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.

Sign up for GitHub

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

Jump to bottom

[OSPRH-12795] Remove tls cert and key from molecule tests in telemetry #849

Merged
merged 2 commits into from
Jan 9, 2025
Merged

[OSPRH-12795] Remove tls cert and key from molecule tests in telemetry #849

merged 2 commits into from
Jan 9, 2025
+46 −103

Conversation

jlarriba
Copy link
Contributor

@jlarriba jlarriba commented Jan 8, 2025

No description provided.

@openshift-ci openshift-ci bot added the approved label Jan 8, 2025
@jlarriba jlarriba force-pushed the tls_generation branch 3 times, most recently from 15c31e4 to fa16901 Compare January 8, 2025 14:40
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/75eeb4219526488a8156d6ec6b16337c

✔️ edpm-ansible-molecule-edpm_bootstrap SUCCESS in 7m 02s
✔️ edpm-ansible-molecule-edpm_podman SUCCESS in 6m 11s
✔️ edpm-ansible-molecule-edpm_module_load SUCCESS in 4m 39s
✔️ edpm-ansible-molecule-edpm_kernel SUCCESS in 9m 52s
✔️ edpm-ansible-molecule-edpm_libvirt SUCCESS in 9m 43s
✔️ edpm-ansible-molecule-edpm_nova SUCCESS in 8m 18s
✔️ edpm-ansible-molecule-edpm_frr SUCCESS in 7m 06s
✔️ edpm-ansible-molecule-edpm_iscsid SUCCESS in 4m 46s
✔️ edpm-ansible-molecule-edpm_ovn_bgp_agent SUCCESS in 7m 37s
✔️ edpm-ansible-molecule-edpm_ovs SUCCESS in 12m 08s
✔️ edpm-ansible-molecule-edpm_tripleo_cleanup SUCCESS in 4m 08s
✔️ edpm-ansible-molecule-edpm_tuned SUCCESS in 6m 22s
❌ edpm-ansible-molecule-edpm_telemetry_power_monitoring FAILURE in 6m 03s
✔️ edpm-ansible-molecule-edpm_update SUCCESS in 6m 24s

@jlarriba jlarriba force-pushed the tls_generation branch 3 times, most recently from 1ca1e40 to 67835f4 Compare January 8, 2025 15:04
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/950043f982474a9ababb3e9949f458c0

✔️ edpm-ansible-molecule-edpm_bootstrap SUCCESS in 5m 39s
✔️ edpm-ansible-molecule-edpm_podman SUCCESS in 4m 40s
✔️ edpm-ansible-molecule-edpm_module_load SUCCESS in 4m 55s
✔️ edpm-ansible-molecule-edpm_kernel SUCCESS in 9m 45s
✔️ edpm-ansible-molecule-edpm_libvirt SUCCESS in 9m 54s
✔️ edpm-ansible-molecule-edpm_nova SUCCESS in 11m 01s
✔️ edpm-ansible-molecule-edpm_frr SUCCESS in 6m 56s
✔️ edpm-ansible-molecule-edpm_iscsid SUCCESS in 4m 48s
✔️ edpm-ansible-molecule-edpm_ovn_bgp_agent SUCCESS in 8m 15s
✔️ edpm-ansible-molecule-edpm_ovs SUCCESS in 12m 42s
✔️ edpm-ansible-molecule-edpm_tripleo_cleanup SUCCESS in 4m 10s
✔️ edpm-ansible-molecule-edpm_tuned SUCCESS in 6m 04s
❌ edpm-ansible-molecule-edpm_telemetry_power_monitoring FAILURE in 5m 57s
✔️ edpm-ansible-molecule-edpm_update SUCCESS in 6m 25s

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/d2e156c96e5142ff917c19a2c8ce7026

✔️ edpm-ansible-molecule-edpm_bootstrap SUCCESS in 6m 58s
✔️ edpm-ansible-molecule-edpm_podman SUCCESS in 5m 55s
✔️ edpm-ansible-molecule-edpm_module_load SUCCESS in 4m 32s
✔️ edpm-ansible-molecule-edpm_kernel SUCCESS in 9m 31s
✔️ edpm-ansible-molecule-edpm_libvirt SUCCESS in 9m 39s
✔️ edpm-ansible-molecule-edpm_nova SUCCESS in 8m 59s
✔️ edpm-ansible-molecule-edpm_frr SUCCESS in 5m 56s
✔️ edpm-ansible-molecule-edpm_iscsid SUCCESS in 4m 40s
✔️ edpm-ansible-molecule-edpm_ovn_bgp_agent SUCCESS in 8m 37s
✔️ edpm-ansible-molecule-edpm_ovs SUCCESS in 12m 08s
✔️ edpm-ansible-molecule-edpm_tripleo_cleanup SUCCESS in 4m 13s
✔️ edpm-ansible-molecule-edpm_tuned SUCCESS in 6m 14s
❌ edpm-ansible-molecule-edpm_telemetry_power_monitoring FAILURE in 6m 00s
✔️ edpm-ansible-molecule-edpm_update SUCCESS in 6m 16s

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/0f4464092e484de6b554d4ce133f154c

