[ci] Add endpoint tests to the logging job#220
Merged
elfiesmelfie merged 3 commits intomasterfrom Feb 14, 2025
Merged
Conversation
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
elfiesmelfie
force-pushed
the
efoley_logging_add_endpoint_tests
branch
from
383bdc4 to
d758adb
Compare
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/b7132c84c89e44598b785246d3c10e53 ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 18m 29s |
ayefimov-1
approved these changes
Feb 11, 2025
Contributor
ayefimov-1
left a comment
ayefimov-1
left a comment
There was a problem hiding this comment.
Looks good. Thank you!!!
elfiesmelfie
commented
Feb 12, 2025
Comment on lines
+30
to
+45
| - [nova,compute,public] | ||
| - [nova,compute,internal] | ||
| - [placement,placement,public] | ||
| - [placement,placement,internal] | ||
| - [swift,object-store,public] | ||
| - [swift,object-store,internal] | ||
| - [cinderv3,volumev3,public] | ||
| - [cinderv3,volumev3,internal] | ||
| - [barbican,key-manager,public] | ||
| - [barbican,key-manager,internal] | ||
| - [keystone,identity,public] | ||
| - [keystone,identity,internal] | ||
| - [glance,image,public] | ||
| - [glance,image,internal] | ||
| - [neutron,network,public] | ||
| - [neutron,network,internal] |
Collaborator
Author
There was a problem hiding this comment.
Technically, we only need the 0th and second items in this list, but there's some new changes in progress in #203 that will address this
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/16a7f18ecbc44560a3031fe29d748fba ✔️ feature-verification-tests-noop SUCCESS in 4s |
Collaborator
Author
|
recheck |
vyzigold
approved these changes
Feb 14, 2025
The test checks for the various openstack endpoints that we expect to exist Co-authored-by: "Alex Yefimov <ayefimov@redhat.com>"
It contains the "openstack_cmd" var required for the endpoint tests
elfiesmelfie
force-pushed
the
efoley_logging_add_endpoint_tests
branch
from
6d900e2 to
47cd563
Compare
ayefimov-1
added a commit
that referenced
this pull request
Mar 6, 2025
and autoscaling jobs Add to irrelevant_files list for graphing, logging and autoscaling jobs Add metrics verification tasks for power monitoring components (#193) Move container image patch to post_ctlplane hook Make autoscaling job voting Add tasks to show queried metrics [ci][logging] Move file tests to the compute nodes (#206) [ci] logging test: fix typo 0->O (#205) [ci][autoscaling] Add a note to the vars file for autoscaling (#215) The note explains why os-net-config is skipped Rename tasks for XML reporting (#181) * [ansible-lint] Update the config to ignore templates in task names * [telemetry_logging] Update task names for XML reporting * [common] Update task names for XML reporting * [common] Update the name for the pod tests to include the status_string and namespace that the pod is running in Add files to the zuul jobs Add files to the zuul jobs uncomment zuul.yaml Add ceilometer_power_state to Metrics_verification test (#221) * Add test Ceilometer_power_state * Add the master_image to the zuul file * Add the master_image to the zuul file Add pod tests for Completed pods in the OpenStack namespace (#214) * [ci] Add logging tests: completed pods in openstack namespace * [ci] logging tests: update names for completed pods in openstack namespace * [ci] logging: update typo on pod name * [ci] Update pod list for completed status in openstack namespace This repo-setup-edpm-deployment-openstack-edpm-ipam doesn't need to be checked. --------- Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci][logging] Add tests for services in the openstack namespace (#225) Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci] Add endpoint tests to the logging job (#220) * [ci] Add endpoint tests to the logging job The test checks for the various openstack endpoints that we expect to exist Co-authored-by: "Alex Yefimov <ayefimov@redhat.com>" * [ci] Add vars/osp18_env to the logging playbook It contains the "openstack_cmd" var required for the endpoint tests * [common] Add documentation and default for openstack_cmd Fix kepler metrics tests (#224) Kepler metrics tests were added in [1] though the merged commit was not the correct one. This PR aplies changes from [2] that had more correct code of the tests. Fixes OSPRH-14029 [1] #193 [2] 0b5d856 Co-authored-by: Jaromír Wysoglad <jwysogla@redhat.com> Change required projects The projects should contain "github.com/" prefix. Fix rabbitmq test [zuul][graphing] Add the new dashboards to the test (#222) * Add the new dashboards to zuul job Update .zuul.yaml [ci] Add manifest tests to logging job (#228) Co-authored-by: Alex Yefimov <ayefimov@redhat.com> [telemetry_verify_metrics]: add openstack-network-exporter feature tests (#231) * feature_tests: add openstack-network-exporter feature tests Add tests for operator of openstack-network-exporter Signed-off-by: Aaron T. Smith <aasmith@rehdat.com> Co-authored-by: Aaron T. Smith <aasmith@redhat.com> Co-authored-by: Jaromír Wysoglad <smulumas@gmail.com> Add to irrelevant_files list for graphing, logging and autoscaling jobs
ayefimov-1
added a commit
that referenced
this pull request
Mar 6, 2025
and autoscaling jobs Add to irrelevant_files list for graphing, logging and autoscaling jobs Add metrics verification tasks for power monitoring components (#193) Move container image patch to post_ctlplane hook Make autoscaling job voting Add tasks to show queried metrics [ci][logging] Move file tests to the compute nodes (#206) [ci] logging test: fix typo 0->O (#205) [ci][autoscaling] Add a note to the vars file for autoscaling (#215) The note explains why os-net-config is skipped Rename tasks for XML reporting (#181) * [ansible-lint] Update the config to ignore templates in task names * [telemetry_logging] Update task names for XML reporting * [common] Update task names for XML reporting * [common] Update the name for the pod tests to include the status_string and namespace that the pod is running in Add files to the zuul jobs Add files to the zuul jobs uncomment zuul.yaml Add ceilometer_power_state to Metrics_verification test (#221) * Add test Ceilometer_power_state * Add the master_image to the zuul file * Add the master_image to the zuul file Add pod tests for Completed pods in the OpenStack namespace (#214) * [ci] Add logging tests: completed pods in openstack namespace * [ci] logging tests: update names for completed pods in openstack namespace * [ci] logging: update typo on pod name * [ci] Update pod list for completed status in openstack namespace This repo-setup-edpm-deployment-openstack-edpm-ipam doesn't need to be checked. --------- Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci][logging] Add tests for services in the openstack namespace (#225) Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci] Add endpoint tests to the logging job (#220) * [ci] Add endpoint tests to the logging job The test checks for the various openstack endpoints that we expect to exist Co-authored-by: "Alex Yefimov <ayefimov@redhat.com>" * [ci] Add vars/osp18_env to the logging playbook It contains the "openstack_cmd" var required for the endpoint tests * [common] Add documentation and default for openstack_cmd Fix kepler metrics tests (#224) Kepler metrics tests were added in [1] though the merged commit was not the correct one. This PR aplies changes from [2] that had more correct code of the tests. Fixes OSPRH-14029 [1] #193 [2] 0b5d856 Co-authored-by: Jaromír Wysoglad <jwysogla@redhat.com> Change required projects The projects should contain "github.com/" prefix. Fix rabbitmq test [zuul][graphing] Add the new dashboards to the test (#222) * Add the new dashboards to zuul job Update .zuul.yaml [ci] Add manifest tests to logging job (#228) Co-authored-by: Alex Yefimov <ayefimov@redhat.com> [telemetry_verify_metrics]: add openstack-network-exporter feature tests (#231) * feature_tests: add openstack-network-exporter feature tests Add tests for operator of openstack-network-exporter Signed-off-by: Aaron T. Smith <aasmith@rehdat.com> Co-authored-by: Aaron T. Smith <aasmith@redhat.com> Co-authored-by: Jaromír Wysoglad <smulumas@gmail.com> Add to irrelevant_files list for graphing, logging and autoscaling jobs
ayefimov-1
added a commit
that referenced
this pull request
Mar 10, 2025
and autoscaling jobs Add to irrelevant_files list for graphing, logging and autoscaling jobs Add metrics verification tasks for power monitoring components (#193) Move container image patch to post_ctlplane hook Make autoscaling job voting Add tasks to show queried metrics [ci][logging] Move file tests to the compute nodes (#206) [ci] logging test: fix typo 0->O (#205) [ci][autoscaling] Add a note to the vars file for autoscaling (#215) The note explains why os-net-config is skipped Rename tasks for XML reporting (#181) * [ansible-lint] Update the config to ignore templates in task names * [telemetry_logging] Update task names for XML reporting * [common] Update task names for XML reporting * [common] Update the name for the pod tests to include the status_string and namespace that the pod is running in Add files to the zuul jobs Add files to the zuul jobs uncomment zuul.yaml Add ceilometer_power_state to Metrics_verification test (#221) * Add test Ceilometer_power_state * Add the master_image to the zuul file * Add the master_image to the zuul file Add pod tests for Completed pods in the OpenStack namespace (#214) * [ci] Add logging tests: completed pods in openstack namespace * [ci] logging tests: update names for completed pods in openstack namespace * [ci] logging: update typo on pod name * [ci] Update pod list for completed status in openstack namespace This repo-setup-edpm-deployment-openstack-edpm-ipam doesn't need to be checked. --------- Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci][logging] Add tests for services in the openstack namespace (#225) Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci] Add endpoint tests to the logging job (#220) * [ci] Add endpoint tests to the logging job The test checks for the various openstack endpoints that we expect to exist Co-authored-by: "Alex Yefimov <ayefimov@redhat.com>" * [ci] Add vars/osp18_env to the logging playbook It contains the "openstack_cmd" var required for the endpoint tests * [common] Add documentation and default for openstack_cmd Fix kepler metrics tests (#224) Kepler metrics tests were added in [1] though the merged commit was not the correct one. This PR aplies changes from [2] that had more correct code of the tests. Fixes OSPRH-14029 [1] #193 [2] 0b5d856 Co-authored-by: Jaromír Wysoglad <jwysogla@redhat.com> Change required projects The projects should contain "github.com/" prefix. Fix rabbitmq test [zuul][graphing] Add the new dashboards to the test (#222) * Add the new dashboards to zuul job Update .zuul.yaml [ci] Add manifest tests to logging job (#228) Co-authored-by: Alex Yefimov <ayefimov@redhat.com> [telemetry_verify_metrics]: add openstack-network-exporter feature tests (#231) * feature_tests: add openstack-network-exporter feature tests Add tests for operator of openstack-network-exporter Signed-off-by: Aaron T. Smith <aasmith@rehdat.com> Co-authored-by: Aaron T. Smith <aasmith@redhat.com> Co-authored-by: Jaromír Wysoglad <smulumas@gmail.com> Add to irrelevant_files list for graphing, logging and autoscaling jobs
ayefimov-1
pushed a commit
that referenced
this pull request
Mar 10, 2025
* [ansible-lint] Update the config to ignore templates in task names * [telemetry_logging] Update task names for XML reporting * [common] Update task names for XML reporting * [common] Update the name for the pod tests to include the status_string and namespace that the pod is running in Add ceilometer_power_state to Metrics_verification test (#221) * Add test Ceilometer_power_state * Add the master_image to the zuul file * Add the master_image to the zuul file Add pod tests for Completed pods in the OpenStack namespace (#214) * [ci] Add logging tests: completed pods in openstack namespace * [ci] logging tests: update names for completed pods in openstack namespace * [ci] logging: update typo on pod name * [ci] Update pod list for completed status in openstack namespace This repo-setup-edpm-deployment-openstack-edpm-ipam doesn't need to be checked. --------- Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci][logging] Add tests for services in the openstack namespace (#225) Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci] Add endpoint tests to the logging job (#220) * [ci] Add endpoint tests to the logging job The test checks for the various openstack endpoints that we expect to exist Co-authored-by: "Alex Yefimov <ayefimov@redhat.com>" * [ci] Add vars/osp18_env to the logging playbook It contains the "openstack_cmd" var required for the endpoint tests * [common] Add documentation and default for openstack_cmd Fix kepler metrics tests (#224) Kepler metrics tests were added in [1] though the merged commit was not the correct one. This PR aplies changes from [2] that had more correct code of the tests. Fixes OSPRH-14029 [1] #193 [2] 0b5d856 Co-authored-by: Jaromír Wysoglad <jwysogla@redhat.com> Change required projects The projects should contain "github.com/" prefix. Fix rabbitmq test [zuul][graphing] Add the new dashboards to the test (#222) * Add the new dashboards to zuul job [ci] Add manifest tests to logging job (#228) Co-authored-by: Alex Yefimov <ayefimov@redhat.com> [telemetry_verify_metrics]: add openstack-network-exporter feature tests (#231) * feature_tests: add openstack-network-exporter feature tests Add tests for operator of openstack-network-exporter Signed-off-by: Aaron T. Smith <aasmith@rehdat.com> Co-authored-by: Aaron T. Smith <aasmith@redhat.com> Co-authored-by: Jaromír Wysoglad <smulumas@gmail.com> Add to irrelevant_files list for graphing, logging and autoscaling jobs Add to irrelevant_files list for graphing, logging and autoscaling jobs Add metrics verification tasks for power monitoring components (#193) Move container image patch to post_ctlplane hook Make autoscaling job voting Add tasks to show queried metrics [ci][logging] Move file tests to the compute nodes (#206) [ci] logging test: fix typo 0->O (#205) [ci][autoscaling] Add a note to the vars file for autoscaling (#215) The note explains why os-net-config is skipped Rename tasks for XML reporting (#181) * [ansible-lint] Update the config to ignore templates in task names * [telemetry_logging] Update task names for XML reporting * [common] Update task names for XML reporting * [common] Update the name for the pod tests to include the status_string and namespace that the pod is running in Add files to the zuul jobs Add files to the zuul jobs uncomment zuul.yaml Add ceilometer_power_state to Metrics_verification test (#221) * Add test Ceilometer_power_state * Add the master_image to the zuul file * Add the master_image to the zuul file Add pod tests for Completed pods in the OpenStack namespace (#214) * [ci] Add logging tests: completed pods in openstack namespace * [ci] logging tests: update names for completed pods in openstack namespace * [ci] logging: update typo on pod name * [ci] Update pod list for completed status in openstack namespace This repo-setup-edpm-deployment-openstack-edpm-ipam doesn't need to be checked. --------- Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci][logging] Add tests for services in the openstack namespace (#225) Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci] Add endpoint tests to the logging job (#220) * [ci] Add endpoint tests to the logging job The test checks for the various openstack endpoints that we expect to exist Co-authored-by: "Alex Yefimov <ayefimov@redhat.com>" * [ci] Add vars/osp18_env to the logging playbook It contains the "openstack_cmd" var required for the endpoint tests * [common] Add documentation and default for openstack_cmd Fix kepler metrics tests (#224) Kepler metrics tests were added in [1] though the merged commit was not the correct one. This PR aplies changes from [2] that had more correct code of the tests. Fixes OSPRH-14029 [1] #193 [2] 0b5d856 Co-authored-by: Jaromír Wysoglad <jwysogla@redhat.com> Change required projects The projects should contain "github.com/" prefix. Fix rabbitmq test [zuul][graphing] Add the new dashboards to the test (#222) * Add the new dashboards to zuul job Update .zuul.yaml [ci] Add manifest tests to logging job (#228) Co-authored-by: Alex Yefimov <ayefimov@redhat.com> [telemetry_verify_metrics]: add openstack-network-exporter feature tests (#231) * feature_tests: add openstack-network-exporter feature tests Add tests for operator of openstack-network-exporter Signed-off-by: Aaron T. Smith <aasmith@rehdat.com> Co-authored-by: Aaron T. Smith <aasmith@redhat.com> Co-authored-by: Jaromír Wysoglad <smulumas@gmail.com> Add to irrelevant_files list for graphing, logging and autoscaling jobs
ayefimov-1
added a commit
that referenced
this pull request
Mar 12, 2025
* Rename tasks for XML reporting (#181) * [ansible-lint] Update the config to ignore templates in task names * [telemetry_logging] Update task names for XML reporting * [common] Update task names for XML reporting * [common] Update the name for the pod tests to include the status_string and namespace that the pod is running in Add ceilometer_power_state to Metrics_verification test (#221) * Add test Ceilometer_power_state * Add the master_image to the zuul file * Add the master_image to the zuul file Add pod tests for Completed pods in the OpenStack namespace (#214) * [ci] Add logging tests: completed pods in openstack namespace * [ci] logging tests: update names for completed pods in openstack namespace * [ci] logging: update typo on pod name * [ci] Update pod list for completed status in openstack namespace This repo-setup-edpm-deployment-openstack-edpm-ipam doesn't need to be checked. --------- Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci][logging] Add tests for services in the openstack namespace (#225) Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci] Add endpoint tests to the logging job (#220) * [ci] Add endpoint tests to the logging job The test checks for the various openstack endpoints that we expect to exist Co-authored-by: "Alex Yefimov <ayefimov@redhat.com>" * [ci] Add vars/osp18_env to the logging playbook It contains the "openstack_cmd" var required for the endpoint tests * [common] Add documentation and default for openstack_cmd Fix kepler metrics tests (#224) Kepler metrics tests were added in [1] though the merged commit was not the correct one. This PR aplies changes from [2] that had more correct code of the tests. Fixes OSPRH-14029 [1] #193 [2] 0b5d856 Co-authored-by: Jaromír Wysoglad <jwysogla@redhat.com> Change required projects The projects should contain "github.com/" prefix. Fix rabbitmq test [zuul][graphing] Add the new dashboards to the test (#222) * Add the new dashboards to zuul job [ci] Add manifest tests to logging job (#228) Co-authored-by: Alex Yefimov <ayefimov@redhat.com> [telemetry_verify_metrics]: add openstack-network-exporter feature tests (#231) * feature_tests: add openstack-network-exporter feature tests Add tests for operator of openstack-network-exporter Signed-off-by: Aaron T. Smith <aasmith@rehdat.com> Co-authored-by: Aaron T. Smith <aasmith@redhat.com> Co-authored-by: Jaromír Wysoglad <smulumas@gmail.com> Add to irrelevant_files list for graphing, logging and autoscaling jobs Add to irrelevant_files list for graphing, logging and autoscaling jobs Add metrics verification tasks for power monitoring components (#193) Move container image patch to post_ctlplane hook Make autoscaling job voting Add tasks to show queried metrics [ci][logging] Move file tests to the compute nodes (#206) [ci] logging test: fix typo 0->O (#205) [ci][autoscaling] Add a note to the vars file for autoscaling (#215) The note explains why os-net-config is skipped Rename tasks for XML reporting (#181) * [ansible-lint] Update the config to ignore templates in task names * [telemetry_logging] Update task names for XML reporting * [common] Update task names for XML reporting * [common] Update the name for the pod tests to include the status_string and namespace that the pod is running in Add files to the zuul jobs Add files to the zuul jobs uncomment zuul.yaml Add ceilometer_power_state to Metrics_verification test (#221) * Add test Ceilometer_power_state * Add the master_image to the zuul file * Add the master_image to the zuul file Add pod tests for Completed pods in the OpenStack namespace (#214) * [ci] Add logging tests: completed pods in openstack namespace * [ci] logging tests: update names for completed pods in openstack namespace * [ci] logging: update typo on pod name * [ci] Update pod list for completed status in openstack namespace This repo-setup-edpm-deployment-openstack-edpm-ipam doesn't need to be checked. --------- Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci][logging] Add tests for services in the openstack namespace (#225) Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com> [ci] Add endpoint tests to the logging job (#220) * [ci] Add endpoint tests to the logging job The test checks for the various openstack endpoints that we expect to exist Co-authored-by: "Alex Yefimov <ayefimov@redhat.com>" * [ci] Add vars/osp18_env to the logging playbook It contains the "openstack_cmd" var required for the endpoint tests * [common] Add documentation and default for openstack_cmd Fix kepler metrics tests (#224) Kepler metrics tests were added in [1] though the merged commit was not the correct one. This PR aplies changes from [2] that had more correct code of the tests. Fixes OSPRH-14029 [1] #193 [2] 0b5d856 Co-authored-by: Jaromír Wysoglad <jwysogla@redhat.com> Change required projects The projects should contain "github.com/" prefix. Fix rabbitmq test [zuul][graphing] Add the new dashboards to the test (#222) * Add the new dashboards to zuul job Update .zuul.yaml [ci] Add manifest tests to logging job (#228) Co-authored-by: Alex Yefimov <ayefimov@redhat.com> [telemetry_verify_metrics]: add openstack-network-exporter feature tests (#231) * feature_tests: add openstack-network-exporter feature tests Add tests for operator of openstack-network-exporter Signed-off-by: Aaron T. Smith <aasmith@rehdat.com> Co-authored-by: Aaron T. Smith <aasmith@redhat.com> Co-authored-by: Jaromír Wysoglad <smulumas@gmail.com> Add to irrelevant_files list for graphing, logging and autoscaling jobs * Update .zuul.yaml * Add test for health monitoring feature (#234) (#239) * Add test for health monitoring feature * Disable RBAC for KSM metrics Co-authored-by: Martin Mágr <martin.magr@gmail.com> --------- Co-authored-by: Emma Foley <elfiesmelfie@users.noreply.github.com> Co-authored-by: Martin Mágr <martin.magr@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The test checks for the various openstack endpoints that we expect to exist
Depends-On: #181