Skip to content

[DTK] Splitting the Dockerfile into 2 Dockerfiles.#75027

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
ybettan:dtk-dual-stream
Mar 12, 2026
Merged

[DTK] Splitting the Dockerfile into 2 Dockerfiles.#75027
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
ybettan:dtk-dual-stream

Conversation

@ybettan
Copy link
Member

@ybettan ybettan commented Feb 18, 2026

This is required in order to support multi-stream in OCP.

/assign @joepvd

@ybettan
Copy link
Member Author

ybettan commented Feb 18, 2026

/pj-rehearse

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Feb 18, 2026

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

@ybettan
Copy link
Member Author

ybettan commented Feb 18, 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 Feb 18, 2026
@ybettan ybettan mentioned this pull request Feb 18, 2026
Open
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 8, 2026
@ybettan ybettan force-pushed the dtk-dual-stream branch 2 times, most recently from 1d8e136 to 55cf3b6 Compare March 8, 2026 14:57
@ybettan
[DTK] Splitting the Dockerfile into 2 Dockerfiles.
1afc13d 
This is required in order to support multi-stream in OCP.

Signed-off-by: Yoni Bettan <yonibettan@gmail.com>
@ybettan
Copy link
Member Author

ybettan commented Mar 8, 2026

/pj-rehearse

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Mar 8, 2026

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

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Mar 8, 2026

[REHEARSALNOTIFIER]
@ybettan: the pj-rehearse plugin accommodates running rehearsal tests for the changes in this PR. Expand 'Interacting with pj-rehearse' for usage details. The following rehearsable tests have been affected by this change:

Test name Repo Type Reason
pull-ci-openshift-driver-toolkit-master-verify-image-content-dtk-for-rhel-10 openshift/driver-toolkit presubmit Presubmit changed
pull-ci-openshift-driver-toolkit-master-verify-image-content-dtk-for-rhel-9 openshift/driver-toolkit presubmit Presubmit changed
pull-ci-openshift-driver-toolkit-master-check-commits-count openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-e2e-aws openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-e2e-upgrade openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-images openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-okd-scos-e2e-aws-ovn openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-okd-scos-images openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-verify openshift/driver-toolkit presubmit Ci-operator config changed
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
Contributor

openshift-ci-robot commented Mar 8, 2026

[REHEARSALNOTIFIER]
@ybettan: the pj-rehearse plugin accommodates running rehearsal tests for the changes in this PR. Expand 'Interacting with pj-rehearse' for usage details. The following rehearsable tests have been affected by this change:

Test name Repo Type Reason
pull-ci-openshift-driver-toolkit-master-verify-image-content-dtk-for-rhel-10 openshift/driver-toolkit presubmit Presubmit changed
pull-ci-openshift-driver-toolkit-master-check-commits-count openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-e2e-aws openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-e2e-upgrade openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-images openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-okd-scos-e2e-aws-ovn openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-okd-scos-images openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-verify openshift/driver-toolkit presubmit Ci-operator config changed
pull-ci-openshift-driver-toolkit-master-verify-image-content openshift/driver-toolkit presubmit Ci-operator config changed
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
Copy link
Contributor

openshift-ci bot commented Mar 8, 2026

@ybettan: 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/rehearse/openshift/driver-toolkit/master/verify-image-content-dtk-for-rhel-9 80123d4 link unknown /pj-rehearse pull-ci-openshift-driver-toolkit-master-verify-image-content-dtk-for-rhel-9
ci/rehearse/openshift/driver-toolkit/master/images 1afc13d link unknown /pj-rehearse pull-ci-openshift-driver-toolkit-master-images
ci/rehearse/openshift/driver-toolkit/master/e2e-upgrade 1afc13d link unknown /pj-rehearse pull-ci-openshift-driver-toolkit-master-e2e-upgrade
ci/rehearse/openshift/driver-toolkit/master/verify-image-content-dtk-for-rhel-10 1afc13d link unknown /pj-rehearse pull-ci-openshift-driver-toolkit-master-verify-image-content-dtk-for-rhel-10
ci/rehearse/openshift/driver-toolkit/master/e2e-aws 1afc13d link unknown /pj-rehearse pull-ci-openshift-driver-toolkit-master-e2e-aws
ci/rehearse/openshift/driver-toolkit/master/check-commits-count 1afc13d link unknown /pj-rehearse pull-ci-openshift-driver-toolkit-master-check-commits-count

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.

@TomerNewman
Copy link
Contributor

TomerNewman commented Mar 12, 2026

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 12, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 12, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: TomerNewman, ybettan

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

@ybettan
Copy link
Member Author

ybettan commented Mar 12, 2026 •
edited
Loading

/unhold
/pj-rehearse ack
(CI is supposed to fail because of the chicken and egg problem with openshift/driver-toolkit#180)

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Mar 12, 2026

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

@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Mar 12, 2026
@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 12, 2026
@openshift-merge-bot openshift-merge-bot bot merged commit 8d861f3 into openshift:main Mar 12, 2026
15 of 20 checks passed
qiliRedHat pushed a commit to qiliRedHat/release that referenced this pull request Mar 13, 2026
@ybettan @qiliRedHat
[DTK] Splitting the Dockerfile into 2 Dockerfiles. (openshift#75027)
836cde4 
This is required in order to support multi-stream in OCP.

Signed-off-by: Yoni Bettan <yonibettan@gmail.com>
copejon pushed a commit to copejon/release that referenced this pull request Mar 19, 2026
@ybettan @copejon
[DTK] Splitting the Dockerfile into 2 Dockerfiles. (openshift#75027)
1881d05 
This is required in order to support multi-stream in OCP.

Signed-off-by: Yoni Bettan <yonibettan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@TomerNewman TomerNewman

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ybettan @openshift-ci-robot @TomerNewman