-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add orgadm for new organization setup #41
Conversation
Pull Request Test Coverage Report for Build 10723930805Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
Whoops - I forgot about tests for this one - I'll request review again when those are ready |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: complete! 1 of 1 approvals obtained
Oops, I missed your comment before I approve the PR in Reviewable.io. I'll wait for the update. |
PTAL? I have added unit tests now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: complete! 1 of 1 approvals obtained
This change adds a new type
adminx.Org
and commandorgadm
to setup a new organization. The setup includes creating service accounts with no keys and secrets with no versions. An organization should be setup once before the first node attempts to Register via the Autojoin API.This change is