Monday, 2021-09-20

opendevreviewChris M proposed openstack/cinder master: Dell PowerVault driver: Fix documentation typos
opendevreviewChris M proposed openstack/cinder master: Seagate driver: fix get_volume_size()
opendevreviewStephen Finucane proposed openstack/cinder master: Fix cinder-manage db version
stephenfin^ FYI, I haven't tested that since I can get tox to run (it's complaining about package mismatches and I haven't had time to investigate). However, as noted in the bug, nova had this issue and the code is the same so...11:27
*** dviroel|out is now known as dviroel11:36
opendevreviewGiulio Fidente proposed openstack/devstack-plugin-ceph master: Use Ceph Pacific for devstack-plugin-ceph-tempest-py3 jobs
opendevreviewBrian Rosmaita proposed openstack/cinder master: Raise min of oslo.policy to 3.8.1
rosmaitawhoami-rajat: when you have a minute, see what you think of Thierry's comment on
whoami-rajatrosmaita, I looked but didn't know how to respond, In cinder we allow driver feature backports and we are mostly going to end up with driver features in every stable release and a minor version bump for every cinder release12:44
whoami-rajatI think i will just go with what the majority thinks it's right12:44
rosmaitawhoami-rajat: i think he had a good point about
rosmaitaso it's not necessarily something we should do every time, but it might make sense for this one12:46
whoami-rajathmm, that upgrade note is specific to rbd which already had that option and we just changed the default value of it and is not a feature but if his concern is more towards the upgrade notes then makes sense to me12:48
rosmaitawhoami-rajat: ok, why don't you update the patch and you can mention something about incrementing the minor release number to draw attention to the upgrade note, that way you won't be stuck with a precedent to have to do it when feature-y changes are included in a driver12:52
whoami-rajatrosmaita, ok, will do, my concern is that, from time to time, the release team asks for a change and I'm not sure about if we are defining a new rule to do this from now on and if not then what's the reason for doing it this time12:56
rosmaitawhoami-rajat: i understand ... you can always say something on the patch, and if they actually put a negative vote on a change that you disagree with, you can always put it on the agenda for a release team meeting for a discussion13:02
whoami-rajatrosmaita, ack, thanks :)13:15
zigoFYI, cinder must require a higher version of oslo.log that contains XENA, otherwise, we get:14:04
zigo  File "/<<PKGBUILDDIR>>/cinder/policies/", line 264, in <module>14:04
zigo    DEPRECATED_SINCE = versionutils.deprecated.XENA14:04
zigoAttributeError: type object 'deprecated' has no attribute 'XENA'14:04
zigo(during unit tests discovery)14:04
zigoNow I'm getting this:14:09
zigoAny clue what's going on?14:09
raghavendrathi core reviewers, this is regarding
raghavendratHPE 3PAR: Reuse existing session - its bugfix14:27
raghavendratIt would be great if someone can take a look. Thanks.14:28
zigorosmaita: Thanks. Any clue regarding ?14:33
rosmaitazigo: looking at that now, not sure14:33
rosmaitazigo: i think i need to bump min version for SQLAlchemy, sqlalchemy-migrate and SQLAlchemy-Utils14:37
zigoOh, sqlalchemy-utils is probably the one.14:37
zigoI had this with other projects.14:37
rosmaitazigo: ty for bringing it up14:37
rosmaitazigo: i thought we had already upped SQLAlchemy to 1.4.0 and fixed some backward incompatibilities, but maybe we fixed and didn't change requirements.txt14:38
zigoFYI, when I package a new OpenStack release, everything is taken from Debian Unstable, except what I explicitely force to the version of Experimental, so I quickly see breakages ...14:39
rosmaitazigo: any objection to SQLAlchemy>=1.4.22, sqlalchemy-migrate>=0.13.0, SQLAlchemy-Utils>=0.37.8 ?14:40
zigopython3-sqlalchemy-utils was the one !14:40
zigoIt didn't work with 0.36.8 (from unstable) but it worked with 0.37.6 (from experimental).14:40
zigorosmaita: 0.37.6 is enough, as much as I can see.14:40
rosmaitazigo: i am ok with 0.37.614:41
zigoFYI, I use SQLAlchemy 1.4.2314:41
rosmaitaok, so >=1.4.22 won't hurt you14:42
zigo(I'm not the package maintainer for that one, but the person doing it learned to be cooperative)14:42
zigoYeah, that's fine to me.14:42
rosmaitai'll put up a quick patch for sqlalchemy changes, and then review the rest of requirements.txt ... i did not think that it needed an update for release, but i am clearly mistaken14:42
zigo <--- This says 1.4.23 as well now.14:43
rosmaitazigo: i will rerun my local tests and see what gets pulled down14:43
* zigo got a successful unit test run and a ready-to-be-uploaded Cinder package... :)14:45
rosmaitazigo: ok, i will go with SQLAlchemy>=1.4.23, sqlalchemy-migrate>=0.13.0, SQLAlchemy-Utils>=0.37.814:55
opendevreviewBrian Rosmaita proposed openstack/cinder master: Update SQLAlchemy related requirements
opendevreviewSofia Enriquez proposed openstack/cinder stable/ussuri: Allow removing NFS snapshots in error status
mlozahello, does cinder backup support multi backend? I have two backends enabled: rbd-1 and rbd-2, when I a backup on that volumes from rbd-2, it goes to rbd-1 backup pool. This is my cinder-backup config
mlozaI added backup_use_same_host = True in [DEFAULT] and restarted cinder but after that, doing backups  on both backends no longer work. 15:46
enriquetasomloza, cinder doesn't support backup multi backend as far as I know, there is a discussion in the mailing list regarding this:
mlozaenriquetaso: I see. Thanks for the info16:16
mlozais it possible to disable backup on a backend?16:46
mlozawhile waiting for backup multi backend to get implemented, as a workaround, I would want to disable backup on the backend 16:49
enriquetasoyes, the c-bak is optional 16:53
opendevreviewMerged openstack/cinder master: Raise minimum oslo.log to 4.5.0
whoami-rajatrosmaita, hey, any reason for using 3.8.1 and not 3.8.0, there's no functional difference between the two and I think the idea is to keep the min possible version here (or maybe I'm wrong)
opendevreviewFábio Oliveira proposed openstack/cinder master: NetApp Solidfire: Fix retype volume fail due to status
rosmaitawhoami-rajat: we could probably go with 3.8.0, but all the validation jobs were run with 3.8.1, so i figured it would be safest19:19
rosmaitaplus, it's about 4 months old19:20
whoami-rajatrosmaita, if there's no point regarding using the min possible version then 3.8.1 sounds good to me19:20
rosmaitayeah, since we don't have lower constraints anymore, i think it makes sense to go with what we know works19:22
whoami-rajatack, +Wed!19:23
whoami-rajatalso we can propose backport of , (if you're not planning to do all 4 backports together)19:24
whoami-rajatrosmaita, ^19:24
rosmaitago ahead and propose the backport, we can do them separately19:25
rosmaitai need to propose os-brick 5.0.019:26
rosmaitaand i'm going to run a local kind of lower-constraints job to see if anything else breaks19:26
opendevreviewRajat Dhasmana proposed openstack/cinder stable/xena: Raise minimum oslo.log to 4.5.0
whoami-rajatdone ^19:28
opendevreviewBrian Rosmaita proposed openstack/cinder master: Specify xena os-brick series in requirements
opendevreviewNahim Alves de Souza proposed openstack/cinder master: NetApp ONTAP: Fix sub-clone zapi call
opendevreviewMerged openstack/devstack-plugin-ceph master: Use Ceph Pacific for devstack-plugin-ceph-tempest-py3 jobs
*** dviroel is now known as dviroel|out21:09
opendevreviewMerged openstack/cinder stable/xena: Raise minimum oslo.log to 4.5.0
opendevreviewChris M proposed openstack/cinder master: Seagate driver: fix get_volume_size()
opendevreviewChris M proposed openstack/cinder master: Seagate driver: fix get_volume_size()
opendevreviewChris M proposed openstack/cinder master: Dell PowerVault driver: Fix documentation typos
opendevreviewChris M proposed openstack/cinder master: Dell PowerVault: Fix "cinder manageable-list"
opendevreviewBrian Rosmaita proposed openstack/cinder master: DNM: add tox env to check requirements
potsrosmaita Thanks for your guidance on Friday--I've split into 3 bugs: is doc-only and is just 1 line.22:51
rosmaitapots: ok, thanks23:04

Generated by 2.17.2 by Marius Gedminas - find it at!