Skip to content

Complete user stanza support - #11

Draft
navaneeth-dev wants to merge 3 commits into
mainfrom
feat/users-support
Draft

navaneeth-dev wants to merge 3 commits into
mainfrom
feat/users-support

Conversation

@navaneeth-dev

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI balanced review requested due to automatic review settings October 4, 2026 06:59
@navaneeth-dev navaneeth-dev self-assigned this Oct 4, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@navaneeth-dev
navaneeth-dev requested review from elmiko and tormath1 and a balanced review from Copilot October 4, 2026 06:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Convert supplementary and primary groups, password-lock policy, and
sudo rules into equivalent Flatcar Butane configuration. Reject
unrepresentable inactive accounts and unsafe generated-configuration
usernames.

Signed-off-by: Navaneeth <me@rizexor.com>
Signed-off-by: Navaneeth <me@rizexor.com>
Require an explicit primary group when a synthesized group matches a
configured username. This prevents Ignition user creation from colliding
with Linux private-group creation.

Signed-off-by: Navaneeth <me@rizexor.com>

@elmiko elmiko left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this is making sense to me, i have a minor question.

also, the documentation work is great, makes it very clear to see the state of the various cloud-init fields. well done!

Comment thread transpile.go
Comment on lines +85 to +87
config.Passwd.Groups = users.Groups
config.Passwd.Users = users.Users
config.Storage.Files = users.Files

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

do we need any sort of check on the right hand values? (eg detect nil or similar)

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.

3 participants