Azure / Azure/azure-cli-extensions

[load] az load test create should not required field test-id in CLI when set in yaml configuration file.

Open
#10,020 2 comments 1 reaction 2 assignees Claimed by @a0x1ab View on GitHub
Service Attention
Dominant language
Python
Stars
454
Forks
1.7k
Avg merge
2d 19h
Merged PRs (30d)
64

Description

Source: https://github.com/Azure/azure-cli/issues/32170 (by @vlahane)
Affected extension: `load` (`src/load/`)

---

<<>>
# Issue #32170 (by @vlahane)
## Title
az load test create should not required field test-id in CLI when set in yaml configuration file.

## Body
### Describe the bug

Even if a required field is present in the configuration file, the CLI should not prompt for it as a required input.

`testId: demosimpleloatst`

but current it throws error:
`
the following arguments are required: --test-id/-t`

### Related command

z load test create --load-test-resource $LOAD_TEST_RESOURCE -g $LOAD_TEST_RESOURCE_GROUP --load-test-config-file test.yaml --autostop disable

### Errors

the following arguments are required: --test-id/-t

### Issue script & Debug output

''' yaml
version: v0.1
testId: demosimpleloatst
displayName: Sample Test
description: Load test website home page
testPlan: SampleTest.jmx
testType: JMX
engineInstances: 1
configurationFiles:
- 'sampledata.csv'
- 'testfragment.jmx'
zipArtifacts:
- bigdata.zip
'''

### Expected behavior

when Load test configuration has ''' testId''' value mentioned then cli should ask for testId in command line or we should remove it from config file

### Environment Summary

az --version
azure-cli 2.76.0 *

core 2.76.0 *
telemetry 1.1.0

Extensions:
load 2.1.0
ml 2.38.0
ssh 2.0.6

Dependencies:
msal 1.33.0b1
azure-mgmt-resource 23.3.0

### Additional context

_No response_

## Comments
### Comment by @azure-client-tools-bot-prd[bot]
Hi @vlahane,

2.76.0 is not the latest Azure CLI(2.77.0).

If you haven't already attempted to do so, please upgrade to the latest Azure CLI version by following https://learn.microsoft.com/en-us/cli/azure/update-azure-cli.

### Comment by @yonzhan

Thank you for opening this issue, we will look into it.
<<>>

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.