Azure / Azure/azure-cli

Can enable containerDeleteRetentionPolicy without giving retention days

Open
#28,450 1 comment 0 reactions 2 assignees Claimed by @calvinhzy View on GitHub
act-codegen-extensibility-squad Auto-Assign Azure CLI Team customer-reported potential-pruning question Storage
Dominant language
Python
Stars
4.6k
Forks
3.5k
Avg merge
3d 2h
Merged PRs (30d)
60

Description

### Describe the bug

Can run the following command without giving this falg `containerDeleteRetentionPolicy.days` then the value is set to be `null`. This value is also accepting 0. That value only accept in the range of 1 to 365.

`az storage account blob-service-properties update --set containerDeleteRetentionPolicy.enabled=true --account-name --resource-group `

### Related command

`az storage account blob-service-properties update`

### Errors

No errors

### Issue script & Debug output

[output.txt](https://github.com/Azure/azure-cli/files/14391089/output.txt)

### Expected behavior

Command should ask how many retention days I want to set

### Environment Summary

azure-cli 2.57.0

core 2.57.0
telemetry 1.1.0

Extensions:
interactive 0.5.3
ssh 1.1.6
storage-preview 0.8.3

Dependencies:
msal 1.26.0
azure-mgmt-resource 23.1.0b2

Python location 'C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users\ganeshm\.azure\cliextensions'

Python (Windows) 3.11.7 (tags/v3.11.7:fa7a6f2, Dec 4 2023, 19:13:08) [MSC v.1937 32 bit (Intel)]

Legal docs and information: aka.ms/AzureCliLegal

Your CLI is up-to-date.

### Additional context

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.