| opendevreview | Sofia Enriquez proposed openstack/cinder master: Implement project personas for group snapshots https://review.opendev.org/c/openstack/cinder/+/759961 | 00:05 |
|---|---|---|
| opendevreview | Ade Lee proposed openstack/cinder master: WIP/DNM: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 05:01 |
| opendevreview | Xuan Yandong proposed openstack/cinder master: Prevent the volume association of the VM from being released when uploading the vol to image https://review.opendev.org/c/openstack/cinder/+/807819 | 05:56 |
| opendevreview | Xuan Yandong proposed openstack/cinder master: Prevent the volume association of the VM from being released when uploading the vol to image https://review.opendev.org/c/openstack/cinder/+/807819 | 06:05 |
| *** pjakuszew7 is now known as pjakuszew | 07:09 | |
| geguileo | whoami-rajat: I don't follow... We just need to modify how we get the volumes in those specific migration methods, not for other cases. | 07:58 |
| geguileo | and those methods don't return any data, just modify it, right? | 07:58 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Implement project personas for volume API https://review.opendev.org/c/openstack/cinder/+/760199 | 08:03 |
| *** elodilles_pto is now known as elodilles | 08:44 | |
| venkatakrishnat | Hi eharney, geguileo, whoami-rajat: Could you please review https://review.opendev.org/c/openstack/cinder/+/776307 which is to fix IBM SVC HyperSwap snapshot and clone related issues for IBM SVC driver. This is must-go patch to Xena release. | 08:44 |
| opendevreview | Xuan Yandong proposed openstack/cinder master: Prevent the volume association of the VM from being released when uploading the vol to image https://review.opendev.org/c/openstack/cinder/+/807819 | 10:23 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Implement project personas for volume transfers https://review.opendev.org/c/openstack/cinder/+/760194 | 10:28 |
| raghavendrat | hi reviewers, it would be great if someone can look at: https://review.opendev.org/c/openstack/cinder/+/806725 | 12:48 |
| raghavendrat | Nimble: Add Alletra 6k related information | 12:49 |
| raghavendrat | Thank you | 12:49 |
| opendevreview | Venkata krishna Thumu proposed openstack/cinder master: [SVF] Fix extend issue for mirroring volumes https://review.opendev.org/c/openstack/cinder/+/784140 | 13:02 |
| zoharm | Hi, requesting quick review of the kumoscale cinder driver patch, it is a simple change and has one +2 | 13:35 |
| zoharm | https://review.opendev.org/c/openstack/cinder/+/803450 | 13:35 |
| zoharm | Also, please, requesting quick review of the brick patch that fixes nvmeof volume attachments on kernels compiled with nvmeof multipath enabled, such as the standard kernel that ships with Ubuntu 20.04 https://review.opendev.org/c/openstack/cinder/+/803450 | 13:37 |
| zoharm | There are mutiple other connector fixes that were bundled into the large "nvmeof agent" patch, but this fix is the most criticial, to allow folks to at least use this connector code on Ubuntu 20.04 without patching in out of tree code | 13:39 |
| zoharm | https://review.opendev.org/c/openstack/os-brick/+/806687 | 13:40 |
| rosmaita | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:58 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna ^^ | 13:58 |
| opendevreview | Merged openstack/cinder-specs master: Add yoga directory for specs https://review.opendev.org/c/openstack/cinder-specs/+/806567 | 14:23 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Implement project personas for volume transfers https://review.opendev.org/c/openstack/cinder/+/760194 | 14:27 |
| opendevreview | Merged openstack/rbd-iscsi-client master: Drop lower constraints testing https://review.opendev.org/c/openstack/rbd-iscsi-client/+/799914 | 14:40 |
| opendevreview | Venkata krishna Thumu proposed openstack/cinder master: [SVF]:Storwize HyperSwap snapshot clone is failing https://review.opendev.org/c/openstack/cinder/+/776307 | 14:54 |
| eharney | anyone familiar w/ this grenade job failure? https://zuul.opendev.org/t/openstack/build/b2dba2079ffd441b96d739bb6c6cad2e | 14:57 |
| eharney | last --subunit >>/tmp/subunit.uqfobcpt | 14:57 |
| eharney | last: unrecognized option '--subunit' | 14:57 |
| eharney | weird | 14:57 |
| rosmaita | very weird | 14:58 |
| eharney | the normal "last" command definitely doesn't do that | 14:58 |
| enriquetaso | #startmeeting cinder_bs | 15:01 |
| opendevmeet | Meeting started Wed Sep 8 15:01:20 2021 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:01 |
| enriquetaso | Welcome back to the cinder bug meeting | 15:01 |
| enriquetaso | Quick meeting today | 15:01 |
| enriquetaso | #topic bug_1 "Upload volume to image will terminate the vm's volume attachment" | 15:01 |
| eharney | comes from something like this where a variable is blank, i guess https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/fetch-subunit-output/tasks/main.yaml#L68 | 15:01 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1942953 | 15:01 |
| rosmaita | eharney: that job history looks ok, hopefully you just got unlucky | 15:02 |
| rosmaita | well, not hopefully, but you know what i mean | 15:03 |
| eharney | i would be surprised if 1942953 was that straightforward and the whole bug is really just "oh we shouldn't call detach from copy_volume_to_image sometimes" (which is what the proposed fix does) | 15:03 |
| enriquetaso | yes | 15:04 |
| enriquetaso | Fix proposed to branch: master | 15:04 |
| enriquetaso | #topic https://review.opendev.org/c/openstack/cinder/+/807819 | 15:04 |
| enriquetaso | Maybe anyone is currently working with this and knows if this is possible? | 15:04 |
| eharney | if what is possible? | 15:05 |
| enriquetaso | I didn't know that cinder allows uploading-to-image to an attached volume. | 15:05 |
| enriquetaso | maybe I'm confused about this | 15:06 |
| eharney | we need actual reproduction steps, i think | 15:07 |
| enriquetaso | #action(enriquetaso) ask for steps and more details | 15:07 |
| enriquetaso | sure | 15:07 |
| enriquetaso | #topic bug_2 "Undefined CONF.quota_metadata_items is used in unit tests" | 15:08 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1934036 | 15:08 |
| enriquetaso | Summary: Currently there are two places where CONF.quota_metadata_items is used in unit tests. However this parameter is not defined anywhere in cinder. It is very likely that the implementation was ported from nova and we missed removing that undefined parameter. | 15:08 |
| enriquetaso | Fix proposed | 15:09 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/798783 | 15:09 |
| enriquetaso | If anyone is working on tests and would like to review ^ please feel free to do so. | 15:09 |
| eharney | seems straightforward enough | 15:09 |
| enriquetaso | cool | 15:10 |
| enriquetaso | OK, that's all I have for today's meeting | 15:10 |
| enriquetaso | #topic open discussion | 15:10 |
| rosmaita | i never filed a bug about quota_volumes not working in the python-cinderclient functional tests, just mentioning that so y'all ask me about it occasionally | 15:12 |
| rosmaita | not sure if i'm not using devstack correctly or if it's a real problem | 15:13 |
| enriquetaso | good question, I haven't either | 15:14 |
| eharney | https://review.opendev.org/c/openstack/cinder/+/806010 could probably use a look before we get too far into the RC phase (since it sort of but not really changes a config option) | 15:15 |
| rosmaita | that's interesting, though, that we don't have a limit on max metadata allowed | 15:15 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/806010 | 15:15 |
| enriquetaso | OK | 15:17 |
| enriquetaso | ending meeting now if we don't have any more bugs for open discussion | 15:18 |
| rosmaita | sounds good, thanks sofia | 15:18 |
| enriquetaso | #endmeeting | 15:18 |
| opendevmeet | Meeting ended Wed Sep 8 15:18:33 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:18 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-09-08-15.01.html | 15:18 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-09-08-15.01.txt | 15:18 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-09-08-15.01.log.html | 15:18 |
| opendevreview | Eric Harney proposed openstack/cinder stable/wallaby: Change 'host' option from HostAddressOpt to StrOpt https://review.opendev.org/c/openstack/cinder/+/807898 | 15:22 |
| *** tosky is now known as Guest6702 | 15:22 | |
| *** tosky_ is now known as tosky | 15:22 | |
| venkatakrishnat | Hi, Can someone review this patch https://review.opendev.org/c/openstack/cinder/+/784140 which is to fix mirror volume-extend related issues for IBM SVC driver. CI passed. | 15:50 |
| opendevreview | Ade Lee proposed openstack/cinder master: WIP/DNM: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 15:55 |
| venkatakrishnat | Hi eharney, hemna, jungleboyj, rosmaita,whoami-rajat: Could you please review this patch https://review.opendev.org/c/openstack/cinder/+/784140 | 17:16 |
| knomura | Hi team, please review the patch for GVG feature in Hitachi driver. https://review.opendev.org/c/openstack/cinder/+/782910/ | 17:35 |
| knomura | It is a registered as FFE, but is large code change, if you have a time , please review it. | 17:35 |
| knomura | thanks. | 17:36 |
| hemna | ok | 18:18 |
| opendevreview | Merged openstack/cinderlib master: Drop lower-constraints file https://review.opendev.org/c/openstack/cinderlib/+/799913 | 18:26 |
| opendevreview | Merged openstack/cinder master: Log connection info returned from driver https://review.opendev.org/c/openstack/cinder/+/803696 | 18:27 |
| opendevreview | Merged openstack/cinder master: Change 'host' option from HostAddressOpt to StrOpt https://review.opendev.org/c/openstack/cinder/+/806010 | 18:27 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Fix missing RBD delete_volume test case https://review.opendev.org/c/openstack/cinder/+/807933 | 18:32 |
| sfernand | rosmaita: was on meetings all days couldn't join the cinder meeting. Fabio told me you mentioned the nas options, right? I didn't added any comments there yet due to the hurry for the feature freeze last week but will take some time to do it asap | 18:48 |
| rosmaita | sfernand: that would be helpful, the next day or so will be fine | 18:48 |
| rosmaita | i figure that you have a lot of knowledge of/opinions about NFS | 18:48 |
| sfernand | ok! | 18:49 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Implement project personas for volume actions https://review.opendev.org/c/openstack/cinder/+/806965 | 19:44 |
| *** elodilles is now known as elodilles_pto | 20:12 | |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org is going offline momentarily for a host migration and zuul upgrade, downtime should be only a few minutes. | 21:05 | |
| opendevreview | Alan Bishop proposed openstack/cinder master: Implement project personas for quotas and quota_classes https://review.opendev.org/c/openstack/cinder/+/760038 | 23:36 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!