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

doc(OFV): Document how to configure cloud-init #5519

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

holmanb
Copy link
Member

@holmanb holmanb commented Jul 17, 2024

Proposed commit message

doc(OFV): Document how to configure cloud-init

Additional context

#5517

Merge type

  • Squash merge using "Proposed Commit Message"
  • Rebase and merge unique commits. Requires commit messages per-commit each referencing the pull request number (#<PR_NUM>)

@holmanb holmanb requested a review from s-makin July 17, 2024 19:23
@holmanb holmanb added the documentation This Pull Request changes documentation label Jul 17, 2024
Copy link
Contributor

@aciba90 aciba90 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@holmanb holmanb merged commit 658d184 into canonical:main Jul 18, 2024
23 checks passed
What platforms support OVF
--------------------------

OFV is an open standard which is supported by various virtualization
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry I was a bit late to the party, but OFV s/b OVF :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Ups, I missed that one. The commit message also contained OFV.

@TobiPeterG
Copy link

Hey thanks. :)
This makes it clearer, however, by default the cloud image I use doesn't have the vapp property network-config.
Could this be added to the ovf image?

@aciba90
Copy link
Contributor

aciba90 commented Jul 18, 2024

Could this be added to the ovf image?

Hi, @TobiPeterG. The OVF image is not provided by cloud-init nor Canonical, I believe, but by the cloud you are running on.

@TobiPeterG
Copy link

TobiPeterG commented Jul 18, 2024

Could this be added to the ovf image?

Hi, @TobiPeterG. The OVF image is not provided by cloud-init nor Canonical, I believe, but by the cloud you are running on.

We got our image from here:
https://cloud-images.ubuntu.com/

So these are not official images?

Whoops, typo, we used the ova image

@aciba90
Copy link
Contributor

aciba90 commented Jul 22, 2024

@TobiPeterG

ova: Ubuntu image
ovf: dynamic disc with user-data / vendor-data / network config

The network config, as it is not static, has to be provided via the OVF disc, and that is build and presented by the cloud to the instance.

@holmanb
Copy link
Member Author

holmanb commented Jul 22, 2024

Could this be added to the ovf image?

I looked internally at getting a field added to the default OVF provided by the official ubuntu OVA downloads. It sounds like such a change comes with a high risk of breaking existing users of this OVA, so there is little interest in modifying the default.

@TobiPeterG You are free to copy the OVF file and add the field (copy the user-data field and rename it) if you are launching directly from an OVA, or if it is cloud-provided @aciba90 suggested, your cloud will need to provide the modification.

holmanb added a commit to holmanb/cloud-init that referenced this pull request Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This Pull Request changes documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants