Tuesday, 2020-03-03

*** slaweq has joined #openstack-sdks00:11
*** slaweq has quit IRC00:17
openstackgerritMerged openstack/python-openstackclient stable/rocky: Fix functional tests for py3  https://review.opendev.org/70368300:37
*** slaweq has joined #openstack-sdks01:11
*** yolanda has quit IRC01:15
*** slaweq has quit IRC01:16
*** mtreinish has quit IRC01:27
*** mtreinish has joined #openstack-sdks01:40
*** slaweq has joined #openstack-sdks02:11
*** slaweq has quit IRC02:15
*** enriquetaso has joined #openstack-sdks02:43
*** enriquetaso has quit IRC03:05
*** slaweq has joined #openstack-sdks03:11
*** slaweq has quit IRC03:16
*** dave-mccowan has quit IRC04:04
*** slaweq has joined #openstack-sdks04:11
*** ricolin has quit IRC04:15
*** slaweq has quit IRC04:17
*** slaweq has joined #openstack-sdks05:11
*** slaweq has quit IRC05:16
*** ccamel has quit IRC05:34
*** evrardjp has quit IRC05:35
*** evrardjp has joined #openstack-sdks05:35
*** camelCaser has joined #openstack-sdks05:37
*** enriquetaso has joined #openstack-sdks06:06
*** slaweq has joined #openstack-sdks06:11
*** slaweq has quit IRC06:16
*** enriquetaso has quit IRC06:51
*** slaweq has joined #openstack-sdks07:11
*** slaweq has quit IRC07:16
*** yolanda has joined #openstack-sdks07:21
*** ricolin has joined #openstack-sdks07:33
*** slaweq has joined #openstack-sdks07:52
*** tosky has joined #openstack-sdks08:20
*** iurygregory has joined #openstack-sdks08:26
*** stingrayza has joined #openstack-sdks08:34
*** iurygregory has quit IRC08:36
*** jpena|off is now known as jpena08:41
*** iurygregory has joined #openstack-sdks08:41
*** ralonsoh has joined #openstack-sdks08:42
*** sshnaidm|pto has quit IRC08:45
*** sshnaidm|pto has joined #openstack-sdks08:59
*** jpich has joined #openstack-sdks09:06
*** sshnaidm|pto is now known as sshnaidm09:17
*** iurygregory has quit IRC09:27
*** jpich has quit IRC09:33
*** jpich has joined #openstack-sdks09:34
*** iurygregory has joined #openstack-sdks09:43
*** dtantsur|afk is now known as dtantsur10:03
*** masterpe has quit IRC10:56
*** openstackstatus has quit IRC10:59
openstackgerritRiccardo Pittau proposed openstack/openstacksdk master: Import generate_temp_url from swiftclient  https://review.opendev.org/71097311:03
*** sshnaidm has quit IRC11:07
*** sshnaidm has joined #openstack-sdks11:12
*** sshnaidm has quit IRC11:13
*** sshnaidm has joined #openstack-sdks11:13
*** masterpe has joined #openstack-sdks11:20
*** enriquetaso has joined #openstack-sdks12:08
*** jpena is now known as jpena|lunch12:32
openstackgerritBernard Cafarelli proposed openstack/openstacksdk stable/train: Handle HTTP errors in add/remove router interface calls  https://review.opendev.org/71099412:41
openstackgerritBernard Cafarelli proposed openstack/openstacksdk stable/train: Fix error handling on add/remove router iface calls  https://review.opendev.org/71099512:41
openstackgerritMichaƂ Dulko proposed openstack/openstacksdk stable/train: Fix error handling in network trunks operations  https://review.opendev.org/71100113:10
*** enriquetaso has quit IRC13:13
KeithMnemonicCould I also please get some reviews on this change https://review.opendev.org/#/c/701066/13:15
*** jpena|lunch is now known as jpena13:32
*** zigo has quit IRC13:58
*** enriquetaso has joined #openstack-sdks14:14
*** senrique_ has joined #openstack-sdks14:24
*** iurygregory has quit IRC14:26
*** enriquetaso has quit IRC14:27
*** iurygregory has joined #openstack-sdks14:28
*** tkajinam has quit IRC14:30
*** tkajinam has joined #openstack-sdks14:31
*** zigo has joined #openstack-sdks14:33
openstackgerritRiccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node  https://review.opendev.org/71103214:46
openstackgerritRodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add "fields" parameter to ListSecurityGroup query  https://review.opendev.org/71082514:46
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Deal with collection build modifying tree  https://review.opendev.org/71103314:50
openstackgerritRodolfo Alonso Hernandez proposed openstack/openstacksdk master: Include "fields" to "SecurityGroup" query parameters  https://review.opendev.org/71082014:54
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Fix license metadata  https://review.opendev.org/71103514:55
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Clean up minor build quibbles  https://review.opendev.org/71103614:57
openstackgerritRiccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node  https://review.opendev.org/71103214:59
*** tkajinam has quit IRC15:09
*** KeithMnemonic has quit IRC15:16
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Add Openstack guidelines spec from Ansible  https://review.opendev.org/70455815:27
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Test with and assert support for python3  https://review.opendev.org/71064115:29
mordreddtantsur: https://review.opendev.org/#/c/711032 has your name all over it15:30
dtantsurdone!15:31
mordreddtantsur: also - looking at futurist (it does seem like a good idea to use it)15:33
mordreddtantsur: I don't suppose we could drop the prettytable dependency ...15:34
dtantsurmordred: I don't recall why we have it, honestly15:39
dtantsurmordred: I would +2 a change making it optional, seems really used in one corner case15:40
mordreddtantsur: https://review.opendev.org/711041 Make PrettyTable optional15:41
*** dave-mccowan has joined #openstack-sdks15:41
mordreddtantsur: 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 similar15:42
dtantsurmordred: one small suggestion inline, I remembered that this code path is often used by default15:43
*** dave-mccowan has quit IRC15:46
mordreddtantsur: updated - and good call15:48
mordredsmcginnis: ^^ have a second to be a second human?15:49
dtantsurmordred: sorry if I seem nitpicking, I've left another idea15:50
mordreddtantsur: I think in a project with zero other open changes it's worthwhile to nitpick15:50
mordreddtantsur: ooh, that's nice.15:51
smcginnismordred: Will look in a bit.15:56
smcginnisMaybe of interest - https://github.com/astanin/python-tabulate/commit/ba3c20364720421fccddb16bb6d9708fe89735b115:56
*** senrique_ has quit IRC16:02
mordreddtantsur: how about "statistics not available, PrettyTable missing"16:03
dtantsurmordred: ship it!16:03
smcginnis+116:03
mordreddtantsur, smcginnis: k. there we go16:04
dtantsur+216:05
mordredsmcginnis: looks decent enough - but it's still one-more-dep for a single print function16:05
smcginnisYeah, for this case not useful.16:06
smcginnisI have a feeling PrettyTable is going to start causing other problems soon though.16:06
smcginnisLast release was ages ago.16:06
mordredyeah16:06
mordredsmcginnis: although sometimes that's nice (/me looks at virtualenv)16:06
smcginnis;)16:06
dtantsur:D16:07
dtantsurhonestly, I'd probably rewrite the statistics reporting to a simpler format. It's not exactly user-facing..16:08
mordredyah16:09
openstackgerritRiccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node  https://review.opendev.org/71103216:26
openstackgerritMerged openstack/ansible-collections-openstack master: Just use shutil.rmtree in build.py  https://review.opendev.org/71064016:38
*** jpena is now known as jpena|brb16:46
*** senrique_ has joined #openstack-sdks16:50
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Add bindep file  https://review.opendev.org/71105516:55
*** dtantsur is now known as dtantsur|afk17:13
*** jpena|brb is now known as jpena17:25
*** jpich has quit IRC17:25
*** evrardjp has quit IRC17:35
*** evrardjp has joined #openstack-sdks17:35
openstackgerritMerged openstack/openstacksdk stable/train: Fix error handling in network trunks operations  https://review.opendev.org/71100118:08
openstackgerritMerged openstack/openstacksdk master: Include user_id attribute in volume information  https://review.opendev.org/71083418:09
*** jpena is now known as jpena|off18:41
cmurphymordred: could you review https://review.opendev.org/703707 for me? it's an easy one18:46
mordredcmurphy: done!18:57
*** senrique_ has quit IRC18:58
cmurphytyvm18:58
*** senrique_ has joined #openstack-sdks18:59
*** ralonsoh has quit IRC19:21
*** tosky has quit IRC20:22
mordredrm_work: ooh - the glance patches seem to work!20:22
rm_workyeah they passed CI, haven't tested myself yet tho20:23
rm_workwish i could take any credit for that XD20:23
mordredrm_work: well, they were red last time20:27
mordredthen you rebased them and they were green20:27
rm_work:D20:27
rm_workmagic!20:27
mordredI should test them by trying to upload an image to rackspace20:27
mordredsince that would not work before this patch20:28
*** senrique_ has quit IRC20:40
rm_workI bet they were red because SDK hadn't released an update including the code it needed yet20:41
rm_workbut that was in december, and now it has20:41
*** slaweq has quit IRC20:46
*** slaweq has joined #openstack-sdks21:11
*** slaweq has quit IRC21:16
*** slaweq has joined #openstack-sdks22:11
*** slaweq has quit IRC22:16
*** tkajinam has joined #openstack-sdks22:48
*** slaweq has joined #openstack-sdks23:11
*** slaweq has quit IRC23:16
openstackgerritMerged openstack/python-openstackclient master: Fix copypaste errors in access rule command  https://review.opendev.org/70370723:40

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!