*** spatel has joined #openstack-cinder | 00:09 | |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: Add features for add backup id to volume metadata https://review.opendev.org/705876 | 00:16 |
---|---|---|
*** n-saito has joined #openstack-cinder | 00:21 | |
*** spatel has quit IRC | 00:33 | |
*** zhanglong has joined #openstack-cinder | 00:58 | |
*** dave-mccowan has quit IRC | 01:01 | |
*** ociuhandu has joined #openstack-cinder | 01:04 | |
*** ociuhandu has quit IRC | 01:08 | |
openstackgerrit | DingDong proposed openstack/cinder master: [Unity] Support consistency group replication https://review.opendev.org/691557 | 01:22 |
*** spatel has joined #openstack-cinder | 01:22 | |
rosmaita | jungleboyj: around? | 01:28 |
jungleboyj | rosmaita: kind of | 01:29 |
rosmaita | ok, if you have a few minutes, i think https://review.opendev.org/#/c/711362/8 is OK now | 01:30 |
rosmaita | patch passes Zuul and Nimble CI | 01:30 |
jungleboyj | Done. | 01:33 |
*** lvbin02 has joined #openstack-cinder | 01:36 | |
*** lvbin01 has quit IRC | 01:38 | |
*** lvbin02 is now known as lvbin01 | 01:38 | |
*** ajitha has joined #openstack-cinder | 01:38 | |
*** eharney has quit IRC | 01:46 | |
*** eharney has joined #openstack-cinder | 01:58 | |
*** yaawang has quit IRC | 02:08 | |
*** yaawang has joined #openstack-cinder | 02:08 | |
openstackgerrit | Brian Rosmaita proposed openstack/cinder master: Move release note to correct directory https://review.opendev.org/716436 | 02:15 |
openstackgerrit | Merged openstack/cinder master: Revert "Remove Nimble Storage Driver" https://review.opendev.org/711362 | 02:51 |
*** lvbin02 has joined #openstack-cinder | 03:17 | |
*** psachin has joined #openstack-cinder | 03:19 | |
*** lvbin01 has quit IRC | 03:20 | |
*** lvbin02 is now known as lvbin01 | 03:20 | |
*** spatel has quit IRC | 03:26 | |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: Add features for add backup id to volume metadata https://review.opendev.org/705876 | 03:41 |
*** ykarel|away is now known as ykarel | 04:10 | |
openstackgerrit | wuqiongdan proposed openstack/cinder master: Add sandstone iscsi driver. https://review.opendev.org/708673 | 04:12 |
*** udesale has joined #openstack-cinder | 05:12 | |
*** udesale has quit IRC | 05:14 | |
*** udesale has joined #openstack-cinder | 05:14 | |
*** ajitha has quit IRC | 05:18 | |
*** vishalmanchanda has joined #openstack-cinder | 05:22 | |
*** raghavendrat has joined #openstack-cinder | 05:50 | |
*** ociuhandu has joined #openstack-cinder | 05:52 | |
*** Liang__ has joined #openstack-cinder | 05:56 | |
*** ociuhandu has quit IRC | 05:57 | |
*** raghavendrat has quit IRC | 06:16 | |
openstackgerrit | Raunak Kumar proposed openstack/cinder master: Mark Nimble Storage Driver supported https://review.opendev.org/716495 | 06:38 |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: Add features for add backup id to volume metadata https://review.opendev.org/705876 | 06:44 |
*** irclogbot_1 has quit IRC | 06:49 | |
*** Reepicheep has quit IRC | 06:51 | |
*** spotz has quit IRC | 06:51 | |
*** Reepicheep has joined #openstack-cinder | 06:51 | |
*** irclogbot_0 has joined #openstack-cinder | 06:51 | |
*** raunak12 has quit IRC | 06:53 | |
*** irclogbot_0 has quit IRC | 06:53 | |
*** psachin has quit IRC | 06:53 | |
*** irclogbot_0 has joined #openstack-cinder | 06:53 | |
*** tonyb has quit IRC | 06:54 | |
*** ociuhandu has joined #openstack-cinder | 06:55 | |
*** Liang__ has quit IRC | 06:56 | |
*** yaawang has quit IRC | 06:56 | |
*** Liang__ has joined #openstack-cinder | 06:57 | |
*** yaawang has joined #openstack-cinder | 06:59 | |
*** tonyb has joined #openstack-cinder | 06:59 | |
*** ociuhandu has quit IRC | 07:00 | |
*** Liang__ is now known as LiangFang | 07:01 | |
*** ociuhandu has joined #openstack-cinder | 07:03 | |
*** ociuhandu has quit IRC | 07:08 | |
*** rcernin has quit IRC | 07:15 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/716515 | 07:19 |
*** raunak12 has joined #openstack-cinder | 07:24 | |
*** ociuhandu has joined #openstack-cinder | 07:28 | |
*** tosky has joined #openstack-cinder | 07:32 | |
*** e0ne has joined #openstack-cinder | 07:35 | |
*** raunak12 has quit IRC | 07:42 | |
*** ociuhandu has quit IRC | 08:08 | |
*** ociuhandu has joined #openstack-cinder | 08:09 | |
*** ociuhandu has quit IRC | 08:14 | |
*** ociuhandu has joined #openstack-cinder | 08:15 | |
*** tkajinam has quit IRC | 08:18 | |
*** lvbin02 has joined #openstack-cinder | 08:29 | |
*** ykarel is now known as ykarel|lunch | 08:30 | |
*** lvbin01 has quit IRC | 08:31 | |
*** lvbin02 is now known as lvbin01 | 08:31 | |
*** martinkennelly has joined #openstack-cinder | 08:42 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Support Glance image data colocation https://review.opendev.org/697636 | 08:58 |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: Add features for add backup id to volume metadata https://review.opendev.org/705876 | 09:06 |
*** zhanglong has quit IRC | 09:12 | |
*** martinkennelly has quit IRC | 09:37 | |
*** n-saito has quit IRC | 09:58 | |
*** ykarel|lunch is now known as ykarel | 09:59 | |
*** LiangFang has quit IRC | 10:00 | |
*** ociuhandu has quit IRC | 10:21 | |
*** ociuhandu has joined #openstack-cinder | 10:22 | |
openstackgerrit | Prabhuraj Kamaraj proposed openstack/cinder master: cgname need to be passed to '_create_consistencygroup' not group object. https://review.opendev.org/716543 | 10:24 |
openstackgerrit | Prabhuraj Kamaraj proposed openstack/cinder master: cgname need to be passed to '_create_consistencygroup' for group created from source group, not group object. https://review.opendev.org/716543 | 10:26 |
*** ociuhandu has quit IRC | 10:28 | |
openstackgerrit | Prabhuraj Kamaraj proposed openstack/cinder master: cgname need to be passed instead of group to '_create_consistencygroup'. https://review.opendev.org/716543 | 10:34 |
*** rcernin has joined #openstack-cinder | 10:34 | |
*** ociuhandu has joined #openstack-cinder | 10:44 | |
*** ociuhandu has quit IRC | 10:45 | |
*** ociuhandu has joined #openstack-cinder | 10:46 | |
*** ociuhandu has quit IRC | 10:59 | |
*** ociuhandu has joined #openstack-cinder | 11:00 | |
*** ruffian_sheep has joined #openstack-cinder | 11:05 | |
*** ociuhandu has quit IRC | 11:05 | |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder master: Add support for VxFlex OS 3.5 to VxFlex OS driver https://review.opendev.org/705176 | 11:31 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder master: Add OpenStack volume replication v2.1 support in VxFlex OS driver https://review.opendev.org/707390 | 11:31 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder master: Add support for volume migration in VxFlex OS driver https://review.opendev.org/711624 | 11:32 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder master: Add revert to snapshot support in VxFlex OS driver https://review.opendev.org/712622 | 11:33 |
*** ociuhandu has joined #openstack-cinder | 11:37 | |
*** ociuhandu has quit IRC | 11:47 | |
*** ociuhandu has joined #openstack-cinder | 11:48 | |
*** enriquetaso has joined #openstack-cinder | 11:48 | |
*** Xuchu_ has joined #openstack-cinder | 11:50 | |
*** Xuchu has quit IRC | 11:50 | |
*** Xuchu_ is now known as Xuchu | 11:50 | |
*** xuanyd has joined #openstack-cinder | 11:52 | |
*** zhanglong has joined #openstack-cinder | 11:52 | |
*** ociuhandu has quit IRC | 11:56 | |
*** zhanglong has quit IRC | 11:57 | |
*** zhanglong has joined #openstack-cinder | 11:58 | |
*** Xuchu has quit IRC | 11:59 | |
*** xuanyd has quit IRC | 11:59 | |
*** ociuhandu has joined #openstack-cinder | 12:01 | |
*** sapd1_x has quit IRC | 12:03 | |
*** spotz has joined #openstack-cinder | 12:07 | |
*** Xuchu has joined #openstack-cinder | 12:07 | |
*** martinkennelly has joined #openstack-cinder | 12:11 | |
*** ociuhandu has quit IRC | 12:37 | |
openstackgerrit | Simon O'Donovan proposed openstack/cinder master: PowerMax Driver - Update single underscores https://review.opendev.org/716572 | 12:37 |
openstackgerrit | Lucio Seki proposed openstack/cinder master: DNM Debug https://review.opendev.org/696626 | 12:39 |
*** Kvasscn has joined #openstack-cinder | 12:51 | |
*** sapd1_x has joined #openstack-cinder | 12:57 | |
Kvasscn | hello, does anyone familiar with cinderlib? I had a problem with it when did some testing | 12:58 |
Kvasscn | Traceback (most recent call last): | 12:59 |
Kvasscn | File "test.py", line 1, in <module> | 12:59 |
Kvasscn | import cinderlib as cl | 12:59 |
Kvasscn | File "/usr/lib/python2.7/site-packages/cinderlib/__init__.py", line 19, in <module> | 12:59 |
Kvasscn | from cinderlib import cinderlib | 12:59 |
Kvasscn | File "/usr/lib/python2.7/site-packages/cinderlib/cinderlib.py", line 28, in <module> | 12:59 |
Kvasscn | cinder_objects.register_all() # noqa | 12:59 |
Kvasscn | File "/tmp/cinder/objects/__init__.py", line 27, in register_all | 12:59 |
Kvasscn | __import__('cinder.objects.backup') | 12:59 |
Kvasscn | File "/tmp/cinder/objects/backup.py", line 34, in <module> | 12:59 |
Kvasscn | base.CinderObjectDictCompat, base.CinderComparableObject): | 12:59 |
Kvasscn | File "/usr/lib/python2.7/site-packages/oslo_versionedobjects/base.py", line 139, in register | 12:59 |
Kvasscn | registry._register_class(obj_cls) | 12:59 |
Kvasscn | File "/usr/lib/python2.7/site-packages/oslo_versionedobjects/base.py", line 134, in _register_class | 12:59 |
smcginnis | Please use paste.openstack.org | 12:59 |
smcginnis | geguileo: Look familiar? ^ | 12:59 |
Kvasscn | http://paste.openstack.org/show/791459/ | 13:04 |
Kvasscn | Paste to the hyperlink above, thanks | 13:04 |
Kvasscn | the version of cinder is openstack-cinder-12.0.7-1.el7.noarch | 13:06 |
Kvasscn | and cinderlib version is cinderlib-1.0.1 | 13:06 |
Kvasscn | The test program is very simple, just import cinderlib as cl | 13:08 |
*** udesale_ has joined #openstack-cinder | 13:09 | |
*** udesale has quit IRC | 13:11 | |
*** ruffian_sheep has quit IRC | 13:14 | |
*** ociuhandu has joined #openstack-cinder | 13:15 | |
Kvasscn | the code work fine with cinderlib 0.9.0, but failed when update to cinderlib-1.0.1 | 13:17 |
*** ociuhandu has quit IRC | 13:20 | |
*** zhanglong has quit IRC | 13:21 | |
openstackgerrit | Merged openstack/cinder stable/rocky: [Unity] Fix TypeError for test case test_delete_host_wo_lock https://review.opendev.org/716156 | 13:27 |
openstackgerrit | Merged openstack/cinder master: Blacklist stestr 2.3.0 https://review.opendev.org/711080 | 13:27 |
*** ociuhandu has joined #openstack-cinder | 13:35 | |
*** Kvasscn has quit IRC | 13:36 | |
geguileo | smcginnis: I think the problem is that they are running cinderlib with the wrong cinder version | 13:38 |
geguileo | I'll check it | 13:40 |
geguileo | I see the problem... I'm no pinning the cinder version in the requirements | 13:42 |
* geguileo facepalms | 13:42 | |
*** sapd1_x has quit IRC | 13:42 | |
*** ykarel is now known as ykarel|afk | 13:43 | |
*** Kvasscn has joined #openstack-cinder | 13:45 | |
Kvasscn | Some additional information was added to http://paste.openstack.org/show/791466 | 13:48 |
geguileo | Kvasscn: I think the problem is the cinder version you are using | 13:52 |
geguileo | Kvasscn: because cinder and cinderlib must be from the same openstack release | 13:52 |
Kvasscn | the version was too old? | 13:53 |
geguileo | Kvasscn: but I forgot to ping the Cinder version in the cinderlib requirements | 13:53 |
geguileo | Kvasscn: what Cinder release where you using? | 13:53 |
Kvasscn | queens | 13:53 |
geguileo | you are trying to use cinder queens with cinderlib from train | 13:53 |
Kvasscn | openstack-cinder-12.0.7-1.el7.noarch | 13:54 |
Kvasscn | python-cinder-12.0.7-1.el7.noarch | 13:54 |
geguileo | yup, queens | 13:54 |
geguileo | any chance you could bump the cinder version? | 13:54 |
geguileo | by bump, I mean update | 13:54 |
*** Liang__ has joined #openstack-cinder | 13:55 | |
*** sapd1_x has joined #openstack-cinder | 13:55 | |
Kvasscn | you mean latest queens version? | 13:55 |
geguileo | no, sorry upgrade to newer release | 13:56 |
*** Liang__ is now known as LiangFang | 13:56 | |
smcginnis | lvbin01: Please discuss in channel. | 13:57 |
smcginnis | job-output.txt.gz does not contain the tempest run output. | 13:58 |
smcginnis | lvbin01: Really large file, so not sure what's all in there. | 13:58 |
Kvasscn | do you mean the latest stein or train version? | 13:58 |
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 lseki sfernand rajinir ^^ | 13:58 |
geguileo | Kvasscn: I'm checking which one is the older version that works with cinderlib 1.0.1 | 13:59 |
jungleboyj | o/ Multi tasking today. :-( | 13:59 |
Kvasscn | thanks | 13:59 |
*** sfernand has joined #openstack-cinder | 14:00 | |
Kvasscn | when i commented out this line, # from cinderlib import _fake_packages , the code can work | 14:01 |
Kvasscn | I think it related with this issue: https://github.com/openstack/cinderlib/commit/76d0164fb009fbd54dd6f1100d55c15943ed63e5#diff-3b57fcb24ec474339c706c4b7bb08fe0 | 14:01 |
Kvasscn | but i think we need this patch | 14:01 |
*** ykarel|afk is now known as ykarel | 14:02 | |
geguileo | Kvasscn: Stein is the minimum version that allows it to start | 14:04 |
Kvasscn | ok, thanks for your confirm and help | 14:05 |
geguileo | Kvasscn: but that has another problem | 14:06 |
Kvasscn | :( | 14:06 |
geguileo | Kvasscn: because we added a non backward compatible change... | 14:06 |
geguileo | Kvasscn: I'm checking with Train now | 14:06 |
openstackgerrit | Merged openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/716515 | 14:07 |
geguileo | Kvasscn: OK, you can use 1.0.1 with Train, which is the right openstack release | 14:08 |
geguileo | Kvasscn: how are you installing cinder and cinderlib? | 14:08 |
Kvasscn | i install cinder by yum, and install cinder by pip | 14:09 |
geguileo | Kvasscn: which repository for cinder? | 14:09 |
geguileo | RDO, OSP? | 14:10 |
Kvasscn | sorry, i dont which this mean. | 14:11 |
Kvasscn | rdo or osp mean what ? ;( | 14:11 |
Kvasscn | sorry | 14:11 |
geguileo | Kvasscn: when you do "yum info openstack-cinder" what do you get? | 14:11 |
geguileo | in the "Repo:" field | 14:12 |
Kvasscn | [root@kataz other]# yum info openstack-cinder | 14:13 |
Kvasscn | Loaded plugins: fastestmirror, langpacks | 14:13 |
Kvasscn | Repodata is over 2 weeks old. Install yum-cron? Or run: yum makecache fast | 14:13 |
Kvasscn | Determining fastest mirrors | 14:13 |
Kvasscn | * base: mirrors.aliyun.com | 14:13 |
Kvasscn | * centos-qemu-ev: mirrors.aliyun.com | 14:13 |
Kvasscn | * extras: mirrors.aliyun.com | 14:13 |
Kvasscn | * updates: mirrors.aliyun.com | 14:13 |
Kvasscn | Installed Packages | 14:13 |
Kvasscn | Name : openstack-cinder | 14:13 |
Kvasscn | Arch : noarch | 14:13 |
Kvasscn | Epoch : 1 | 14:13 |
*** throne82_ has joined #openstack-cinder | 14:13 | |
geguileo | Kvasscn: better use pastebin or they are going to kill us both lol | 14:13 |
Kvasscn | ok | 14:14 |
geguileo | Kvasscn: did you get a row with the Repo information in that command? | 14:17 |
*** sapd1_x has quit IRC | 14:17 | |
*** Kvasscn has quit IRC | 14:23 | |
*** Kvasscn has joined #openstack-cinder | 14:23 | |
Kvasscn | http://paste.openstack.org/show/791470/ i have pasted the yum info | 14:24 |
openstackgerrit | Monty Taylor proposed openstack/cinder master: DNM Testing zuul job change https://review.opendev.org/716604 | 14:27 |
geguileo | Kvasscn: ok, the enabled repo is RDO's | 14:28 |
geguileo | Kvasscn: You should uninstall the cinder packages, disable the queens repository, enable the one from stein, and install from there | 14:28 |
openstackgerrit | Monty Taylor proposed openstack/cinder master: DNM Testing zuul job change https://review.opendev.org/716604 | 14:29 |
Kvasscn | ok | 14:29 |
geguileo | Kvasscn: to enable the stein repository: sudo yum install -y centos-release-openstack-stein | 14:29 |
Kvasscn | Is it possible to porting the patch to cinder 0.9.0 version? i mean this patch https://github.com/openstack/cinderlib/commit/76d0164fb009fbd54dd6f1100d55c15943ed63e5 | 14:30 |
*** sapd1_x has joined #openstack-cinder | 14:30 | |
geguileo | Kvasscn: that patch wouldn't help, because the problem is not due to a missing requirement :-( | 14:30 |
Kvasscn | Thanks for the detailed instructions ;) | 14:31 |
geguileo | Kvasscn: np, let me know how it goes :-) | 14:31 |
Kvasscn | my initial purpose is to achieve faster load times through this patch | 14:33 |
Kvasscn | so the latest cinderlib version is not necessary | 14:34 |
geguileo | Kvasscn: oh, yeah, it would load a bit faster, but probably the slowest part is the DB migration code that is run on every start... | 14:35 |
Kvasscn | ok | 14:37 |
geguileo | Kvasscn: how are you using cinderlib, as a command line tool? | 14:38 |
Kvasscn | yes | 14:38 |
geguileo | and are you using a DB to store the metadata? | 14:38 |
Kvasscn | no, i use it just to create/delete volume, and add acl to san target | 14:40 |
*** martinkennelly has quit IRC | 14:40 | |
geguileo | Kvasscn: but how are you initializing cinderlib's persistence? | 14:40 |
geguileo | Kvasscn: because by default everything is just stored in memory, and you lose all references to the volumes when you stop the program | 14:41 |
*** sapd1_x has quit IRC | 14:41 | |
Kvasscn | My colleague do some modify to the cinderlib code. | 14:41 |
geguileo | Kvasscn: what features did they need that were not there? | 14:42 |
geguileo | Kvasscn: here you can see an example of using a sqlite file to store the volume information: https://docs.openstack.org/cinderlib/latest/topics/metadata.html#database-plugin | 14:43 |
Kvasscn | we store the related info in a standalone standalone db | 14:43 |
geguileo | Kvasscn: ah, ok | 14:43 |
geguileo | Kvasscn: but yes, cinderlib does not load all that fast :-( | 14:44 |
Kvasscn | we get some voume infomation directly from san system, not use cinder persistence db. | 14:45 |
*** martinkennelly has joined #openstack-cinder | 14:45 | |
geguileo | Kvasscn: ok, that's not something I can add to cinderlib :-( | 14:45 |
Kvasscn | or it mean cinderlib itself does not store any configuration. | 14:45 |
*** sapd1_x has joined #openstack-cinder | 14:46 | |
Kvasscn | In our use case. | 14:46 |
Kvasscn | Thanks for your feedback and help. I'm going offline | 14:47 |
Kvasscn | Thank you very much, :) | 14:48 |
geguileo | Kvasscn: ok, np | 14:48 |
geguileo | Kvasscn: if you have cinderlib issues you can ping me :-) | 14:48 |
Kvasscn | ok :) | 14:48 |
*** Kvasscn has quit IRC | 14:51 | |
enriquetaso | rosmaita, one question about the cinder bug: if the incremental backup must be in the same project as the full backup that means admin + owner permissions? | 14:54 |
openstackgerrit | Gorka Eguileor proposed openstack/cinderlib master: Pin cinder version https://review.opendev.org/716617 | 14:56 |
*** lpetrut has joined #openstack-cinder | 14:59 | |
eharney | what are "inline comments" for pylint? | 15:00 |
rosmaita | eharney: did you look at rajat's patch? | 15:00 |
rosmaita | looks like the new "helpful" way for pylint to report | 15:01 |
rosmaita | take a look at this: https://review.opendev.org/#/c/697636/10/cinder/tests/unit/volume/drivers/vmware/test_vmware_vmdk.py | 15:01 |
openstackgerrit | Monty Taylor proposed openstack/cinder master: DNM Testing zuul job change https://review.opendev.org/716604 | 15:01 |
eharney | ohh | 15:02 |
rosmaita | yeah, remarkably ugly | 15:02 |
rosmaita | enriquetaso: gimme a minute to think about your question | 15:02 |
*** lpetrut has quit IRC | 15:06 | |
rosmaita | enriquetaso: i think the idea is that since a volume exists in a particular project, the stuff related to that volume (backups & snapshots) should exist in that same project | 15:06 |
eharney | i really don't think i buy the idea that backups are the same as snapshots in that way | 15:07 |
rosmaita | eharney: i am thinking they are more the same than snapshots | 15:07 |
rosmaita | actually, that doesn't make any sense | 15:08 |
rosmaita | :) | 15:08 |
enriquetaso | rosmaita, ok.. so if admin and demo are in diff project the scenario of https://bugs.launchpad.net/cinder/+bug/1869746 isn't valid | 15:08 |
openstack | Launchpad bug 1869746 in Cinder "Cinder throws error creating incremental backup from parent in another project" [Undecided,New] | 15:08 |
rosmaita | enriquetaso: well, you need to think of the admin as making the call on behalf of the project user | 15:09 |
smcginnis | We just need to pick - do we have separate backup trees and allow admin's to keep their own backups of tenant's volumes? Or is a backup initiated by an admin on behalf of the tenant. We just need to pick which use case we want to support. | 15:09 |
rosmaita | so when the admin is making the call, we should consider the admin an honorary member of that project | 15:09 |
smcginnis | Because really it's someone in the project or the admin. I don't think there would be a different way to get two different projects taking a backup of the same volume otherwise. | 15:09 |
rosmaita | smcginnis: the only use case i see for the admin making their own backup would be for forensics | 15:10 |
rosmaita | in that case, i think admin just clones the volume | 15:10 |
rosmaita | and then makes own backup | 15:10 |
rosmaita | or not | 15:10 |
eharney | volume transfer after backup results in different ownership | 15:10 |
smcginnis | Yeah. Or maybe as an extra value add service to cover their own butt in case of a data corruption due to an infrastructure issue. | 15:10 |
smcginnis | rosmaita: Cloning would make more sense to me for that scenario. | 15:11 |
smcginnis | Yeah, volume transfer needs to be considered too. If it really is something tied to a volume, then in that case I think it needs to be treated like snapshots and transfered with the volume. | 15:11 |
enriquetaso | ok, so admin is the admin and has superpowers as usual | 15:11 |
smcginnis | That's getting complicated though. | 15:11 |
eharney | but backups shouldn't be tied directly to volumes... they need to be separate to actually be useful backups | 15:12 |
smcginnis | And if we don't transfer a backup and someone tries to restore to the original, we need to make sure that is blocked. | 15:12 |
eharney | why? whoever owns the backup can restore the backup | 15:12 |
eharney | oh, never mind | 15:13 |
rosmaita | eharney: i may be thinking too much like a glance person here, but if there are backups of volume V, do not allow transfer of volume V. You could always clone it and transfer the clone | 15:13 |
eharney | to the original, yeah | 15:13 |
eharney | rosmaita: i don't think that's a reasonable restriction... backups are supposed to be off somewhere else unrelated to the volume after they're created | 15:14 |
eharney | if you want backups that are tied to the volume... use snapshots | 15:14 |
*** sapd1_x has quit IRC | 15:15 | |
smcginnis | This reminds me, we really need what someone had requested awhile back that I said I would look into implementing. Unmanage backup. | 15:15 |
smcginnis | That's what really addresses eharney's point. | 15:15 |
smcginnis | A backup moved off somewhere else should be unmanaged and no longer tracked by cinder. | 15:16 |
eharney | i thought we had that | 15:16 |
smcginnis | Then whoever has that backup, they can import it and restore to whatever volume they want. | 15:16 |
smcginnis | I don't think so. | 15:16 |
openstackgerrit | Michael McAleer proposed openstack/cinder master: PowerMax Driver - SRDF Replication Fixes https://review.opendev.org/714723 | 15:16 |
smcginnis | And I don't think it makes sense for cinder to track a backup that is supposed to be separated from a volume. Because cinder then is enforcing an artificial coupling in our database for something we don't want tightly coupled. | 15:17 |
rosmaita | let's continue this discussion once ganso puts together that etherpad | 15:17 |
eharney | ah, we have backup import/export, but it's not exactly the same thing as unmanage would be | 15:17 |
*** martinkennelly has quit IRC | 15:18 | |
smcginnis | Yeah. | 15:19 |
enriquetaso | thanks! | 15:20 |
smcginnis | When I looked into it, I was going to extend export to have something like a "--detach" argument that would export the backup metadata that needs to be saved, but then also just drop entries from the DB. | 15:20 |
rosmaita | enriquetaso: did i answer your question? | 15:20 |
smcginnis | Then import would theoretically be the same. | 15:20 |
smcginnis | Someone actually requested that to tie into their backup software. They wanted their backup solution to manage the backup lifecycle, not cinder. | 15:20 |
smcginnis | Just use cinder to get a good integrated way to get at the backup data. | 15:21 |
enriquetaso | rosmaita: yes, but I would like to read that etherpad to think a fix or a client fix | 15:21 |
enriquetaso | anyway I'm going to try to reproduce the bug in devstack | 15:21 |
rosmaita | enriquetaso: sounds good | 15:21 |
*** ykarel is now known as ykarel|away | 15:22 | |
*** LiangFang has quit IRC | 15:23 | |
e0ne | I'm busy with horizon meeting now, but if you need any help with backups - feel free to ping me here | 15:26 |
*** sapd1_x has joined #openstack-cinder | 15:28 | |
openstackgerrit | Yong Huang proposed openstack/cinder stable/queens: [Unity] Fix TypeError for test case test_delete_host_wo_lock https://review.opendev.org/716629 | 15:30 |
*** pcaruana has quit IRC | 15:32 | |
openstackgerrit | Simon O'Donovan proposed openstack/cinder master: PowerMax Driver - Limit replication devices https://review.opendev.org/716631 | 15:35 |
*** eharney has left #openstack-cinder | 15:35 | |
*** eharney has joined #openstack-cinder | 15:36 | |
openstackgerrit | Helen Walsh proposed openstack/cinder stable/stein: PowerMax Driver - Legacy volume not found https://review.opendev.org/715004 | 15:42 |
*** pcaruana has joined #openstack-cinder | 15:48 | |
openstackgerrit | Michael McAleer proposed openstack/cinder master: PowerMax Driver - Compression Change Bug Fix https://review.opendev.org/715965 | 16:07 |
openstackgerrit | Gorka Eguileor proposed openstack/cinderlib master: Pin cinder version https://review.opendev.org/716617 | 16:22 |
openstackgerrit | Gorka Eguileor proposed openstack/cinderlib master: Switch from mock to unittest.mock https://review.opendev.org/716652 | 16:22 |
*** raunak12 has joined #openstack-cinder | 16:23 | |
*** sfernand has quit IRC | 16:29 | |
openstackgerrit | Raunak Kumar proposed openstack/cinder master: Mark Nimble Storage Driver supported https://review.opendev.org/716495 | 16:37 |
*** rishabhhpe has joined #openstack-cinder | 16:38 | |
*** martinkennelly has joined #openstack-cinder | 16:43 | |
openstackgerrit | Brian Rosmaita proposed openstack/os-brick master: Add release note for ussuri cycle release https://review.opendev.org/716662 | 16:44 |
rosmaita | smcginnis: jungleboyj: lmk what you think of ^^ (i thought it would look weird with no release notes) | 16:45 |
smcginnis | rosmaita: We didn't have anything merge with release notes? | 16:45 |
*** rishabhhpe has quit IRC | 16:45 | |
rosmaita | no, believe it or not | 16:45 |
rosmaita | not since 3.0.0 anyway | 16:46 |
smcginnis | Yeah, I think having those couple mentions is a good idea. | 16:46 |
rosmaita | ok, cool | 16:46 |
openstackgerrit | Merged openstack/cinder master: Move release note to correct directory https://review.opendev.org/716436 | 17:12 |
*** udesale_ has quit IRC | 17:29 | |
*** ociuhandu has quit IRC | 17:45 | |
*** ociuhandu has joined #openstack-cinder | 17:49 | |
openstackgerrit | Brian Rosmaita proposed openstack/cinder stable/train: Move release note to correct directory https://review.opendev.org/716690 | 17:50 |
*** ociuhandu has quit IRC | 17:56 | |
*** gmann is now known as gmann_lunch | 17:56 | |
jungleboyj | rosmaita: ++ | 17:58 |
rosmaita | jungleboyj: ty, that was just what i was going to ask you! | 18:00 |
openstackgerrit | Merged openstack/os-brick master: Add release note for ussuri cycle release https://review.opendev.org/716662 | 18:05 |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Add hacking check for 3rd party mock https://review.opendev.org/716380 | 18:19 |
smcginnis | rosmaita: One snuck in that needed to be fixed. ^ | 18:20 |
rosmaita | smcginnis: ok | 18:21 |
rosmaita | smcginnis: thanks for catching that | 18:22 |
*** ociuhandu has joined #openstack-cinder | 18:25 | |
*** gmann_lunch is now known as gmann | 18:28 | |
rosmaita | smcginnis: should the brick release be 3.0.1 or 3.1.0 ? | 18:28 |
*** ociuhandu has quit IRC | 18:28 | |
*** ociuhandu has joined #openstack-cinder | 18:29 | |
smcginnis | Let me take a look at what's all in there. | 18:29 |
rosmaita | thanks | 18:29 |
smcginnis | rosmaita: There's a requirements change, but that's only for testing. So we could go with either one. I think 3.0.1 is probably fine. | 18:31 |
rosmaita | works for me! | 18:31 |
*** e0ne has quit IRC | 18:38 | |
*** e0ne has joined #openstack-cinder | 18:38 | |
*** e0ne has quit IRC | 18:39 | |
*** sapd1_x has quit IRC | 18:43 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Add hacking check for 3rd party mock https://review.opendev.org/716380 | 18:51 |
smcginnis | rosmaita: Found another pattern that needed to be covered. Also addresses one of the py38 failures. | 18:51 |
rosmaita | nice! | 18:51 |
openstackgerrit | Brian Rosmaita proposed openstack/cinder master: Remove HPE Lefthand Driver https://review.opendev.org/716413 | 18:51 |
*** enriquetaso has quit IRC | 19:11 | |
*** Lucas_Gray has joined #openstack-cinder | 19:20 | |
*** Lucas_Gray has quit IRC | 19:21 | |
*** Lucas_Gray has joined #openstack-cinder | 19:22 | |
*** throne82_ has quit IRC | 19:22 | |
*** Lucas_Gray has quit IRC | 19:26 | |
*** Lucas_Gray has joined #openstack-cinder | 19:32 | |
*** Lucas_Gray has quit IRC | 19:32 | |
*** ociuhandu has quit IRC | 19:39 | |
*** ociuhandu has joined #openstack-cinder | 19:40 | |
*** ociuhandu has quit IRC | 19:45 | |
*** ociuhandu has joined #openstack-cinder | 19:57 | |
*** lvbin02 has joined #openstack-cinder | 20:02 | |
*** lvbin01 has quit IRC | 20:05 | |
*** lvbin02 is now known as lvbin01 | 20:05 | |
*** e0ne has joined #openstack-cinder | 20:11 | |
*** vishalmanchanda has quit IRC | 20:39 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Update HACKING document to match current checks https://review.opendev.org/716774 | 20:49 |
*** jv has quit IRC | 20:51 | |
*** Lucas_Gray has joined #openstack-cinder | 21:01 | |
*** jv has joined #openstack-cinder | 21:05 | |
*** ociuhandu has quit IRC | 21:16 | |
*** ociuhandu has joined #openstack-cinder | 21:17 | |
*** ociuhandu has quit IRC | 21:22 | |
*** jv has quit IRC | 21:30 | |
*** jv has joined #openstack-cinder | 21:30 | |
*** enriquetaso has joined #openstack-cinder | 21:56 | |
*** Lucas_Gray has quit IRC | 22:03 | |
*** martinkennelly has quit IRC | 22:04 | |
*** Lucas_Gray has joined #openstack-cinder | 22:06 | |
*** melwitt has quit IRC | 22:08 | |
*** aspiers has quit IRC | 22:08 | |
*** rmk has quit IRC | 22:08 | |
*** fnordahl has quit IRC | 22:08 | |
*** Lucas_Gray has quit IRC | 22:10 | |
*** Lucas_Gray has joined #openstack-cinder | 22:12 | |
*** Lucas_Gray has quit IRC | 22:12 | |
*** Lucas_Gray has joined #openstack-cinder | 22:13 | |
*** spatel has joined #openstack-cinder | 22:19 | |
*** e0ne has quit IRC | 22:20 | |
openstackgerrit | Brian Rosmaita proposed openstack/cinder master: RBD: add support for revert-to-snapshot https://review.opendev.org/710334 | 22:20 |
*** e0ne has joined #openstack-cinder | 22:21 | |
*** rmk has joined #openstack-cinder | 22:30 | |
openstackgerrit | Merged openstack/cinder master: Remove HPE Lefthand Driver https://review.opendev.org/716413 | 22:36 |
*** aspiers has joined #openstack-cinder | 22:37 | |
*** spatel has quit IRC | 22:44 | |
*** melwitt has joined #openstack-cinder | 22:51 | |
*** tkajinam has joined #openstack-cinder | 22:55 | |
*** tosky has quit IRC | 23:03 | |
*** enriquetaso has quit IRC | 23:11 | |
*** e0ne has quit IRC | 23:13 | |
*** Lucas_Gray has quit IRC | 23:13 | |
hemna | anyone else messing around with multipass on ubuntu for vms? | 23:18 |
*** enriquetaso has joined #openstack-cinder | 23:20 | |
*** Kvasscn has joined #openstack-cinder | 23:21 | |
*** Kvasscn has quit IRC | 23:25 | |
*** n-saito has joined #openstack-cinder | 23:30 | |
*** enriquetaso has quit IRC | 23:41 | |
*** Xuchu has quit IRC | 23:50 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!