Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-08-08
15:33:47 mordred cool
15:33:51 mordred yeah - that's where I got to
15:34:16 mordred kmalloc: in fact- what if I don't even have admin endpoints for anything in the v3 catalog?
15:34:17 kmalloc otherwise i would let v2 just kindof sit. my guess is we will rapidly get to a point where no one will be using v2 even with SDK.
15:34:40 mordred (so that we catch places where people are doing something that they probably shouldnt be)
15:34:46 kmalloc someone was working on fixing that in the general case (no admin endpoints in gate in v3) recently
15:34:50 kmalloc i am not sure what landed.
15:35:08 kmalloc but some things were still a little broken. i support going to no admin endpoints in v3
15:35:14 kmalloc esp. in SDK testing.
15:35:24 kmalloc no explicit admin endpoints*
15:35:44 mordred ++
16:00:00 edleafe What's that you say? The API-SIG Office Hour is upon us?
16:00:21 elmiko \o/
16:57:51 edleafe Guess all that lurking wore dtantsur|afk out!
16:58:19 elmiko haha, yeah!
16:58:34 elmiko we can blame it on the /other/ meeting XD
17:00:47 elmiko i won't be around next week, i'm on conference duty ;)
17:00:54 elmiko have a good weekend edleafe o/
17:01:03 edleafe you too, elmiko!
18:49:23 openstackgerrit David Shrewsbury proposed openstack/openstacksdk master: WIP: Retry SLO manifest upload https://review.opendev.org/675134
18:50:24 frickler mordred: kmalloc: for what happens with no admin endpoint see https://review.opendev.org/651492 , heat blows up when using the OS-TRUST interface, so that might be related to your v2 issue
18:51:06 frickler anyway help with fixing that remaining issue would be greatly appreciated, I dug through the tracebacks and didn't find anything helpful
19:00:21 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP: Retry SLO manifest upload https://review.opendev.org/675134
19:21:16 Shrews mordred: omg, i got the test working
19:23:07 Shrews mordred: i'm assuming we want the same retry logic on dlo object manifests, yeah? Would it also need to delete the segments?
19:44:35 openstackgerrit David Shrewsbury proposed openstack/openstacksdk master: Retry large object manifest upload https://review.opendev.org/675134
19:45:16 Shrews mordred: I just realized that we have the same problem if any of the segment uploads fail. :/ So no limited to manifest failure.
19:45:32 Shrews So not*
20:27:32 openstackgerrit Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049
20:43:35 mordred Shrews: this is quite the big beastie
20:43:45 mordred Shrews: but yay!
20:44:40 mordred Shrews: I believe with DLO objects we just pass the "delete your manifests" parameter and deleting the manifest object will cause the segment objects to be deleted
20:45:42 Shrews mordred: but that requires actually uploading the manifest, yeah?
20:46:16 mordred oh - joy. yes
20:46:21 openstackgerrit Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049
20:46:21 mordred Shrews: you have the smart brain
20:46:25 Shrews can't delete it if you can't upload it :-P
20:53:10 efried mordred: I loaded up your patch from yesterday and was able to get nova talking to cyborg just fine.
20:53:23 mordred efried: woot
20:53:23 efried I've got the setup running in a public cloud, do you want to poke at anything?
21:02:47 efried mordred: ...but afaict it doesn't look any different than the ksa version.
21:03:24 mordred efried: I think as long as it worked like you expected that's good enough for me
21:04:33 mordred Shrews: if you have a sec in between bouts of yelling at object-store apis ... https://review.opendev.org/#/c/675135/ is easy-ish
21:05:36 mordred Shrews: (and this is the patch efried is mentioning that he tested in the scrollback just above that makes cyborg and placement integration in nova work better)
21:06:04 efried mordred: We expected that to get rid of a bunch of warnings to placement as well, let me see if I can verify that...
21:06:39 mordred efried: if it doesn't, something is horribly wrong
21:27:59 efried mordred: Okay, I confirmed. Went back through the logs before I applied your patch and found those UnsupportedServiceVersion warnings, and they then went away once the patch was applied.
21:28:35 mordred efried: woot!
21:28:48 mordred senseless warnings in logs are like fungus
21:28:56 mordred you gotta get rid of them before the spread
21:29:40 efried left a +1 accordingly.
21:29:55 efried mordred: though a guy wonders if there should be some unit test for this....
21:30:23 mordred efried: some days I feel like you and Shrews are the same person what with your "shouldn't there be a test for this" ;)
21:31:05 efried I can think of worse company to be in.
21:31:29 efried I mean, I'm happy having tested this live, BUT I only tested it with a nice clean default devstack service catalog and conf
21:31:31 mordred efried: oh - I don't know if you saw - patch one on on the path to getting you a reusable fixture is actually working now
21:31:43 efried nice
21:31:58 mordred efried: people use openstack in some configuration that isn't a nice clean default devstack?
21:32:20 efried ...I didn't put it through the paces of endpoint_override, weird service catalog entries, non-default interfaces, weird version requests....
21:35:00 mordred yeah. well - so - the nice thing here (and we should do all of those things) - but the nice thing about this patch is it basically says "if we don't know anything, just defer 100% to the existing and already tested elsewhere adapter creation logic"
21:35:26 mordred so all the other combos *should* be covered already
21:35:43 mordred but that's me rationalizing a scenario that clearly needs some coverage :)
21:47:48 Shrews mordred: i insist on a test for the test as well
#openstack-sdks - 2019-08-09
08:29:06 openstackgerrit Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113
08:45:44 openstackgerrit Brin Zhang proposed openstack/python-openstackclient master: Add server migration list CLI https://review.opendev.org/675304
09:48:45 openstackgerrit Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113
10:04:08 openstackgerrit Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113
10:54:50 frickler kmalloc: mordred: I've refreshed the logs on my devstack patch dropping the admin endpoint, this is the error in h-eng that I'm failing to resolve https://logs.opendev.org/92/651492/7/check/openstacksdk-functional-devstack/afeaec0/controller/logs/screen-h-eng.txt.gz#_Aug_09_08_44_55_817151
10:56:34 frickler the traceback looks slightly similar to what cmurphy fixed for https://review.opendev.org/651790
11:15:46 mordred frickler: yeah - I think it means the devstack patch should also add interface=public to the middleware config - maybe that's just a thing that needs to be done for heat?
11:28:28 frickler mordred: I tested that locally for all different heat config sections I have found, but it didn't seem to make any difference.
11:28:39 mordred frickler: boo
11:28:52 mordred frickler: maybe cmurphy will know the magical answers whence she awakens
11:56:47 openstackgerrit Merged openstack/openstacksdk master: Initalize pool_executor so close works https://review.opendev.org/675120
12:12:18 openstackgerrit Shubham Potale proposed openstack/openstacksdk master: Add support for nfv-orchestration service https://review.opendev.org/675604
12:17:40 openstackgerrit Shubham Potale proposed openstack/python-openstackclient master: Add python-openstackclient support to vnf package APIs https://review.opendev.org/675609
12:17:40 openstackgerrit Shubham Potale proposed openstack/python-openstackclient master: Add commands for create and delete vnfpackage APIs https://review.opendev.org/675610
12:17:41 openstackgerrit Shubham Potale proposed openstack/python-openstackclient master: Add commands for list and show vnf package API's https://review.opendev.org/675611
12:17:41 openstackgerrit Shubham Potale proposed openstack/python-openstackclient master: Add command for upload vnf package API https://review.opendev.org/675612
12:23:27 openstackgerrit Shubham Potale proposed openstack/python-openstackclient master: Add python-openstackclient support to vnf package APIs https://review.opendev.org/675609
12:23:27 openstackgerrit Shubham Potale proposed openstack/python-openstackclient master: Add commands for create and delete vnfpackage APIs https://review.opendev.org/675610
12:23:28 openstackgerrit Shubham Potale proposed openstack/python-openstackclient master: Add commands for list and show vnf package API's https://review.opendev.org/675611
12:23:28 openstackgerrit Shubham Potale proposed openstack/python-openstackclient master: Add command for upload vnf package API https://review.opendev.org/675612
12:59:47 openstackgerrit Bence Romsics proposed openstack/openstacksdk master: Add router add/remove route operations https://review.opendev.org/674324
13:01:55 openstackgerrit Bence Romsics proposed openstack/python-openstackclient master: fixup! Add command: router add/remove route --route https://review.opendev.org/675618
13:04:55 openstackgerrit Bence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325
13:53:59 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Rename cloud_config to cloud_region https://review.opendev.org/675627
13:54:07 mordred dtantsur: ^^ I did the variable renaming in a followup
14:03:13 dtantsur cool!
14:06:29 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework statistics reporting https://review.opendev.org/659841
14:35:17 cmurphy mordred: frickler no idea
15:10:44 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187
15:10:45 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Remove container-infra.example.com hack https://review.opendev.org/675378
15:10:46 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Rename cloud_config to cloud_region https://review.opendev.org/675627
15:28:15 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Skip most service_description for unknown services https://review.opendev.org/675135
15:28:27 mordred Shrews, efried, mriedem: ^^ this time with a test
15:28:39 mordred (which even tests that no warnings are thrown)
15:35:58 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Allow services to override proxy creation. https://review.opendev.org/674110
15:36:05 mordred that has a test now too
15:36:40 mordred Shrews: that one should avoid the annoying warnings we get from swift about not being able to talk to the root url

Earlier   Later