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

scripts: Add bash completions for lxc auth #13159

Merged
merged 1 commit into from
Mar 15, 2024

Conversation

MggMuggins
Copy link
Contributor

I'm vacillating over including idp-groups in the top-level completion (L198). I think idp ought to complete to idp-groups, but it makes identity less convenient. Happy to change it if needed.

Resolves #12989

I've tested this pretty thoroughly but If @markylaing is willing to give them a quick spin I'd appreciate someone with a little more knowledge of the domain making sure they don't do anything too dumb.

I'm vacillating over including idp-groups in the top-level completion (L198).
I think `idp` ought to complete to idp-groups, but it makes `identity` less
convenient. Happy to patch it.

Resolves canonical#12989

Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
@markylaing
Copy link
Contributor

I can't say that I understand the bash in this file all that well (I've never actually touched the bash completions before) but I'll try it out today :)

@tomponline
Copy link
Member

Just want to check @simondeziel is happy with this.

Copy link
Member

@simondeziel simondeziel left a comment

Choose a reason for hiding this comment

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

Awesome, thanks!

@tomponline tomponline merged commit 38f0e22 into canonical:main Mar 15, 2024
28 checks passed
@MggMuggins MggMuggins deleted the auth-bash-completions branch March 26, 2024 21:37
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.

Add lxc auth * to bash completion
4 participants