ansible-modules-hashivault icon indicating copy to clipboard operation
ansible-modules-hashivault copied to clipboard

Ansible collection

Open TerryHowe opened this issue 5 years ago • 8 comments

This PR converts the module into an Ansible collection.

Fixes: #234 Closes: #263

Still to do: Update the documentation and remove PyPi scripts.

It can be packaged with:

ansible-galaxy collection build

The package can then be installed into the user's home with:

ansible-galaxy collection install terryhowe-hashivault-4.5.2.tar.gz

To try it out, use the following playbook:

- hosts: test
  gather_facts: no
  collections:
   - terryhowe.hashivault
  tasks:
  - name: get test secret
    delegate_to: localhost
    hashivault_read:
      secret: test/secret
    register: new
  - debug:
      var: new.value

TerryHowe avatar Nov 08 '20 21:11 TerryHowe

Looks like the tox tests will need to be adapted as well. You could also add a "Closes: #263" to the description to have the other PR closed automatically when this gets merged.

In any case, thanks for looking into the PR. I'll give your modifications a try later.

onitake avatar Nov 09 '20 12:11 onitake

@onitake any updates ?

freeseacher avatar Dec 11 '20 05:12 freeseacher

I'm looking forward to completing this feature.

elcomtik avatar Feb 06 '21 21:02 elcomtik

It looks like a bunch of symbols have been removing from module_utils/hashivault.py yet are still imported. Can this branch be maybe restarted from the latest changes (4.6.2) instead of try and replay the work that's gone into main, but untested and partial.

bendem avatar Feb 23 '21 14:02 bendem

Any updates on this?

ericsysmin avatar May 26 '21 21:05 ericsysmin

It looks like a bunch of symbols have been removing from module_utils/hashivault.py yet are still imported. Can this branch be maybe restarted from the latest changes (4.6.2) instead of try and replay the work that's gone into main, but untested and partial.

This can be done via a git rebase.

@TerryHowe any plans on having this converted to a collection and pushed to Ansible Galaxy?

fastlorenzo avatar Jul 12 '21 10:07 fastlorenzo

This can be done via a git rebase.

Have you checked out the branch? Someone did try to rebase it, it's a mess. Some symbols are used but their definitions disappeared, some stuff got merged one way some stuff got merged the other way...

bendem avatar Jul 16 '21 13:07 bendem

This can be done via a git rebase.

Have you checked out the branch? Someone did try to rebase it, it's a mess. Some symbols are used but their definitions disappeared, some stuff got merged one way some stuff got merged the other way...

If you're talking about PR #350 , it's me 😆 I've tried something but not sure if the rebate all went well.

fastlorenzo avatar Jul 16 '21 18:07 fastlorenzo