Skip to content

Conversation

@rh-max
Copy link
Contributor

@rh-max rh-max commented Sep 3, 2019

No description provided.

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 3, 2019
. Examine and edit the current Alertmanager configuration.
. Save the updated Alertmanager configuration.

Also, next to the title of each of these pages is a link to the old Alertmanager interface.

Choose a reason for hiding this comment

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

..is a link to the old Alertmanager interface Not sure if 'old' is the best term here. Maybe 'Promethus interface'?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's the term we agreed to use to imply that people should switch to the new OCP interface. And it's not the Prometheus interface anyway, the Prometheus one is linked from the Metrics page. This is the AM one.

@rh-max
Copy link
Contributor Author

rh-max commented Sep 18, 2019

@sheriff-rh Could you please take a look at this? If it's good, please merge & cherry-pick to 4.2 too. Thanks.

@sheriff-rh
Copy link
Contributor

sheriff-rh commented Sep 18, 2019

Sure thing @rh-max !

Edit: This is against 4.2 already, can you make one against master and tag me? I'll merge that too. Thanks!

@sheriff-rh sheriff-rh merged commit 8702888 into openshift:enterprise-4.2 Sep 18, 2019
@sheriff-rh
Copy link
Contributor

/cherrypick enterprise-4.2

@openshift-cherrypick-robot

@sheriff-rh: base branch (enterprise-4.2) needs to differ from target branch (enterprise-4.2)

Details

In response to this:

/cherrypick enterprise-4.2

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.

@rh-max
Copy link
Contributor Author

rh-max commented Sep 18, 2019

@sheriff-rh Thanks! Yeah, I should have said cherry-pick into master. How about just running /cherrypick master? The effect will be the same as opening a new PR against master & merging that.

@sheriff-rh
Copy link
Contributor

/cherrypick master

@openshift-cherrypick-robot

@sheriff-rh: new pull request created: #16741

Details

In response to this:

/cherrypick master

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.

@sheriff-rh
Copy link
Contributor

@sheriff-rh Thanks! Yeah, I should have said cherry-pick into master. How about just running /cherrypick master? The effect will be the same as opening a new PR against master & merging that.

Yep that got it, good call @rh-max !

@rh-max
Copy link
Contributor Author

rh-max commented Sep 18, 2019

Thanks @sheriff-rh !

The main two pages of the Alerting UI are the *Alerts* and the *Silences* pages.
The main three pages of the Alerting UI are the *Alerts*, the *Silences*, and the *YAML* pages.

The *Alerts* page is located in *Monitoring* -> *Alerts* of the {product-title} web console.
Copy link

Choose a reason for hiding this comment

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

4.2
The Alerts page is located in Monitoring -> Alerting -> Alerts tab
The Silences page is located in Monitoring -> Alerting -> Silences tab

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Implemented in #16978.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

6 participants