Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2021-02-22
17:31:32 openstackgerrit Anuradha proposed openstack/openstacksdk master: Improve Ironic API on OpenStack SDK https://review.opendev.org/c/openstack/openstacksdk/+/766531
18:16:49 tosky mordred, zigo: sorry for the partial OT, but it seems you are the only cores still around for the old sqlalchemy-migrate, and I've tried to fix the jobs a bit: https://review.opendev.org/c/x/sqlalchemy-migrate/+/756926
18:17:19 tosky I understand the project is not really alive but there are still projects which depend on it
18:17:28 tosky could you please take a look?
18:45:26 zigo tosky: Will review it.
20:18:55 zigo tosky: +2w
20:19:10 zigo I'm not sure if mordred would review.
20:40:19 tosky zigo, mordred: thank you both :)
20:40:34 tosky now let's hope nothing broke since October, but in case I will fix it
#openstack-sdks - 2021-02-23
11:55:43 openstackgerrit Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node traits https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774089
13:01:04 openstackgerrit Anuradha proposed openstack/openstacksdk master: Improve Ironic API on OpenStack SDK https://review.opendev.org/c/openstack/openstacksdk/+/766531
13:52:45 openstackgerrit Andrew Karpow proposed openstack/python-openstackclient master: Use network service for Server AddFixedIP https://review.opendev.org/c/openstack/python-openstackclient/+/698749
17:08:46 openstackgerrit Anuradha proposed openstack/openstacksdk master: Improve Ironic API on OpenStack SDK https://review.opendev.org/c/openstack/openstacksdk/+/766531
18:54:41 openstackgerrit Anuradha proposed openstack/openstacksdk master: Support Introspection rules for ironic API https://review.opendev.org/c/openstack/openstacksdk/+/775878
22:09:35 mordred dtantsur: you know lots of things - did we ever write a plugin for ksa or sdk to allow people to use some of the non-keystone auths?
22:10:14 mordred (I remember it being a weird thing to think about but I feel like one we did to allow for deprecating old non-sdk/osc things)
22:10:16 mordred corvus: ^^
22:11:55 corvus (i'm looking at whether i can use a swift aio installation without keystone in an integration test with an app that uses openstacksdk)
22:17:47 mordred as a followup - python-swiftclient has a keystoneauth plugin for v1auth
22:22:24 mordred I kinda feel like we shoudl import that in to either sdk or ksa - I could understand why we wouldn't do it in ksa - but we put http basic in there already
22:23:04 mordred it seems like supporting swauth would be friendly to our users who sometimes might need to interact with a swift that's not using keystone
22:23:17 mordred but - you know - I'm here like an hour a week, so I'm not going to make too much of a fuss about it P)
22:23:18 mordred :)
22:41:39 corvus mordred: that worked, thanks!
22:43:26 mordred \o/
#openstack-sdks - 2021-02-24
00:00:27 openstackgerrit Brian Haley proposed openstack/python-openstackclient master: Add --name to port list https://review.opendev.org/c/openstack/python-openstackclient/+/777264
06:16:49 openstackgerrit sue proposed openstack/openstacksdk master: add masakari enabled to segment https://review.opendev.org/c/openstack/openstacksdk/+/777299
06:26:22 openstackgerrit Dylan Zapzalka proposed openstack/openstacksdk master: Added support for the Limits resource https://review.opendev.org/c/openstack/openstacksdk/+/777303
06:36:25 openstackgerrit sue proposed openstack/openstacksdk master: add masakari enabled to segment https://review.opendev.org/c/openstack/openstacksdk/+/777299
07:19:26 openstackgerrit Dylan Zapzalka proposed openstack/openstacksdk master: Added support for the Limits resource https://review.opendev.org/c/openstack/openstacksdk/+/776318
09:23:25 openstackgerrit Dr. Jens Harbott proposed openstack/openstacksdk master: Keep creating keystone admin endpoint for heat https://review.opendev.org/c/openstack/openstacksdk/+/777343
10:07:50 dtantsur mordred: we have http_basic
10:43:58 openstackgerrit Radosław Piliszek proposed openstack/openstacksdk master: Fix a trivial error in one of the error messages https://review.opendev.org/c/openstack/openstacksdk/+/777354
10:43:58 openstackgerrit Radosław Piliszek proposed openstack/openstacksdk master: Fix a trivial error in one of the error messages https://review.opendev.org/c/openstack/openstacksdk/+/777354
12:35:25 openstackgerrit Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node traits https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774089
12:45:23 openstackgerrit Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node traits https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774089
15:44:33 openstackgerrit James Palmer proposed openstack/openstacksdk master: WIP Add support for Resource Filters https://review.opendev.org/c/openstack/openstacksdk/+/776271
15:50:34 twilit_dragon stephenfin: Thanks for the code review comments you sent yesterday. I used autopep8 for formatting so hopefully that resolves any pep8 issues.
16:00:33 stephenfin ack
16:57:01 iurygregory stephenfin, hey o/ I'm trying to understand your comment https://review.opendev.org/c/openstack/openstacksdk/+/766531/9/openstack/baremetal/v1/conductor.py =)
16:57:25 iurygregory or the only things that need to change are the docstrings in _proxy.py? =)
17:02:26 iurygregory I've asked the intern to do the changes in _proxy.py only atm (she will push a new patchset)
17:06:30 openstackgerrit Anuradha proposed openstack/openstacksdk master: Improve Ironic API on OpenStack SDK https://review.opendev.org/c/openstack/openstacksdk/+/766531
22:22:12 openstackgerrit Merged openstack/openstacksdk master: Fix a trivial error in one of the error messages https://review.opendev.org/c/openstack/openstacksdk/+/777354
#openstack-sdks - 2021-02-25
02:37:02 openstackgerrit Ashley Rodriguez proposed openstack/openstacksdk master: Add share resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/775707
07:21:51 openstackgerrit Hudson Dahly proposed openstack/openstacksdk master: Added support for Messages resource https://review.opendev.org/c/openstack/openstacksdk/+/777500
07:37:31 openstackgerrit Hudson Dahly proposed openstack/openstacksdk master: Added support for Messages resource https://review.opendev.org/c/openstack/openstacksdk/+/777502
07:47:11 openstackgerrit Dylan Zapzalka proposed openstack/openstacksdk master: Added support for the Limits resource https://review.opendev.org/c/openstack/openstacksdk/+/776318
08:00:57 openstackgerrit Alexander Wilcox proposed openstack/openstacksdk master: Attempting to pass Zuul tests https://review.opendev.org/c/openstack/openstacksdk/+/777503
11:00:01 openstackgerrit Radosław Piliszek proposed openstack/openstacksdk stable/victoria: Fix a trivial error in one of the error messages https://review.opendev.org/c/openstack/openstacksdk/+/777540
11:00:01 openstackgerrit Radosław Piliszek proposed openstack/openstacksdk stable/victoria: Fix a trivial error in one of the error messages https://review.opendev.org/c/openstack/openstacksdk/+/777540
11:52:21 iurygregory stephenfin, if you have some time today can you re-review https://review.opendev.org/c/openstack/openstacksdk/+/766531 ? ty =)
13:25:24 iurygregory stephenfin, tyvm!
14:47:32 openstackgerrit Stephen Finucane proposed openstack/openstacksdk master: Add support for Messages resource https://review.opendev.org/c/openstack/openstacksdk/+/777502
14:50:02 stephenfin iurygregory: np :)
15:08:11 stephenfin gtema: Can you look at https://review.opendev.org/c/openstack/openstacksdk/+/776271/4/openstack/block_storage/v3/resource_filter.py#23 ?
15:08:25 gtema yes
15:08:28 openstackgerrit Dylan Zapzalka proposed openstack/openstacksdk master: Added support for the Limits resource https://review.opendev.org/c/openstack/openstacksdk/+/776318
15:08:29 stephenfin I have a question and think you might know the answer :)
15:13:45 gtema I was sure I missed one patch for reviews. This was the one
15:14:51 gtema we might hint James to change his gerrit account to be full name or a nick.
15:15:59 stephenfin good idea
15:16:14 gtema I actually missed this change exactly because of that
16:00:26 openstackgerrit Alexander Wilcox proposed openstack/openstacksdk master: Attempting to pass Zuul tests https://review.opendev.org/c/openstack/openstacksdk/+/777503
16:19:40 openstackgerrit Alexander Wilcox proposed openstack/openstacksdk master: This is the Host Extension Resource, for capstone. https://review.opendev.org/c/openstack/openstacksdk/+/777617
16:30:33 openstackgerrit Alexander Wilcox proposed openstack/openstacksdk master: This is the Host Extension Resource, for capstone. https://review.opendev.org/c/openstack/openstacksdk/+/776320
22:20:35 openstackgerrit Merged openstack/openstacksdk master: Improve Ironic API on OpenStack SDK https://review.opendev.org/c/openstack/openstacksdk/+/766531
#openstack-sdks - 2021-02-26
03:47:58 openstackgerrit Dylan Zapzalka proposed openstack/openstacksdk master: Added support for the Limits resource https://review.opendev.org/c/openstack/openstacksdk/+/776318
10:40:36 openstackgerrit Simon Li proposed openstack/keystoneauth master: fix: miss attr _discovery_cache in ServiceTokenAuthWrapper https://review.opendev.org/c/openstack/keystoneauth/+/777739
15:13:14 openstackgerrit Thiago Paiva Brito proposed openstack/openstacksdk master: Add set_readonly_volume to BlockStorageCloudMixin https://review.opendev.org/c/openstack/openstacksdk/+/776266
15:36:08 openstackgerrit Thiago Paiva Brito proposed openstack/openstacksdk master: Adding retype_volume to BlockStorageCloudMixin https://review.opendev.org/c/openstack/openstacksdk/+/776272
15:43:13 openstackgerrit Thiago Paiva Brito proposed openstack/openstacksdk master: Add set_readonly_volume to BlockStorageCloudMixin https://review.opendev.org/c/openstack/openstacksdk/+/776266
18:03:55 openstackgerrit Thiago Paiva Brito proposed openstack/openstacksdk master: Implement metadata operation support for Volumes https://review.opendev.org/c/openstack/openstacksdk/+/777801
18:05:03 openstackgerrit Thiago Paiva Brito proposed openstack/openstacksdk master: Implement metadata operation support for Volumes https://review.opendev.org/c/openstack/openstacksdk/+/777801
#openstack-sdks - 2021-02-27
23:19:11 openstackgerrit James Palmer proposed openstack/openstacksdk master: Add support for Resource Filters https://review.opendev.org/c/openstack/openstacksdk/+/776271
#openstack-sdks - 2021-03-01
00:05:36 openstackgerrit Dylan Zapzalka proposed openstack/openstacksdk master: WIP Added support for the Capabilities resource https://review.opendev.org/c/openstack/openstacksdk/+/777895
04:14:41 openstackgerrit Dylan Zapzalka proposed openstack/openstacksdk master: WIP Added support for the Capabilities resource https://review.opendev.org/c/openstack/openstacksdk/+/777895
04:15:50 openstackgerrit Dylan Zapzalka proposed openstack/openstacksdk master: WIP Added support for the Capabilities resource https://review.opendev.org/c/openstack/openstacksdk/+/777895
07:21:00 openstackgerrit Dylan Zapzalka proposed openstack/openstacksdk master: WIP Added support for the Quota class set extension resource https://review.opendev.org/c/openstack/openstacksdk/+/777934
07:59:14 openstackgerrit Maxim Babushkin proposed openstack/ansible-collections-openstack master: Add binding profile to port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/777936
08:03:33 openstackgerrit Maxim Babushkin proposed openstack/ansible-collections-openstack master: Add binding profile to port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/777936
08:52:03 openstackgerrit Maxim Babushkin proposed openstack/ansible-collections-openstack master: Add binding profile to port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/777936
12:17:03 openstackgerrit Maxim Babushkin proposed openstack/ansible-collections-openstack master: Add binding profile to port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/777936
13:40:52 yoctozepto morning
13:44:15 yoctozepto how could I help/motivate the merging of https://review.opendev.org/c/openstack/openstacksdk/+/777299 ? :-)
13:44:38 yoctozepto this is important for masakari this cycle
13:44:59 yoctozepto I truly hope we did not miss any deadlines
15:01:49 frickler yoctozepto: mentioning it here often enough will help, not sure whom best to ping about it in addition, possibly gtema as ptl
15:03:24 frickler weird error regarding volumes, if I do "openstack --os-volume-api-version 3.42 volume set --size 3 volume_name", I get "No volume with a name or ID of 'volume_name' exists.". using the volume id works fine.
15:03:58 frickler using the same command without the special api version also resolves the volume_name correctly, but obviously fails when the volume is in use
15:07:03 yoctozepto frickler: thanks, will keep bugging then
15:07:51 gtema yoctozepto, just having a look at this, you might want also to add _max_microversion on the resource to ensure microversion header is send (if it is required)
15:09:56 yoctozepto gtema: thanks; how does it work though?
15:10:17 gtema you specify resource attribute (i.e. https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/baremetal/v1/chassis.py#L23)
15:11:23 gtema it is then used in the underlaying methods (i.e. https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/resource.py#L1218) to be send as a header
15:11:54 gtema https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/resource.py#L1453
15:25:39 yoctozepto gtema: ah, ok, ack, thank you; I think I have wrapped my head around it now
15:26:14 yoctozepto gtema: I have a related question now: is it wrong that python-masakariclient sets the default microversion?
15:26:29 yoctozepto as that seems to just override any other smartness at the sdk level
15:27:03 gtema there is nothing right and nothing wrong. Each client is now behaving differently and thus we try to normalize OSC to use SDK to have less of differencies

Earlier   Later