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

CLI: Rework cmpInstanceKeys for contextual completions based on instance type #14684

Merged

Conversation

kadinsayani
Copy link
Contributor

@kadinsayani kadinsayani commented Dec 18, 2024

Fixes #14689.

Summary of changes:

  • Correctly provides instance config key completions based on instance type.

@kadinsayani kadinsayani changed the title lxc storage volume completion fixes and description updates CLI: lxc storage volume completion fixes and description updates Dec 18, 2024
@kadinsayani kadinsayani force-pushed the 14682-storage-volume-completion branch from 1c67d9f to 2f53f43 Compare December 18, 2024 00:09
@kadinsayani kadinsayani changed the title CLI: lxc storage volume completion fixes and description updates CLI: lxc storage volume completion fixes and command description updates Dec 18, 2024
@kadinsayani kadinsayani force-pushed the 14682-storage-volume-completion branch from 2f53f43 to 14d9643 Compare December 18, 2024 16:48
@kadinsayani kadinsayani changed the title CLI: lxc storage volume completion fixes and command description updates CLI: Completion fixes and command description updates Dec 18, 2024
@kadinsayani kadinsayani linked an issue Dec 18, 2024 that may be closed by this pull request
@kadinsayani kadinsayani force-pushed the 14682-storage-volume-completion branch 3 times, most recently from 463ac01 to 33c5c93 Compare December 18, 2024 21:05
@kadinsayani kadinsayani marked this pull request as ready for review December 18, 2024 22:01
@kadinsayani kadinsayani force-pushed the 14682-storage-volume-completion branch 2 times, most recently from 9a7d059 to 5495fc8 Compare December 19, 2024 15:06
@tomponline
Copy link
Member

Is it possible to split this pr into two, so we can merge the non contentious parts?

@kadinsayani kadinsayani force-pushed the 14682-storage-volume-completion branch 2 times, most recently from 4307f76 to fb65418 Compare December 19, 2024 22:22
@kadinsayani kadinsayani changed the title CLI: Completion fixes and command description updates CLI: Completion fixes Dec 19, 2024
@kadinsayani kadinsayani marked this pull request as draft December 19, 2024 22:32
@kadinsayani
Copy link
Contributor Author

kadinsayani commented Dec 19, 2024

Is it possible to split this pr into two, so we can merge the non contentious parts?

Sure thing. I've updated this PR and created #14702. Please note the addition of 5b7d222.

@kadinsayani kadinsayani changed the title CLI: Completion fixes CLI: Rework cmpInstanceKeys for contextual completions based on instance type Dec 19, 2024
@kadinsayani kadinsayani force-pushed the 14682-storage-volume-completion branch from fb65418 to 34b3579 Compare January 17, 2025 17:06
@kadinsayani kadinsayani marked this pull request as ready for review January 17, 2025 17:08
This commit adds contextual completions for instance config keys based
on instance type.

Signed-off-by: Kadin Sayani <[email protected]>
@kadinsayani kadinsayani force-pushed the 14682-storage-volume-completion branch from 34b3579 to 50b3454 Compare January 17, 2025 18:14
@kadinsayani kadinsayani force-pushed the 14682-storage-volume-completion branch from 50b3454 to 2da14fd Compare January 17, 2025 18:15
…Split` in storage pool volume completion functions

Signed-off-by: Kadin Sayani <[email protected]>
@kadinsayani kadinsayani force-pushed the 14682-storage-volume-completion branch from 2da14fd to f4dc670 Compare January 17, 2025 18:22
@tomponline tomponline merged commit 01bcd6d into canonical:main Jan 18, 2025
27 of 28 checks passed
@kadinsayani kadinsayani deleted the 14682-storage-volume-completion branch January 20, 2025 15:08
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.

cloud-init.* keys are not suggested during tab completion
3 participants