-
Notifications
You must be signed in to change notification settings - Fork 53
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
Enable watcher-dashboard panels if ENABLE_WATCHER is yes #242
Conversation
/approve |
/test precommit-check |
/test images |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/008169655f424b4eaa0ff3e5be121493 ✔️ openstack-meta-content-provider SUCCESS in 3h 57m 56s |
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.
|
recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/1cf82b8c2546494499b167ba7dfb6b56 ✔️ openstack-meta-content-provider SUCCESS in 4h 02m 54s |
recheck same tempest failure vanished after recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/2c7f88efd7a54aedbb4ea435eb77346f ✔️ openstack-meta-content-provider SUCCESS in 3h 55m 33s |
recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/2ac066c8b38f4cc7b423d8c4a89e8d3f ✔️ openstack-meta-content-provider SUCCESS in 3h 13m 57s |
recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a5e8e127c76b4384b8be20cacd2f32c2 ✔️ openstack-meta-content-provider SUCCESS in 4h 08m 25s |
recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/6efc32a7e4d34913b5bce0d350b10918 ✔️ openstack-meta-content-provider SUCCESS in 3h 02m 24s |
recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/ba8643650774455b910f1dfc5541fd1d ❌ openstack-meta-content-provider FAILURE in 50m 10s |
recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/33bab90cb40e45bf9270b219ebe01a0c ✔️ openstack-meta-content-provider SUCCESS in 3h 14m 05s |
I have reviewed the tobiko logs and there is something wrong. It looks like a bug in test-operator. I will investigate tomorrow. At [1] environment is empty (wrong). And there are some other issues. I think we should disable tobiko tests to avoid blocking tcib, for the moment. |
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[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 |
01ff037
into
openstack-k8s-operators:main
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