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

Add NRPE check tests for Designate #436

Merged
merged 2 commits into from
Aug 26, 2021

Conversation

xavpaice
Copy link
Contributor

@xavpaice xavpaice commented Oct 2, 2020

Adds a simple test for the Designate application to ensure that the NRPE
checks for services have been created.

Related-Bug: #1897809

Adds a simple test for the Designate application to ensure that the NRPE
checks for services have been created.

Related-Bug: #1897809
@xavpaice xavpaice marked this pull request as draft October 2, 2020 05:21
@xavpaice xavpaice marked this pull request as ready for review October 2, 2020 22:22
Change-Id: Ib36cf72fe20aa2b46a80f2d71bcd1400fd70fb8e
@lourot
Copy link
Contributor

lourot commented Jun 25, 2021

Gerrit review making use of it: https://review.opendev.org/c/openstack/charm-designate/+/755691

Copy link
Contributor

@lourot lourot left a comment

Choose a reason for hiding this comment

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

Happy to land this together with https://review.opendev.org/c/openstack/charm-designate/+/755691 , thanks!

@lourot lourot merged commit 15add17 into openstack-charmers:master Aug 26, 2021
openstack-mirroring pushed a commit to openstack/charm-designate that referenced this pull request Aug 26, 2021
Adds the nrpe-external-master layer and checks for services managed by
this charm.

func-test-pr: openstack-charmers/zaza-openstack-tests#436

Change-Id: Ice5eada838626059f8cb2c427f7b9b0884317660
Closes-Bug: #1897809
Related-Bug: #1933643
Co-authored-by: Aurelien Lourot <aurelien.lourot@canonical.com>
openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request Aug 26, 2021
* Update charm-designate from branch 'master'
  to 3ae0be599b4433919e3db6526a26070fea536db5
  - Add NRPE checks for services
    
    Adds the nrpe-external-master layer and checks for services managed by
    this charm.
    
    func-test-pr: openstack-charmers/zaza-openstack-tests#436
    
    Change-Id: Ice5eada838626059f8cb2c427f7b9b0884317660
    Closes-Bug: #1897809
    Related-Bug: #1933643
    Co-authored-by: Aurelien Lourot <aurelien.lourot@canonical.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants