Skip to content

USHIFT-345: MicroShift: skip sig-arch tests #27407

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
pacevedom wants to merge 1 commit into from
Closed

USHIFT-345: MicroShift: skip sig-arch tests #27407

pacevedom wants to merge 1 commit into from

Conversation

@pacevedom
Copy link
Contributor

@pacevedom pacevedom commented Sep 7, 2022

No description provided.

@openshift-ci openshift-ci bot requested review from deads2k and mfojtik September 7, 2022 15:33
@pacevedom
Copy link
Contributor Author

pacevedom commented Sep 13, 2022

/hold
Need to rework.

@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 13, 2022
@pacevedom
Copy link
Contributor Author

pacevedom commented Sep 13, 2022

/unhold
Ready for review. Requires #27345
/assign @ingvagabund

@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 Sep 13, 2022
ingvagabund
ingvagabund reviewed Sep 19, 2022
View reviewed changes
ingvagabund
ingvagabund reviewed Sep 21, 2022
View reviewed changes
@ingvagabund
Copy link
Member

ingvagabund commented Sep 22, 2022

/lgtm

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

pacevedom commented Sep 22, 2022

/retest-required

3 similar comments
@pacevedom
Copy link
Contributor Author

pacevedom commented Sep 23, 2022

/retest-required

@pacevedom
Copy link
Contributor Author

pacevedom commented Sep 26, 2022

/retest-required

@pacevedom
Copy link
Contributor Author

pacevedom commented Sep 26, 2022

/retest-required

ingvagabund
ingvagabund reviewed Sep 29, 2022
View reviewed changes
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 4, 2022
ingvagabund
ingvagabund reviewed Oct 5, 2022
View reviewed changes
@ingvagabund
Copy link
Member

ingvagabund commented Oct 5, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 5, 2022
@pacevedom
Copy link
Contributor Author

pacevedom commented Oct 5, 2022

/retest-required

@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 5, 2022
@pacevedom
Copy link
Contributor Author

pacevedom commented Oct 6, 2022

/retest-required

@ingvagabund
Copy link
Member

ingvagabund commented Oct 10, 2022

/lgtm
/retest-required

@pacevedom
Copy link
Contributor Author

pacevedom commented Oct 20, 2022

/retest-required

1 similar comment
@pacevedom
Copy link
Contributor Author

pacevedom commented Oct 20, 2022

/retest-required

@pacevedom
Copy link
Contributor Author

pacevedom commented Oct 21, 2022

/retest-requirede

@pacevedom
Copy link
Contributor Author

pacevedom commented Oct 21, 2022

/retest-required

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 22, 2022

/retest-required

Remaining retests: 0 against base HEAD 6f17d51 and 2 for PR HEAD eb2d299 in total

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 22, 2022
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 24, 2022
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 24, 2022
@ingvagabund
Copy link
Member

ingvagabund commented Oct 24, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 24, 2022
@pacevedom
Copy link
Contributor Author

pacevedom commented Oct 24, 2022

/retest-required

1 similar comment
@ingvagabund
Copy link
Member

ingvagabund commented Oct 25, 2022

/retest-required

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 26, 2022

/retest-required

Remaining retests: 0 against base HEAD de3cdf6 and 2 for PR HEAD 12cf9b8 in total

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 26, 2022
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 26, 2022
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 26, 2022
@ingvagabund
Copy link
Member

ingvagabund commented Oct 26, 2022

/lgtm

@ingvagabund
Copy link
Member

ingvagabund commented Oct 26, 2022

/retest-required

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

openshift-ci bot commented Oct 26, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ingvagabund, mfojtik, pacevedom

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

@pmtk
Copy link
Member

pmtk commented Oct 26, 2022

/retest-required

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 26, 2022

@pacevedom: 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/prow/e2e-aws-csi 20b9344 link false /test e2e-aws-csi
ci/prow/e2e-aws-ovn-cgroupsv2 20b9344 link false /test e2e-aws-ovn-cgroupsv2
ci/prow/e2e-metal-ipi-ovn-ipv6 20b9344 link false /test e2e-metal-ipi-ovn-ipv6
ci/prow/e2e-aws-ovn-fips 20b9344 link true /test e2e-aws-ovn-fips
ci/prow/e2e-gcp-ovn-rt-upgrade 20b9344 link false /test e2e-gcp-ovn-rt-upgrade
ci/prow/e2e-gcp-ovn 20b9344 link true /test e2e-gcp-ovn
ci/prow/e2e-gcp-csi 20b9344 link false /test e2e-gcp-csi
ci/prow/e2e-metal-ipi-sdn 20b9344 link false /test e2e-metal-ipi-sdn
ci/prow/e2e-openstack-ovn 20b9344 link false /test e2e-openstack-ovn
ci/prow/e2e-gcp-ovn-upgrade 20b9344 link true /test e2e-gcp-ovn-upgrade
ci/prow/e2e-aws-ovn-single-node-upgrade 20b9344 link false /test e2e-aws-ovn-single-node-upgrade
ci/prow/e2e-aws-ovn-single-node 20b9344 link false /test e2e-aws-ovn-single-node
ci/prow/e2e-agnostic-ovn-cmd 20b9344 link false /test e2e-agnostic-ovn-cmd
ci/prow/e2e-aws-ovn-single-node-serial 20b9344 link false /test e2e-aws-ovn-single-node-serial

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.

@ingvagabund
Copy link
Member

ingvagabund commented Oct 26, 2022 •
edited
Loading

From pull-ci-openshift-origin-master-e2e-aws-ovn-fips and pull-ci-openshift-origin-master-e2e-gcp-ovn [sig-arch] ClusterOperators [apigroup:config.openshift.io] should define valid related objects [Suite:openshift/conformance/parallel] is constantly failing with:

{  fail [github.com/openshift/origin/test/extended/util/client.go:750]: Oct 21 00:06:57.788: open : no such file or directory
Ginkgo exit error 1: exit with code 1}

For pull-ci-openshift-origin-master-e2e-gcp-ovn-upgrade [sig-instrumentation] Prometheus [apigroup:image.openshift.io] when installed on the cluster shouldn't report any alerts in firing state apart from Watchdog and AlertmanagerReceiversNotConfigured [Early][apigroup:config.openshift.io] [Suite:openshift/conformance/parallel] failed two times in a row. https://github.com/openshift/origin/blob/master/test/extended/util/annotate/rules.go#L242 needs to be updated as well.

@ingvagabund ingvagabund mentioned this pull request Oct 26, 2022
Merged
@ingvagabund
Copy link
Member

ingvagabund commented Oct 27, 2022

Merged through #27498
/close

@openshift-ci openshift-ci bot closed this Oct 27, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 27, 2022

@ingvagabund: Closed this PR.

Details

In response to this:

Merged through #27498
/close

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@deads2k deads2k Awaiting requested review from deads2k

@mfojtik mfojtik Awaiting requested review from mfojtik

3 more reviewers

@jan--f jan--f jan--f left review comments

@simonpasquier simonpasquier simonpasquier left review comments

@ingvagabund ingvagabund ingvagabund left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@ingvagabund ingvagabund

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.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@pacevedom @ingvagabund @mfojtik @openshift-ci-robot @pmtk @jan--f @simonpasquier @openshift-merge-robot