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

RI-661 Implement RPC-Designate MNAIO job #3484

Open
wants to merge 1 commit into
base: rocky
Choose a base branch
from

Conversation

Frank-ZhangXin
Copy link
Contributor

@Frank-ZhangXin Frank-ZhangXin commented Feb 18, 2019

Implement rpc-designate job as one 'action', which could be used
later in rpc-gating script. This way is re-using rpc-artifacts's
mnaio vm image and then apply rpc-designate testing upon it.

Issue: RI-661

Implement rpc-designate job as one 'action', which could be used
later in rpc-gating script. This way is re-using rpc-artifacts's
mnaio vm image and then apply rpc-designate testing upon it.
@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.

@Frank-ZhangXin
Copy link
Contributor Author

Got unexpected exception:

MNAIO RPC-O deploy completed...

  • [[ designate == \d\e\p\l\o\y ]]
  • [[ designate == system* ]]
  • [[ designate == \s\d\q\c ]]
  • [[ designate == \e\l\k ]]
  • [[ designate == \d\e\s\i\g\n\a\t\e ]]
    +++ dirname ./gating/check/run
    ++ readlink -f ./gating/check/run_designate_tests.sh
  • bash -c /opt/rpc-openstack/gating/check/run_designate_tests.sh
    bash: /opt/rpc-openstack/gating/check/run_designate_tests.sh: Permission denied

Copy link

@jpward1981 jpward1981 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, fix the permissions issue and we should be good to merge

@odyssey4me
Copy link
Contributor

Until #3509 merges, the syntax/lint tests will fail.

@odyssey4me
Copy link
Contributor

recheck_container_ansible-lint_tox-test
recheck_container_ansible-syntax_tox-test
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.

4 participants