Skip to content

chore: propagate filters to the images stanza#76548

Open
Prucek wants to merge 2 commits intoopenshift:mainfrom
Prucek:prowgen-images-change
Open

chore: propagate filters to the images stanza#76548
Prucek wants to merge 2 commits intoopenshift:mainfrom
Prucek:prowgen-images-change

Conversation

@Prucek
Copy link
Copy Markdown
Member

@Prucek Prucek commented Mar 19, 2026

Summary

  • Propagate run_if_changed/skip_if_only_changed from manually-configured -images prow jobs into the corresponding ci-operator config images: stanza
  • Move build_images_if_affected to build_if_affected inside images: config for openshift/ci-tools

Companion PR: openshift/ci-tools#5009 & openshift/ci-tools#5078

🤖 Generated with Claude Code

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek, pj-rehearse: unable to determine affected jobs. This could be due to a branch that needs to be rebased. ERROR:

could not load configuration from candidate revision of release repo: failed to load ci-operator configuration from release repo: failed to load ci-operator config (error unmarshaling JSON: json: cannot unmarshal object into Go struct field ReleaseBuildConfiguration.images of type []api.ProjectDirectoryImageBuildStepConfiguration)
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@openshift-ci openshift-ci bot requested review from 0xFelix and 86254860 March 19, 2026 14:20
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 19, 2026
@Prucek
Copy link
Copy Markdown
Member Author

Prucek commented Mar 19, 2026

/hold

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 19, 2026
@Prucek Prucek force-pushed the prowgen-images-change branch from ee80fb2 to 34e7907 Compare April 1, 2026 07:53
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek, pj-rehearse: unable to determine affected jobs. This could be due to a branch that needs to be rebased. ERROR:

could not load configuration from candidate revision of release repo: failed to load ci-operator configuration from release repo: [failed to load ci-operator config (error unmarshaling JSON: json: cannot unmarshal object into Go struct field ReleaseBuildConfiguration.images of type []api.ProjectDirectoryImageBuildStepConfiguration), failed to load ci-operator config (error converting YAML to JSON: yaml: line 35: could not find expected ':'), failed to load ci-operator config (error converting YAML to JSON: yaml: line 28: could not find expected ':'), failed to load ci-operator config (error converting YAML to JSON: yaml: line 36: could not find expected ':')]
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@Prucek Prucek force-pushed the prowgen-images-change branch from 34e7907 to 75638e0 Compare April 1, 2026 07:59
@Prucek
Copy link
Copy Markdown
Member Author

Prucek commented Apr 1, 2026

/label tide/merge-method-squash

@openshift-ci openshift-ci bot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Apr 1, 2026
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek, pj-rehearse: unable to determine affected jobs. This could be due to a branch that needs to be rebased. ERROR:

could not load configuration from candidate revision of release repo: failed to load ci-operator configuration from release repo: [failed to load ci-operator config (error unmarshaling JSON: json: cannot unmarshal object into Go struct field ReleaseBuildConfiguration.images of type []api.ProjectDirectoryImageBuildStepConfiguration), failed to load ci-operator config (error converting YAML to JSON: yaml: line 35: could not find expected ':'), failed to load ci-operator config (error converting YAML to JSON: yaml: line 28: could not find expected ':'), failed to load ci-operator config (error converting YAML to JSON: yaml: line 36: could not find expected ':')]
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@Prucek
Copy link
Copy Markdown
Member Author

Prucek commented Apr 1, 2026

/restest
/pj-rehearse

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek, pj-rehearse: unable to determine affected jobs ERROR:

could not load configuration from candidate revision of release repo: failed to load ci-operator configuration from release repo: [failed to load ci-operator config (error converting YAML to JSON: yaml: line 35: could not find expected ':'), failed to load ci-operator config (error converting YAML to JSON: yaml: line 28: could not find expected ':'), failed to load ci-operator config (error converting YAML to JSON: yaml: line 36: could not find expected ':')]

If the problem persists, please contact Test Platform.

@Prucek
Copy link
Copy Markdown
Member Author

Prucek commented Apr 1, 2026

/retest-required

@Prucek Prucek force-pushed the prowgen-images-change branch from 75638e0 to 904a80d Compare April 1, 2026 14:48
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek, pj-rehearse: unable to determine affected jobs. This could be due to a branch that needs to be rebased. ERROR:

could not load configuration from candidate revision of release repo: failed to load ci-operator configuration from release repo: [failed to load ci-operator config (error unmarshaling JSON: json: cannot unmarshal object into Go struct field ReleaseBuildConfiguration.images of type []api.ProjectDirectoryImageBuildStepConfiguration), failed to load ci-operator config (error converting YAML to JSON: yaml: line 28: could not find expected ':')]
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@Prucek Prucek force-pushed the prowgen-images-change branch from 904a80d to 089c020 Compare April 1, 2026 15:03
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek, pj-rehearse: unable to determine affected jobs. This could be due to a branch that needs to be rebased. ERROR:

