Fix azure_rm_storageaccount failure due to new version of Azure Pytho… #33897

ZhijunZhao · 2017-12-14T07:44:57Z

…n SDK

SUMMARY

The failure is as blow

\"/tmp/ansible_frHOyW/ansible_module_azure_rm_storageaccount.py\", line 416, in create_account\n    poller = self.storage_client.storage_accounts.create(self.resource_group, self.name, parameters)\n  File \"/home/travis/virtualenv/python2.7.14/lib/python2.7/site-packages/azure/mgmt/storage/v2017_06_01/operations/storage_accounts_operations.py\", line 159, in create\n    body_content = self._serialize.body(parameters, 'StorageAccountCreateParameters')\n  File \"/home/travis/virtualenv/python2.7.14/lib/python2.7/site-packages/msrest/serialization.py\", line 460, in body\n    SerializationError, \"Unable to build a model: \"+str(err), err)\n  File \"/home/travis/virtualenv/python2.7.14/lib/python2.7/site-packages/msrest/exceptions.py\", line 48, in raise_with_traceback\n    raise error\nmsrest.exceptions.SerializationError: Unable to build a model: <SkuName.standard_lrs: 'Standard_LRS'> is not valid value for enum <enum 'SkuName'>, DeserializationError: <SkuName.standard_lrs: 'Standard_LRS'> is not valid value for enum <enum 'SkuName'>\n"

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

azure_rm_storageaccount

ANSIBLE VERSION

