origin icon indicating copy to clipboard operation
origin copied to clipboard

OCPBUGS-62875: images: test that creating an imagestream with --dry-run does not actually create the object

Open liouk opened this issue 2 months ago • 37 comments

This test will fail without https://github.com/openshift/oc/pull/2117 and must not be merged before the fix.

/hold

liouk avatar Oct 21 '25 08:10 liouk

@liouk: This pull request references Jira Issue OCPBUGS-62875, which is valid.

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

Requesting review from QA contact: /cc @wangke19

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

In response to this:

This test will fail without https://github.com/openshift/oc/pull/2117.

/hold

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 08:10 openshift-ci-robot

/lgtm

ardaguclu avatar Oct 21 '25 11:10 ardaguclu

/retest

wangke19 avatar Oct 22 '25 02:10 wangke19

The failed jobs I think there is no fixed oc for execution. we can ignore them. /unhold

wangke19 avatar Oct 22 '25 07:10 wangke19

/verified later @wangke19

wangke19 avatar Oct 22 '25 07:10 wangke19

@wangke19: This PR has been marked to be verified later by @wangke19.

In response to this:

/verified later @wangke19

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 22 '25 07:10 openshift-ci-robot

/retest

liouk avatar Oct 23 '25 08:10 liouk

/approve

flavianmissi avatar Oct 23 '25 08:10 flavianmissi

/retest-required

Remaining retests: 0 against base HEAD 618d3bd983f70de7e1c1d077675dcefea20bee6d and 2 for PR HEAD 7872e885127aeb367d214408b167d63cd108b09e in total

openshift-ci-robot avatar Oct 23 '25 11:10 openshift-ci-robot

/retest-required

Remaining retests: 0 against base HEAD 0734d3add1c389e16b228db2f577391b97059c53 and 1 for PR HEAD 7872e885127aeb367d214408b167d63cd108b09e in total

openshift-ci-robot avatar Oct 23 '25 13:10 openshift-ci-robot

/retest-required

Remaining retests: 0 against base HEAD d629cf124084ed97ce122dabbcc9e64be273a414 and 0 for PR HEAD 7872e885127aeb367d214408b167d63cd108b09e in total

openshift-ci-robot avatar Oct 23 '25 19:10 openshift-ci-robot

/hold

Revision 7872e885127aeb367d214408b167d63cd108b09e was retested 3 times: holding

openshift-ci-robot avatar Oct 23 '25 21:10 openshift-ci-robot

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: 7872e885127aeb367d214408b167d63cd108b09e

Job Name New Test Risk
pull-ci-openshift-origin-main-e2e-aws-ovn-fips High - "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" is a new test that failed 5 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-aws-ovn-image-registry High - "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" is a new test that was not present in all runs against the current commit, and also failed 4 time(s).
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift High - "Import the release payload "nightly-arm64" from an external source" is a new test that was not present in all runs against the current commit.
pull-ci-openshift-origin-main-e2e-aws-ovn-microshift-serial High - "Import the release payload "nightly-arm64" from an external source" is a new test that was not present in all runs against the current commit.
pull-ci-openshift-origin-main-e2e-gcp-ovn High - "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" is a new test that failed 5 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-metal-ipi-ovn-ipv6 High - "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" is a new test that failed 5 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-vsphere-ovn High - "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" is a new test that failed 5 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-vsphere-ovn-upi High - "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" is a new test that failed 5 time(s) against the current commit
pull-ci-openshift-origin-main-okd-scos-e2e-aws-ovn High - "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" is a new test that failed 3 time(s) against the current commit

New tests seen in this PR at sha: 7872e885127aeb367d214408b167d63cd108b09e

  • "Import the release payload "nightly-arm64" from an external source" [Total: 6, Pass: 6, Fail: 0, Flake: 0]
  • "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" [Total: 33, Pass: 1, Fail: 32, Flake: 0]

openshift-trt[bot] avatar Oct 23 '25 23:10 openshift-trt[bot]

/retest-required

liouk avatar Oct 24 '25 08:10 liouk

Most recent push just changed a description message, verification stands as before.

/verified later @wangke19

liouk avatar Oct 24 '25 08:10 liouk

@liouk: This PR has been marked to be verified later by @wangke19.

In response to this:

Most recent push just changed a description message, verification stands as before.

/verified later @wangke19

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 24 '25 08:10 openshift-ci-robot

/lgtm

ardaguclu avatar Oct 24 '25 08:10 ardaguclu

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: d1af17977b31c85b591c1990c92a48cccc2dc159

Job Name New Test Risk
pull-ci-openshift-origin-main-e2e-aws-ovn-fips High - "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-aws-ovn-image-registry High - "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-vsphere-ovn-upi High - "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit

New tests seen in this PR at sha: d1af17977b31c85b591c1990c92a48cccc2dc159

  • "[sig-imageregistry] Image --dry-run should not create resources [apigroup:image.openshift.io] [Suite:openshift/conformance/parallel]" [Total: 3, Pass: 0, Fail: 3, Flake: 0]

openshift-trt[bot] avatar Oct 24 '25 12:10 openshift-trt[bot]

/lgtm

ardaguclu avatar Oct 24 '25 13:10 ardaguclu

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ardaguclu, flavianmissi, liouk

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 Oct 24 '25 13:10 openshift-ci[bot]

/verified by e2e tests

wangke19 avatar Oct 27 '25 09:10 wangke19

@wangke19: This PR has been marked as verified by e2e tests.

In response to this:

/verified by e2e tests

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 27 '25 09:10 openshift-ci-robot

/unhold

wangke19 avatar Oct 27 '25 09:10 wangke19

/test e2e-aws-ovn-fips

wangke19 avatar Oct 27 '25 09:10 wangke19

/retest

wangke19 avatar Oct 27 '25 09:10 wangke19

/retest-required

Remaining retests: 0 against base HEAD 32a91eb0ee3b8f2f5be28305f1b725750d7208d1 and 2 for PR HEAD 15f0885328d43f8dfe0127dc5548e4b883f0ceea in total

openshift-ci-robot avatar Oct 27 '25 11:10 openshift-ci-robot

/test e2e-aws-ovn-fips

wangke19 avatar Oct 27 '25 13:10 wangke19

Backport to 4.20 PR https://github.com/openshift/origin/pull/30389

wangke19 avatar Oct 27 '25 13:10 wangke19

/hold until we confirm that changes are ok.

ardaguclu avatar Oct 27 '25 13:10 ardaguclu

/retest

liouk avatar Nov 12 '25 14:11 liouk