compliance-operator icon indicating copy to clipboard operation
compliance-operator copied to clipboard

OCPBUGS-42396: Fix variables not listed in the annotations of rule issue

Open xiaojiey opened this issue 9 months ago • 15 comments

xiaojiey avatar Mar 18 '25 07:03 xiaojiey

@xiaojiey: This pull request references Jira Issue OCPBUGS-42396, which is invalid:

  • expected the bug to target the "4.19.0" version, but no target version was set

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

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 Mar 19 '25 01:03 openshift-ci-robot

:robot: To deploy this PR, run the following command:

make catalog-deploy CATALOG_IMG=ghcr.io/complianceascode/compliance-operator-catalog:706-0bf00f44288d480ffe218711099532e47190a1ef

github-actions[bot] avatar Mar 20 '25 02:03 github-actions[bot]

:robot: To deploy this PR, run the following command:

make catalog-deploy CATALOG_IMG=ghcr.io/complianceascode/compliance-operator-catalog:706-cf6711c5712956fb67e6eb491237606a145a15d9

github-actions[bot] avatar Mar 21 '25 04:03 github-actions[bot]

:robot: To deploy this PR, run the following command:

make catalog-deploy CATALOG_IMG=ghcr.io/complianceascode/compliance-operator-catalog:706-6081872ebc2b6dd6ef4da0c2ab6ad73c84568547

github-actions[bot] avatar Mar 21 '25 16:03 github-actions[bot]

:robot: To deploy this PR, run the following command:

make catalog-deploy CATALOG_IMG=ghcr.io/complianceascode/compliance-operator-catalog:706-a6f27023a95b24be015a9a79d8c552cd34c6cc4f

github-actions[bot] avatar Mar 21 '25 17:03 github-actions[bot]

I can see the variable is listed correctly with catalogsource image ghcr.io/complianceascode/compliance-operator-catalog:706-a6f27023a95b24be015a9a79d8c552cd34c6cc4f

% oc get rule ocp4-resource-requests-limits-in-daemonset   -o=jsonpath='{.metadata.annotations.compliance\.openshift\.io/rule-variable}'
var-daemonset-limit-namespaces-exempt-rege

xiaojiey avatar Mar 24 '25 07:03 xiaojiey

/hold for test

BhargaviGudi avatar Mar 24 '25 08:03 BhargaviGudi

/jira-refresh

xiaojiey avatar Mar 24 '25 09:03 xiaojiey

Verification passed with 4.19.0-0.nightly-2025-03-21-030708 + PR #706 code This PR fixes both OCPBUGS-42396 and OCPBUGS-35260

$ oc get rule -o custom-columns=NAME:metadata.name,VARIABLE:metadata.annotations.compliance\\.openshift\\.io/rule-variable --no-headers | grep ocp4-configure-network-policies-namespaces
ocp4-configure-network-policies-namespaces                                          var-network-policies-namespaces-exempt-regex
$ oc get rule -o custom-columns=NAME:metadata.name,VARIABLE:metadata.annotations.compliance\\.openshift\\.io/rule-variable --no-headers | grep resource-requests-limits
ocp4-resource-requests-limits-in-daemonset                                          var-daemonset-limit-namespaces-exempt-regex
ocp4-resource-requests-limits-in-deployment                                         var-deployment-limit-namespaces-exempt-regex
ocp4-resource-requests-limits-in-statefulset                                        var-statefulset-limit-namespaces-exempt-regex

BhargaviGudi avatar Mar 24 '25 12:03 BhargaviGudi

/label qe-approved

BhargaviGudi avatar Mar 24 '25 12:03 BhargaviGudi

@xiaojiey: This pull request references Jira Issue OCPBUGS-42396, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.19.0) matches configured target version for branch (4.19.0)
  • bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact: /cc @xiaojiey

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

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 Mar 24 '25 12:03 openshift-ci-robot

/unhold

BhargaviGudi avatar Mar 24 '25 12:03 BhargaviGudi

@openshift-ci-robot: GitHub didn't allow me to request PR reviews from the following users: xiaojiey.

Note that only ComplianceAsCode members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

@xiaojiey: This pull request references Jira Issue OCPBUGS-42396, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.19.0) matches configured target version for branch (4.19.0)
  • bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact: /cc @xiaojiey

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

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.

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-sigs/prow repository.

openshift-ci[bot] avatar Mar 24 '25 12:03 openshift-ci[bot]

:robot: To deploy this PR, run the following command:

make catalog-deploy CATALOG_IMG=ghcr.io/complianceascode/compliance-operator-catalog:706-a79aff67ca5df2c8e01467391fb5d3098a78b5a9

github-actions[bot] avatar Oct 17 '25 14:10 github-actions[bot]

:robot: To deploy this PR, run the following command:

make catalog-deploy CATALOG_IMG=ghcr.io/complianceascode/compliance-operator-catalog:706-99073bbcc32443c377a9aa9ffd6b1a6a07cac92e

github-actions[bot] avatar Oct 17 '25 14:10 github-actions[bot]

:robot: To deploy this PR, run the following command:

make catalog-deploy CATALOG_IMG=ghcr.io/complianceascode/compliance-operator-catalog:706-bb68efb9841a651f9aa6cc5c770eb1dac051a1a7

github-actions[bot] avatar Oct 17 '25 15:10 github-actions[bot]

:robot: To deploy this PR, run the following command:

make catalog-deploy CATALOG_IMG=ghcr.io/complianceascode/compliance-operator-catalog:706-4c4363975510de605a688075e28ce6e10a03d31c

github-actions[bot] avatar Oct 20 '25 00:10 github-actions[bot]

@xiaojiey: This pull request references Jira Issue OCPBUGS-42396, which is invalid:

  • expected the bug to target the "4.21.0" version, but no target version was set
  • expected the bug to be in one of the following states: NEW, ASSIGNED, POST, but it is In Progress instead

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

The main changes are:

  1. Extract Variables from Rule Warnings
  2. Improve Variable Extraction Efficiency. Enhanced trimToValue() function to deduplicate variables at the source.
  3. Add a e2e test for the variables check in the annotation.

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 Oct 20 '25 01:10 openshift-ci-robot

/retest-required

xiaojiey avatar Oct 20 '25 09:10 xiaojiey

/retest-required

xiaojiey avatar Oct 20 '25 12:10 xiaojiey

/retest-required

xiaojiey avatar Oct 21 '25 00:10 xiaojiey

/retest-required

xiaojiey avatar Oct 21 '25 07:10 xiaojiey

/retest-required as cluster failed to create for arm test

xiaojiey avatar Oct 21 '25 13:10 xiaojiey

@xiaojiey: This pull request references CMP-3557 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.21.0" version, but no target version was set.

In response to this:

The main changes are:

  1. Extract Variables from Rule Warnings
  2. Improve Variable Extraction Efficiency. Enhanced trimToValue() function to deduplicate variables at the source.
  3. Add a e2e test for the variables check in the annotation.

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 Oct 21 '25 21:10 openshift-ci-robot

@xiaojiey: This pull request references CMP-3582 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.21.0" version, but no target version was set.

In response to this:

The main changes are:

  1. Extract Variables from Rule Warnings
  2. Improve Variable Extraction Efficiency. Enhanced trimToValue() function to deduplicate variables at the source.
  3. Add a e2e test for the variables check in the annotation.

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 Oct 21 '25 21:10 openshift-ci-robot

@xiaojiey: This pull request references CMP-3582 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.21.0" version, but no target version was set.

This pull request references CMP-3557 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.21.0" version, but no target version was set.

In response to this:

The main changes are:

  1. Extract Variables from Rule Warnings
  2. Improve Variable Extraction Efficiency. Enhanced trimToValue() function to deduplicate variables at the source.
  3. Add a e2e test for the variables check in the annotation.

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 Oct 21 '25 21:10 openshift-ci-robot

/test e2e-aws-parallel-arm

xiaojiey avatar Oct 22 '25 01:10 xiaojiey

Verification passed with CO installed from this PR.

$ oc get rule -o custom-columns=NAME:metadata.name,VARIABLE:metadata.annotations.compliance\\.openshift\\.io/rule-variable --no-headers | grep ocp4-configure-network-policies-namespaces
ocp4-configure-network-policies-namespaces                                          var-network-policies-namespaces-exempt-regex
$ oc get rule -o custom-columns=NAME:metadata.name,VARIABLE:metadata.annotations.compliance\\.openshift\\.io/rule-variable --no-headers | grep resource-requests-limits
ocp4-resource-requests-limits-in-daemonset                                          var-daemonset-limit-namespaces-exempt-regex
ocp4-resource-requests-limits-in-deployment                                         var-deployment-limit-namespaces-exempt-regex
ocp4-resource-requests-limits-in-statefulset                                        var-statefulset-limit-namespaces-exempt-regex

Also I have run test cases OCP-33578, 33859, 47044, 42720 on my cluster to look for regressions and it has all passed

Anna-Koudelkova avatar Oct 22 '25 19:10 Anna-Koudelkova

/lgtm

Anna-Koudelkova avatar Oct 22 '25 19:10 Anna-Koudelkova

@Anna-Koudelkova: 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-sigs/prow repository.

openshift-ci[bot] avatar Oct 22 '25 19:10 openshift-ci[bot]