could not load configuration from candidate revision of release repo: failed to load ci-operator configuration from release repo: [failed to load ci-operator config (error unmarshaling JSON: json: cannot unmarshal object into Go struct field ReleaseBuildConfiguration.images of type []api.ProjectDirectoryImageBuildStepConfiguration), failed to load ci-operator config (error converting YAML to JSON: yaml: line 52: did not find expected key)]
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@Prucek
Copy link
Copy Markdown
Member Author

Prucek commented Apr 1, 2026

/unhold
/label priority/ci-critical

@openshift-ci openshift-ci bot added priority/ci-critical Critical priority - CI infrastructure is broken or severely impacted and removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Apr 1, 2026
@Prucek Prucek force-pushed the prowgen-images-change branch from 089c020 to d48aeb0 Compare April 1, 2026 15:11
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek, pj-rehearse: unable to determine affected jobs. This could be due to a branch that needs to be rebased. ERROR:

could not load configuration from candidate revision of release repo: failed to load ci-operator configuration from release repo: failed to load ci-operator config (error converting YAML to JSON: yaml: line 52: did not find expected key)
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@Prucek Prucek force-pushed the prowgen-images-change branch from d48aeb0 to e2eb3b3 Compare April 1, 2026 15:20
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek, pj-rehearse: unable to determine affected jobs. This could be due to a branch that needs to be rebased. ERROR:

could not load configuration from candidate revision of release repo: failed to load ci-operator configuration from release repo: failed to load ci-operator config (error unmarshaling JSON: json: cannot unmarshal object into Go struct field ReleaseBuildConfiguration.images of type []api.ProjectDirectoryImageBuildStepConfiguration)
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@Prucek Prucek force-pushed the prowgen-images-change branch from e2eb3b3 to 73722d8 Compare April 1, 2026 16:16
@Prucek
Copy link
Copy Markdown
Member Author

Prucek commented Apr 1, 2026

/pj-rehearse pull-ci-openshift-ironic-rhcos-downloader-release-4.20-images

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@Prucek Prucek force-pushed the prowgen-images-change branch from 73722d8 to 1ef81ed Compare April 1, 2026 16:27
@Prucek
Copy link
Copy Markdown
Member Author

Prucek commented Apr 1, 2026

/pj-rehearse pull-ci-openshift-ironic-rhcos-downloader-release-4.20-images

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek, pj-rehearse: unable to determine affected jobs. This could be due to a branch that needs to be rebased. ERROR:

could not load configuration from candidate revision of release repo: failed to load ci-operator configuration from release repo: failed to load ci-operator config (error unmarshaling JSON: json: cannot unmarshal object into Go struct field ReleaseBuildConfiguration.images of type []api.ProjectDirectoryImageBuildStepConfiguration)
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@Prucek, pj-rehearse: unable to determine affected jobs ERROR:

could not load configuration from candidate revision of release repo: failed to load ci-operator configuration from release repo: failed to load ci-operator config (error unmarshaling JSON: json: cannot unmarshal object into Go struct field ReleaseBuildConfiguration.images of type []api.ProjectDirectoryImageBuildStepConfiguration)

If the problem persists, please contact Test Platform.

Prucek and others added 2 commits April 1, 2026 18:32
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Prucek Prucek force-pushed the prowgen-images-change branch from 1ef81ed to e69ee71 Compare April 1, 2026 16:34
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Apr 1, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Prucek

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

@hector-vido
Copy link
Copy Markdown
Contributor

/pj-rehearse pull-ci-openshift-ironic-rhcos-downloader-release-4.20-images

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@hector-vido: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@hector-vido: job(s): pull-ci-openshift-ironic-rhcos-downloader-release-4.20-images either don't exist or were not found to be affected, and cannot be rehearsed

@hector-vido
Copy link
Copy Markdown
Contributor

/pj-rehearse auto-ack

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@hector-vido: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Apr 1, 2026

@Prucek: The following test 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/rehearse/openstack-k8s-operators/ovn-operator/main/precommit-check e69ee71 link unknown /pj-rehearse pull-ci-openstack-k8s-operators-ovn-operator-main-precommit-check

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.

@Prucek Prucek changed the title chore: migrate images stanza to use items key and propagate filters chore: propagate filters to the images stanza Apr 2, 2026
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. priority/ci-critical Critical priority - CI infrastructure is broken or severely impacted tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants