Friday, 2020-03-06

openstackgerritMerged openstack/openstacksdk master: Fix service_type test for magnum in gate  https://review.opendev.org/71153300:23
openstackgerritMerged openstack/openstacksdk master: Normalise create_coe_cluster{,_template} results  https://review.opendev.org/71152600:27
*** enriquetaso has quit IRC00:40
*** tosky has quit IRC01:39
*** KeithMnemonic1 has quit IRC01:52
*** KeithMnemonic1 has joined #openstack-sdks01:53
*** adriant has quit IRC01:53
*** adriant has joined #openstack-sdks01:54
*** dave-mccowan has quit IRC05:17
*** evrardjp has quit IRC05:35
*** evrardjp has joined #openstack-sdks05:35
brtknrmordred: hmm I don’t see any functional tests for magnum, am I looking in the wrong place?05:44
*** vishakha has joined #openstack-sdks06:59
vishakhaamotoki: Regarding https://review.opendev.org/#/c/697444/. Could you please look over the cmurphy's suggestion, so that I can proceed further on the patch.07:04
*** iurygregory has joined #openstack-sdks07:18
*** slaweq has joined #openstack-sdks07:23
*** LarsErikP has joined #openstack-sdks07:41
LarsErikPHi guys! Spoke to johnsom over at #openstack-horizon about a thread issue i'm having with octavia-dashboard in stein. He was pretty sure this patch fixed it: https://review.opendev.org/#/c/695795/07:42
LarsErikPBut I'm a little bit confused, because: I have one horizon-server running 16.0.0 (train) and sdk 0.36.0 which does not have the issue; and one server running 15.1.1 and sdk 0.26.0 which has this issue. The confusion is, neither of those SDK versions has the aforementioned patch included...07:44
LarsErikPThe issue is basically that leaving a browser open at the list of load-balancers will accumulate threads on the horizon server, and eventually kill the server..07:45
*** sshnaidm_ has joined #openstack-sdks07:57
*** sshnaidm|afk has quit IRC07:58
LarsErikPand; I tried to "ninja-apply" that patch into my horizon-server running 15.1.1 and SDK 0.26.0, but I stil got the issue - so I guess there is more to this?07:59
*** tkajinam has quit IRC08:29
*** tosky has joined #openstack-sdks08:40
*** jpena|off is now known as jpena08:47
*** dtantsur|afk is now known as dtantsur08:55
*** jpich has joined #openstack-sdks08:58
*** lxkong has quit IRC09:04
*** lxkong has joined #openstack-sdks09:04
*** ricolin has quit IRC09:12
openstackgerritTom Stappaerts proposed openstack/openstacksdk master: Support for stateless security groups  https://review.opendev.org/71151309:19
*** yolanda has quit IRC09:29
*** gtema has joined #openstack-sdks09:46
*** stingrayza has quit IRC09:58
*** stingrayza has joined #openstack-sdks09:59
*** gtema has quit IRC10:10
openstackgerritRiccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node  https://review.opendev.org/71103210:15
openstackgerritRiccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node  https://review.opendev.org/71103210:15
*** ralonsoh has joined #openstack-sdks10:17
openstackgerritTom Stappaerts proposed openstack/python-openstackclient master: Support for stateless security groups  https://review.opendev.org/71151510:31
*** sshnaidm_ is now known as sshnaidm|afk11:13
LarsErikPdid some more testing. Seems like SDK 0.27 acutally will fix the issue. Who should I nudge to get that released in UCA?11:16
fricklerLarsErikP: you can create a bug against UCA in lp or ping coreycb+jamespage in #ubuntu-server11:22
LarsErikPfrickler: fantastic! thaks :)11:29
LarsErikPthanks*11:29
LarsErikPgod damn. Can't even spell easy words now... #friday11:29
*** zigo has quit IRC12:02
*** purplerbot has quit IRC12:04
*** irclogbot_2 has quit IRC12:05
*** amotoki has quit IRC12:05
*** irclogbot_0 has joined #openstack-sdks12:08
*** amotoki has joined #openstack-sdks12:09
*** purplerbot has joined #openstack-sdks12:10
*** zigo has joined #openstack-sdks12:13
*** jpich has quit IRC12:27
*** jpich has joined #openstack-sdks12:27
openstackgerritTom Stappaerts proposed openstack/openstacksdk master: Support for stateless security groups  https://review.opendev.org/71151312:28
*** zigo has quit IRC12:31
*** dtantsur is now known as dtantsur|afk12:33
*** jpena is now known as jpena|lunch12:37
*** zigo has joined #openstack-sdks12:50
openstackgerritTom Stappaerts proposed openstack/python-openstackclient master: Support for stateless security groups  https://review.opendev.org/71151512:54
*** Luzi has joined #openstack-sdks13:11
*** jpena|lunch is now known as jpena13:33
*** Luzi has quit IRC13:50
openstackgerritRiccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node  https://review.opendev.org/71103213:57
*** enriquetaso has joined #openstack-sdks13:58
openstackgerritJames Denton proposed openstack/openstacksdk stable/train: Include "fields" to "SecurityGroup" query parameters  https://review.opendev.org/71162213:58
amotokivishakha: cmurphy's suggestion works for me.14:01
vishakhaamotoki: Thanks for the response.14:01
amotokivishakha: I originally advised to follow the OSC convention for boolean opts, then you proposed --enable/disable-ignore-foo, I suggested to avoid verb-verb style and finally you proposed the current version.14:02
amotokivishakha: during this, the route went wrong a bit.... :(14:02
vishakhaamotoki: Yes I agree.14:03
openstackgerritTom Stappaerts proposed openstack/openstacksdk master: Support for stateless security groups  https://review.opendev.org/71151314:03
mordredbrtknr: openstack.tests.functional.cloud.test_cluster_templates.TestClusterTemplate and openstack.tests.functional.cloud.test_magnum_services.TestMagnumServices ... so not many :)14:16
*** iurygregory has quit IRC14:34
*** iurygregory has joined #openstack-sdks14:40
openstackgerritRiccardo Pittau proposed openstack/openstacksdk master: Extract check temp_url_key logic  https://review.opendev.org/71097315:16
openstackgerritRiccardo Pittau proposed openstack/openstacksdk master: [WIP] Import generate_temp_url from swiftclient  https://review.opendev.org/71164015:16
*** jpich has quit IRC15:45
*** jpich has joined #openstack-sdks15:46
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Add ansible stable-2.9 job and run 2.8 and 2.9  https://review.opendev.org/71147116:51
umbSublimeHi again, bouncing back on the issue we fixed related to microversion on the compute.hypervisor and the work-around of adding `compute_api_version: <target_MV>` in clouds.yaml. I noticed it works as expected with the SDK, but the OSC doesn't seem to pick it up. (I still get id=type(int) and not id=type(uuid)). I only get the header `-H "X-OpenStack-Nova-API-Version: 2.53"` added when I use --os-compute-api-version=2.5316:51
umbSublime`openstack 5.0.0`16:56
*** iurygregory has quit IRC17:05
*** jpich has quit IRC17:13
*** dtroyer has quit IRC17:33
*** evrardjp has quit IRC17:35
*** evrardjp has joined #openstack-sdks17:35
mordredumbSublime: well - joy17:51
umbSublimeno biggie, just wanted to point it out17:53
mordredumbSublime: that'll be a little longer to sort out - but it's actually an important one to do. I'm goign to see if there's a quicker way to plumb those through than what my hunch is17:55
umbSublimeYah I figured this one would be a bit trickier. Would you like me to file a bug for tracking this ?17:56
mordredsure!17:57
*** jpena is now known as jpena|off17:57
* mordred starts spelunking in the code ...17:58
umbSublimeI'll get to it a bit later today ;)17:59
umbSublimeI'm building my first cliff app right now, and I'm not sure how to access to apps parser from within a command (or if that's even possible). cmd_obj.get_parser() only returns the parser local to the command.18:00
umbSublimeWelp, again I asked a question too quick. I can use cmd_obj.app_args to access the parsed args18:06
mordredwoot!18:08
umbSublimeI want to centralize all my "operator" scripts under a CLI. I'm still debating if I want a standalone CLI, or an OSC plugin. In any case transitioning for a stand-alone CLI to OSC plugin seems very easy18:10
*** dtroyer has joined #openstack-sdks18:14
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Honor api config version for compute calls  https://review.opendev.org/71173218:17
mordredumbSublime: ^^ that fixes the _specific_ issue - but obviously this is a thing that should be generalized and apply to all of the osc config reading18:18
*** gmann is now known as gmann_lunch18:27
*** gmann_lunch is now known as gmann18:32
*** ralonsoh has quit IRC18:40
rm_workmordred: did chances for a release get sidetracked due to a bug yesterday? :D18:43
mordredrm_work: yeah. I think we've landed all the patches now - but I think it'll be first thing monday before the release team will cut a release (they don't tend to release on friday)18:44
rm_workkk, WFM18:44
umbSublimemordred, I applied the patch, but I still get `-H "X-OpenStack-Nova-API-Version: 2.1`18:55
mordredumbSublime: boo. it worked for me locally ...19:01
umbSublimeShould we populate L40's _compute_api_version. If not it will always evaluate to None19:01
mordredno - I think it gets populated in check_api_version19:03
umbSublimeIs it this call that looks in the config  `instance._cli_options.get_default_microversion(API_NAME)`?19:04
mordredyeah - that one and the one before it - get_api_version will get it if you set compute_api_version and get_default_microversion gets compute_default_microversion19:05
mordredI'm curious why it didn't work for you but it did work for me19:05
mordreddo you have any env vars set like OS_COMPUTE_API_VERSION or anything?19:05
umbSublimeI have this is my config "compute_api_version: 2.53", no Env vars19:06
umbSublimeI fall in the `if` on L5319:07
mordredoh. interesting19:07
mordredso you have something in compute_api_version by there19:08
umbSublimeBoth instance._cli_options.get_api_version(API_NAME) and instance._cli_options.get_default_microversion(API_NAME) return None19:10
mordredOH19:11
mordredit works for me without that patch19:11
umbSublimecompute_api_version --> API Version Major: 2, Minor: 119:11
mordredI think I may know what the issue is19:11
umbSublimeWhat config param did you use `compute_default_microversion` or `compute_api_version` ?19:12
mordredcompute_api_verison ... but I think there is a different issue19:12
mordredI think your nova has the discovery document auth protected19:12
mordredcan you try making a bare curl GET call to your compute endpoint?19:12
umbSublimeI the bare endpoint discovery works, but if I add /v2.1 I get 40119:13
mordredyeah. that's it19:13
umbSublimewow19:13
mordredwe fixed the bug in nova the last cycle I think - and we work around it in sdk19:13
mordredthe novaclient that osc is using can't fetch your discovery document19:14
umbSublimeishh, I'm sorry for the waste of time. I should've tested in a fresh venv with everything up to date19:14
mordredno - it's the service ... I think this is something we're working around in sdk but not in osc19:14
mordredso a fresh venv won't fix it for you19:14
umbSublimeAhh ok I get what you mean now19:14
mordredI think what we need to do in osc is to replace the contents of check_api_version with the sdk equiv19:15
mordredsince that knows how to do discovery even with a broken nova service19:15
mordredI'll work on getting that - might take me a few - but also should fix nova across the board for OSC19:16
umbSublime+1 !!, I'll try and document the result of our discussion in the bug19:17
mordredI think the difference is that --os-compute-api-version is hard-setting the value and ignoring discovery19:17
mordredthanks!19:17
umbSublimeThank you ! Through all our discussions in the last few days I learned so much about the SDK's and OSC's inner workings !19:18
mordredyay! what a successful week then!19:20
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Honor api version settings in config like CLI  https://review.opendev.org/71173219:39
mordredumbSublime: ^^ can you try that and see if it fixes the issue for you when you set the compute_api_version on the CLI?19:40
mordredgah19:40
mordredin the clouds.yaml19:40
mordredit injects the value into the stream similarly to how it works with --os-compute-api-version which WAS working for you19:40
umbSublimeI'll give it a shot shortly and report back19:44
umbSublimeBefore and after the patch `print(version_opt)` returns 2.1 (to make sure it's not a lib version issue let me just test in fresh venv)19:49
mordredso weird19:50
umbSublimeSobasically  i don't enter the patch because `version_opt = str(self.cloud.config.get(option, default_version))` appears to be setting the value to '2.1'. Where does that 'default_version' come from19:51
mordredoh. that is weird ... let me see19:55
mordredah. it's in the compute module19:56
mordredbut why does the cli arg work19:56
mordredit's possible it's too late in the week to fully wrap my head around this :)19:57
mordredumbSublime: actually ...19:59
mordredso - in that line you just referenced ...20:00
mordredoption should be "compute_api_version" right?20:00
mordredif you print self.cloud.config it should be a dict - and if you have compute_api_version in your clouds.yaml it should have compute_api_version in that dict - which means that line _should_ be returning your setting and not the default fallback value20:01
umbSublimechecking now, (sorry was AFK)20:15
mordredno worries!20:16
mordredthanks for trying out all the things :)20:17
umbSublimeoption isn't in the dic, so the the .get returns default_version20:17
umbSublimeI see though that for other services I get and int.  for example for identity Option=320:18
umbSublimeherr nvm I did this without thinking20:20
umbSublimeI didn't notice option wasn't the key name, but a var containing the key name20:21
umbSublimemordred, here is self.cloud.config -> https://hastebin.com/ugiketotup.py20:25
mordredwow. yeah - that sure doesn't have compute_api_version in there20:25
mordredyou sure it's in your clouds.yaml?20:25
mordredhttp://paste.openstack.org/show/790413/20:27
umbSublimehttps://hastebin.com/ifulonekaq.cpp20:27
mordredthere's mine20:27
umbSublimevexx <320:27
mordredyou're calling this like openstack --os-cloud=ucne1 hypvisor list20:28
umbSublimeno :(, with env_var OS_CLOUD_CONFIG_NAME=ucne120:29
mordredthat shold still be fine - do you have the other things set via env vars?20:29
umbSublimenothing related to OS_COMPUTE_API_VERSION20:30
umbSublimeJust noticed I also have `OS_REGION_NAME=ncsa-east-ne1`, but I don't think that should cause issues20:31
mordredoh wait ... no, OS_REGION_NAME is still a selector and fine20:31
mordredBUT20:31
umbSublimeit works with --os-cloud though  :O20:31
mordredchange OS_CLOUD_CONFIG_NAME=ucne1 to OS_CLOUD=20:31
mordredchange OS_CLOUD_CONFIG_NAME=ucne1 to OS_CLOUD=ucne120:31
umbSublime'compute_api_version': '2.53',20:32
mordred\o/20:32
mordrednow does the hypervisor list work without --os-compute-api-version ? (eithe with or without that patch)20:32
umbSublimeworks with the patch, let me check without20:32
umbSublime:facepalm: it works without the patch. It was env all along :/20:33
mordredyay!20:33
mordredwell - still ... there are some things here I want to refactor :)20:34
mordredbut even better ... now we don't have to get any patches released for you20:34
umbSublimeXD20:34
*** lxkong has quit IRC21:11
*** Shrews has quit IRC21:11
*** dayou has quit IRC21:11
*** iokiwi has quit IRC21:11
*** melwitt has quit IRC21:13
*** noonedeadpunk has quit IRC21:13
*** aspiers has quit IRC21:13
*** corvus has quit IRC21:13
*** Hazelesque has quit IRC21:13
*** mugsie has quit IRC21:13
*** Jeffrey4l has quit IRC21:13
*** fyx has quit IRC21:13
*** knikolla has quit IRC21:13
*** logan- has quit IRC21:13
*** dustinc has quit IRC21:13
*** jroll has quit IRC21:13
*** LarsErikP has quit IRC21:13
*** zigo has quit IRC21:13
*** mgariepy has quit IRC21:13
*** mgoddard has quit IRC21:13
*** eandersson has quit IRC21:13
*** jpena|off has quit IRC21:13
*** bverschueren has quit IRC21:13
*** gmann has quit IRC21:13
*** vkmc has quit IRC21:13
*** smcginnis has quit IRC21:13
*** odyssey4me has quit IRC21:13
*** slaweq has quit IRC21:13
*** __ministry has quit IRC21:13
*** yoctozepto has quit IRC21:13
*** Qiming has quit IRC21:13
*** tacco has quit IRC21:13
*** lifeless has quit IRC21:13
*** dtruong has quit IRC21:13
*** tbarron has quit IRC21:13
*** ianw has quit IRC21:13
*** tobberydberg has quit IRC21:13
*** dtroyer has quit IRC21:13
*** enriquetaso has quit IRC21:13
*** purplerbot has quit IRC21:13
*** irclogbot_0 has quit IRC21:13
*** KeithMnemonic1 has quit IRC21:13
*** efried_gone has quit IRC21:13
*** dmellado has quit IRC21:13
*** evrardjp has quit IRC21:13
*** masayukig has quit IRC21:13
*** amito has quit IRC21:13
*** dasp has quit IRC21:13
*** mnasiadka has quit IRC21:13
*** tonyb has quit IRC21:13
*** rm_work has quit IRC21:13
*** stingrayza has quit IRC21:13
*** factor has quit IRC21:13
*** tobiash has quit IRC21:13
*** jistr has quit IRC21:13
*** ebbex has quit IRC21:13
*** timburke has quit IRC21:13
*** jkulik has quit IRC21:13
*** Blinkiz has quit IRC21:13
*** gryf has quit IRC21:13
*** guilhermesp has quit IRC21:13
*** gouthamr has quit IRC21:13
*** frickler has quit IRC21:13
*** mordred has quit IRC21:13
*** dtantsur|afk has quit IRC21:13
*** tosky has quit IRC21:13
*** sshnaidm|afk has quit IRC21:13
*** ccamel has quit IRC21:13
*** mtreinish has quit IRC21:13
*** elmiko has quit IRC21:13
*** larsks has quit IRC21:13
*** vishakha has quit IRC21:13
*** TheJulia has quit IRC21:13
*** amotoki has quit IRC21:13
*** umbSublime has quit IRC21:13
*** openstackgerrit has quit IRC21:13
*** stephenfin has quit IRC21:13
*** mmedvede has quit IRC21:13
*** vdrok has quit IRC21:13
*** masterpe has quit IRC21:13
*** mnaser has quit IRC21:13
*** gundalow has quit IRC21:13
*** cmurphy has quit IRC21:13
*** dtroyer has joined #openstack-sdks21:13
*** evrardjp has joined #openstack-sdks21:13
*** enriquetaso has joined #openstack-sdks21:13
*** zigo has joined #openstack-sdks21:13
*** purplerbot has joined #openstack-sdks21:13
*** amotoki has joined #openstack-sdks21:13
*** irclogbot_0 has joined #openstack-sdks21:13
*** stingrayza has joined #openstack-sdks21:13
*** lxkong has joined #openstack-sdks21:13
*** tosky has joined #openstack-sdks21:13
*** sshnaidm|afk has joined #openstack-sdks21:13
*** LarsErikP has joined #openstack-sdks21:13
*** slaweq has joined #openstack-sdks21:13
*** vishakha has joined #openstack-sdks21:13
*** KeithMnemonic1 has joined #openstack-sdks21:13
*** umbSublime has joined #openstack-sdks21:13
*** ccamel has joined #openstack-sdks21:13
*** mgariepy has joined #openstack-sdks21:13
*** iokiwi has joined #openstack-sdks21:13
*** dayou has joined #openstack-sdks21:13
*** Shrews has joined #openstack-sdks21:13
*** efried_gone has joined #openstack-sdks21:13
*** factor has joined #openstack-sdks21:13
*** mgoddard has joined #openstack-sdks21:13
*** dmellado has joined #openstack-sdks21:13
*** openstackgerrit has joined #openstack-sdks21:13
*** vdrok has joined #openstack-sdks21:13
*** masayukig has joined #openstack-sdks21:13
*** amito has joined #openstack-sdks21:13
*** TheJulia has joined #openstack-sdks21:13
*** __ministry has joined #openstack-sdks21:13
*** masterpe has joined #openstack-sdks21:13
*** mtreinish has joined #openstack-sdks21:13
*** tobiash has joined #openstack-sdks21:13
*** larsks has joined #openstack-sdks21:13
*** dasp has joined #openstack-sdks21:13
*** jistr has joined #openstack-sdks21:13
*** stephenfin has joined #openstack-sdks21:13
*** elmiko has joined #openstack-sdks21:13
*** dtantsur|afk has joined #openstack-sdks21:13
*** gouthamr has joined #openstack-sdks21:13
*** frickler has joined #openstack-sdks21:13
*** jkulik has joined #openstack-sdks21:13
*** Blinkiz has joined #openstack-sdks21:13
*** tonyb has joined #openstack-sdks21:13
*** rm_work has joined #openstack-sdks21:13
*** vkmc has joined #openstack-sdks21:13
*** smcginnis has joined #openstack-sdks21:13
*** odyssey4me has joined #openstack-sdks21:13
*** Qiming has joined #openstack-sdks21:13
*** lifeless has joined #openstack-sdks21:13
*** dtruong has joined #openstack-sdks21:13
*** tbarron has joined #openstack-sdks21:13
*** ianw has joined #openstack-sdks21:13
*** tobberydberg has joined #openstack-sdks21:13
*** jpena|off has joined #openstack-sdks21:13
*** bverschueren has joined #openstack-sdks21:13
*** eandersson has joined #openstack-sdks21:13
*** melwitt has joined #openstack-sdks21:13
*** aspiers has joined #openstack-sdks21:13
*** corvus has joined #openstack-sdks21:13
*** Hazelesque has joined #openstack-sdks21:13
*** logan- has joined #openstack-sdks21:13
*** dustinc has joined #openstack-sdks21:13
*** jroll has joined #openstack-sdks21:13
*** gmann has joined #openstack-sdks21:13
*** knikolla has joined #openstack-sdks21:13
*** fyx has joined #openstack-sdks21:13
*** gundalow has joined #openstack-sdks21:13
*** mnaser has joined #openstack-sdks21:13
*** cmurphy has joined #openstack-sdks21:13
*** gryf has joined #openstack-sdks21:13
*** yoctozepto has joined #openstack-sdks21:13
*** noonedeadpunk has joined #openstack-sdks21:13
*** mnasiadka has joined #openstack-sdks21:13
*** guilhermesp has joined #openstack-sdks21:13
*** tacco has joined #openstack-sdks21:13
*** Jeffrey4l has joined #openstack-sdks21:13
*** mmedvede has joined #openstack-sdks21:13
*** mordred has joined #openstack-sdks21:13
*** timburke has joined #openstack-sdks21:13
*** ebbex has joined #openstack-sdks21:13
*** mugsie has joined #openstack-sdks21:13
*** factor has quit IRC21:13
*** factor has joined #openstack-sdks21:13
*** lastmikoi has quit IRC21:13
*** factor has quit IRC21:16
*** factor has joined #openstack-sdks21:16
*** lastmikoi has joined #openstack-sdks21:16
*** enriquetaso has quit IRC21:56
*** factor has quit IRC22:15
*** tosky has quit IRC23:51

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!