*** slaweq has joined #openstack-sdks | 00:11 | |
*** slaweq has quit IRC | 00:17 | |
openstackgerrit | Merged openstack/python-openstackclient stable/rocky: Fix functional tests for py3 https://review.opendev.org/703683 | 00:37 |
---|---|---|
*** slaweq has joined #openstack-sdks | 01:11 | |
*** yolanda has quit IRC | 01:15 | |
*** slaweq has quit IRC | 01:16 | |
*** mtreinish has quit IRC | 01:27 | |
*** mtreinish has joined #openstack-sdks | 01:40 | |
*** slaweq has joined #openstack-sdks | 02:11 | |
*** slaweq has quit IRC | 02:15 | |
*** enriquetaso has joined #openstack-sdks | 02:43 | |
*** enriquetaso has quit IRC | 03:05 | |
*** slaweq has joined #openstack-sdks | 03:11 | |
*** slaweq has quit IRC | 03:16 | |
*** dave-mccowan has quit IRC | 04:04 | |
*** slaweq has joined #openstack-sdks | 04:11 | |
*** ricolin has quit IRC | 04:15 | |
*** slaweq has quit IRC | 04:17 | |
*** slaweq has joined #openstack-sdks | 05:11 | |
*** slaweq has quit IRC | 05:16 | |
*** ccamel has quit IRC | 05:34 | |
*** evrardjp has quit IRC | 05:35 | |
*** evrardjp has joined #openstack-sdks | 05:35 | |
*** camelCaser has joined #openstack-sdks | 05:37 | |
*** enriquetaso has joined #openstack-sdks | 06:06 | |
*** slaweq has joined #openstack-sdks | 06:11 | |
*** slaweq has quit IRC | 06:16 | |
*** enriquetaso has quit IRC | 06:51 | |
*** slaweq has joined #openstack-sdks | 07:11 | |
*** slaweq has quit IRC | 07:16 | |
*** yolanda has joined #openstack-sdks | 07:21 | |
*** ricolin has joined #openstack-sdks | 07:33 | |
*** slaweq has joined #openstack-sdks | 07:52 | |
*** tosky has joined #openstack-sdks | 08:20 | |
*** iurygregory has joined #openstack-sdks | 08:26 | |
*** stingrayza has joined #openstack-sdks | 08:34 | |
*** iurygregory has quit IRC | 08:36 | |
*** jpena|off is now known as jpena | 08:41 | |
*** iurygregory has joined #openstack-sdks | 08:41 | |
*** ralonsoh has joined #openstack-sdks | 08:42 | |
*** sshnaidm|pto has quit IRC | 08:45 | |
*** sshnaidm|pto has joined #openstack-sdks | 08:59 | |
*** jpich has joined #openstack-sdks | 09:06 | |
*** sshnaidm|pto is now known as sshnaidm | 09:17 | |
*** iurygregory has quit IRC | 09:27 | |
*** jpich has quit IRC | 09:33 | |
*** jpich has joined #openstack-sdks | 09:34 | |
*** iurygregory has joined #openstack-sdks | 09:43 | |
*** dtantsur|afk is now known as dtantsur | 10:03 | |
*** masterpe has quit IRC | 10:56 | |
*** openstackstatus has quit IRC | 10:59 | |
openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Import generate_temp_url from swiftclient https://review.opendev.org/710973 | 11:03 |
*** sshnaidm has quit IRC | 11:07 | |
*** sshnaidm has joined #openstack-sdks | 11:12 | |
*** sshnaidm has quit IRC | 11:13 | |
*** sshnaidm has joined #openstack-sdks | 11:13 | |
*** masterpe has joined #openstack-sdks | 11:20 | |
*** enriquetaso has joined #openstack-sdks | 12:08 | |
*** jpena is now known as jpena|lunch | 12:32 | |
openstackgerrit | Bernard Cafarelli proposed openstack/openstacksdk stable/train: Handle HTTP errors in add/remove router interface calls https://review.opendev.org/710994 | 12:41 |
openstackgerrit | Bernard Cafarelli proposed openstack/openstacksdk stable/train: Fix error handling on add/remove router iface calls https://review.opendev.org/710995 | 12:41 |
openstackgerrit | MichaĆ Dulko proposed openstack/openstacksdk stable/train: Fix error handling in network trunks operations https://review.opendev.org/711001 | 13:10 |
*** enriquetaso has quit IRC | 13:13 | |
KeithMnemonic | Could I also please get some reviews on this change https://review.opendev.org/#/c/701066/ | 13:15 |
*** jpena|lunch is now known as jpena | 13:32 | |
*** zigo has quit IRC | 13:58 | |
*** enriquetaso has joined #openstack-sdks | 14:14 | |
*** senrique_ has joined #openstack-sdks | 14:24 | |
*** iurygregory has quit IRC | 14:26 | |
*** enriquetaso has quit IRC | 14:27 | |
*** iurygregory has joined #openstack-sdks | 14:28 | |
*** tkajinam has quit IRC | 14:30 | |
*** tkajinam has joined #openstack-sdks | 14:31 | |
*** zigo has joined #openstack-sdks | 14:33 | |
openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node https://review.opendev.org/711032 | 14:46 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add "fields" parameter to ListSecurityGroup query https://review.opendev.org/710825 | 14:46 |
openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Deal with collection build modifying tree https://review.opendev.org/711033 | 14:50 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: Include "fields" to "SecurityGroup" query parameters https://review.opendev.org/710820 | 14:54 |
openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Fix license metadata https://review.opendev.org/711035 | 14:55 |
openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Clean up minor build quibbles https://review.opendev.org/711036 | 14:57 |
openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node https://review.opendev.org/711032 | 14:59 |
*** tkajinam has quit IRC | 15:09 | |
*** KeithMnemonic has quit IRC | 15:16 | |
openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Add Openstack guidelines spec from Ansible https://review.opendev.org/704558 | 15:27 |
openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Test with and assert support for python3 https://review.opendev.org/710641 | 15:29 |
mordred | dtantsur: https://review.opendev.org/#/c/711032 has your name all over it | 15:30 |
dtantsur | done! | 15:31 |
mordred | dtantsur: also - looking at futurist (it does seem like a good idea to use it) | 15:33 |
mordred | dtantsur: I don't suppose we could drop the prettytable dependency ... | 15:34 |
dtantsur | mordred: I don't recall why we have it, honestly | 15:39 |
dtantsur | mordred: I would +2 a change making it optional, seems really used in one corner case | 15:40 |
mordred | dtantsur: https://review.opendev.org/711041 Make PrettyTable optional | 15:41 |
*** dave-mccowan has joined #openstack-sdks | 15:41 | |
mordred | dtantsur: I think with that it's a great idea to use futurist in sdk - and if we do that global executor idea it should make it really nice for people instantiating a Connection in an eventlet service or similar | 15:42 |
dtantsur | mordred: one small suggestion inline, I remembered that this code path is often used by default | 15:43 |
*** dave-mccowan has quit IRC | 15:46 | |
mordred | dtantsur: updated - and good call | 15:48 |
mordred | smcginnis: ^^ have a second to be a second human? | 15:49 |
dtantsur | mordred: sorry if I seem nitpicking, I've left another idea | 15:50 |
mordred | dtantsur: I think in a project with zero other open changes it's worthwhile to nitpick | 15:50 |
mordred | dtantsur: ooh, that's nice. | 15:51 |
smcginnis | mordred: Will look in a bit. | 15:56 |
smcginnis | Maybe of interest - https://github.com/astanin/python-tabulate/commit/ba3c20364720421fccddb16bb6d9708fe89735b1 | 15:56 |
*** senrique_ has quit IRC | 16:02 | |
mordred | dtantsur: how about "statistics not available, PrettyTable missing" | 16:03 |
dtantsur | mordred: ship it! | 16:03 |
smcginnis | +1 | 16:03 |
mordred | dtantsur, smcginnis: k. there we go | 16:04 |
dtantsur | +2 | 16:05 |
mordred | smcginnis: looks decent enough - but it's still one-more-dep for a single print function | 16:05 |
smcginnis | Yeah, for this case not useful. | 16:06 |
smcginnis | I have a feeling PrettyTable is going to start causing other problems soon though. | 16:06 |
smcginnis | Last release was ages ago. | 16:06 |
mordred | yeah | 16:06 |
mordred | smcginnis: although sometimes that's nice (/me looks at virtualenv) | 16:06 |
smcginnis | ;) | 16:06 |
dtantsur | :D | 16:07 |
dtantsur | honestly, I'd probably rewrite the statistics reporting to a simpler format. It's not exactly user-facing.. | 16:08 |
mordred | yah | 16:09 |
openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node https://review.opendev.org/711032 | 16:26 |
openstackgerrit | Merged openstack/ansible-collections-openstack master: Just use shutil.rmtree in build.py https://review.opendev.org/710640 | 16:38 |
*** jpena is now known as jpena|brb | 16:46 | |
*** senrique_ has joined #openstack-sdks | 16:50 | |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Add bindep file https://review.opendev.org/711055 | 16:55 |
*** dtantsur is now known as dtantsur|afk | 17:13 | |
*** jpena|brb is now known as jpena | 17:25 | |
*** jpich has quit IRC | 17:25 | |
*** evrardjp has quit IRC | 17:35 | |
*** evrardjp has joined #openstack-sdks | 17:35 | |
openstackgerrit | Merged openstack/openstacksdk stable/train: Fix error handling in network trunks operations https://review.opendev.org/711001 | 18:08 |
openstackgerrit | Merged openstack/openstacksdk master: Include user_id attribute in volume information https://review.opendev.org/710834 | 18:09 |
*** jpena is now known as jpena|off | 18:41 | |
cmurphy | mordred: could you review https://review.opendev.org/703707 for me? it's an easy one | 18:46 |
mordred | cmurphy: done! | 18:57 |
*** senrique_ has quit IRC | 18:58 | |
cmurphy | tyvm | 18:58 |
*** senrique_ has joined #openstack-sdks | 18:59 | |
*** ralonsoh has quit IRC | 19:21 | |
*** tosky has quit IRC | 20:22 | |
mordred | rm_work: ooh - the glance patches seem to work! | 20:22 |
rm_work | yeah they passed CI, haven't tested myself yet tho | 20:23 |
rm_work | wish i could take any credit for that XD | 20:23 |
mordred | rm_work: well, they were red last time | 20:27 |
mordred | then you rebased them and they were green | 20:27 |
rm_work | :D | 20:27 |
rm_work | magic! | 20:27 |
mordred | I should test them by trying to upload an image to rackspace | 20:27 |
mordred | since that would not work before this patch | 20:28 |
*** senrique_ has quit IRC | 20:40 | |
rm_work | I bet they were red because SDK hadn't released an update including the code it needed yet | 20:41 |
rm_work | but that was in december, and now it has | 20:41 |
*** slaweq has quit IRC | 20:46 | |
*** slaweq has joined #openstack-sdks | 21:11 | |
*** slaweq has quit IRC | 21:16 | |
*** slaweq has joined #openstack-sdks | 22:11 | |
*** slaweq has quit IRC | 22:16 | |
*** tkajinam has joined #openstack-sdks | 22:48 | |
*** slaweq has joined #openstack-sdks | 23:11 | |
*** slaweq has quit IRC | 23:16 | |
openstackgerrit | Merged openstack/python-openstackclient master: Fix copypaste errors in access rule command https://review.opendev.org/703707 | 23:40 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!