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

Generate cloud-config outside of cidata.iso too #2271

Merged
merged 8 commits into from
Oct 15, 2024

Conversation

afbjorklund
Copy link
Member

@afbjorklund afbjorklund commented Apr 5, 2024

This does not include any mounts, networks, nor boot scripts.
It is assumed that "reverse-sshfs" is being used, for mounts.

It also does not include lima-guestagent, nerdctl-full.tgz,
or any of the provisioning scripts that are in the cidata...

Closes #2268

@afbjorklund
Copy link
Member Author

afbjorklund commented Jul 1, 2024

Note for testing, it is possible to extract the user-data of a started instance:

isoinfo -R -i ~/.lima/default/cidata.iso -x /user-data

You could also mount the CD and copy the user-data file, but that is more work.

The new file is in ~/.lima/default/cloud-config.yaml

@afbjorklund afbjorklund force-pushed the cloud-config branch 3 times, most recently from 10b182f to 89b92e3 Compare August 31, 2024 11:25
@afbjorklund

This comment was marked as resolved.

@AkihiroSuda AkihiroSuda added this to the v1.0 milestone Oct 8, 2024
Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
This does not include any mounts, networks, nor boot scripts.
It is assumed that "reverse-sshfs" is being used, for mounts.

It also does not include lima-guestagent, nerdctl-full.tgz,
or any of the provisioning scripts that are in the cidata...

Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
It was already generated by "create", but it might have
changed since. So make sure that it is updated by "start".

It is up to the user if they already used this file, and
if so they need to update the file for the next boot too.

Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
Since it is a pointer, it is always true (non-nil).

And structs are always true, so they can't be tested.

Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda merged commit f1babc0 into lima-vm:master Oct 15, 2024
26 checks passed
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.

Generate separate cloud-config.yaml for when not using cidata.iso
2 participants