theforeman / foreman_puppet

https://theforeman.org/plugins/foreman_puppet/
GNU General Public License v3.0
7 stars 24 forks source link

API working in hammer and fam #150

Closed nadjaheitmann closed 2 years ago

nadjaheitmann commented 3 years ago

Puppet related Foreman Ansible Modules need to be updated after changes in the API.

There might be need for more discussion about API changes in general.

ezr-ondrej commented 3 years ago

This should probably get to the 1.0 release, I'll align that. If there is any volunteer who would want to take this effort, split it into more actionable tasks and be in charge of that, it's most definitelly welcome!

Thanks for reporting :+1:

nadjaheitmann commented 3 years ago

I don't think that there is a straight way forward to go here. So I put this up for discussion: https://community.theforeman.org/t/the-road-to-making-puppet-optional/17983/23

I highly appreciate your opinions @ezr-ondrej @timogoebel @sbernhard and everyone else.

nadjaheitmann commented 3 years ago

Can we close this issue? I think that we have a good base for the API and can open further issues on demand. @ezr-ondrej @timogoebel @evgeni @ofedoren @amirfefer Are you aware of any hammer/FAM issues related to Puppet API?

ezr-ondrej commented 2 years ago

I agree, this can be closed as it mostly works, if something does not, we should open specific issues :)