Skip to content

collection and sap_hostagent: ansible-lint fixes#973

Merged
ja9fuchs merged 1 commit into
sap-linuxlab:devfrom
ja9fuchs:fix-linting
Feb 14, 2025
Merged

collection and sap_hostagent: ansible-lint fixes#973
ja9fuchs merged 1 commit into
sap-linuxlab:devfrom
ja9fuchs:fix-linting

Conversation

@ja9fuchs
Copy link
Copy Markdown
Contributor

  • collection: added .ansible/ to excludes, this is auto-created by pre-commit and contains the collection, on which the role excludes do not work

  • sap_hostagent: removed ansible_become from defaults, it does not belong in the role but in the playbook, and it triggers the no-role-prefix rule

  • sap_hostagent: changed some truthy values for consistency

  • sap_hostagent is now free of linter errors and included in checks

- added .ansible/ to excludes, this is auto-created by pre-commit and
  contains the collection, on which the role excludes do not work
- sap_hostagent: removed `ansible_become` from defaults, it does not belong in the role
  but in the playbook, and it triggers the no-role-prefix rule
- sap_hostagent: changed some truthy values for consistency
- sap_hostagent linting is fine now, it is commented out in the exclude
  list
@ja9fuchs ja9fuchs requested review from berndfinger and rhmk February 14, 2025 12:54
@ja9fuchs ja9fuchs changed the title collection and sap_hostagend: ansible-lint fixes collection and sap_hostagent: ansible-lint fixes Feb 14, 2025
Copy link
Copy Markdown
Member

@berndfinger berndfinger left a comment

Choose a reason for hiding this comment

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

LGTM.

@ja9fuchs ja9fuchs merged commit 73c5774 into sap-linuxlab:dev Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants