add-cloud vsphere prompts for vCenter and datacenter
Bug #1676539 reported by
Curtis Hovey
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-ci-tools |
Fix Released
|
Critical
|
Curtis Hovey |
Bug Description
As seen at
http://
Several tests are failing because of
https:/
you should be asked for "the vCenter address or URL" (endpoint) and "datacenter name" (region)
Enter the vCenter address or URL: ERROR:root:Adding cloud failed: pexpect session timed out
The test needs to change. We need to support juju 2.1, so the expect rules need to OR old and new prompt.
Related branches
lp:~sinzui/juju-ci-tools/vsphere-add-cloud
- Aaron Bentley (community): Approve
-
Diff: 132 lines (+52/-11)3 files modifiedjujupy/client.py (+8/-5)
jujupy/fake.py (+18/-6)
jujupy/tests/test_version_client.py (+26/-0)
Changed in juju-ci-tools: | |
assignee: | nobody → s. (salamcainc) |
status: | Triaged → Fix Released |
Changed in juju-ci-tools: | |
status: | Fix Released → Triaged |
assignee: | s. (salamcainc) → nobody |
Changed in juju-ci-tools: | |
status: | In Progress → Fix Released |
To post a comment you must log in.
ModelClient. add_cloud_ interactive( )'s vsphere block could expect
child. expect( 'Enter the API endpoint url for the cloud:')
child. sendline( cloud[' endpoint' ]) cloud[' regions' ].items( )):
child. expect( "(Enter region name:)|"
"( Can't validate endpoint)")
if child.match. group(2) is not None:
raise InvalidEndpoint()
child. sendline( name)
child. expect( 'Enter another region\? \(Y/n\):')
if num + 1 < len(cloud[ 'regions' ]):
child. sendline( 'y')
else:
child. sendline( 'n')
"Enter the vCenter address or URL:|Enter the API endpoint url for the cloud:"
and
"Enter region name:|Enter datacenter name:"
and other prompts
...
if cloud['type'] == 'vsphere':
for num, (name, values) in enumerate(