Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

[FLOSS] VISION.md: example for ansible no longer working #4333

Closed
lukashartl opened this issue Apr 18, 2022 · 3 comments
Closed

[FLOSS] VISION.md: example for ansible no longer working #4333

lukashartl opened this issue Apr 18, 2022 · 3 comments

Comments

@lukashartl
Copy link
Contributor

Steps to Reproduce the Problem

- name: setup samba workgroup
  connection: local
  hosts: localhost
  tasks:
    - name: set workgroup
      elektra:
        key: "system:/sw/samba/#0/current/global/workgroup"
        value: "MYGROUP"

Actual Result

TASK [just a test for example01] **************************************************************
fatal: [example01.example.com]: FAILED! => {"changed": false, "msg": "Unsupported parameters for (elektra) module: key. Supported parameters include: recommends, keeporder, keys, filename, resolver, plugins, mountpoint."}

Description

The ansible-libelektra module has changed, but the documentation in this repo is still old.

@flo91
Copy link
Collaborator

flo91 commented Oct 4, 2022

@markus2330 Maybe also a small issue for the FLOSS course.

@markus2330 markus2330 added floss2022w lang/config triage needed Issue needs clarifications. labels Oct 9, 2022
@markus2330
Copy link
Contributor

Absolutely! @flo91 btw. I added the label lang/config if config files need to be fixed. It's not 100% correct here but Ansible is written in YAML files so it is also not completely wrong.

Btw. the task would be to find a new example that works. There are probably several problems here, in the ansible code and also that we do not have an INI parser compatible to samba. Fixing TOML is a different issue: #3490.

In any case first we need a triage to find out what everything is wrong.

@Bujuhu Bujuhu self-assigned this Oct 19, 2022
@Bujuhu Bujuhu added the H1 label Oct 19, 2022
@Bujuhu
Copy link
Contributor

Bujuhu commented Oct 19, 2022

Hi!

Here are changes I've made to fix the outdated ansible playbook, which should be functionally identical to the one posted above
cat playbook.yml

- name: setup samba workgroup
  connection: local
  hosts: localhost
  collections:
    - elektra_initiative.libelektra
  tasks:
    - name: set workgroup
      elektra:
        mountpoint: system:/sw/samba/#0/current/global
        keys:
          workgroup:
            value: MYGROUP
sudo kdb get system:/sw/samba/#0/current/global/workgroup
MYGROUP
cat /etc/samba/smb.conf
;Ni1
; Generated by the ni plugin using Elektra (see libelektra.org).

workgroup = MYGROUP

The only place I could find a full explanation of the options in the most update to date version was here. Everything else seems to be outdated.

@Bujuhu Bujuhu removed the triage needed Issue needs clarifications. label Oct 19, 2022
@flo91 flo91 changed the title VISION.md: example for ansible no longer working [FLOSS] VISION.md: example for ansible no longer working Oct 23, 2022
Bujuhu added a commit to Bujuhu/libelektra that referenced this issue Nov 8, 2022
Bujuhu added a commit to Bujuhu/libelektra that referenced this issue Nov 8, 2022
- Update ansible playbook in VISION.md to be compatible with current ansible-libelektra version
- Fixes ElektraInitiative#4333
Bujuhu added a commit to Bujuhu/libelektra that referenced this issue Nov 9, 2022
- Update ansible playbook in VISION.md to be compatible with current ansible-libelektra version
- Fixes ElektraInitiative#4333
@flo91 flo91 closed this as completed in 0f462f5 Nov 23, 2022
joni1993 pushed a commit to joni1993/libelektra that referenced this issue Dec 5, 2022
- Update ansible playbook in VISION.md to be compatible with current ansible-libelektra version
- Fixes ElektraInitiative#4333
Dynamichost96 pushed a commit to Dynamichost96/libelektra that referenced this issue Jan 15, 2023
- Update ansible playbook in VISION.md to be compatible with current ansible-libelektra version
- Fixes ElektraInitiative#4333
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants