| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: fix to initialize a lock counter https://review.opendev.org/c/openstack/cinder/+/892616 | 05:18 |
|---|---|---|
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi and OEM: masking REST API token on log https://review.opendev.org/c/openstack/cinder/+/899113 | 05:19 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Restore to support ``AIX`` as host OS type https://review.opendev.org/c/openstack/cinder/+/899548 | 05:19 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder stable/2023.1: Make default-types APIs compatible with V3.67 https://review.opendev.org/c/openstack/cinder/+/930863 | 05:48 |
| whoami-rajat | rosmaita, jbernard_ last backport for this patch (then i will bother the unmaintained folks for zed and yoga) https://review.opendev.org/c/openstack/cinder/+/930863 | 05:49 |
| opendevreview | Bertrand Lanson proposed openstack/cinder master: Fix type passed to write function during backup restoration https://review.opendev.org/c/openstack/cinder/+/930682 | 06:48 |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: HPE 3PAR: Add HPE Alletra MP related information https://review.opendev.org/c/openstack/cinder/+/930872 | 06:55 |
| *** geguileo is now known as Guest4975 | 07:54 | |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Use builtin hashlib.md5 https://review.opendev.org/c/openstack/cinder/+/930882 | 08:49 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Use builtin hashlib.md5 https://review.opendev.org/c/openstack/cinder/+/930882 | 10:38 |
| opendevreview | Takashi Kajinami proposed openstack/os-brick master: Remove python 3.8 support https://review.opendev.org/c/openstack/os-brick/+/930899 | 11:30 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Use builtin hashlib.md5 https://review.opendev.org/c/openstack/cinder/+/930882 | 11:31 |
| opendevreview | Takashi Kajinami proposed openstack/os-brick master: Remove Python 3.8 support https://review.opendev.org/c/openstack/os-brick/+/930899 | 11:31 |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: HPE 3PAR: Add HPE Alletra MP related information https://review.opendev.org/c/openstack/cinder/+/930872 | 12:16 |
| *** kgube1 is now known as kgube | 12:30 | |
| ganso | rosmaita, whoami-rajat_: hi! quick question about a bug, do you think solution (a) or (b) is better? both of them break the rules a little bit: https://bugs.launchpad.net/cinder/+bug/2083061 | 15:34 |
| rosmaita | ganso: will take a look | 15:34 |
| rosmaita | well, with regard to (a), it's a fairly large patch, but about 1/3 is unit tests, and it's isolated to a single driver, so an argument for backporting could be made, especially if there's evidence that it hasn't caused regressions in caracal+ | 15:43 |
| rosmaita | we should probably ask eharney for an assessment of whether it would be a clean backport, or would require other supporting patches in bobcat and earlier | 15:43 |
| rosmaita | ganso: ^^ | 15:43 |
| ganso | rosmaita: on the other hand, (b) is a harmless one-liner :) | 15:44 |
| ganso | rosmaita: thinking about the chance of regressions and impact, (b) seems safer even if we break the rules by patching directly into stable/branches | 15:46 |
| ganso | rosmaita: I can't make the argument that (a) could be safer or just as safe as (b) | 15:47 |
| rosmaita | ganso: it's ok per openstack policy to patch directly into stable branches if the fix doesn't apply to more recent branches because the code has changed | 15:47 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [IBM SVf Driver] Added support of clone operation for volumegroup https://review.opendev.org/c/openstack/cinder/+/929777 | 16:01 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [IBM SVf Driver] Added support of clone operation for volumegroup https://review.opendev.org/c/openstack/cinder/+/929777 | 16:10 |
| ganso | rosmaita: ok, that is good, it is not strictly breaking rules then. Let's see what eharney and whoami-rajat_ think | 16:12 |
| *** jbernard_ is now known as jbernard | 16:42 | |
| *** whoami-rajat_ is now known as whoami-rajat | 17:08 | |
| whoami-rajat | ganso, rosmaita eharney i remember we treated the big RBD patch as a bug fix, on the other hand it also has upgrades impact and a new config option | 17:10 |
| whoami-rajat | the retry mechanism might solve one of the issues but we had several issues with RBD image dependencies that Eric's patch addresses | 17:10 |
| whoami-rajat | I'm not against the idea of backporting if the upgrade impact is safe enough that it won't break existing deployments i.e. a suitable value for config options | 17:11 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [IBM SVf Driver] Added support of clone operation for volumegroup https://review.opendev.org/c/openstack/cinder/+/929777 | 17:31 |
| rosmaita | ganso: let's discuss https://bugs.launchpad.net/cinder/+bug/2083061 at the cinder meeting on wednesday 1400 UTC ... also, how far back do you want this backported? | 18:32 |
| ganso | rosmaita: yoga. The backport for the unmaintained branches will be done downstream | 18:35 |
| opendevreview | Merged openstack/cinder stable/2023.1: Make default-types APIs compatible with V3.67 https://review.opendev.org/c/openstack/cinder/+/930863 | 21:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!