Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-03-26
15:53:27 gtema mordred, so I rebase vendor patch on top of 715249, right?
15:54:45 smcginnis I will watch for the release and try to get it through asap.
15:54:51 smcginnis And the requirements update.
15:55:11 vphilippon @mordred: Alrighty, thanks a lot for the quick reaction. And thanks to everyone else too :)
15:55:54 tremble gtema, to_dict() still seems to exclude the resource.URI entry
15:56:29 gtema ah, and you need those explicitely?
15:56:49 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Change default image type in the OTC vendor profile https://review.opendev.org/715183
15:56:49 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add requires-python entry indicating 3.6 is required https://review.opendev.org/715243
15:57:03 mordred gtema: rebase done ^^
15:57:09 gtema ok, thks mordred
15:57:17 mordred gtema: I'm guessing this is for returning data to an ansible module - which needs json-able dict
15:57:40 mordred smcginnis: I think the constraints patch that's there for us already is now safe
15:58:15 mordred smcginnis: but - we can also just wait to reduce churn
15:58:21 mordred smcginnis: I'll get release patches up asap
15:58:34 smcginnis OK. I need to update that so it's python_version aware. Might as well wait a bit.
15:59:11 tremble gtema, I wanted one for completeness, I can manually copy it over
15:59:34 gtema we can add a parameter for it in to_dict if necessary
15:59:59 tremble yeah, which then requires another release before I could use it in my Ansible module :)
16:00:13 gtema sure :D
16:00:35 elmiko API SIG office hour now open \o/
16:00:39 gtema lots of releases last days, therefore you might need to wait just one day, or one hour
16:00:40 mordred we're killing tremble here
16:00:51 tremble It'd be nice as a parameter, but I wouldn't prioritise it.
16:02:28 openstackgerrit Duc Truong proposed openstack/openstacksdk master: Remove kwargs validation for identity project updates https://review.opendev.org/715255
16:03:27 openstackgerrit Duc Truong proposed openstack/openstacksdk master: Remove kwargs validation for identity project updates https://review.opendev.org/715255
16:04:16 openstackgerrit Duc Truong proposed openstack/openstacksdk master: Remove kwargs validation for identity project updates https://review.opendev.org/715255
16:04:32 tremble gtema, I just got really confused because I tried to do "dict(my_resource)" and some of the parameters disappears compared to just passing the resource directly to Ansible's "exit_json"
16:05:12 gtema oh yes, this is really fun - was also enjoying it writing some modules
16:09:00 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991
16:09:01 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992
16:13:14 tremble gtema, On the plus side my integration tests actually caught it :)
16:14:41 gtema ah, there is one more thing to remember - switch on proxy in the cloud layer is not complete, therefore you might have different object types (munch vs resource) for different resources when using cloud layer
16:16:01 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to include URI attributes in the resource dict https://review.opendev.org/715256
16:17:08 gtema I was starting to work on that, but there are everywhere things to clear, so stuck at some point
16:17:18 gtema will try to revive this
16:17:25 mordred yeah - same
16:17:44 gtema ah, now I know what was the problem - resource object is not cacheable
16:18:01 gtema and this is where we stuck wanting to discuss at some point
16:25:17 mordred oh right
16:26:16 gtema I guess make sense to discuss it before PTG :D
16:30:28 gtema ok guys, need to go off while there is still bit of sun outside
16:31:13 elmiko carpe them diems ;)
16:31:21 gtema thks
16:44:12 openstackgerrit Duc Truong proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266
17:04:21 elmiko have a good weekend all, stay safe out there o/
17:09:28 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444
17:09:28 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: [WIP] Add immutable option https://review.opendev.org/715269
17:13:54 openstackgerrit Merged openstack/python-openstackclient master: Add libc6-dev to bindep https://review.opendev.org/715210
17:25:38 openstackgerrit Merged openstack/python-openstackclient master: Be explicit about python version in image https://review.opendev.org/715211
18:15:40 openstackgerrit Duc Truong proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266
19:14:32 openstackgerrit Merged openstack/ansible-collections-openstack master: New Module: os_keystone_mapping https://review.opendev.org/713643
20:22:57 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Ignore more things for the final artifact https://review.opendev.org/715302
20:34:29 openstackgerrit Monty Taylor proposed openstack/cliff master: Re-add support for python 3.5 https://review.opendev.org/715305
20:36:32 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add requires-python entry indicating 3.5 is required https://review.opendev.org/715243
20:37:26 openstackgerrit Monty Taylor proposed openstack/cliff master: Re-add support for python 3.5 https://review.opendev.org/715305
20:38:19 openstackgerrit Monty Taylor proposed openstack/cliff master: Re-add support for python 3.5 https://review.opendev.org/715305
20:46:58 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add requires-python entry indicating 3.5 is required https://review.opendev.org/715243
20:49:50 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add requires-python entry indicating 3.5 is required https://review.opendev.org/715243
20:51:38 openstackgerrit Andreas Jaeger proposed openstack/openstacksdk master: Add requires-python entry indicating 3.5 is required https://review.opendev.org/715243
20:52:40 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add requires-python entry indicating 3.5 is required https://review.opendev.org/715243
20:52:56 mordred smcginnis: good catch
20:53:21 smcginnis Good, I feel better with that. ;)
20:53:39 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add requires-python entry indicating 3.5 is required https://review.opendev.org/715243
20:53:41 smcginnis mordred: Looks like you had a clash with Andreas. He made a minor typo fix.
20:53:47 mordred smcginnis: yeah - just picked it up
20:53:50 smcginnis And there you go..
20:55:05 openstackgerrit Andreas Jaeger proposed openstack/openstacksdk master: Small cleanups after Python2 drop https://review.opendev.org/715309
20:57:07 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Small cleanups after Python2 drop https://review.opendev.org/715309
21:01:41 openstackgerrit Andreas Jaeger proposed openstack/openstacksdk master: Re-add nodepool functional test https://review.opendev.org/715245
21:12:33 openstackgerrit Duc Truong proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266
21:40:11 openstackgerrit Merged openstack/openstacksdk master: Revert "Switch to futurist for concurrency" https://review.opendev.org/715249
22:19:00 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add requires-python entry indicating 3.5 is required https://review.opendev.org/715243
22:22:25 vphilippon @mordred: so just to make sure, the plan is still to make a release (which supports py2) before adding the py3.5 constraint?
22:23:02 mordred vphilippon: yes, that's right ... I've submitted this: https://review.opendev.org/#/c/715317/
22:23:23 mordred which will trigger the release of 0.45 with py2 supported properly
22:23:41 mordred then we'll land the 3.5 patch above and cut a 0.46
22:23:51 mordred so the world should be in a sensible position again after that
22:26:38 vphilippon Great, thanks! I can make a quick test of 0.45 when it lands to validate before you go forward :)
22:26:53 mordred vphilippon: sweet!
22:50:27 openstackgerrit Sam Morrison proposed openstack/openstacksdk master: Add availability zone and availability zone profile resources to load balancer https://review.opendev.org/714345
23:43:49 openstackgerrit Merged openstack/cliff master: Re-add support for python 3.5 https://review.opendev.org/715305
#openstack-sdks - 2020-03-27
00:37:45 openstackgerrit Merged openstack/openstacksdk master: Change default image type in the OTC vendor profile https://review.opendev.org/715183
02:02:30 openstackgerrit Sam Morrison proposed openstack/openstacksdk master: Add availability zone and availability zone profile resources to load balancer https://review.opendev.org/714345
09:14:37 AJaeger mordred: https://review.opendev.org/#/c/715243/ needs an update for requirements first - or done differently :( That's the py35 change
11:18:51 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add '--force; parameter to 'openstack quota set' https://review.opendev.org/713881
11:19:16 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add '--force' parameter to 'openstack quota set' https://review.opendev.org/713881
11:36:20 openstackgerrit Merged openstack/ansible-collections-openstack master: Ignore more things for the final artifact https://review.opendev.org/715302
11:44:32 openstackgerrit Merged openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444
12:27:03 smcginnis AJaeger, mordred: Actually, we might have an issue with that 3.5 patch.
12:27:25 smcginnis Since OpenStack itself is dropping support, this will also need to do something about requirements constraints.
12:27:40 smcginnis We're only tracking 3.6 and 3.7 there now.
12:35:12 AJaeger smcginnis: yeah, that's what I feared ;(
12:41:54 smcginnis Not sure what the best option is here. We may need to track our own upper-constraints on this repo. Or at least for the "unofficial" python versions we want to still support.
12:42:47 smcginnis We can have py35 jobs in tox refer to a local file, so something like "deps = -c./py35upper-constraints.txt" or something.
12:43:16 smcginnis Then each time the rest of openstack moves along to a new version, we would need to take a snapshot of the last supported u-c file and add it locally.
12:43:30 smcginnis I don't really like that, but it's the simplest option I can think of at the moment.
12:43:56 AJaeger What fun ;( Let's see what mordred thinks
12:56:54 smcginnis Another option. Mostly the same concept, but instead of keeping a local snapshot of u-c we point to the stable branch URL to grab it. Then is things get updated in stable/train to fix an issue, we automatically pick it up.
12:58:57 smcginnis But... we may also need multiple versions of [test-]requirements.txt, otherwise we will have conflicts between versions and the requirements-check job will fail...
12:59:08 smcginnis This isn't as easy as it first appears.

Earlier   Later