*** itlinux has joined #openstack-cinder | 00:10 | |
*** AlexeyAbashkin has joined #openstack-cinder | 00:10 | |
*** catintheroof has joined #openstack-cinder | 00:11 | |
openstackgerrit | Merged openstack/cinder master: Fix scheduler_host_manager limitation https://review.openstack.org/479775 | 00:12 |
---|---|---|
*** AlexeyAbashkin has quit IRC | 00:14 | |
*** abhitechie has quit IRC | 00:16 | |
*** abhitechie has joined #openstack-cinder | 00:16 | |
openstackgerrit | Merged openstack/cinder master: Completely remove mox from Cinder unittest https://review.openstack.org/514640 | 00:19 |
openstackgerrit | Merged openstack/cinder master: Remove doc/build before running docs job https://review.openstack.org/512507 | 00:19 |
openstackgerrit | Merged openstack/cinder master: Support count info in List&Detail APIs https://review.openstack.org/505101 | 00:19 |
openstackgerrit | Merged openstack/cinder master: Don't fail when deleting missing backup https://review.openstack.org/515026 | 00:19 |
*** abhitechie has quit IRC | 00:20 | |
*** rcernin has joined #openstack-cinder | 00:21 | |
*** abhitechie has joined #openstack-cinder | 00:21 | |
*** gkadam has joined #openstack-cinder | 00:23 | |
*** gkadam_ has quit IRC | 00:25 | |
*** abhinavtechie has joined #openstack-cinder | 00:26 | |
*** abhitechie has quit IRC | 00:26 | |
*** aagrawal has joined #openstack-cinder | 00:28 | |
*** aagrawal has quit IRC | 00:30 | |
openstackgerrit | Merged openstack/python-cinderclient master: Add .stestr.conf configuration https://review.openstack.org/505422 | 00:30 |
*** abhitechie has joined #openstack-cinder | 00:31 | |
*** abhinavtechie has quit IRC | 00:31 | |
*** catintheroof has quit IRC | 00:32 | |
*** chhavi has joined #openstack-cinder | 00:34 | |
*** chhavi has quit IRC | 00:38 | |
*** catintheroof has joined #openstack-cinder | 00:38 | |
*** wanghao has joined #openstack-cinder | 00:41 | |
*** catintheroof has quit IRC | 00:42 | |
*** Apoorva_ has joined #openstack-cinder | 00:43 | |
*** catintheroof has joined #openstack-cinder | 00:44 | |
openstackgerrit | Merged openstack/os-brick master: Remove legacy connector constants https://review.openstack.org/509502 | 00:46 |
*** AlexeyAbashkin has joined #openstack-cinder | 00:46 | |
*** Apoorva has quit IRC | 00:47 | |
*** Apoorva_ has quit IRC | 00:47 | |
*** gouthamr has joined #openstack-cinder | 00:49 | |
*** AlexeyAbashkin has quit IRC | 00:51 | |
*** catintheroof has quit IRC | 00:55 | |
*** catintheroof has joined #openstack-cinder | 00:56 | |
*** zzzeek has quit IRC | 00:58 | |
*** zzzeek has joined #openstack-cinder | 01:00 | |
*** gkadam has quit IRC | 01:07 | |
*** mriedem has joined #openstack-cinder | 01:15 | |
*** mriedem_away has quit IRC | 01:15 | |
*** catintheroof has quit IRC | 01:20 | |
*** namnh has joined #openstack-cinder | 01:21 | |
*** catintheroof has joined #openstack-cinder | 01:21 | |
*** fanzhang has quit IRC | 01:24 | |
*** fanzhang_ has joined #openstack-cinder | 01:24 | |
*** catintheroof has quit IRC | 01:24 | |
*** dave-mccowan has quit IRC | 01:26 | |
*** dave-mccowan has joined #openstack-cinder | 01:28 | |
*** wes_dillingham has quit IRC | 01:34 | |
*** psachin has joined #openstack-cinder | 01:35 | |
*** rcernin has quit IRC | 01:43 | |
*** rcernin has joined #openstack-cinder | 01:44 | |
*** dhinesh has quit IRC | 01:44 | |
*** fanzhang_ is now known as fanzhang | 01:44 | |
*** wes_dillingham has joined #openstack-cinder | 01:48 | |
openstackgerrit | zhuangzejian proposed openstack/cinder master: Update setup instructions https://review.openstack.org/515564 | 01:49 |
*** catintheroof has joined #openstack-cinder | 01:55 | |
*** salv-orlando has joined #openstack-cinder | 01:55 | |
*** salv-orlando has quit IRC | 02:00 | |
*** catintheroof has quit IRC | 02:00 | |
*** dalgaaf has quit IRC | 02:07 | |
*** dalgaaf has joined #openstack-cinder | 02:08 | |
*** stakeda has joined #openstack-cinder | 02:09 | |
*** zhaochao has quit IRC | 02:15 | |
*** zhaochao has joined #openstack-cinder | 02:16 | |
*** gcb has joined #openstack-cinder | 02:16 | |
*** thurloat has joined #openstack-cinder | 02:22 | |
*** catintheroof has joined #openstack-cinder | 02:28 | |
*** catintheroof has quit IRC | 02:33 | |
*** lhx__ has joined #openstack-cinder | 02:34 | |
*** AlexeyAbashkin has joined #openstack-cinder | 02:47 | |
*** junboli has joined #openstack-cinder | 02:47 | |
openstackgerrit | Merged openstack/cinder master: Dell EMC: Update PS and SC CI wiki names https://review.openstack.org/515159 | 02:49 |
*** AlexeyAbashkin has quit IRC | 02:52 | |
openstackgerrit | yixuan zhang proposed openstack/cinder master: Fix migate_volume API with replication_status ‘not-capable’ https://review.openstack.org/514988 | 02:56 |
*** armaan has joined #openstack-cinder | 02:59 | |
*** AlexeyAbashkin has joined #openstack-cinder | 03:08 | |
*** AlexeyAbashkin has quit IRC | 03:12 | |
*** dave-mccowan has quit IRC | 03:14 | |
*** markstur_ has quit IRC | 03:17 | |
*** thurloat has quit IRC | 03:21 | |
*** markstur has joined #openstack-cinder | 03:21 | |
openstackgerrit | TommyLike proposed openstack/cinder master: Fix earlier backup records can't be restored https://review.openstack.org/515577 | 03:28 |
*** abhitechie has quit IRC | 03:29 | |
*** nicolasbock has quit IRC | 03:30 | |
*** markvoelker has quit IRC | 03:43 | |
openstackgerrit | Merged openstack/cinder master: FlashSystem: Add CMMVC6045E CLI error for multi-host mapping https://review.openstack.org/511645 | 03:54 |
*** salv-orlando has joined #openstack-cinder | 03:56 | |
*** salv-orlando has quit IRC | 04:00 | |
*** catintheroof has joined #openstack-cinder | 04:01 | |
*** psachin has quit IRC | 04:03 | |
*** catintheroof has quit IRC | 04:05 | |
*** yangyape_ has joined #openstack-cinder | 04:09 | |
*** chhavi has joined #openstack-cinder | 04:11 | |
*** yangyapeng has quit IRC | 04:11 | |
*** mriedem has quit IRC | 04:15 | |
*** bhagyashri_s has joined #openstack-cinder | 04:15 | |
*** pooja-jadhav has joined #openstack-cinder | 04:15 | |
*** Apoorva has joined #openstack-cinder | 04:16 | |
*** pooja_jadhav has quit IRC | 04:17 | |
*** bhagyashris has quit IRC | 04:17 | |
openstackgerrit | Merged openstack/cinder master: Dell EMC PS: Fix Duplicate ACL records Issue https://review.openstack.org/515107 | 04:19 |
*** gkadam has joined #openstack-cinder | 04:19 | |
*** psachin has joined #openstack-cinder | 04:20 | |
*** abhi89 has joined #openstack-cinder | 04:21 | |
*** bhagyashri_s is now known as bhagyashris | 04:24 | |
*** abhitechie has joined #openstack-cinder | 04:31 | |
*** abhinavtechie has joined #openstack-cinder | 04:32 | |
*** abhitechie has quit IRC | 04:32 | |
*** logan- has quit IRC | 04:38 | |
*** logan- has joined #openstack-cinder | 04:42 | |
*** bhagyashris has quit IRC | 04:46 | |
*** Apoorva has quit IRC | 04:46 | |
*** rmk has quit IRC | 04:50 | |
*** cburgess has quit IRC | 04:51 | |
*** wes_dillingham has quit IRC | 05:00 | |
*** cburgess has joined #openstack-cinder | 05:03 | |
*** rmk has joined #openstack-cinder | 05:05 | |
*** rmk has quit IRC | 05:05 | |
*** rmk has joined #openstack-cinder | 05:05 | |
*** markstur has quit IRC | 05:10 | |
openstackgerrit | Merged openstack/cinder master: Make service object UUID not nullable https://review.openstack.org/514853 | 05:11 |
*** psachin has quit IRC | 05:14 | |
*** masuberu has joined #openstack-cinder | 05:15 | |
*** psachin has joined #openstack-cinder | 05:15 | |
*** lpetrut has joined #openstack-cinder | 05:16 | |
*** masber has quit IRC | 05:18 | |
*** masber has joined #openstack-cinder | 05:20 | |
*** masuberu has quit IRC | 05:23 | |
*** markstur has joined #openstack-cinder | 05:24 | |
*** salv-orlando has joined #openstack-cinder | 05:25 | |
*** newmember has joined #openstack-cinder | 05:31 | |
*** wanghao_ has joined #openstack-cinder | 05:33 | |
*** junboli has quit IRC | 05:34 | |
*** wanghao has quit IRC | 05:35 | |
*** salv-orlando has quit IRC | 05:38 | |
*** salv-orlando has joined #openstack-cinder | 05:38 | |
*** markvoelker has joined #openstack-cinder | 05:43 | |
*** junboli has joined #openstack-cinder | 05:55 | |
openstackgerrit | TommyLike proposed openstack/cinder master: Fix earlier backup records can't be restored https://review.openstack.org/515577 | 06:00 |
*** ntpttr_laptop___ has joined #openstack-cinder | 06:01 | |
*** ntpttr_laptop__ has quit IRC | 06:01 | |
*** zengyingzhe has joined #openstack-cinder | 06:03 | |
*** ntpttr_laptop___ has quit IRC | 06:05 | |
*** lpetrut has quit IRC | 06:17 | |
*** markvoelker has quit IRC | 06:18 | |
*** gouthamr has quit IRC | 06:21 | |
*** spectr has joined #openstack-cinder | 06:22 | |
*** bhagyashris has joined #openstack-cinder | 06:26 | |
*** markstur has quit IRC | 06:27 | |
*** lpetrut has joined #openstack-cinder | 06:27 | |
*** lpetrut has quit IRC | 06:33 | |
*** lpetrut has joined #openstack-cinder | 06:34 | |
*** lpetrut has quit IRC | 06:44 | |
*** pgadiya has joined #openstack-cinder | 06:48 | |
*** armaan has quit IRC | 07:04 | |
*** armaan has joined #openstack-cinder | 07:05 | |
*** markvoelker has joined #openstack-cinder | 07:14 | |
*** armaan has quit IRC | 07:15 | |
*** armaan has joined #openstack-cinder | 07:15 | |
*** fanzhang has left #openstack-cinder | 07:16 | |
*** abhinavtechie has quit IRC | 07:17 | |
*** armax has quit IRC | 07:21 | |
*** tesseract has joined #openstack-cinder | 07:22 | |
*** armax has joined #openstack-cinder | 07:22 | |
*** armax has quit IRC | 07:22 | |
*** armax has joined #openstack-cinder | 07:23 | |
*** armax has quit IRC | 07:23 | |
*** armax has joined #openstack-cinder | 07:24 | |
*** armax has quit IRC | 07:24 | |
*** armax has joined #openstack-cinder | 07:24 | |
*** armax has joined #openstack-cinder | 07:25 | |
*** armax has quit IRC | 07:25 | |
*** armax has joined #openstack-cinder | 07:26 | |
*** armax has quit IRC | 07:26 | |
*** armax has joined #openstack-cinder | 07:27 | |
*** armax has quit IRC | 07:27 | |
*** salv-orlando has quit IRC | 07:30 | |
*** salv-orlando has joined #openstack-cinder | 07:31 | |
*** abhitechie has joined #openstack-cinder | 07:33 | |
openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Add async replication support https://review.openstack.org/509222 | 07:34 |
*** namnh has quit IRC | 07:34 | |
*** salv-orlando has quit IRC | 07:35 | |
openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Enable CHAP authentication for vmax backend https://review.openstack.org/510614 | 07:36 |
openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Cannot extend volumes in a replication relationship https://review.openstack.org/511812 | 07:38 |
*** abhishek has joined #openstack-cinder | 07:39 | |
*** josecastroleon has joined #openstack-cinder | 07:41 | |
*** abhi89 has quit IRC | 07:41 | |
*** itlinux has quit IRC | 07:43 | |
*** e0ne has joined #openstack-cinder | 07:45 | |
*** markvoelker has quit IRC | 07:48 | |
*** amoralej|off is now known as amoralej | 07:51 | |
*** salv-orlando has joined #openstack-cinder | 07:52 | |
*** yikun has joined #openstack-cinder | 07:55 | |
*** AlexeyAbashkin has joined #openstack-cinder | 07:56 | |
*** rcernin has quit IRC | 07:58 | |
*** gcb has quit IRC | 08:00 | |
openstackgerrit | yixuan zhang proposed openstack/cinder master: Storwize: add backup snapshots support https://review.openstack.org/512562 | 08:03 |
openstackgerrit | zhuangzejian proposed openstack/cinder master: Update setup instructions https://review.openstack.org/515564 | 08:03 |
*** gcb has joined #openstack-cinder | 08:03 | |
*** armaan has quit IRC | 08:18 | |
*** armaan has joined #openstack-cinder | 08:19 | |
*** junboli has quit IRC | 08:22 | |
openstackgerrit | Alexandru Muresan proposed openstack/cinder master: SMBFS: add fixed image support https://review.openstack.org/514659 | 08:38 |
*** markvoelker has joined #openstack-cinder | 08:45 | |
*** ntpttr_ has joined #openstack-cinder | 08:50 | |
*** namnh has joined #openstack-cinder | 08:51 | |
*** ntpttr has quit IRC | 08:52 | |
*** sdague has joined #openstack-cinder | 08:54 | |
openstackgerrit | Dinesh Bhor proposed openstack/cinder master: V3 jsonschema validation: base schema framework https://review.openstack.org/515649 | 08:56 |
openstackgerrit | zhuangzejian proposed openstack/cinder master: Update setup instructions https://review.openstack.org/515564 | 09:00 |
openstackgerrit | Jack Lu proposed openstack/os-brick master: Update supported transports for iscsi connector https://review.openstack.org/510776 | 09:00 |
*** lpetrut has joined #openstack-cinder | 09:09 | |
*** markvoelker has quit IRC | 09:18 | |
johnthetubaguy | ildikov: just reading matt's comment in the review, do we have a way forward on that? | 09:22 |
ildikov | johnthetubaguy: it depends on what we want to do :) | 09:22 |
ildikov | johnthetubaguy: as the behavior is there for live migrate right now | 09:22 |
johnthetubaguy | so I think we pass the host and mountpoint to cinder? | 09:23 |
ildikov | we don't pass the host to reserve currently | 09:23 |
johnthetubaguy | what I thought we said for live-migration was same instance, different host, is allowed | 09:23 |
johnthetubaguy | true, this would fail on sending the connector | 09:23 |
ildikov | host is missing at the moment, we need to tweak that in Cinder | 09:24 |
ildikov | for live migrate I think we send the connector for the second attachment with create | 09:24 |
ildikov | so we do that in one step | 09:24 |
johnthetubaguy | yeah, makes sense | 09:24 |
ildikov | so we might be able to leverage that here | 09:25 |
ildikov | if we want to build on the current behavior | 09:26 |
johnthetubaguy | did we want to limit the number of attachments to two, I remember we mentioned that at some point? | 09:26 |
ildikov | I mean the fact that the second attachment_create contains extra information | 09:26 |
ildikov | I would leave that decision to Cinder | 09:27 |
ildikov | and I don't remember that we actually would've agreed on this | 09:27 |
ildikov | not yet at least | 09:27 |
johnthetubaguy | yeah, its more my curiosity really | 09:27 |
ildikov | I think we would like to try to limit the checks we are doing to find the middle ground between trying to be overly protective and complex but still bullet proof enough | 09:28 |
ildikov | if you're around in the afternoon when the US folks wake up it would be great to talk about this today or Monday afternoon the latest so we can get progress on this part | 09:29 |
ildikov | as it's still the regular attach part | 09:29 |
*** wanghao_ has quit IRC | 09:29 | |
johnthetubaguy | We can check on the Nova side, its just we were trying to not do that. | 09:30 |
ildikov | and with the Summit approaching we're losing two weeks... | 09:30 |
ildikov | I know, I was thinking about this too | 09:30 |
ildikov | we need to think about what check might make sense in Nova, but in general they should all happen in Cinder | 09:30 |
johnthetubaguy | I mean Nova has the same goals on complexity vs robustness, its a hard problem. | 09:30 |
johnthetubaguy | adding the host check to attach create and update seems like it would give us the protection we need | 09:31 |
johnthetubaguy | (its not perfect, but it would stop bad things) | 09:31 |
ildikov | as for attaching the same volume to the same instance multiple times I believe it causes more of administrational problems rather than data corruption | 09:31 |
johnthetubaguy | depends if you try and mount all those at the same time, and start writing to them | 09:32 |
ildikov | I mean having multiple attachment records, but physically I would think it's still one connection | 09:32 |
*** armaan has quit IRC | 09:32 | |
*** armaan has joined #openstack-cinder | 09:32 | |
johnthetubaguy | so automatically, we would plug each connection as a different device, I believe | 09:32 |
johnthetubaguy | although I would have to go check | 09:32 |
ildikov | hmm, ok, if that happens that's not good | 09:34 |
johnthetubaguy | ildikov: a have a nuts idea, for comparison, we could pass in the attachment-id we are replacing when we do the move | 09:35 |
ildikov | so technically the host check is doable on the Cinder side, we just don't do it yet as we don't have all the info for it necessarily at the time of the first create call | 09:36 |
*** abhi89 has joined #openstack-cinder | 09:36 | |
ildikov | hmm, I think that's too much context | 09:36 |
johnthetubaguy | ildikov: probably, its more bait and switch for the host check | 09:37 |
ildikov | I mean we would like to keep Cinder somewhat decoupled from what the consumer is doing still | 09:37 |
*** gcb has quit IRC | 09:37 | |
johnthetubaguy | sure, until you need to know | 09:37 |
*** stakeda has quit IRC | 09:38 | |
*** abhishek has quit IRC | 09:38 | |
openstackgerrit | Vivek Soni proposed openstack/cinder master: HPE 3PAR: Handle single path in FC configuration https://review.openstack.org/514952 | 09:38 |
ildikov | johnthetubaguy: yeah, I wish we could just say that we trust the sanity of our users :) | 09:39 |
johnthetubaguy | heh, dangerous | 09:40 |
ildikov | I know :) | 09:40 |
*** links has quit IRC | 09:42 | |
*** dpawlik has joined #openstack-cinder | 09:52 | |
*** danpawlik has quit IRC | 09:55 | |
*** armaan has quit IRC | 09:58 | |
*** kaisers1 has quit IRC | 10:05 | |
*** psachin has quit IRC | 10:05 | |
*** kaisers has joined #openstack-cinder | 10:14 | |
*** markvoelker has joined #openstack-cinder | 10:15 | |
*** armaan has joined #openstack-cinder | 10:31 | |
*** gkadam has quit IRC | 10:34 | |
*** namnh has quit IRC | 10:36 | |
*** armaan has quit IRC | 10:43 | |
*** gkadam has joined #openstack-cinder | 10:47 | |
*** abhishek has joined #openstack-cinder | 10:48 | |
*** abhitechie has quit IRC | 10:48 | |
*** markvoelker has quit IRC | 10:49 | |
*** abhi89 has quit IRC | 10:51 | |
*** robcresswell has quit IRC | 11:03 | |
openstackgerrit | Vivek Soni proposed openstack/cinder master: HPE 3PAR: Handle single path in FC configuration https://review.openstack.org/514952 | 11:06 |
*** markstur has joined #openstack-cinder | 11:15 | |
*** markstur has quit IRC | 11:20 | |
*** sdague has quit IRC | 11:31 | |
*** sdague has joined #openstack-cinder | 11:32 | |
*** sdague has quit IRC | 11:37 | |
*** sdague has joined #openstack-cinder | 11:38 | |
*** sdague has quit IRC | 11:38 | |
openstackgerrit | Vivek Soni proposed openstack/cinder master: [WIP]HPE 3PAR: Handle single path in FC configuration https://review.openstack.org/514952 | 11:41 |
*** nicolasbock has joined #openstack-cinder | 11:43 | |
*** belmoreira has joined #openstack-cinder | 11:45 | |
*** sdague has joined #openstack-cinder | 11:46 | |
*** markvoelker has joined #openstack-cinder | 11:46 | |
*** abishop has joined #openstack-cinder | 11:47 | |
openstackgerrit | Vivek Soni proposed openstack/cinder master: [WIP]HPE 3PAR: Handle single path in FC configuration https://review.openstack.org/514952 | 11:54 |
*** dillaman has joined #openstack-cinder | 12:03 | |
*** liverpooler has joined #openstack-cinder | 12:05 | |
*** pooja_jadhav has joined #openstack-cinder | 12:07 | |
*** nsingh has joined #openstack-cinder | 12:08 | |
*** niraj_singh has quit IRC | 12:09 | |
*** bhagyashris has quit IRC | 12:10 | |
*** pooja-jadhav has quit IRC | 12:10 | |
*** wes_dillingham has joined #openstack-cinder | 12:16 | |
*** markvoelker has quit IRC | 12:20 | |
*** markvoelker has joined #openstack-cinder | 12:27 | |
*** pgadiya has quit IRC | 12:28 | |
*** lhx__ has quit IRC | 12:30 | |
*** gkadam has quit IRC | 12:37 | |
*** armaan has joined #openstack-cinder | 12:37 | |
*** amoralej is now known as amoralej|lunch | 12:55 | |
*** mdbooth has quit IRC | 12:58 | |
*** armaan has quit IRC | 13:02 | |
*** catintheroof has joined #openstack-cinder | 13:04 | |
*** dustins has joined #openstack-cinder | 13:05 | |
*** mriedem has joined #openstack-cinder | 13:18 | |
*** mdbooth has joined #openstack-cinder | 13:21 | |
*** hemna_ has joined #openstack-cinder | 13:25 | |
*** dansmith is now known as superdan | 13:30 | |
*** sheel has joined #openstack-cinder | 13:32 | |
*** catintheroof has quit IRC | 13:34 | |
*** catintheroof has joined #openstack-cinder | 13:42 | |
*** spectr has quit IRC | 13:49 | |
*** zhaochao has quit IRC | 13:53 | |
*** dave-mccowan has joined #openstack-cinder | 13:53 | |
*** pooja-jadhav has joined #openstack-cinder | 13:54 | |
*** niraj_singh has joined #openstack-cinder | 13:55 | |
*** pooja_jadhav has quit IRC | 13:55 | |
*** nsingh has quit IRC | 13:55 | |
*** amoralej|lunch is now known as amoralej | 13:56 | |
*** gouthamr has joined #openstack-cinder | 14:02 | |
*** Dinesh_Bhor has quit IRC | 14:04 | |
*** mchlumsky has joined #openstack-cinder | 14:08 | |
*** spectr has joined #openstack-cinder | 14:09 | |
*** armax has joined #openstack-cinder | 14:17 | |
*** mchlumsky has quit IRC | 14:17 | |
*** mchlumsky has joined #openstack-cinder | 14:18 | |
*** dave-mccowan has quit IRC | 14:20 | |
*** mchlumsky has quit IRC | 14:26 | |
*** felipemonteiro_ has joined #openstack-cinder | 14:26 | |
*** mchlumsky has joined #openstack-cinder | 14:28 | |
*** felipemonteiro__ has joined #openstack-cinder | 14:32 | |
*** felipemonteiro_ has quit IRC | 14:36 | |
*** spectr has quit IRC | 14:39 | |
*** e0ne has quit IRC | 14:45 | |
*** josecastroleon has quit IRC | 14:46 | |
*** armaan has joined #openstack-cinder | 14:48 | |
*** e0ne has joined #openstack-cinder | 14:53 | |
*** e0ne has quit IRC | 14:53 | |
*** e0ne has joined #openstack-cinder | 14:54 | |
*** e0ne has quit IRC | 14:54 | |
*** e0ne has joined #openstack-cinder | 14:55 | |
*** e0ne has quit IRC | 14:56 | |
*** e0ne has joined #openstack-cinder | 14:56 | |
*** e0ne has quit IRC | 14:57 | |
*** markstur has joined #openstack-cinder | 15:02 | |
*** belmoreira has quit IRC | 15:09 | |
*** dave-mccowan has joined #openstack-cinder | 15:11 | |
*** markstur has quit IRC | 15:11 | |
*** dustins has quit IRC | 15:13 | |
*** markstur has joined #openstack-cinder | 15:13 | |
*** AlexeyAbashkin has quit IRC | 15:13 | |
*** crose has joined #openstack-cinder | 15:14 | |
*** jmlowe_ has joined #openstack-cinder | 15:15 | |
*** newmember has quit IRC | 15:15 | |
*** hemna_ has quit IRC | 15:15 | |
*** jmlowe has quit IRC | 15:16 | |
*** links has joined #openstack-cinder | 15:19 | |
*** dave-mccowan has quit IRC | 15:21 | |
*** dhinesh has joined #openstack-cinder | 15:22 | |
*** armax_ has joined #openstack-cinder | 15:22 | |
*** salv-orlando has quit IRC | 15:22 | |
*** armax has quit IRC | 15:22 | |
*** armax_ is now known as armax | 15:22 | |
*** salv-orlando has joined #openstack-cinder | 15:23 | |
*** salv-orlando has quit IRC | 15:27 | |
*** markstur_ has joined #openstack-cinder | 15:30 | |
*** itlinux has joined #openstack-cinder | 15:30 | |
*** itlinux has quit IRC | 15:30 | |
*** markstur has quit IRC | 15:33 | |
*** ntpttr_laptop has joined #openstack-cinder | 15:34 | |
*** sheel has quit IRC | 15:41 | |
rajinir | jungleboyj: smcginnis: eharney;: The driverfixes/ocata gate is still broken looks like. https://review.openstack.org/#/c/514364/ Any idea? | 15:47 |
jungleboyj | :-( eharney Any update there? | 15:48 |
eharney | rajinir: jungleboyj: patch needs to be rebased -- see the yellow dot on parents, it's based on an outdated patch which didn't have the fix included. | 15:49 |
*** markstur has joined #openstack-cinder | 15:50 | |
*** markstur_ has quit IRC | 15:53 | |
*** itlinux has joined #openstack-cinder | 15:57 | |
rajinir | eharney. thanks, will do | 15:57 |
*** lpetrut has quit IRC | 16:03 | |
*** Apoorva has joined #openstack-cinder | 16:03 | |
jungleboyj | eharney: Ah, good to know. Do you mean 'related changes' ? | 16:03 |
*** dave-mccowan has joined #openstack-cinder | 16:03 | |
eharney | jungleboyj: well, that shows it, but the Parent sha on the left shows its direct git parent | 16:04 |
jungleboyj | Ah, thank you. | 16:09 |
*** ntpttr_laptop has quit IRC | 16:10 | |
jungleboyj | Never knew what that dot meant. Learn something new every day. | 16:10 |
*** ntpttr_laptop has joined #openstack-cinder | 16:10 | |
eharney | it says "Not current" if you mouse over it | 16:10 |
*** hemna_ has joined #openstack-cinder | 16:18 | |
*** robcresswell has joined #openstack-cinder | 16:19 | |
*** crose has quit IRC | 16:19 | |
*** e0ne has joined #openstack-cinder | 16:21 | |
*** catintheroof has quit IRC | 16:23 | |
*** salv-orlando has joined #openstack-cinder | 16:23 | |
*** jmlowe_ has quit IRC | 16:23 | |
*** dave-mccowan has quit IRC | 16:26 | |
*** catintheroof has joined #openstack-cinder | 16:27 | |
*** salv-orlando has quit IRC | 16:27 | |
jungleboyj | eharney: Nice! | 16:30 |
*** ntpttr_laptop has quit IRC | 16:31 | |
*** dustins has joined #openstack-cinder | 16:34 | |
*** catintheroof has quit IRC | 16:35 | |
*** catintheroof has joined #openstack-cinder | 16:36 | |
*** shvepsy has quit IRC | 16:37 | |
*** shvepsy has joined #openstack-cinder | 16:37 | |
*** links has quit IRC | 16:38 | |
*** catintheroof has quit IRC | 16:39 | |
*** catintheroof has joined #openstack-cinder | 16:39 | |
*** dave-mccowan has joined #openstack-cinder | 16:50 | |
jgriffith | smcginnis: thingee geguileo Looks like this broke the default devstack deploy of cinderclient: https://review.openstack.org/#/c/490961/ | 16:52 |
*** AlexeyAbashkin has joined #openstack-cinder | 16:53 | |
jgriffith | AttributeError: "'Namespace' object has no attribute 'os_domain_id'" | 16:54 |
smcginnis | jgriffith: Are you sure that's from this patch? | 16:54 |
jgriffith | which appears to be because auth_type is set to `password` | 16:54 |
jgriffith | mriedem: yes, it does let you do a second attach to the Instance, BUT it's just reusing the attachment record that already exists | 16:56 |
*** AlexeyAbashkin has quit IRC | 16:57 | |
jgriffith | mriedem: at least it *was*, let me go have a look.... | 16:57 |
jgriffith | may have broken something along the way | 16:57 |
*** dave-mccowan has quit IRC | 17:03 | |
*** dave-mccowan has joined #openstack-cinder | 17:03 | |
*** dhinesh has quit IRC | 17:07 | |
*** ntpttr_laptop has joined #openstack-cinder | 17:08 | |
jgriffith | smcginnis: not 100% | 17:19 |
jgriffith | smcginnis: although reverting it did solve the problem | 17:19 |
jgriffith | it's also highly possible that Keystone screwed us again.. | 17:19 |
jgriffith | or the constant devstack churn has caught something | 17:19 |
jgriffith | I'd be curious if it fails for anybody else running client from source and not pypi | 17:20 |
jgriffith | mriedem: ildikov so yeah... since there were some changes for things like live-migrate and the dance that goes with it, we did this: https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1972 | 17:21 |
e0ne | jgriffith: is it broken on master? | 17:21 |
*** aarefiev is now known as aarefiev_afk | 17:21 | |
jgriffith | so the side effect is that you *can* call attach on a volume to the same instance as many times as you'd like | 17:21 |
jgriffith | it will create a new attachment record each time, then reuse the connection if it's there, or init a new one in the case of instance migration | 17:22 |
jgriffith | e0ne: it was when I just tried it this morning yes | 17:23 |
jgriffith | but who knows.. maybe it was one of those "one time glitch" things :) | 17:23 |
e0ne | jgriffith: do you mean it was reproduced on your env? | 17:23 |
jgriffith | e0ne: it's only occured in my env that I know of | 17:24 |
ildikov | jgriffith: we discussed to check the host too at the time when we were writing the attach spec for Nova | 17:24 |
ildikov | jgriffith: I need to re-read, we might have that part in the final spec too | 17:24 |
e0ne | jgriffith: got it. will try to reproduce on my env on Monday | 17:24 |
jgriffith | e0ne: awesome... I'll probably circle back and look at it again on my side later as well | 17:25 |
ildikov | jgriffith: so the idea was to let the volume be attached twice to the same instance if the host is different | 17:25 |
e0ne | jgriffith: ok, let me know if it works or not, please | 17:25 |
jgriffith | e0ne: I'll shoot you a note if I do so you don't waste any time | 17:25 |
e0ne | jgriffith: thanks | 17:25 |
jgriffith | ildikov: I'll leave that up to you and the Nova team | 17:25 |
jgriffith | easy enough to force that if that's what's desired | 17:26 |
jgriffith | ildikov: but I'm not letting this raise my blood pressure any more :) | 17:26 |
ildikov | jgriffith: I thought to discuss it with you on what and what not to do to avoid your blood pressure raising on this again :) | 17:28 |
*** itlinux has quit IRC | 17:28 | |
jgriffith | ildikov: well, I just mean all the changes we made already to behave like the old code, but let's ignore that | 17:29 |
jgriffith | it doesn't matter | 17:29 |
jgriffith | so the way I see it there are two choices, maybe 3: | 17:29 |
*** dave-mccowan has quit IRC | 17:29 | |
jgriffith | 1. Leave it as it works, but just gives a new attachment-id / entry (I'm sure people will hate that) | 17:30 |
*** jmlowe has joined #openstack-cinder | 17:30 | |
jgriffith | 2. Do as you mention and look at host-id and fail if it's the same (that's kinda ugly though IMO) | 17:30 |
jgriffith | 3. Add something on the Nova side that indicates to Cinder that the volume is part of an Instance/Live migration process and handle it explicitly that way | 17:31 |
jgriffith | Personally I'm good with option 1 :) | 17:31 |
*** dhinesh has joined #openstack-cinder | 17:31 | |
jgriffith | But wouldn't feel horrible about option 3 | 17:31 |
jgriffith | I think option 2 sucks | 17:31 |
jgriffith | that's my vote | 17:31 |
ildikov | yeah, I hate option 2 as well, I was thinking about it but couldn't get to actually liking it | 17:31 |
ildikov | and I had seven hours on the train today and it wasn't enough... :) | 17:32 |
smcginnis | jgriffith: You said we actually create a new attachment for it now? It doesn't just return the existing attachment? | 17:32 |
jgriffith | smcginnis: nope | 17:32 |
ildikov | option 3 sounds good from the perspective that at least we don't just best guess what's going on | 17:32 |
jgriffith | smcginnis: so the whole point of this was supposed to be attachments were ephemeral things | 17:32 |
ildikov | I'm not sure I fully get option 1 though | 17:32 |
jgriffith | but we've morphed that a bit now so it's not really quite true any more | 17:33 |
jgriffith | smcginnis: so what happens in this particular case is the reserve gets called, sets the special case flag because it's the same instance | 17:33 |
jgriffith | smcginnis: creates a new attachment object/id | 17:34 |
jgriffith | there's no connector passed in so it doesn't need to do a completion | 17:34 |
ildikov | jgriffith: live migrate only calls create I think | 17:34 |
jgriffith | updates the attachment objects and returns | 17:34 |
smcginnis | OK, make sense. | 17:35 |
jgriffith | ildikov: nah | 17:35 |
jgriffith | ildikov: it comes back later and does a round of updating and such | 17:35 |
ildikov | but anyone else can randomly make that call independently from live migrate anyway | 17:35 |
jgriffith | ildikov: it has to, because it needs to move the connection | 17:35 |
jgriffith | to antoher host | 17:35 |
jgriffith | I remember this all pretty well because I didn't want to do the whole reserve and status check thing that mandated this special case handling here | 17:36 |
jgriffith | the status checking stuff inparticular was the challenge | 17:36 |
ildikov | jgriffith: here: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5709 | 17:39 |
jgriffith | ildikov: "here" what? | 17:39 |
ildikov | it got updated I think after we realized we don't want an attachment_update call in refresh_connection_info | 17:40 |
jgriffith | Oh, the connector | 17:40 |
ildikov | yep :) | 17:40 |
jgriffith | ildikov: yeah, I haven't kept up with all 148 version of the patch :) | 17:41 |
ildikov | jgriffith: that's another patch :) | 17:41 |
jgriffith | ildikov: even better :) | 17:42 |
ildikov | jgriffith: :) | 17:42 |
ildikov | jgriffith: so what is option 1 again? | 17:43 |
ildikov | jgriffith: just to ensure I understand | 17:43 |
jgriffith | ildikov: option 1 is leave it like it is | 17:46 |
ildikov | jgriffith: ah ok, I guess the 'but' in the sentence is what got me confused :) | 17:47 |
ildikov | jgriffith: if we decide to tweak the code and option 1 is not good enough my vote is option 3 | 17:48 |
ildikov | mriedem: johnthetubaguy: ^^ ? | 17:48 |
mriedem | being able to attach a multiattach=False volume to the same instance that it's already attached to needs to result in an error at least from the nova side, since it would today as a 400, otherwise it's a behavior change, | 17:50 |
mriedem | so if we have to, we'll just do a check_attach() on the nova side | 17:50 |
ildikov | sadness :( | 17:51 |
mriedem | well, it's a behavior change, | 17:52 |
mriedem | so it has to be dealt with somewhere right? | 17:52 |
ildikov | yeah, I just hoped we will not get back to check_attach | 17:52 |
ildikov | I guess it's the matter of taste, like if I needed to choose between a flag and check_attach, I would choose the flag | 17:53 |
mriedem | nova should probably be enforcing a unique constraint in the bdm table on volumeid and instanceid, which would also catch this, but it's something we haven't done - and doing it now might break some existing deployments in weird ways on upgrade | 17:53 |
mriedem | flag == config option? | 17:53 |
*** itlinux has joined #openstack-cinder | 17:53 | |
ildikov | flag == telling Cinder we're live migrating so it can make an exception | 17:54 |
mriedem | i'm not really sure what that would look like | 17:55 |
mriedem | that's a flag passed during attachment_create? | 17:55 |
ildikov | if we could use the BDM long term though that might be better | 17:55 |
mriedem | so normal attach, if the flag isn't there, cinder kicks you out if the instance/volume already has an attachment? | 17:55 |
mriedem | nova's going to have bdms long term regardless, it's how we model the block devices (including ephemeral and swap devices) | 17:55 |
ildikov | yes, I meant like that, so when we create the second attachment we tell Cinder we need that for live migrate | 17:56 |
ildikov | otherwise fail | 17:57 |
ildikov | do you think using the BDM for solving this issue would be better/acceptable? | 17:58 |
*** newmember has joined #openstack-cinder | 17:58 | |
ildikov | as in that case we can hide behind a check_attach for now that we remove once we figured out the BDM solution | 17:59 |
mriedem | bdm solution == unique constraints? | 18:02 |
mriedem | so i don't want to have to do another cinder microversion of this, which is what you'd need to pass a new parameter to attachment_create, | 18:03 |
mriedem | and passing a "seriously i know what i'm doing" flag just for nova-isms sucks | 18:03 |
mriedem | i'm sure jgriffith would agree | 18:03 |
mriedem | so for now yeah i think we do a check_attach in nova | 18:03 |
mriedem | if we can eventually replace that with a bdm unique constraint in the db, that would be cool too | 18:03 |
mriedem | but i won't hold my breath for that since it would require some thought to not blow people up on upgrade | 18:03 |
mriedem | e.g. we are upgrading and find out there are 2 bdms in the db for the same instance and volume, what do we do? | 18:04 |
ildikov | fair enough | 18:05 |
*** harlowja has quit IRC | 18:05 | |
*** harlowja has joined #openstack-cinder | 18:05 | |
ildikov | I might name the new check_attach function to bleeding_ildiko or smth like | 18:05 |
ildikov | or just add it to check_attach_and_reserve_volume silently | 18:06 |
mriedem | it might be possible to only do the check_attach thing if we know we're dealing with new style attachments | 18:06 |
mriedem | i was thinking check_attach_and_reserve_volume | 18:06 |
mriedem | problem is i think we don't know if we're doing new style until we try attachment_create and it fails or not | 18:07 |
jgriffith | mriedem: I was referring to flag in terms of the attachment-create call | 18:07 |
jgriffith | gimmie a second to read scrollback | 18:07 |
jgriffith | Ok, all sounds good; how about I just put the enforcements back in Cinder to catch this | 18:08 |
jgriffith | and if we break something somewhere we can revisit | 18:08 |
mriedem | that would break the live migration flow i think | 18:08 |
jgriffith | mriedem: it looks like that may not | 18:08 |
jgriffith | at least if I add some Cinder checks for things like connectors and Instance UUID's | 18:09 |
mriedem | oh b/c of the host? | 18:09 |
jgriffith | mriedem: yeah | 18:09 |
mriedem | yeah i guess for live migration we pass the connector in attachment_create, | 18:09 |
mriedem | during volume attach, we don't | 18:09 |
jgriffith | as much as I sort of disliked that it might be the best idea | 18:09 |
jgriffith | ie safest, most explicit and expedient with OUT a mv | 18:09 |
mriedem | at 1pm on a friday and my head has been in something else all day, i can't really say at this point | 18:10 |
jgriffith | which I don't want to do antoher one of those either ;) | 18:10 |
jgriffith | mriedem: yeah, fair | 18:10 |
jgriffith | mriedem: and I haven't looked closely at it | 18:10 |
jgriffith | I'll see if I can come up with something before Monday without changing the Nova side including mv bump on Cinder | 18:10 |
ildikov | so I think live migrate itself is fine | 18:10 |
jgriffith | should be doable | 18:10 |
ildikov | and we want to prevent the user from themselves | 18:11 |
ildikov | for which we don't pass the connector | 18:11 |
jgriffith | ildikov: in either case, what we have now the more I think about it is no good | 18:11 |
jgriffith | even if we wanted to ignore the whole multi-attach False thing we shouldn't swap the attachment-id's anyway | 18:12 |
ildikov | either way we can say either that 'Cinder shouldn't care how the volume gets attached and it's on Nova to prevent this' or that 'Nova should only issue the attach call and let Cinder decide on everything as the single source of truth' and now we need to pick one of the above for this particular case | 18:13 |
ildikov | we create a new one and don't swap it and yes we shouldn't in the future either | 18:13 |
ildikov | as by calling create with the connector it creates a new attachment I would think | 18:14 |
*** wes_dillingham has quit IRC | 18:14 | |
*** Apoorva has quit IRC | 18:15 | |
ildikov | jgriffith: let's give it some thought and if I need to check it on the Nova side by the end, I will | 18:15 |
*** e0ne has quit IRC | 18:15 | |
*** ntpttr_laptop has quit IRC | 18:16 | |
*** Apoorva has joined #openstack-cinder | 18:17 | |
*** salv-orlando has joined #openstack-cinder | 18:24 | |
*** salv-orlando has quit IRC | 18:29 | |
*** crose has joined #openstack-cinder | 18:31 | |
*** armaan has quit IRC | 18:34 | |
*** rtjure has quit IRC | 18:34 | |
*** rtjure has joined #openstack-cinder | 18:35 | |
*** markstur has quit IRC | 18:36 | |
*** dave-mccowan has joined #openstack-cinder | 18:38 | |
*** rtjure has quit IRC | 18:39 | |
*** dave-mccowan has quit IRC | 18:44 | |
*** rtjure has joined #openstack-cinder | 18:45 | |
*** wes_dillingham has joined #openstack-cinder | 18:45 | |
*** amoralej is now known as amoralej|off | 18:48 | |
*** rtjure has quit IRC | 18:50 | |
*** chhavi has quit IRC | 19:02 | |
*** crose has quit IRC | 19:07 | |
*** lpetrut has joined #openstack-cinder | 19:08 | |
*** crose has joined #openstack-cinder | 19:10 | |
*** abhishek has quit IRC | 19:11 | |
*** crose has quit IRC | 19:19 | |
*** ChubYann has joined #openstack-cinder | 19:23 | |
*** ChubYann has left #openstack-cinder | 19:28 | |
*** Apoorva has quit IRC | 19:28 | |
*** Apoorva has joined #openstack-cinder | 19:29 | |
*** catintheroof has quit IRC | 19:29 | |
*** e0ne has joined #openstack-cinder | 19:29 | |
*** e0ne has quit IRC | 19:30 | |
*** Apoorva has quit IRC | 19:33 | |
*** tesseract has quit IRC | 19:33 | |
*** dave-mccowan has joined #openstack-cinder | 19:37 | |
*** catintheroof has joined #openstack-cinder | 19:41 | |
*** LobsterRoll has joined #openstack-cinder | 19:41 | |
*** markstur has joined #openstack-cinder | 19:43 | |
*** wes_dillingham has quit IRC | 19:43 | |
*** LobsterRoll is now known as wes_dillingham | 19:43 | |
*** Apoorva has joined #openstack-cinder | 19:43 | |
*** catintheroof has quit IRC | 19:45 | |
*** markstur has quit IRC | 19:47 | |
*** dave-mccowan has quit IRC | 19:49 | |
*** rcernin has joined #openstack-cinder | 19:57 | |
*** itlinux has quit IRC | 20:01 | |
*** wes_dillingham has quit IRC | 20:11 | |
*** jmlowe has quit IRC | 20:14 | |
*** KeithMnemonic1 has quit IRC | 20:17 | |
*** abhishek has joined #openstack-cinder | 20:22 | |
*** salv-orlando has joined #openstack-cinder | 20:25 | |
*** salv-orlando has quit IRC | 20:29 | |
*** dave-mccowan has joined #openstack-cinder | 20:33 | |
*** dustins has quit IRC | 20:35 | |
*** salv-orlando has joined #openstack-cinder | 20:35 | |
*** ntpttr_laptop has joined #openstack-cinder | 20:35 | |
*** markstur has joined #openstack-cinder | 20:36 | |
*** mdbooth has quit IRC | 20:37 | |
*** dave-mccowan has quit IRC | 20:38 | |
*** itlinux has joined #openstack-cinder | 20:41 | |
*** mdbooth has joined #openstack-cinder | 20:43 | |
*** ntpttr_laptop has quit IRC | 20:48 | |
*** ntpttr_laptop has joined #openstack-cinder | 20:48 | |
*** felipemonteiro__ has quit IRC | 20:48 | |
*** abhishek has quit IRC | 20:53 | |
*** Apoorva_ has joined #openstack-cinder | 20:59 | |
*** abishop has quit IRC | 21:00 | |
*** hemna_ has quit IRC | 21:01 | |
*** Apoorva has quit IRC | 21:02 | |
*** ntpttr_laptop__ has joined #openstack-cinder | 21:03 | |
*** ntpttr_laptop has quit IRC | 21:05 | |
*** itlinux has quit IRC | 21:07 | |
*** itlinux has joined #openstack-cinder | 21:15 | |
*** ntpttr_laptop__ has quit IRC | 21:15 | |
openstackgerrit | Merged openstack/cinder master: Vzstorage: improvement of create_cloned_volume https://review.openstack.org/496828 | 21:23 |
openstackgerrit | Merged openstack/cinder master: Mark Cisco FC ZM driver as unsupported https://review.openstack.org/508275 | 21:28 |
*** rcernin has quit IRC | 21:28 | |
*** wes_dillingham has joined #openstack-cinder | 21:33 | |
*** newmember has quit IRC | 21:33 | |
*** itlinux has quit IRC | 21:44 | |
*** ganso has quit IRC | 21:47 | |
*** wes_dillingham has quit IRC | 21:49 | |
*** salv-orlando has quit IRC | 21:50 | |
*** salv-orlando has joined #openstack-cinder | 21:50 | |
*** mchlumsky has quit IRC | 21:53 | |
*** salv-orlando has quit IRC | 21:55 | |
*** salv-orlando has joined #openstack-cinder | 22:03 | |
*** dave-mccowan has joined #openstack-cinder | 22:15 | |
*** dave-mccowan has quit IRC | 22:24 | |
*** salv-orlando has quit IRC | 22:27 | |
*** salv-orlando has joined #openstack-cinder | 22:28 | |
*** itlinux has joined #openstack-cinder | 22:33 | |
*** salv-orlando has quit IRC | 22:35 | |
*** salv-orlando has joined #openstack-cinder | 22:36 | |
*** lpetrut has quit IRC | 22:36 | |
*** salv-orlando has quit IRC | 22:40 | |
*** itlinux has quit IRC | 22:46 | |
*** sdague has quit IRC | 22:50 | |
*** gnufied has quit IRC | 23:13 | |
SamYaple | hello friends. a couple of years back there was a spec make cinder-volume more HA like with shared backends. ceph as the use case, if you submit a create it will be assigned to one cinder-volume and if that cinder-volume service does down you can no longer manage that ovlume you created until it returns (or you db edit) does anyone know the status of that? | 23:46 |
*** ntpttr_laptop has joined #openstack-cinder | 23:47 | |
*** stvnoyes has quit IRC | 23:48 | |
*** ntpttr_laptop has quit IRC | 23:53 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!