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

Broken build "CloudStackException ... More than 1 default Isolated networks are found for account" #49541

Closed
aknrdureegaesr opened this issue Dec 5, 2018 · 10 comments · Fixed by #49678
Assignees
Labels
affects_2.8 This issue/PR affects Ansible v2.8 bug This issue/PR relates to a bug. cloud cloudstack module This issue/PR relates to a module. support:community This issue/PR relates to code supported by the Ansible community. test This PR relates to tests.

Comments

@aknrdureegaesr
Copy link
Contributor

aknrdureegaesr commented Dec 5, 2018

SUMMARY

I'm trying to get PR #48416 merged (which has been discussed at the docs meeting. I'm also known as dj3ei.)

I get some test failures in the Cloudstack area which I felt were unearned. To test solidity of the build, I opened up a dummy PR #49538 with only a comment change from a recent devel commit.

When testing that PR, I get the same kind of test failure:

failure: CloudStackException: ('HTTP 431 response from CloudStack', <Response [431]>, {u'errorcode': 431, u'uuidList': [], u'cserrorcode': 4350, u'errortext': u'More than 1 default Isolated networks are found for account Acct[79dffa6a-aa16-11e8-bbe5-0242ac110003-admin]; please specify networkIds'})

Hence, I assume that I have found a bug in the ansible test setup that checks PRs.

ISSUE TYPE
  • Bug Report
COMPONENT NAME

cs_snapshot_policy

ANSIBLE VERSION

Not sure what to write here. Test the check setup.

CONFIGURATION

Not sure what to write here. Check the test setup.

OS / ENVIRONMENT

Cloudstack

STEPS TO REPRODUCE
git clone -b no_real_change origin/devel

Add a comment to some important file, e.g., lib/ansible/module_utils/common/file.py. You need to do this, as opposed to an empty commit, or else the build setup will only run a reduced set of tests.

Commit, and open a pull request with that change. I recommend the title contains "WIP".

EXPECTED RESULTS

As this has no real change from the devel branch, it should build.

ACTUAL RESULTS

See the shippable log

@ansibot
Copy link
Contributor

ansibot commented Dec 5, 2018

Files identified in the description:

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

@ansibot
Copy link
Contributor

ansibot commented Dec 5, 2018

@ansibot ansibot added affects_2.8 This issue/PR affects Ansible v2.8 bug This issue/PR relates to a bug. needs_triage Needs a first human triage before being processed. support:core This issue/PR relates to code supported by the Ansible Engineering Team. test This PR relates to tests. labels Dec 5, 2018
@aknrdureegaesr
Copy link
Contributor Author

I'm not sure my original phoney pull request #49538 is still needed. Whoever tackles this, it's yours. Feel free to close at the earliest convenience.

@gundalow
Copy link
Contributor

gundalow commented Dec 5, 2018

@resmo Got any ideas about this

