Skip to content
This repository has been archived by the owner on Dec 27, 2023. It is now read-only.

[FIX]Don't use password if not defined #22

Closed
wants to merge 1 commit into from

Conversation

flotho
Copy link

@flotho flotho commented Dec 5, 2022

proposal to fix #21

@ulvida
Copy link
Member

ulvida commented Dec 7, 2022

Merci, @flotho !
Next wednesday we are moving ouyr office, but I'll see it after. Before testing, idea, for our DRY: maybe we could use module_defaults to factorize kvm common modules (almost all).

@ulvida
Copy link
Member

ulvida commented Dec 14, 2022

Following up your MR (gitlab's PR) in the collection where this role has moved, I proposed you back a code improvement MR.

Let's continue there!

@flotho
Copy link
Author

flotho commented Dec 14, 2022

thanks

@flotho flotho closed this Dec 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE]don't use the api_password when token and secret are used
2 participants