cluster-logging-operator icon indicating copy to clipboard operation
cluster-logging-operator copied to clipboard

LOG-5308: add watcher for grafana-dashboard-cluster-logging configMap to support it reconciliation

Open vparfonov opened this issue 1 year ago • 12 comments

Description

This PR introduces a new watcher for grafana-dashboard-cluster-logging ConfigMap resources to enable their reconciliation within our operator. With this enhancement, our operator can now react to changes/removing in ConfigMap objects, ensuring that the system remains synchronized and operates effectively in response to ConfigMap modifications.

/cc @Clee2691 @cahartma /assign @jcantrill

/cherry-pick

Links

  • Depending on PR(s):
  • Bugzilla:
  • Github issue:
  • JIRA: https://issues.redhat.com/browse/LOG-5308
  • Enhancement proposal:

vparfonov avatar Apr 08 '24 14:04 vparfonov

@vparfonov: This pull request references LOG-5308 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.8.0" version, but no target version was set.

In response to this:

Description

This PR introduces a new watcher for grafana-dashboard-cluster-logging ConfigMap resources to enable their reconciliation within our operator. With this enhancement, our operator can now react to changes/removing in ConfigMap objects, ensuring that the system remains synchronized and operates effectively in response to ConfigMap modifications.

/cc @Clee2691 @cahartma /assign @jcantrill

/cherry-pick

Links

  • Depending on PR(s):
  • Bugzilla:
  • Github issue:
  • JIRA: https://issues.redhat.com/browse/LOG-5308
  • Enhancement proposal:

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 openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot avatar Apr 08 '24 14:04 openshift-ci-robot

/hold

vparfonov avatar Apr 08 '24 14:04 vparfonov

/retest

vparfonov avatar Apr 08 '24 18:04 vparfonov

/retest

vparfonov avatar Apr 08 '24 21:04 vparfonov

/test e2e-target

jcantrill avatar Apr 09 '24 19:04 jcantrill

/approve /lgtm /hold cancel

jcantrill avatar Apr 09 '24 19:04 jcantrill

/cherrypick release-5.9

jcantrill avatar Apr 09 '24 19:04 jcantrill

@jcantrill: once the present PR merges, I will cherry-pick it on top of release-5.9 in a new PR and assign it to you.

In response to this:

/cherrypick release-5.9

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.

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jcantrill, vparfonov

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

openshift-ci[bot] avatar Apr 09 '24 19:04 openshift-ci[bot]

/test e2e-target

vparfonov avatar Apr 09 '24 20:04 vparfonov

/retest-required

Remaining retests: 0 against base HEAD 1b05746042c5c6ea06e77280a0c434e199a45057 and 2 for PR HEAD a8260da0021a7ec63a929d16abb0c62fc0611e07 in total

openshift-ci-robot avatar Apr 09 '24 21:04 openshift-ci-robot

/test e2e-target

vparfonov avatar Apr 10 '24 09:04 vparfonov

/test e2e-target

vparfonov avatar Apr 10 '24 11:04 vparfonov

/retest

vparfonov avatar Apr 10 '24 12:04 vparfonov

/retest-required

Remaining retests: 0 against base HEAD 5b4f827bc1ee4aa692f5d4ea35e23eb6d10e159f and 1 for PR HEAD a8260da0021a7ec63a929d16abb0c62fc0611e07 in total

openshift-ci-robot avatar Apr 12 '24 14:04 openshift-ci-robot

/retest

jcantrill avatar Apr 12 '24 18:04 jcantrill

/cherrypick release-5.9

jcantrill avatar Apr 12 '24 18:04 jcantrill

@jcantrill: once the present PR merges, I will cherry-pick it on top of release-5.9 in a new PR and assign it to you.

In response to this:

/cherrypick release-5.9

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.

/test e2e-target

vparfonov avatar Apr 14 '24 15:04 vparfonov

/retest

jcantrill avatar Apr 15 '24 20:04 jcantrill

/test e2e-target

jcantrill avatar Apr 16 '24 12:04 jcantrill

/test e2e-target

vparfonov avatar Apr 17 '24 14:04 vparfonov

/retest-required

Remaining retests: 0 against base HEAD dc121294fdb0e142b4e118094310918967de8d05 and 0 for PR HEAD a8260da0021a7ec63a929d16abb0c62fc0611e07 in total

openshift-ci-robot avatar Apr 17 '24 22:04 openshift-ci-robot

/hold

Revision a8260da0021a7ec63a929d16abb0c62fc0611e07 was retested 3 times: holding

openshift-ci-robot avatar Apr 18 '24 01:04 openshift-ci-robot

/test e2e-target

vparfonov avatar Apr 19 '24 10:04 vparfonov

/hold cancel

vparfonov avatar Apr 19 '24 13:04 vparfonov

/refresh

vparfonov avatar Apr 19 '24 13:04 vparfonov

/retest-required

Remaining retests: 0 against base HEAD 0e262a48494d46b6a69865d799e3c1fe1f805665 and 2 for PR HEAD a8260da0021a7ec63a929d16abb0c62fc0611e07 in total

openshift-ci-robot avatar Apr 19 '24 19:04 openshift-ci-robot

/retest-required

Remaining retests: 0 against base HEAD 085bdc1fd735d430a59a1ae92e24f18284c6e956 and 1 for PR HEAD a8260da0021a7ec63a929d16abb0c62fc0611e07 in total

openshift-ci-robot avatar Apr 20 '24 00:04 openshift-ci-robot

/refresh

vparfonov avatar Apr 23 '24 07:04 vparfonov