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

ipa_dnszone fails for reverse zone #6905

Closed
1 task done
irozet12 opened this issue Jul 10, 2023 · 2 comments · Fixed by #6906
Closed
1 task done

ipa_dnszone fails for reverse zone #6905

irozet12 opened this issue Jul 10, 2023 · 2 comments · Fixed by #6906
Labels
bug This issue/PR relates to a bug module module plugins plugin (any type)

Comments

@irozet12
Copy link
Contributor

Summary

The following code fails:

  community.general.ipa_dnszone:
    ipa_host: "{{ freeipa_master }}"
    ipa_pass: "{{ freeipa_admin_password }}"
    state: present
    zone_name: "10.13.10.in-addr.arpa."
    dynamicupdate: true
    validate_certs: false

Error:

fatal: [dc02 -> dc01(10.13.19.33)]: FAILED! => changed=false 
  msg: '''idnsallowsyncptr''

Reason: this is dict key error. Freeipa response from dnszone_find method for reverse zone does not contain 'idnsallowsyncptr' field

Issue Type

Bug Report

Component Name

ipa_dnszone

Ansible Version

$ ansible --version
ansible [core 2.15.0]
  config file = ~/ansible/ansible.cfg
  configured module search path = ['~/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/Cellar/ansible/8.0.0/libexec/lib/python3.11/site-packages/ansible
  ansible collection location = ~/.ansible/collections:/usr/share/ansible/collections
  executable location = /usr/local/bin/ansible
  python version = 3.11.4 (main, Jun  7 2023, 00:42:15) [Clang 14.0.3 (clang-1403.0.22.14.1)] (/usr/local/Cellar/ansible/8.0.0/libexec/bin/python3.11)
  jinja version = 3.1.2
  libyaml = True

Community.general Version

$ ansible-galaxy collection list community.general
# ~.ansible/collections/ansible_collections
Collection        Version
----------------- -------
community.general 7.0.1  

# /usr/local/Cellar/ansible/8.0.0/libexec/lib/python3.11/site-packages/ansible_collections
Collection        Version
----------------- -------
community.general 7.0.1  

Configuration

$ ansible-config dump --only-changed
CONFIG_FILE() = ~/ansible/ansible.cfg
DEFAULT_HOST_LIST(~/ansible/ansible.cfg) = ['~/ansible/environments/dev/inventory']
DEFAULT_ROLES_PATH(~/ansible/ansible.cfg) = ['~/ansible/roles']
DEFAULT_STDOUT_CALLBACK(~/ansible/ansible.cfg) = yaml
INTERPRETER_PYTHON(~/ansible/ansible.cfg) = /usr/bin/python3

OS / Environment

Darwin Kernel Version 22.5.0

Steps to Reproduce

- name: Test IPA reverse zone 
  community.general.ipa_dnszone:
    ipa_host: "{{ freeipa_master }}"
    ipa_pass: "{{ freeipa_admin_password }}"
    state: present
    zone_name: "10.13.10.in-addr.arpa."
    dynamicupdate: true
    validate_certs: false

Expected Results

Reverse zone is added

Actual Results

fatal: [dc02 -> dc01(10.13.19.33)]: FAILED! => changed=false 
  msg: '''idnsallowsyncptr''

Code of Conduct

  • I agree to follow the Ansible Code of Conduct
@ansibullbot
Copy link
Collaborator

Files identified in the description:

If these files are incorrect, please update the component name section of the description or use the !component bot command.

click here for bot help

@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added bug This issue/PR relates to a bug module module plugins plugin (any type) labels Jul 10, 2023
irozet12 added a commit to irozet12/community.general that referenced this issue Jul 10, 2023
irozet12 added a commit to irozet12/community.general that referenced this issue Jul 13, 2023
felixfontein added a commit that referenced this issue Jul 16, 2023
* Fix key error for reverse zone (#6905)

* Add changelog fragment for #6905

* Fix changelog (6905)

Co-authored-by: Felix Fontein <felix@fontein.de>

---------

Co-authored-by: Felix Fontein <felix@fontein.de>
patchback bot pushed a commit that referenced this issue Jul 16, 2023
* Fix key error for reverse zone (#6905)

* Add changelog fragment for #6905

* Fix changelog (6905)

Co-authored-by: Felix Fontein <felix@fontein.de>

---------

Co-authored-by: Felix Fontein <felix@fontein.de>
(cherry picked from commit b818afd)
patchback bot pushed a commit that referenced this issue Jul 16, 2023
* Fix key error for reverse zone (#6905)

* Add changelog fragment for #6905

* Fix changelog (6905)

Co-authored-by: Felix Fontein <felix@fontein.de>

---------

Co-authored-by: Felix Fontein <felix@fontein.de>
(cherry picked from commit b818afd)
felixfontein pushed a commit that referenced this issue Jul 16, 2023
…#6905) (#6950)

Fix key error for reverse zone (#6905) (#6906)

* Fix key error for reverse zone (#6905)

* Add changelog fragment for #6905

* Fix changelog (6905)

Co-authored-by: Felix Fontein <felix@fontein.de>

---------

Co-authored-by: Felix Fontein <felix@fontein.de>
(cherry picked from commit b818afd)

Co-authored-by: irozet12 <119814380+irozet12@users.noreply.github.com>
felixfontein pushed a commit that referenced this issue Jul 16, 2023
…#6905) (#6951)

Fix key error for reverse zone (#6905) (#6906)

* Fix key error for reverse zone (#6905)

* Add changelog fragment for #6905

* Fix changelog (6905)

Co-authored-by: Felix Fontein <felix@fontein.de>

---------

Co-authored-by: Felix Fontein <felix@fontein.de>
(cherry picked from commit b818afd)

Co-authored-by: irozet12 <119814380+irozet12@users.noreply.github.com>
valeriopoggi pushed a commit to valeriopoggi/community.general that referenced this issue Jul 17, 2023
…llections#6906)

* Fix key error for reverse zone (ansible-collections#6905)

* Add changelog fragment for ansible-collections#6905

* Fix changelog (6905)

Co-authored-by: Felix Fontein <felix@fontein.de>

---------

Co-authored-by: Felix Fontein <felix@fontein.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug module module plugins plugin (any type)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants