| *** spatel_ has joined #openstack-ansible-sig | 02:13 | |
| *** spatel_ is now known as spatel | 02:13 | |
| *** spatel has quit IRC | 02:36 | |
| *** spatel_ has joined #openstack-ansible-sig | 02:41 | |
| *** spatel_ is now known as spatel | 02:41 | |
| *** spatel has quit IRC | 03:05 | |
| *** gtema_ has joined #openstack-ansible-sig | 06:14 | |
| *** gtema_ has quit IRC | 06:18 | |
| *** gtema_ has joined #openstack-ansible-sig | 06:18 | |
| *** gtema has joined #openstack-ansible-sig | 06:28 | |
| gtema | test from matrix | 06:28 |
|---|---|---|
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch role_assignment module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792416 | 07:26 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch Recordset module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792417 | 07:28 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch Quota module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792418 | 07:29 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch ProjectAccess module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792419 | 07:29 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch port_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792420 | 07:29 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch port module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792421 | 07:30 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch object module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792422 | 07:31 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch lb_pool module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792423 | 07:32 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch lb_member module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792424 | 07:32 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch lb_listener module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792425 | 07:33 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch KeystoneFederationProtocolInfo module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792426 | 07:33 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch federation_protocol module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792427 | 07:36 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch iuser_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792428 | 07:36 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch identity_user module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792429 | 07:36 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch identity_role module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792430 | 07:37 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch identity_group_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792431 | 07:38 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch identity_group module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792432 | 07:38 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch identity_domain_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792433 | 07:38 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch identity_domain module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792434 | 07:39 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch hostaggregate module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792435 | 07:39 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch group_assignment module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792436 | 07:39 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch floating_ip module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792437 | 07:40 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch federation_mapping_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792438 | 07:40 |
| *** gtema__ has joined #openstack-ansible-sig | 07:41 | |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch federation_mapping module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792439 | 07:42 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch federation_idp_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792440 | 07:42 |
| *** tremble has quit IRC | 07:43 | |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch federation_idp module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792441 | 07:43 |
| *** gtema_ has quit IRC | 07:43 | |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch endpoint module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792442 | 07:43 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch flavor_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792443 | 07:44 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch flavor module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792444 | 07:44 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch coe_cluster_template module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792445 | 07:44 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch coe_cluster module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792446 | 07:46 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch catalog_service module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792447 | 07:46 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch auth module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792448 | 07:47 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch Snapshot module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792449 | 07:48 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch Stack module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792450 | 07:51 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch ServerMetadata module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792451 | 07:51 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch ServerGroup module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792452 | 07:51 |
| openstackgerrit | Polina Gubina proposed openstack/ansible-collections-openstack master: Floating ip info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792548 | 09:13 |
| openstackgerrit | Polina Gubina proposed openstack/ansible-collections-openstack master: Floating ip info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792548 | 09:19 |
| *** gtema_ has joined #openstack-ansible-sig | 09:54 | |
| *** gtema__ has quit IRC | 09:57 | |
| *** itd27m01 has joined #openstack-ansible-sig | 10:03 | |
| *** gtema_ has quit IRC | 10:13 | |
| itd27m01 | Hi Guys. ansible-4.0.0 brokes the inventory in some way. The token for plugin should be in fqdn fromat (namespace.collection.plugin) but carrently it is just "openstack" | 10:14 |
| itd27m01 | https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/inventory/openstack.py#L25 | 10:14 |
| gtema | https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792138 | 10:15 |
| gtema | i think that addresses it | 10:15 |
| itd27m01 | Yes, it is, thank you | 10:15 |
| gtema | can you please verify it works for you and leave a review | 10:16 |
| gtema | it it works also for you I will approve it and we land it today | 10:17 |
| gtema | but not the release | 10:17 |
| itd27m01 | gtema What do you think about tests, should they be fixed accordingly? https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/tests/unit/plugins/inventory/test_openstack.py#L30 | 10:19 |
| gtema | actually yes, makes sense | 10:20 |
| itd27m01 | gtema Also it would be good to fix the example accordingly: https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/inventory/openstack.py#L115 | 10:23 |
| gtema | good points | 10:23 |
| itd27m01 | gtema because the documentation is not obvious currently: https://docs.ansible.com/ansible/latest/collections/openstack/cloud/openstack_inventory.html | 10:24 |
| gtema | I know. I actually was never using it personally | 10:24 |
| itd27m01 | gtema I can confirm that change works for me, could you highlight the points regarding tests and examples to the merge owners? | 10:33 |
| gtema | you can simply update the patch addressing the concerns | 10:33 |
| itd27m01 | Ah, yes, thank you | 10:34 |
| gtema | otherwise if I find time today I will do that | 10:34 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch Snapshot module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792449 | 11:48 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch Stack module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792450 | 11:55 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch ServerMetadata module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792451 | 11:56 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch ServerGroup module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792452 | 11:57 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch Quota module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792418 | 12:00 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch user_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792428 | 12:02 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch port module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792421 | 12:02 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch Recordset module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792417 | 12:04 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch identity_group module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792432 | 12:05 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch KeystoneFederationProtocolInfo module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792426 | 12:07 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch federation_protocol module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792427 | 12:12 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch group_assignment module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792436 | 12:13 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch federation_idp module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792441 | 12:14 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch identity_role module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792430 | 12:15 |
| openstackgerrit | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch identity_domain module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792434 | 12:15 |
| *** spatel_ has joined #openstack-ansible-sig | 13:24 | |
| *** spatel_ is now known as spatel | 13:24 | |
| *** sshnaidm|pto has quit IRC | 13:49 | |
| *** tremble has joined #openstack-ansible-sig | 14:35 | |
| *** tremble has left #openstack-ansible-sig | 14:40 | |
| *** sshnaidm has joined #openstack-ansible-sig | 14:50 | |
| *** sshnaidm is now known as sshnaidm|pto | 14:52 | |
| *** tremble has joined #openstack-ansible-sig | 15:04 | |
| *** itd27m01 has quit IRC | 16:20 | |
| *** spatel has quit IRC | 22:10 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!