Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

Fix adding users with correct primary and secondary groups #23

Closed
wants to merge 3 commits into from

Conversation

ktsakalozos
Copy link
Contributor

No description provided.

@johnsca
Copy link
Contributor

johnsca commented Dec 3, 2015

I didn't realize when i acquiesced to Stuart's suggested param name change that it would still leave us with a mismatch between our fork and upstream, which was the whole point of getting that change upstream.

Unfortunately, that means that this change will break all of the current production charms. We'll need to either update our fork to match the param names, or provide values for all of the params and only use positional args to get around the difference in param name.

@ktsakalozos ktsakalozos closed this Dec 5, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants