Skip to content

Fixed result type of "when" conditions.#855

Merged
insatomcat merged 1 commit intoseapath:mainfrom
dahoat-sprecher:main
Jan 30, 2026
Merged

Fixed result type of "when" conditions.#855
insatomcat merged 1 commit intoseapath:mainfrom
dahoat-sprecher:main

Conversation

@dahoat-sprecher
Copy link
Contributor

@dahoat-sprecher dahoat-sprecher commented Jan 15, 2026

The "when" condition in tasks of role network_systemdnetworkd and detect_seapath_distro returned values which could be evaluated to a boolean but were no real booleans and thus rejected by Ansible (2.19.4).
OS: Debian Trixie (6.12.63-1 (2025-12-30) x86_64)

Best regards,
Daniel

@eroussy
Copy link
Member

eroussy commented Jan 16, 2026

Thanks for you PR @dahoat-sprecher
Just for you to know, we are right now in the process to move from ansible 2.10 to ansible 2.16 (See #843 )

It is an important stuff for the project, so we will first merge this ansible 2.16 PR and then come back to you

@eroussy
Copy link
Member

eroussy commented Jan 30, 2026

Hi @dahoat-sprecher
#843 is merged now. Could you rebase your work on main so that we can review this PR ?

The "when" condition in tasks of role network_systemdnetworkd and detect_seapath_distro returned values which could be evaluated to a boolean but were no real booleans and thus rejected by Ansible (2.19.4).

Signed-off-by: Daniel Hofer <daniel.hofer@sprecher-automation.com>
@dahoat-sprecher
Copy link
Contributor Author

Hi @eroussy
Thanks for notifying me. My fork should be synced ad the PR updated now.

@insatomcat insatomcat merged commit 5896aab into seapath:main Jan 30, 2026
4 checks passed
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.

3 participants