*** holser has quit IRC | 00:10 | |
*** tosky has quit IRC | 00:14 | |
*** eandersson has quit IRC | 00:57 | |
*** eandersson has joined #openstack-sdks | 00:57 | |
*** holser has joined #openstack-sdks | 01:08 | |
*** holser has quit IRC | 01:31 | |
*** LinPeiWen has joined #openstack-sdks | 01:48 | |
*** evrardjp has quit IRC | 05:33 | |
*** evrardjp has joined #openstack-sdks | 05:33 | |
openstackgerrit | zhufl proposed openstack/openstacksdk master: Fix invalid assertIsNotNone statement https://review.opendev.org/761053 | 05:48 |
---|---|---|
*** melwitt has quit IRC | 07:20 | |
*** melwitt has joined #openstack-sdks | 07:21 | |
*** gtema has joined #openstack-sdks | 07:24 | |
*** slaweq has joined #openstack-sdks | 07:54 | |
*** ralonsoh has joined #openstack-sdks | 07:59 | |
*** tosky has joined #openstack-sdks | 08:13 | |
*** sshnaidm|rover has quit IRC | 08:36 | |
*** sshnaidm|rover has joined #openstack-sdks | 08:38 | |
*** rpittau|afk is now known as rpittau | 08:39 | |
*** sshnaidm|rover has quit IRC | 08:43 | |
*** sshnaidm|rover has joined #openstack-sdks | 08:45 | |
*** sshnaidm|rover has quit IRC | 08:52 | |
openstackgerrit | jayaditya gupta proposed openstack/python-openstackclient master: Add a few selectable fields to the "openstack server list" output https://review.opendev.org/741181 | 08:54 |
*** sshnaidm|rover has joined #openstack-sdks | 08:56 | |
*** sshnaidm|rover has quit IRC | 09:00 | |
*** jpich has joined #openstack-sdks | 09:11 | |
*** Hybrid512 has joined #openstack-sdks | 09:20 | |
*** sshnaidm has joined #openstack-sdks | 09:49 | |
stephenfin | gtema: amotoki: Thanks for the reviews :) Any chance you could look at https://review.opendev.org/#/c/761421/ too? | 10:06 |
gtema | stephenfin - sure. I requested release of osc-lib | 10:06 |
noonedeadpunk | o/ | 10:07 |
*** holser has joined #openstack-sdks | 10:07 | |
stephenfin | gtema++ great, thanks :) | 10:07 |
noonedeadpunk | I'm trying to get why `openstackclient/volume/v2` is used and where does it comes from... | 10:07 |
noonedeadpunk | as it seems it has limitations v3 should not have, but kind of v3 is just empty for me | 10:08 |
openstackgerrit | Stephen Finucane proposed openstack/osc-lib stable/victoria: zuul: Stop testing against Tempest https://review.opendev.org/761525 | 10:09 |
gtema | noonedeadpunk - mostly it comes from whatever version is being pointed in the service catalog (service_type=volume) | 10:09 |
gtema | normally operators forget to switch it to v3 | 10:09 |
noonedeadpunk | well, I have both v2 and v3 | 10:09 |
gtema | yes, a well known practice to have: volume, volumev2, volumev3 | 10:10 |
gtema | look where the volume type is pointing you to | 10:10 |
noonedeadpunk | the most thing that concerns me is http://paste.openstack.org/show/799728/ | 10:10 |
gtema | this is another interesting case: v2=v3 in the CLI | 10:10 |
noonedeadpunk | aha | 10:11 |
gtema | https://opendev.org/openstack/python-openstackclient/src/branch/master/setup.cfg#L662 | 10:12 |
noonedeadpunk | well, I think it's not... What I'm trying to do - volume resize when it's in-use. | 10:12 |
noonedeadpunk | So client has check https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/volume/v2/volume.py#L608 | 10:12 |
noonedeadpunk | however it's not true for modern cinder https://opendev.org/openstack/cinder/src/branch/stable/train/cinder/volume/manager.py#L2766 | 10:12 |
noonedeadpunk | but true for v2 I guess | 10:13 |
noonedeadpunk | Not sure what's the best way to workaround here... | 10:13 |
openstackgerrit | Stephen Finucane proposed openstack/osc-lib master: Add Python3 wallaby unit tests https://review.opendev.org/750335 | 10:16 |
noonedeadpunk | well, or not.... | 10:18 |
noonedeadpunk | it is https://opendev.org/openstack/cinder/src/branch/master/cinder/api/openstack/rest_api_version_history.rst#user-content-section-39 | 10:20 |
*** noonedeadpunk has quit IRC | 10:32 | |
*** noonedeadpunk has joined #openstack-sdks | 10:32 | |
noonedeadpunk | gtema: oh, that is neat, means I can kind of overwrite with v3 only single method? | 10:50 |
gtema | not the method, but you need more or less to copy/modify complete resource | 10:51 |
noonedeadpunk | yeah, agree, sure | 10:52 |
frickler | doesn't the volume-resize-while-in-use require a bumped microversion? | 10:54 |
noonedeadpunk | it does | 10:55 |
noonedeadpunk | but I think that issue here is that openstackclient additionally checks for the volume status | 10:56 |
noonedeadpunk | moreover http://paste.openstack.org/show/799733/ | 10:58 |
frickler | hmm, weird, I was pretty sure that it did work for me earlier. will need to double-check after the meeting starting now | 10:59 |
*** noonedeadpunk has quit IRC | 11:21 | |
frickler | noonedeadpunk: you are right, it's only "cinder --os-volume-api-version 3.42 extend" that worked to me. would be great to make that work in OSC, but it seems one would first need to implement v3 and microversions for cinder | 11:22 |
openstackgerrit | Merged openstack/osc-lib master: bump py37 to py38 in tox.ini https://review.opendev.org/752613 | 11:25 |
*** dtantsur|afk is now known as dtantsur | 11:26 | |
*** noonedeadpunk has joined #openstack-sdks | 11:26 | |
noonedeadpunk | uh, everything is so weird about volumes part in openstackclient | 12:07 |
*** jpich has quit IRC | 12:11 | |
*** jpich has joined #openstack-sdks | 12:11 | |
*** gtema has quit IRC | 12:16 | |
openstackgerrit | Merged openstack/osc-lib master: Add Python3 wallaby unit tests https://review.opendev.org/750335 | 12:32 |
openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server * -f yaml' output https://review.opendev.org/761205 | 12:34 |
openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'usage * -f yaml' output https://review.opendev.org/761595 | 12:34 |
openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server group * -f yaml' output https://review.opendev.org/761596 | 12:34 |
openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Add support for 'server group create --rule' parameter https://review.opendev.org/761597 | 12:34 |
openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: trivial: Use plural for appended parameters https://review.opendev.org/761598 | 12:34 |
*** gtema has joined #openstack-sdks | 12:41 | |
stephenfin | smcginnis: Question: lower constraints are failing for osc-lib because the job is running on Python 3.8 but we didn't merge the patch to fix compatibility with this Python version soon enough https://opendev.org/openstack/osc-lib/commit/8ff3f80102ae184227d4f8cb5f82bedee987b804 | 12:42 |
stephenfin | smcginnis: How important is Python 3.8 compatibility for Victoria. Do I backport that patch, thus bumping minimums on a stable branch, or simply set the basepython in 'tox.ini' to something lower? | 12:43 |
openstackgerrit | Stephen Finucane proposed openstack/osc-lib stable/victoria: tox: Set minimum Python version for lower-constraints https://review.opendev.org/761600 | 12:47 |
openstackgerrit | Stephen Finucane proposed openstack/osc-lib stable/ussuri: zuul: Stop testing against Tempest https://review.opendev.org/761603 | 12:49 |
openstackgerrit | jayaditya gupta proposed openstack/python-openstackclient master: Add a few selectable fields to the "openstack server list" output https://review.opendev.org/741181 | 13:01 |
smcginnis | stephenfin: I think either approach. | 13:11 |
smcginnis | stephenfin: py38 was an official runtime for victoria, but there were differing opinions on whether that meant lower constraints should work for 3.8 or our actual minimum runtime (3.6). | 13:12 |
noonedeadpunk | any way to get cinder client to use different api version? http://paste.openstack.org/show/799741/ | 13:12 |
smcginnis | stephenfin: There were a few other projects that ended up backporting those lower-constraints updates to work with 3.8. | 13:13 |
smcginnis | For consistency, that's probably the right thing to do. | 13:13 |
smcginnis | noonedeadpunk: That .version value doesn't reflect the version sent when making API calls. | 13:18 |
noonedeadpunk | yeah already found it.... | 13:18 |
smcginnis | k | 13:18 |
smcginnis | api_version, right? | 13:18 |
noonedeadpunk | yep | 13:18 |
* noonedeadpunk tries to find a way to make CLI client to do volume resize with attached volumes | 13:19 | |
smcginnis | noonedeadpunk: In case you need to read up on any background on that: https://specs.openstack.org/openstack/cinder-specs/specs/pike/extend-attached-volume.html | 13:20 |
noonedeadpunk | well, I can do it with curl ofc, but it's not really handy^) | 13:21 |
noonedeadpunk | and openstackclient just fails because of https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/volume/v2/volume.py#L608 | 13:21 |
noonedeadpunk | oh, btw, once I've catched you:) | 13:22 |
noonedeadpunk | smcginnis: is it safe to execute this with auth_type with service credentials? | 13:22 |
noonedeadpunk | I'm a bit worried about privilege escalation here | 13:23 |
noonedeadpunk | (as there were probably reasons to use session token by default instead) | 13:23 |
smcginnis | Good question. I believe it should be safe. | 13:23 |
noonedeadpunk | probbly would be awesome to add these options to the list of supported somehow, but that's complete another topic.... Thanks! | 13:25 |
*** jpich has quit IRC | 14:09 | |
*** jpich has joined #openstack-sdks | 14:10 | |
openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Remove references to Python 2.7 https://review.opendev.org/761615 | 14:16 |
openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Remove unnecessary test https://review.opendev.org/761616 | 14:16 |
*** dtantsur has quit IRC | 14:16 | |
openstackgerrit | Stephen Finucane proposed openstack/osc-lib stable/victoria: Fix l-c testing for ubuntu focal https://review.opendev.org/761617 | 14:17 |
stephenfin | smcginnis: Okay, makes sense. Care to take a look at that change ^ in that case? | 14:17 |
*** dtantsur has joined #openstack-sdks | 14:37 | |
*** dtantsur has quit IRC | 14:37 | |
*** dtantsur has joined #openstack-sdks | 14:38 | |
*** belmoreira has joined #openstack-sdks | 14:56 | |
*** dtantsur has quit IRC | 14:56 | |
*** dtantsur has joined #openstack-sdks | 14:56 | |
*** Hybrid512 has quit IRC | 15:23 | |
openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/python-openstackclient master: Allow to resize in-use volumes https://review.opendev.org/761633 | 15:51 |
*** jpich has quit IRC | 16:04 | |
*** jpich has joined #openstack-sdks | 16:05 | |
*** Hybrid512 has joined #openstack-sdks | 16:07 | |
openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/python-openstackclient master: Allow to resize in-use volumes https://review.opendev.org/761633 | 16:08 |
*** belmoreira has quit IRC | 16:09 | |
*** gtema has quit IRC | 16:56 | |
openstackgerrit | Merged openstack/osc-lib stable/ussuri: zuul: Stop testing against Tempest https://review.opendev.org/761603 | 17:13 |
*** rpittau is now known as rpittau|afk | 17:21 | |
*** jpich has quit IRC | 17:24 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add "fields" parameter to ListPort query https://review.opendev.org/754117 | 17:26 |
openstackgerrit | Merged openstack/osc-lib stable/victoria: Fix l-c testing for ubuntu focal https://review.opendev.org/761617 | 17:45 |
openstackgerrit | Stephen Finucane proposed openstack/osc-lib stable/victoria: zuul: Stop testing against Tempest https://review.opendev.org/761525 | 17:50 |
*** ralonsoh has quit IRC | 18:41 | |
*** sshnaidm is now known as sshnaidm|afk | 18:44 | |
*** dtantsur is now known as dtantsur|afk | 18:53 | |
*** gtema has joined #openstack-sdks | 18:57 | |
*** gtema has quit IRC | 19:02 | |
*** lbragstad has quit IRC | 19:18 | |
*** Hybrid512 has quit IRC | 19:47 | |
*** hyang has joined #openstack-sdks | 20:22 | |
*** hyang has quit IRC | 21:14 | |
*** slaweq has quit IRC | 22:06 | |
*** tosky has quit IRC | 23:55 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!