Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-12-16
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
14:46:11 gtema dtantsur - comments?
14:47:02 mordred gtema: ++
14:49:30 openstackgerrit Vishakha Agarwal proposed openstack/keystoneauth master: [ussuri][goal] Drop python 2.7 support and testing https://review.opendev.org/699121
15:54:56 gtema thanks Shrews
15:55:05 Shrews np
17:12:34 openstackgerrit Keith Berger proposed openstack/python-openstackclient master: Fix openstack server list --deleted --marker option https://review.opendev.org/691045
20:02:31 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Fix openstack server list --deleted --marker option https://review.opendev.org/691045
20:34:06 openstackgerrit Merged openstack/openstacksdk master: Support uploading image from data and stdin https://review.opendev.org/698972
22:59:53 openstackgerrit Merged openstack/openstacksdk master: Stop supporting python2 https://review.opendev.org/699132
23:36:26 openstackgerrit Merged openstack/openstacksdk master: Remove extra python2 test jobs https://review.opendev.org/699133
23:47:43 openstackgerrit Merged openstack/openstacksdk master: Switch stable ansible job to 2.8 https://review.opendev.org/699134
#openstack-sdks - 2019-12-18
07:28:56 openstackgerrit wangfaxin proposed openstack/api-sig master: update reviewer url. https://review.opendev.org/699570
10:23:03 openstackgerrit Bo Tran proposed openstack/openstacksdk master: Add return cluster_id when query actions list https://review.opendev.org/699650
11:56:53 sshnaidm mordred, dtantsur Shrews pabelanger gtema odyssey4me and all others I'd like to have your opinions about the subject, in your time: http://lists.openstack.org/pipermail/openstack-discuss/2019-December/011597.html
11:58:41 gtema ack
13:48:00 openstackgerrit Merged openstack/cliff master: Add autoprogram_cliff_app_dist_name config opt https://review.opendev.org/692464
18:26:19 openstackgerrit Merged openstack/openstacksdk master: Add dns_publish_fixed_ip attribute to subnets https://review.opendev.org/679833
18:46:40 openstackgerrit David Shaughnessy proposed openstack/python-openstackclient master: [POC] Neutron Classifier CLI Mock up of Neutron Classifier CLI https://review.opendev.org/687256
18:46:41 openstackgerrit David Shaughnessy proposed openstack/python-openstackclient master: Extension to the DSCP Marking Rule This extends the CLI to include the classification_group_id and the classification_priority value for deciding the relitive priority of the rules. https://review.opendev.org/699832
#openstack-sdks - 2019-12-19
02:17:19 adriant mordred, it appears that the SDK is busted when setting metadata on objectstorage container creation
02:17:46 adriant none of the header values are being marked as dirty, even though the resource does have them
02:18:19 adriant I have a feeling the issue is because of: https://github.com/openstack/openstacksdk/blob/master/openstack/object_store/v1/container.py#L116
02:18:37 adriant any reason why you are setting "_synchronized=True" ?
02:18:45 adriant when by default, it never is?
02:20:24 adriant when I do: conn.object_store.create_container("test_5", read_ACL="stuff")
02:20:43 adriant I'd expect that read_ACL to be set, but it isn't, because while it does get added to resource headers, it isn't dirty
02:22:05 adriant yeah... based on your own docs. this is wrong: https://github.com/openstack/openstacksdk/blob/master/openstack/resource.py#L876-L880

Earlier   Later