Skip to content

Comments

sshkeys: enable SSH keys injection from Scaleway IMDS#135

Merged
tormath1 merged 1 commit intoflatcar-masterfrom
tormath1/scaleway
Sep 16, 2025
Merged

sshkeys: enable SSH keys injection from Scaleway IMDS#135
tormath1 merged 1 commit intoflatcar-masterfrom
tormath1/scaleway

Conversation

@tormath1
Copy link
Contributor

@tormath1 tormath1 commented Sep 16, 2025

In this PR, we enable SSH keys injection from Scaleway IMDS. In case one wants to add SSH keys to the instance without using Ignition.


I've tested it and this has been tested by the user reporting the issue as well.

This has to be backported on some maintainance branches for init.

@tormath1 tormath1 self-assigned this Sep 16, 2025
@tormath1 tormath1 marked this pull request as ready for review September 16, 2025 14:47
@tormath1 tormath1 requested a review from a team as a code owner September 16, 2025 14:47
ConditionKernelCommandLine=|ignition.platform.id=proxmoxve
ConditionKernelCommandLine=|flatcar.oem.id=proxmoxve

ConditionKernelCommandLine=|flatcar.oem.id=scaleway
Copy link
Member

Choose a reason for hiding this comment

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

It certainly is an odd one out here, without the ignition.platform.id variant. I suppose it is on purpose, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We initially added this ignition.platform.id to be closer from the upstream but I honestly doubt we will change flatcar.oem.id -> ignition.platform.id one day.
But I'll add the ignition.platform.id to be consistent with the other entries (and we never know!)

Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
@tormath1 tormath1 merged commit ba8eefb into flatcar-master Sep 16, 2025
1 check passed
@tormath1 tormath1 deleted the tormath1/scaleway branch September 16, 2025 15:14
@tormath1
Copy link
Contributor Author

Cherry-picked to flatcar-4230-backport

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