Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-09-17
10:38:18 tobiash dtantsur: at least for the last 5 or 6 changes the same set of jobs fails deterministically
10:38:24 tobiash and I think also the same test cases
10:38:25 dtantsur meh
10:40:21 tobiash these started to fail somewhere between friday and saturday last week
10:40:41 dtantsur tobiash: oh, they did like a full rewrite of the API O___O https://github.com/openstack/keystone/commit/27f226554a21667eab62cfc61c13abfc2e780518
10:40:48 dtantsur gotta have some fun
10:42:47 tobiash dtantsur: thanks for checking, I think I'll just wait and try a recheck tomorrow ;)
10:48:21 dtantsur tobiash: mind filing a bug against keystone? I'm pretty sure that code is broken
10:50:37 tobiash dtantsur: can do that in an hour
11:03:03 dtantsur tobiash: never mind, did it https://bugs.launchpad.net/keystone/+bug/1792913
11:03:03 openstack Launchpad bug 1792913 in OpenStack Identity (keystone) "Keystone HTTP 500: AttributeError: type object 'GroupsResource' has no attribute '_get_domain_id_from_token'" [Undecided,New]
11:13:40 tobiash thanks :)
11:17:00 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: [DO NOT MERGE] Testing the gate fix https://review.openstack.org/603098
11:17:34 AJaeger dtantsur, mordred, shade is also broken...
11:17:38 dtantsur tobiash: you can watch https://review.openstack.org/#/c/602808/
11:17:42 dtantsur AJaeger: ^^^
11:18:09 AJaeger dtantsur: thanks
12:58:41 openstackgerrit Matthias Lisin proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303
13:51:32 ml what's the status of openstackcloud.py? Will this still be used in future releases or will this be deprecated in favor of proxy implementations through the Resource interface?
13:53:09 ml or is it still ok to implement convenience methods to query/update/delete resources by name_or_id ?
15:41:31 mordred ml: openstackcloud.py is still totally a supported interface and will continue to be - so it's great to implement convenience methods there
15:41:59 mordred ml: we will hopefully start to implement some of those methods using the underlying proxy methods, but that should be transparent to the user
16:00:03 dtantsur mordred: hey, welcome back! mind checking a few baremetal patches? https://review.openstack.org/599420 https://review.openstack.org/599533 https://review.openstack.org/600452
16:00:18 dtantsur we're getting really close, probably one patch away, from switching metalsmith away from ironicclient :)
17:27:01 ml mordred: alright. thanks for the information
18:09:07 mordred dtantsur|afk: woot! reviewing now
18:30:26 Shrews mnaser: regarding https://github.com/ansible/ansible/pull/45701 , we have a policy of not having the service name (e.g., "nova") in the name of the module
18:30:52 Shrews mnaser: i think mordred wrote the naming rules down somewhere, but i forget where
18:31:48 Shrews mordred: https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/cloud/openstack/README.md
18:32:09 Shrews mnaser: oh, i forgot the "admin" module rule
18:32:19 Shrews heh
18:33:04 mnaser Shrews: i guess i indirectly followed the rules :D
18:33:23 mnaser the reason is it is managing nova services, and other services have their own 'services' sooo
18:34:27 Shrews mnaser: yeah
18:34:35 Shrews my bad. strange test failure though
18:45:34 openstackgerrit Brian Haley proposed openstack/python-openstackclient master: Fix some spaces in help messages https://review.openstack.org/603206
#openstack-sdks - 2018-09-18
00:16:27 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Use discovery instead of config to create proxies https://review.openstack.org/600292
03:26:35 openstackgerrit Deepak Mourya proposed openstack/python-openstackclient master: Now we can add description for role creation in OSC https://review.openstack.org/484355
05:07:22 openstackgerrit Wenran Xiao proposed openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279
06:39:12 openstackgerrit Wenran Xiao proposed openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279
07:38:06 openstackgerrit Andreas Jaeger proposed openstack/openstacksdk master: [DO NOT MERGE] Testing the gate fix https://review.openstack.org/603098
11:30:48 openstackgerrit Dmitry Tantsur proposed openstack/api-sig master: Add document describing consuming version discovery https://review.openstack.org/459405
13:16:58 mordred dtantsur|brb: I think we still don't quite have version discovery working right with sdk and keystoneauth and noauth endpoint_override in ironic
13:19:43 mordred I think it's hitting lines 105/106 in openstack/service_description.py
13:33:20 openstackgerrit Matthias Lisin proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303
14:10:10 dtantsur mordred: :(
14:18:59 dtantsur mordred: why do we even need that fallback to base Proxy?
14:26:37 mordred dtantsur: I *think* it's useful for people developing against things that aren't in os-service-types yet? but I think if we detect things properly we shouldn't hit that fallback
14:26:42 mordred I'll try to figure that out in a little bit
14:31:59 dtantsur mordred: I feel like this fallback may be addressing a non-existing case..
14:32:41 mordred dtantsur: it probably is
14:50:32 dtantsur mordred: oh, could you check another baremetal patch? https://review.openstack.org/#/c/594242/ I forgot about ti
15:40:24 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Wire in retries for all baremetal actions https://review.openstack.org/603427
15:40:26 dtantsur comments on the direction are welcome ^^^
16:50:10 openstackgerrit Merged openstack/openstacksdk master: baremetal: support newer microversions in {get,update,patch}_machine https://review.openstack.org/594242
17:03:01 AJaeger mordred , could you look at remaining python3-first changes, please? https://review.openstack.org/#/q/is:open+topic:python3-first+(project:openstack/os-client-config+OR+project:openstack-infra/requestsexceptions+OR+project:openstack/openstacksdk+OR+project:openstack-infra/shade)
17:03:09 AJaeger Who else reviews these and can help?
17:21:44 mordred AJaeger: done. I see you hit a bunch of rechecks already - those should work this time, the issues before were due to a keystone bug which has since been fixed
17:25:11 AJaeger mordred: yeah, waited for those ;)
17:25:13 AJaeger mordred: thanks
17:29:20 openstackgerrit Merged openstack-infra/requestsexceptions master: import zuul job settings from project-config https://review.openstack.org/594367
17:57:11 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Update baremetal objects with fields added up to Rocky https://review.openstack.org/599420
18:05:20 openstackgerrit Merged openstack/openstacksdk master: Consolidate cloud/base.py into functional/base.py https://review.openstack.org/591031
19:00:09 openstackgerrit Merged openstack/openstacksdk master: Add sjc1 to vexxhost profile https://review.openstack.org/602985
20:41:59 openstackgerrit Merged openstack-infra/shade master: import zuul job settings from project-config https://review.openstack.org/594368
20:44:37 openstackgerrit Merged openstack-infra/shade master: switch documentation job to new PTI https://review.openstack.org/594369
20:44:39 openstackgerrit Merged openstack-infra/shade master: add python 3.6 unit test job https://review.openstack.org/594370
20:57:34 openstackgerrit Merged openstack-infra/shade master: Cleanup .zuul.yaml https://review.openstack.org/598585
21:26:51 openstackgerrit Merged openstack/openstacksdk master: switch documentation job to new PTI https://review.openstack.org/594374
#openstack-sdks - 2018-09-19
00:34:03 openstackgerrit Ian Wienand proposed openstack/osc-lib master: Mask password when specified as an argument with a space https://review.openstack.org/603528
02:08:35 openstackgerrit Wenran Xiao proposed openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279
02:12:16 openstackgerrit Wenran Xiao proposed openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279
03:34:12 openstackgerrit Ian Wienand proposed openstack/osc-lib master: Mask password when specified as an argument with a space https://review.openstack.org/603528
05:34:40 openstackgerrit Merged openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279
06:03:14 AJaeger dtantsur|afk, mordred, https://review.openstack.org/#/c/594393/ fails on queens - any ideas how to fix the test suite? That's the final python3-first change to import for SDK...
06:20:58 openstackgerrit Andreas Jaeger proposed openstack/openstacksdk master: Update .zuul.yaml https://review.openstack.org/603563
08:22:13 openstackgerrit Ian Wienand proposed openstack/openstacksdk master: Handle empty values in regions https://review.openstack.org/601485
08:59:29 dtantsur AJaeger: ouch. mordred, shade CI seems to use new ansible with old shade/openstacksdk
08:59:46 dtantsur so it actually does not exercise shade. need to cap ansible?
09:01:07 dtantsur AJaeger: I'd wait for mordred, I'm not overly familiar with shade. But I suspect we may need to upper-cap ansible on all its ansible jobs.
09:18:28 AJaeger dtantsur: thanks for digging into it, hope mordred has an idea.
09:57:25 mordred AJaeger, dtantsur: I just pushed up another rev on 594393
09:57:36 dtantsur wow, an early mordred
09:57:45 dtantsur (or is it late?)
09:57:59 mordred dtantsur: it's early - and for no reason I can determine :(
09:58:16 dtantsur happens :(
09:58:21 mordred but - maybe that means I'm going to be productive today
09:58:53 dtantsur early wake up - and productive? gotta need a few litres of coffee :)
10:03:00 mordred oh. yes. very much so
10:03:21 ml I got some issues with the CI too, maybe you can help
10:03:34 AJaeger mordred: if that job passes, we might need to merge the project-config change first to remove the "broken" ansible job to merge 594393. But since 594393 is the last one, this is fine...
10:03:54 ml want to enable neutron-fwaas for my implementation over here: https://review.openstack.org/#/c/592303/ but didn't have any success yet
10:04:06 openstackgerrit Merged openstack/openstacksdk master: Add support for configured NAT source variable https://review.openstack.org/602618
10:05:31 ml trying to enable neutron-fwaas for the devstack checks
10:06:07 mordred AJaeger: ++
10:07:40 mordred ml: looking
10:07:48 ml thank you
10:13:52 mordred ml: I added a couple of comments - I think we need to move those devstack/settings into the .zuul.yaml
10:16:44 ml thanks. I will apply those changes right now
10:22:37 mordred dtantsur: my god. my brain is bleeding reading the microversion doc change - how did I ever write the initial versions of that

Earlier   Later