Skip to content
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

Implement basic permission checks and Improve OIDC support. #187

Merged
merged 21 commits into from
Oct 25, 2019

Conversation

jankaspar
Copy link
Collaborator

No description provided.

@jankaspar jankaspar added the do-not-merge Do not merge this PR label Oct 23, 2019
docs/helm/server.md Outdated Show resolved Hide resolved
Co-Authored-By: JamesMurkin <[email protected]>
JamesMurkin
JamesMurkin previously approved these changes Oct 24, 2019
@jankaspar
Copy link
Collaborator Author

Fixes #166.

@jankaspar jankaspar merged commit 84a9ced into master Oct 25, 2019
@JamesMurkin JamesMurkin deleted the auth-permissions branch October 25, 2019 11:28
svc-gh-ghzonetrans-p pushed a commit that referenced this pull request Aug 2, 2024
* Plumbing queue cordoning into armadactl

* Making armadactl cordoning commands more intuitive, some formatting changes

* Formatting

* Making queue get more intuitive

* Correcting comments and error messages

* Removing single queue cordoning code in favour of cobra alias. Adding only-cordoned flag when fetching queues

* Removing unused methods

* Removing method prefix from error messages

* Fixing import order
MustafaI added a commit that referenced this pull request Aug 2, 2024
* Plumbing queue cordoning into armadactl

* Making armadactl cordoning commands more intuitive, some formatting changes

* Formatting

* Making queue get more intuitive

* Correcting comments and error messages

* Removing single queue cordoning code in favour of cobra alias. Adding only-cordoned flag when fetching queues

* Removing unused methods

* Removing method prefix from error messages

* Fixing import order

Co-authored-by: Mustafa Ilyas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge Do not merge this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants