Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve resource usage in nginx controller #1387

Merged
merged 1 commit into from
Sep 19, 2017
Merged

Improve resource usage in nginx controller #1387

merged 1 commit into from
Sep 19, 2017

Conversation

aledbf
Copy link
Member

@aledbf aledbf commented Sep 19, 2017

No description provided.

@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 19, 2017
@k8s-reviewable
Copy link

This change is Reviewable

@k8s-ci-robot k8s-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Sep 19, 2017
@aledbf aledbf mentioned this pull request Sep 19, 2017
Closed
@aledbf aledbf force-pushed the profile branch 3 times, most recently from 0380ff7 to 7419cab Compare September 19, 2017 01:49
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 44.038% when pulling 7419cabd0fe7541321a867ddb618039b20e9c9f5 on aledbf:profile into 1a68536 on kubernetes:master.

@k8s-ci-robot k8s-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Sep 19, 2017
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 43.809% when pulling 1f09e1a1f1cd1fd627513a0fe16ca08be7dda611 on aledbf:profile into 1a68536 on kubernetes:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 43.727% when pulling faedd71524147dab6bce316e606858e722724b19 on aledbf:profile into 1a68536 on kubernetes:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 43.727% when pulling 4830ad82cc2122ffdcde98c4ed6a85591122f5d4 on aledbf:profile into 1a68536 on kubernetes:master.

@aledbf aledbf force-pushed the profile branch 2 times, most recently from c2dbd1c to 4fb5b93 Compare September 19, 2017 16:00
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 43.752% when pulling cd288b9 on aledbf:profile into 1a68536 on kubernetes:master.

@aledbf aledbf changed the title WIP: Improve resource usage in nginx controller Improve resource usage in nginx controller Sep 19, 2017
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 19, 2017
@aledbf aledbf merged commit 8584b25 into kubernetes:master Sep 19, 2017
@aledbf aledbf deleted the profile branch September 19, 2017 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aledbf @k8s-reviewable @coveralls @k8s-ci-robot