Evgeni Golov comments

Results 620 comments of


                                            Evgeni Golov

[WIP] Add foreman_ansible_roles module

Thanks @branic! And once more, sorry for keeping you busy by catching up on changes we made to our helpers. I've tried your module, and it works as advertised. However,...

[WIP] Add foreman_ansible_roles module

Oh, and I didn't see any difference between list and show in my tests. What extra info should show expose?

[WIP] Add foreman_ansible_roles module

Yeah. I think list and show are just default methods of the api and thus exist here. Show is cheaper if you only need to see one item. But in...

[WIP] Add foreman_ansible_roles module

Long term I'd love to see us having separate "info" modules for the various resources, but it's far from a priority as long foreman_search_facts covers the same functionality with a...

Compute attributes needs to be dictionary, they do not accept lists

Yeah, better examples are indeed a good idea. The second idea is also good, but more work. Are you planning to work on this?

"Current password Incorrect password" when trying to set the password of the current user

triage: ``` if user == username: current_password = password ``` carry on

add compute_attributes to foreman_host

so we added the non-idempotent version in https://github.com/theforeman/foreman-ansible-modules/pull/932, please test :)

use modern role documentation

an example can be found in https://github.com/sensu/sensu-go-ansible/pull/289

use modern role documentation

@wbclark to raise his issues/concerns :)

use modern role documentation

FWIW, I think the metadata structure comes from Ansible, not antsibull.