Monday, 2020-11-16

*** iurygregory has quit IRC02:07
*** dmsimard has quit IRC02:49
*** dmsimard has joined #openstack-ansible-sig02:50
*** evrardjp has quit IRC05:33
*** evrardjp has joined #openstack-ansible-sig05:33
*** iurygregory has joined #openstack-ansible-sig07:29
*** sshnaidm|off is now known as sshnaidm07:46
*** gtema has joined #openstack-ansible-sig08:13
noonedeadpunkHi everyone! How do you think - should we fail here https://opendev.org/openstack/ansible-collections-openstack/src/tag/1.0.0/plugins/modules/endpoint.py#L161-L162if endpoint state is set to absent?08:20
*** gtema has quit IRC08:27
*** gtema has joined #openstack-ansible-sig08:31
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/ansible-collections-openstack master: Do not fail when endpoint state is absent  https://review.opendev.org/76279808:35
noonedeadpunkie ^08:35
tremblenoonedeadpunk, Trying to ensure that something you want to be gone is gone should return "result is not changed" rather than throwing an exception, this is the standard Ansible module behaviour.10:13
sshnaidm+1 ^^10:16
trembleIf the service is gone, then the endpoint should be gone, so changed=False feels like the right behaviour to me...10:16
noonedeadpunkok, thanks for the feedback :) I t felt right to me as well, but was not sure 100% so decided to ask as well10:18
openstackgerritMerged openstack/ansible-collections-openstack master: Do not fail when endpoint state is absent  https://review.opendev.org/76279811:48
*** frenzy_friday has joined #openstack-ansible-sig12:17
*** redrobot has quit IRC15:35
*** Guest64097 has joined #openstack-ansible-sig15:51
*** Guest64097 is now known as redrobot15:53
*** gtema has quit IRC16:03
*** gtema has joined #openstack-ansible-sig16:03
*** gtema has quit IRC17:32
*** gtema has joined #openstack-ansible-sig18:04
*** gtema has quit IRC18:10
*** gtema has joined #openstack-ansible-sig19:23
*** gtema has quit IRC19:39
*** iurygregory has quit IRC20:40
*** iurygregory has joined #openstack-ansible-sig20:43
*** zeestrat has quit IRC21:12
*** zeestrat has joined #openstack-ansible-sig21:13
*** gtema has joined #openstack-ansible-sig21:40
*** gtema has quit IRC21:45

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!