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

Update ubuntu-latest images: set them to Ubuntu 22.04 XOR comment they are latest intentionally #5006

Merged
merged 3 commits into from
Jan 10, 2023

Conversation

konrad-jamrozik
Copy link
Contributor

@konrad-jamrozik konrad-jamrozik commented Dec 21, 2022

This is a companion PR to:

In this PR, I migrate occurrences of ubuntu-latest to Ubuntu 22.04 XOR I add a comment explaining they are intentionally set to latest.

I do not have enough context to guess which images should be updated - this is just my best guess. Thus I kindly ask reviewers to verify if my guess is correct.

This PR has revealed following issue:

@konrad-jamrozik konrad-jamrozik self-assigned this Dec 21, 2022
@konrad-jamrozik konrad-jamrozik added the Central-EngSys This issue is owned by the Engineering System team. label Dec 21, 2022

pool:
name: azsdk-pool-mms-ubuntu-2204-general
vmImage: MMSUbuntu22.04
Copy link
Contributor Author

@konrad-jamrozik konrad-jamrozik Dec 21, 2022

Choose a reason for hiding this comment

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

This pool is the actual change. Not sure why entire file shows as diffed; perhaps due to automatic line endings conversion.

@konrad-jamrozik
Copy link
Contributor Author

@weshaggard I am going to override check-enforcer and merge this PR in spite of tools - mock-service-host - ci failing. I looked into it and I believe the failure is unrelated to this change. I filed an issue with my findings:

@weshaggard
Copy link
Member

/check-enforcer override

@konrad-jamrozik konrad-jamrozik merged commit 292b1f8 into main Jan 10, 2023
@konrad-jamrozik konrad-jamrozik deleted the users/kojamroz/iss_4967_latest branch January 10, 2023 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Central-EngSys This issue is owned by the Engineering System team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants