Skip to content

Comments

Bug 1976753: move jenkins e2e's that were using mongodb, then mariadb, to postgresql, to get them to pass again#26274

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
gabemontero:alt-fix-jenkins-only-e2e
Jun 29, 2021
Merged

Bug 1976753: move jenkins e2e's that were using mongodb, then mariadb, to postgresql, to get them to pass again#26274
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
gabemontero:alt-fix-jenkins-only-e2e

Conversation

@gabemontero
Copy link
Contributor

Replacement to #26268

/assign @yselkowitz
/assign @adambkaplan

@akram @waveywaves @jkhelil FYI ... the reader's digest of the #26268 and #26072 is that in the attempt to move a bunch of e2e's off of mongodb, as well as onto ubi8, e2e-aws-jenkins slipped through the cracks

I've updated openshift/release to make sure e2e-aws-jenkins runs when the files in question are touched (we'll see if that works here in a sec)

I was able to run the jenkins suite locally with these changes

@gabemontero
Copy link
Contributor Author

ok e2e-aws-jenkins has fired ... good news there

@gabemontero
Copy link
Contributor Author

ok e2e-aws-jenkins passed that is what this PR addresses

@yselkowitz
Copy link
Contributor

/lgtm
thanks and sorry for all the extra work on this

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

the cmd failure is unrelated but I'll try it again before skipping

/retest

@gabemontero
Copy link
Contributor Author

all green e2e's @adambkaplan

need an approved label to get this to merge an unblock e2e-aws-jenkins e2e's for @akram @waveywaves @jkhelil

jenkins e2e's in e2e-*-builds do pass currently

Copy link
Contributor

@adambkaplan adambkaplan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

We need to move off mongodb so we can test ARM, Power, and Z

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 28, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adambkaplan, gabemontero, yselkowitz

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 28, 2021
@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.

@openshift-merge-robot openshift-merge-robot merged commit 516e761 into openshift:master Jun 29, 2021
@gabemontero gabemontero deleted the alt-fix-jenkins-only-e2e branch June 29, 2021 14:45
@gabemontero gabemontero changed the title move jenkins e2e's that were using mongodb, then mariadb, to postgresql, to get them to pass again Bug 1976753: move jenkins e2e's that were using mongodb, then mariadb, to postgresql, to get them to pass again Jul 6, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 6, 2021

@gabemontero: All pull requests linked via external trackers have merged:

Bugzilla bug 1976753 has been moved to the MODIFIED state.

Details

In response to this:

Bug 1976753: move jenkins e2e's that were using mongodb, then mariadb, to postgresql, to get them to pass again

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. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants