Tuesday, 2022-06-07

opendevreviewArun KV proposed openstack/cinder master: Reintroduce DataCore driver  https://review.opendev.org/c/openstack/cinder/+/83699603:36
Zer0Bytehey03:37
Zer0Bytei have a little issue :(03:37
Zer0Bytei create a volume type and everything was working fine after adding a second backend03:37
Zer0Bytethe first backend doesn't have specified volume_backend_name  and now i can't set it because i have around 200 volumes03:38
Zer0Byteused with this volume type03:38
Zer0Byteis there a way to setup the volume_backend_name 03:39
Zer0Byteafter you have volumes03:39
Zer0Byteon it03:39
Zer0Byte?03:39
opendevreviewFrancesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/82648406:11
simondodsleyAnyone in Berlin?06:53
opendevreview王昊 proposed openstack/cinder master: Support multiple filters together for volume time comparison filter  https://review.opendev.org/c/openstack/cinder/+/74014607:41
opendevreview王昊 proposed openstack/cinder master: Support to query snapshot filter by updated_at/created_at  https://review.opendev.org/c/openstack/cinder/+/73306207:44
*** bhagyashris_ is now known as bhagyashris08:03
opendevreviewRajat Dhasmana proposed openstack/cinder master: Remove redundant line from host_manager.py  https://review.opendev.org/c/openstack/cinder/+/84492008:09
gokhanihi folks, how can I migrate volume-backed instance to new openstack environment. we can't detach root volume. is there any way for migrating bootable volume instead of delete vm?  08:16
opendevreviewArun KV proposed openstack/cinder master: Reintroduce DataCore driver  https://review.opendev.org/c/openstack/cinder/+/83699608:28
whoami-rajatgokhani, i think using --block-migrate with openstack live migrate command should work08:29
whoami-rajatgokhani, here's the doc on live migration https://docs.openstack.org/nova/pike/admin/live-migration-usage.html08:29
whoami-rajathttps://docs.openstack.org/nova/latest/admin/live-migration-usage.html -- latest08:29
whoami-rajatgeguileo, minor followup on one of your patch https://review.opendev.org/c/openstack/cinder/+/84492008:33
geguileoouch, I must have missed it in one of the merge conflict resolutions08:35
whoami-rajatyep, looks like it08:37
whoami-rajatbut doesn't change anything so we're good08:38
gokhaniwhoami-rajat, I mean not live migration, for example to different pike environment, openstack env1 and openstack env2. I want to snapshot of vm image but its root disk is on volume. 08:45
gokhaniwhen I try to get volume snapshot, it throws root device is not detached error. this is pike environment  08:47
opendevreviewMerged openstack/os-brick master: Avoid volume extension errors caused by multipath-tools version  https://review.opendev.org/c/openstack/os-brick/+/76277609:14
whoami-rajatgokhani, ack, i think it would be more useful to ask this in #openstack-nova as they might have strategies to support moving instances between different deployments10:35
gokhanithanks whoami-rajat I find solution with this link https://docs.fuga.cloud/how-to-migrate-a-volume-from-one-openstack-provider-to-another10:53
gokhanifirstly we need to get snapshot of bootable volume 10:54
whoami-rajatgokhani, i can see it requires a detach operation, if you just want to move volumes then you can unmanage the volume from one deployment and manage it in another but moving instances is a different thing11:00
sfernandhey folks am I the only cinder dev attending to the Berlin summit today!? :(11:00
whoami-rajatsfernand, maybe, none of the attendees of the mid cycle were in berlin ...11:00
sfernandOh I see, hope to see you at the ptg so11:03
opendevreviewMerged openstack/cinder master: db: Migrate "driver initiator data" APIs to enginefacade  https://review.opendev.org/c/openstack/cinder/+/83753412:02
yuval2Hey I am in the summit, anyone else is around?12:06
opendevreviewMerged openstack/cinder master: db: Migrate "image volume cache" APIs to enginefacade  https://review.opendev.org/c/openstack/cinder/+/83753512:07
opendevreviewMerged openstack/cinder master: db: Migrate "worker" APIs to enginefacade  https://review.opendev.org/c/openstack/cinder/+/83753612:07
opendevreviewStephen Finucane proposed openstack/cinder master: db: Remove unnecessary engine facade decorator  https://review.opendev.org/c/openstack/cinder/+/84496112:23
opendevreviewStephen Finucane proposed openstack/cinder master: db: Remove weird error handling code  https://review.opendev.org/c/openstack/cinder/+/84496212:23
opendevreviewWalt proposed openstack/cinder master: Add reporting of capacity factors in API  https://review.opendev.org/c/openstack/cinder/+/84460112:25
opendevreviewStephen Finucane proposed openstack/cinder master: db: Move comment  https://review.opendev.org/c/openstack/cinder/+/84496412:27
opendevreviewWalt proposed openstack/cinder master: Fix tracking allocated_capacity_gb  https://review.opendev.org/c/openstack/cinder/+/83608312:29
opendevreviewWalt proposed openstack/cinder master: Fix and unify capacity calculations  https://review.opendev.org/c/openstack/cinder/+/83124712:30
rosmaitayuval2: sfernand is also at the summit12:44
abishop_requirements-check ci job is failing for a couple of patches13:02
abishop_eharney: https://review.opendev.org/c/openstack/cinder/+/84487913:02
abishop_yours is one of them13:02
rosmaitaabishop_: probably due to this commit: https://opendev.org/openstack/requirements/commit/5fab5913b5225e5b96c424f7975bd76003662fd713:10
rosmaitai have an idea, hopefully will have a fix up soon13:10
opendevreviewBrian Rosmaita proposed openstack/cinder master: Don't limit use of importlib-metadata to Python < 3.8  https://review.opendev.org/c/openstack/cinder/+/84497413:28
rosmaitalet's see if ^^ fixes the requirements job13:28
abishop_dc_arunkv: you can try rebasing your patch on ^^13:31
opendevreviewBrian Rosmaita proposed openstack/cinderlib master: Don't limit use of importlib-metadata to Python < 3.8  https://review.opendev.org/c/openstack/cinderlib/+/84498114:08
rosmaitawhoami-rajat__: https://review.opendev.org/c/openstack/cinder/+/84497414:43
*** abishop_ is now known as abishop14:47
opendevreviewEric Harney proposed openstack/cinder master: Bump mypy version to 0.960  https://review.opendev.org/c/openstack/cinder/+/84487915:04
whoami-rajat__rosmaita, I still don't understand why the restriction of py<3.8 was introduced?15:08
rosmaitawhoami-rajat__: i think because in py3.8, the functionality is in stdlib, so you don't really need to use importlib_metadata any more15:09
rosmaitabut the reason to still use it in 3.8 and 3.9 is that there's some py3.10 features that are in importlib_metadata that's not in importlib.metadata15:10
whoami-rajat__rosmaita, ah ok, so since we've removed py36 support from Zed, shouldn't we drop using this lib altogether?15:10
rosmaitai think nova wants to use some of the py3.10 features that it makes available in 3.8 and 3.915:11
whoami-rajat__ack, sounds good then, thanks for the info15:11
whoami-rajat__rosmaita, do we have a precedent of requiring 2 +2s for requirement changes? I remember we don't need it in doc patches but not sure about this one15:12
rosmaitawhoami-rajat__: i'm not sure we have a policy, i think two +2s are better, but sometimes only one +2 for something that's holding up CI15:13
whoami-rajat__rosmaita, yeah this one is gate unblocker so will just approve it15:14
eharneyi just bumped my patch that was failing the check ^15:14
eharneyso i could probably review it15:14
whoami-rajat__oh, i approved...15:14
eharneywell i reviewed it anyway since i was sitting here looking at it15:16
whoami-rajat__cool, that helps, thanks15:17
rosmaitathanks!15:17
opendevreviewMerged openstack/cinder master: Don't limit use of importlib-metadata to Python < 3.8  https://review.opendev.org/c/openstack/cinder/+/84497415:35
*** dviroel|afk is now known as dviroel16:41
opendevreviewRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add test to create multiple volume from resource  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/79575317:09
*** dviroel is now known as dviroel|afk19:46
opendevreviewBrian Rosmaita proposed openstack/cinderlib master: Add requirements-check job  https://review.opendev.org/c/openstack/cinderlib/+/84502620:03
opendevreviewSofia Enriquez proposed openstack/cinder-tempest-plugin master: Update README.md with Tempest information  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/84505020:23
opendevreviewBrian Rosmaita proposed openstack/cinder master: WIP: Prospective policy matrix for Zed  https://review.opendev.org/c/openstack/cinder/+/83552521:14

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!