| *** mgoddard- is now known as mgoddard | 08:40 | |
| gouthamr | artom diablo_rojo_phone: o/ hey there, sorry for the PM last evening, thought i could catch you here during the day :) | 16:38 |
|---|---|---|
| gouthamr | so, we've a number of manila changes to the sdk - and they've not been getting much review traction | 16:38 |
| gtema | gouthamr, I put them into my list, will try to look over the weekend/Monday | 16:38 |
| gouthamr | i brought up the possibility of adding one or more of us to the review team to +2 manila specific changes so we can reduce the time it takes to get them in | 16:39 |
| diablo_rojo_phone | gouthamr: you wanted to be added as core? | 16:39 |
| diablo_rojo_phone | Lol | 16:39 |
| diablo_rojo_phone | Yeah that | 16:39 |
| gouthamr | yeah - i don't really want to be, honestly - i'd do a terrible job keeping up; but i'm trying to get a few people up to speed here | 16:39 |
| gouthamr | and that's going to take time, and time's something our interns typically lack | 16:40 |
| gouthamr | heh, sorry artom - i forgot gtema | 16:40 |
| gtema | no problem | 16:40 |
| diablo_rojo_phone | mordred had added a bunch of project representatives. | 16:40 |
| diablo_rojo_phone | I think it's a good idea. | 16:40 |
| diablo_rojo_phone | I guess the understanding is that you'd only +2 manila ones? | 16:41 |
| gouthamr | yes | 16:42 |
| gouthamr | i'll ofcourse add the rest of you for that final +W too, if you've not already looked :) | 16:42 |
| gouthamr | kafilat's internship ends in a few weeks; i'm hoping we can get her work in; and iterate - it's all going to have to be merged back from the feature/r1 branch into master anyway | 16:44 |
| artom | gouthamr, hehe, I'm the wrong Art(e|o)m :) | 16:44 |
| artom | (Not often I get to say that in a non-Russophone environment) | 16:44 |
| gouthamr | so i'll help with that cherry-picking etc; but having patch dependencies (resources that have a fk dependency on other resources, "sub" resources etc) is hurting her productivity for now | 16:45 |
| gouthamr | artom: :D i blame freenode chaos on my nick confusions | 16:47 |
| gouthamr | also would be nice to see this one merged: https://review.opendev.org/c/openstack/project-config/+/799323 | 16:51 |
| *** holser is now known as holser_ | 16:52 | |
| *** holser_ is now known as holser | 16:52 | |
| mordred | ++ to project cores | 16:55 |
| *** melwitt is now known as Guest322 | 17:31 | |
| *** melwitt_ is now known as melwitt | 17:57 | |
| *** melwitt is now known as jgwentworth | 17:58 | |
| *** TheJulia is now known as needssleep | 18:04 | |
| gouthamr | \o/ | 18:14 |
| gouthamr | i've a question regarding microversions in the API requests in the functional tests | 18:14 |
| gouthamr | when i write a functional test, and set a min_version in the class, it only seems to be used for validating if the version is supported: https://opendev.org/openstack/openstacksdk/src/commit/9dc4a841145200583ac3f8c56df775a165e5f2f4/openstack/tests/functional/shared_file_system/test_availability_zone.py#L18 | 18:15 |
| gouthamr | i'd like the request to be made with that microversion | 18:16 |
| gouthamr | is there any way i can do that? i tried setting the user and operator cloud versions to a specific version, but that didn't seem to make a difference: https://review.opendev.org/c/openstack/openstacksdk/+/799208/2/openstack/tests/functional/shared_file_system/base.py | 18:17 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!