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

Enable cloud-init to inject ssh key on applehv #64

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

lstocchi
Copy link
Collaborator

@lstocchi lstocchi commented Feb 27, 2025

with crc-org/vfkit#263 and cfergeau/podman#3 we can leverage cloud-init to inject ssh key into a rhel vm.
this patch allows to set the cloudinit flag from macadam that would define if the vm should be customized by using cloud-init or the default ignition.

This needs to be updated when cfergeau/podman#3 is merged

with crc-org/vfkit#263 and cfergeau/podman#3 we can leverage cloud-init to inject ssh key into a rhel vm.
this patch allows to set the cloudinit flag from macadam that would define if the vm should be customized by using cloud-init or the default ignition.

Signed-off-by: Luca Stocchi <[email protected]>
it fixes how the local path is generated in the noop image puller.
Before the name ended up having 2 dots <name>..raw

Signed-off-by: Luca Stocchi <[email protected]>
@@ -196,6 +196,7 @@ func initMachine(cmd *cobra.Command, args []string) error {
initOpts.DiskSize = 50
initOpts.SSHIdentityPath = sshIdentityPath
initOpts.Username = username
initOpts.CloudInit = true // this should be calculated based on the image we want to start ??
Copy link
Collaborator

Choose a reason for hiding this comment

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

most likely, but we can deal with this later, I don't think it's going to be a problem to add cloudinit config to an image which does not use it.

@cfergeau
Copy link
Collaborator

Not pushing this as the go module replacement needs to be updated otherwise it won't build.

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.

2 participants