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

designate: test new configuration options #457

Merged
merged 1 commit into from Aug 4, 2021

Conversation

freyes
Copy link
Member

@freyes freyes commented Nov 11, 2020

New configuration options:

  • default-ttl
  • default-soa-minimum
  • default-soa-refresh-min
  • default-soa-refresh-max
  • default-soa-retry

Related-Bug: #1902922

@codecov-io
Copy link

codecov-io commented Nov 12, 2020

Codecov Report

Merging #457 (cf5fbfa) into master (bb95752) will increase coverage by 0.49%.
The diff coverage is 0.00%.

❗ Current head cf5fbfa differs from pull request most recent head bcff352. Consider uploading reports for the commit bcff352 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #457      +/-   ##
==========================================
+ Coverage   17.79%   18.28%   +0.49%     
==========================================
  Files         165      160       -5     
  Lines       10539     9359    -1180     
==========================================
- Hits         1875     1711     -164     
+ Misses       8664     7648    -1016     
Impacted Files Coverage Δ
zaza/openstack/charm_tests/designate/tests.py 0.00% <0.00%> (ø)
zaza/openstack/charm_tests/test_utils.py 39.17% <ø> (+7.57%) ⬆️
zaza/openstack/utilities/openstack.py 57.22% <0.00%> (-3.78%) ⬇️
zaza/openstack/utilities/__init__.py 100.00% <0.00%> (ø)
zaza/openstack/charm_tests/ovn/tests.py 0.00% <0.00%> (ø)
zaza/openstack/utilities/os_versions.py 100.00% <0.00%> (ø)
zaza/openstack/charm_tests/aodh/tests.py 0.00% <0.00%> (ø)
zaza/openstack/charm_tests/nova/tests.py 0.00% <0.00%> (ø)
zaza/openstack/charm_tests/mysql/tests.py 0.00% <0.00%> (ø)
zaza/openstack/charm_tests/vault/setup.py 0.00% <0.00%> (ø)
... and 32 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb95752...bcff352. Read the comment docs.

@freyes freyes force-pushed the bug/1902923 branch 2 times, most recently from 7292ad7 to cf5fbfa Compare January 14, 2021 19:28
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.

LGTM in general, thanks! Just a few typos

@@ -33,6 +33,8 @@
class BaseDesignateTest(test_utils.OpenStackBaseTest):
"""Base for Designate charm tests."""

DESIGNAME_CONF = '/etc/designate/designate.conf'
Copy link
Contributor

Choose a reason for hiding this comment

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

s/DESIGNAME/DESIGNATE/


logging.debug('Tidy up delete test record %s' % domain_id)
self._wait_on_domain_gone(domain_id)
logging.debug('Donde with deletion of domain %s' % domain_id)
Copy link
Contributor

Choose a reason for hiding this comment

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

s/Donde/Done/

New configuration options:

- default-ttl
- default-soa-minimum
- default-soa-refresh-min
- default-soa-refresh-max
- default-soa-retry

Related-Bug: #1902922
@lourot
Copy link
Contributor

lourot commented Jul 15, 2021

@freyes Can you please not force-push once someone has started reviewing? Now I can't tell what exactly you have changed between cf5fbfa and bcff352. Thanks

@lourot lourot merged commit 6a4f3d4 into openstack-charmers:master Aug 4, 2021
openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request Aug 4, 2021
* Update charm-designate from branch 'master'
  to fa48d3d7cf5fafd157b3d3f1570c63660f4a59dc
  - Merge "Add config options to handle SOA records."
  - Add config options to handle SOA records.
    
    New configuration options:
    
    - default-ttl
    - default-soa-minimum
    - default-soa-refresh-min
    - default-soa-refresh-max
    - default-soa-retry
    
    Change-Id: I2ad0aec7d50a36e9e5227a08295aef4e99130d78
    Closes-Bug: #1902923
    Func-Test-Pr: openstack-charmers/zaza-openstack-tests#457
openstack-mirroring pushed a commit to openstack/charm-designate that referenced this pull request Aug 4, 2021
New configuration options:

- default-ttl
- default-soa-minimum
- default-soa-refresh-min
- default-soa-refresh-max
- default-soa-retry

Change-Id: I2ad0aec7d50a36e9e5227a08295aef4e99130d78
Closes-Bug: #1902923
Func-Test-Pr: openstack-charmers/zaza-openstack-tests#457
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

3 participants