Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-11-04
17:47:03 stephenfin I don't know if we want to wait for cliff and osc-lib releases, but if not that patch should clean up the tests and let me keep working on the compute issues in a sane manner :) ^
19:59:36 openstackgerrit Merged openstack/osc-lib master: zuul: Stop testing against Tempest https://review.opendev.org/760343
21:27:56 openstackgerrit Merged openstack/osc-lib master: formatters: Cast columns before outputting https://review.opendev.org/761394
21:38:28 openstackgerrit Merged openstack/osc-lib master: tox: Set 'ignore_basepython_conflict' https://review.opendev.org/761428
#openstack-sdks - 2020-11-05
05:48:00 openstackgerrit zhufl proposed openstack/openstacksdk master: Fix invalid assertIsNotNone statement https://review.opendev.org/761053
08:54:24 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: Add a few selectable fields to the "openstack server list" output https://review.opendev.org/741181
10:06:18 stephenfin gtema: amotoki: Thanks for the reviews :) Any chance you could look at https://review.opendev.org/#/c/761421/ too?
10:06:59 gtema stephenfin - sure. I requested release of osc-lib
10:07:22 noonedeadpunk o/
10:07:50 stephenfin gtema++ great, thanks :)
10:07:55 noonedeadpunk I'm trying to get why `openstackclient/volume/v2` is used and where does it comes from...
10:08:43 noonedeadpunk as it seems it has limitations v3 should not have, but kind of v3 is just empty for me
10:09:09 openstackgerrit Stephen Finucane proposed openstack/osc-lib stable/victoria: zuul: Stop testing against Tempest https://review.opendev.org/761525
10:09:22 gtema noonedeadpunk - mostly it comes from whatever version is being pointed in the service catalog (service_type=volume)
10:09:31 gtema normally operators forget to switch it to v3
10:09:42 noonedeadpunk well, I have both v2 and v3
10:10:06 gtema yes, a well known practice to have: volume, volumev2, volumev3
10:10:22 gtema look where the volume type is pointing you to
10:10:25 noonedeadpunk the most thing that concerns me is http://paste.openstack.org/show/799728/
10:10:54 gtema this is another interesting case: v2=v3 in the CLI
10:11:06 noonedeadpunk aha
10:12:19 gtema https://opendev.org/openstack/python-openstackclient/src/branch/master/setup.cfg#L662
10:12:20 noonedeadpunk well, I think it's not... What I'm trying to do - volume resize when it's in-use.
10:12:35 noonedeadpunk So client has check https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/volume/v2/volume.py#L608
10:12:56 noonedeadpunk however it's not true for modern cinder https://opendev.org/openstack/cinder/src/branch/stable/train/cinder/volume/manager.py#L2766
10:13:06 noonedeadpunk but true for v2 I guess
10:13:37 noonedeadpunk Not sure what's the best way to workaround here...
10:16:16 openstackgerrit Stephen Finucane proposed openstack/osc-lib master: Add Python3 wallaby unit tests https://review.opendev.org/750335
10:18:44 noonedeadpunk well, or not....
10:20:57 noonedeadpunk it is https://opendev.org/openstack/cinder/src/branch/master/cinder/api/openstack/rest_api_version_history.rst#user-content-section-39
10:50:47 noonedeadpunk gtema: oh, that is neat, means I can kind of overwrite with v3 only single method?
10:51:21 gtema not the method, but you need more or less to copy/modify complete resource
10:52:36 noonedeadpunk yeah, agree, sure
10:54:56 frickler doesn't the volume-resize-while-in-use require a bumped microversion?
10:55:27 noonedeadpunk it does
10:56:01 noonedeadpunk but I think that issue here is that openstackclient additionally checks for the volume status
10:58:38 noonedeadpunk moreover http://paste.openstack.org/show/799733/
10:59:18 frickler hmm, weird, I was pretty sure that it did work for me earlier. will need to double-check after the meeting starting now
11:22:49 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:25:32 openstackgerrit Merged openstack/osc-lib master: bump py37 to py38 in tox.ini https://review.opendev.org/752613
12:07:01 noonedeadpunk uh, everything is so weird about volumes part in openstackclient
12:32:19 openstackgerrit Merged openstack/osc-lib master: Add Python3 wallaby unit tests https://review.opendev.org/750335
12:34:55 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'usage * -f yaml' output https://review.opendev.org/761595
12:34:55 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server * -f yaml' output https://review.opendev.org/761205
12:34:56 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add support for 'server group create --rule' parameter https://review.opendev.org/761597
12:34:56 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server group * -f yaml' output https://review.opendev.org/761596
12:34:57 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: trivial: Use plural for appended parameters https://review.opendev.org/761598
12:42:09 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:43:04 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:47:15 openstackgerrit Stephen Finucane proposed openstack/osc-lib stable/victoria: tox: Set minimum Python version for lower-constraints https://review.opendev.org/761600
12:49:56 openstackgerrit Stephen Finucane proposed openstack/osc-lib stable/ussuri: zuul: Stop testing against Tempest https://review.opendev.org/761603
13:01:22 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:11:48 smcginnis stephenfin: I think either approach.
13:12:26 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:35 noonedeadpunk any way to get cinder client to use different api version? http://paste.openstack.org/show/799741/
13:13:06 smcginnis stephenfin: There were a few other projects that ended up backporting those lower-constraints updates to work with 3.8.
13:13:21 smcginnis For consistency, that's probably the right thing to do.
13:18:08 smcginnis noonedeadpunk: That .version value doesn't reflect the version sent when making API calls.
13:18:20 noonedeadpunk yeah already found it....
13:18:30 smcginnis k
13:18:39 smcginnis api_version, right?
13:18:43 noonedeadpunk yep
13:20:35 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:21:20 noonedeadpunk well, I can do it with curl ofc, but it's not really handy^)
13:21:43 noonedeadpunk and openstackclient just fails because of https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/volume/v2/volume.py#L608
13:22:07 noonedeadpunk oh, btw, once I've catched you:)
13:22:50 noonedeadpunk smcginnis: is it safe to execute this with auth_type with service credentials?
13:23:06 noonedeadpunk I'm a bit worried about privilege escalation here
13:23:22 noonedeadpunk (as there were probably reasons to use session token by default instead)
13:23:54 smcginnis Good question. I believe it should be safe.
13:25:46 noonedeadpunk probbly would be awesome to add these options to the list of supported somehow, but that's complete another topic.... Thanks!
14:16:04 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Remove references to Python 2.7 https://review.opendev.org/761615
14:16:05 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Remove unnecessary test https://review.opendev.org/761616
14:17:28 openstackgerrit Stephen Finucane proposed openstack/osc-lib stable/victoria: Fix l-c testing for ubuntu focal https://review.opendev.org/761617
14:17:50 stephenfin smcginnis: Okay, makes sense. Care to take a look at that change ^ in that case?
15:51:43 openstackgerrit Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/python-openstackclient master: Allow to resize in-use volumes https://review.opendev.org/761633
16:08:57 openstackgerrit Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/python-openstackclient master: Allow to resize in-use volumes https://review.opendev.org/761633
17:13:22 openstackgerrit Merged openstack/osc-lib stable/ussuri: zuul: Stop testing against Tempest https://review.opendev.org/761603
17:26:49 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add "fields" parameter to ListPort query https://review.opendev.org/754117
17:45:17 openstackgerrit Merged openstack/osc-lib stable/victoria: Fix l-c testing for ubuntu focal https://review.opendev.org/761617
17:50:44 openstackgerrit Stephen Finucane proposed openstack/osc-lib stable/victoria: zuul: Stop testing against Tempest https://review.opendev.org/761525
#openstack-sdks - 2020-11-06
08:05:58 openstackgerrit Merged openstack/cliff master: columns: Make 'FormattableColumn' comparable https://review.opendev.org/761421
08:12:00 openstackgerrit Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/python-openstackclient master: Allow to resize in-use volumes https://review.opendev.org/761633
08:23:48 openstackgerrit Merged openstack/cliff master: Update requirements URLs in tox config https://review.opendev.org/760342
09:58:50 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server * -f yaml' output https://review.opendev.org/761205
09:58:51 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server group * -f yaml' output https://review.opendev.org/761596
09:58:51 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'usage * -f yaml' output https://review.opendev.org/761595
09:58:52 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: trivial: Use plural for appended parameters https://review.opendev.org/761598
09:58:52 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add support for 'server group create --rule' parameter https://review.opendev.org/761597
10:16:17 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Remove unnecessary test https://review.opendev.org/761616
10:31:37 openstackgerrit Stephen Finucane proposed openstack/osc-lib master: Replace 'assertFalse(a in b)' with 'assertNotIn(a, b)' https://review.opendev.org/535733
10:36:41 openstackgerrit Stephen Finucane proposed openstack/osc-lib master: Check auth_url as well as identity version api https://review.opendev.org/526413
10:40:04 openstackgerrit Merged openstack/osc-lib stable/victoria: zuul: Stop testing against Tempest https://review.opendev.org/761525
11:13:54 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Add 'server show --topology' option https://review.opendev.org/680928
11:15:17 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Add 'server show --topology' option https://review.opendev.org/680928
11:15:17 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server * -f yaml' output https://review.opendev.org/761205
12:19:18 openstackgerrit Merged openstack/osc-lib stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/750332
12:23:50 openstackgerrit Merged openstack/osc-lib stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/750333
12:46:54 openstackgerrit Merged openstack/osc-lib stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/719015
12:46:55 openstackgerrit Merged openstack/osc-lib stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/719016

Earlier   Later