19:12 The full traceback is:
19:12 WARNING: The below traceback may *not* be related to the actual failure.
19:12   File "/tmp/ansible_cs_instance_payload_vjlvHQ/ansible_cs_instance_payload.zip/ansible/module_utils/cloudstack.py", line 221, in query_api
19:12     res = getattr(self.cs, command)(**args)
19:12   File "/usr/local/lib/python2.7/dist-packages/cs/client.py", line 128, in handler
19:12     return self._request(command, **kwargs)
19:12   File "/usr/local/lib/python2.7/dist-packages/cs/client.py", line 198, in _request
19:12     response.status_code), response, data)
19:12 
19:12 fatal: [testhost]: FAILED! => {
19:12     "api_http_method": "get", 
19:12     "api_key": "s7T2jkUw4lZNJd8d3931KuCDYcSS0TtbFDTZR-DQoL9-3jn_11UqE1LekwGXzPJLx87xF7nU-fi5yZzpDkOpIw", 
19:12     "api_region": "cloudstack", 
19:12     "api_timeout": "60", 
19:12     "api_url": "http://cloudstack-sim:8888/client/api", 
19:12     "changed": true, 
19:12     "diff": {
19:12         "after": {}, 
19:12         "before": {}
19:12     }, 
19:12     "invocation": {
19:12         "module_args": {
19:12             "account": null, 
19:12             "affinity_groups": null, 
19:12             "allow_root_disk_shrink": false, 
19:12             "api_http_method": null, 
19:12             "api_key": null, 
19:12             "api_region": "cloudstack", 
19:12             "api_secret": null, 
19:12             "api_timeout": null, 
19:12             "api_url": null, 
19:12             "cpu": null, 
19:12             "cpu_speed": null, 
19:12             "details": null, 
19:12             "disk_offering": null, 
19:12             "disk_size": null, 
19:12             "display_name": null, 
19:12             "domain": null, 
19:12             "force": false, 
19:12             "group": null, 
19:12             "host": null, 
19:12             "hypervisor": null, 
19:12             "ip6_address": null, 
19:12             "ip_address": null, 
19:12             "ip_to_networks": null, 
19:12             "iso": null, 
19:12             "keyboard": null, 
19:12             "memory": null, 
19:12             "name": "shippable-96761-89-vm-snapshot-policy", 
19:12             "networks": null, 
19:12             "poll_async": true, 
19:12             "project": null, 
19:12             "root_disk_size": null, 
19:12             "security_groups": null, 
19:12             "service_offering": "Small Instance", 
19:12             "ssh_key": null, 
19:12             "state": "present", 
19:12             "tags": null, 
19:12             "template": "CentOS 5.6 (64-bit) no GUI (Simulator)", 
19:12             "template_filter": "executable", 
19:12             "user_data": null, 
19:12             "zone": "Sandbox-simulator-advanced"
19:12         }
19:12     }, 
19:12     "msg": "CloudStackException: ('HTTP 431 response from CloudStack', <Response [431]>, {u'errorcode': 431, u'uuidList': [], u'cserrorcode': 4350, u'errortext': u'More than 1 default Isolated networks are found for account Acct[79dffa6a-aa16-11e8-bbe5-0242ac110003-admin]; please specify networkIds'})", 
19:12     "zone": "Sandbox-simulator-advanced"
19:12 }

@ansibot
Copy link
Contributor

ansibot commented Dec 5, 2018

Files identified in the description:

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

@ansibot
Copy link
Contributor

ansibot commented Dec 5, 2018

@ansibot ansibot added cloud cloudstack module This issue/PR relates to a module. support:community This issue/PR relates to code supported by the Ansible community. and removed 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 5, 2018
@resmo
Copy link
Contributor

resmo commented Dec 5, 2018

hmm.. weird. The only thing how this can happen AFAICS is when there was an error once and the CI worker was reused, so it uses the same setup with an undefined state?

@mattclay mattclay added this to To do in Broken Tests via automation Dec 5, 2018
@mattclay
Copy link
Member

mattclay commented Dec 5, 2018

That's the error from the retry. Here's the original error:

9:05 TASK [cs_snapshot_policy : setup instance] *************************************
19:06 fatal: [testhost]: FAILED! => {"api_http_method": "get", "api_key": "s7T2jkUw4lZNJd8d3931KuCDYcSS0TtbFDTZR-DQoL9-3jn_11UqE1LekwGXzPJLx87xF7nU-fi5yZzpDkOpIw", "api_region": "cloudstack", "api_timeout": "60", "api_url": "http://cloudstack-sim:8888/client/api", "changed": true, "msg": "CloudStackException: ('HTTP 431 response from CloudStack', <Response [431]>, {u'errorcode': 431, u'uuidList': [], u'cserrorcode': 4350, u'errortext': u'More than 1 default Isolated networks are found for account Acct[79dffa6a-aa16-11e8-bbe5-0242ac110003-admin]; please specify networkIds'})", "zone": "Sandbox-simulator-advanced"}

@mattclay
Copy link
Member

mattclay commented Dec 5, 2018

The test has been marked unstable in #49543

The test failed on the first run of the PR, which included multiple CS tests:

https://app.shippable.com/github/ansible/ansible/runs/96780/89/console

The second run passed, but that only included the cs_snapshot_policy test:

https://app.shippable.com/github/ansible/ansible/runs/96819/89/console

That may mean that the test passes when run by itself, but fails when run after other CS tests.

@resmo
Copy link
Contributor

resmo commented Dec 9, 2018

resolved_by_pr #49678

Broken Tests automation moved this from To do to Done Dec 10, 2018
@ansible ansible locked and limited conversation to collaborators Jul 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.8 This issue/PR affects Ansible v2.8 bug This issue/PR relates to a bug. cloud cloudstack module This issue/PR relates to a module. support:community This issue/PR relates to code supported by the Ansible community. test This PR relates to tests.
Projects
No open projects
Broken Tests
  
Done
Development

Successfully merging a pull request may close this issue.

5 participants