Julian @ 42 N.E.R.D.S. Comments

Julian @ 42 N.E.R.D.S.

•Created by heavytobi on 10/22/2024 in #help

Ansible not usable

@wendrul oh yeah \o/, this is what I overlooked. That name-Parameter seems to solve the issue. I just gave it a try. I will look further into this the next days. Pretty awesome support, thanks a lot.

15 replies

WWindmill

•Created by heavytobi on 10/22/2024 in #help

Ansible not usable

Hi, I'm really looking forward to using ansible with windmill. But I have a showstopper for us to use it: When I read your code correct, you hardcoded the inventory file name https://github.com/windmill-labs/windmill/blob/c194e124dac3b9cf261f7f504dcbc306c37ea4f4/backend/parsers/windmill-parser-yaml/src/lib.rs#L237 with an .ini extension. This makes your ansible implementation unusable with dynamic inventory plugins.

dependencies:
  galaxy:
    collections:
      - name: hetzner.hcloud
  python:
    - hcloud

dependencies:
  galaxy:
    collections:
      - name: hetzner.hcloud
  python:
    - hcloud

in this setup i want to use an inventory file which ist not in ini format.

# hcloud.yml
---
plugin: hetzner.hcloud.hcloud

# hcloud.yml
---
plugin: hetzner.hcloud.hcloud

then you call this locally with for example

$  ansible-inventory -i hcloud.yml --graph
@all:
  |--@ungrouped:
  |--@hcloud:
  |  |--node-1
  |  |--node-2

$  ansible-inventory -i hcloud.yml --graph
@all:
  |--@ungrouped:
  |--@hcloud:
  |  |--node-1
  |  |--node-2

this works locally but not in windmill, as I get:

[WARNING]:  * Failed to parse /tmp/windmill/wk-default-
fd0172ad9b40-bFAfZ/019354e4-7358-5833-262a-026b281a13d2/inventory.ini with ini
plugin: Invalid host pattern '---' supplied, '---' is normally a sign this is a
YAML file.
[WARNING]: Unable to parse /tmp/windmill/wk-default-
fd0172ad9b40-bFAfZ/019354e4-7358-5833-262a-026b281a13d2/inventory.ini as an
inventory source

[WARNING]:  * Failed to parse /tmp/windmill/wk-default-
fd0172ad9b40-bFAfZ/019354e4-7358-5833-262a-026b281a13d2/inventory.ini with ini
plugin: Invalid host pattern '---' supplied, '---' is normally a sign this is a
YAML file.
[WARNING]: Unable to parse /tmp/windmill/wk-default-
fd0172ad9b40-bFAfZ/019354e4-7358-5833-262a-026b281a13d2/inventory.ini as an
inventory source

Please note: I added a custom inventory source type of plain text format yaml Did I miss something? Any chance the inventory: parameter will accept something like inventory_type: yaml or (better) infer the file extension from the resource type extension and not hardcoded?

15 replies

WWindmill