Bug 2134505

Summary: [ansible-freeipa] Do not require 'enable-sid' for 'add-sids' and 'netbios-name'.
Product: Red Hat Enterprise Linux 9 Reporter: Rafael Jeffman <rjeffman>
Component: ansible-freeipaAssignee: Rafael Jeffman <rjeffman>
Status: CLOSED ERRATA QA Contact: Varun Mylaraiah <mvarun>
Severity: high Docs Contact:
Priority: unspecified    
Version: 9.1CC: mvarun, tscherf, twoerner
Target Milestone: rcKeywords: Triaged
Target Release: ---Flags: pm-rhel: mirror+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: ansible-freeipa-1.9.0-1.el9 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 2134530 2135776 (view as bug list) Environment:
Last Closed: 2023-05-09 07:25:56 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 2134530, 2135776    
Deadline: 2022-12-12   

Description Rafael Jeffman 2022-10-13 13:46:48 UTC
When using plugin 'ipaconfig', parameters 'add-sids' and 'netbios-name' can only be used if 'enable-sid: yes' is used.

It would provide a better experience if the parameters can be used without having to set 'enable-sid', and the plugin simply fails if SID generation is not enabled and 'enable-sid' is not set to 'yes'.

Comment 1 Rafael Jeffman 2022-10-13 16:35:48 UTC
Upstream PR: https://github.com/freeipa/ansible-freeipa/pull/961

Comment 7 Varun Mylaraiah 2022-12-20 13:34:21 UTC
Verified
ansible-core-2.14.1-1.el9.x86_64
ansible-freeipa-1.9.0-1.el9.noarch

Passed	ansible_freeipa_tests/config/test_config.py::TestConfigInvalid::test_update_netbios_name_without_enable_sid

DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 PLAYBOOK: config_module.yml ****************************************************
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 1 plays in config_module.yml
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 PLAY [Playbook to ensure able to update netbios_name without enable_sid.] ******
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 TASK [Gathering Facts] *********************************************************
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 task path: /root/config_module.yml:2
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 ok: [master.ipadomain.test]
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 TASK [ipaconfig] ***************************************************************
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 task path: /root/config_module.yml:6
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 changed: [master.ipadomain.test] => {"changed": true, "config": {}}
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 PLAY RECAP *********************************************************************
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 master.ipadomain.test      : ok=2    changed=1    unreachable=0    failed=0    skipped=0    rescued=0    ignored=0   
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:563 
DEBUG    pytest_multihost.host.Host.ansible.cmd171:transport.py:217 Exit code: 0
INFO     pytest_multihost.host.Host.master.OpenSSHTransport:transport.py:397 RUN ['kinit', 'admin']
DEBUG    pytest_multihost.host.Host.master.cmd210:transport.py:519 RUN ['kinit', 'admin']
DEBUG    pytest_multihost.host.Host.master.cmd210:transport.py:563 bash: line 1: cd: /root/multihost_tests: No such file or directory
DEBUG    pytest_multihost.host.Host.master.cmd210:transport.py:563 bash: line 2: /root/multihost_tests/env.sh: No such file or directory
DEBUG    pytest_multihost.host.Host.master.cmd210:transport.py:563 Password for admin: 
DEBUG    pytest_multihost.host.Host.master.cmd210:transport.py:217 Exit code: 0
INFO     pytest_multihost.host.Host.master.OpenSSHTransport:transport.py:397 RUN ['ipa', 'trustconfig-show']
DEBUG    pytest_multihost.host.Host.master.cmd211:transport.py:519 RUN ['ipa', 'trustconfig-show']
DEBUG    pytest_multihost.host.Host.master.cmd211:transport.py:563 bash: line 1: cd: /root/multihost_tests: No such file or directory
DEBUG    pytest_multihost.host.Host.master.cmd211:transport.py:563 bash: line 2: /root/multihost_tests/env.sh: No such file or directory
DEBUG    pytest_multihost.host.Host.master.cmd211:transport.py:563   Domain: ipadomain.test
DEBUG    pytest_multihost.host.Host.master.cmd211:transport.py:563   Security Identifier: S-1-5-21-2538360992-4063131553-938975944
DEBUG    pytest_multihost.host.Host.master.cmd211:transport.py:563   NetBIOS name: NEWTESTNETBIOS
DEBUG    pytest_multihost.host.Host.master.cmd211:transport.py:563   Domain GUID: dab96bc0-e73a-4da8-a249-0e691bc66b73
DEBUG    pytest_multihost.host.Host.master.cmd211:transport.py:563   Fallback primary group: Default SMB Group
DEBUG    pytest_multihost.host.Host.master.cmd211:transport.py:217 Exit code: 0
INFO     pytest_multihost.host.Host.master.OpenSSHTransport:transport.py:397 RUN ['kdestroy', '-A']


Based on the test result, marking the bug Verified

Comment 10 errata-xmlrpc 2023-05-09 07:25:56 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (ansible-freeipa bug fix and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2023:2168