Skip to content
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.

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

Bump versions of MDM + MDSD to latest (rebased) #1644

Merged
merged 1 commit into from
Jul 28, 2021

Conversation

jewzaam
Copy link
Contributor

@jewzaam jewzaam commented Jul 27, 2021

Which issue this PR addresses:

Fixes https://msazure.visualstudio.com/AzureRedHatOpenShift/_workitems/edit/10409039

What this PR does / why we need it:

Bumps tags of MDM + MDSD
Rebase of #1640 as @hawkowl is expected to be unavailable for next day or so.

Test plan for issue:

Existing E2E

Is there any documentation that needs to be updated for this PR?

No

@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

/azp run ci

@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

/azp run e2e

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

/azp run e2e

@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

/azp run ci

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 1644 in repo Azure/ARO-RP

1 similar comment
@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 1644 in repo Azure/ARO-RP

@troy0820 troy0820 added LGTM Looks Good To Me priority-high High priority issue or pull request ready-for-review size-small Size small labels Jul 27, 2021
troy0820
troy0820 previously approved these changes Jul 27, 2021
@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

/azp run ci

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

/azp run e2e

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

mjudeikis
mjudeikis previously approved these changes Jul 27, 2021
@troy0820
Copy link
Contributor

/azp run e2e

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@github-actions github-actions bot added the needs-rebase branch needs a rebase label Jul 27, 2021
@github-actions
Copy link

Please rebase pull request.

@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

Deployed the updated operator in an INT cluster and realized after trying to deploy updated logging DS that the mdsd image isn't going to exist until we merge and mirroring is run... so with only the genevalogging test failing I'll merge, run mirror, then run e2e in INT. After I rebase.. update coming soon.

@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

I take that back, I can run mirror on a commit!

@jewzaam jewzaam dismissed stale reviews from mjudeikis and troy0820 via bebdee4 July 27, 2021 19:36
@github-actions github-actions bot removed the needs-rebase branch needs a rebase label Jul 27, 2021
@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

/azp run ci

@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

/azp run e2e

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

/azp run e2e

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 1644 in repo Azure/ARO-RP

@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

/azp run e2e

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jewzaam
Copy link
Contributor Author

jewzaam commented Jul 27, 2021

I wasn't able to get the mirror pipeline to work but have manually pushed it to INT. Manually verified in an INT cluster. Running e2e again. Mirror will push it again which is fine or we can delete it and run mirror after.

@mjudeikis mjudeikis merged commit 8026987 into Azure:master Jul 28, 2021
@jewzaam jewzaam deleted the hawkowl/mdm-update-jul branch July 28, 2021 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LGTM Looks Good To Me next-up priority-high High priority issue or pull request ready-for-review size-small Size small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants