Skip to content

Modifying image-mirroring periodic to use new image#51150

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:masterfrom
deepsm007:modify_mirroring_periodic
Apr 19, 2024
Merged

Modifying image-mirroring periodic to use new image#51150
openshift-merge-bot[bot] merged 1 commit intoopenshift:masterfrom
deepsm007:modify_mirroring_periodic

Conversation

@deepsm007
Copy link
Copy Markdown
Contributor

@deepsm007 deepsm007 commented Apr 19, 2024

/cc @hongkailiu

/hold checking all rehearsal...

@openshift-ci openshift-ci Bot requested a review from hongkailiu April 19, 2024 19:36
@openshift-ci openshift-ci Bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Apr 19, 2024
@deepsm007
Copy link
Copy Markdown
Contributor Author

deepsm007 commented Apr 19, 2024

/pj-rehearse more

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Apr 19, 2024

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

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

openshift-ci-robot commented Apr 19, 2024

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

@deepsm007
Copy link
Copy Markdown
Contributor Author

deepsm007 commented Apr 19, 2024

/pj-rehearse periodic-image-mirroring-konveyor, periodic-image-mirroring-ovirt

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Apr 19, 2024

@deepsm007: 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

openshift-ci-robot commented Apr 19, 2024

@deepsm007: job(s): periodic-image-mirroring-konveyor, either don't exist or were not found to be affected, and cannot be rehearsed

@deepsm007 deepsm007 force-pushed the modify_mirroring_periodic branch from 6ed1fa6 to 7a953f2 Compare April 19, 2024 20:02
@deepsm007
Copy link
Copy Markdown
Contributor Author

deepsm007 commented Apr 19, 2024

/pj-rehearse periodic-image-mirroring-aws-load-balancer-operator, /pj-rehearse periodic-image-mirroring-openshift-kni, /pj-rehearse periodic-image-mirroring-konveyor

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Apr 19, 2024

@deepsm007: 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

openshift-ci-robot commented Apr 19, 2024

@deepsm007: job(s): periodic-image-mirroring-aws-load-balancer-operator,, /pj-rehearse, periodic-image-mirroring-openshift-kni,, /pj-rehearse either don't exist or were not found to be affected, and cannot be rehearsed

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Apr 19, 2024

@deepsm007: 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

openshift-ci-robot commented Apr 19, 2024

@deepsm007: requesting more than one rehearsal in one comment is not supported. If you would like to rehearse multiple specific jobs, please separate the job names by a space in a single command.

@deepsm007
Copy link
Copy Markdown
Contributor Author

deepsm007 commented Apr 19, 2024

/pj-rehearse periodic-image-mirroring-openshift-kni

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Apr 19, 2024

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

@deepsm007
Copy link
Copy Markdown
Contributor Author

deepsm007 commented Apr 19, 2024

Dry run looks good
edge-infrastructure: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/51150/rehearse-51150-periodic-image-mirroring-edge-infrastructure/1781407814321704960
external-dns-operator: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/51150/rehearse-51150-periodic-image-mirroring-external-dns-operator/1781407810949484544
hypershift: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/51150/rehearse-51150-periodic-image-mirroring-hypershift/1781407824371257344
lightspeed-service: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/51150/rehearse-51150-periodic-image-mirroring-lightspeed-service/1781407809326288896
node-observability-operator: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/51150/rehearse-51150-periodic-image-mirroring-node-observability-operator/1781407790011518976
openshift-azure: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/51150/rehearse-51150-periodic-image-mirroring-openshift-azure/1781407790825213952
osa: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/51150/rehearse-51150-periodic-image-mirroring-osa/1781407790070239232
ovirt: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/51150/rehearse-51150-periodic-image-mirroring-ovirt/1781411598758318080
redhat-developer: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/51150/rehearse-51150-periodic-image-mirroring-redhat-developer/1781407790305120256
konveyor: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/51150/rehearse-51150-periodic-image-mirroring-konveyor/1781414427904446464
aws-load-balancer-operator: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_rele[…]e-mirroring-aws-load-balancer-operator/1781415145491140608
openshift-kni: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/51150/rehearse-51150-periodic-image-mirroring-openshift-kni/1781416077993971712

@deepsm007 deepsm007 force-pushed the modify_mirroring_periodic branch from 7a953f2 to f9aa4da Compare April 19, 2024 20:25
@deepsm007
Copy link
Copy Markdown
Contributor Author

deepsm007 commented Apr 19, 2024

/hold cancel

@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 Apr 19, 2024
@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Apr 19, 2024
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Apr 19, 2024

[REHEARSALNOTIFIER]
@deepsm007: no rehearsable tests are affected by this change

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 abort to abort all active rehearsals

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.

@hongkailiu
Copy link
Copy Markdown
Member

hongkailiu commented Apr 19, 2024

/lgtm

/hold

Free to chance if you are going to watch the rolling ...
It is late Friday afternoon. I would suggest to do it next week.

@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 Apr 19, 2024
@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Apr 19, 2024
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 19, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deepsm007, hongkailiu

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

@deepsm007
Copy link
Copy Markdown
Contributor Author

deepsm007 commented Apr 19, 2024

/hold cancel
Will watch the rolling before calling it a day

@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 Apr 19, 2024
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 19, 2024

@deepsm007: all tests passed!

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

@openshift-merge-bot openshift-merge-bot Bot merged commit 8396468 into openshift:master Apr 19, 2024
@deepsm007
Copy link
Copy Markdown
Contributor Author

deepsm007 commented Apr 19, 2024

https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-aws-load-balancer-operator/1781427683977072640
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-external-dns-operator/1781427681468878848
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-lightspeed-service/1781427691551985664
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-osa/1781427689010237440
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-edge-infrastructure/1781427672220438528
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-hypershift/1781427674737020928
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-konveyor/1781427676427325440
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-node-observability-operator/1781427678931324928
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-openshift-azure/1781427654654693376
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-openshift-kni/1781427654881185792
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-ovirt/1781427654969266176
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-image-mirroring-redhat-developer/1781431177941356544

@alebedev87 alebedev87 mentioned this pull request Sep 17, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hongkailiu hongkailiu Awaiting requested review from hongkailiu

Assignees

@hongkailiu hongkailiu

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

@deepsm007 @openshift-ci-robot @hongkailiu