Skip to content

Move workflows to new packet cluster profiles #21859

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
lranjbar wants to merge 1 commit into from
Closed

Move workflows to new packet cluster profiles #21859

lranjbar wants to merge 1 commit into from

Conversation

@lranjbar
Copy link
Contributor

@lranjbar lranjbar commented Sep 14, 2021 •
edited
Loading

Moving jobs that use the following workflows to their respective cluster profiles:

  • baremetalds-assisted -> packet-assisted
  • baremetalds-sno -> packet-sno

@lranjbar lranjbar force-pushed the packet-cluster-profiles-round-2 branch 2 times, most recently from be5faf7 to 700a0df Compare September 14, 2021 19:25
@lranjbar
Copy link
Contributor Author

lranjbar commented Sep 14, 2021

This PR blocks: #21739

@lranjbar
Copy link
Contributor Author

lranjbar commented Sep 14, 2021

/assign @eliorerz @hexfusion @sinnykumari @stbenjam @wking

Lots of teams involved in this one. For those interested in the errors from the packet-setup steps, check out the channel #forum-metal-alerts on Slack.

@hexfusion
Copy link
Contributor

hexfusion commented Sep 16, 2021

LGTM

@sinnykumari
Copy link
Contributor

sinnykumari commented Sep 16, 2021

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 16, 2021
@lranjbar
Copy link
Contributor Author

lranjbar commented Sep 17, 2021

/hold

Lets table this while I work on ocm-2.4 release

@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 Sep 17, 2021
@andfasano andfasano mentioned this pull request Sep 20, 2021
Closed
@lranjbar
Copy link
Contributor Author

lranjbar commented Sep 22, 2021

/unhold

@openshift-ci openshift-ci bot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Sep 22, 2021
@lranjbar lranjbar force-pushed the packet-cluster-profiles-round-2 branch from 700a0df to bed95d2 Compare September 27, 2021 23:48
@openshift-ci openshift-ci bot removed lgtm Indicates that a PR is ready to be merged. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Sep 27, 2021
@andfasano
Copy link
Contributor

andfasano commented Oct 4, 2021

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 4, 2021
@andfasano
Copy link
Contributor

andfasano commented Nov 22, 2021

@lranjbar looks like there are some conflicts to be resolved. Can you please fix them, so that we could ping the other missing owners? Thanks

@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 22, 2021
@lranjbar lranjbar force-pushed the packet-cluster-profiles-round-2 branch from bed95d2 to dfc110e Compare November 29, 2021 18:30
@openshift-ci openshift-ci bot removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. lgtm Indicates that a PR is ready to be merged. labels Nov 29, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 29, 2021

New changes are detected. LGTM label has been removed.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 29, 2021

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: andfasano, lranjbar, sinnykumari
To complete the pull request process, please ask for approval from hexfusion, wking after the PR has been reviewed.

The full list of commands accepted by this bot can be found 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

@lranjbar
Copy link
Contributor Author

lranjbar commented Nov 29, 2021

Looks like there are unwanted changes coming in after rebase.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 29, 2021

@lranjbar: 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/assisted-test-infra/master/e2e-metal-assisted-static-ip 700a0df46b00bc39567dea01387e34a178eac031 link true /test pj-rehearse
ci/rehearse/periodic-ci-openshift-assisted-test-infra-master-e2e-metal-assisted-static-ip-periodic 700a0df46b00bc39567dea01387e34a178eac031 link true /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/release-4.7/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/machine-config-operator/release-4.7/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-version-operator/release-4.7/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/release-4.9/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/release-4.10/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/release-4.8/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/installer/release-4.7/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/installer/release-4.9/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/installer/master/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/installer/release-4.10/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/release-4.6/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-version-operator/release-4.10/e2e-metal-assisted bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/release-4.10/e2e-metal-single-node-live-iso bed95d21664aaa17c35c7db9a3514f5c74764bab link unknown /test pj-rehearse
ci/prow/build-clusters dfc110e link true /test build-clusters
ci/rehearse/openshift/cluster-etcd-operator/master/e2e-metal-assisted dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/master/configmap-scale dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/release-4.10/configmap-scale dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-baremetal-operator/master/e2e-metal-assisted dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-baremetal-operator/release-4.7/e2e-metal-assisted dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-baremetal-operator/release-4.8/e2e-metal-assisted dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/master/e2e-azure dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-baremetal-operator/release-4.10/e2e-metal-assisted dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-baremetal-operator/release-4.9/e2e-metal-assisted dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/master/e2e-gcp-disruptive-five-control-plane-replicas dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/master/e2e-gcp-upgrade-five-control-plane-replicas dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/master/e2e-aws-disruptive dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/master/e2e-gcp-disruptive dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-baremetal-operator/release-4.9/e2e-metal-ipi-virtualmedia dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-baremetal-operator/master/e2e-metal-ipi-ovn-dualstack dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/master/e2e-gcp-disruptive-ovn dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-baremetal-operator/release-4.9/e2e-metal-ipi-ovn-dualstack dfc110e link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-etcd-operator/master/e2e-aws-disruptive-ovn dfc110e link unknown /test pj-rehearse
ci/prow/pj-rehearse dfc110e link false /test pj-rehearse

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.

@andfasano
Copy link
Contributor

andfasano commented Nov 30, 2021

Looks like there are unwanted changes coming in after rebase.

@lranjbar can you please re-check the rebase, I wasn't expecting updates on the always_run fields (at least for our repos).

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 30, 2021

@lranjbar: PR needs rebase.

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.

@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 30, 2021
@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 30, 2021

Issues in openshift/release go stale after 30d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 15d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@openshift-ci openshift-ci bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Dec 30, 2021
@lranjbar
Copy link
Contributor Author

lranjbar commented Jan 6, 2022

/lifecycle frozen

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 6, 2022

@lranjbar: The lifecycle/frozen label cannot be applied to Pull Requests.

Details

In response to this:

/lifecycle frozen

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.

@lranjbar
Copy link
Contributor Author

lranjbar commented Jan 6, 2022

Will be splitting this PR into multiple PRs for better chance of merging.

@openshift-bot
Copy link
Contributor

openshift-bot commented Jan 21, 2022

Stale issue in openshift/release rot after 15d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle rotten.
Rotten issues close after an additional 15d of inactivity.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle rotten
/remove-lifecycle stale

@openshift-ci openshift-ci bot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Jan 21, 2022
@lranjbar lranjbar closed this Jan 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bbguimaraes bbguimaraes Awaiting requested review from bbguimaraes

@cheesesashimi cheesesashimi Awaiting requested review from cheesesashimi

Assignees

@wking wking

@stbenjam stbenjam

@hexfusion hexfusion

@sinnykumari sinnykumari

@eliorerz eliorerz

@andfasano andfasano

Labels

lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@lranjbar @hexfusion @sinnykumari @andfasano @openshift-bot @wking @stbenjam @eliorerz