Skip to content

Uuids#159

Merged
jjasghar merged 5 commits intotest-kitchen:masterfrom
cas-ei:uuids
Mar 13, 2017
Merged

Uuids#159
jjasghar merged 5 commits intotest-kitchen:masterfrom
cas-ei:uuids

Conversation

@boc-tothefuture
Copy link
Copy Markdown

This PR is an optimization that enables the end user to specify the UUIDs of network, image and flavor.

The calls to lookup these elements consume up to 7 seconds if the openstack box is located remotely. Making this change reduced the total time before a create is called in openstack from 9 seconds to 2 seconds.

More details on motivation and testing are available here

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 462fd7a on cas-ei:uuids into ** on test-kitchen:master**.

1 similar comment
@coveralls
Copy link
Copy Markdown

coveralls commented Mar 3, 2017

Coverage Status

Changes Unknown when pulling 462fd7a on cas-ei:uuids into ** on test-kitchen:master**.

@jjasghar
Copy link
Copy Markdown
Contributor

Looks good, can you add some docs on the README around this change too?

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 1cdeb9d on cas-ei:uuids into ** on test-kitchen:master**.

1 similar comment
@coveralls
Copy link
Copy Markdown

coveralls commented Mar 12, 2017

Coverage Status

Changes Unknown when pulling 1cdeb9d on cas-ei:uuids into ** on test-kitchen:master**.

@boc-tothefuture
Copy link
Copy Markdown
Author

@jjasghar I updated the README.md let me know if you want me to tweak anything.

@jjasghar jjasghar merged commit 7f2d448 into test-kitchen:master Mar 13, 2017
@jjasghar
Copy link
Copy Markdown
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants