| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-03-23 | |||
| 14:10: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:44 | gtema | well, 99% of what ospurge could do (AFAIR) is now done | |
| 14:11: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:19 | gtema | oh, no, it was another similar project | |
| 14:11:49 | openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: image: autogenerate docs https://review.opendev.org/693025 | |
| 14:11:49 | openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: identity: autogenerate docs https://review.opendev.org/692931 | |
| 14:11:52 | gtema | basically my change gives a framework for that and the individual services can be easily covered | |
| 14:12:32 | gtema | so with followups we can integrate more stuff from ospurge, but I would prefer to put it into the cleanup framework | |
| 14:13:06 | mordred | oh - totally. we should definitely put it in the cleanup framework | |
| 14:13:13 | mordred | gtema: left a quick question on that patch | |
| 14:13:20 | gtema | especially that it does not seem to support parallel execution | |
| 14:14:14 | gtema | so you mean I should drop this "single thread fallback"? | |
| 14:14:48 | mordred | I think so? cause I think it's covered with futurist - but maybe dtantsur has a different pov | |
| 14:15:26 | gtema | if he don't mind - will definitely do this | |
| 14:16:52 | 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:17:04 | gtema | cool. | |
| 14:17:28 | gtema | I have seen discussion and adding OSC to cores in SDK, but haven't seen SDKs in OSC | |
| 14:17:37 | openstackgerrit | Akihiro Motoki proposed openstack/python-openstackclient master: Fix network segment range "_get_ranges" function https://review.opendev.org/710031 | |
| 14:17:51 | gtema | but am now glad to have possibility to put more efforts in OSC as well ;) | |
| 14:18:05 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374 | |
| 14:18:11 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Complete switch from glanceclient to SDK for image service https://review.opendev.org/699416 | |
| 14:18:16 | 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:23 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Optimize adding project to image slightly more https://review.opendev.org/712813 | |
| 14:18:44 | gtema | any thoughts already on how the virtual PTG will look like? | |
| 14:18:45 | mordred | gtema: we should merge that stack before the ussuri feature freeze :) | |
| 14:18:54 | gtema | oh sure | |
| 14:18:58 | mordred | I think people are working on it | |
| 14:19:11 | mordred | but I don't know what it will look like yet | |
| 14:19:18 | gtema | oki | |
| 14:19:46 | mordred | corvus deployed a POC of jitsi-meet integrated with etherpad that I believe we're going to deploy as a new service | |
| 14:20:00 | mordred | so it's like etherpad as the main screen, but also with video chat capabilities | |
| 14:20:24 | gtema | would be cool, but definitely TZ will play a nice role :D | |
| 14:20:30 | mordred | so hopefully that should help some with virtual ptg | |
| 14:20:31 | mordred | yeah | |
| 14:20:59 | mordred | maybe we should all just give ourselves jetlag in our local location for a week | |
| 14:21:08 | 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:13 | mordred | and "Fly" to vancouver virtually | |
| 14:21:40 | gtema | I was hoping so much to go to Vancouver (cry) | |
| 14:21:45 | mordred | yeah | |
| 14:21:54 | mordred | it's gonna be a rough next several months | |
| 14:22:18 | gtema | sure. Hopefully next Berlin will take place | |
| 14:22:28 | mordred | I certainly hope so | |
| 14:22:33 | gtema | we can then make SDK hiking tour over germany | |
| 14:23:48 | mordred | sounds like fun! :) | |
| 14:24:08 | mordred | (anything other than an SDK "sitting in a room for months" will be wonderful) | |
| 14:24:13 | gtema | we need to compensate vancouver | |
| 14:24:30 | gtema | "sitting in a virtual room" you mean | |
| 14:30:35 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Lay a foundation for the project cleanup https://review.opendev.org/700219 | |
| 14:31:02 | gtema | I assumed dtantsur will not object on futurist and removed bypass | |
| 14:35:20 | openstackgerrit | Mark Chappell proposed openstack/openstacksdk master: Add support for Federation Protocols https://review.opendev.org/714431 | |
| 14:36:21 | openstackgerrit | Mark Chappell proposed openstack/openstacksdk master: Add support for Identity Providers https://review.opendev.org/714122 | |
| 14:37:48 | mordred | cmurphy: if you have some spare time - mind looking at ^^ ? | |
| 14:38:00 | mordred | gtema: cool | |
| 14:39:22 | tremble | gtema, Anywhere you'd like me to add a link to the relevant API docs? | |
| 14:39:50 | gtema | since we have all already found it - nope, it's ok | |
| 15:21:27 | mgoddard | hi, is this the place to ask for cliff reviews? https://review.opendev.org/708882 | |
| 15:45:20 | 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:48:38 | 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:46 | smcginnis | This should be fine though. | |
| 15:49:14 | KeithMnemonic | ok thanks for checking again | |
| 15:49:33 | smcginnis | np, thanks for pointing that out. | |
| 15:53:39 | openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Introduce 'quota unset' command https://review.opendev.org/376311 | |
| 15:59:16 | tremble | mordred, Any suggestions what I can do to work around: http://paste.openstack.org/show/791031/ | |
| 16:04:25 | smcginnis | tremble: Is that coming out of generated code? | |
| 16:05:09 | tremble | smcginnis, It's documentation from https://review.opendev.org/#/c/714431/ | |
| 16:05:40 | tremble | I was trying to keep the object name consistent with how the openstack CLI refers to the object | |
| 16:06:32 | smcginnis | tremble: You'll have to put in some line break in there to keep them <80 characters long. | |
| 16:07:25 | tremble | smcginnis, Any idea how to do that without causing documentation oddness ? | |
| 16:09:27 | 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:58 | smcginnis | As far as reading the direct source, probably good to add a break after the last "." in the class path maybe. | |
| 16:25:40 | 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:30:10 | tremble | Thanks smcginnis | |
| 16:32:49 | smcginnis | tremble: no problem! | |
| 16:35:02 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374 | |
| 16:35:03 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Complete switch from glanceclient to SDK for image service https://review.opendev.org/699416 | |
| 16:35:04 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Optimize adding project to image slightly more https://review.opendev.org/712813 | |
| 16:35:04 | 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:16 | mordred | stephenfin: ^^ thanks for the catch there. | |
| 16:42:34 | openstackgerrit | Mark Chappell proposed openstack/openstacksdk master: Add support for Federation Protocols https://review.opendev.org/714431 | |
| 16:47:36 | cmurphy | mordred: added to my list | |
| 16:47:44 | mordred | cmurphy: thanks | |
| 16:47:49 | cmurphy | happy to see those being added | |
| 16:48:06 | mordred | me too! | |
| 17:15:38 | openstackgerrit | Merged openstack/python-openstackclient master: Change dockerhub password https://review.opendev.org/714436 | |
| 17:55:36 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Remove trailing newline from dockerhub secret https://review.opendev.org/714502 | |
| 17:55:56 | mordred | stephenfin: ^^ if you don't mind ... I think it's going to work this time | |
| 18:03:00 | smcginnis | Odd that osc-build-image passed on the original patch. | |
| 18:19:48 | openstackgerrit | Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add description field to portforwarding NAT rules https://review.opendev.org/714512 | |
| 18:50:20 | 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:51 | 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:51:05 | mordred | (We also have changes up to update the job to strip and also to strip in encrypt_secret) | |
| 18:51:28 | 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:55:08 | smcginnis | mordred: Makes sense. | |
| 19:26:28 | mordred | smcginnis: so - I feel like I need to udpate requirements here: https://zuul.opendev.org/t/openstack/build/e4bde7dbcff74624ba844e11db5bef97 | |
| 19:27:04 | mordred | actually - maybe I should switch channels | |
| 19:38:53 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374 | |
| 20:11:00 | 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:35:25 | sshnaidm|pto | folks, please review: https://review.opendev.org/#/c/714302/ required for ansible job on openstacksdk | |
| 20:35:28 | sshnaidm|pto | mordred, ^^ | |
| 20:38:19 | sshnaidm|pto | seems like rocky and train gates are broken: https://review.opendev.org/#/c/714086/ https://review.opendev.org/#/c/714089/ | |