Skip to content

Commit

Permalink
Revert "[tls] Add CA bundle from OpenStackCtlplane to controller"
Browse files Browse the repository at this point in the history
This is a *partial* revert of 00e8d24.

We've seen constant failures in a CI job, linked to certificate
validation:
https://review.rdoproject.org/zuul/builds?job_name=podified-multinode-edpm-e2e-nobuild-tagged-crc&project=openstack-k8s-operators/ci-framework

An example:
FAILED - RETRYING: [localhost]: Wait for keystone endpoint to exist in DNS (1 retries left).
fatal: [localhost]: FAILED! => {"attempts": 20, "changed": false, "elapsed": 0, "msg": "Status code was -1 and not [200, 300, 301, 302, 401, 402, 403]: Request failed: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self-signed certificate in certificate chain (_ssl.c:1129)>", "redirected": false, "status": -1, "url": "https://keystone-public-openstack.apps-crc.testing"}

Reverting the CA verification toggle patch seems the safest way to get
back to a green CI.
It's still supposed to fetch and install the CA at this point.

We're seeing tempest failures when we do a complete reverse.

Note: the new failure may be related to a recent patch:
openstack-k8s-operators/openstack-operator#502
  • Loading branch information
cjeanner committed Oct 27, 2023
1 parent ed7f187 commit be6189c
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 5 deletions.
2 changes: 1 addition & 1 deletion ci_framework/roles/edpm_prepare/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,5 +14,5 @@ This role doesn't need privilege escalation.
* `cifmw_edpm_prepare_oc_delay`: (Integer) Delay, in seconds, between failed oc call retries. Defaults to `30`.
* `cifmw_edpm_prepare_update_os_containers`: (Boolean) Updates the openstack services containers env variable. Defaults to `false`.
* `cifmw_edpm_prepare_timeout`: (Integer) Time, in minutes to wait for the deployment to be ready. Defaults to `30`.
* `cifmw_edpm_prepare_verify_tls`: (Boolean) In case of TLS enabled for OpenStack endpoint, validates against the CA. Defaults to `true`.
* `cifmw_edpm_prepare_verify_tls`: (Boolean) In case of TLS enabled for OpenStack endpoint, validates against the CA. Defaults to `false`.
* `cifmw_edpm_prepare_skip_patch_ansible_runner`: (Boolean) Intentionally skips setting ansible runner image to `latest` from quay.io. Defaults to `False`.
2 changes: 1 addition & 1 deletion ci_framework/roles/edpm_prepare/defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,5 +25,5 @@ cifmw_edpm_prepare_dry_run: false
cifmw_edpm_prepare_skip_crc_storage_creation: false
cifmw_edpm_prepare_update_os_containers: false
cifmw_edpm_prepare_timeout: 30
cifmw_edpm_prepare_verify_tls: true
cifmw_edpm_prepare_verify_tls: false
cifmw_edpm_prepare_skip_patch_ansible_runner: false
7 changes: 6 additions & 1 deletion ci_framework/roles/edpm_prepare/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,11 @@
KUBECONFIG: "{{ cifmw_openshift_kubeconfig }}"
PATH: "{{ cifmw_path }}"
ansible.builtin.command:
cmd: 'oc get secret combined-ca-bundle -n openstack -o "jsonpath={.data.tls-ca-bundle\.pem}" --ignore-not-found'
cmd: >-
oc get secret combined-ca-bundle
-n {{ cifmw_install_yamls_defaults.NAMESPACE }}
-o "jsonpath={.data.tls-ca-bundle\.pem}"
--ignore-not-found'
register: ca_bundle_data

- name: Get CA bundle
Expand All @@ -252,6 +256,7 @@
ansible.builtin.include_role:
role: install_ca


- name: Wait for keystone to be ready
tags:
- control-plane
Expand Down
2 changes: 1 addition & 1 deletion ci_framework/roles/os_net_setup/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,4 @@ That is provided by `openshift_login` role.
* `cifmw_os_net_setup_config`: See an example in ci_framework/roles/os_net_setup/defaults/main.yml
* `cifmw_os_net_setup_osp_calls_retries`: (Integer) Number of attempts to retry an OSP action if it fails. Defaults to `10`.
* `cifmw_os_net_setup_osp_calls_delay`: (Integer) Delay, in seconds, between failed OSP call retries. Defaults to `5`.
* `cifmw_os_net_setup_verify_tls`: (Boolean) In case of TLS enabled for OpenStack endpoint, validates against the CA. Defaults to `true`.
* `cifmw_os_net_setup_verify_tls`: (Boolean) In case of TLS enabled for OpenStack endpoint, validates against the CA. Defaults to `false`.
2 changes: 1 addition & 1 deletion ci_framework/roles/os_net_setup/defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,4 @@ cifmw_os_net_setup_config:
allocation_pool_end: 192.168.122.240
gateway_ip: 192.168.122.1
enable_dhcp: false
cifmw_os_net_setup_verify_tls: true
cifmw_os_net_setup_verify_tls: false
2 changes: 2 additions & 0 deletions ci_framework/roles/tempest/vars/main.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
cifmw_tempest_tempestconf_profile_default:
overrides:
identity.v3_endpoint_type: public
identity.disable_ssl_certificate_validation: true
dashboard.disable_ssl_certificate_validation: true

0 comments on commit be6189c

Please sign in to comment.