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

Add check for offline installs and allow to specify custom xml config #108

Merged

Conversation

guidograzioli
Copy link
Member

@guidograzioli guidograzioli commented Apr 24, 2023

Add a new variable wildfly_config_custom_file which allows to pass a custom file/templated standalone.xml configuration to use as a base config. Defaults to empty string (do not use the feature).

Variable Description Default
wildfly_config_base Base standalone.xml config for instance standalone.xml unless wildfly_config_custom_file is used
wildfly_config_custom_file Custom standalone.xml config to be copied to target instance and used as base ''

wildfly_config_base changes it default to standalone.xml only when wildfly_config_custom_file is not used, otherwise it uses whatever filename is passed in.

Also, improves fix for #101 with a pre-req conditional which fails the play at a very early stage when offline_install is enabled, but the necessary zipfiles are not available on the controller.

@guidograzioli guidograzioli added the minor_changes New parameters added to modules, or non-breaking behavior changes to existing parameters; no bugfix label Apr 25, 2023
@guidograzioli guidograzioli changed the title add prereq for filename existing, add fix to apply_cp when no version… Add check for offline installs and allow to specify custom xml config Apr 25, 2023
Copy link
Contributor

@rpelisse rpelisse left a comment

Choose a reason for hiding this comment

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

LGTM

owner: "{{ wildfly_install.user }}"
mode: 0640
become: yes
when: wildfly_install.config_custom | length > 0
Copy link
Contributor

Choose a reason for hiding this comment

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

I would add wildfly_install is defined and wildfly_install.config_custom is defined, but I'm on the paranoid side of validation :)

@guidograzioli guidograzioli merged commit c292bcc into ansible-middleware:main Apr 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor_changes New parameters added to modules, or non-breaking behavior changes to existing parameters; no bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants