Friday, 2019-11-15

*** gtema has quit IRC00:01
*** gtema has joined #openstack-sdks00:04
*** gtema has quit IRC00:08
*** dave-mccowan has quit IRC00:10
*** slaweq has joined #openstack-sdks00:11
*** gtema has joined #openstack-sdks00:12
*** slaweq has quit IRC00:16
*** KeithMnemonic has joined #openstack-sdks00:16
*** dave-mccowan has joined #openstack-sdks00:16
*** KeithMnemonic has quit IRC00:17
*** KeithMnemonic has joined #openstack-sdks00:17
*** gtema has quit IRC00:20
*** KeithMnemonic has quit IRC00:23
*** slaweq has joined #openstack-sdks00:28
*** slaweq has quit IRC00:33
*** gtema has joined #openstack-sdks00:35
*** gtema has quit IRC00:39
*** gtema has joined #openstack-sdks00:45
*** mriedem has quit IRC00:45
*** gtema has quit IRC00:49
*** slaweq has joined #openstack-sdks00:50
*** gtema has joined #openstack-sdks00:50
*** gtema has quit IRC00:54
*** slaweq has quit IRC00:55
*** gtema has joined #openstack-sdks00:58
*** tosky has quit IRC00:59
*** dave-mccowan has quit IRC01:01
*** gtema has quit IRC01:02
*** gtema has joined #openstack-sdks01:04
*** slaweq has joined #openstack-sdks01:06
*** gtema has quit IRC01:09
*** gtema has joined #openstack-sdks01:10
*** mnasiadka has quit IRC01:11
*** guilhermesp has quit IRC01:11
*** kmalloc has quit IRC01:12
*** TheJulia has quit IRC01:12
*** mnasiadka has joined #openstack-sdks01:12
*** TheJulia has joined #openstack-sdks01:13
*** guilhermesp has joined #openstack-sdks01:13
*** kmalloc has joined #openstack-sdks01:13
*** jkulik has quit IRC01:14
*** slaweq has quit IRC01:15
*** jkulik has joined #openstack-sdks01:17
*** gtema has quit IRC01:18
*** slaweq has joined #openstack-sdks01:18
*** gtema has joined #openstack-sdks01:21
*** slaweq has quit IRC01:23
*** gtema has quit IRC01:26
*** gtema has joined #openstack-sdks01:31
*** slaweq has joined #openstack-sdks01:37
*** gtema has quit IRC01:37
*** slaweq has quit IRC01:45
*** gtema has joined #openstack-sdks01:45
*** gtema has quit IRC01:50
*** gtema has joined #openstack-sdks01:51
*** slaweq has joined #openstack-sdks01:51
*** gtema has quit IRC01:56
*** slaweq has quit IRC01:56
*** gtema has joined #openstack-sdks02:01
*** openstackstatus has joined #openstack-sdks02:03
*** ChanServ sets mode: +v openstackstatus02:03
*** gtema has quit IRC02:10
*** slaweq has joined #openstack-sdks02:11
*** slaweq has quit IRC02:15
*** gtema has joined #openstack-sdks02:19
*** gtema has quit IRC02:27
*** gtema has joined #openstack-sdks02:35
*** gtema has quit IRC02:39
*** gtema has joined #openstack-sdks03:12
*** gtema has quit IRC03:16
*** gtema has joined #openstack-sdks03:18
*** gtema has quit IRC03:24
*** gtema has joined #openstack-sdks03:34
*** gtema has quit IRC03:38
*** gtema has joined #openstack-sdks03:45
*** gtema has quit IRC03:52
*** goldyfruit_ has quit IRC03:53
*** gtema has joined #openstack-sdks04:06
*** gtema has quit IRC04:10
*** gtema has joined #openstack-sdks04:17
*** gtema has quit IRC04:27
*** gtema has joined #openstack-sdks04:34
*** gtema has quit IRC04:42
*** Luzi has joined #openstack-sdks06:04
*** Blinkiz5 has joined #openstack-sdks06:29
*** gouthamr_ has joined #openstack-sdks06:35
*** Jeffrey4l_ has joined #openstack-sdks06:36
*** jpena|off has quit IRC06:37
*** mtreinish has quit IRC06:39
*** gouthamr has quit IRC06:39
*** brtknr has quit IRC06:39
*** Blinkiz has quit IRC06:39
*** lastmikoi has quit IRC06:39
*** Jeffrey4l has quit IRC06:39
*** evrardjp has quit IRC06:39
*** Blinkiz5 is now known as Blinkiz06:39
*** openstackstatus has quit IRC06:40
*** jpena|off has joined #openstack-sdks06:48
*** mtreinish has joined #openstack-sdks06:48
*** lastmikoi has joined #openstack-sdks06:48
*** brtknr has joined #openstack-sdks06:48
*** evrardjp has joined #openstack-sdks06:48
*** jpena|off has quit IRC07:15
*** mtreinish has quit IRC07:15
*** brtknr has quit IRC07:15
*** lastmikoi has quit IRC07:15
*** evrardjp has quit IRC07:15
*** jpena|off has joined #openstack-sdks07:16
*** mtreinish has joined #openstack-sdks07:16
*** lastmikoi has joined #openstack-sdks07:16
*** brtknr has joined #openstack-sdks07:16
*** evrardjp has joined #openstack-sdks07:16
*** KeithMnemonic has joined #openstack-sdks07:29
*** KeithMnemonic has quit IRC07:34
*** gtema has joined #openstack-sdks07:45
*** slaweq has joined #openstack-sdks07:46
*** slaweq_ has joined #openstack-sdks07:53
*** slaweq has quit IRC07:56
*** gtema_ has joined #openstack-sdks07:59
*** gtema has quit IRC08:01
*** slaweq_ is now known as slaweq08:03
openstackgerritYongli He proposed openstack/python-openstackclient master: Microversion 2.78 - server show topology  https://review.opendev.org/68092808:06
*** slaweq_ has joined #openstack-sdks08:19
*** slaweq has quit IRC08:19
*** ralonsoh has joined #openstack-sdks08:32
*** tosky has joined #openstack-sdks08:39
*** slaweq_ is now known as slaweq08:45
*** jpena|off is now known as jpena08:47
*** gtema_ has quit IRC08:49
*** jpich has joined #openstack-sdks08:50
*** jangutter has joined #openstack-sdks08:56
openstackgerritDaniel Bengtsson proposed openstack/python-openstackclient master: Stop configuring install_command in tox.  https://review.opendev.org/69448409:07
*** dave-mccowan has joined #openstack-sdks09:13
*** gtema has joined #openstack-sdks09:32
openstackgerritDaniel Bengtsson proposed openstack/python-openstackclient master: Stop configuring install_command in tox and stop use pip.  https://review.opendev.org/69448409:33
*** gtema has quit IRC09:37
*** cdent has joined #openstack-sdks09:45
*** gtema has joined #openstack-sdks09:47
*** jpich has quit IRC10:16
*** jpich has joined #openstack-sdks10:17
*** slaweq has quit IRC10:34
openstackgerritDaniel Bengtsson proposed openstack/osc-lib master: Stop configuring install_command in tox and stop use pip.  https://review.opendev.org/69451010:43
*** slaweq has joined #openstack-sdks10:45
*** slaweq has quit IRC10:50
*** cdent has left #openstack-sdks10:53
*** jawad_axd has joined #openstack-sdks11:32
jawad_axdHi folks, How to get vm console url (as openstack console url show server) from openstackSDK ?11:48
*** slaweq has joined #openstack-sdks11:55
*** jpena is now known as jpena|lunch12:00
*** goldyfruit_ has joined #openstack-sdks12:12
*** goldyfruit_ has quit IRC12:16
*** jangutter has quit IRC12:33
fricklerjawad_axd: this seems to be a missing feature. some tried to implement this three years ago, but failed, I haven't found anything happening after that https://review.openstack.org/31982812:45
fricklerosc is using novaclient for that action iiuc12:48
*** ricolin has joined #openstack-sdks12:50
jawad_axdfrickler: okay..I am looking into link you provided. Thanks for reply.12:50
*** Luzi has quit IRC13:05
*** Luzi has joined #openstack-sdks13:20
*** jangutter has joined #openstack-sdks13:20
*** mriedem has joined #openstack-sdks13:23
*** gtema_ has joined #openstack-sdks13:56
*** gtema has quit IRC14:00
*** jpena|lunch is now known as jpena14:01
mriedemdtroyer: when you get a chance, can you weigh in on an option name on this osc-placement patch? https://review.opendev.org/#/c/691422/ - it's updating a set command to add a --no-provider option which means, remove allocations for the given provider (but not all providers),14:13
mriedemthe --no-provider name is trying to mirror CLIs like port set --no-tag,14:13
mriedembut at least 2 people (efried and melwitt) were confused by it14:13
mriedemmy alternative suggestions are (1) change the option name to be --clear-provider or --remove-provider and keep using the set command,14:14
mriedemor (2) add an unset command with a --provider option14:14
mriedeme.g. openstack resource provider allocation unset --provider prov1 --provider prov2 consumer1  // removes consumer1's allocations from prov1 and prov214:15
*** Luzi has quit IRC14:16
*** gtema_ has quit IRC14:16
*** gtema has joined #openstack-sdks14:17
*** goldyfruit has joined #openstack-sdks14:30
*** jawad_axd has quit IRC14:37
*** jawad_axd has joined #openstack-sdks14:38
*** jawad_ax_ has joined #openstack-sdks14:40
*** gtema has quit IRC14:40
*** KeithMnemonic has joined #openstack-sdks14:41
*** jawad_axd has quit IRC14:43
*** jawad_ax_ has quit IRC14:45
*** jangutter has quit IRC15:00
*** ricolin has quit IRC15:18
openstackgerritNate Johnston proposed openstack/python-openstackclient master: Handle NotFoundException when listing floating IPs  https://review.opendev.org/69457115:20
efriedmriedem: (2) jumped out as being pretty clean/intuitive. I haven't stared at it yet.15:28
mriedemi left a comment on the review but one fear i have with adding unset is someone expecting it to also have a --no-allocation or some option to also remove specific resource classes from an existing provider allocation combo, which would be really annoying15:31
efriedmriedem: I was about to respond to that. IMO15:41
efriedopenstack resource provider allocation unset --consumer C [--provider P] [--resource-class R1 [--resource-class R2 ...]]15:41
efriedmakes enough sense.15:41
efriedIf you don't specify a --provider, all providers are affected.15:41
efriedYou can specify at most one --provider, in which case only that provider is affected.15:41
efriedYou can specify zero or more15:41
efried... --resource-class, in which case only those resource classes are affected.15:41
efriedFor a first iteration you could leave off the --resource-class arg, but if/when it needs to be added later, does that seem like a reasonable design?15:41
*** KeithMnemonic1 has joined #openstack-sdks15:41
efriedIf you need to affect a >1 subset of providers, repeat the command. Are we allowed to say that?15:42
mriedemi think consumer and at least one provider should be required, consumer obviously (it's an arg, not an option)15:45
mriedemspecifying zero providers means what? clear all allocations?15:45
mriedemand if so, <1.28 means DELETE, >= 1.28 means PUT with empty allocs dict and consumer_generation15:46
mriedemi'm not dealing with that --resource-class garbage15:46
efriedYes, zero providers means remove all allocations, the base use case for 'allocation unset $consumer'.15:47
mriedemand that's only useful over allocation delete b/c of 1.28 and consumer generations?15:47
efriedDo you think it's necessary to have a generation'd code path in `allocation delete`?15:48
mriedemnot really15:49
mriedemif someone wanted that, you could provide 1.28 and a --safe or --validate option or something like that15:49
efriedIMO this kind of reflects the semantics of DELETE vs PUT{}: one you're saying "Blow it away. I don't care what's there." The other you're saying "Remove what I think is there right now. If something changes, hold up, I need to reevaluate"15:49
mriedemopenstack --os-placement-api-version 1.28 resource provider allocation delete --safe <consumer_id>15:49
efriedBut in a CLI like this, the window between the GET and PUT is so small, I'm not sure it adds any value.15:49
efriedso no, unset with no provider (and no RC) doesn't really add value. It's just a reducto case of two CLIs providing the same function.15:50
efriedIf you include (or at least plan for) the RC option, though, it does make sense for --provider to be optional.15:51
efried"Remove all bandwidth allocations"15:51
efried"Remove all GPUs"15:51
efriedthings that can be on (multiple) nested providers15:52
mriedemcongratulations, you've talked me out of wanting to work on this anymore :)15:52
mriedemwhat i'd probably do is unset with --provider using nargs='+' to start meaning at least one --provider is required,15:55
mriedemif at some point you want to make --provider really optional, you drop the nargs15:55
*** goldyfruit_ has joined #openstack-sdks15:55
*** goldyfruit has quit IRC15:57
efriedI don't think we should allow more than one provider.15:58
efriedmm, hold that thought.15:58
efriedI guess you're more likely to want to remove all (nested) allocs from one host, huh.15:59
efried... unset --root-provider $P ?15:59
efried--tree $P ?15:59
mriedemmelwitt specifically wants to be able to remove allocations from more than one provider in a single call15:59
efriedalways all and only in the same tree?15:59
efriedbrbwc16:00
mriedemno16:00
mriedemno trees16:00
mriedemsimple old separate compute node providers16:00
mriedeme.g. you try migrating a server to hosts A, B, C, all fail, all orphan providers b/c nova has a bug16:00
efriedand it doesn't help us to just fix the bug, because osc needs to be usable against old still-broken clouds.16:01
mriedemcorrect16:02
efriedI can't think of reasonable way to allow granular updates on the CLI because you need to specify RP+RC in pairs.16:03
mriedemright, which is what allocation set --allocation does and it's a really gross interface on a command line16:04
mriedemthere is a good example of the gross in here https://docs.openstack.org/nova/latest/admin/troubleshooting/orphaned-allocations.html16:04
mriedemhttps://docs.openstack.org/nova/latest/admin/troubleshooting/orphaned-allocations.html#solution16:04
efriedyeah, rp=$u,RC=$n16:05
mriedemthat's the reason i'm working on the osc-placement command16:05
efriedokay, well, if we have a precedent for this --allocation syntax (I didn't realize that) then IMO it makes the path forward *somewhat* clearer.16:06
efriedDoes `allocation set` have anything yet that does partial update?16:08
efriedIf not, I'd like to keep it that way. "set" means full overwrite, end of story.16:08
efriedAnd we should put partials in a different CLI.16:09
efriedwhether `update` or `unset`16:09
efried`update` could take --allocation {same syntax as `set`} to add a rp/rc to what's there, and --no-allocation rp=$uuid[,$rcname] to remove. Omit $rcname and everything from that provider goes away, which is melwitt's use case. And we can defer implementing [,$rcname] to later.16:11
mriedemthere is no 'update' in osc16:12
mriedemthere is set16:12
melwittefried: I tried to add 'update' awhile back and got nacked and ended up instead adding --amend to 'set'16:12
efriedOkay, `set --amend` then. Point being, `set` (without something really stark) should be full overwrite.16:13
efriedalternative to --no-allocation could be --allocation rp=$uuid,$rc=0 to remove one16:13
melwittit's not a huge deal if it's not possible to remove for multiple providers, I was just saying it would be one call vs several16:13
mriedemtetsuro also asked for multiple provider support16:13
melwittheh16:14
efriedmelwitt: your --amend was for inventories, right?16:14
efriedWe still don't have any form of amending for allocs yet?16:15
melwittefried: that was my original reason (allocation ratios) but it works for anything16:15
melwittoh, sorry yeah16:15
efried"works for anything" theoretically, but is only implemented for inventories currently, right?16:15
melwittinventory set is its own thing yeah?16:15
efriedyeah, separate subcommand.16:15
efriedafaiu16:15
melwittif so then yes it's only for inventories16:15
melwittI can't remember rn16:16
dtroyermriedem, efried: I just skimmed the scrollback, it seems like you guys are on the right track.  The things to keep in mind are that —no-whatever options are used to remove all of whatever in set commands, to remove a specific one needs an unset command.  —all-whatever in unset also has a precedence in network commands.16:16
efriedokay, then s/update/set --amend/ for my suggestions above.16:16
*** goldyfruit___ has joined #openstack-sdks16:16
dtroyerI'll leave the particulars of the semantics of multiple providers, etc to you guys, I don't think we have a precedent for that sort of thing.  If so iit'll be in netowrk commands16:16
efrieddtroyer: Roger that. In this case I feel like --no-provider is really jarring semantically so I'm grasping for *any* better option.16:17
melwittok yeah inventory set is the only one that has --amend16:18
dtroyerI would expect that to remove all providers, which may or may not make sense at all there16:18
*** jawad_axd has joined #openstack-sdks16:18
efrieddtroyer: does --no-foo ever take an arg?16:18
efriedcause that's what's being proposed here.16:18
*** goldyfruit_ has quit IRC16:18
efriedto narrow down what we remove16:18
mriedembased on what dean just said i'd drop the set --no-provider thing,16:18
mriedemand just do: unset --provider16:18
mriedemnargs='+'16:19
dtroyerefried: no, it is used to clear the list (like properites) to allow setting new ones in the same command16:19
efriedokay16:19
efriedmriedem: having learned about the existing --allocation syntax, I'm not in favor of unset --provider16:19
*** tbarron has quit IRC16:20
efriedHindsight: if we had called it `consumer allocation` instead of `resource provider allocation` then we could have made "futz with a resource provider's allocations" the latter.16:21
mriedemso what are you asking for now? allocation set --amend --allocation rp=uuid to remove all allocatoins for a given provider?16:21
efriedyes16:21
efriedfor symmetry16:21
mriedemand --amend with rp=uuid,rc=n means add/update new/existing allocation?16:22
*** jawad_axd has quit IRC16:22
efriedyes16:22
efriedHold on, you said --amend --allocation rp=$uuid to remove? As opposed to --amend --allocation rp=$uuid,$rc=0 [... repeat for all $rc] ?16:23
efriedI guess that could work.16:24
mriedemi would prefer just a simple unset --provider16:24
mriedemthan dork with the existing set --allocation <gdi this is a mess like bdms> format thing16:25
mriedembut that's just me16:25
efriedI won't die on a hill for it, but the reasons I don't like that are:16:25
efried1) it's asymmetrical16:25
efried2) it makes it impossible to narrow down to RC granularity in the future16:25
mriedemi don't think it's impossible, you add a --resource-class option16:26
mriedemif specified, unset the resource class for the given provider,16:26
efriednot if you can specify multiple providers16:26
mriedemif no providers are specified, remove the resource class from all providers on which the consumer has allocations16:26
mriedem1. openstack resource provider allocation unset --provider A --provider B --resource-class VGPU consumer1 == remove VGPU allocatoins from providers A and B for consumer116:27
mriedem2. openstack resource provider allocation unset --resource-class VGPU consumer1 == remove VGPU allocations from all providers on which consumer1 has allocations16:27
mriedem3. openstack resource provider allocation unset --provider A consumer1 == remove all allocations consumer1 has on provider A16:28
mriedemdoesn't seem impossible to me16:28
efriedand --resource-class is repeatable too?16:28
mriedemsure16:28
efriedOkay. Then I just hate the asymmetry.16:29
efried4. openstack resource provider allocation unset consumer1  ? Same as `delete`?16:29
mriedemyeah i guess16:31
mriedemwell,16:31
mriedemif you make --provider use nargs=+ to start, then that's not possible as at least one provider is required,16:31
mriedemwhich you could do until you add support for --resource-class16:32
mriedembut that's a behavior change and would require a major version release probably16:32
efriedYeah, but the code is nearly identical because you still have to deal with having specified all --provider Z16:32
mriedemanyway, sure16:32
efriedwhich is effectively the same.16:32
mriedemso yes to 1-416:32
efriedare you going to skip implementing --resource-class for now?16:33
mriedemi would yes16:34
mriedemunlike this irc conversation, i don't really want to boil the ocean on providing something simple to just remove a single provider from a set of allocations :)16:34
mriedemthis started as like a "this should be relatively simple" kind of thing for something useful16:35
mriedemi appreciate forward-thinking, but i'm also close to just abandoning16:35
efried"remove *multiple* providers from a set of allocations" yeah?16:37
mriedemit started as one,16:39
mriedemtetsuro and mel asked for multiple which was simple to accomodate16:39
efriedor so you thought16:44
mriedemyeah this eric guy got involved16:45
mriedemblew it all up!16:45
efriedYou'll thank me later.16:46
efriedIn therapy.16:46
mnasermordred: i hope you're resting well after your entertainment-less flight --- can you leave some comments on https://review.opendev.org/#/c/684740/ please?16:53
mnaserwe kinda discussed that briefly at ansiblefest and i think ansible-sig might be a better home16:54
*** jpich has quit IRC16:55
*** goldyfruit_ has joined #openstack-sdks17:02
*** goldyfruit___ has quit IRC17:05
*** tbarron has joined #openstack-sdks17:06
*** mriedem is now known as mriedem_afk17:19
*** jpena is now known as jpena|off17:58
*** gtema has joined #openstack-sdks18:14
*** KeithMnemonic1 has quit IRC18:19
*** ralonsoh has quit IRC18:20
*** gtema has quit IRC18:22
*** jawad_axd has joined #openstack-sdks18:55
*** KeithMnemonic1 has joined #openstack-sdks19:00
*** tosky has quit IRC19:14
*** mriedem_afk is now known as mriedem19:23
johnsomefried It seems Dean is ok with that osc-lib hack for the neutron docs. Any chance we can move that from WIP?19:33
efriedjohnsom: which one?19:35
johnsomefried lol, https://review.opendev.org/#/c/69326719:35
efriedjohnsom: oh, yeah, needs tests, thought you might be willing to, ahem, co-author :P19:36
johnsomUm, no19:36
efriedokay then, I'll see what I can work up.19:37
johnsomOk, thank you19:37
*** mriedem has quit IRC19:37
*** mriedem has joined #openstack-sdks19:42
*** jawad_axd has quit IRC19:43
*** goldyfruit___ has joined #openstack-sdks20:09
*** goldyfruit_ has quit IRC20:12
openstackgerritEric Fried proposed openstack/osc-lib master: Add enhance_help kwarg to tags option generators  https://review.opendev.org/69326720:15
efriedjohnsom: ^20:15
efrieddtroyer: ^20:15
*** gtema has joined #openstack-sdks20:23
*** gtema has quit IRC20:27
*** yolanda has quit IRC20:35
*** yolanda has joined #openstack-sdks20:43
*** goldyfruit_ has joined #openstack-sdks20:46
*** goldyfruit___ has quit IRC20:48
*** slaweq has quit IRC20:50
openstackgerritNate Johnston proposed openstack/python-openstackclient master: Handle NotFoundException when listing floating IPs  https://review.opendev.org/69457120:58
johnsomThanks21:10
*** slaweq has joined #openstack-sdks21:11
*** slaweq has quit IRC21:16
openstackgerritRuby Loo proposed openstack/openstacksdk master: baremetal node: 'error' is a failed state  https://review.opendev.org/69463422:07
*** KeithMnemonic1 has quit IRC22:50
*** KeithMnemonic has quit IRC23:08
*** slaweq has joined #openstack-sdks23:11
*** slaweq has quit IRC23:16
*** mriedem has quit IRC23:50

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