ansible 2.5.0
  config file = None
  configured module search path = [u'/Users/zhijzhao/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python2.7/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.14 (default, Sep 25 2017, 09:53:22) [GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.37)]

ADDITIONAL INFORMATION

@zikalino @yuwzho @yaweiw

…n SDK

ansibot · 2017-12-14T07:52:08Z

cc @haroldwongms @nitzmahone @tstringer @xscript
click here for bot help

ansibot · 2017-12-14T08:06:52Z

The test ansible-test sanity --test azure-requirements [?] failed with the following error:

Command "test/sanity/code-smell/azure-requirements.py" returned exit status 1.
>>> Standard Output
Update the Azure integration test requirements with the packaging test requirements:
cp packaging/requirements/requirements-azure.txt test/runner/requirements/integration.cloud.azure.txt

click here for bot help

yuwzho · 2017-12-14T09:03:39Z

shipit

ZhijunZhao · 2017-12-19T02:22:27Z

@nitzmahone

ansibot · 2017-12-25T10:14:52Z

@ZhijunZhao this PR contains the following merge commits:

9efac32

Please rebase your branch to remove these commits.

click here for bot help

ZhijunZhao · 2017-12-25T10:17:46Z

@lmazuel Please help to review the code. If you think this is the appropriate solution, I'll make similar changes to the models of other clients.

ansibot · 2017-12-25T10:57:52Z

The test ansible-test sanity --test pep8 [?] failed with the following errors:

lib/ansible/modules/cloud/azure/azure_rm_storageaccount.py:354:43: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_storageaccount.py:413:82: E127 continuation line over-indented for visual indent

click here for bot help

ansibot · 2017-12-28T09:17:03Z

The test ansible-test sanity --test pep8 [?] failed with the following errors:

lib/ansible/module_utils/azure_rm_common.py:688:34: E128 continuation line under-indented for visual indent
lib/ansible/module_utils/azure_rm_common.py:695:34: E128 continuation line under-indented for visual indent
lib/ansible/module_utils/azure_rm_common.py:697:34: E128 continuation line under-indented for visual indent
lib/ansible/module_utils/azure_rm_common.py:708:34: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_image.py:239:28: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_image.py:240:28: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_image.py:241:28: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_image.py:242:28: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_image.py:250:34: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_image.py:251:34: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_image.py:252:34: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_networkinterface.py:458:71: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_networkinterface.py:459:71: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_networkinterface.py:501:75: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_networkinterface.py:502:75: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_subnet.py:241:78: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_subnet.py:242:78: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_subnet.py:253:78: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_subnet.py:254:78: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py:941:37: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py:1648:66: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py:1649:66: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py:1651:77: E128 continuation line under-indented for visual indent
lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py:1652:77: E128 continuation line under-indented for visual indent

The test ansible-test sanity --test pylint [?] failed with the following error:

lib/ansible/modules/cloud/azure/azure_rm_loadbalancer.py:429:46: undefined-variable Undefined variable 'Subnet'

click here for bot help

ansibot · 2017-12-28T11:14:42Z

The test ansible-test sanity --test pylint [?] failed with the following error:

lib/ansible/modules/cloud/azure/azure_rm_loadbalancer.py:429:46: undefined-variable Undefined variable 'Subnet'

click here for bot help

ansibot · 2018-01-10T07:08:36Z

@ZhijunZhao this PR contains the following merge commits:

Please rebase your branch to remove these commits.

click here for bot help

* resolved a number of conflicts

nitzmahone · 2018-01-10T07:25:59Z

merge conflict corrected out-of-band, direct committed to devel, cherry picked to stable-2.4 via #34682 for 2.4.3rc1

* reverts ansible#33897/ansible#34682 * revisit in 2.4.4/2.5.0

* reverts #33897/#34682 * revisit in 2.4.4/2.5.0

Fix azure_rm_storageaccount failure due to new version of Azure Pytho…

85e22b7

…n SDK

ansibot added affects_2.5 This issue/PR affects Ansible v2.5 azure bugfix_pull_request cloud needs_triage Needs a first human triage before being processed. support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Dec 14, 2017

ansibot added the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label Dec 14, 2017

fix test failure

8a1a4ea

ansibot added the test This PR relates to tests. label Dec 14, 2017

ansibot removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. needs_triage Needs a first human triage before being processed. labels Dec 14, 2017

ZhijunZhao mentioned this pull request Dec 19, 2017

fix failure got multiple values for keyword argument id #33624

Merged

Zhijun Zhao added 3 commits December 25, 2017 13:59

Merge branch 'devel' into zhijzhao-storage

9efac32

pin api version of azure storage models

082e67b

pin api version of azure storage models

f0f7124

ansibot added merge_commit This PR contains at least one merge commit. Please resolve! module This issue/PR relates to a module. needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html support:certified This issue/PR relates to certified code. labels Dec 25, 2017

use semantic versioning

fff438c

ansibot added ci_verified Changes made in this PR are causing tests to fail. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Dec 25, 2017

Zhijun Zhao added 3 commits December 28, 2017 16:27

network models

cd180a5

resources models

929eda6

compute models

7704942

ansibot added the support:community This issue/PR relates to code supported by the Ansible community. label Dec 28, 2017

ansibot added the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label Dec 28, 2017

fix lint errors

6eb2ec8

ZhijunZhao added 2 commits December 28, 2017 19:17

fix typo

7e659e5

used wrong models

2c42aca

ansibot removed the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label Dec 28, 2017

yuwzho mentioned this pull request Jan 4, 2018

Upgrade the azure storage version #34442

Merged

ansibot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. labels Jan 5, 2018

nitzmahone added 2 commits January 9, 2018 23:03

Merge branch 'devel' into zhijzhao-storage

05a36fe

Merge branch 'devel' into zhijzhao-storage

2f7da79

ansibot removed the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Jan 10, 2018

nitzmahone added a commit to nitzmahone/ansible that referenced this pull request Jan 10, 2018

cherry-pick ansible#33897

b010638

* resolved a number of conflicts

nitzmahone added a commit that referenced this pull request Jan 10, 2018

cherry-pick #33897 (#34682)

bb95774

* resolved a number of conflicts

nitzmahone closed this Jan 10, 2018

nitzmahone mentioned this pull request Jan 10, 2018

Azure Working Group agenda - Nov 2017-Jun 2019 ansible/community#281

Closed

nitzmahone added a commit to nitzmahone/ansible that referenced this pull request Jan 17, 2018

revert azure_rm model pinning

b86efcb

* reverts ansible#33897/ansible#34682 * revisit in 2.4.4/2.5.0

nitzmahone mentioned this pull request Jan 17, 2018

revert azure_rm model pinning #35011

Merged

abadger pushed a commit that referenced this pull request Jan 17, 2018

revert azure_rm model pinning (#35011)

949e2f0

* reverts #33897/#34682 * revisit in 2.4.4/2.5.0

ansibot added bug This issue/PR relates to a bug. and removed bugfix_pull_request labels Mar 6, 2018

ansible locked and limited conversation to collaborators Apr 26, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix azure_rm_storageaccount failure due to new version of Azure Pytho… #33897

Fix azure_rm_storageaccount failure due to new version of Azure Pytho… #33897

ZhijunZhao commented Dec 14, 2017

ansibot commented Dec 14, 2017

ansibot commented Dec 14, 2017

yuwzho commented Dec 14, 2017

ZhijunZhao commented Dec 19, 2017

ansibot commented Dec 25, 2017

ZhijunZhao commented Dec 25, 2017

ansibot commented Dec 25, 2017

ansibot commented Dec 28, 2017

ansibot commented Dec 28, 2017

ansibot commented Jan 10, 2018

nitzmahone commented Jan 10, 2018 •

edited

Loading

Fix azure_rm_storageaccount failure due to new version of Azure Pytho… #33897

Fix azure_rm_storageaccount failure due to new version of Azure Pytho… #33897

Conversation

ZhijunZhao commented Dec 14, 2017

SUMMARY

ISSUE TYPE

COMPONENT NAME

ANSIBLE VERSION

ADDITIONAL INFORMATION

ansibot commented Dec 14, 2017

ansibot commented Dec 14, 2017

yuwzho commented Dec 14, 2017

ZhijunZhao commented Dec 19, 2017

ansibot commented Dec 25, 2017

ZhijunZhao commented Dec 25, 2017

ansibot commented Dec 25, 2017

ansibot commented Dec 28, 2017

ansibot commented Dec 28, 2017

ansibot commented Jan 10, 2018

nitzmahone commented Jan 10, 2018 • edited Loading

nitzmahone commented Jan 10, 2018 •

edited

Loading