| *** ysandeep|out is now known as ysandeep | 05:21 | |
| *** ysandeep is now known as ysandeep|afk | 06:44 | |
| *** ysandeep|afk is now known as ysandeep | 07:31 | |
| opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update role_assignment to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843830 | 08:15 |
|---|---|---|
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Backport enhancements to host_aggregate module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844150 | 09:15 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Change network modules to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | 09:42 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Change network modules to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | 10:11 |
| *** rlandy|out is now known as rlandy | 10:29 | |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Change network modules to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | 10:36 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Backport enhancements to host_aggregate module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844150 | 10:45 |
| *** ysandeep is now known as ysandeep|break | 10:49 | |
| *** dviroel|out is now known as dviroel | 11:12 | |
| *** ysandeep|break is now known as ysandeep | 11:27 | |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Change network modules to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | 11:46 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Change network modules to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | 12:02 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Change network modules to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | 12:08 |
| opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves security_group_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840660 | 12:41 |
| *** ysandeep is now known as ysandeep|afk | 13:06 | |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Change network modules to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | 13:25 |
| sshnaidm | jm1, ^^ | 13:25 |
| jm1 | arxcruz: is working on role_assignment atm, maybe he can help? | 13:27 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Change network modules to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | 13:33 |
| arxcruz | jm1 ? | 13:48 |
| jm1 | arxcruz: FlexibleToast had a question about role_assignment on tuesday. maybe you have an idea what is going on? https://meetings.opendev.org/irclogs/%23openstack-ansible-sig/%23openstack-ansible-sig.2022-05-31.log.html | 13:58 |
| arxcruz | jm1 seems to be in the sdk side, maybe the grant_role have some constraints, from the role_assignment point of view, it only fails if the group, user, domain doesn't exist | 14:07 |
| arxcruz | i'll check later the sdk side | 14:07 |
| gtema | if someone is able to reproduce (or at least to give a concrete error description) it I will have a look | 14:08 |
| FlexibleToast | Yes, in check mode it succeeds. So it is passing the checks for the group/domain/project/role | 14:08 |
| gtema | FlexibleToast - if you can pls post how you invoke module (all the params) | 14:09 |
| arxcruz | gtema https://github.com/openstack/openstacksdk/blob/master/openstack/cloud/_identity.py#L1062= | 14:10 |
| arxcruz | error come from this | 14:10 |
| gtema | I know, but it will only come if neither group nor user are set | 14:10 |
| gtema | and I simply can't map the real invocation parameters | 14:10 |
| gtema | there are even plenty of tests for all of the possible combinations and I wonder whether particular combination is missing | 14:13 |
| arxcruz | agree | 14:13 |
| arxcruz | FlexibleToast maybe this is what happening? https://github.com/openstack/openstacksdk/blob/master/openstack/cloud/_identity.py#L1114-L1117= | 14:13 |
| gtema | if a referred "Must specify either a user or a group" error is returned we are not there and exit much earlier | 14:14 |
| FlexibleToast | The domain is definitely being passed and I believe they should all be in the same domain as we're really only using one domain in this lab environment | 14:14 |
| gtema | but that error is only returned when neither group nor user are found | 14:14 |
| arxcruz | FlexibleToast is it possible to test it with this patch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843830/ ? | 14:15 |
| FlexibleToast | This is an air-gapped environment. It takes a little bit of time to import something new like this. I can certainly give it a try though. | 14:17 |
| *** ysandeep|afk is now known as ysandeep | 14:28 | |
| opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves security_group_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840660 | 15:00 |
| opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves security_group_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840660 | 15:08 |
| *** ysandeep is now known as ysandeep|out | 15:38 | |
| *** dviroel is now known as dviroel|lunch | 15:44 | |
| opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Backport enhancements to host_aggregate module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844150 | 15:53 |
| opendevreview | Merged openstack/ansible-collections-openstack master: Change network modules to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | 15:54 |
| *** dviroel|lunch is now known as dviroel | 16:26 | |
| *** rlandy is now known as rlandy|dr_appt | 17:56 | |
| *** rlandy|dr_appt is now known as rlandy | 19:57 | |
| *** dviroel is now known as dviroel|afk | 20:28 | |
| *** dviroel|afk is now known as dviroel | 23:02 | |
| *** dviroel|afk is now known as dviroel | 23:03 | |
| *** dviroel is now known as dviroel|afk | 23:23 | |
| *** rlandy is now known as rlandy|out | 23:34 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!