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

[az load] fix empty object on cli when file upload #8148

Merged
merged 3 commits into from
Oct 18, 2024
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/load/HISTORY.rst
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,11 @@

Release History
===============
1.1.1
++++++
* Fix empty response object on CLI when using 'az load test file upload'
* Add termination status 'NOT_VALIDATED' for LRO Poller
mbhardwaj-msft marked this conversation as resolved.
Show resolved Hide resolved

1.1.0
++++++
* Add support for ZIP artifacts upload to a test. Artifacts can be uploaded through YAML config when using --load-test-config-file and through cmd `az load test file upload`, the associate --file-type is `ZIPPED_ARTIFACTS`
Expand Down
2 changes: 1 addition & 1 deletion src/load/azext_load/data_plane/load_test/custom.py
Original file line number Diff line number Diff line change
Expand Up @@ -386,7 +386,7 @@ def upload_test_file(
)
logger.debug("Upload test file response: %s", response)
logger.info("Upload test file completed")
return response
return response.as_dict()
mbhardwaj-msft marked this conversation as resolved.
Show resolved Hide resolved


def list_test_file(
Expand Down
924 changes: 462 additions & 462 deletions src/load/azext_load/tests/latest/recordings/test_load_app_component.yaml

Large diffs are not rendered by default.

496 changes: 248 additions & 248 deletions src/load/azext_load/tests/latest/recordings/test_load_scenarios.yaml

Large diffs are not rendered by default.

Large diffs are not rendered by default.

974 changes: 529 additions & 445 deletions src/load/azext_load/tests/latest/recordings/test_load_test_create.yaml

Large diffs are not rendered by default.

Large diffs are not rendered by default.

374 changes: 208 additions & 166 deletions src/load/azext_load/tests/latest/recordings/test_load_test_delete.yaml

Large diffs are not rendered by default.

Large diffs are not rendered by default.

770 changes: 406 additions & 364 deletions src/load/azext_load/tests/latest/recordings/test_load_test_file.yaml

Large diffs are not rendered by default.

430 changes: 278 additions & 152 deletions src/load/azext_load/tests/latest/recordings/test_load_test_list.yaml

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

896 changes: 406 additions & 490 deletions src/load/azext_load/tests/latest/recordings/test_load_test_run_list.yaml

Large diffs are not rendered by default.

1,432 changes: 611 additions & 821 deletions src/load/azext_load/tests/latest/recordings/test_load_test_run_metrics.yaml

Large diffs are not rendered by default.

Large diffs are not rendered by default.

896 changes: 427 additions & 469 deletions src/load/azext_load/tests/latest/recordings/test_load_test_run_show.yaml

Large diffs are not rendered by default.

558 changes: 232 additions & 326 deletions src/load/azext_load/tests/latest/recordings/test_load_test_run_stop.yaml

Large diffs are not rendered by default.

1,060 changes: 488 additions & 572 deletions src/load/azext_load/tests/latest/recordings/test_load_test_run_update.yaml

Large diffs are not rendered by default.

Large diffs are not rendered by default.

322 changes: 161 additions & 161 deletions src/load/azext_load/tests/latest/recordings/test_load_test_show.yaml

Large diffs are not rendered by default.

404 changes: 202 additions & 202 deletions src/load/azext_load/tests/latest/recordings/test_load_test_update.yaml

Large diffs are not rendered by default.

Large diffs are not rendered by default.

6 changes: 3 additions & 3 deletions src/load/azext_load/tests/latest/test_load_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -886,15 +886,15 @@ def test_load_test_file(self):
"file_type": LoadTestConstants.ZIP_ARTIFACT_TYPE,
"file_path": LoadTestConstants.ZIP_ARTIFACT_FILE
})
self.cmd(
response = self.cmd(
"az load test file upload "
"--test-id {test_id} "
"--load-test-resource {load_test_resource} "
"--resource-group {resource_group} "
"--file-type {file_type} "
'--path "{file_path}" '
)
time.sleep(10)
mbhardwaj-msft marked this conversation as resolved.
Show resolved Hide resolved
).get_output_in_json()
assert self.kwargs["file_name"] == response["fileName"]
files = self.cmd(
"az load test file list "
"--test-id {test_id} "
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ def __init__(self, interval=5) -> None:
self._polling_interval = interval
self._status = None
self._termination_statuses = [
"NOT_VALIDATED",
"VALIDATION_SUCCESS",
"VALIDATION_FAILED",
"VALIDATION_NOT_REQUIRED",
Expand Down
2 changes: 1 addition & 1 deletion src/load/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@


# HISTORY.rst entry.
VERSION = '1.1.0'
VERSION = '1.1.1'

# The full list of classifiers is available at
# https://pypi.python.org/pypi?%3Aaction=list_classifiers
Expand Down
Loading