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

Enable watcher-dashboard panels if ENABLE_WATCHER is yes #242

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

amoralej
Copy link
Contributor

@amoralej amoralej commented Jan 24, 2025

Following the same pattern that other plugins, this patch enable the pannels provided in the wathcher-dashboard if environmental value is set to yes.

To make it backwards compatible it is setting it as no by default and checks if the dashboard is installed before trying to enable it.

Related: OSPRH-11277

@openshift-ci openshift-ci bot requested review from bshephar and lewisdenny January 24, 2025 13:08
@raukadah
Copy link
Collaborator

/approve

@amoralej
Copy link
Contributor Author

/test precommit-check

@amoralej
Copy link
Contributor Author

/test images

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/008169655f424b4eaa0ff3e5be121493

✔️ openstack-meta-content-provider SUCCESS in 3h 57m 56s
tcib-crc-podified-edpm-baremetal FAILURE in 2h 57m 41s
tcib-podified-multinode-edpm-deployment-crc FAILURE in 2h 26m 13s

@raukadah
Copy link
Collaborator

https://logserver.rdoproject.org/42/242/90cd66b9cfed9d9ed382ffbcc19d443981c8ff30/github-check/tcib-podified-multinode-edpm-deployment-crc/243f72b/controller/ci-framework-data/tests/test_operator/tempest-tests-tempest/stestr_results.html test network basic ops failure. Not related this pr.

}}}

Traceback (most recent call last):
  File "/usr/lib/python3.9/site-packages/tempest/common/utils/__init__.py", line 65, in wrapper
    return f(*func_args, **func_kwargs)
  File "/usr/lib/python3.9/site-packages/tempest/scenario/test_network_basic_ops.py", line 568, in test_update_router_admin_state
    self._check_public_network_connectivity(
  File "/usr/lib/python3.9/site-packages/tempest/scenario/test_network_basic_ops.py", line 211, in _check_public_network_connectivity
    self.check_vm_connectivity(
  File "/usr/lib/python3.9/site-packages/tempest/scenario/manager.py", line 1082, in check_vm_connectivity
    self.assertTrue(self.ping_ip_address(ip_address,
  File "/usr/lib64/python3.9/unittest/case.py", line 688, in assertTrue
    raise self.failureException(msg)
AssertionError: False is not true : Public network connectivity check failed. Reason: after updating admin_state_up of router to True
Timed out waiting for 192.168.122.199 to become reachable

@raukadah
Copy link
Collaborator

recheck

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/1cf82b8c2546494499b167ba7dfb6b56

✔️ openstack-meta-content-provider SUCCESS in 4h 02m 54s
tcib-crc-podified-edpm-baremetal FAILURE in 3h 06m 56s
tcib-podified-multinode-edpm-deployment-crc FAILURE in 2h 15m 23s

@raukadah
Copy link
Collaborator

recheck same tempest failure vanished after recheck

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/2c7f88efd7a54aedbb4ea435eb77346f

✔️ openstack-meta-content-provider SUCCESS in 3h 55m 33s
tcib-crc-podified-edpm-baremetal FAILURE in 3h 02m 42s
tcib-podified-multinode-edpm-deployment-crc FAILURE in 2h 32m 06s

@amoralej
Copy link
Contributor Author

recheck

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/2ac066c8b38f4cc7b423d8c4a89e8d3f

✔️ openstack-meta-content-provider SUCCESS in 3h 13m 57s
tcib-crc-podified-edpm-baremetal RETRY_LIMIT in 13m 26s
tcib-podified-multinode-edpm-deployment-crc FAILURE in 1h 56m 48s

@amoralej
Copy link
Contributor Author

recheck

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/a5e8e127c76b4384b8be20cacd2f32c2

✔️ openstack-meta-content-provider SUCCESS in 4h 08m 25s
✔️ tcib-crc-podified-edpm-baremetal SUCCESS in 2h 14m 59s
tcib-podified-multinode-edpm-deployment-crc FAILURE in 1h 24m 27s

@amoralej
Copy link
Contributor Author

recheck
seems flaky tests

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/6efc32a7e4d34913b5bce0d350b10918

✔️ openstack-meta-content-provider SUCCESS in 3h 02m 24s
tcib-crc-podified-edpm-baremetal FAILURE in 2h 01m 20s
tcib-podified-multinode-edpm-deployment-crc FAILURE in 1h 23m 11s

@eduolivares
Copy link
Contributor

recheck
Depends-On added to the PR description should fix the tobiko failures.

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/ba8643650774455b910f1dfc5541fd1d

openstack-meta-content-provider FAILURE in 50m 10s
⚠️ tcib-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ tcib-podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-meta-content-provider

@amoralej
Copy link
Contributor Author

recheck

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/33bab90cb40e45bf9270b219ebe01a0c

✔️ openstack-meta-content-provider SUCCESS in 3h 14m 05s
tcib-crc-podified-edpm-baremetal FAILURE in 2h 06m 01s
tcib-podified-multinode-edpm-deployment-crc FAILURE in 1h 30m 37s

@rabi
Copy link
Collaborator

rabi commented Jan 31, 2025

I've propsed to remove them #245. Let's discuss before we introduce these again as I'm not sure if we should run them here.

Following the same pattern that other plugins, this patch enable the
pannels provided in the wathcher-dashboard if environmental value is set
to `yes`.

To make it backwards compatible it is setting it as no by default and
checks if the dashboard is installed before trying to enable it.
Copy link
Collaborator

@rabi rabi left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Feb 4, 2025
Copy link
Contributor

openshift-ci bot commented Feb 4, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: amoralej, deshipu, rabi, raukadah

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

@openshift-merge-bot openshift-merge-bot bot merged commit 01ff037 into openstack-k8s-operators:main Feb 4, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants