| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/818037 | 02:16 |
|---|---|---|
| xuany | rosmaita: hello, | 03:11 |
| opendevreview | Abhishek Kekane proposed openstack/devstack-plugin-ceph master: Allow glance ceph osd cinder pool access https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/819476 | 07:14 |
| *** hemna0 is now known as hemna | 07:37 | |
| opendevreview | Tushar Trambak Gite proposed openstack/python-cinderclient master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/python-cinderclient/+/819259 | 07:58 |
| *** redrobot6 is now known as redrobot | 10:34 | |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Fix formatting of database models https://review.opendev.org/c/openstack/cinder/+/813219 | 10:47 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Correct 'nullable' mismatches on models https://review.opendev.org/c/openstack/cinder/+/813220 | 10:47 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Add missing foreign keys, indexes to models https://review.opendev.org/c/openstack/cinder/+/813221 | 10:47 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Resolve additional migration-model mismatches https://review.opendev.org/c/openstack/cinder/+/813222 | 10:47 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Add tests to ensure we keep migrations in sync https://review.opendev.org/c/openstack/cinder/+/813223 | 10:47 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: docs: Add whereto for testing redirect rules https://review.opendev.org/c/openstack/cinder/+/813224 | 10:47 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: docs: Rework and move upgrades guide https://review.opendev.org/c/openstack/cinder/+/813225 | 10:47 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Enable auto-generation of database migrations https://review.opendev.org/c/openstack/cinder/+/813226 | 10:47 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: docs: Remove unnecessary 'blockstorage-' prefix https://review.opendev.org/c/openstack/cinder/+/813227 | 10:47 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Move utility methods to top of file https://review.opendev.org/c/openstack/cinder/+/813228 | 10:47 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: WIP: db: Remove unnecessary sqlalchemy abstraction https://review.opendev.org/c/openstack/cinder/+/813229 | 10:47 |
| *** dviroel_ is now known as dviroel|rover | 11:14 | |
| opendevreview | Stephen Finucane proposed openstack/cinder master: tests: Restore - don't reset - warning filters https://review.opendev.org/c/openstack/cinder/+/821824 | 11:15 |
| *** tkajinam is now known as Guest8517 | 11:58 | |
| tosky | can please anyone give a final kick to this victoria backport of the matrix job to zuul v3? https://review.opendev.org/c/openstack/cinder/+/809447 | 13:36 |
| opendevreview | Brian Rosmaita proposed openstack/cinder-specs master: Os-brick NVMeoF connector multipathing spec https://review.opendev.org/c/openstack/cinder-specs/+/815613 | 13:54 |
| rosmaita | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira ^^ | 13:59 |
| *** whoami-rajat__ is now known as whoami-rajat | 14:00 | |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: backup https://review.opendev.org/c/openstack/cinder/+/787195 | 14:06 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: initiator https://review.opendev.org/c/openstack/os-brick/+/786732 | 14:18 |
| *** AndreBeltrami[m] is now known as andrebeltrami[m] | 14:19 | |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: lvm.py https://review.opendev.org/c/openstack/os-brick/+/786733 | 14:19 |
| shoffmann | Hi eharney, I missed to check for updates on the nas_secure option and retyping volumes discussion. Last time we discussed the proposal here https://etherpad.opendev.org/p/gSotXYAZ3JfJE8FEpMpS it is already implemented at the change https://review.opendev.org/c/openstack/cinder/+/802882 | 14:57 |
| shoffmann | Can you please have a look, if I missed something and update your review vote? | 14:58 |
| *** simondodsley_ is now known as simondodsley | 14:58 | |
| *** mnaser_ is now known as mnaser | 14:58 | |
| *** angiolucci_ is now known as angiolucci | 14:59 | |
| *** manpreetk_ is now known as manpreetk | 15:00 | |
| *** viks___ is now known as viks__ | 15:00 | |
| *** bbezak_ is now known as bbezak | 15:00 | |
| *** erbarr_ is now known as erbarr | 15:01 | |
| *** walshh__ is now known as walshh_ | 15:01 | |
| rosmaita | i think i am back | 15:07 |
| enriquetaso | hey | 15:11 |
| enriquetaso | not sure about the bug meeting | 15:11 |
| enriquetaso | I'll start and we'll see | 15:11 |
| enriquetaso | #startmeeting cinder_bs | 15:11 |
| opendevmeet | Meeting started Wed Dec 15 15:11:53 2021 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:11 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:11 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:11 |
| enriquetaso | Welcome back! | 15:12 |
| enriquetaso | Full list of bugs | 15:12 |
| enriquetaso | #link http://lists.openstack.org/pipermail/openstack-discuss/2021-December/026351.html | 15:12 |
| enriquetaso | We have two incomplete bugs and one regarding volume type. | 15:12 |
| enriquetaso | #topic VolumeType cannot be deleted with image_volume_cache enabled | 15:12 |
| enriquetaso | Summary: VolumeType cannot be deleted when "image_volume_cache_enabled = True" is set in "cinder.conf". | 15:12 |
| enriquetaso | A new bug brings back an older bug. New bug: | 15:12 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1953704 | 15:12 |
| enriquetaso | oops, people is still joining | 15:12 |
| enriquetaso | Older bug reported by Lee a while ago: | 15:13 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1823880 | 15:13 |
| enriquetaso | The tempest merged https://review.opendev.org/c/openstack/tempest/+/637527 but cinder hasn't a fix proposed yet. | 15:13 |
| enriquetaso | From Lee's comment: Reviewing the cinder code in this area shows that these cache volumes need to be manually deleted by an admin in order for them to then be considered evicted from the cache and no longer in use: | 15:13 |
| enriquetaso | https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L508-L511 | 15:13 |
| enriquetaso | Lee suggests that we should automatically remove these when the last volume associated with this cached volume is removed. | 15:13 |
| whoami-rajat | enriquetaso, i don't think so, it stores image inside it to optimize the operation, maybe at a later stage, we would like to create another volume from that image and expect the optimization | 15:14 |
| eharney | later analysis in the bug rejects that idea | 15:14 |
| enriquetaso | oh | 15:15 |
| enriquetaso | sorry about that | 15:15 |
| whoami-rajat | i do agree it's extra efforts to clear out the volume if we've removed the associated image in glance, then it serves no purpose, but how frequent a case like that occur? | 15:17 |
| enriquetaso | OK, so we need a patch for this ^ and the bug report is valid. | 15:18 |
| enriquetaso | I'll to add a note on 1953704 | 15:19 |
| whoami-rajat | i remember we had a discussion about APIs for operators to manage the cached images, that will be useful in this case | 15:20 |
| whoami-rajat | but that's a time taking effort | 15:20 |
| enriquetaso | the discussion was on 1823880? whoami-rajat | 15:20 |
| whoami-rajat | this looks similar, i think that was for a midcycle topic when we want to pin images in cache | 15:22 |
| whoami-rajat | maybe simondodsley proposed it | 15:22 |
| whoami-rajat | but i won't waste much time of this meeting on it | 15:23 |
| enriquetaso | I can look for that and add a reference on the new bug in case to have some more context in case anyone would like to work on that. Thanks whoami-rajat ! | 15:24 |
| enriquetaso | we don't have any other topic for today's meeting anyway :P | 15:24 |
| whoami-rajat | np, thanks enriquetaso | 15:24 |
| enriquetaso | #topic open discussion | 15:24 |
| enriquetaso | Do we have any bugs to discuss today? | 15:24 |
| whoami-rajat | https://etherpad.opendev.org/p/cinder-yoga-midcycles#L86 | 15:24 |
| whoami-rajat | this is the topic if anyone's interested | 15:25 |
| enriquetaso | \o/ | 15:25 |
| enriquetaso | cool, thanks | 15:25 |
| enriquetaso | OK, no meetings for 22 and 29 December :P | 15:26 |
| enriquetaso | only the bug report on the mailing list | 15:27 |
| opendevreview | Eric Harney proposed openstack/cinder master: Remove various unused code https://review.opendev.org/c/openstack/cinder/+/800703 | 15:27 |
| enriquetaso | happy end of the year! | 15:27 |
| enriquetaso | #endmeeting | 15:28 |
| opendevmeet | Meeting ended Wed Dec 15 15:28:02 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:28 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-12-15-15.11.html | 15:28 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-12-15-15.11.txt | 15:28 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-12-15-15.11.log.html | 15:28 |
| opendevreview | Eric Harney proposed openstack/cinder master: Drivers: remove unused code https://review.opendev.org/c/openstack/cinder/+/800704 | 15:28 |
| opendevreview | Rafael Weingartner proposed openstack/cinder master: Filter reserved image properties https://review.opendev.org/c/openstack/cinder/+/812685 | 15:34 |
| EmilienM | hello, could I get a review on https://review.opendev.org/c/openstack/cinder/+/821737 and https://review.opendev.org/c/openstack/cinder/+/821738 ? so we can deploy Cinder via devstack on ussuri & train | 15:36 |
| opendevreview | Danny Webb proposed openstack/cinder master: rbd backend qos implementation https://review.opendev.org/c/openstack/cinder/+/820027 | 15:36 |
| rosmaita | cinder-stable-cores: https://review.opendev.org/c/openstack/cinder/+/821737 and https://review.opendev.org/c/openstack/cinder/+/821738 are blocking deployment of Cinder via devstack on ussuri & train | 15:40 |
| rosmaita | e0ne eharney geguileo jungleboyj smcginnis whoami-rajat ^^ | 15:40 |
| e0ne | looking on it | 15:41 |
| jungleboyj | Looks ok to me. | 15:45 |
| *** dviroel|rover is now known as dviroel|rover|lunch | 15:47 | |
| e0ne | rosmaita: approved | 15:50 |
| rosmaita | e0ne: ty! | 15:50 |
| e0ne | np | 15:50 |
| jungleboyj | Excellent. | 15:50 |
| *** dviroel|rover|lunch is now known as dviroel|rover | 16:32 | |
| opendevreview | Rafael Weingartner proposed openstack/cinder master: Filter reserved image properties https://review.opendev.org/c/openstack/cinder/+/812685 | 18:06 |
| opendevreview | Merged openstack/cinder stable/ussuri: [stable-only] Pin virtualenv and setuptools https://review.opendev.org/c/openstack/cinder/+/821737 | 18:10 |
| opendevreview | Merged openstack/cinder stable/train: [stable-only] Pin virtualenv and setuptools https://review.opendev.org/c/openstack/cinder/+/821738 | 18:13 |
| opendevreview | Fábio Oliveira proposed openstack/cinder stable/xena: NetApp ONTAP: Fix sub-clone zapi call https://review.opendev.org/c/openstack/cinder/+/821890 | 18:25 |
| hemna | mep | 18:34 |
| lmcgann | Hello, I was wondering if cinder supported erasure encoded ceph pools? I can't seem to find a consistent answer on the subject | 18:54 |
| opendevreview | Fábio Oliveira proposed openstack/cinder stable/wallaby: NetApp ONTAP: Fix sub-clone zapi call https://review.opendev.org/c/openstack/cinder/+/821892 | 20:10 |
| opendevreview | Fábio Oliveira proposed openstack/cinder stable/victoria: NetApp ONTAP: Fix sub-clone zapi call https://review.opendev.org/c/openstack/cinder/+/821893 | 20:11 |
| opendevreview | Rafael Weingartner proposed openstack/cinder master: Filter reserved image properties https://review.opendev.org/c/openstack/cinder/+/812685 | 20:28 |
| opendevreview | Fábio Oliveira proposed openstack/cinder stable/victoria: NetApp ONTAP: Fix sub-clone zapi call https://review.opendev.org/c/openstack/cinder/+/821893 | 20:28 |
| opendevreview | Fábio Oliveira proposed openstack/cinder stable/ussuri: NetApp ONTAP: Fix sub-clone zapi call https://review.opendev.org/c/openstack/cinder/+/821894 | 20:31 |
| opendevreview | Fábio Oliveira proposed openstack/cinder stable/ussuri: NetApp ONTAP: Fix sub-clone zapi call https://review.opendev.org/c/openstack/cinder/+/821894 | 20:34 |
| *** dviroel|rover is now known as dviroel|out | 21:11 | |
| opendevreview | Merged openstack/cinder-specs master: Propose image-encryption spec for Yoga https://review.opendev.org/c/openstack/cinder-specs/+/818671 | 21:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!