| openstackgerrit | Sagi Shnaidman proposed openstack/openstacksdk master: Add ansible stable-2.9 job and run 2.8 and 2.9 https://review.opendev.org/711471 | 01:28 |
|---|---|---|
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Don't fetch subunit in jobs https://review.opendev.org/714342 | 01:33 |
| openstackgerrit | Sagi Shnaidman proposed openstack/openstacksdk master: DNM test fetch_subunit https://review.opendev.org/714343 | 01:37 |
| openstackgerrit | Sam Morrison proposed openstack/openstacksdk master: Add availability zone and availability zone profile resources to load balancer https://review.opendev.org/714345 | 02:26 |
| *** __ministry has quit IRC | 03:23 | |
| *** ricolin_ has joined #openstack-sdks | 03:46 | |
| *** ricolin_ has quit IRC | 04:17 | |
| openstackgerrit | Sam Morrison proposed openstack/openstacksdk master: Add availability zone and availability zone profile resources to load balancer https://review.opendev.org/714345 | 04:19 |
| *** evrardjp has quit IRC | 05:36 | |
| *** evrardjp has joined #openstack-sdks | 05:36 | |
| *** slaweq_ has joined #openstack-sdks | 06:31 | |
| *** dtantsur|afk is now known as dtantsur | 07:18 | |
| *** gtema has joined #openstack-sdks | 07:37 | |
| *** lxkong has quit IRC | 07:42 | |
| *** jawad_axd has joined #openstack-sdks | 07:49 | |
| *** gtema has quit IRC | 07:59 | |
| *** ralonsoh has joined #openstack-sdks | 07:59 | |
| *** gtema has joined #openstack-sdks | 08:02 | |
| *** tkajinam has quit IRC | 08:15 | |
| *** tosky has joined #openstack-sdks | 08:32 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Lay a foundation for the project cleanup https://review.opendev.org/700219 | 08:34 |
| *** amoralej|off is now known as amoralej | 08:41 | |
| *** jpena|off is now known as jpena | 08:52 | |
| *** rpittau|afk is now known as rpittau | 08:52 | |
| openstackgerrit | Mark Chappell proposed openstack/openstacksdk master: Add support for not including the ID in creation requests https://review.opendev.org/714120 | 08:55 |
| *** jpich has joined #openstack-sdks | 08:59 | |
| openstackgerrit | Daniel Bengtsson proposed openstack/osc-lib master: Stop configuring install_command in tox and stop use pip. https://review.opendev.org/694510 | 09:01 |
| *** slaweq_ is now known as slaweq | 09:13 | |
| openstackgerrit | Mark Chappell proposed openstack/openstacksdk master: Add support for Identity Providers https://review.opendev.org/714122 | 09:17 |
| *** ccamel has quit IRC | 09:29 | |
| rpittau | good morning everyone, just a doubt looking at the code, in the service_description module, shouldn't we set skip_discovery to True for the proxy object if we have both endpoint_override and version_string? Precisely inside this "if" https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/service_description.py#L174 | 09:29 |
| *** camelCaser has joined #openstack-sdks | 09:31 | |
| *** ralonsoh has quit IRC | 09:44 | |
| *** ralonsoh has joined #openstack-sdks | 09:44 | |
| *** gtema_ has joined #openstack-sdks | 09:56 | |
| *** gtema has quit IRC | 10:00 | |
| dtantsur | I'll look after the mandatory tea | 10:38 |
| *** sshnaidm is now known as sshnaidm|pto | 10:53 | |
| dtantsur | rpittau: why so? there is still information to discover, e.g. the precise location of a major version or available microversions | 10:59 |
| rpittau | dtantsur: this is related to the migration of ironic-inspector to use openstacksdk for the ironic module, when running the functional test it fails with the current conf, I tracked down the issue to the discovery part between openstacksdk and keystone, when creating the proxy object | 11:09 |
| *** gtema has joined #openstack-sdks | 11:09 | |
| dtantsur | rpittau: if you have problems with discovery, it usually means you've got a wrong URL | 11:12 |
| dtantsur | and I do agree that the error message may be quite confusing | 11:12 |
| *** gtema_ has quit IRC | 11:13 | |
| rpittau | dtantsur: yeah, the current config has endpoint_override=http://url , I tried with different ones with no luck | 11:14 |
| dtantsur | rpittau: unless you mock out keystoneauth, it will try accessing that | 11:15 |
| rpittau | dtantsur: I also tried that, maybe I'm mocking it in the wrong place :/ | 11:15 |
| dtantsur | rpittau: just mock the hell out of get_client or how it's called now | 11:16 |
| dtantsur | I don't think it's useful to go deeper | 11:16 |
| *** ouroboros8 has joined #openstack-sdks | 11:21 | |
| rpittau | dtantsur: tried that too :/ again maybe I'm doing something wrong while mocking | 11:22 |
| dtantsur | rpittau: do you have a patch with get_client mocked? if it's mocked properly, it cannot end up in keystoneauth | 11:23 |
| *** ricolin_ has joined #openstack-sdks | 11:23 | |
| rpittau | dtantsur: no, I didn't upload it as it was failing | 11:24 |
| ouroboros8 | I have an open review ( https://review.opendev.org/#/c/713698/ ) for a change to the openstack cli, but one of the later Zuul build steps is failing (osc-promote-image). I've had a look at it but the logs don't really make much sense to me, and I'm not sure what the step is trying to achieve. Anyone able to give me some pointers? | 11:24 |
| dtantsur | ouroboros8: your patch has merged. I'm not sure what that job does, but you may talk to #openstack-infra about it. | 11:26 |
| ouroboros8 | thanks! | 11:27 |
| dtantsur | rpittau: I suggest you upload it and we walk from there? | 11:30 |
| rpittau | dtantsur: alright, will do | 11:30 |
| *** evrardjp has quit IRC | 11:44 | |
| *** rpittau is now known as rpittau|bbl | 11:47 | |
| *** evrardjp has joined #openstack-sdks | 11:50 | |
| *** gtema has quit IRC | 11:55 | |
| *** gtema has joined #openstack-sdks | 11:57 | |
| *** mgariepy has quit IRC | 12:15 | |
| *** evrardjp has quit IRC | 12:27 | |
| *** evrardjp has joined #openstack-sdks | 12:38 | |
| openstackgerrit | Mark Chappell proposed openstack/openstacksdk master: Add support for Federation Protocols https://review.opendev.org/714431 | 12:48 |
| *** Luzi has joined #openstack-sdks | 12:55 | |
| *** jpena is now known as jpena|lunch | 12:56 | |
| *** Luzi_ has joined #openstack-sdks | 13:00 | |
| *** Luzi has quit IRC | 13:01 | |
| *** mgariepy has joined #openstack-sdks | 13:01 | |
| openstackgerrit | jayaditya gupta proposed openstack/python-openstackclient master: FIX for openstack quota set --force https://review.opendev.org/713881 | 13:02 |
| *** rpittau|bbl is now known as rpittau | 13:08 | |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Change dockerhub password https://review.opendev.org/714436 | 13:09 |
| *** amoralej is now known as amoralej|lunch | 13:09 | |
| *** jpich has quit IRC | 13:41 | |
| *** jpich has joined #openstack-sdks | 13:42 | |
| openstackgerrit | Mark Chappell proposed openstack/openstacksdk master: Add support for Federation Protocols https://review.opendev.org/714431 | 13:42 |
| *** Luzi_ has quit IRC | 13:44 | |
| *** gtema_ has joined #openstack-sdks | 13:54 | |
| mordred | dtantsur, rpittau: another option past normal mocking is to use requests-mock ... although that's probably not a _awesome_ choice in the short-term if this isn't in sdk itself. I started work on building a fixture in sdk that can be used in other places: https://review.opendev.org/#/c/675187/ - but am not yet done | 13:55 |
| mordred | not that it helps today - but the idea there would be to be able to make a FakeOpenStackCloud object and tell it what you want it to do and it'll do all the underlying requests-mock magic that we do in the sdk unit tests | 13:56 |
| rpittau | mordred: hey, thanks :) in the end it was actually a bug in ironic-inspector | 13:56 |
| mordred | but in a form that other projects consuming sdk could actually use | 13:56 |
| mordred | rpittau: cool! | 13:56 |
| *** gtema has quit IRC | 13:58 | |
| *** jpena|lunch is now known as jpena | 13:59 | |
| openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Add OpenStack Client for Manila docs https://review.opendev.org/706252 | 14:02 |
| gtema_ | dtantsur, mordred: I'm back, and so the https://review.opendev.org/#/c/700219 (project cleanup) is now rebased and waiting for your ok | 14:03 |
| dtantsur | thx | 14:06 |
| *** gtema_ is now known as gtema | 14:06 | |
| *** amoralej|lunch is now known as amoralej | 14:06 | |
| mordred | gtema: yay! | 14:07 |
| mordred | gtema: welcome back! | 14:07 |
| gtema | thks | 14:07 |
| mordred | gtema: so ... don't hate me ... but I may have *completely* forgotten to tell you about a discussion we had at the shanghai summit | 14:08 |
| gtema | okaaayy, and...??? | 14:08 |
| mordred | apparently https://opendev.org/x/ospurge has project cleanup logic and people were suggesting we suck that into sdk and start from there. at this point since you've got the underlying framework maybe instead of doing that we just port in whatever knowledge is in there? | 14:10 |
| gtema | well, 99% of what ospurge could do (AFAIR) is now done | 14:10 |
| mordred | yeah. I just feel bad I didn't mention "just copy in ospurge as a starting point" before you started work :) | 14:11 |
| gtema | oh, no, it was another similar project | 14:11 |
| openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: identity: autogenerate docs https://review.opendev.org/692931 | 14:11 |
| openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: image: autogenerate docs https://review.opendev.org/693025 | 14:11 |
| gtema | basically my change gives a framework for that and the individual services can be easily covered | 14:11 |
| gtema | so with followups we can integrate more stuff from ospurge, but I would prefer to put it into the cleanup framework | 14:12 |
| mordred | oh - totally. we should definitely put it in the cleanup framework | 14:13 |
| mordred | gtema: left a quick question on that patch | 14:13 |
| gtema | especially that it does not seem to support parallel execution | 14:13 |
| gtema | so you mean I should drop this "single thread fallback"? | 14:14 |
| mordred | I think so? cause I think it's covered with futurist - but maybe dtantsur has a different pov | 14:14 |
| gtema | if he don't mind - will definitely do this | 14:15 |
| mordred | gtema: also - don't know if you saw - but we merged the sdk and osc teams - so welcome to being core on osc now too | 14:16 |
| gtema | cool. | 14:17 |
| gtema | I have seen discussion and adding OSC to cores in SDK, but haven't seen SDKs in OSC | 14:17 |
| openstackgerrit | Akihiro Motoki proposed openstack/python-openstackclient master: Fix network segment range "_get_ranges" function https://review.opendev.org/710031 | 14:17 |
| gtema | but am now glad to have possibility to put more efforts in OSC as well ;) | 14:17 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374 | 14:18 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Complete switch from glanceclient to SDK for image service https://review.opendev.org/699416 | 14:18 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Don't look up project by id if given id https://review.opendev.org/712810 | 14:18 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Optimize adding project to image slightly more https://review.opendev.org/712813 | 14:18 |
| gtema | any thoughts already on how the virtual PTG will look like? | 14:18 |
| mordred | gtema: we should merge that stack before the ussuri feature freeze :) | 14:18 |
| gtema | oh sure | 14:18 |
| mordred | I think people are working on it | 14:18 |
| mordred | but I don't know what it will look like yet | 14:19 |
| gtema | oki | 14:19 |
| mordred | corvus deployed a POC of jitsi-meet integrated with etherpad that I believe we're going to deploy as a new service | 14:19 |
| mordred | so it's like etherpad as the main screen, but also with video chat capabilities | 14:20 |
| gtema | would be cool, but definitely TZ will play a nice role :D | 14:20 |
| mordred | so hopefully that should help some with virtual ptg | 14:20 |
| mordred | yeah | 14:20 |
| mordred | maybe we should all just give ourselves jetlag in our local location for a week | 14:20 |
| gtema | anyway glad it is not finally decided to do it this way - sitting here in prison and not knowing how the June will look like was a bit ... | 14:21 |
| mordred | and "Fly" to vancouver virtually | 14:21 |
| gtema | I was hoping so much to go to Vancouver (cry) | 14:21 |
| mordred | yeah | 14:21 |
| mordred | it's gonna be a rough next several months | 14:21 |
| gtema | sure. Hopefully next Berlin will take place | 14:22 |
| mordred | I certainly hope so | 14:22 |
| gtema | we can then make SDK hiking tour over germany | 14:22 |
| mordred | sounds like fun! :) | 14:23 |
| mordred | (anything other than an SDK "sitting in a room for months" will be wonderful) | 14:24 |
| gtema | we need to compensate vancouver | 14:24 |
| gtema | "sitting in a virtual room" you mean | 14:24 |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Lay a foundation for the project cleanup https://review.opendev.org/700219 | 14:30 |
| gtema | I assumed dtantsur will not object on futurist and removed bypass | 14:31 |
| openstackgerrit | Mark Chappell proposed openstack/openstacksdk master: Add support for Federation Protocols https://review.opendev.org/714431 | 14:35 |
| openstackgerrit | Mark Chappell proposed openstack/openstacksdk master: Add support for Identity Providers https://review.opendev.org/714122 | 14:36 |
| *** tremble has joined #openstack-sdks | 14:36 | |
| mordred | cmurphy: if you have some spare time - mind looking at ^^ ? | 14:37 |
| mordred | gtema: cool | 14:38 |
| tremble | gtema, Anywhere you'd like me to add a link to the relevant API docs? | 14:39 |
| gtema | since we have all already found it - nope, it's ok | 14:39 |
| *** ouroboros8 has quit IRC | 14:49 | |
| *** mgariepy has quit IRC | 14:55 | |
| *** ricolin_ has quit IRC | 15:01 | |
| *** ricolin_ has joined #openstack-sdks | 15:01 | |
| *** evrardjp has quit IRC | 15:07 | |
| *** evrardjp has joined #openstack-sdks | 15:11 | |
| mgoddard | hi, is this the place to ask for cliff reviews? https://review.opendev.org/708882 | 15:21 |
| *** tremble has left #openstack-sdks | 15:34 | |
| *** mgariepy has joined #openstack-sdks | 15:35 | |
| *** tremble has joined #openstack-sdks | 15:39 | |
| *** KeithMnemonic has joined #openstack-sdks | 15:42 | |
| KeithMnemonic | smcginnis: Hello Sean. I hope you are holding up ok. Could you please let me know exactly what is needed for here? https://review.opendev.org/#/c/702260/4 The "cherry picked" from does point to Train so I am not sure I understand exactly what is needed. Thank you. | 15:45 |
| smcginnis | KeithMnemonic: Oh, I saw only the one line and assumed it was directly from master. Typically we keep each line, so by the time you get back to stein you would have a couple cherry-pick lines. | 15:48 |
| smcginnis | This should be fine though. | 15:48 |
| KeithMnemonic | ok thanks for checking again | 15:49 |
| smcginnis | np, thanks for pointing that out. | 15:49 |
| openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Introduce 'quota unset' command https://review.opendev.org/376311 | 15:53 |
| tremble | mordred, Any suggestions what I can do to work around: http://paste.openstack.org/show/791031/ | 15:59 |
| smcginnis | tremble: Is that coming out of generated code? | 16:04 |
| tremble | smcginnis, It's documentation from https://review.opendev.org/#/c/714431/ | 16:05 |
| tremble | I was trying to keep the object name consistent with how the openstack CLI refers to the object | 16:05 |
| smcginnis | tremble: You'll have to put in some line break in there to keep them <80 characters long. | 16:06 |
| tremble | smcginnis, Any idea how to do that without causing documentation oddness ? | 16:07 |
| smcginnis | tremble: Should be able to just stick a break anywhere in there. Generated docs will combine strings together in the resulting HTML. | 16:09 |
| smcginnis | As far as reading the direct source, probably good to add a break after the last "." in the class path maybe. | 16:09 |
| *** sshnaidm|pto has quit IRC | 16:15 | |
| *** sshnaidm|pto has joined #openstack-sdks | 16:17 | |
| nightmare_unreal | hello i am kind of stuck with writing a test case for heal_allocation. I am trying to mimic cell-uuid using uuidutils.generate_uuid() . Here's what i have written so far : https://review.opendev.org/#/c/714459/ | 16:25 |
| tremble | Thanks smcginnis | 16:30 |
| smcginnis | tremble: no problem! | 16:32 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374 | 16:35 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Complete switch from glanceclient to SDK for image service https://review.opendev.org/699416 | 16:35 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Don't look up project by id if given id https://review.opendev.org/712810 | 16:35 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Optimize adding project to image slightly more https://review.opendev.org/712813 | 16:35 |
| mordred | stephenfin: ^^ thanks for the catch there. | 16:35 |
| openstackgerrit | Mark Chappell proposed openstack/openstacksdk master: Add support for Federation Protocols https://review.opendev.org/714431 | 16:42 |
| cmurphy | mordred: added to my list | 16:47 |
| mordred | cmurphy: thanks | 16:47 |
| cmurphy | happy to see those being added | 16:47 |
| mordred | me too! | 16:48 |
| *** rpittau is now known as rpittau|afk | 17:08 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Change dockerhub password https://review.opendev.org/714436 | 17:15 |
| *** jawad_axd has quit IRC | 17:17 | |
| *** gtema has quit IRC | 17:22 | |
| *** jpich has quit IRC | 17:23 | |
| *** evrardjp has quit IRC | 17:36 | |
| *** dtantsur is now known as dtantsur|afk | 17:36 | |
| *** evrardjp has joined #openstack-sdks | 17:36 | |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Remove trailing newline from dockerhub secret https://review.opendev.org/714502 | 17:55 |
| mordred | stephenfin: ^^ if you don't mind ... I think it's going to work this time | 17:55 |
| smcginnis | Odd that osc-build-image passed on the original patch. | 18:03 |
| openstackgerrit | Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add description field to portforwarding NAT rules https://review.opendev.org/714512 | 18:19 |
| *** jpena is now known as jpena|off | 18:27 | |
| *** amoralej is now known as amoralej|off | 18:44 | |
| mordred | smcginnis: the osc-upload-image job turns out to use docker login which writes the creds to a config file and then docker push uses them - so the trailing whitespace naturally just gets eaten in the process | 18:50 |
| mordred | smcginnis: the promote job just does a remote API call to retag an existing image on dockerhub and passing teh value to the ansible uri module keeps the trailing whitespace intact | 18:50 |
| mordred | (We also have changes up to update the job to strip and also to strip in encrypt_secret) | 18:51 |
| mordred | smcginnis: but yeah - took a few days to figure out what was going wrong - I'd like for other people to not have to do that | 18:51 |
| smcginnis | mordred: Makes sense. | 18:55 |
| *** diablo_rojo__ has joined #openstack-sdks | 19:13 | |
| *** ralonsoh has quit IRC | 19:16 | |
| *** ricolin_ has quit IRC | 19:24 | |
| mordred | smcginnis: so - I feel like I need to udpate requirements here: https://zuul.opendev.org/t/openstack/build/e4bde7dbcff74624ba844e11db5bef97 | 19:26 |
| mordred | actually - maybe I should switch channels | 19:27 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374 | 19:38 |
| *** jawad_axd has joined #openstack-sdks | 19:48 | |
| *** diablo_rojo__ has quit IRC | 19:49 | |
| *** diablo_rojo__ has joined #openstack-sdks | 19:51 | |
| openstackgerrit | jayaditya gupta proposed openstack/python-openstackclient master: Add '--force; parameter to 'openstack quota set'/ The compute service allows us to to force set a quota, setting a quota value that is less than the amount of the resource currently consumed. Expose this feature by way of a '--force' boolean parameter. https://review.opendev.org/713881 | 20:11 |
| *** nightmare_unreal has quit IRC | 20:17 | |
| *** jawad_axd has quit IRC | 20:28 | |
| *** jawad_axd has joined #openstack-sdks | 20:29 | |
| *** jawad_axd has quit IRC | 20:33 | |
| sshnaidm|pto | folks, please review: https://review.opendev.org/#/c/714302/ required for ansible job on openstacksdk | 20:35 |
| sshnaidm|pto | mordred, ^^ | 20:35 |
| sshnaidm|pto | seems like rocky and train gates are broken: https://review.opendev.org/#/c/714086/ https://review.opendev.org/#/c/714089/ | 20:38 |
| *** mgariepy has quit IRC | 20:45 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Remove trailing newline from dockerhub secret https://review.opendev.org/714502 | 20:47 |
| mordred | woot! it worked! https://hub.docker.com/repository/docker/osclient/python-openstackclient corvus <-- | 20:55 |
| mordred | smcginnis: ^^ | 20:55 |
| corvus | mordred: that's a private url, but i assume the public one works so yay! | 20:57 |
| mordred | oh. heh | 20:58 |
| mordred | https://hub.docker.com/r/osclient/python-openstackclient | 20:58 |
| *** dtroyer has joined #openstack-sdks | 21:14 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Fix network segment range "_get_ranges" function https://review.opendev.org/710031 | 21:41 |
| *** mgariepy has joined #openstack-sdks | 21:50 | |
| *** tkajinam has joined #openstack-sdks | 22:57 | |
| *** lxkong has joined #openstack-sdks | 23:03 | |
| openstackgerrit | Sam Morrison proposed openstack/openstacksdk master: Add availability zone and availability zone profile resources to load balancer https://review.opendev.org/714345 | 23:05 |
| *** slaweq has quit IRC | 23:06 | |
| *** slaweq has joined #openstack-sdks | 23:18 | |
| *** diablo_rojo__ is now known as diablo_rojo | 23:21 | |
| *** slaweq has quit IRC | 23:23 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!