Skip to content

Comments

[release-4.8] Bug 1973995: Add failing tests in OVN-K into skipped list#26251

Merged
bparees merged 1 commit intoopenshift:release-4.8from
openshift-cherrypick-robot:cherry-pick-26241-to-release-4.8
Jun 30, 2021
Merged

[release-4.8] Bug 1973995: Add failing tests in OVN-K into skipped list#26251
bparees merged 1 commit intoopenshift:release-4.8from
openshift-cherrypick-robot:cherry-pick-26241-to-release-4.8

Conversation

@openshift-cherrypick-robot

This is an automated cherry-pick of #26241

/assign ardaguclu

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 19, 2021

@openshift-cherrypick-robot: Bugzilla bug 1972684 has been cloned as Bugzilla bug 1973995. Retitling PR to link against new bug.
/retitle [release-4.8] Bug 1973995: Add failing tests in OVN-K into skipped list

Details

In response to this:

[release-4.8] Bug 1972684: Add failing tests in OVN-K into skipped list

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 changed the title [release-4.8] Bug 1972684: Add failing tests in OVN-K into skipped list [release-4.8] Bug 1973995: Add failing tests in OVN-K into skipped list Jun 19, 2021
@openshift-ci openshift-ci bot added bugzilla/severity-high Referenced Bugzilla bug's severity is high for the branch this PR is targeting. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. labels Jun 19, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 19, 2021

@openshift-cherrypick-robot: This pull request references Bugzilla bug 1973995, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker.

6 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target release (4.8.0) matches configured target release for branch (4.8.0)
  • bug is in the state NEW, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)
  • dependent bug Bugzilla bug 1972684 is in the state MODIFIED, which is one of the valid states (MODIFIED, ON_QA, VERIFIED)
  • dependent Bugzilla bug 1972684 targets the "4.9.0" release, which is one of the valid target releases: 4.9.0
  • bug has dependents

No GitHub users were found matching the public email listed for the QA contact in Bugzilla (anusaxen@redhat.com), skipping review request.

Details

In response to this:

[release-4.8] Bug 1973995: Add failing tests in OVN-K into skipped list

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.

@ardaguclu
Copy link
Member

/retest

1 similar comment
@ardaguclu
Copy link
Member

/retest

@ardaguclu
Copy link
Member

@smarterclayton could you please have a look at this backport PR?, thanks

@knobunc
Copy link
Contributor

knobunc commented Jun 21, 2021

/approve
/lgtm

@openshift-ci openshift-ci bot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jun 21, 2021
@knobunc knobunc added staff-eng-approved Indicates a release branch PR has been approved by a staff engineer (formerly group/pillar lead). and removed approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jun 21, 2021
@trozet
Copy link
Contributor

trozet commented Jun 21, 2021

/retest

@trozet
Copy link
Contributor

trozet commented Jun 21, 2021

/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 21, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: knobunc, openshift-cherrypick-robot, trozet

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 21, 2021
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

6 similar comments
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@gabemontero
Copy link
Contributor

#26275 should address the e2e-aws-jenkins problem you are seeing here, but its merge is blocked by this PR among other things

classic chicken/egg problem

as it is a non-required test, I'm going to

/skip

@gabemontero
Copy link
Contributor

/test e2e-gcp-disruptive

was also not required and was impacted by my skip

I'll let others decide if that needs to be skipped long term

@gabemontero
Copy link
Contributor

ok now that the job has failed let's try skipping it again

/skip

@gabemontero
Copy link
Contributor

OK I don't understand why skip is not removing e2e-aws-jenkins from this list

@ardaguclu we may need @knobunc or someone with sufficient perms to override e2e-aws-jenkins to unravel the cross dependencies between this PR and #26275

we get this merged, and then the tests this PR is skipping will no longer block #26275

as one of the jenkins SME's I'm good with that

finally,

/test e2e-gcp-disruptive

as my skip is removing that .... I'll defer to others if that is needed for this PR

@astoycos
Copy link
Contributor

@knobunc I think we more overrides here

@akram
Copy link
Contributor

akram commented Jun 29, 2021

Maybe I have override permission for jenkins, let's see
/override ci/prow/e2e-aws-jenkins

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 29, 2021

@akram: akram unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file.

Details

In response to this:

Maybe I have override permission for jenkins, let's see
/override ci/prow/e2e-aws-jenkins

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-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@yselkowitz
Copy link
Contributor

Overrides required:
e2e-aws-jenkins is #26275
e2e-gcp is #26267

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

7 similar comments
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@sttts
Copy link
Contributor

sttts commented Jun 30, 2021

These are upstream tests. I don't think we want to skip them in this repository, but in o/k and vendor back into origin.

These tests are not OVN-K specific, are they? We are skipping them for every environment now. Is that intended and why? And if yes, what's the plan to reenable them?

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

1 similar comment
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@bparees
Copy link
Contributor

bparees commented Jun 30, 2021

/override ci/prow/e2e-aws-jenkins
/override ci/prow/e2e-gcp
/override ci/prow/e2e-gcp-disruptive

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 30, 2021

@bparees: Overrode contexts on behalf of bparees: ci/prow/e2e-aws-jenkins, ci/prow/e2e-gcp, ci/prow/e2e-gcp-disruptive

Details

In response to this:

/override ci/prow/e2e-aws-jenkins
/override ci/prow/e2e-gcp
/override ci/prow/e2e-gcp-disruptive

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
Copy link
Contributor

openshift-ci bot commented Jun 30, 2021

@openshift-cherrypick-robot: The following test failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/e2e-agnostic-cmd 1e2fc74 link /test e2e-agnostic-cmd

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.

@bparees
Copy link
Contributor

bparees commented Jun 30, 2021

this is a clean cherry pick and the issue it fixes is blocking critical 4.8 origin PRs all over the place. Manually merging.

@bparees bparees merged commit f9a8499 into openshift:release-4.8 Jun 30, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 30, 2021

@openshift-cherrypick-robot: All pull requests linked via external trackers have merged:

Bugzilla bug 1973995 has been moved to the MODIFIED state.

Details

In response to this:

[release-4.8] Bug 1973995: Add failing tests in OVN-K into skipped list

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

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. bugzilla/severity-high Referenced Bugzilla bug's severity is high for the branch this PR is targeting. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. lgtm Indicates that a PR is ready to be merged. staff-eng-approved Indicates a release branch PR has been approved by a staff engineer (formerly group/pillar lead).

Projects

None yet

Development

Successfully merging this pull request may close these issues.