Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-06-25
08:25:18 openstackgerrit Vishakha Agarwal proposed openstack/openstacksdk master: Add access rule CRUD support https://review.opendev.org/734549
12:32:11 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Upload image via interop import if needed https://review.opendev.org/737608
12:32:11 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for multiple image stores https://review.opendev.org/737845
12:33:46 mordred gtema: ^^ interop import patch there should be working now - I've checked https://review.opendev.org/#/c/734184/ - which is a nova patch that exercises it in the gate via openstackclient
12:34:25 mordred gtema: silly me, I thought type=list would turn a comma separated header into a list
12:34:25 gtema oki
12:34:38 gtema do we have our tests repaired?
12:35:10 mordred gtema: looks like you rechecked the slaweq patch
12:35:16 gtema yes
12:35:29 gtema I see in zuul the only remaining job is networking
12:35:37 mordred fingers crossed
12:35:41 gtema hopefully it will go through this time
12:36:43 gtema I see you now follow the formatting style of the black for new functions
12:36:45 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Drop python3.5 support https://review.opendev.org/736058
12:36:46 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Upload image via interop import if needed https://review.opendev.org/737608
12:37:05 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for multiple image stores https://review.opendev.org/737845
12:37:23 mordred gtema: I rebased those on top of the network fix
12:37:28 mordred gtema: yes - I really like it actually
12:37:31 gtema oki
12:37:35 gtema yeah, mee to
12:37:48 mordred I think it's great for later refactoring to touch fewer lines
12:37:56 gtema agree
12:38:07 mordred I *almost* want to reformat our codebase to black
12:38:11 mordred but - like - oy
12:38:19 gtema :D
13:20:04 mordred gtema: ok - so I just went down a time-wasting hole about linters and black - and wound up adding flake8-bugbear ... which actually did catch some new bugs :(
13:20:24 gtema oh, that's interesting
13:20:40 mordred yeah. we have a bunch of {} in argument lists
13:21:29 gtema okay
13:22:09 frickler mordred: did you see http://lists.openstack.org/pipermail/openstack-discuss/2020-June/015610.html ?
13:29:20 mordred frickler: no, I didn't - but thanks, that should be fixable
13:43:28 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Provide hypervisor_hostname in Hypervisor object https://review.opendev.org/738026
14:20:11 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add use of flake8-bugbear https://review.opendev.org/738041
14:21:11 mordred gtema: ^^ enjoy
14:21:27 gtema thks, already noticed amount of files
14:21:29 mordred gtema: (there's also a bunch of "loop var not used in loop" warnings fixed)
14:21:49 gtema good
14:21:56 mordred but all of the mutable arguments are, you know, broken
14:22:08 gtema yupp
14:25:38 gtema looks good. Let's see what tests will say
16:02:26 elmiko API SIG office hour open
16:05:42 elmiko well, today's the big day. there have been no objections on the version discovery docs and i am going to merge them =)
16:05:45 elmiko \o/
16:13:05 dtantsur \o/
16:13:19 dtantsur hey elmiko, how's weather there? we're boiling.
16:13:38 gtema_ oh yeah, boiling
16:13:41 elmiko it's really nice here, clear skies, feels like about 20-21C
16:14:02 elmiko supposed to go up to 24-25C later though
16:15:06 dtantsur awesome (both)
16:15:06 elmiko our summers are usually very mild where i live though, we rarely go above 30-32C
16:15:18 dtantsur 30-32 is what we have now
16:15:22 elmiko ugh
16:15:26 elmiko i'm melting...
16:15:55 elmiko otoh, air conditioning is basically default here. most folks have strong central ac
16:17:13 openstackgerrit Merged openstack/api-sig master: Add document describing consuming version discovery https://review.opendev.org/459405
16:17:14 openstackgerrit Merged openstack/api-sig master: Add guidelines on Version Discovery https://review.opendev.org/459710
16:17:53 dtantsur elmiko: that's useful :) AC is pretty much non-existing here, people assume we don't have enough hot days
16:18:06 dtantsur "Hold my beer", said the global warming :)
16:20:12 elmiko haha, totally
16:20:34 elmiko mordred: ^^ it merged!!!!
16:20:46 elmiko i'm just sad that cdent and edleafe aren't here to see it
16:20:54 dtantsur indeed
16:21:13 elmiko oh, we need to decide if we want to have a slide in the retrospective thing
16:21:36 mordred elmiko: ZOMG
16:21:38 dtantsur right
16:21:54 dtantsur maybe just funny pictures of all us involved? :)
16:21:58 elmiko i'm not sure we really have enough material to warrant it, but maybe we could get a mention from the sdk folks?
16:22:02 elmiko hahaha
16:22:04 elmiko i like that
16:24:53 dtantsur otherwise we could have a short description of what we do. dunno if it's of any interest for anyone nowadays..
16:25:02 elmiko yeah, that's my concern also
16:25:21 elmiko we aren't very active these days and i'm not sure there is much community excitement for our original mission
16:26:40 dtantsur yup
16:26:52 dtantsur maybe we should skip it..
16:27:27 elmiko i tend to agree
16:27:41 elmiko i'll send an email to Helena to let her know what we've decided
18:40:55 openstackgerrit Merged openstack/openstacksdk master: Remove neutron-fwaas from the jobs' required project https://review.opendev.org/735858
18:49:53 mordred \o/
#openstack-sdks - 2020-06-26
01:13:37 openstackgerrit Merged openstack/python-openstackclient master: port: add --host to list command https://review.opendev.org/736005
05:48:34 openstackgerrit Merged openstack/openstacksdk master: Drop python3.5 support https://review.opendev.org/736058
10:13:20 openstackgerrit Merged openstack/openstacksdk master: baremetal: support for volume connectors API https://review.opendev.org/718746
10:44:03 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Update 'openstack server create --config-drive' value https://review.opendev.org/738182
12:16:55 openstackgerrit Vishakha Agarwal proposed openstack/openstacksdk master: Add access rule CRUD support https://review.opendev.org/734549
12:19:14 openstackgerrit Vishakha Agarwal proposed openstack/openstacksdk master: Add access rule CRUD support https://review.opendev.org/734549
16:15:52 mordred gtema_: so - I've got a weird one ... in the image import patch, I added image_import_methods as a Header attribute. the whole thing worked and dansmith was able to test the import stuff in his nova patch. BUT - the image_import_methods value got added to the image as properties with a key of the original header name
16:16:08 mordred gtema_: http://eavesdrop.openstack.org/irclogs/%23openstack-glance/%23openstack-glance.2020-06-25.log.html#t2020-06-25T21:57:50
16:16:19 mordred I'm going to fix that just with a special case ... but it seems like we should perhaps invent a way to say "this is a read-only attribute never attempt to send it" - but also why would we have sent a Header as part of the body? (I'm assuming the uknown attributes code for image is involved here)
16:16:48 gtema_ ugh, ogh
16:17:08 gtema_ tough question at the end of the work week
16:17:46 gtema_ unknown_attributes should have not affected headers, those are only for body
16:18:19 gtema_ this __image__ stuff
16:18:39 gtema_ readonly attributes make totally sense for me
16:24:46 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Upload image via interop import if needed https://review.opendev.org/737608
16:25:03 mordred gtema_: there is a different approach from listing this particular thing as a Resource attribute
16:25:35 gtema_ ok, might be
16:25:50 mordred I'll try to think of some sort of readonly something - as well as try to figure out why the header attribute got added to properties
16:27:58 gtema_ good, thks
22:08:34 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Upload image via interop import if needed https://review.opendev.org/737608
22:08:34 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for multiple image stores https://review.opendev.org/737845
22:08:35 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add use of flake8-bugbear https://review.opendev.org/738041

Earlier   Later