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

{compute-recommender} Remove default --ids from cli parameters #30108

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

joal42
Copy link
Contributor

@joal42 joal42 commented Oct 16, 2024

Related command
az compute-recommender spot-placement-recommender

Description
In order to resolve the cli user confusion, this pr removes the default --ids from the az compute-recommender spot-placement-recommender cli parameters
Testing Guide
az compute-recommender spot-placement-recommender --help
Now, the help output has no --ids
History Notes


This checklist is used to make sure that common guidelines for a pull request are followed.

Copy link

Validation for Azure CLI Full Test Starting...

Thanks for your contribution!

Copy link

Validation for Breaking Change Starting...

Thanks for your contribution!

@yonzhan
Copy link
Collaborator

yonzhan commented Oct 16, 2024

Thank you for your contribution! We will review the pull request and get back to you soon.

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.

3 participants