ansible-jsonpatch icon indicating copy to clipboard operation
ansible-jsonpatch copied to clipboard

wish: add ansible-jsonpatch to galaxy

Open tpo opened this issue 4 years ago • 13 comments

Hi,

currently ansible-jsonpatch is living on my system in my ansible repo as a submodule. Submodule handling in git sucks in various ways. I could manage it outside of my ansible repo, but then, why not do the next step too and have ansible-jsonpatch on galaxy. That way I could manage it just like any other galaxy role/module...

?

Would be nice!

In case you loathe learning new galaxy processes then I can assist you through it having done it recently. -> DM.

Thanks for ansible-jsonpatch! *t

tpo avatar May 17 '20 17:05 tpo

Yep, this is a good idea. I already have content on galaxy.ansible.com so I should go ahead and move more of my stuff in there. I'll try and get on this sometime this week.

particledecay avatar May 18 '20 17:05 particledecay

@tpo I wouldn't mind chatting about the best way to get this in there. You can find me on PySlackers (pythondev.slack.com). I'm on a million other slack teams too but I figure that one's the most relevant to ansible (there isn't an ansible slack though).

particledecay avatar May 20 '20 03:05 particledecay

Well, here's my latest attempt. Take 2! https://github.com/ansible-collections/community.general/pull/378

particledecay avatar May 20 '20 04:05 particledecay

@tpo I wouldn't mind chatting about the best way to get this in there. You can find me on PySlackers (pythondev.slack.com). I'm on a million other slack teams too but I figure that one's the most relevant to ansible (there isn't an ansible slack though).

There doesn't seem to be a way to sign on to Slack any more: https://slack.com/intl/en-ch/get-started#/

It seems you need to invite me. Please invite the emai address tpo_pythondev.slack.com at sourcepole.ch .

tpo avatar May 20 '20 10:05 tpo

Well, here's my latest attempt. Take 2! ansible-collections/community.general#378

currently some of the module's tests are failing... https://github.com/ansible-collections/community.general/pull/378#issuecomment-631230151

tpo avatar May 20 '20 10:05 tpo

@tpo you can join the slack from this page: https://pyslackers.com/web. They have a "Join" button right in the middle.

As far as the tests... I'll check out some details tonight, though that ansible-doc failure looks suspicious.

particledecay avatar May 20 '20 14:05 particledecay

I noticed the PR on ansible-collections is closed now - is the work on getting it merged there dead now?

godeater avatar May 10 '22 23:05 godeater

@godeater yes, unfortunately I've had a pretty poor experience with Ansible folks, so I haven't pushed that any further. If someone's willing to help a bit I'd be willing to pick it back up?

particledecay avatar May 11 '22 13:05 particledecay

This issue was closed, but it doesn't look like it was resolved - I think the original request was to publish this module in a separate, standalone collection on ansible galaxy under a personal namespace? It doesn't need to be merged into any ansible-maintained git repositories.

See the documentation for Creating collections, but to summarize I think the main steps required would be:

  • Move files around a bit in the git repo to follow the collection structure
    • json_patch.py goes in plugins/modules/json_patch.py
    • test_json_patch.py goes in tests/unit/plugins/modules/test_json_patch.py to be run by ansible-test
  • Add the galaxy.yml metadata file
  • At this point, people could use ansible-galaxy to install directly from the git repo, but to publish a release:
  • Follow the instructions for Distributing collections to build and publish a release to ansible galaxy.

kepstin avatar Nov 20 '23 17:11 kepstin

I don't mind hosting it there under my own account. If there's still value in this, I can look into it tonight.

particledecay avatar Nov 20 '23 17:11 particledecay

I'd say it's still useful for people, so I think you should publish it if you can - I'm actually hosting it in a collection in our internal (and therefore private) galaxy hub for our teams to use - we definitely find it useful!

godeater avatar Nov 21 '23 02:11 godeater

I'd say it's still useful for people, so I think you should publish it if you can - I'm actually hosting it in a collection in our internal (and therefore private) galaxy hub for our teams to use - we definitely find it useful!

I just want to let you @particledecay know that we're still using ansible-jsonpatch, it's being accessed by some playbooks of ours, and we are very happy and grateful to be able to do so.

Publishing it on galaxy would make the module a bit cleaner and easier to use, however this by no means we request or demand that. Having it on galaxy would just "polish" the module a bit more.

Thanks a lot for having made the module available to the universe in the first place!!!

tpo avatar Nov 22 '23 10:11 tpo

Sorry, I only meant whether having it on Galaxy would still be useful haha, not the module. But great to hear you're enjoying it! Don't worry I'm not going to archive or drop this anytime soon 😉

particledecay avatar Nov 22 '23 11:11 particledecay