| *** mhen_ is now known as mhen | 01:21 | |
| rmnijgh | If a bug report is open for a long time, but has a propossed fix that seems to work. What can I do to get this checked-up and merged by someone? (I've never done this) | 06:08 |
|---|---|---|
| frickler | rmnijgh: well it certainly could be helpful to link to the bug and the fix here ;) | 07:57 |
| rmnijgh | frickler: yea deff helpfull sorry https://bugs.launchpad.net/cinder/+bug/1995204 | 08:58 |
| frickler | rmnijgh: https://review.opendev.org/c/openstack/cinder/+/863600 seems to be failing some unit tests, which indicates that the proposed fix isn't completely without issues, you may want to look into that | 09:11 |
| opendevreview | Tobias Urdin proposed openstack/cinder master: Add finished_at column for backup https://review.opendev.org/c/openstack/cinder/+/933216 | 09:18 |
| frickler | (another possibility is that the unit tests are actually verifying the broken behaviour and need amending, but that needs someone with more detailed knowledge than me to look at it) | 09:21 |
| rmnijgh | I've tested the commit on my devstack enviremont with 2 cinder controllers. and it worked without issues. After looking into the logs of those build, one of them was complaining it couldnt get admin_context, so i assumed some of them where just broke. | 09:44 |
| opendevreview | Fernando Ferraz proposed openstack/cinder stable/2025.1: NFS driver: Fix fail creating volume with multiple snapshots https://review.opendev.org/c/openstack/cinder/+/947820 | 12:29 |
| opendevreview | Tobias Urdin proposed openstack/cinder master: Add finished_at column for backup https://review.opendev.org/c/openstack/cinder/+/933216 | 12:56 |
| opendevreview | Cyprian Kleist proposed openstack/cinder stable/2025.1: Fix type passed to write function during backup restoration https://review.opendev.org/c/openstack/cinder/+/947827 | 12:59 |
| rosmaita | frickler: thanks for triaging cinder issues! | 13:00 |
| rosmaita | rmnijgh: i suspect some of the unit test failures may be that some tests check to see what was called, and you are adding a new function that maybe needs to be mocked for those tests, something like that | 13:00 |
| fungi | does anyone know this user who seems to manage macrosan's third-party ci? they seem to be having trouble understanding the template instructions for creating a new page, so i've had to roll back their destructive edits to the instructions twice now: https://wiki.openstack.org/wiki/Special:Contributions/Xielongfei | 13:54 |
| frickler | rosmaita: sure thing, not completely without self-interest in this case ;) | 14:03 |
| opendevreview | jayaanand borra proposed openstack/cinder master: [NetApp]:Cinder support for self-signed transport Current NetApp Cinder driver is not supporting https transport using self-singed certificate. This patch will enable transport using self-signed certificate. https://review.opendev.org/c/openstack/cinder/+/947663 | 15:52 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NVMe-oF Target: Fix incorrect check for initiator in connector data https://review.opendev.org/c/openstack/cinder/+/946103 | 18:37 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NVMe-oF Target: Fix incorrect check for initiator in connector data https://review.opendev.org/c/openstack/cinder/+/946103 | 18:40 |
| opendevreview | Hao Wang proposed openstack/cinder master: _process_stack in wsgi disables admin API policy https://review.opendev.org/c/openstack/cinder/+/939154 | 18:41 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: nfs: minimal mypy coverage https://review.opendev.org/c/openstack/cinder/+/947745 | 20:07 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Fix fail to resize NFS volume with snapshots https://review.opendev.org/c/openstack/cinder/+/943659 | 20:07 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Set volume format for snapshot operations https://review.opendev.org/c/openstack/cinder/+/943660 | 20:07 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Use database format in initialize_connection https://review.opendev.org/c/openstack/cinder/+/941173 | 20:07 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Fix driver fails to verify virtual size for Glance with Cinder/NFS https://review.opendev.org/c/openstack/cinder/+/941174 | 20:07 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: nfs: minimal mypy coverage https://review.opendev.org/c/openstack/cinder/+/947745 | 20:11 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Fix fail to resize NFS volume with snapshots https://review.opendev.org/c/openstack/cinder/+/943659 | 20:11 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Set volume format for snapshot operations https://review.opendev.org/c/openstack/cinder/+/943660 | 20:11 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Use database format in initialize_connection https://review.opendev.org/c/openstack/cinder/+/941173 | 20:11 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Fix driver fails to verify virtual size for Glance with Cinder/NFS https://review.opendev.org/c/openstack/cinder/+/941174 | 20:11 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: nfs: minimal mypy coverage https://review.opendev.org/c/openstack/cinder/+/947745 | 22:22 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Fix fail to resize NFS volume with snapshots https://review.opendev.org/c/openstack/cinder/+/943659 | 22:22 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Set volume format for snapshot operations https://review.opendev.org/c/openstack/cinder/+/943660 | 22:22 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Use database format in initialize_connection https://review.opendev.org/c/openstack/cinder/+/941173 | 22:22 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NFS driver: Fix driver fails to verify virtual size for Glance with Cinder/NFS https://review.opendev.org/c/openstack/cinder/+/941174 | 22:22 |
| opendevreview | Fernando Ferraz proposed openstack/devstack-plugin-nfs master: [DNM] Run Glance with Cinder as backend https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/940482 | 23:34 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!