Thursday, 2020-11-05

*** holser has quit IRC00:10
*** tosky has quit IRC00:14
*** eandersson has quit IRC00:57
*** eandersson has joined #openstack-sdks00:57
*** holser has joined #openstack-sdks01:08
*** holser has quit IRC01:31
*** LinPeiWen has joined #openstack-sdks01:48
*** evrardjp has quit IRC05:33
*** evrardjp has joined #openstack-sdks05:33
openstackgerritzhufl proposed openstack/openstacksdk master: Fix invalid assertIsNotNone statement  https://review.opendev.org/76105305:48
*** melwitt has quit IRC07:20
*** melwitt has joined #openstack-sdks07:21
*** gtema has joined #openstack-sdks07:24
*** slaweq has joined #openstack-sdks07:54
*** ralonsoh has joined #openstack-sdks07:59
*** tosky has joined #openstack-sdks08:13
*** sshnaidm|rover has quit IRC08:36
*** sshnaidm|rover has joined #openstack-sdks08:38
*** rpittau|afk is now known as rpittau08:39
*** sshnaidm|rover has quit IRC08:43
*** sshnaidm|rover has joined #openstack-sdks08:45
*** sshnaidm|rover has quit IRC08:52
openstackgerritjayaditya gupta proposed openstack/python-openstackclient master: Add a few selectable fields to the "openstack server list" output  https://review.opendev.org/74118108:54
*** sshnaidm|rover has joined #openstack-sdks08:56
*** sshnaidm|rover has quit IRC09:00
*** jpich has joined #openstack-sdks09:11
*** Hybrid512 has joined #openstack-sdks09:20
*** sshnaidm has joined #openstack-sdks09:49
stephenfingtema: amotoki: Thanks for the reviews :) Any chance you could look at https://review.opendev.org/#/c/761421/ too?10:06
gtemastephenfin - sure. I requested release of osc-lib10:06
noonedeadpunko/10:07
*** holser has joined #openstack-sdks10:07
stephenfingtema++ great, thanks :)10:07
noonedeadpunkI'm trying to get why `openstackclient/volume/v2` is used and where does it comes from...10:07
noonedeadpunkas it seems it has limitations v3 should not have, but kind of v3 is just empty for me10:08
openstackgerritStephen Finucane proposed openstack/osc-lib stable/victoria: zuul: Stop testing against Tempest  https://review.opendev.org/76152510:09
gtemanoonedeadpunk - mostly it comes from whatever version is being pointed in the service catalog (service_type=volume)10:09
gtemanormally operators forget to switch it to v310:09
noonedeadpunkwell, I have both v2 and v310:09
gtemayes, a well known practice to have: volume, volumev2, volumev310:10
gtemalook where the volume type is pointing you to10:10
noonedeadpunkthe most thing that concerns me is http://paste.openstack.org/show/799728/10:10
gtemathis is another interesting case: v2=v3 in the CLI10:10
noonedeadpunkaha10:11
gtemahttps://opendev.org/openstack/python-openstackclient/src/branch/master/setup.cfg#L66210:12
noonedeadpunkwell, I think it's not... What I'm trying to do - volume resize when it's in-use.10:12
noonedeadpunkSo client has check https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/volume/v2/volume.py#L60810:12
noonedeadpunkhowever it's not true for modern cinder https://opendev.org/openstack/cinder/src/branch/stable/train/cinder/volume/manager.py#L276610:12
noonedeadpunkbut true for v2 I guess10:13
noonedeadpunkNot sure what's the best way to workaround here...10:13
openstackgerritStephen Finucane proposed openstack/osc-lib master: Add Python3 wallaby unit tests  https://review.opendev.org/75033510:16
noonedeadpunkwell, or not....10:18
noonedeadpunkit is https://opendev.org/openstack/cinder/src/branch/master/cinder/api/openstack/rest_api_version_history.rst#user-content-section-3910:20
*** noonedeadpunk has quit IRC10:32
*** noonedeadpunk has joined #openstack-sdks10:32
noonedeadpunkgtema: oh, that is neat, means I can kind of overwrite with v3 only single method?10:50
gtemanot the method, but you need more or less to copy/modify complete resource10:51
noonedeadpunkyeah, agree, sure10:52
fricklerdoesn't the volume-resize-while-in-use require a bumped microversion?10:54
noonedeadpunkit does10:55
noonedeadpunkbut I think that issue here is that openstackclient additionally checks for the volume status10:56
noonedeadpunkmoreover http://paste.openstack.org/show/799733/10:58
fricklerhmm, weird, I was pretty sure that it did work for me earlier. will need to double-check after the meeting starting now10:59
*** noonedeadpunk has quit IRC11:21
fricklernoonedeadpunk: 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 cinder11:22
openstackgerritMerged openstack/osc-lib master: bump py37 to py38 in tox.ini  https://review.opendev.org/75261311:25
*** dtantsur|afk is now known as dtantsur11:26
*** noonedeadpunk has joined #openstack-sdks11:26
noonedeadpunkuh, everything is so weird about volumes part in openstackclient12:07
*** jpich has quit IRC12:11
*** jpich has joined #openstack-sdks12:11
*** gtema has quit IRC12:16
openstackgerritMerged openstack/osc-lib master: Add Python3 wallaby unit tests  https://review.opendev.org/75033512:32
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server * -f yaml' output  https://review.opendev.org/76120512:34
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'usage * -f yaml' output  https://review.opendev.org/76159512:34
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server group * -f yaml' output  https://review.opendev.org/76159612:34
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Add support for 'server group create --rule' parameter  https://review.opendev.org/76159712:34
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: trivial: Use plural for appended parameters  https://review.opendev.org/76159812:34
*** gtema has joined #openstack-sdks12:41
stephenfinsmcginnis: 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/8ff3f80102ae184227d4f8cb5f82bedee987b80412:42
stephenfinsmcginnis: 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
openstackgerritStephen Finucane proposed openstack/osc-lib stable/victoria: tox: Set minimum Python version for lower-constraints  https://review.opendev.org/76160012:47
openstackgerritStephen Finucane proposed openstack/osc-lib stable/ussuri: zuul: Stop testing against Tempest  https://review.opendev.org/76160312:49
openstackgerritjayaditya gupta proposed openstack/python-openstackclient master: Add a few selectable fields to the "openstack server list" output  https://review.opendev.org/74118113:01
smcginnisstephenfin: I think either approach.13:11
smcginnisstephenfin: 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
noonedeadpunkany way to get cinder client to use different api version? http://paste.openstack.org/show/799741/13:12
smcginnisstephenfin: There were a few other projects that ended up backporting those lower-constraints updates to work with 3.8.13:13
smcginnisFor consistency, that's probably the right thing to do.13:13
smcginnisnoonedeadpunk: That .version value doesn't reflect the version sent when making API calls.13:18
noonedeadpunkyeah already found it....13:18
smcginnisk13:18
smcginnisapi_version, right?13:18
noonedeadpunkyep13:18
* noonedeadpunk tries to find a way to make CLI client to do volume resize with attached volumes13:19
smcginnisnoonedeadpunk: In case you need to read up on any background on that: https://specs.openstack.org/openstack/cinder-specs/specs/pike/extend-attached-volume.html13:20
noonedeadpunkwell, I can do it with curl ofc, but it's not really handy^)13:21
noonedeadpunkand openstackclient just fails because of https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/volume/v2/volume.py#L60813:21
noonedeadpunkoh, btw, once I've catched you:)13:22
noonedeadpunksmcginnis: is it safe to execute this with auth_type with service credentials?13:22
noonedeadpunkI'm a bit worried about privilege escalation here13:23
noonedeadpunk(as there were probably reasons to use session token by default instead)13:23
smcginnisGood question. I believe it should be safe.13:23
noonedeadpunkprobbly 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 IRC14:09
*** jpich has joined #openstack-sdks14:10
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Remove references to Python 2.7  https://review.opendev.org/76161514:16
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Remove unnecessary test  https://review.opendev.org/76161614:16
*** dtantsur has quit IRC14:16
openstackgerritStephen Finucane proposed openstack/osc-lib stable/victoria: Fix l-c testing for ubuntu focal  https://review.opendev.org/76161714:17
stephenfinsmcginnis: Okay, makes sense. Care to take a look at that change ^ in that case?14:17
*** dtantsur has joined #openstack-sdks14:37
*** dtantsur has quit IRC14:37
*** dtantsur has joined #openstack-sdks14:38
*** belmoreira has joined #openstack-sdks14:56
*** dtantsur has quit IRC14:56
*** dtantsur has joined #openstack-sdks14:56
*** Hybrid512 has quit IRC15:23
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/python-openstackclient master: Allow to resize in-use volumes  https://review.opendev.org/76163315:51
*** jpich has quit IRC16:04
*** jpich has joined #openstack-sdks16:05
*** Hybrid512 has joined #openstack-sdks16:07
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/python-openstackclient master: Allow to resize in-use volumes  https://review.opendev.org/76163316:08
*** belmoreira has quit IRC16:09
*** gtema has quit IRC16:56
openstackgerritMerged openstack/osc-lib stable/ussuri: zuul: Stop testing against Tempest  https://review.opendev.org/76160317:13
*** rpittau is now known as rpittau|afk17:21
*** jpich has quit IRC17:24
openstackgerritRodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add "fields" parameter to ListPort query  https://review.opendev.org/75411717:26
openstackgerritMerged openstack/osc-lib stable/victoria: Fix l-c testing for ubuntu focal  https://review.opendev.org/76161717:45
openstackgerritStephen Finucane proposed openstack/osc-lib stable/victoria: zuul: Stop testing against Tempest  https://review.opendev.org/76152517:50
*** ralonsoh has quit IRC18:41
*** sshnaidm is now known as sshnaidm|afk18:44
*** dtantsur is now known as dtantsur|afk18:53
*** gtema has joined #openstack-sdks18:57
*** gtema has quit IRC19:02
*** lbragstad has quit IRC19:18
*** Hybrid512 has quit IRC19:47
*** hyang has joined #openstack-sdks20:22
*** hyang has quit IRC21:14
*** slaweq has quit IRC22:06
*** tosky has quit IRC23:55

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!