Skip to content

Conversation

@bburt-rh bburt-rh requested a review from simonpasquier October 7, 2022 19:37
@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 7, 2022
@bburt-rh bburt-rh force-pushed the RHDEVDOCS-4310-clarify-contradictory-guidance-in-autoscaling-topic branch from cd75218 to 82e1331 Compare October 7, 2022 20:03
@bburt-rh bburt-rh force-pushed the RHDEVDOCS-4310-clarify-contradictory-guidance-in-autoscaling-topic branch from 82e1331 to ea08902 Compare October 11, 2022 13:57
@bburt-rh bburt-rh requested a review from juzhao October 11, 2022 13:57
@bburt-rh
Copy link
Contributor Author

@juzhao PTAL this PR and provide feedback and suggestions. Thank you!

Copy link
Contributor

@gabriel-rh gabriel-rh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - some very minor suggestions

@bburt-rh bburt-rh force-pushed the RHDEVDOCS-4310-clarify-contradictory-guidance-in-autoscaling-topic branch from ea08902 to e915a91 Compare October 12, 2022 17:58
@bburt-rh bburt-rh requested a review from gabriel-rh October 12, 2022 17:58
@bburt-rh bburt-rh force-pushed the RHDEVDOCS-4310-clarify-contradictory-guidance-in-autoscaling-topic branch from e915a91 to 52f928a Compare October 12, 2022 18:08
@bburt-rh bburt-rh requested a review from juzhao October 13, 2022 12:38
Copy link
Contributor

@gabriel-rh gabriel-rh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 13, 2022
@juzhao
Copy link

juzhao commented Oct 14, 2022

/lgtm

@bburt-rh bburt-rh added peer-review-done Signifies that the peer review team has reviewed this PR branch/enterprise-4.6 dev-tools Label for all Odo/Pipelines/Helm/Developer Console/Perspective PRs branch/enterprise-4.7 qe-approved Signifies that QE has signed off on this PR labels Oct 14, 2022
@bburt-rh bburt-rh added this to the Continuous Release milestone Oct 14, 2022
@bburt-rh bburt-rh merged commit bf9f313 into openshift:enterprise-4.7 Oct 14, 2022
@bburt-rh
Copy link
Contributor Author

/cherry-pick enterprise-4.6

@openshift-cherrypick-robot

@bburt-rh: new pull request created: #51696

In response to this:

/cherry-pick enterprise-4.6

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@bburt-rh bburt-rh deleted the RHDEVDOCS-4310-clarify-contradictory-guidance-in-autoscaling-topic branch June 6, 2023 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.6 branch/enterprise-4.7 dev-tools Label for all Odo/Pipelines/Helm/Developer Console/Perspective PRs lgtm Indicates that a PR is ready to be merged. peer-review-done Signifies that the peer review team has reviewed this PR qe-approved Signifies that QE has signed off on this PR 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.

[enterprise-4.7] Issue in file monitoring/exposing-custom-application-metrics-for-autoscaling.adoc

5 participants