Friday, 2019-04-05

*** samueldmq has quit IRC00:29
*** whoami-rajat has joined #openstack-sdks00:48
*** ttsiouts has joined #openstack-sdks01:32
*** ttsiouts has quit IRC02:05
*** markvoelker has joined #openstack-sdks02:18
*** markvoelker has quit IRC02:52
*** markvoelker has joined #openstack-sdks03:49
*** ttsiouts has joined #openstack-sdks04:03
*** markvoelker has quit IRC04:22
*** ttsiouts has quit IRC04:36
*** ttsiouts has joined #openstack-sdks04:46
*** markvoelker has joined #openstack-sdks05:19
*** markvoelker has quit IRC05:52
*** Luzi has joined #openstack-sdks05:58
stephenfinmordred: Are we collecting things for osc 4.0? Is so, thoughts on converting the likes of 'openstack security group create' to 'openstack security-group create' (with aliases, maybe)? The former feels wrong 'openstack security' isn't a command and I think it was only done because of cliff's use of underscores in entry point names06:09
*** ttsiouts has quit IRC06:13
*** ignatenkobrain has quit IRC06:31
*** e0ne has joined #openstack-sdks06:35
*** ignatenkobrain has joined #openstack-sdks06:40
*** slaweq has joined #openstack-sdks06:42
*** gtema has joined #openstack-sdks06:51
*** ttsiouts has joined #openstack-sdks07:04
*** e0ne has quit IRC07:07
*** ITD27M01 has joined #openstack-sdks07:09
*** gouthamr has quit IRC07:11
*** dulek has quit IRC07:11
*** dmellado has quit IRC07:12
*** gkadam has joined #openstack-sdks07:14
*** tosky has joined #openstack-sdks07:16
*** dmellado has joined #openstack-sdks07:18
*** gouthamr has joined #openstack-sdks07:20
*** e0ne has joined #openstack-sdks07:21
*** dulek has joined #openstack-sdks07:22
*** e0ne has quit IRC07:25
*** holser_ has joined #openstack-sdks07:36
*** markvoelker has joined #openstack-sdks07:49
*** jpena|off is now known as jpena07:52
*** jpich has joined #openstack-sdks07:52
*** ralonsoh has joined #openstack-sdks07:59
*** Luzi has quit IRC08:16
*** slaweq has quit IRC08:18
fricklerstephenfin: mordred: I'd say keep those consistent with "floating ip", "subnet pool" and others. you also most certainly don't want "security-group-rule create" in addition08:22
*** markvoelker has quit IRC08:23
fricklerif you want to change something in that regard, my vote would be to make "key pair" instead of "keypair", because to me that is inconsistent with all the other variants and always makes me need two attempts to get right08:23
*** slaweq has joined #openstack-sdks08:27
*** slaweq has quit IRC08:32
*** slaweq has joined #openstack-sdks08:34
*** dtantsur|afk is now known as dtantsur08:41
*** e0ne has joined #openstack-sdks09:00
*** e0ne has quit IRC09:02
*** e0ne has joined #openstack-sdks09:20
*** markvoelker has joined #openstack-sdks09:20
*** e0ne has quit IRC09:42
*** e0ne has joined #openstack-sdks09:45
*** ignatenkobrain has quit IRC09:53
*** markvoelker has quit IRC09:54
*** e0ne has quit IRC09:59
*** ignatenkobrain has joined #openstack-sdks10:01
*** yolanda has joined #openstack-sdks10:07
*** holser_ is now known as holser|long_long10:15
stephenfinfrickler: Oh yeah, I mean anything where the top level command ('openstack floating' would fall into this) doesn't make sense or means something entirely different ('openstack subnet' would fall into this)10:16
*** cdent has joined #openstack-sdks10:17
*** cdent has quit IRC10:45
dtantsurstephenfin: I think the idea is to keep it as close to a human language sentence as possible10:46
dtantsurand it's not friendly to users to make them remember where we use dashes and where spaces10:46
dtantsurlike, 'security-group create' makes as much sense as security-group-create10:46
*** markvoelker has joined #openstack-sdks10:50
*** ttsiouts has quit IRC10:54
*** ttsiouts has joined #openstack-sdks10:55
stephenfindtantsur: I'd think of it in terms of 'openstack [resource] [action]'10:58
stephenfinSo 'openstack security-group create' makes far more sense to me. Helps from an autocomplete perspective too10:59
*** ttsiouts has quit IRC10:59
*** ttsiouts has joined #openstack-sdks11:01
openstackgerritJeremy Houser proposed openstack/python-openstackclient master: Remove hard-coded "admin" in test_port  https://review.openstack.org/65008211:06
*** gtema has quit IRC11:10
*** ttsiouts has quit IRC11:11
*** ttsiouts has joined #openstack-sdks11:12
*** cdent has joined #openstack-sdks11:13
dtantsurstephenfin: that's more correct technically, but more confusing to those who don't have a clear idea of what openstack resources are11:15
stephenfinfair point11:15
stephenfinGuess this isn't quite the no-brainer I thought it was :)11:15
dtantsuralso this simple schema can be quite confusing, e.g. when action has a complex subject, like "server add floating ip"11:15
*** ttsiouts has quit IRC11:16
stephenfinyeah, nested resources are trickier11:16
stephenfin'server-floating-ip add' seems icky11:16
dtantsurour schema is not without problems, of course. I still remember TripleO people once ending up with "openstack overcloud scale stack overcloud overcloud" :D11:17
* dtantsur tried to sing that sentence as kind of rapping11:18
*** ttsiouts has joined #openstack-sdks11:21
*** markvoelker has quit IRC11:24
mordredwow. I just woke up to that sentence11:25
dtroyerstephenfin: the multiple work resource names is intentional.  'security group' is the name11:27
dtroyerkeypair is unfortuante and I'm all for fixing that one, there are a couple more that I'd like to change too.  we already did change ip floating to floating ip moving to 3.0, it isn't hard, and aliases are easy too, just add to setup.cfg whatever you like11:28
dtroyerfor a long time I kept the old verb-object format for commands that we started with...11:29
dtroyerdtantsur: that is partyl a result of thinking you have to prefix every resource name, you don't…11:30
*** e0ne has joined #openstack-sdks11:37
*** jpena is now known as jpena|lunch11:38
*** yolanda has quit IRC11:39
*** gtema has joined #openstack-sdks11:48
cdentdtroyer: since you're about and in an osc naming frame of mind, your input on: https://review.openstack.org/#/c/640898/ would be most appreciated11:48
cdent(see gibi's comment on ps5)11:48
mordredgtema: re: https://review.openstack.org/#/c/649329 - something to think about - in the shade layer we put unknown object properties into a 'properties' property on the main object - largely to make it clear to people that they can't necessarily be depended on as part of the object model (this is a thing glance did in v1 and broke in v2) - now granted, we ALSO would leave them on the main body of the12:01
mordredobject so that people could do whatever if stict mode wasn't on12:01
mordredgtema: in looking at that patch, I'm mostly pondering how it'll interact with the goal of being able to eventually get the shade methods to just return Resource objects - whether what you're doing there will make it harder or easier12:02
mordredgtema: I don't have enough coffee in my blood yet to have smart thoughts - mostly wanted to word-vomit at you about that while I try to caffiene myself12:03
gtemayes, it is clear to me this was done in shade/cloud. But we are not able to get "unknown" props for Image service12:03
gtemaand that somehow blocks normal switch from direct get/post/delete in the cloud._image services to proxy methods12:03
mordredyeah. oh - I totally understand the motivation of the patch - I think I'm saying - I'm not sure we should put them on the object itself vs. making a magical properties attribute12:04
mordredwe should ABSOLUTELY expose the unknown stuff12:04
gtemado you want to say that everything unknown should go dynamically under properties?12:05
mordredMAYBE? I think I need more coffee :)12:06
mordredI like your add_attribute thing too ... it actually makes me think it wouldn't be completely insane to fetch the jsonschema objects glance publishes and use that metadata somehow12:06
gtemamaybe, I just got my second one and that should be it for today, otherwise I will create some crazy patches12:06
* mordred connects coffee IV to gtema and stands back to watch12:06
gtemaas far as I remember there were also some issues with setting "sdk" shaXXX properties in the image uploads and objects12:07
gtemado not joke with me and coffee - it might end bad12:07
gtemait's affects me like a drug12:08
gtemareally :D12:08
mordredah. then yes - please to only have the second cup :)12:08
gtemaso I was not somehow able to switch all of the _image and _object methods to use proxy due to those "dynamic" things12:09
gtemayou might have fixed _object however12:09
gtemayeah - _SHADE_IMAGE_OBJECT_KEY12:11
gtemathis was one of the problem s12:11
gtemaso basically glance accepts anything and we need to be able to set this "anything" in Resource12:12
*** e0ne has quit IRC12:15
mordredyeah12:16
*** e0ne has joined #openstack-sdks12:33
*** cdent has quit IRC12:37
*** whoami-rajat has quit IRC12:37
dtroyercdent: looking… at a glance, the way OSC typically sorts these things is to start with the object being manipulated, that is the name that appears first in the command followed by a verb and then the rest of the options.  I'll need to tap into mordred's coffee IV and digest that in detail to get a sense of what I would have done...12:39
*** jpena|lunch is now known as jpena12:40
* mordred passes the bag to dtroyer12:40
*** whoami-rajat has joined #openstack-sdks12:54
*** cdent has joined #openstack-sdks13:11
openstackgerritJeremy Houser proposed openstack/python-openstackclient master: Remove hard-coded "admin" in test_port  https://review.openstack.org/65008213:11
openstackgerritJeremy Houser proposed openstack/python-openstackclient master: Remove hard-coded "admin" in test_port  https://review.openstack.org/65008213:11
openstackgerritJeremy Houser proposed openstack/python-openstackclient master: Remove hard-coded "admin" in test_port  https://review.openstack.org/65008213:12
openstackgerritMerged openstack/openstacksdk master: Expand on a brief error message  https://review.openstack.org/65016113:30
dtroyercdent: ok, there, I stuck my nose in y'alls business… https://review.openstack.org/#/c/640898/513:30
*** stephenfin is now known as finucannot13:31
cdentthanks dtroyer. I'm attempting to decode what you've written. Should have it once my post lunch fuzz has waned.13:34
*** e0ne has quit IRC13:45
*** e0ne has joined #openstack-sdks13:50
*** e0ne has quit IRC13:54
*** ITD27M01 has quit IRC14:01
*** e0ne has joined #openstack-sdks14:07
*** e0ne has quit IRC14:14
*** e0ne has joined #openstack-sdks14:15
*** ttsiouts has quit IRC14:22
*** ttsiouts has joined #openstack-sdks14:23
*** ttsiouts has quit IRC14:24
*** ttsiouts has joined #openstack-sdks14:24
*** e0ne has quit IRC14:57
*** gtema has quit IRC14:58
dtroyercdent: hey, you found my gabbi experiment :)   since I am working against a running devstack I think I need to actually get a token with SDK to set up for the test to include, right?  anyway, that's where I am right now…I'll try one more thing then push it up to show you where I've wandered14:58
cdentyeah, if you're running like you need a token14:59
cdentyou might be able to clean some magic from my gabbi-tempest plugin (which is on git.o.o)14:59
cdents/like/live/14:59
*** ttsiouts has quit IRC15:00
cdents/clean/glean/15:00
dtroyercool, I had not studied that yet15:00
cdentapparently I have not recovered from post-lunch fuzz at all15:00
* cdent makes coffee15:00
*** gtema has joined #openstack-sdks15:00
*** ttsiouts has joined #openstack-sdks15:00
dtroyeris my agregate working to opaque?  I may have still been in that sort of shape myself when I wrote it15:00
dtroyers/working/wording/15:00
cdentI gave you my disease!15:01
cdenthuzzah15:01
cdentI think mel was able to grok it properly, and likes it, so gtg15:01
*** ttsiouts has quit IRC15:01
* dtroyer looks15:01
*** ttsiouts has joined #openstack-sdks15:02
dtroyeryeah, she seems to have deciphered it.15:02
*** e0ne has joined #openstack-sdks15:03
*** whoami-rajat has quit IRC15:07
openstackgerritLIU Yulong proposed openstack/openstacksdk master: Add floating IP port forwarding related methods  https://review.openstack.org/65006315:11
*** yolanda has joined #openstack-sdks15:13
*** e0ne has quit IRC15:16
*** ttsiouts has quit IRC15:25
cdentdtroyer: Instead of post-lunch fuzz, I'm going with "this is a day where I need my reading glasses"15:25
*** ttsiouts has joined #openstack-sdks15:26
*** ttsiouts has quit IRC15:30
*** holser_ has joined #openstack-sdks15:32
openstackgerritDmitry Tantsur proposed openstack/openstacksdk master: [WIP] Support for the baremetal introspection service  https://review.openstack.org/65004415:36
*** yolanda has quit IRC15:52
openstackgerritDmitry Tantsur proposed openstack/openstacksdk master: [WIP] Support for the baremetal introspection service  https://review.openstack.org/65004415:59
*** dtantsur is now known as dtantsur|afk16:01
*** jpich has quit IRC16:06
*** jangutter has quit IRC16:06
openstackgerritLIU Yulong proposed openstack/python-openstackclient master: Add floating IP Port Forwarding commands  https://review.openstack.org/65006216:15
openstackgerritArtem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK  https://review.openstack.org/65037416:20
*** gkadam has quit IRC16:25
*** holser_ has quit IRC16:34
*** holser_ has joined #openstack-sdks16:41
*** cmurphy is now known as cmorpheus16:47
openstackgerritArtem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK  https://review.openstack.org/65037417:04
*** jpena is now known as jpena|off17:04
*** gtema has quit IRC17:11
*** e0ne has joined #openstack-sdks17:12
*** holser_ has quit IRC17:22
*** e0ne has quit IRC17:24
*** whoami-rajat has joined #openstack-sdks17:37
*** e0ne has joined #openstack-sdks17:37
*** Jeremy has joined #openstack-sdks17:42
*** Jeremy is now known as Guest7260117:43
*** Guest72601 is now known as JeremyHouser17:43
JeremyHouserQuick question: are there any current known bugs related to openstack quotas in python-openstackclient?17:44
*** kmalloc is now known as needscoffee17:46
*** TheJulia is now known as needssleep17:47
openstackgerritMerged openstack/python-openstackclient master: Volume backup functional test tweak  https://review.openstack.org/64932717:52
*** JeremyHouser has quit IRC18:29
*** ralonsoh has quit IRC18:45
*** e0ne has quit IRC19:02
*** e0ne has joined #openstack-sdks19:03
*** slaweq has quit IRC19:51
*** e0ne has quit IRC19:53
openstackgerritMerged openstack/os-service-types master: add python 3.7 unit test job  https://review.openstack.org/64739720:16
*** slaweq has joined #openstack-sdks20:18
*** slaweq has quit IRC20:23
*** holser_ has joined #openstack-sdks21:32
*** cdent has quit IRC21:55
*** manfred has joined #openstack-sdks21:58
*** tobiash_ has joined #openstack-sdks21:59
*** dasp has joined #openstack-sdks21:59
*** mgariepy has quit IRC22:01
*** tobiash has quit IRC22:01
*** tbarron has quit IRC22:01
*** zigo has quit IRC22:01
*** frickler has quit IRC22:01
*** brtknr has quit IRC22:01
*** aspiers has quit IRC22:01
*** gtmanfred has quit IRC22:01
*** dayou_ has quit IRC22:01
*** dasp_ has quit IRC22:01
*** frickler has joined #openstack-sdks22:01
*** dayou_ has joined #openstack-sdks22:02
*** manfred is now known as gtmanfred22:02
*** holser_ has quit IRC22:16
*** aspiers has joined #openstack-sdks22:24
*** whoami-rajat has quit IRC22:27
*** openstackgerrit has quit IRC22:28
*** tosky has quit IRC23:11
*** needscoffee is now known as kmalloc23:32

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