✔️ edpm-ansible-molecule-edpm_bootstrap SUCCESS in 7m 29s
✔️ edpm-ansible-molecule-edpm_podman SUCCESS in 6m 42s
✔️ edpm-ansible-molecule-edpm_module_load SUCCESS in 3m 50s
✔️ edpm-ansible-molecule-edpm_kernel SUCCESS in 7m 10s
✔️ edpm-ansible-molecule-edpm_libvirt SUCCESS in 9m 47s
✔️ edpm-ansible-molecule-edpm_nova SUCCESS in 11m 09s
✔️ edpm-ansible-molecule-edpm_frr SUCCESS in 7m 20s
✔️ edpm-ansible-molecule-edpm_iscsid SUCCESS in 5m 05s
✔️ edpm-ansible-molecule-edpm_ovn_bgp_agent SUCCESS in 8m 28s
✔️ edpm-ansible-molecule-edpm_ovs SUCCESS in 13m 44s
✔️ edpm-ansible-molecule-edpm_tripleo_cleanup SUCCESS in 4m 40s
✔️ edpm-ansible-molecule-edpm_tuned SUCCESS in 6m 11s
❌ edpm-ansible-molecule-edpm_telemetry_power_monitoring FAILURE in 6m 32s
✔️ edpm-ansible-molecule-edpm_update SUCCESS in 6m 13s

@openshift-ci openshift-ci bot added the lgtm label Jan 8, 2025
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Jan 8, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jlarriba, slagle

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

slagle
slagle reviewed Jan 8, 2025
View reviewed changes
Copy link
Contributor

@slagle slagle left a comment

Choose a reason for hiding this comment

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

actually, the molecule failure is related:

TASK [Create certs directory] ************************************************** Wednesday 08 January 2025 13:18:36 -0500 (0:00:02.932) 0:02:21.068 ***** fatal: [compute-1]: FAILED! => changed=false msg: 'There was an issue creating /var/lib/openstack/certs as requested: [Errno 13] Permission denied: b''/var/lib/openstack/certs''' path: /var/lib/openstack/certs/telemetry-power-monitoring/default

looks like become:true is needed on those tasks

@openshift-ci openshift-ci bot removed the lgtm label Jan 9, 2025
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/fc93d4c336e94162ad41275e2a91ad22

✔️ edpm-ansible-molecule-edpm_bootstrap SUCCESS in 6m 02s
✔️ edpm-ansible-molecule-edpm_podman SUCCESS in 4m 47s
✔️ edpm-ansible-molecule-edpm_module_load SUCCESS in 4m 37s
✔️ edpm-ansible-molecule-edpm_kernel SUCCESS in 10m 05s
✔️ edpm-ansible-molecule-edpm_libvirt SUCCESS in 9m 44s
✔️ edpm-ansible-molecule-edpm_nova SUCCESS in 10m 13s
✔️ edpm-ansible-molecule-edpm_frr SUCCESS in 7m 21s
✔️ edpm-ansible-molecule-edpm_iscsid SUCCESS in 4m 51s
✔️ edpm-ansible-molecule-edpm_ovn_bgp_agent SUCCESS in 8m 01s
✔️ edpm-ansible-molecule-edpm_ovs SUCCESS in 11m 57s
✔️ edpm-ansible-molecule-edpm_tripleo_cleanup SUCCESS in 4m 32s
✔️ edpm-ansible-molecule-edpm_tuned SUCCESS in 6m 13s
❌ edpm-ansible-molecule-edpm_telemetry_power_monitoring FAILURE in 6m 02s
✔️ edpm-ansible-molecule-edpm_update SUCCESS in 6m 38s

@jlarriba
Copy link
Contributor Author

jlarriba commented Jan 9, 2025

/test pre-commit-test

@jlarriba
Copy link
Contributor Author

jlarriba commented Jan 9, 2025

pre-commit-test refuses to run, it is not related with the change itself

@slagle
Copy link
Contributor

slagle commented Jan 9, 2025

recheck

@slagle
Copy link
Contributor

slagle commented Jan 9, 2025

/retest

@vyzigold
Copy link
Contributor

vyzigold commented Jan 9, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Jan 9, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 7d4ade2 into openstack-k8s-operators:main Jan 9, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@slagle slagle slagle approved these changes

@elfiesmelfie elfiesmelfie Awaiting requested review from elfiesmelfie

@paramite paramite Awaiting requested review from paramite

Assignees

@slagle slagle

@vyzigold vyzigold

Labels
approved lgtm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jlarriba @slagle @vyzigold