Skip to content

Conversation

@dilyn-corner
Copy link
Contributor

The newline in the assertion =""" line passes an unparseable assertion to snapd, resulting in syntax error messages. Removing this newline allows the assertion to be successfully acknowledged by snapd.

In order for this assertion to be acknowledged by snapd, it must also have acknowledged the account and account-key assertions for the authority-id and signing key of the assertion. snapd itself supports the --chain flag to append these assertions to the output of the sign command, so the example now accurately reflects what should be submitted to snapd.

Closes #22.

The newline in the `assertion ="""` line passes an unparseable assertion
to snapd, resulting in syntax error messages. Removing this newline
allows the assertion to be successfully acknowledged by snapd.

In order for this assertion to be acknowledged by snapd, it must
also have acknowledged the account and account-key assertions for the
authority-id and signing key of the assertion. snapd itself supports
the --chain flag to append these assertions to the output of the sign
command, so the example now accurately reflects what should be submitted
to snapd.

Signed-off-by: Dilyn Corner <dilyn.corner@canonical.com>
Copy link

@Perfect5th Perfect5th 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!

@Perfect5th Perfect5th merged commit 00b4b61 into canonical:main Dec 12, 2025
1 check 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.

Issues with add-system-user.py

2 participants