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

[Backport][ipa-4-9] dnszone-add: deprecate option for setting SOA serial #5949

Closed

Conversation

antoniotorresm
Copy link
Contributor

This PR was opened automatically because PR #5841 was pushed to master and backport to ipa-4-9 is required.

Since IPA 3 [1] SOA serial is managed automatically via autoincrement,
and the option of disabling this behavior was deprecated in IPA 3.3.3 [2].
As a result, the option '--serial' during DNS zone addition would be
ignored as it is set during the creation. This commit adds a deprecation
warning if this option is used.

[1]: https://www.freeipa.org/page/V3/DNS_SOA_serial_auto-incrementation
[2]: https://www.freeipa.org/page/Releases/3.3.3

Fixes: https://pagure.io/freeipa/issue/8227
Signed-off-by: Antonio Torres <antorres@redhat.com>
Tests must be updated to expect the new deprecation warning.

Related: https://pagure.io/freeipa/issue/8227
Signed-off-by: Antonio Torres <antorres@redhat.com>
@antoniotorresm antoniotorresm added the ack Pull Request approved, can be merged label Aug 4, 2021
@antoniotorresm
Copy link
Contributor Author

PR was ACKed automatically because this is backport of PR #5841. Wait for CI to finish before pushing. In case of questions or problems contact @antoniotorresm who is author of the original PR.

@antoniotorresm antoniotorresm added the pushed Pull Request has already been pushed label Aug 4, 2021
@antoniotorresm
Copy link
Contributor Author

ipa-4-9:

  • 4c0dcab dnszone: deprecate option for setting SOA serial
  • 1d75124 ipatests: expect SOA serial option deprecation warning

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
1 participant