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

roachprod: upgrade Azure Ubuntu image to 20.04 #70944

Merged
merged 1 commit into from
Oct 5, 2021

Conversation

rail
Copy link
Member

@rail rail commented Sep 30, 2021

Previously, currently used Ubuntu 18.04 doesn't support systemd-run --same-dir, which is used by some roachprod scripts. Additionally, GCE
and AWS already use Ubuntu 20.04 based images for roachprod.

Updating the base image to Ubuntu 20.04 fixes the issue above and aligns
the version with other cloud providers.

Release note: None

@rail rail self-assigned this Sep 30, 2021
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Contributor

@xun-cockroachlabs xun-cockroachlabs left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @rail)


pkg/cmd/roachprod/vm/azure/azure.go, line 517 at r1 (raw file):

			},
			StorageProfile: &compute.StorageProfile{
				// You can find available versions by running the following command:

Guess you also get the info from https://discourse.ubuntu.com/t/find-ubuntu-images-on-microsoft-azure/18918, you may want to add this into the comment part for readers to reference.

Previously, currently used Ubuntu 18.04 doesn't support `systemd-run
--same-dir`, which is used by some roachprod scripts. Additionally, GCE
and AWS already use Ubuntu 20.04 based images for roachprod.

Updating the base image to Ubuntu 20.04 fixes the issue above and aligns
the version with other cloud providers.

Release note: None
@rail rail force-pushed the azure-ubuntu-20.04 branch from b46d2a1 to 9bf0789 Compare October 5, 2021 15:20
@rail rail marked this pull request as ready for review October 5, 2021 15:20
@rail rail requested a review from a team as a code owner October 5, 2021 15:20
@rail rail requested a review from jlinder October 5, 2021 15:21
@rail
Copy link
Member Author

rail commented Oct 5, 2021

bors r=jlinder

@craig
Copy link
Contributor

craig bot commented Oct 5, 2021

Build succeeded:

@craig craig bot merged commit d9eecf8 into cockroachdb:master Oct 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants