Skip to content

OCPBUGS-77878:auditloganalyzer: update test name, remove duplicated test#29930

Merged
openshift-merge-bot[bot] merged 2 commits intoopenshift:mainfrom
vrutkovs:readyz-test-name
Jun 20, 2025
Merged

OCPBUGS-77878:auditloganalyzer: update test name, remove duplicated test#29930
openshift-merge-bot[bot] merged 2 commits intoopenshift:mainfrom
vrutkovs:readyz-test-name

Conversation

@vrutkovs
Copy link
Copy Markdown
Contributor

Ensure that we create a JUnit for an existing test - so it should have expected prefix and suffix. Currently these tests are incorrectly duplicated

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jun 20, 2025
@openshift-ci-robot
Copy link
Copy Markdown

@vrutkovs: This pull request explicitly references no jira issue.

Details

In response to this:

Ensure that we create a JUnit for an existing test - so it should have expected prefix and suffix. Currently these tests are incorrectly duplicated

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 openshift-ci Bot requested review from p0lyn0mial and sjenning June 20, 2025 12:04
@vrutkovs vrutkovs changed the title NO-JIRA: auditloganalyzer: update test name to fail NO-JIRA: auditloganalyzer: update test name, remove duplicated test Jun 20, 2025
}

testName = "API LBs follow /readyz of kube-apiserver and stop sending requests before server shutdowns for external clients"
testName = "[sig-api-machinery][Feature:APIServer][Late] API LBs follow /readyz of kube-apiserver and stop sending requests before server shutdowns for external clients [Suite:openshift/conformance/parallel]"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for deduping. Small problems, I believe the late and suite tags in the test name are now quite misleading and inaccurate. And yet if a rename is done it needs to be reflected in the mapping repo.

Lets make it [sig-api-machinery][Feature:APIServer] API LBs follow /readyz of kube-apiserver and stop sending requests before server shutdowns for external clients

I will handle the test rename mapping in this instance over in https://github.com/openshift-eng/ci-test-mapping

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated

Vadim Rutkovsky added 2 commits June 20, 2025 14:32
Ensure that we create a JUnit for an existing test - so it should have expected prefix and suffix
This test is already performed by auditloganalyzer
@dgoodwin
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jun 20, 2025
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 20, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dgoodwin, vrutkovs

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

The pull request process is described here

Details 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 openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 20, 2025
@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD bea636f and 2 for PR HEAD 544ac99 in total

1 similar comment
@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD bea636f and 2 for PR HEAD 544ac99 in total

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 20, 2025

@vrutkovs: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-aws-ovn-kube-apiserver-rollout 544ac99 link false /test e2e-aws-ovn-kube-apiserver-rollout
ci/prow/e2e-openstack-serial 544ac99 link false /test e2e-openstack-serial
ci/prow/e2e-gcp-ovn-rt-upgrade 544ac99 link false /test e2e-gcp-ovn-rt-upgrade
ci/prow/e2e-vsphere-ovn-dualstack-primaryv6 544ac99 link false /test e2e-vsphere-ovn-dualstack-primaryv6
ci/prow/e2e-aws-ovn-serial-publicnet-1of2 544ac99 link false /test e2e-aws-ovn-serial-publicnet-1of2
ci/prow/e2e-aws 544ac99 link false /test e2e-aws
ci/prow/e2e-aws-ovn-etcd-scaling 544ac99 link false /test e2e-aws-ovn-etcd-scaling
ci/prow/e2e-gcp-ovn-etcd-scaling 544ac99 link false /test e2e-gcp-ovn-etcd-scaling
ci/prow/e2e-gcp-csi 544ac99 link false /test e2e-gcp-csi
ci/prow/e2e-aws-ovn-single-node-upgrade 544ac99 link false /test e2e-aws-ovn-single-node-upgrade
ci/prow/e2e-gcp-fips-serial-2of2 544ac99 link false /test e2e-gcp-fips-serial-2of2
ci/prow/e2e-azure-ovn-etcd-scaling 544ac99 link false /test e2e-azure-ovn-etcd-scaling
ci/prow/e2e-gcp-fips-serial-1of2 544ac99 link false /test e2e-gcp-fips-serial-1of2
ci/prow/e2e-azure-ovn-upgrade 544ac99 link false /test e2e-azure-ovn-upgrade
ci/prow/okd-scos-e2e-aws-ovn 544ac99 link false /test okd-scos-e2e-aws-ovn
ci/prow/e2e-aws-disruptive 544ac99 link false /test e2e-aws-disruptive
ci/prow/e2e-gcp-disruptive 544ac99 link false /test e2e-gcp-disruptive
ci/prow/e2e-vsphere-ovn-etcd-scaling 544ac99 link false /test e2e-vsphere-ovn-etcd-scaling
ci/prow/okd-e2e-gcp 544ac99 link false /test okd-e2e-gcp

Full PR test history. Your PR dashboard.

Details

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. I understand the commands that are listed here.

@openshift-trt
Copy link
Copy Markdown

openshift-trt Bot commented Jun 20, 2025

Risk analysis has seen new tests most likely introduced by this PR.
Please ensure that new tests meet guidelines for naming and stability.

New Test Risks for sha: 544ac99

Job Name New Test Risk
pull-ci-openshift-origin-main-e2e-gcp-ovn High - "[sig-api-machinery][Feature:APIServer] API LBs follow /readyz of kube-apiserver and stop sending requests before server shutdowns for external clients" is a new test that was not present in all runs against the current commit.
pull-ci-openshift-origin-main-e2e-gcp-ovn-upgrade High - "[sig-api-machinery][Feature:APIServer] API LBs follow /readyz of kube-apiserver and stop sending requests before server shutdowns for external clients" is a new test that was not present in all runs against the current commit.

New tests seen in this PR at sha: 544ac99

  • "[sig-api-machinery][Feature:APIServer] API LBs follow /readyz of kube-apiserver and stop sending requests before server shutdowns for external clients" [Total: 48, Pass: 48, Fail: 0, Flake: 0]

@openshift-merge-bot openshift-merge-bot Bot merged commit 14f4646 into openshift:main Jun 20, 2025
39 of 58 checks passed
@openshift-bot
Copy link
Copy Markdown
Contributor

[ART PR BUILD NOTIFIER]

Distgit: openshift-enterprise-tests
This PR has been included in build openshift-enterprise-tests-container-v4.20.0-202506210042.p0.g14f4646.assembly.stream.el9.
All builds following this will include this PR.

@vrutkovs
Copy link
Copy Markdown
Contributor Author

/cherry-pick release-4.19

@openshift-cherrypick-robot
Copy link
Copy Markdown

@vrutkovs: new pull request created: #29992

Details

In response to this:

/cherry-pick release-4.19

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.

@wangke19
Copy link
Copy Markdown
Contributor

wangke19 commented Mar 5, 2026

/retitle OCPBUGS-77878:auditloganalyzer: update test name, remove duplicated test

@openshift-ci openshift-ci Bot changed the title NO-JIRA: auditloganalyzer: update test name, remove duplicated test OCPBUGS-77878:auditloganalyzer: update test name, remove duplicated test Mar 5, 2026
@openshift-ci-robot
Copy link
Copy Markdown

@vrutkovs: Jira Issue OCPBUGS-77878: All pull requests linked via external trackers have merged:

Jira Issue OCPBUGS-77878 has been moved to the MODIFIED state.

Details

In response to this:

Ensure that we create a JUnit for an existing test - so it should have expected prefix and suffix. Currently these tests are incorrectly duplicated

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.

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

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants