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

aixPB: add check for each of the RBAC auth used in the config #3090

Merged
merged 2 commits into from
Jun 9, 2023

Conversation

aixtools
Copy link
Contributor

@aixtools aixtools commented Jun 5, 2023

   and make auth as needed before calling loop
Checklist
  • commit message has one of the standard prefixes
  • faq.md updated if appropriate
  • other documentation is changed or added (if applicable)
  • playbook changes run through VPC or QPC (if you have access)
  • VPC/QPC not applicable for this PR
  • for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

       and make auth as needed before calling loop
@aixtools
Copy link
Contributor Author

aixtools commented Jun 5, 2023

@Haroon-Khel : please test this for me. I do not have a setup suitable for running the playbook. Many thanks.

@Haroon-Khel
Copy link
Contributor

Getting this while testing

fatal: [test-osuosl-aix715-ppc64-1]: FAILED! => {"msg": "The conditional check 'ojdk_exists.rc == 2 OR rtclk_exists.rc == 2 OR rtcore_exists.rc == 2' failed. The error was: template error while templating string: expected token 'end of statement
 block', got 'OR'. String: {% if ojdk_exists.rc == 2 OR rtclk_exists.rc == 2 OR rtcore_exists.rc == 2 %} True {% else %} False 
{% endif %}\n\nThe error appears to be in 
'/tmp/awx_1370_cd1xlrjb/project/ansible/playbooks/AdoptOpenJDK_AIX_Playbook/roles/rbac/tasks/main.yml': line 58, 
column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to 
be:\n\n\n- name: Update RBAC Security Tables to kernel\n  ^ here\n"}

ill try tweaking some things

Copy link
Member

@sxa sxa left a comment

Choose a reason for hiding this comment

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

Approving on the basis it seems to have been tested and validated as working

@karianna karianna merged commit 3cecbf3 into adoptium:master Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants