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

feat: add azure image builder #727

Merged
merged 1 commit into from
May 16, 2024
Merged

Conversation

matt-boris
Copy link
Contributor

@matt-boris matt-boris commented May 7, 2024

Fixes #728

@matt-boris matt-boris force-pushed the matt-boris/azureImageBuilder branch 4 times, most recently from 51ec715 to 24f8ffd Compare May 10, 2024 01:26
Copy link
Member

@petemoore petemoore left a comment

Choose a reason for hiding this comment

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

This is so awesome!!! I realise it is a lot of work that packer could have done for us, but I love it that we can easily see what is going on behind the scenes. I love that we don't depend on some version of packer and its dependencies etc, and only rely on the az CLI and that there is less magic about how this instance is actually getting created and snapshotted etc. Really awesome work, Boris!

imagesets/imageset.sh Outdated Show resolved Hide resolved
imagesets/imageset.sh Outdated Show resolved Hide resolved
imagesets/imageset.sh Outdated Show resolved Hide resolved
imagesets/imageset.sh Outdated Show resolved Hide resolved
imagesets/imageset.sh Outdated Show resolved Hide resolved
imagesets/imageset.sh Show resolved Hide resolved
imagesets/imageset.sh Outdated Show resolved Hide resolved
imagesets/imageset.sh Outdated Show resolved Hide resolved
imagesets/imageset.sh Show resolved Hide resolved
imagesets/imageset.sh Show resolved Hide resolved
@petemoore
Copy link
Member

Also would be good to update README.md...

@matt-boris matt-boris force-pushed the matt-boris/azureImageBuilder branch 10 times, most recently from 748a9a4 to 8811f68 Compare May 16, 2024 16:57
@matt-boris matt-boris marked this pull request as ready for review May 16, 2024 16:57
@matt-boris matt-boris requested a review from a team as a code owner May 16, 2024 16:57
@matt-boris matt-boris force-pushed the matt-boris/azureImageBuilder branch from 8811f68 to a54bac8 Compare May 16, 2024 17:04
petemoore
petemoore previously approved these changes May 16, 2024
config/azure.yml Outdated Show resolved Hide resolved
apt-get install -y google-cloud-sdk
apt-get install -y google-cloud-sdk && \
curl -sL https://aka.ms/InstallAzureCLIDeb | bash && \
rm -rf /var/lib/apt/lists/*
Copy link
Member

Choose a reason for hiding this comment

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

Do you also need to install flock here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope! Comes installed on ubuntu:22.04 :)

@matt-boris matt-boris merged commit b6ab637 into main May 16, 2024
3 checks passed
@matt-boris matt-boris deleted the matt-boris/azureImageBuilder branch May 16, 2024 20:06
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.

Create Azure image builder
2 participants