| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-12-13 | |||
| 17:32:35 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Support uploading image from data and stdin https://review.opendev.org/698972 | |
| 17:33:21 | gtema | need to fall out. See ya | |
| 17:34:17 | mriedem | "_MIGHT__" - the api-ref wording shouldn't probably say that | |
| 17:34:21 | mriedem | it will be an empty string if bfv | |
| 17:35:07 | mriedem | https://github.com/openstack/nova/blob/2f0f8b8cf13b7441ce6a7a524b33d30d40f1d468/nova/api/openstack/compute/views/servers.py#L540 | |
| 17:37:47 | mriedem | https://review.opendev.org/#/c/698976/ | |
| 17:40:17 | gtema | mriedem, according to this block image_ref is populated in my case when BFV. Can this happen in some cases? | |
| 17:40:36 | gtema | I have no access to sources, but will try to figure out | |
| 17:41:38 | mriedem | it shouldn't be i don't think https://github.com/openstack/nova/blob/2f0f8b8cf13b7441ce6a7a524b33d30d40f1d468/nova/compute/api.py#L1433 | |
| 17:41:50 | gtema | ok | |
| 17:42:04 | gtema | means some extra patch on my cloud | |
| 17:42:14 | mriedem | maybe a problem when creating a server with an image specified and a bdm list, or that | |
| 17:42:26 | mriedem | but when booting from volume the specified imageRef should be ignored | |
| 21:10:14 | openstackgerrit | Keith Berger proposed openstack/python-openstackclient master: WIP:Fix openstack server list --deleted --marker option https://review.opendev.org/691045 | |
| 23:12:18 | openstackgerrit | Keith Berger proposed openstack/python-openstackclient master: WIP:Fix openstack server list --deleted --marker option https://review.opendev.org/691045 | |
| #openstack-sdks - 2019-12-14 | |||
| 14:14:19 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Support uploading image from data and stdin https://review.opendev.org/698972 | |
| 14:36:07 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374 | |
| 15:56:51 | mordred | dtantsur|afk: if you get a chance, could you re-review https://review.opendev.org/#/c/698972 ? | |
| #openstack-sdks - 2019-12-15 | |||
| 13:43:50 | openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: [ussuri][goal] Drop python 2.7 support and testing https://review.opendev.org/699121 | |
| 15:19:38 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove python2 from project-template https://review.opendev.org/699130 | |
| 15:21:16 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Stop testing python2 https://review.opendev.org/686590 | |
| 15:21:17 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Remove python2 from project-template https://review.opendev.org/699131 | |
| 18:24:54 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Stop running python2 test jobs https://review.opendev.org/699132 | |
| 18:24:55 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove extra python2 test jobs https://review.opendev.org/699133 | |
| 18:24:55 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Switch stable ansible job to 2.8 https://review.opendev.org/699134 | |
| #openstack-sdks - 2019-12-16 | |||
| 15:58:33 | mordred | Shrews: if you're bored there's a stack of patches ending in https://review.opendev.org/#/c/699134 that should be happy making | |
| 15:59:11 | mordred | Shrews: although I think I should maybe add a release note to one of them | |
| 16:00:44 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Stop supporting python2 https://review.opendev.org/699132 | |
| 16:00:45 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove extra python2 test jobs https://review.opendev.org/699133 | |
| 16:00:45 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Switch stable ansible job to 2.8 https://review.opendev.org/699134 | |
| 16:01:11 | mordred | gtema: ^^ that stack was green - but I just added a release note to 699132 | |
| 16:01:41 | gtema | oki | |
| 16:20:19 | Shrews | mordred: ok | |
| 16:33:14 | openstackgerrit | Keith Berger proposed openstack/python-openstackclient master: WIP:Fix openstack server list --deleted --marker option https://review.opendev.org/691045 | |
| 19:34:44 | openstackgerrit | Keith Berger proposed openstack/python-openstackclient master: Fix openstack server list --deleted --marker option https://review.opendev.org/691045 | |
| 19:59:32 | openstackgerrit | Merged openstack/openstacksdk master: Remove python2 from project-template https://review.opendev.org/699130 | |
| 21:49:17 | openstackgerrit | Keith Berger proposed openstack/python-openstackclient master: Fix openstack server list --deleted --marker option https://review.opendev.org/691045 | |
| #openstack-sdks - 2019-12-17 | |||
| 13:40:51 | mordred | gtema: heya! - wanna +A https://review.opendev.org/#/c/699132 ? | |
| 13:41:18 | gtema | done | |
| 13:41:36 | gtema | a milestone with mixed feeling | |
| 13:41:39 | mordred | \o/ | |
| 13:41:54 | mordred | python2 has served us well for many years | |
| 13:42:15 | gtema | sure, and it is still used in the backend of perhaps lots of installations | |
| 13:42:20 | mordred | yup | |
| 13:42:25 | gtema | at least unfortunately my cloud case | |
| 13:43:27 | mordred | yeah - I really wish a default transition to 3 had happened sooner | |
| 13:43:56 | gtema | yes, and perhaps even "harder" to force people | |
| 13:43:59 | mordred | I remember the first python3 summit session - during the *grizzly* summit | |
| 13:44:13 | Shrews | https://pythonclock.org | |
| 13:44:24 | mordred | we could never quite get people to take this one proactively | |
| 13:45:11 | mordred | although I blame rhel7 not shipping with even an optional python3 as a large part of the problem | |
| 13:45:24 | gtema | agree | |
| 13:45:29 | mordred | because for many people that made opting in to python3 a non-starter | |
| 13:46:04 | gtema | and the one available through EPEL (3.4) is not a good deal either | |
| 13:47:01 | mordred | yup | |
| 13:47:37 | mordred | this is an area where I really do think containers improve the story for many use cases | |
| 13:47:41 | mordred | not all | |
| 13:47:42 | mordred | but many | |
| 13:48:05 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374 | |
| 13:48:06 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: Complete switch from glanceclient to SDK for image service https://review.opendev.org/699416 | |
| 13:48:22 | mordred | we've been working on switching gerrit to run in a container - and one of the big wins is that we don't have to care what version of java is shipped with the OS we're running on the gerrit server :) | |
| 13:49:05 | gtema | oh god, Java is the same for me. There are definitely still lots of Enterprises running JEE1.6 | |
| 13:49:26 | mordred | yup | |
| 13:49:43 | mordred | and - you know - that's fine - build it in your container image and stop caring about it :) | |
| 13:49:57 | mordred | dtroyer: heya - got a sec for an easy one: https://review.opendev.org/#/c/695427/ ? | |
| 13:50:16 | gtema | he, in environments, which still depend on J1.6 there is most likely no container support as well | |
| 13:51:56 | gtema | mordred: please have a look at https://review.opendev.org/#/c/698972/ for stdin support image upload in SDK | |
| 13:52:39 | mordred | gtema: yes - I'm excited about that! (was waiting for the jobs to go green) | |
| 13:53:15 | gtema | they are actually green, remaining red ones (non-voting) are not related failures | |
| 13:53:28 | mordred | Shrews: ^^ that one from gtema is needef for replacing glanceclient with sdk in osc | |
| 13:54:13 | gtema | I was actually a bit disappointed with summary from Glance team after summit. They claimed to have removed all OSC references from their docs | |
| 13:54:21 | gtema | and went back to "glance" | |
| 13:56:30 | mordred | gtema: question inline - it seems liek the hashes thing can't quite work right, no? | |
| 13:56:56 | mordred | gtema: that's very sad | |
| 13:57:18 | gtema | exactly, that's why in stdin case I'm passing validate_checksum as False | |
| 13:58:03 | gtema | surely it's sad, but I have no clue how else can we workaround this | |
| 13:58:20 | mordred | oh - nod. yeah | |
| 13:58:38 | mordred | gtema: I think that's just life if you decide that for some reason you need to stream the image over stdin | |
| 13:58:46 | gtema | exactly | |
| 14:00:11 | gtema | so basically if we are in stdin we should not come into this calculation logic due to if isinstance(data, bytes) | |
| 14:01:25 | mordred | gtema: that patch looks great - definitely less scary than I was thinking it would be | |
| 14:01:29 | mordred | oh - wait | |
| 14:01:35 | mordred | you should add a release note for that perhaps | |
| 14:01:41 | gtema | me as well | |
| 14:01:44 | gtema | ok | |
| 14:02:02 | mordred | since that'll be a key feature osc depends on | |
| 14:03:13 | Shrews | mordred: will look when I return from the gym | |
| 14:03:19 | mordred | Shrews: ew. gym | |
| 14:05:15 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Support uploading image from data and stdin https://review.opendev.org/698972 | |
| 14:05:20 | gtema | added RN only | |
| 14:07:31 | mordred | gtema: also - https://review.opendev.org/#/c/679914 looks good to me when you have a sec | |
| 14:07:49 | gtema | okay | |
| 14:12:08 | gtema | mordred: having a brief look there - in resources attributes are not in alphabetical order. While don't want to be neat-picky, do we want to get it this way? | |
| 14:13:44 | gtema | if it's ok for me - I can skip this comment | |
| 14:14:18 | gtema | I mean if ok for you | |
| 14:39:32 | mordred | gtema: I don't know that I care too much - I doubt a ton of people are going to be adding new attributes in parallel | |
| 14:39:47 | mordred | so I doubt we'll have too many merge conflicts | |
| 14:40:32 | gtema | it'S not about conflicts, it's more about keeping code clean. Especially if you look at image resources, where list of attrs take few pages - it'S really hard to orient | |
| 14:44:41 | mordred | nod. I'm also supportive of that comment if you want to make it | |
| 14:46:02 | gtema | Let us then +A this change with comment to order attributes. And in future really take care of this during review | |