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.

Sign up for GitHub

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

Jump to bottom

[REFACTOR] Remove defined variable in vars directory #97

Closed
xtianae7 opened this issue Nov 15, 2023 · 0 comments · Fixed by #98
Closed

[REFACTOR] Remove defined variable in vars directory #97

xtianae7 opened this issue Nov 15, 2023 · 0 comments · Fixed by #98
Assignees
@xtianae7
Labels
bug

Comments

@xtianae7
Copy link
Member

xtianae7 commented Nov 15, 2023 •
edited
Loading

Description

Remove a declared variable in vars directory so it can be modified from a playbook.
mysql_required_libs is defined in both, vars and defaults.

The one declared in "vars" is always being used, and the one in "defauts" ignored at all.

[Description of the issue]

Link/s to the specific code that needs a refactor (if applicable)

[Links]

mysql_role/defaults/main.yml

Line 15 in 5efb72f

mysql_required_libs:

mysql_role/vars/mariadb.yml

Line 2 in 5efb72f

mysql_required_libs:

@xtianae7 xtianae7 added the bug label Nov 15, 2023
@xtianae7 xtianae7 self-assigned this Nov 15, 2023
xtianae7 added a commit that referenced this issue Nov 15, 2023
@xtianae7
[#97] Remove variable in "vars" in favor of "defauts"
874a966
@xtianae7 xtianae7 mentioned this issue Nov 15, 2023
Merged
@xtianae7 xtianae7 linked a pull request Nov 15, 2023 that will close this issue
[#97] Remove variable in "vars" in favor of "defauts" #98
Merged
xtianae7 added a commit that referenced this issue Nov 15, 2023
@xtianae7
Merge pull request #98 from idealista/bugs/97
a7743b2 
[#97] Remove variable in "vars" in favor of "defauts"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xtianae7 xtianae7

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#97] Remove variable in "vars" in favor of "defauts" idealista/mysql_role
1 participant
@xtianae7