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

Add COS integration testing for ceph-mon #1254

Merged
merged 1 commit into from
Aug 19, 2024
Merged

Add COS integration testing for ceph-mon #1254

merged 1 commit into from
Aug 19, 2024

Conversation

sabaini
Copy link
Contributor

@sabaini sabaini commented Aug 12, 2024

Test retrieval of metrics, alert rules and dashboards

@sabaini
Add COS integration testing for ceph-mon
ab347e1 
Test retrieval of metrics, alert rules and dashboards

Signed-off-by: Peter Sabaini <peter.sabaini@canonical.com>
@sabaini
Copy link
Contributor Author

sabaini commented Aug 12, 2024 •
edited
Loading

https://review.opendev.org/c/openstack/charm-ceph-mon/+/926155

Edit: now in https://review.opendev.org/c/openstack/charm-ceph-mon/+/926204

openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request Aug 13, 2024
@sabaini @openstack-gerrit
Update git submodules
918168e 
* Update charm-ceph-mon from branch 'master'
  to 9f6d98c308d731f689b6d58e73fd1fc18a13718e
  - Fix unit tests for py312
    
    Needs liblzma-dev header files for installing Python3.12 from
    source.
    
    Also test functest update
    
    func-test-pr: openstack-charmers/zaza-openstack-tests#1254
    Change-Id: I13b8f95bd4f7360204a42991c276951e3c0c44db
openstack-mirroring pushed a commit to openstack/charm-ceph-mon that referenced this pull request Aug 13, 2024
@sabaini
Fix unit tests for py312
9f6d98c 
Needs liblzma-dev header files for installing Python3.12 from
source.

Also test functest update

func-test-pr: openstack-charmers/zaza-openstack-tests#1254
Change-Id: I13b8f95bd4f7360204a42991c276951e3c0c44db
lmlg
lmlg approved these changes Aug 14, 2024
View reviewed changes
Copy link
Contributor

@lmlg lmlg left a comment

Choose a reason for hiding this comment

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

lgtm, cheers!

@sabaini sabaini merged commit a08ef84 into openstack-charmers:master Aug 19, 2024
3 checks passed
@nobuto-m
Copy link
Contributor

@sabaini In https://review.opendev.org/c/openstack/charm-ceph-mon/+/926204, I don't see any output related to COSIntegrationTest. Am I missing something?

$ grep 'Running Test zaza' job-output.txt 
2024-08-13 16:28:47.495649 | focal-medium | 2024-08-13 16:28:47 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CheckPoolTypes ##
2024-08-13 16:28:49.846339 | focal-medium | 2024-08-13 16:28:49 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CephLowLevelTest ##
2024-08-13 16:29:11.498140 | focal-medium | 2024-08-13 16:29:11 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CephTest ##
2024-08-13 16:30:24.156484 | focal-medium | 2024-08-13 16:30:24 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.osd.tests.SecurityTest ##
2024-08-13 16:30:25.477339 | focal-medium | 2024-08-13 16:30:25 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CephPrometheusTest ##
2024-08-13 16:30:26.609841 | focal-medium | 2024-08-13 16:30:26 [INFO] ## Running Test zaza.charm_tests.lifecycle.tests.UpgradeCharmsToPath;ceph-mon ##
2024-08-13 16:30:32.776860 | focal-medium | 2024-08-13 16:30:32 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CephMonJujuPersistent ##
2024-08-13 16:31:44.873439 | focal-medium | 2024-08-13 16:31:44 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CheckPoolTypes ##
2024-08-13 16:31:46.129491 | focal-medium | 2024-08-13 16:31:46 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CephLowLevelTest ##
2024-08-13 16:32:07.839153 | focal-medium | 2024-08-13 16:32:07 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CephTest ##
2024-08-13 16:33:18.514145 | focal-medium | 2024-08-13 16:33:18 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.osd.tests.SecurityTest ##
2024-08-13 16:33:19.840755 | focal-medium | 2024-08-13 16:33:19 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CephPrometheusTest ##
2024-08-13 16:33:21.012524 | focal-medium | 2024-08-13 16:33:21 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CephAuthTest ##
2024-08-13 16:33:32.803361 | focal-medium | 2024-08-13 16:33:32 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CephMonActionsTest ##
2024-08-13 16:33:57.210691 | focal-medium | 2024-08-13 16:33:57 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.mon.tests.CephPermissionUpgradeTest ##
2024-08-13 16:35:25.463498 | focal-medium | 2024-08-13 16:35:25 [INFO] ## Running Test zaza.openstack.charm_tests.ceph.tests.CephMonKeyRotationTests ##

@sabaini
Copy link
Contributor Author

sabaini commented Aug 19, 2024

These tests are only being run if there's grafana-agent present. Due to resource constraints we don't plan to run them in the regular CI but will add a COS model + grafana-agent into the SQA tests

lmlg pushed a commit to canonical/ceph-charms that referenced this pull request Jan 22, 2025
@sabaini
Fix unit tests for py312
093380f 
Needs liblzma-dev header files for installing Python3.12 from
source.

Also test functest update

func-test-pr: openstack-charmers/zaza-openstack-tests#1254
Change-Id: I13b8f95bd4f7360204a42991c276951e3c0c44db
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lmlg lmlg lmlg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sabaini @nobuto-m @lmlg