Friday, 2020-06-12

*** mnasiadka has quit IRC01:19
*** mnasiadka has joined #openstack-sdks01:22
*** ricolin has joined #openstack-sdks02:41
*** enriquetaso has quit IRC02:53
*** also_stingrayza has joined #openstack-sdks04:19
*** stingrayza has quit IRC04:23
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-sdks04:33
*** lbragstad has quit IRC04:36
*** lifeless has quit IRC04:47
*** lifeless has joined #openstack-sdks04:48
frickleris there a reason why "router create" doesn't allow additional settings like external-gateway? according to the api-ref that should be possible05:53
*** dustinc has quit IRC06:02
*** ttsiouts has joined #openstack-sdks06:19
*** ttsiouts has quit IRC06:37
*** jawad_axd has joined #openstack-sdks06:37
*** nightmare_unreal has joined #openstack-sdks06:52
*** gtema has joined #openstack-sdks06:55
*** ttsiouts has joined #openstack-sdks07:03
*** slaweq has joined #openstack-sdks07:05
*** slaweq has quit IRC07:10
*** vishakha has joined #openstack-sdks07:18
*** yolanda has quit IRC07:31
*** also_stingrayza is now known as stingrayza07:54
*** slaweq has joined #openstack-sdks07:58
*** iurygregory has joined #openstack-sdks07:59
*** iurygregory has quit IRC08:02
*** tosky has joined #openstack-sdks08:04
*** slaweq has quit IRC08:22
*** tkajinam has quit IRC08:23
*** slaweq has joined #openstack-sdks08:28
*** slaweq has quit IRC08:32
openstackgerritMerged openstack/openstacksdk master: NIT: Fix application credential  https://review.opendev.org/73475608:34
*** donnyd has joined #openstack-sdks09:37
*** ttsiouts has quit IRC09:50
*** gtema has quit IRC10:54
*** gtema has joined #openstack-sdks11:15
*** ttsiouts has joined #openstack-sdks11:16
*** ralonsoh has joined #openstack-sdks11:21
*** ttsiouts has quit IRC11:56
*** ttsiouts has joined #openstack-sdks12:12
*** ricolin has quit IRC12:19
*** jawad_axd has quit IRC12:53
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add support for filters into the project cleanup  https://review.opendev.org/73533613:11
mordredfrickler: I doubt it's on purpose13:27
mordredgtema: if you get a chance this morning, could you look at https://review.opendev.org/#/c/734000/ - it's green now that we released cliff13:28
gtemabtw, just few hours ago noticed, that when we do now "openstack command list" - everything is doubled13:29
mordredneat13:30
*** ttsiouts has quit IRC13:31
mordredgtema: that is not true for me13:33
gtemaokay, pretty interesting13:33
gtemaevery plugin is doubled for me and for my colleague13:33
gtemalemme check again13:34
mordredI'm using the container image with sha a865640d8bef13:34
gtemahmm, you are right. Then something is corrupted in our env setup13:35
gtemabut it started really very recently13:36
mordredcool. to double-check I pulled the latest image and it also still works13:36
gtemaokay, will dig further13:36
mordredgtema: maybe it's something with the latest cliff change? (don't know how that would do it though)13:36
gtemabut funny that you exactly work in this area13:37
mordredgtema: yeah13:37
gtemait happens though only for our project, which adds plugins13:37
mordredoh - wait!13:37
mordredno13:37
mordredit totally happens for me13:37
* mordred dives in13:38
mordredgtema: definitely caused by latest cliff13:41
mordredlet me see if applying the osc-lib patch fixes it13:41
gtemaokay13:41
mordredyes13:41
mordred https://review.opendev.org/#/c/734000/ fixes it13:41
mordredI don't understand WHY - but since that wants to land anyway maybe I'm not too worried about it13:42
gtemaokay, great13:42
mordredgtema: I went ahead and +A'd it. I'll cut a release as soon as that lands13:42
gtemaperfect13:42
vkmco/13:56
vkmchi folks, we have a quick doubt on format13:56
vkmcwe want to comply with the human interface guidelines as much as possible13:57
vkmcwe ran into this https://docs.openstack.org/python-openstackclient/latest/contributor/command-options.html#required-options13:57
vkmcin the docs13:57
*** vishakha has quit IRC13:57
vkmcand we are a bit confused on the "--" usage13:57
vkmcshouldn't double dashed params be optional params?13:57
gtemavkmc - mostly13:58
gtemabut there are situation, where you have so many mandatory params that you simply can't do everything positional13:58
vkmcwell, yes, that makes sense13:59
gtemaso rule of a thumb - everything "important to breathe" is positional, everything else is "--"13:59
vkmcconcern came up in this patch set https://review.opendev.org/#/c/732991/13:59
vkmcit's a command with very few params and both of them mandatory... openstack share resize <share> <size>14:00
vkmcIMHO there shouldn't be dashes involved there14:00
gtemaagree14:00
gtemashare at least "must" be positional14:00
gtemasince this is a root element of the operation14:00
gtemaif there would be multiple props you can set on the share - they would be likely dashed-ones14:01
gtemaif it's a single one - it's ok to make it positional either14:01
gtemabut still - size is up to you how to name14:01
vkmcok, it's clearer now14:02
vkmcthanks gtema :)14:03
gtemawelcome14:03
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update babel and webpack  https://review.opendev.org/73063914:05
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update karma, jasmine and istanbul  https://review.opendev.org/73064014:05
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update yaml-loader and json-loader  https://review.opendev.org/73064314:05
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update eslint and switch to standard  https://review.opendev.org/73064414:05
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Remove jsdoc  https://review.opendev.org/73067814:05
*** yoctozepto has joined #openstack-sdks14:09
yoctozeptohi mordred14:09
yoctozeptomust have lost this channel on failed irc client upgrade14:10
mordredyoctozepto: it happens :)14:12
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: CI: New jobs  https://review.opendev.org/70213214:17
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update babel and webpack  https://review.opendev.org/73063914:17
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update karma, jasmine and istanbul  https://review.opendev.org/73064014:17
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update yaml-loader and json-loader  https://review.opendev.org/73064314:17
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update eslint and switch to standard  https://review.opendev.org/73064414:17
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Remove jsdoc  https://review.opendev.org/73067814:17
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: WIP Re-add functional tests  https://review.opendev.org/73534414:17
*** ttsiouts has joined #openstack-sdks14:23
*** gtema has quit IRC14:23
*** ttsiouts has quit IRC14:28
openstackgerritMonty Taylor proposed openstack/osc-lib master: Fix the misspelling of "argument"  https://review.opendev.org/62979014:28
openstackgerritMonty Taylor proposed openstack/osc-lib master: Implement auth caching  https://review.opendev.org/35948914:30
dtroyervkmc: I would encourage you to conider your command set as a whole and look for consistency.  in some commands —size is optional as it can be inferred, like volume create with —image.  User feedback has strongly valued the consistency of the OSc commands as a whole...14:51
openstackgerritMerged openstack/osc-lib master: Remove commandmanager subclass  https://review.opendev.org/73400014:58
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Cache auth token in keyring  https://review.opendev.org/73535215:02
mordreddtroyer: ^^ lookie - reviving the 2.5 year old jamielennox patch!15:02
mordred(although gonna make a stab at doing it down in sdk so it's not just an osc thing)15:03
*** gtema has joined #openstack-sdks15:16
dtroyerWoot!!15:41
dtroyerI was hoping to look that up soon… the cloud I am blessed to use has Duo enabled so psuedo-mfa, we're leveraging interactive mode so we don't get auth pings on damn near every API call…that would be the real fix15:42
dtroyermordred: know anyone who can fix fog too? :)15:43
mordreddtroyer: sure - just replace it with sdk and you should be fine ;)15:44
dtroyerah, right, the ole s/chef/anisble/ solution!15:44
mordred:)15:44
mordreddtroyer: I'm thinking - if we can get this token caching thing to work - we should/could use a similar mechanism to allow optional password storage - and add an 'openstack cloud login' command that would trigger the password prompt and then save it in your keyring for you - but in a way that if you do that with osc, then any sdk consumers for that cloud will be able to pull the password from your15:46
mordredkeyring15:46
mordredI think gtema was asking about the ability to save passwords separately but not just in the secure.yaml file15:46
mordredalso - gtema - see 735352 :)15:47
dtroyermordred: that parallels the dreams I had about leveraging keychain/ssh-agent-type stuff15:47
mordredyeah15:47
openstackgerritRodolfo Alonso Hernandez proposed openstack/cliff master: Remove cap on cmd2  https://review.opendev.org/73536415:49
ralonsohmordred, ^^15:51
ralonsohI think this will fix some problems in the CI15:51
ralonsohimporting cliff in rally_openstack15:51
ralonsohhttps://4ca4d8620dc864739feb-f83d06667d580e000031601b82c71a43.ssl.cf5.rackcdn.com/735193/1/check/neutron-rally-task/acf1423/job-output.txt15:51
openstackgerritMerged openstack/osc-lib master: Revert "Add error message when occurrence Forbidden error"  https://review.opendev.org/71359515:53
*** gtema_ has joined #openstack-sdks16:04
mordredralonsoh: awesome16:05
*** gtema has quit IRC16:07
openstackgerritRodolfo Alonso Hernandez proposed openstack/cliff master: Remove cap on cmd2  https://review.opendev.org/73536416:08
*** gtema_ has quit IRC16:23
*** ttsiouts has joined #openstack-sdks16:24
openstackgerritMerged openstack/keystoneauth master: Remove mox3 from local upper-constraints  https://review.opendev.org/73407116:39
*** gtema has joined #openstack-sdks16:42
openstackgerritMonty Taylor proposed openstack/cliff master: Add load_plugins option  https://review.opendev.org/73396116:47
*** ttsiouts has quit IRC16:58
openstackgerritMerged openstack/osc-lib master: Fix the misspelling of "argument"  https://review.opendev.org/62979017:00
*** __ministry has quit IRC17:04
*** ttsiouts has joined #openstack-sdks17:05
openstackgerritRadosław Piliszek proposed openstack/js-openstack-lib master: CI: New jobs  https://review.opendev.org/70213217:06
*** ralonsoh has quit IRC17:12
yoctozeptomordred: https://review.opendev.org/702132 - modified commit message and moved the functional to exp to save some CI cycles17:13
yoctozeptomordred: we know it's broken for the time being, no need to "check"17:14
yoctozeptomordred: I've written a tiny TODO note for the future17:14
yoctozeptomordred: also started an etherpad for basic tracking https://etherpad.opendev.org/p/js-openstack-lib17:18
mordred++17:19
yoctozeptomordred: seems it does not like new babel/webpack17:21
mordredyoctozepto: weird - it's happy locally - let me see if i can reproduce locally17:23
*** gtema has quit IRC17:26
openstackgerritMerged openstack/js-openstack-lib master: CI: New jobs  https://review.opendev.org/70213217:26
mordredyoctozepto: oh - I think the babel and the karma patches need to be squashed17:27
*** gtema has joined #openstack-sdks17:27
*** ttsiouts has quit IRC17:28
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update babel, webpack, karma, jasmine and istanbul  https://review.opendev.org/73063917:28
mordredyoctozepto: ^^ there's still a failure locally - but it actually runs the tests17:28
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update babel, webpack, karma, jasmine and istanbul  https://review.opendev.org/73063917:30
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update eslint and switch to standard  https://review.opendev.org/73064417:30
mordredok. that's a smaller stack17:31
mordredyoctozepto: I think it's that we're not fully doing the right thing WRT browser and non-broswer so it's trying to call non-browser fs. module from browser tests17:33
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update babel, webpack, karma, jasmine and istanbul  https://review.opendev.org/73063917:38
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update eslint and switch to standard  https://review.opendev.org/73064417:39
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Re-enable browser tests  https://review.opendev.org/73537817:39
*** diablo_rojo has joined #openstack-sdks17:42
*** nightmare_unreal has quit IRC17:53
openstackgerritMerged openstack/js-openstack-lib master: Update babel, webpack, karma, jasmine and istanbul  https://review.opendev.org/73063917:54
*** ttsiouts has joined #openstack-sdks18:04
openstackgerritMonty Taylor proposed openstack/js-openstack-lib master: Update eslint and switch to standard  https://review.opendev.org/73064418:05
*** gtema has quit IRC18:07
*** ttsiouts has quit IRC18:09
*** stingrayza has quit IRC18:14
*** gtema has joined #openstack-sdks18:26
*** gtema has quit IRC18:28
*** irclogbot_1 has quit IRC18:29
*** irclogbot_2 has joined #openstack-sdks18:33
openstackgerritMonty Taylor proposed openstack/cliff master: Add load_plugins option  https://review.opendev.org/73396118:44
*** stingrayza has joined #openstack-sdks19:10
*** diablo_rojo has quit IRC19:24
*** ttsiouts has joined #openstack-sdks20:05
*** mgariepy has quit IRC20:16
*** gtema has joined #openstack-sdks20:29
*** mgariepy has joined #openstack-sdks20:30
*** gtema has quit IRC20:35
*** ttsiouts has quit IRC20:39
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org is going offline momentarily at 21:00 UTC for project rename maintenance, but should return within a few minutes: http://lists.opendev.org/pipermail/service-announce/2020-June/000004.html20:57
*** lbragstad has joined #openstack-sdks21:07
openstackgerritMonty Taylor proposed openstack/cliff master: Add load_plugins option  https://review.opendev.org/73396121:41
openstackgerritMonty Taylor proposed openstack/cliff master: Remove cap on cmd2  https://review.opendev.org/73536421:43
openstackgerritMonty Taylor proposed openstack/cliff master: Add load_plugins option  https://review.opendev.org/73396121:43
-openstackstatus- NOTICE: gerrit is being taken offline for emergency cleanup, will return to service again shortly22:00
*** lbragstad has quit IRC22:20
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org is available again22:49
*** tosky has quit IRC23:25
*** lbragstad has joined #openstack-sdks23:34
*** ttsiouts has joined #openstack-sdks23:35
*** dasp has quit IRC23:57

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!