*** markvoelker has quit IRC | 00:17 | |
*** markvoelker has joined #openstack-sdks | 00:17 | |
*** markvoelker has quit IRC | 00:21 | |
*** Qiming has quit IRC | 01:06 | |
*** Qiming has joined #openstack-sdks | 01:09 | |
*** openstackgerrit has joined #openstack-sdks | 01:30 | |
openstackgerrit | Merged openstack/python-openstackclient master: Add metavar for name parameter in subnet create https://review.openstack.org/602062 | 01:30 |
---|---|---|
*** dave-mccowan has joined #openstack-sdks | 01:56 | |
*** markvoelker has joined #openstack-sdks | 02:18 | |
openstackgerrit | Andriy Shevchenko proposed openstack/python-openstackclient master: fix tox python3 overrides https://review.openstack.org/607453 | 02:32 |
*** markvoelker has quit IRC | 02:51 | |
*** dave-mccowan has quit IRC | 03:08 | |
*** markvoelker has joined #openstack-sdks | 03:48 | |
*** markvoelker has quit IRC | 04:21 | |
*** gkadam has joined #openstack-sdks | 04:32 | |
*** gkadam has quit IRC | 04:32 | |
*** markvoelker has joined #openstack-sdks | 05:18 | |
*** markvoelker has quit IRC | 05:52 | |
*** dims has quit IRC | 06:24 | |
*** e0ne has joined #openstack-sdks | 06:24 | |
*** dims has joined #openstack-sdks | 06:26 | |
*** dims has quit IRC | 06:34 | |
*** dims has joined #openstack-sdks | 06:35 | |
*** tosky has joined #openstack-sdks | 07:03 | |
*** Horrorcat has quit IRC | 07:19 | |
*** jroll has quit IRC | 07:19 | |
*** dmellado has quit IRC | 07:19 | |
*** odyssey4me has quit IRC | 07:19 | |
*** amotoki has quit IRC | 07:23 | |
*** olivierb has joined #openstack-sdks | 07:24 | |
*** Horrorcat has joined #openstack-sdks | 07:32 | |
*** jroll has joined #openstack-sdks | 07:32 | |
*** dmellado has joined #openstack-sdks | 07:32 | |
*** odyssey4me has joined #openstack-sdks | 07:32 | |
*** jpich has joined #openstack-sdks | 07:35 | |
*** gkadam has joined #openstack-sdks | 07:44 | |
*** amotoki has joined #openstack-sdks | 07:52 | |
*** amotoki has quit IRC | 07:52 | |
*** amotoki has joined #openstack-sdks | 07:53 | |
*** amotoki has quit IRC | 07:57 | |
*** Horrorcat has quit IRC | 08:13 | |
*** jroll has quit IRC | 08:13 | |
*** dmellado has quit IRC | 08:13 | |
*** odyssey4me has quit IRC | 08:13 | |
*** markvoelker has joined #openstack-sdks | 08:18 | |
*** amotoki has joined #openstack-sdks | 08:25 | |
*** Horrorcat has joined #openstack-sdks | 08:27 | |
*** jroll has joined #openstack-sdks | 08:27 | |
*** dmellado has joined #openstack-sdks | 08:27 | |
*** odyssey4me has joined #openstack-sdks | 08:27 | |
*** amotoki_ has joined #openstack-sdks | 08:27 | |
*** amotoki has quit IRC | 08:28 | |
*** cdent has joined #openstack-sdks | 08:31 | |
*** ttsiouts has joined #openstack-sdks | 08:51 | |
*** Horrorcat has quit IRC | 08:51 | |
*** jroll has quit IRC | 08:51 | |
*** dmellado has quit IRC | 08:51 | |
*** odyssey4me has quit IRC | 08:51 | |
*** markvoelker has quit IRC | 08:51 | |
*** ttsiouts has quit IRC | 08:53 | |
*** ttsiouts has joined #openstack-sdks | 08:53 | |
*** ttsiouts has quit IRC | 08:58 | |
*** ttsiouts has joined #openstack-sdks | 09:00 | |
*** Horrorcat has joined #openstack-sdks | 09:05 | |
*** jroll has joined #openstack-sdks | 09:05 | |
*** dmellado has joined #openstack-sdks | 09:05 | |
*** odyssey4me has joined #openstack-sdks | 09:05 | |
*** ttsiouts has quit IRC | 09:07 | |
*** ttsiouts has joined #openstack-sdks | 09:07 | |
*** ttsiouts_ has joined #openstack-sdks | 09:10 | |
*** ttsiouts has quit IRC | 09:12 | |
*** gildub has joined #openstack-sdks | 09:15 | |
*** ttsiouts_ has quit IRC | 10:07 | |
*** pepijn has joined #openstack-sdks | 10:17 | |
*** gkadam has quit IRC | 10:24 | |
*** markvoelker has joined #openstack-sdks | 10:49 | |
*** ttsiouts has joined #openstack-sdks | 11:02 | |
*** amotoki_ is now known as amotoki | 11:07 | |
*** markvoelker has quit IRC | 11:22 | |
*** gildub has quit IRC | 11:38 | |
*** pepijn has quit IRC | 11:59 | |
*** pepijn has joined #openstack-sdks | 12:03 | |
*** e0ne has quit IRC | 12:04 | |
*** ttsiouts has quit IRC | 12:05 | |
*** ttsiouts has joined #openstack-sdks | 12:07 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Split advanced networking services into a job https://review.openstack.org/607199 | 12:11 |
*** ttsiouts has quit IRC | 12:11 | |
ml | mordred: https://review.openstack.org/#/c/592303/ | 12:13 |
ml | am I able to trigger a recheck myself or do you have to reset the workflow? | 12:13 |
mordred | ml: you can - but hold off for now- the issue that's happening is the test node running out of memory | 12:13 |
ml | alright | 12:14 |
mordred | ml: I'm trying to fix with https://review.openstack.org/607199 - if it fixes it, I'll probably just squash it into your patch and merge it that way | 12:14 |
ml | alright, I'll keep an eye on this change. thanks for your work | 12:15 |
*** e0ne has joined #openstack-sdks | 12:16 | |
*** dave-mccowan has joined #openstack-sdks | 12:20 | |
*** pepijn has quit IRC | 12:25 | |
*** ttsiouts has joined #openstack-sdks | 12:31 | |
*** ttsiouts has quit IRC | 12:33 | |
*** ttsiouts has joined #openstack-sdks | 12:34 | |
*** pepijn has joined #openstack-sdks | 12:36 | |
*** rosmaita has joined #openstack-sdks | 12:46 | |
*** rosmaita has left #openstack-sdks | 12:46 | |
*** pepijn has quit IRC | 13:05 | |
*** smcginnis has joined #openstack-sdks | 13:23 | |
*** rosmaita has joined #openstack-sdks | 13:30 | |
rosmaita | cdent: have an api question for you when you have a few minutes | 13:31 |
cdent | rosmaita: i've got time, for you, right now | 13:33 |
rosmaita | cdent: thanks! can you take a look at https://review.openstack.org/#/c/597648/7 / | 13:34 |
rosmaita | (meant that to be a ? at the end) | 13:34 |
* cdent looks | 13:34 | |
*** mriedem has joined #openstack-sdks | 13:37 | |
cdent | rosmaita: okay, I think I'm caught up on the text of the spec and the comments | 13:41 |
cdent | what's the main issue you're concerned with? | 13:42 |
rosmaita | cdent: whether i'm right about us having to introduce a new json-patch mime type for this | 13:42 |
rosmaita | and whether that's a good idea | 13:43 |
smcginnis | Mimes are never a good idea. | 13:43 |
rosmaita | smcginnis: i can't hear you | 13:43 |
rosmaita | :) | 13:43 |
smcginnis | :) | 13:43 |
cdent | you both need to go sit on the naughty step | 13:44 |
cdent | rosmaita: I'd say the issue here is that the validation_data is being marked out as a separate in the representation. If checksum, os_hash_algo, and os_hash_value are legit members of the representation of the 'locations' attribute, then just pass those and _in_code_ be aware that they are validation bits | 13:47 |
cdent | otherwise you are not truly doing a replace operation (from a json-patch standpoint) | 13:47 |
rosmaita | cdent: tjat | 13:47 |
rosmaita | that's the key issue, we aren't doing a replace | 13:48 |
cdent | it sure looks like you are :) | 13:48 |
rosmaita | the locations attribute has a 'metadata' property, which is a json object iirc, but it doesn't have individual checksum, os_hash_* properties | 13:49 |
rosmaita | and we don't really want to leave 'checksum' as a location metadata item, we want there to be only one true checksum, as an image property | 13:50 |
*** tosky__ has joined #openstack-sdks | 13:51 | |
*** tosky has quit IRC | 13:51 | |
rosmaita | cdent: but that's probably too much detail. i guess the key thing is that a json-patch replace operation should actually replace stuff or fail (and not do weird side-effect stuff like we're proposing) | 13:51 |
*** tosky__ is now known as tosky | 13:52 | |
cdent | you say in one of your initial comments " because the "validation_data" object does not exist in the image/location schema; it's sort of an instruction, really, that produces a side effect (the population of other properties on that image)". Can you just populate those properties? | 13:52 |
cdent | yes: if you're using json-patch replace, it should do json-patch replace, not some special glance-only magic | 13:52 |
rosmaita | cdent: yes, that's the alternative (but those properties are currently read-only in the schema, which is a sticking point) | 13:52 |
rosmaita | the issue is that we only want these properties to be settable under one specific circumstance, and never at any other time | 13:53 |
cdent | making them writable during certain states is probably OK | 13:53 |
cdent | jinxish | 13:53 |
cdent | rosmaita: Have I helped or just made it worse? | 13:58 |
rosmaita | cdent: thinking! | 13:58 |
rosmaita | cdent: last question ... if we make these writable under one tiny circumstance, then we have to remove readOnly: true from the schema for them, right? | 13:59 |
cdent | I think that depends on how you are using and publishing the schema. In my mind schema are a validation tool used indepdently for each method:resource pair, not a contract in an of themselves | 14:01 |
*** olivierb_ has joined #openstack-sdks | 14:02 | |
rosmaita | cdent: thanks, that is helpful and will give us something to think about | 14:03 |
cdent | you're welcome | 14:03 |
*** olivierb has quit IRC | 14:03 | |
*** adriant has quit IRC | 14:03 | |
*** adriant has joined #openstack-sdks | 14:04 | |
*** bobh has joined #openstack-sdks | 14:04 | |
*** ttsiouts has quit IRC | 14:30 | |
*** ttsiouts has joined #openstack-sdks | 14:40 | |
*** Shrews has quit IRC | 14:45 | |
*** slaweq has quit IRC | 14:48 | |
*** Shrews has joined #openstack-sdks | 14:53 | |
*** bobh has quit IRC | 14:54 | |
*** slaweq has joined #openstack-sdks | 14:55 | |
*** bobh has joined #openstack-sdks | 15:31 | |
*** ttsiouts has quit IRC | 15:31 | |
*** Luzi has joined #openstack-sdks | 15:47 | |
*** bobh has quit IRC | 16:02 | |
*** slaweq has quit IRC | 16:09 | |
*** slaweq has joined #openstack-sdks | 16:09 | |
*** ttsiouts has joined #openstack-sdks | 16:12 | |
*** ttsiouts has quit IRC | 16:15 | |
*** ttsiouts has joined #openstack-sdks | 16:16 | |
*** ttsiouts has quit IRC | 16:20 | |
*** dims has quit IRC | 16:28 | |
*** olivierb_ has quit IRC | 16:29 | |
*** dims_ has joined #openstack-sdks | 16:35 | |
Luzi | mordred are you there? | 16:37 |
mordred | Luzi: *sortof* - I'm at a conference so I'm in and out | 16:37 |
Luzi | okay, short question: it is planned to use openstacksdk in more projects right? | 16:38 |
Luzi | like nova and cinder maybe? | 16:38 |
smcginnis | Nothing active that I've seen, but not out of the question. | 16:39 |
mordred | Luzi: yes - it's already in use in heat, senlin and masakari | 16:42 |
mordred | Luzi: once the 1.0 is cut, I'll start poking at making patches for folks to make that possible | 16:42 |
mordred | Luzi: I don't know that the various services have agreed that they want such a migration yet - but from my/sdk point of view, once 1.0 is out it's a great idea | 16:43 |
smcginnis | mordred: Forum topic? | 16:43 |
Luzi | mordred, thank you for now :) | 16:44 |
smcginnis | Too late now to be proposed, but we may be able to find a time for it if there is interest/need. | 16:44 |
*** mhen has joined #openstack-sdks | 16:44 | |
*** e0ne has quit IRC | 16:47 | |
mordred | smcginnis: yah. I was supposed to talk to nova about it in denver - but I missed the irc ping | 16:50 |
*** Luzi has quit IRC | 17:02 | |
*** bobh has joined #openstack-sdks | 17:14 | |
*** jpich has quit IRC | 17:32 | |
*** olivierb has joined #openstack-sdks | 17:41 | |
*** e0ne has joined #openstack-sdks | 18:02 | |
*** imacdonn has quit IRC | 18:21 | |
*** imacdonn has joined #openstack-sdks | 18:21 | |
*** cdent has quit IRC | 18:33 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Split advanced networking services into a job https://review.openstack.org/607199 | 18:45 |
*** e0ne has quit IRC | 18:45 | |
*** e0ne has joined #openstack-sdks | 18:48 | |
*** e0ne has quit IRC | 19:10 | |
kmalloc | mordred: how far is SDK away from being ... usable say, in lieu of keystoneclient? | 19:35 |
kmalloc | mordred: i kindof want to leap early towards dropping ksc to the curb (new stuff only in SDK) | 19:36 |
* kmalloc is asking your opinion and will make a personal assessment. | 19:36 | |
mordred | kmalloc: we're very close to a 1.0 | 19:37 |
mordred | kmalloc: as soon asI can get the tests to stop OOMing | 19:37 |
kmalloc | mordred: re-stating what i said in -keystone | 19:38 |
kmalloc | mordred: because i want to jump early like we did for OSC as the cli | 19:38 |
kmalloc | mordred: i would love to draw the line in the sand this release and say "all new stuff goes in SDK, ksc is deprecated and will see no new changes except for compat/security if an API is materially changed [see security reasons]" | 19:39 |
kmalloc | mordred: specifically... for new things like oslo.limit | 19:39 |
kmalloc | mordred: so.. yay! | 19:39 |
kmalloc | mordred: and boo OOMing | 19:39 |
mordred | kmalloc: yes - I tihnk we're in a good place for that | 19:40 |
mordred | kmalloc: I don't know that we're at feature parity for the existing thing | 19:40 |
kmalloc | cool. I'll make my -1 a little less soft on forklifting in ksc into oslo.limit | 19:41 |
kmalloc | and say "lets use SDK" | 19:41 |
kmalloc | (referencing this convo) | 19:42 |
mordred | ++ | 19:42 |
kmalloc | https://review.openstack.org/#/c/596520/13 if you want to weigh in | 19:44 |
kmalloc | commented on it saying it's better to add support to SDK if needed than lean on ksc at this point. | 19:44 |
kmalloc | it's a super small set of API calls. | 19:44 |
kmalloc | we use there. | 19:44 |
mordred | kmalloc: I _certainly_ think it's valuable to start new dev that way. adding comment now | 19:44 |
kmalloc | :) | 19:44 |
mordred | kmalloc: well, and you can just use the rest layer worst case | 19:45 |
kmalloc | yeah, i said in my initial comment, use KSA not KSC. | 19:45 |
kmalloc | but figured I should talk here before firming up the -1 | 19:45 |
kmalloc | now i'm saying "SDK" | 19:45 |
kmalloc | i will also push hard that all new stuff will be in SDK vs in KSC. | 19:45 |
kmalloc | it does mean we'll need to look at how hard it will be to re-write our ksc-plugin for OSC under SDK. | 19:46 |
kmalloc | should be... doable | 19:46 |
kmalloc | if we can make our plugin SDK, then i know everything else could lean on SDK and be 90% parity | 19:46 |
mordred | yes! | 19:46 |
kmalloc | and 90% is good enough imo to deprecate (freeze) ksc. | 19:47 |
kmalloc | and that saves us the headache of fixing ksc's weird session stuff that is there for compat | 19:48 |
kmalloc | mordred: in the interest in sharing eyebleeding patches... because... well we can | 19:50 |
kmalloc | mordred: https://review.openstack.org/#/c/603461/ ::will make your eyes bleed:: | 19:50 |
kmalloc | mordred: but that is the worst forklift of keystone -> flask | 19:51 |
*** gildub has joined #openstack-sdks | 20:06 | |
*** slaweq has quit IRC | 20:10 | |
*** slaweq has joined #openstack-sdks | 20:11 | |
*** gildub has quit IRC | 20:15 | |
*** olivierb has quit IRC | 20:47 | |
*** bobh has quit IRC | 20:51 | |
*** bobh has joined #openstack-sdks | 20:56 | |
dtroyer | kmalloc: it's very easy to change commands to use the SDK, have a look at the network commands for how to use it directly, specifically security groups which handle both neutron and nova-net show how to hop between two at the same time | 21:05 |
kmalloc | Yep. Was looking at it just now | 21:06 |
kmalloc | About to embark on a "strip ksc from osc" crusade :) | 21:06 |
dtroyer | there are also some uses of sdk-network in the compute server commands IIRC | 21:06 |
kmalloc | If osc is on SDK for keystone, I can make the case ksc can die a slow death of bit rot. | 21:07 |
kmalloc | And all new things go sdk (carrot) for keystone. | 21:07 |
kmalloc | Similar to how ksc command line did a while back. | 21:07 |
dtroyer | second verse same as the first… | 21:08 |
kmalloc | Exactly | 21:09 |
* dtroyer now has Herman's Hermits singing away in his head | 21:10 | |
*** slaweq has quit IRC | 21:30 | |
*** rosmaita has left #openstack-sdks | 21:35 | |
kmalloc | woo, just got a new mechanical keyboard, and loving it | 22:04 |
kmalloc | yay cherry silver switches | 22:04 |
kmalloc | (and my other one for my workstation is cherry clears) | 22:04 |
* kmalloc has a mechanical keyboard problem. | 22:05 | |
*** bobh has quit IRC | 22:07 | |
*** slaweq has joined #openstack-sdks | 22:11 | |
*** slaweq has quit IRC | 22:16 | |
*** tosky has quit IRC | 22:24 | |
*** mriedem has quit IRC | 23:51 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!