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

Do not ignore failures in edpm_deploy_instance #462

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions devsetup/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -167,8 +167,8 @@ edpm_compute_cleanup: ## Delete EDPM compute VM

.PHONY: edpm_deploy_instance
edpm_deploy_instance: ## Spin a instance on edpm node
-oc cp scripts/edpm-deploy-instance.sh openstackclient:/tmp/
-oc rsh openstackclient bash /tmp/edpm-deploy-instance.sh
id;oc cp scripts/edpm-deploy-instance.sh openstackclient:/tmp/
oc rsh openstackclient bash /tmp/edpm-deploy-instance.sh

.PHONY: standalone
standalone: export STANDALONE=true
Expand Down
6 changes: 5 additions & 1 deletion devsetup/scripts/edpm-deploy-instance.sh
Original file line number Diff line number Diff line change
Expand Up @@ -44,4 +44,8 @@ openstack security group rule create --protocol icmp --ingress --icmp-type -1 $(
openstack security group rule create --protocol tcp --ingress --dst-port 22 $(openstack security group list --project admin -f value -c ID)

# check connectivity via FIP
ping -c4 192.168.122.20
if type ping >/dev/null 2>&1; then
ping -c4 192.168.122.20
else
echo "ping command not available"
fi
6 changes: 3 additions & 3 deletions zuul.d/projects.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@
github-check:
jobs:
- install-yamls-content-provider
- install-yamls-crc-podified-edpm-baremetal: &content_provider
- install-yamls-crc-podified-edpm-deployment:
dependencies:
- install-yamls-content-provider
- install-yamls-crc-podified-galera-deployment: *content_provider
- install-yamls-crc-podified-edpm-deployment: *content_provider
vars:
cifmw_edpm_deploy_run_validation: true
gibizer marked this conversation as resolved.
Show resolved Hide resolved