-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Fix blob type validator #1656
Fix blob type validator #1656
Conversation
@@ -122,7 +122,8 @@ def validate_source_uri(namespace): | |||
namespace.copy_source = uri | |||
|
|||
def validate_blob_type(namespace): | |||
namespace.blob_type = 'page' if namespace.file_path.endswith('.vhd') else 'block' | |||
if namespace.blob_type is None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use if not namespace.blob_type
instead of is None
To fix the error, it looks like you need to re-record the blob scenario test. In the storage test recordings delete the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Onboard with the change. Just please re-record the associated test.
* Azure/master: Fix PEP8 for azure-cli (Azure#1661) [VM/VMSS] Enable Multi-Cloud VM/VMSS Create (Azure#1654) Fix blob type validator (Azure#1656) Improve @file handling logic (Azure#1648) Name private_config.json differently in VM test (Azure#1653) Update regex of parsing sdk function argument comments (Azure#1650) Reorganize test run in automation (Azure#1624) Fix issue 1644. (Azure#1647) [Network] VNet Gateway Fixes and Enhancements (Azure#1643) [Network] Application Gateway Commands and Fixes (Azure#1606) [Docs] Correcting unittest location in docs (Azure#1642) [ACS] Add a table transform for acs show to match acs list. (Azure#1641) [ACS] Add a unit test and fix a bug with az browse ... (Azure#1640) [Component] Fix pip seg fault on 'az component update' (Azure#1639) fixed typo (Azure#1634)
The blob_type cannot be set to
append
because of this validator, such as inaz storage blob upload
command by--type
argument.