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-1552) Add health check tests before running system tests #3479

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

Conversation

phongdly
Copy link
Contributor

@phongdly phongdly commented Feb 11, 2019

This PR adds an experimental test of running health check tests
in repo https://github.com/rcbops/rpc-openstack-healthcheck.
The health check tests in the above repo will be executed
right after RPC-O deployment without modification anything to ensure
that the deployment is in good status before entering system tests.

Issue: ASC-1552

This PR adds an experimental test of running health check tests
in repo https://github.com/rcbops/rpc-openstack-healthcheck.
The health check tests in the above repo will be executed
right after RPC-O deployment without modification anything to ensure
that the deployment is in good status before entering system tests.
@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.

@phongdly
Copy link
Contributor Author

phongdly commented Feb 13, 2019

recheck_bionic_no_artifacts_swift_deploy
recheck_xenial_no_artifacts_swift_deploy

@johnduarte
Copy link
Contributor

recheck_bionic_no_artifacts_swift_deploy
recheck_xenial_no_artifacts_swift_deploy

2 similar comments
@phongdly
Copy link
Contributor Author

recheck_bionic_no_artifacts_swift_deploy
recheck_xenial_no_artifacts_swift_deploy

@phongdly
Copy link
Contributor Author

recheck_bionic_no_artifacts_swift_deploy
recheck_xenial_no_artifacts_swift_deploy

@phongdly
Copy link
Contributor Author

recheck_all

1 similar comment
@phongdly
Copy link
Contributor Author

recheck_all

@odyssey4me
Copy link
Contributor

Please refrain from rechecking - the failures here are legitimate and need to be resolved. Constant rechecks is just wasting CI resources.

@phongdly
Copy link
Contributor Author

@odyssey4me yes, that's absolutely correct. I think I am kinda abuse the CI resources by rechecking it again and again. I'll get the logs and open a ticket for these failures, I have seen the failure of ping: not found is very consistent in the utility container

@phongdly
Copy link
Contributor Author

phongdly commented Feb 21, 2019

RI-670 PR has been merged, recheck the jobs now.

@phongdly
Copy link
Contributor Author

recheck_all

@phongdly phongdly requested a review from ryan-rs February 21, 2019 17:31
@phongdly
Copy link
Contributor Author

recheck_bionic_no_artifacts_swift_deploy
recheck_xenial_no_artifacts_swift_deploy

@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_bionic_no_artifacts_swift_deploy
recheck_xenial_no_artifacts_swift_deploy

@odyssey4me
Copy link
Contributor

recheck_container_releasenotes_tox-test

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