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

ASC-978 Update "rpc-zigzag" Version #3493

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ryan-rs
Copy link
Contributor

@ryan-rs ryan-rs commented Feb 25, 2019

Update the script to pin to the 1.0 series for "rpc-zigzag". Also, add a
variable for the "rpc-zigzag" PyPI package at the top so it can be easily
updated in the future without hunting around in the script.

Issue: ASC-978

@rpc-jenkins-svc
Copy link

Thank you for submitting a pull request to this project!

Once all check jobs have passed and the pull request has received a
sufficient number of reviews, the repository's gate jobs can be triggered by
adding a comment of :shipit: to this pull request.

When the gate jobs have completed successfully, this pull request will get
merged automatically.

@ryan-rs ryan-rs force-pushed the ryan-rs/ASC-978/update_rpc-zigzag branch from 0eb779b to 71076fc Compare February 25, 2019 18:02
@ryan-rs ryan-rs requested a review from zreichert February 25, 2019 18:03
zreichert
zreichert previously approved these changes Feb 25, 2019
johnduarte
johnduarte previously approved these changes Feb 25, 2019
@johnduarte
Copy link
Contributor

@ryan-rs can you articulate why this PR should not be merged?

phongdly
phongdly previously approved these changes Feb 25, 2019
@ryan-rs ryan-rs dismissed stale reviews from phongdly, johnduarte, and zreichert via b137093 February 25, 2019 21:26
@ryan-rs ryan-rs force-pushed the ryan-rs/ASC-978/update_rpc-zigzag branch from b137093 to a947894 Compare February 25, 2019 23:07
ryan-rs and others added 2 commits February 25, 2019 15:09
Update the script to pin to the 1.0 series for "rpc-zigzag". Also, added a
variable for the "rpc-zigzag" PyPI package at the top so it can be easily
updated in the future without hunting around in the script.

The ZigZag config file is contained as a heredoc within the script which is
written out to disk at runtime.
This pins the openstack-ansible-tests to a known working SHA.

(cherry picked from commit 284cf7a)
@ryan-rs ryan-rs force-pushed the ryan-rs/ASC-978/update_rpc-zigzag branch from a947894 to b6e8e8e Compare February 25, 2019 23:23
@ryan-rs
Copy link
Contributor Author

ryan-rs commented Feb 26, 2019

recheck_bionic_no_artifacts_swift_deploy
recheck_xenial_no_artifacts_swift_deploy

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.

5 participants