Skip to content

Conversation

benjaminapetersen
Copy link
Contributor

Part of the work to tolerate v2beta1 autoscalers.

  • if the user sets the cpu input to 0, the input would disappear
  • this change makes the attribute bound one-way, which ensures we can hide it on load (if the appropriate conditions are met) but that the user input doesn't flip the ng-show to a falsy value while editing. It should be set to false by the controller only if it is already falsy && metrics are in use.

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 14, 2018
@jwforres
Copy link
Member

lgtm but you are missing a dist file it looks like

@benjaminapetersen benjaminapetersen force-pushed the bug/1545011/autoscaler-cpu-input-disappears branch from b11397b to 249c5fd Compare February 14, 2018 20:17
@benjaminapetersen
Copy link
Contributor Author

great, thx. yup, there is my dist/template change.

@jwforres
Copy link
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 14, 2018
@openshift-merge-robot
Copy link
Contributor

/test all [submit-queue is verifying that this PR is safe to merge]

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit 39db87b into openshift:master Feb 14, 2018
@benjaminapetersen benjaminapetersen deleted the bug/1545011/autoscaler-cpu-input-disappears branch February 15, 2018 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants