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

Don't allow both a zone name and --name-from-ip to be provided #5091

Closed
wants to merge 2 commits into from

Conversation

rcritten
Copy link
Contributor

Don't allow both a zone name and --name-from-ip to be provided

--name-from-ip will generate a zone name so there is no point in
the user providing one. If one is provided and doesn't match the
generated name then a validation exception is raised.

https://pagure.io/freeipa/issue/8446

@rcritten rcritten added the ipa-4-8 Mark for backport to ipa 4.8 label Sep 10, 2020
Copy link
Member

@rjeffman rjeffman left a comment

Choose a reason for hiding this comment

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

LGTM.

--name-from-ip will generate a zone name so there is no point in
the user providing one. If one is provided and doesn't match the
generated name then a validation exception is raised.

https://pagure.io/freeipa/issue/8446

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
If a zone name is provided then name-from-ip makes little sense,
don't allow it.

https://pagure.io/freeipa/issue/8446

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
@rcritten
Copy link
Contributor Author

/azp run

@rcritten rcritten added the re-run Trigger a new run of PR-CI label Sep 10, 2020
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Sep 10, 2020
@abbra abbra added the ack Pull Request approved, can be merged label Sep 13, 2020
@abbra
Copy link
Contributor

abbra commented Sep 13, 2020

LGTM. ACK.

@abbra abbra added the pushed Pull Request has already been pushed label Sep 14, 2020
@abbra
Copy link
Contributor

abbra commented Sep 14, 2020

master:

  • 2265cb8 Don't allow both a zone name and --name-from-ip to be provided
  • e92a4ba ipatests: test that a zone name and name-from-ip will be rejected

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 ipa-4-8 Mark for backport to ipa 4.8 pushed Pull Request has already been pushed
Projects
None yet
4 participants