| *** bobh has joined #openstack-sdks | 01:24 | |
| *** zigo has quit IRC | 02:29 | |
| *** zigo has joined #openstack-sdks | 02:31 | |
| *** lbragstad has joined #openstack-sdks | 02:45 | |
| *** bobh has quit IRC | 03:02 | |
| *** bobh has joined #openstack-sdks | 03:05 | |
| *** dave-mccowan has quit IRC | 03:05 | |
| openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | 03:09 |
|---|---|---|
| *** bobh has quit IRC | 03:16 | |
| *** dayou has quit IRC | 04:06 | |
| *** smcginnis has quit IRC | 04:06 | |
| *** dayou has joined #openstack-sdks | 04:06 | |
| *** lbragstad has quit IRC | 04:21 | |
| *** gildub_ has quit IRC | 05:13 | |
| *** olivierb has joined #openstack-sdks | 05:30 | |
| *** Luzi has joined #openstack-sdks | 07:29 | |
| *** tosky has joined #openstack-sdks | 07:45 | |
| *** jpich has joined #openstack-sdks | 07:46 | |
| openstackgerrit | Merged openstack/openstacksdk master: Add functional tests for clustering https://review.openstack.org/607061 | 07:53 |
| *** e0ne has joined #openstack-sdks | 07:59 | |
| *** d0ugal has quit IRC | 08:07 | |
| *** d0ugal has joined #openstack-sdks | 08:08 | |
| *** ttsiouts has joined #openstack-sdks | 08:19 | |
| *** ttsiouts has quit IRC | 08:22 | |
| *** ttsiouts_ has joined #openstack-sdks | 08:22 | |
| *** ttsiouts_ has quit IRC | 08:42 | |
| *** ttsiouts has joined #openstack-sdks | 08:42 | |
| *** e0ne has quit IRC | 08:44 | |
| *** e0ne has joined #openstack-sdks | 08:46 | |
| *** slaweq has quit IRC | 08:46 | |
| *** ttsiouts has quit IRC | 08:47 | |
| *** ttsiouts has joined #openstack-sdks | 08:56 | |
| *** e0ne has quit IRC | 09:13 | |
| *** ttsiouts has quit IRC | 09:16 | |
| *** e0ne has joined #openstack-sdks | 09:16 | |
| *** ttsiouts has joined #openstack-sdks | 09:16 | |
| *** ttsiouts_ has joined #openstack-sdks | 09:18 | |
| *** ttsiouts has quit IRC | 09:21 | |
| *** dtantsur|afk is now known as dtantsur | 09:33 | |
| *** ttsiouts_ has quit IRC | 09:37 | |
| *** ttsiouts has joined #openstack-sdks | 09:39 | |
| openstackgerrit | Vishakha Agarwal proposed openstack/python-openstackclient master: Modify the help message for 'registered limit set' https://review.openstack.org/609985 | 09:50 |
| *** ttsiouts has quit IRC | 09:53 | |
| *** imacdonn has quit IRC | 09:53 | |
| *** imacdonn has joined #openstack-sdks | 09:53 | |
| *** Luzi has quit IRC | 09:55 | |
| *** slaweq has joined #openstack-sdks | 09:56 | |
| *** ttsiouts has joined #openstack-sdks | 09:57 | |
| *** ttsiouts has quit IRC | 10:01 | |
| *** e0ne has quit IRC | 10:34 | |
| *** jpena|off has quit IRC | 10:41 | |
| *** jpena|off has joined #openstack-sdks | 10:42 | |
| *** ttsiouts has joined #openstack-sdks | 11:08 | |
| *** dayou has quit IRC | 11:09 | |
| *** dayou has joined #openstack-sdks | 11:14 | |
| *** ttsiouts has quit IRC | 11:16 | |
| *** ttsiouts has joined #openstack-sdks | 11:17 | |
| *** ttsiouts has quit IRC | 11:21 | |
| *** ttsiouts has joined #openstack-sdks | 11:23 | |
| *** e0ne has joined #openstack-sdks | 11:27 | |
| *** ttsiouts has quit IRC | 11:27 | |
| openstackgerrit | Matthias Lisin proposed openstack/openstacksdk master: openstackcloud.py: Implement FWaaS wrapper methods. https://review.openstack.org/604811 | 11:32 |
| *** dave-mccowan has joined #openstack-sdks | 11:38 | |
| *** ttsiouts has joined #openstack-sdks | 11:39 | |
| *** bobh has joined #openstack-sdks | 12:01 | |
| *** ttsiouts has quit IRC | 12:03 | |
| *** ttsiouts has joined #openstack-sdks | 12:03 | |
| *** ttsiouts has quit IRC | 12:08 | |
| *** bobh has quit IRC | 12:09 | |
| *** ttsiouts has joined #openstack-sdks | 12:11 | |
| *** e0ne has quit IRC | 12:16 | |
| *** e0ne has joined #openstack-sdks | 12:17 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Switch bare metal NIC actions in OpenStackCloud to baremetal Proxy calls https://review.openstack.org/610024 | 12:51 |
| *** mriedem has joined #openstack-sdks | 12:58 | |
| dtantsur | Shrews: hi, if you have a minute: https://review.openstack.org/#/c/609399/ | 13:01 |
| *** bobh has joined #openstack-sdks | 13:06 | |
| *** dtantsur is now known as dtantsur|afk | 13:07 | |
| *** markvoelker has quit IRC | 13:15 | |
| Shrews | dtantsur|afk: sure, will take a look in a little bit | 13:16 |
| *** ttsiouts has quit IRC | 13:20 | |
| *** ttsiouts has joined #openstack-sdks | 13:22 | |
| *** mriedem is now known as hansmoleman | 13:22 | |
| *** lbragstad has joined #openstack-sdks | 13:23 | |
| *** munimeha1 has joined #openstack-sdks | 13:23 | |
| *** munimeha1 has quit IRC | 13:38 | |
| *** stephenfin is now known as finucannot | 13:41 | |
| mordred | oh my god the discovery patch landed | 13:43 |
| *** irclogbot_0 has joined #openstack-sdks | 13:44 | |
| edleafe | mordred: ?? Which patch? | 13:46 |
| mordred | edleafe: Merged openstack/openstacksdk master: Use discovery instead of config to create proxies https://review.openstack.org/600292 | 13:46 |
| mordred | edleafe: sdk is now fully discovery driven | 13:46 |
| mordred | with config values only used for override | 13:47 |
| edleafe | mordred: ah. For a second there, I thought you were referring to https://review.openstack.org/459405 | 13:47 |
| mordred | edleafe: bah. like we'll ever finish that. it's too hard :) | 13:48 |
| edleafe | nah, we have dtantsur|afk on it. | 13:48 |
| mordred | good point | 13:49 |
| *** irclogbot_0 has quit IRC | 13:49 | |
| Shrews | mordred: good grief. i +A'd that DAYS ago | 13:49 |
| mordred | mnaser: do you know who a good person is to ping about the broken magnum job for sdk? | 13:49 |
| Shrews | i assumed it merged already | 13:49 |
| mordred | Shrews: there was an oslo.messaging release which broke bifrost which broke the sdk bifrost gate | 13:50 |
| Shrews | see... this is why we should test in a silo | 13:50 |
| mnaser | mordred: I think spyros (PTL) is usually pretty good about that stuff. I can’t recall his IRC nick off the top of my head | 13:50 |
| mnaser | mordred: strigazi | 13:51 |
| mordred | awesome- thanks! | 13:51 |
| mordred | mnaser: openstack-magnum doesn't seem to be their irc channel ... | 13:51 |
| *** irclogbot_0 has joined #openstack-sdks | 13:53 | |
| hansmoleman | dtroyer: i was just reminded of this bug i opened for osc https://storyboard.openstack.org/#!/story/2002158 - curious as to how to handle that. you can't set a volume as read-only while it's being created, but the option is on the volume create CLI. | 13:55 |
| hansmoleman | there is a --wait option on the volume create CLI, so if --wait and --read-only, then wait for it to be available and then make the read-only change? | 13:55 |
| hansmoleman | or make --read-only imply --wait? | 13:55 |
| hansmoleman | oh nvm, i guess there is no --wait option for volume create | 13:56 |
| hansmoleman | that's on server create | 13:56 |
| hansmoleman | welp, i could add --wait to volume create and then build on that for --read-only, but that wouldn't work for anyone using older versions of osc | 13:56 |
| *** irclogbot_0 has quit IRC | 13:59 | |
| *** irclogbot_0 has joined #openstack-sdks | 13:59 | |
| mnaser | mordred: #openstack-containers is the one to look at | 14:04 |
| *** irclogbot_0 has quit IRC | 14:18 | |
| *** lbragstad is now known as elbragstad | 14:25 | |
| openstackgerrit | Chuck Short proposed openstack/os-client-config master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610051 | 14:25 |
| *** markvoelker has joined #openstack-sdks | 14:41 | |
| mordred | hansmoleman: does the volume have to be in ready state to have its readonly attribute set? | 14:42 |
| hansmoleman | yeah it has to be available | 14:43 |
| mordred | how fun | 14:43 |
| hansmoleman | $ openstack volume create --size 1 --read-only read-only-vol1 | 14:43 |
| hansmoleman | Failed to set volume read-only access mode flag: Invalid volume: Volume 70539836-fbb9-478d-9cd1-0e733c0eb0cc status must be available to update readonly flag, but current status is: creating. (HTTP 400) (Request-ID: req-ac54552a-3784-4028-8262-8a600844c186) | 14:43 |
| mordred | well - I'd certainly support --read-only implying --wait | 14:43 |
| mordred | because otherwise it's a pretty useless flag | 14:43 |
| hansmoleman | either that or that option just needs to die because it never worked | 14:43 |
| hansmoleman | and was clearly not tested when it was added | 14:44 |
| mordred | yah | 14:44 |
| openstackgerrit | Chuck Short proposed openstack/os-client-config master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610051 | 14:44 |
| mordred | hansmoleman: I say add a wait, make --read-only imply it, and don't worry about backwards compat bcaues the previous thing never worked | 14:44 |
| hansmoleman | there is no --wait on the volume create CLI, so i figure i can just make --read-only wait and i'll update the help on the option to say if you use this, the command will wait | 14:44 |
| hansmoleman | or that | 14:44 |
| hansmoleman | ok that wfm | 14:45 |
| mordred | \o/ | 14:47 |
| * mordred is helpful today | 14:47 | |
| openstackgerrit | Merged openstack/openstacksdk master: cloud: rename with deprecation validate_node -> validate_machine https://review.openstack.org/609399 | 14:57 |
| *** e0ne has quit IRC | 14:58 | |
| *** e0ne has joined #openstack-sdks | 15:06 | |
| *** irclogbot_0 has joined #openstack-sdks | 15:15 | |
| * edleafe checks the alignment of the planets | 15:16 | |
| *** irclogbot_0 has quit IRC | 15:21 | |
| *** irclogbot_0 has joined #openstack-sdks | 15:21 | |
| *** ttsiouts has quit IRC | 15:31 | |
| *** e0ne has quit IRC | 15:37 | |
| *** e0ne has joined #openstack-sdks | 15:40 | |
| *** e0ne has quit IRC | 15:40 | |
| *** irclogbot_0 has quit IRC | 15:50 | |
| *** markvoelker has quit IRC | 15:51 | |
| *** markvoelker has joined #openstack-sdks | 15:52 | |
| *** finucannot is now known as stephenfin | 16:01 | |
| *** bobh has quit IRC | 16:12 | |
| *** jpich has quit IRC | 16:17 | |
| *** hansmoleman is now known as mriedem_away | 16:18 | |
| *** bobh has joined #openstack-sdks | 16:23 | |
| *** e0ne has joined #openstack-sdks | 17:28 | |
| *** olivierb has quit IRC | 18:21 | |
| *** e0ne has quit IRC | 18:37 | |
| *** kmalloc is now known as needscoffee | 18:48 | |
| *** pooja-jadhav has joined #openstack-sdks | 19:03 | |
| *** pooja_jadhav has quit IRC | 19:05 | |
| *** pooja_jadhav has joined #openstack-sdks | 19:10 | |
| *** pooja-jadhav has quit IRC | 19:13 | |
| openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Handle not having cinderclient.v1 available https://review.openstack.org/609473 | 19:19 |
| *** smcginnis has joined #openstack-sdks | 19:51 | |
| smcginnis | mordred or dtroyer: If one of you has a few minutes - I have a couple openstack volume command additions with one +2 that I would like to make sure are good before proceeding on more updates. | 19:52 |
| smcginnis | https://review.openstack.org/#/q/owner:sean.mcginnis%2540gmail.com+status:open+project:openstack/python-openstackclient | 19:53 |
| dtroyer | smcginnis: I'll have a look in a bit | 19:53 |
| smcginnis | Thanks dtroyer! | 19:54 |
| *** needscoffee is now known as kmalloc | 20:20 | |
| dtroyer | smcginnis: around? now that I see both commands I have a question… | 20:31 |
| smcginnis | Sure! | 20:31 |
| dtroyer | I had assumed 'volume backend capability' and 'volume backend pool' were your resource names | 20:31 |
| dtroyer | you added 'volume backend' to commands.rst | 20:32 |
| dtroyer | maybe that should be the resource name and —capability and —pool are attributes? dunno, trying to sort out how to thing about these | 20:32 |
| smcginnis | I think that may be confusing. Thinking though... | 20:33 |
| dtroyer | it isn't obvious to me, but I don't have the data models around the backends in my head either | 20:34 |
| smcginnis | I guess you could look at it that way that a backend has a set of capabilities and one or more pools, so I suppose you could use flags for those attributes of the backend. | 20:35 |
| smcginnis | Doesn't really sit right with me though. | 20:35 |
| dtroyer | will there ever be other verbs for this/these resources? | 20:35 |
| dtroyer | maybe it feels like capabilities are attributes but pools are resources | 20:35 |
| smcginnis | What would be the expected behavior of not providing a flag. List both capabilities and pools? | 20:35 |
| dtroyer | so let's go with what you have… and just update commands.rst to list ehm both separately | 20:35 |
| smcginnis | I don't think there will be any other verbs. At least nothing on the horizon that I can tell. | 20:36 |
| dtroyer | ok, yeah, that seals it, too ambiguous | 20:36 |
| smcginnis | Yeah, I kind of like how unambiguous this turned out after changing directions from where I started off. | 20:36 |
| smcginnis | So I should update commands.rst to have each one separately? | 20:36 |
| smcginnis | And it would just be "openstack volume backend capability" and "openstack volume backend pool", no verbs, right? | 20:37 |
| dtroyer | right… I'm leaving comments | 20:38 |
| smcginnis | OK, great. Would it be OK if I did that in a follow up patch? There's one or two other internal things I would like to clean up too. | 20:39 |
| smcginnis | Guess that's what I get for using one of Monty's release notes as a template. :D | 20:40 |
| dtroyer | that works for me, they both had tweaks needed :) | 20:40 |
| smcginnis | Will get that going right now... | 20:40 |
| dtroyer | I re-write release notes before a release to make then have at least a similar voice | 20:40 |
| dtroyer | ok, if you're going to do a follow-up I'll +W these now | 20:41 |
| smcginnis | Great. Often overlooked, but I think that makes them much easier to consume when projects do that. | 20:41 |
| dtroyer | in the v1 change, is importing from cinderclient.v3 going to be a problem for v2 commands? | 20:43 |
| dtroyer | smcginnis: basically I just want to be sure that importing v3 but not setting microversions keeps us at v2-compat? | 20:45 |
| mordred | smcginnis: never copy from me :) | 20:45 |
| smcginnis | dtroyer: No, the base v3.0 is identical to v2. It's only after additional microversions (>=3.1) that things start to diverge. | 20:45 |
| smcginnis | mordred: ;) | 20:45 |
| dtroyer | ok, good | 20:45 |
| dtroyer | as soon as an SDK 1.0 hits we can start playing with microversions with gusto | 20:46 |
| smcginnis | I thought of going with v2 as the default, but since they are the same and we eventually someday maybe would like to get down to one version, I thought v3 would be best. | 20:46 |
| mordred | discovery landed today | 20:46 |
| mordred | so I'm thinking of cutting an rc next week | 20:46 |
| dtroyer | \o/ | 20:46 |
| mordred | although I just started hitting the existing sdk glance code with a giant stick | 20:46 |
| dtroyer | just when I'm tied up with releasing stx… | 20:46 |
| smcginnis | Great, I was going to ask about mv support. We have a bunch of cinder commands I would like to add but they are for microversioned things. | 20:46 |
| mordred | but - you know - we'll be 1.0 enough for osc :) | 20:46 |
| smcginnis | :) | 20:46 |
| mordred | smcginnis: dtantsur|afk has been doing a decent amount with mv and ironic in sdk - so not only do we support discovering/configuring them - we've even got some examples of doing something with them :) | 20:47 |
| mordred | and - of course - the joy that is live-migrate | 20:47 |
| smcginnis | Perfect, maybe I can take a look at that work and try to contribute some for cinder. | 20:48 |
| mordred | ++ | 20:48 |
| mordred | speaking of - I actually have something in a glance patch i should do for cinder too | 20:48 |
| *** bobh has quit IRC | 20:48 | |
| mordred | namely - adding support for a base proxy class that has shared code - so for places where it's all the same we don't have to duplicate as much | 20:49 |
| mordred | I think that'll be nice for cinder v2/v3 | 20:49 |
| mordred | I may put in the basics and then ask for your eyes on that? | 20:49 |
| *** mriedem_away is now known as mriedem | 20:51 | |
| dtroyer | fwiw, I explicitly did not do that in OSC so we would never have to decide which version that shared code might affect. I _think_ that is still a good decision, maybe the effects are different at that level… | 20:52 |
| smcginnis | mordred: Perfect, I love reducing code. | 20:52 |
| dtroyer | also, we'll remove cinder v1 with mostly a git rm | 20:52 |
| smcginnis | I think that should be OK here with cinder v2/v3. If they are not the same without microversions, then we messed up. | 20:53 |
| dtroyer | someday when I'm old(er) | 20:53 |
| smcginnis | ;) | 20:53 |
| mordred | dtroyer: yah - I think for the most part it's clearer to read when it's just separate | 20:53 |
| smcginnis | If we can have v2 just duplicate off of v3, then removal should be quick and easy if we ever get to that point. | 20:53 |
| smcginnis | Either way. | 20:53 |
| dtroyer | anyway, enough from me, back to setting up docs for the flock-o-birds | 20:53 |
| mordred | dtroyer: the specific case where I added a base class was so I could do create_image - which is super complex and has logic that applies regardless of version - and then forking logic per-version | 20:53 |
| dtroyer | mordred: ah, makes sense in cases like that | 20:54 |
| mordred | to handle that, I made a BaseImageProxy - and then had image.v1.Proxy and image.v2.Proxy both subclass it | 20:54 |
| mordred | dtroyer: speaking of - when we start ripping out glanceclient - I have a some very shiny image upload code we can take care of | 20:55 |
| openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Address issues from volume backend commands https://review.openstack.org/610161 | 20:55 |
| mordred | andby shiny, I mean dirty dirty dirty nasty dirty | 20:55 |
| smcginnis | dtroyer: Good luck with the release. | 20:56 |
| mordred | \o/ | 20:56 |
| * mordred hands dtroyer a bunny rabbit of luck | 20:56 | |
| *** dave-mccowan has quit IRC | 20:57 | |
| dtroyer | thanks guys, we've got 12 days so no crunching sounds from behind… yet… | 20:57 |
| smcginnis | dtroyer: It's probably completely different than what we've got here, but let me know if you run into anything I can help with. | 20:58 |
| dtroyer | smcginnis: not completely different, I didn't want to invent new, mostly we're just not doing a bunch of things. But I'm finding I may regret choosing 'r/' for the release branch prefix. Many of the exiting tools assume 'stable/' | 21:00 |
| dtroyer | like we don't publish anything other than docs, the final release is really just the tags in git. but the docs… whee! I'm learning a LOT | 21:02 |
| smcginnis | Heh, yeah. I think it's sprinkled everywhere assuming stable/ | 21:02 |
| dtroyer | I'm trying to decide if it is too late to change. Maybe for this release it is but it's also basically an alpha release. MAybe for the one in MArch we will change it… | 21:03 |
| smcginnis | You may end up having changed enough by that point not to, but if we ever think there may be some convergence point in the future between these, it might make sense to try to stay with stable. | 21:04 |
| smcginnis | Not sure if that's a realistic thing or not. | 21:04 |
| smcginnis | But at least picking up future release automation changes from openstack, it might make it easier if there's less to tweak in adopting it. | 21:05 |
| dtroyer | honestly the convergence I see is possibly taking a project or two and making them look even more like OpenStack projects, they might be useful for your average Joe Cloud too… | 21:06 |
| smcginnis | Yeah, hopefully that's the case. | 21:07 |
| dtroyer | I am stealing^H^H^H^Hborrowing as much as I can, thanks for all the fish! | 21:07 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add stackviz processing to functional tests https://review.openstack.org/610167 | 21:08 |
| smcginnis | I would definitely do the same. | 21:15 |
| samueldmq | hi, is there a description of what each one of SDK's gate jobs does? | 21:22 |
| samueldmq | mordred: ^ | 21:22 |
| samueldmq | it's not too hard to have an idea with the job name, but I was wondering if I could confirm that somehow | 21:23 |
| openstackgerrit | Merged openstack/os-client-config master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610051 | 21:37 |
| smcginnis | samueldmq: You should be able to take a look at the logs to see what commands the jobs are running. | 21:39 |
| smcginnis | samueldmq: Or search for the job name in codesearch.openstack.org to track down its definition. | 21:40 |
| *** tosky has quit IRC | 22:02 | |
| *** mriedem is now known as mriedem_away | 22:05 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Add volume backend capability show command https://review.openstack.org/609122 | 22:41 |
| openstackgerrit | Merged openstack/python-openstackclient master: Add volume backend pool list command https://review.openstack.org/608740 | 22:41 |
| openstackgerrit | Merged openstack/python-openstackclient master: Allow endpoint filtering on both project and project-domain https://review.openstack.org/608912 | 22:41 |
| *** elbragstad has quit IRC | 22:56 | |
| *** elbragstad has joined #openstack-sdks | 23:15 | |
| *** elbragstad has quit IRC | 23:15 | |
| samueldmq | smcginnis: Awesome. Thanks | 23:19 |
| *** slaweq has quit IRC | 23:32 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!