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.

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

MON-1841 4.9 Monitoring Release Notes #37264

Merged

Conversation

bburt-rh
Copy link
Contributor

@bburt-rh bburt-rh commented Oct 8, 2021

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 8, 2021
@netlify
Copy link

netlify bot commented Oct 8, 2021

✔️ Deploy Preview for osdocs ready!

🔨 Explore the source changes: e029070

🔍 Inspect the deploy log: https://app.netlify.com/sites/osdocs/deploys/6165aa197a3682000710b5b8

😎 Browse the preview: https://deploy-preview-37264--osdocs.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-9-release-notes

@bburt-rh bburt-rh force-pushed the monitoring-4-9-release-notes branch from f625fee to bce5d28 Compare October 8, 2021 20:28
*** `ThanosQueryHttpRequestQueryRangeErrorRateHigh`
*** `ThanosSidecarBucketOperationsFailed`
*** `ThanosSidecarPrometheusDown`
*** `ThanosSidecarUnhealthy alerts`
Copy link

Choose a reason for hiding this comment

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

these alerts actually added namespace label since 4.9 compared with 4.8

AlertmanagerReceiversNotConfigured
Watchdog
KubeCPUOvercommit
KubeMemoryOvercommit
KubeletDown
MultipleContainersOOMKilled

if we just need to list the changes by cluster monitoring, just add

AlertmanagerReceiversNotConfigured
Watchdog

is fine

@bburt-rh bburt-rh force-pushed the monitoring-4-9-release-notes branch 3 times, most recently from 0090d89 to 7957436 Compare October 11, 2021 20:18
Copy link
Contributor

@jc-berger jc-berger left a comment

Choose a reason for hiding this comment

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

Approving all the changes I've seen in this commit.

Please see further suggestions I have!

Copy link
Contributor

@jc-berger jc-berger 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
Copy link

openshift-ci bot commented Oct 11, 2021

@jc-berger: changing LGTM is restricted to collaborators

In response to this:

/lgtm

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.

@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 12, 2021
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Oct 12, 2021
Copy link
Contributor

@jc-berger jc-berger 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
Copy link

openshift-ci bot commented Oct 12, 2021

@jc-berger: changing LGTM is restricted to collaborators

In response to this:

/lgtm

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.

Copy link

@simonpasquier simonpasquier 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 12, 2021
@abrennan89 abrennan89 merged commit b64bdc3 into openshift:enterprise-4.9 Oct 12, 2021
@abrennan89 abrennan89 added this to the Future Release milestone Oct 12, 2021
@bburt-rh bburt-rh deleted the monitoring-4-9-release-notes branch June 6, 2023 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.9 lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants