Skip to content

Improve pscloud pool CLI definitions#1

Open
DhritiJindal27 wants to merge 11 commits into
mainfrom
cli_improv_aaz
Open

Improve pscloud pool CLI definitions#1
DhritiJindal27 wants to merge 11 commits into
mainfrom
cli_improv_aaz

Conversation

@DhritiJindal27
Copy link
Copy Markdown
Owner

Summary

This PR updates the aaz command definitions for pscloud pool to improve CLI usability.

Changes

1. Removed Unsupported Identity Arguments

  • Removed Identity argument group from create and update commands
  • Removed identity property from request body schemas
  • PureStorage.Block/storagePools doesn't properly support identity types

2. Marked Required Parameters as Mandatory

  • zone (availability zone)
  • provisioned-bandwidth
  • reservation-id
  • subnet-id
  • vnet-id
  • Enables client-side validation for better UX

3. Updated Documentation

  • Fixed example in _create.md to use new --subnet-id and --vnet-id flags
  • Removed references to removed identity parameters
  • Shows realistic Azure resource ID formats

Related PR

Impact

These changes generate cleaner CLI code with better parameter validation and improved user experience.

- Rename -n and --name
- Add --zone and -z
- Remove unsupported Identity argument group from create and update commands
- Remove identity property from request body schemas
- Mark required parameters as mandatory (zone, provisioned-bandwidth, reservation-id, subnet-id, vnet-id)
- Update example to use cleaner --subnet-id and --vnet-id flags instead of --vnet-injection object
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants