*** leeantho has quit IRC | 00:04 | |
*** sage has joined #openstack-cinder | 00:06 | |
*** smoriya has joined #openstack-cinder | 00:06 | |
*** annashen has joined #openstack-cinder | 00:14 | |
*** rmesta has quit IRC | 00:18 | |
*** annashen has quit IRC | 00:19 | |
*** erlon has quit IRC | 00:20 | |
*** tsekiyam_ has joined #openstack-cinder | 00:27 | |
*** tsekiyama has quit IRC | 00:30 | |
*** tsekiyam_ has quit IRC | 00:31 | |
*** mtanino has quit IRC | 00:31 | |
*** haomaiwa_ has quit IRC | 00:32 | |
*** r-daneel has quit IRC | 00:34 | |
openstackgerrit | Kurt Martin proposed openstack/cinder: Port LeftHand driver to use ABCMeta driver model https://review.openstack.org/193374 | 00:38 |
---|---|---|
*** dims has joined #openstack-cinder | 00:39 | |
*** merooney has quit IRC | 00:46 | |
*** dims has quit IRC | 00:47 | |
*** dims has joined #openstack-cinder | 00:47 | |
*** openstack has joined #openstack-cinder | 00:56 | |
*** markvoelker has joined #openstack-cinder | 00:56 | |
*** gouthamr has quit IRC | 01:00 | |
*** gouthamr has joined #openstack-cinder | 01:01 | |
*** markvoelker has quit IRC | 01:02 | |
*** hodos has quit IRC | 01:02 | |
*** MentalRay has joined #openstack-cinder | 01:08 | |
*** gouthamr has quit IRC | 01:08 | |
*** mriedem_afk is now known as mriedem | 01:09 | |
*** haomaiwang has joined #openstack-cinder | 01:11 | |
*** ianbrown has quit IRC | 01:22 | |
*** merooney has joined #openstack-cinder | 01:24 | |
*** ianbrown has joined #openstack-cinder | 01:25 | |
*** haomaiwang has quit IRC | 01:28 | |
*** lixiaoy11 has joined #openstack-cinder | 01:30 | |
*** Longgeek has joined #openstack-cinder | 01:31 | |
*** Apoorva has quit IRC | 01:33 | |
*** haomaiwang has joined #openstack-cinder | 01:34 | |
*** _cjones_ has quit IRC | 01:44 | |
*** Longgeek has quit IRC | 01:50 | |
*** vilobhmm1 has quit IRC | 01:51 | |
*** laughterwym has joined #openstack-cinder | 01:52 | |
*** markvoelker has joined #openstack-cinder | 01:57 | |
*** merooney has quit IRC | 01:59 | |
*** aix has quit IRC | 02:02 | |
*** markvoelker has quit IRC | 02:02 | |
*** davechen has joined #openstack-cinder | 02:07 | |
*** davechen1 has joined #openstack-cinder | 02:10 | |
*** davechen has quit IRC | 02:12 | |
*** daneyon_ has quit IRC | 02:13 | |
*** vincent_hou has joined #openstack-cinder | 02:22 | |
*** tobe has joined #openstack-cinder | 02:24 | |
*** jungleboyj has joined #openstack-cinder | 02:30 | |
*** cknight has joined #openstack-cinder | 02:33 | |
*** cburgess has quit IRC | 02:39 | |
openstackgerrit | LisaLi proposed openstack/cinder: Add check whether cloned images are dependent on the snapshot to delete in ceph delete_snapshot function. Add error message to make clear. https://review.openstack.org/193003 | 02:42 |
*** vincent_hou has quit IRC | 02:43 | |
*** cburgess has joined #openstack-cinder | 02:44 | |
*** vilobhmm has joined #openstack-cinder | 02:45 | |
*** jamielennox is now known as jamielennox|away | 02:46 | |
*** vincent_hou has joined #openstack-cinder | 02:46 | |
*** Longgeek has joined #openstack-cinder | 02:47 | |
*** jamielennox|away is now known as jamielennox | 02:53 | |
*** cburgess has quit IRC | 03:04 | |
*** MentalRay has quit IRC | 03:04 | |
*** cburgess has joined #openstack-cinder | 03:05 | |
*** dims has quit IRC | 03:06 | |
*** cburgess has quit IRC | 03:07 | |
*** cburgess has joined #openstack-cinder | 03:10 | |
*** annashen has joined #openstack-cinder | 03:17 | |
*** Longgeek has quit IRC | 03:19 | |
*** annashen has quit IRC | 03:22 | |
*** coolsvap|away is now known as coolsvap | 03:30 | |
*** cknight has quit IRC | 03:43 | |
*** sgotliv_ has quit IRC | 03:44 | |
*** vincent_hou has quit IRC | 03:45 | |
*** markvoelker has joined #openstack-cinder | 03:46 | |
*** vincent_hou_ has joined #openstack-cinder | 03:48 | |
*** vincent_hou_ is now known as vincent_hou | 03:51 | |
*** markvoelker has quit IRC | 03:51 | |
*** vincent_hou_ has joined #openstack-cinder | 03:53 | |
*** mriedem has quit IRC | 03:55 | |
*** vincent_hou has quit IRC | 03:55 | |
*** vincent_hou_ is now known as vincent_hou | 03:56 | |
*** Longgeek has joined #openstack-cinder | 04:00 | |
*** Longgeek has quit IRC | 04:06 | |
*** Longgeek has joined #openstack-cinder | 04:08 | |
*** Longgeek_ has joined #openstack-cinder | 04:12 | |
*** vincent_hou has quit IRC | 04:12 | |
*** Longgeek has quit IRC | 04:15 | |
*** harlowja_at_home has joined #openstack-cinder | 04:21 | |
openstackgerrit | Daisuke Fujita proposed openstack/cinder: Adds missing argument to delete_keys method https://review.openstack.org/174241 | 04:31 |
openstackgerrit | Daisuke Fujita proposed openstack/cinder: Add missing argument to delete_keys method https://review.openstack.org/174241 | 04:31 |
*** _cjones_ has joined #openstack-cinder | 04:45 | |
*** markvoelker has joined #openstack-cinder | 04:47 | |
*** ianbrown has quit IRC | 04:48 | |
*** _cjones_ has quit IRC | 04:49 | |
*** markvoelker has quit IRC | 04:51 | |
*** ianbrown has joined #openstack-cinder | 05:00 | |
openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: replace management command from collie to dog https://review.openstack.org/193408 | 05:08 |
*** boris-42 has quit IRC | 05:12 | |
*** jwcroppe has joined #openstack-cinder | 05:18 | |
*** nkrinner has joined #openstack-cinder | 05:26 | |
*** vilobhmm has quit IRC | 05:28 | |
*** harlowja_at_home has quit IRC | 05:29 | |
*** yamada-h has joined #openstack-cinder | 05:31 | |
*** Maike has joined #openstack-cinder | 05:32 | |
*** haomaiwang has quit IRC | 05:33 | |
*** haomaiwang has joined #openstack-cinder | 05:35 | |
openstackgerrit | Angela Smith proposed openstack/cinder: Adds friendly zone name support https://review.openstack.org/180518 | 05:35 |
*** nihilifer has joined #openstack-cinder | 05:44 | |
*** BharatK has joined #openstack-cinder | 05:48 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add migration progress indication https://review.openstack.org/189614 | 05:49 |
*** belmoreira has joined #openstack-cinder | 05:57 | |
*** laughterwym has quit IRC | 06:05 | |
*** Lee1092 has joined #openstack-cinder | 06:05 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add migration progress indication https://review.openstack.org/189614 | 06:05 |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add migration progress indication https://review.openstack.org/189614 | 06:08 |
anish | hemna: pretty much done with the code, but I keep hitting this http://paste.openstack.org/show/303985/ | 06:11 |
anish | though the relevant iscsi part is working fine | 06:11 |
*** ianbrown has quit IRC | 06:11 | |
anish | wanted to check if this was a known issue | 06:11 |
*** vilobhmm has joined #openstack-cinder | 06:15 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add migration progress indication https://review.openstack.org/189614 | 06:17 |
*** annashen has joined #openstack-cinder | 06:20 | |
*** e0ne has joined #openstack-cinder | 06:25 | |
*** annashen has quit IRC | 06:25 | |
*** e0ne has quit IRC | 06:31 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add migration progress indication https://review.openstack.org/189614 | 06:33 |
*** lpetrut has joined #openstack-cinder | 06:34 | |
*** anshul has joined #openstack-cinder | 06:35 | |
*** markvoelker has joined #openstack-cinder | 06:36 | |
*** vilobhmm has quit IRC | 06:38 | |
*** vilobhmm has joined #openstack-cinder | 06:39 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add migration progress indication https://review.openstack.org/189614 | 06:40 |
*** markvoelker has quit IRC | 06:40 | |
openstackgerrit | Ryan Lucio proposed openstack/cinder: Adds the Violin Memory V7000 series FC driver. https://review.openstack.org/189920 | 06:42 |
winston-d | dulek: the reason we end up using the strange-named retry filter in oslo is because we were asked to do so. | 06:45 |
*** vilobhmm has quit IRC | 06:46 | |
winston-d | otherwise, filter scheduler wouldn't be able to pass review. long story. | 06:46 |
winston-d | dulek: so I wonder why retry filter didn't work for you, it should. unless your config didn't have it in default filters. | 06:47 |
*** vilobhmm has joined #openstack-cinder | 06:49 | |
dulek | winston-d: So basically despite of having IgnoreAttemptedHostsFilter renamed to RetryFilter in setup.cfg, only the long name worked for me in cinder.conf. | 06:57 |
dulek | winston-d: But it worked on the end. :) | 06:58 |
*** vilobhmm has quit IRC | 07:01 | |
openstackgerrit | LisaLi proposed openstack/cinder: Add dependency check in RBD delete_snapshot https://review.openstack.org/193003 | 07:06 |
*** chlong has quit IRC | 07:06 | |
*** markus_z has joined #openstack-cinder | 07:16 | |
winston-d | dulek: did you actually 'install' cinder? | 07:17 |
winston-d | dulek: that's how entrypoint works | 07:18 |
*** ronis has joined #openstack-cinder | 07:21 | |
dulek | If "pip install -e ." is enough - then yes. | 07:22 |
*** jwcroppe_ has joined #openstack-cinder | 07:22 | |
*** jwcroppe has quit IRC | 07:24 | |
*** deepakcs has joined #openstack-cinder | 07:30 | |
openstackgerrit | Dave Chen proposed openstack/python-cinderclient: Add commands for modifying image metadata https://review.openstack.org/147077 | 07:32 |
openstackgerrit | wanghao proposed openstack/cinder: Fix weird change of volume status in re-scheduling https://review.openstack.org/185545 | 07:32 |
openstackgerrit | Abhijeet Malawade proposed openstack/cinder: Validate outermost request body element name consistently https://review.openstack.org/187886 | 07:33 |
rlucio | hey anyone ever seen the jenkins check gate ceph test fail like this? http://logs.openstack.org/20/189920/12/check/check-tempest-dsvm-full-ceph/f69bf2a/console.html | 07:41 |
rlucio | doesn't look like an issue from my patch... maybe i should run a recheck? | 07:42 |
rlucio | oh, nevermind, answered my own question | 07:45 |
*** IlyaG has quit IRC | 07:50 | |
*** jamielennox is now known as jamielennox|away | 08:00 | |
*** jordanP has joined #openstack-cinder | 08:08 | |
*** yamada-h has quit IRC | 08:16 | |
*** boris-42 has joined #openstack-cinder | 08:17 | |
*** e0ne has joined #openstack-cinder | 08:18 | |
*** rushiagr_away is now known as rushiagr | 08:18 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Handle incorrect '--config-dir' param https://review.openstack.org/157749 | 08:21 |
openstackgerrit | Anish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 08:24 |
*** lpetrut has quit IRC | 08:24 | |
*** markvoelker has joined #openstack-cinder | 08:25 | |
*** afazekas has joined #openstack-cinder | 08:25 | |
openstackgerrit | Anish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 08:25 |
*** e0ne has quit IRC | 08:25 | |
*** jwcroppe_ has quit IRC | 08:26 | |
*** jwcroppe has joined #openstack-cinder | 08:27 | |
*** markvoelker has quit IRC | 08:29 | |
*** yamada-h has joined #openstack-cinder | 08:29 | |
*** jwcroppe has quit IRC | 08:31 | |
*** aix has joined #openstack-cinder | 08:32 | |
*** alexpilotti has joined #openstack-cinder | 08:33 | |
*** yamada-h has quit IRC | 08:39 | |
*** ankit_ag has joined #openstack-cinder | 08:41 | |
*** hodos has joined #openstack-cinder | 08:49 | |
hodos | thingee: hi Mike, so what about our ns4 driver? Can it be merged? | 08:50 |
*** Longgeek_ has quit IRC | 08:51 | |
*** lixiaoy11 has quit IRC | 08:51 | |
*** Longgeek has joined #openstack-cinder | 08:51 | |
*** lpetrut has joined #openstack-cinder | 08:56 | |
*** aarefiev has quit IRC | 09:00 | |
*** aarefiev has joined #openstack-cinder | 09:06 | |
*** ndipanov has joined #openstack-cinder | 09:09 | |
*** haomaiwang has quit IRC | 09:11 | |
*** annashen has joined #openstack-cinder | 09:22 | |
*** sgotliv_ has joined #openstack-cinder | 09:25 | |
*** annashen has quit IRC | 09:27 | |
*** e0ne has joined #openstack-cinder | 09:28 | |
*** tobe has quit IRC | 09:28 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Tests: Make fake_notifier per-instance https://review.openstack.org/193343 | 09:29 |
*** coolsvap is now known as coolsvap|away | 09:30 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: Get updated volume status in begin_detaching https://review.openstack.org/192628 | 09:36 |
*** coolsvap|away is now known as coolsvap | 09:37 | |
*** jasondotstar has joined #openstack-cinder | 09:41 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Handle incorrect '--config-dir' param https://review.openstack.org/157749 | 09:45 |
*** e0ne is now known as e0ne_ | 09:52 | |
*** p_deore has joined #openstack-cinder | 09:53 | |
*** e0ne_ is now known as e0ne | 09:54 | |
*** davechen1 has left #openstack-cinder | 09:56 | |
p_deore | DuncanT: Hi, I have a question regarding patch https://review.openstack.org/#/c/186688/ | 09:57 |
p_deore | DuncanT: Your concern is specifically about HostBinaryNotFound or all other exception messages? | 09:57 |
*** dims has joined #openstack-cinder | 10:09 | |
*** coolsvap is now known as coolsvap|away | 10:11 | |
*** markvoelker has joined #openstack-cinder | 10:13 | |
openstackgerrit | Anton Arefiev proposed openstack/cinder: Add driver list generator https://review.openstack.org/192751 | 10:14 |
openstackgerrit | Anton Arefiev proposed openstack/cinder: Add drivers list generator https://review.openstack.org/192751 | 10:17 |
*** markvoelker has quit IRC | 10:18 | |
*** anshul has quit IRC | 10:24 | |
*** anshul has joined #openstack-cinder | 10:24 | |
*** abehl has joined #openstack-cinder | 10:24 | |
*** yuriy_n17 has joined #openstack-cinder | 10:24 | |
*** anshul has quit IRC | 10:24 | |
*** abehl has quit IRC | 10:24 | |
*** anshul_ has joined #openstack-cinder | 10:25 | |
*** anshul_ has quit IRC | 10:26 | |
*** anshul has joined #openstack-cinder | 10:28 | |
*** anshul has quit IRC | 10:29 | |
*** aarefiev has quit IRC | 10:29 | |
*** anshul has joined #openstack-cinder | 10:29 | |
*** anshul has quit IRC | 10:29 | |
*** anshul has joined #openstack-cinder | 10:30 | |
geguileo | We are getting a lot of failures on pytho27 gate (or maybe it's just me) | 10:40 |
geguileo | And eharney found the reason | 10:40 |
geguileo | The patch would love some reviews... https://review.openstack.org/#/c/193343/ | 10:40 |
*** afazekas has quit IRC | 10:44 | |
*** haomaiwang has joined #openstack-cinder | 10:45 | |
e0ne | geguileo: thanks! i'll take a look on it | 10:55 |
geguileo | e0ne: Thank you! :) | 10:55 |
*** dims has quit IRC | 10:57 | |
*** macaptain has left #openstack-cinder | 10:58 | |
*** IanGovett has joined #openstack-cinder | 11:01 | |
*** dims has joined #openstack-cinder | 11:06 | |
tbarron | geguileo: yes, thank you! | 11:07 |
*** afazekas has joined #openstack-cinder | 11:07 | |
geguileo | tbarron: No problem | 11:08 |
tbarron | geguileo: dumb question, but why do you use side_effect rather than return_value when mocking cinder.rpc.get_notifier | 11:08 |
tbarron | geguileo: i'm just learning mock as I go along ... | 11:09 |
geguileo | Because if I use return_value I have to give a value | 11:09 |
geguileo | But with side effect the method gets all the arguments | 11:09 |
geguileo | So then the constructor can be properly called | 11:09 |
tbarron | kk, that makes sense | 11:10 |
tbarron | geguileo: and can you explain why we needed to join the notifications, as eric wasn't doing that? | 11:10 |
geguileo | Because library calls to notification create new instances | 11:11 |
*** aarefiev has joined #openstack-cinder | 11:11 | |
tbarron | righto | 11:11 |
geguileo | And we want to receive the notifications from all joined in the common notifier | 11:11 |
tbarron | ok, it's what I thought | 11:12 |
tbarron | thanks again! | 11:12 |
geguileo | Maybe there is another way to do it if I mock something else in RPC | 11:12 |
geguileo | But I thought this was easier to follow the logic | 11:12 |
geguileo | If you think it needs additional comments let me know | 11:12 |
tbarron | geguileo: not really, the explanation of joining the notifications is what I was thinking from the code | 11:13 |
geguileo | Ok | 11:13 |
tbarron | geguileo: the side-effect vs return is I guess something that might benefit, but your call | 11:14 |
tbarron | maybe everyone else would understand that right off | 11:14 |
geguileo | You prefer I add a comment on why side effect is used? | 11:15 |
tbarron | geguileo: I would mildly prefer that, but mostly I just want to see this patch merge! | 11:15 |
geguileo | tbarron: Me too, that's why I hijacked the patch while eharney is asleep | 11:16 |
tbarron | geguileo: and maybe everyone else would understand that right off anyways | 11:16 |
geguileo | Ok, then I'll add a comment if I have to submit another patch | 11:16 |
*** e0ne is now known as e0ne_ | 11:19 | |
*** merooney has joined #openstack-cinder | 11:22 | |
*** markvoelker has joined #openstack-cinder | 11:29 | |
*** e0ne_ has quit IRC | 11:29 | |
*** markvoelker has quit IRC | 11:34 | |
*** jasondotstar has quit IRC | 11:43 | |
*** erlon has joined #openstack-cinder | 11:50 | |
*** boris-42 has quit IRC | 11:52 | |
*** deepakcs has quit IRC | 11:56 | |
*** marcusvrn has joined #openstack-cinder | 11:59 | |
*** marcusvrn has quit IRC | 12:01 | |
*** marcusvrn has joined #openstack-cinder | 12:04 | |
*** markvoelker has joined #openstack-cinder | 12:05 | |
*** BharatK has quit IRC | 12:05 | |
*** e0ne has joined #openstack-cinder | 12:08 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Dell SC: Enable use of Storage Profiles https://review.openstack.org/193334 | 12:09 |
*** coolsvap|away is now known as coolsvap | 12:12 | |
*** laughterwym has joined #openstack-cinder | 12:17 | |
*** coolsvap is now known as coolsvap|away | 12:22 | |
*** kjnelson has joined #openstack-cinder | 12:32 | |
*** coolsvap|away is now known as coolsvap | 12:34 | |
*** xyang1 has joined #openstack-cinder | 12:36 | |
*** Longgeek has quit IRC | 12:41 | |
*** bswartz has quit IRC | 12:41 | |
*** Longgeek has joined #openstack-cinder | 12:42 | |
*** akerr has joined #openstack-cinder | 12:42 | |
*** jasondotstar has joined #openstack-cinder | 12:43 | |
*** dims has quit IRC | 12:46 | |
*** dims has joined #openstack-cinder | 12:46 | |
*** cdelatte has joined #openstack-cinder | 12:47 | |
*** haomaiwang has quit IRC | 12:48 | |
*** zhenguo has quit IRC | 12:51 | |
*** jaypipes has quit IRC | 12:51 | |
*** ankit_ag has quit IRC | 12:51 | |
openstackgerrit | Andrey Pavlov proposed openstack/cinder: Small fix in doc string definition https://review.openstack.org/193124 | 12:54 |
*** Yogi1 has joined #openstack-cinder | 12:55 | |
*** haomaiwang has joined #openstack-cinder | 12:56 | |
*** Longgeek has quit IRC | 12:57 | |
*** Longgeek has joined #openstack-cinder | 12:59 | |
*** coolsvap is now known as coolsvap|away | 12:59 | |
*** ganso_ has joined #openstack-cinder | 13:00 | |
*** eharney has joined #openstack-cinder | 13:03 | |
*** julim has joined #openstack-cinder | 13:05 | |
*** HenryG has quit IRC | 13:05 | |
*** HenryG has joined #openstack-cinder | 13:05 | |
*** HenryG has quit IRC | 13:06 | |
*** julim has quit IRC | 13:07 | |
*** HenryG has joined #openstack-cinder | 13:07 | |
*** julim has joined #openstack-cinder | 13:08 | |
*** m3m0 has quit IRC | 13:09 | |
*** gouthamr has joined #openstack-cinder | 13:10 | |
*** laughterwym has quit IRC | 13:10 | |
*** laughterwym has joined #openstack-cinder | 13:10 | |
*** m3m0 has joined #openstack-cinder | 13:11 | |
*** dustins has joined #openstack-cinder | 13:13 | |
*** gouthamr has quit IRC | 13:16 | |
*** nkrinner has quit IRC | 13:17 | |
*** bswartz has joined #openstack-cinder | 13:19 | |
*** gouthamr has joined #openstack-cinder | 13:21 | |
*** mriedem has joined #openstack-cinder | 13:24 | |
*** rmesta has joined #openstack-cinder | 13:25 | |
*** smoriya has quit IRC | 13:25 | |
*** jaypipes has joined #openstack-cinder | 13:26 | |
*** dannywilson has joined #openstack-cinder | 13:32 | |
*** asselin has quit IRC | 13:40 | |
*** dansmith is now known as superdan | 13:45 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter snapshots data on the DB side https://review.openstack.org/184267 | 13:45 |
*** dannywilson has quit IRC | 13:46 | |
*** haomaiwang has quit IRC | 13:46 | |
*** dannywilson has joined #openstack-cinder | 13:46 | |
*** gouthamr has quit IRC | 13:47 | |
*** nihilifer has quit IRC | 13:48 | |
openstackgerrit | xing-yang proposed openstack/cinder: Support SMI-S provider v8.0.3 in VMAX driver https://review.openstack.org/189517 | 13:50 |
*** jaypipes is now known as leakypipes | 13:50 | |
smcginnis | xyang1: ping | 13:53 |
*** dims is now known as dimsum__ | 13:57 | |
*** merooney has quit IRC | 13:57 | |
openstackgerrit | Dmitry Guryanov proposed openstack/os-brick: remotefs: add virtuozzo storage support https://review.openstack.org/188805 | 13:57 |
xyang1 | smcginnis: Hi | 14:00 |
*** merooney has joined #openstack-cinder | 14:01 | |
*** gouthamr has joined #openstack-cinder | 14:02 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Dell SC: Enable use of Storage Profiles https://review.openstack.org/193334 | 14:02 |
*** dannywilson has quit IRC | 14:02 | |
*** morgan has quit IRC | 14:02 | |
*** morgan has joined #openstack-cinder | 14:03 | |
*** morgan is now known as Guest93126 | 14:03 | |
*** jungleboyj has quit IRC | 14:05 | |
*** morganfainberg is now known as morgan-devserver | 14:06 | |
*** jecarey has joined #openstack-cinder | 14:07 | |
*** morgan-devserver is now known as CaptainMorgan | 14:08 | |
*** MentalRay has joined #openstack-cinder | 14:09 | |
*** ronis has quit IRC | 14:14 | |
*** krtaylor has quit IRC | 14:15 | |
*** asselin has joined #openstack-cinder | 14:16 | |
*** jwcroppe has joined #openstack-cinder | 14:21 | |
*** garthb_ has joined #openstack-cinder | 14:27 | |
*** garthb has joined #openstack-cinder | 14:27 | |
*** gouthamr has quit IRC | 14:27 | |
*** leopoldj has joined #openstack-cinder | 14:27 | |
*** r-daneel has joined #openstack-cinder | 14:27 | |
*** Maike has quit IRC | 14:28 | |
*** archers has joined #openstack-cinder | 14:29 | |
*** cknight has joined #openstack-cinder | 14:29 | |
openstackgerrit | Daniel proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 14:33 |
*** jasondotstar has quit IRC | 14:35 | |
*** kjnelson has quit IRC | 14:36 | |
*** gouthamr has joined #openstack-cinder | 14:37 | |
*** jasondotstar has joined #openstack-cinder | 14:37 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Revert "Disable backup progress notifications for unit tests" https://review.openstack.org/193579 | 14:38 |
*** timcl has joined #openstack-cinder | 14:42 | |
*** rushil has joined #openstack-cinder | 14:43 | |
*** krtaylor has joined #openstack-cinder | 14:45 | |
jgriffith | eharney: your patch looks pretty good so far... I'm at about 20/100 with no errors yet | 14:46 |
jgriffith | eharney: going to let it finish then I'll +2/A | 14:47 |
jgriffith | assuming no failures :) | 14:47 |
*** cknight has left #openstack-cinder | 14:47 | |
*** archers has quit IRC | 14:50 | |
*** daneyon has joined #openstack-cinder | 14:52 | |
*** krtaylor has quit IRC | 14:53 | |
*** Longgeek has quit IRC | 14:53 | |
*** Longgeek has joined #openstack-cinder | 14:54 | |
*** jungleboyj has joined #openstack-cinder | 14:54 | |
*** diemt has joined #openstack-cinder | 14:54 | |
*** jungleboyj has quit IRC | 14:59 | |
*** daneyon_ has joined #openstack-cinder | 14:59 | |
*** MentalRay has quit IRC | 15:02 | |
*** rushiagr is now known as rushiagr_away | 15:02 | |
*** daneyon has quit IRC | 15:02 | |
*** gouthamr has quit IRC | 15:02 | |
diemt | Hello, I'd like to backport Oracle ZFSSA iSCSI driver to stable/kilo, is there any process that needs to be followed? I'd appreciate any insight | 15:02 |
jgriffith | diemt: you can't do that | 15:04 |
jgriffith | diemt: we don't allow backports of features or new drivers | 15:05 |
jgriffith | diemt: Check out the wiki here for more info: https://wiki.openstack.org/wiki/StableBranch | 15:05 |
jgriffith | diemt: particularly the "Appropriate Fixes" section | 15:06 |
diemt | jgriffith: I see, however the iSCSI driver was in the gate back in juno time, it was pulled out because of no active CI. So it's not really a "new" driver | 15:06 |
diemt | jgriffith: in that sense, is it possible to make an exception? | 15:08 |
*** belmoreira has quit IRC | 15:08 | |
*** rushil has quit IRC | 15:09 | |
*** gouthamr has joined #openstack-cinder | 15:10 | |
*** kjnelson has joined #openstack-cinder | 15:10 | |
*** esker has joined #openstack-cinder | 15:11 | |
jgriffith | diemt: nope | 15:12 |
*** Tross1 has quit IRC | 15:12 | |
*** daneyon_ has quit IRC | 15:12 | |
jgriffith | diemt: but you can certainly give it to your customers outside of trunk | 15:12 |
*** kjnelson has quit IRC | 15:14 | |
*** kjnelson has joined #openstack-cinder | 15:14 | |
*** mriedem has left #openstack-cinder | 15:15 | |
*** mriedem has joined #openstack-cinder | 15:16 | |
smcginnis | thingee: ping | 15:17 |
smcginnis | xyang1: Sorry, had to run to a meeting. | 15:17 |
xyang1 | smcginnis: No problem | 15:17 |
thingee | dhellmann: got it, thanks | 15:17 |
thingee | smcginnis: hi | 15:17 |
*** Zanatoz has joined #openstack-cinder | 15:17 | |
smcginnis | thingee: Morning! Wondering if you want blueprints for something like this: https://review.openstack.org/#/c/193334 | 15:18 |
smcginnis | thingee: The wiki says "blueprints are used to track the implementation of significant features in OpenStack" | 15:18 |
*** mtanino has joined #openstack-cinder | 15:18 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter snapshots data on the DB side https://review.openstack.org/184267 | 15:18 |
smcginnis | But I could see maybe for tracking things. | 15:18 |
diemt | jgriffith: Thank you | 15:19 |
*** jdurgin1 has joined #openstack-cinder | 15:20 | |
*** timcl1 has joined #openstack-cinder | 15:20 | |
openstackgerrit | Dmitry Guryanov proposed openstack/os-brick: remotefs: add virtuozzo storage support https://review.openstack.org/188805 | 15:21 |
*** timcl has quit IRC | 15:21 | |
openstackgerrit | Andrey Pavlov proposed openstack/cinder: Avoid race condition at snapshot deletion stage https://review.openstack.org/193596 | 15:21 |
openstackgerrit | Bob Callaway proposed openstack/cinder: Fix NetApp cDOT driver use of Glance image locations https://review.openstack.org/193597 | 15:21 |
e0ne | thingee, jgriffith: could you please merge backport https://review.openstack.org/#/c/192428/? | 15:25 |
*** jasondotstar has quit IRC | 15:26 | |
jgriffith | e0ne: no way | 15:26 |
jgriffith | :) | 15:26 |
e0ne | jgriffith: looks like it's not my day:) | 15:26 |
*** rushil has joined #openstack-cinder | 15:26 | |
jgriffith | e0ne: maybe tomorrow | 15:27 |
jgriffith | :) | 15:27 |
e0ne | jgriffith: will you work over the weekend? | 15:27 |
*** annashen has joined #openstack-cinder | 15:27 | |
*** jasondotstar has joined #openstack-cinder | 15:28 | |
e0ne | jgriffith: thank you | 15:28 |
*** MentalRay has joined #openstack-cinder | 15:29 | |
thingee | smcginnis: yeah I'm fine not having a bp for this | 15:30 |
*** edmondsw has joined #openstack-cinder | 15:30 | |
smcginnis | thingee: OK, thanks. | 15:30 |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Dell SC: Enable use of Storage Profiles https://review.openstack.org/193334 | 15:31 |
*** vilobhmm has joined #openstack-cinder | 15:31 | |
Zanatoz | sorry to interrupt. can anyone tell me what the format of my host entry would look like if I enabled multibackends with ceph. my current host entry is cinderhostname#RBD. Do I have to include the #RBD at the end when I enable multibackends? | 15:31 |
Zanatoz | so it would be cinderhostname@mybackend#RBD | 15:32 |
*** annashen has quit IRC | 15:32 | |
smcginnis | Zanatoz: I believe that is correct, but maybe someone that has actually used that can comment. | 15:33 |
smcginnis | eharney: ^^ That you? | 15:34 |
*** jordanP has quit IRC | 15:36 | |
eharney | include it where, exactly? | 15:37 |
*** _cjones_ has joined #openstack-cinder | 15:38 | |
Zanatoz | volumes table in cinder host column | 15:39 |
*** brad[] has joined #openstack-cinder | 15:39 | |
*** IanGovett has quit IRC | 15:39 | |
thingee | casual review friday | 15:40 |
thingee | DuncanT: ^ | 15:40 |
*** vilobhmm has quit IRC | 15:40 | |
eharney | Zanatoz: yes, it will show cinderhostname@mybackend#RBD there | 15:41 |
smcginnis | thingee: Didn't DuncanT realize that was a bad idea for him? Isreal has Friday's off or something. | 15:41 |
*** p_deore has quit IRC | 15:42 | |
smcginnis | Wow, 15 drivers have landed in Liberty. | 15:43 |
*** yuriy_n17 has quit IRC | 15:45 | |
thingee | yeah everyone did a good job in getting those through | 15:46 |
*** laughterwym has quit IRC | 15:46 | |
*** daneyon has joined #openstack-cinder | 15:46 | |
*** IanGovett has joined #openstack-cinder | 15:46 | |
thingee | smcginnis: this was rebased and you previously approved it https://review.openstack.org/#/c/157085/ | 15:48 |
*** krtaylor has joined #openstack-cinder | 15:48 | |
*** leopoldj has quit IRC | 15:49 | |
smcginnis | thingee: Done | 15:52 |
*** tsekiyama has joined #openstack-cinder | 15:54 | |
*** belmoreira has joined #openstack-cinder | 15:58 | |
*** gouthamr has quit IRC | 15:59 | |
openstackgerrit | Mike Perez proposed openstack/python-cinderclient: Add encryption-type-update to cinderclient https://review.openstack.org/58782 | 15:59 |
jgriffith | eharney: so apparantly nobody else was interested and they just ninja'd your patch :) | 16:00 |
jgriffith | eharney: but out of 75 runs I had one failure and it was NOT related to notifications at all | 16:00 |
jgriffith | eharney: xyang1 cinder.tests.unit.test_emc_vnxdirect.EMCVNXCLIDriverISCSITestCase.test_snapshot_preparing_volume | 16:00 |
xyang1 | jgriffith: What is this | 16:01 |
jgriffith | eharney: in other words, AWESOME job on that patch!! It seems to have addressed all of the various issues | 16:01 |
jgriffith | xyang1: I've been testing the notification stuff but running a looop of 100 test runs using concurrency =8 | 16:01 |
jgriffith | xyang1: with eharney 's latest patch I had NO notification errors so far | 16:01 |
jgriffith | xyang1: BUT that test referenced above did fail once | 16:01 |
*** vilobhmm has joined #openstack-cinder | 16:02 | |
xyang1 | jgriffith: I'll take a look | 16:02 |
jgriffith | xyang1: http://paste.openstack.org/show/304737/ | 16:02 |
jgriffith | xyang1: note, that out of 80 runs it only failed once | 16:02 |
xyang1 | jgriffith: Ok, thanks | 16:03 |
jgriffith | xyang1: so you're not likely to have a ton of luck reproducing it. But who knows, maybe you can see something from inspection | 16:03 |
* jgriffith really isn't worried about it... just a data point | 16:03 | |
kmartin | jgriffith, just a follow up on the ABC stuff, are we not suppose to be moving drivers to using the existing ABC solution? Is something in the works to change it and should I hold off doing the 3PAR drivers? | 16:04 |
*** leeantho has joined #openstack-cinder | 16:04 | |
xyang1 | jgriffith: Thanks, So this has to do with the wait | 16:04 |
*** esker has quit IRC | 16:05 | |
xyang1 | jgriffith: I'll send this to the VNX team | 16:05 |
*** markus_z has quit IRC | 16:06 | |
*** e0ne has quit IRC | 16:10 | |
*** dannywilson has joined #openstack-cinder | 16:11 | |
*** crose has joined #openstack-cinder | 16:11 | |
kmartin | jgriffith, the specs for the abc-driver-update was approved https://review.openstack.org/#/c/177114 without any objections. | 16:12 |
*** cs_0x6373 has quit IRC | 16:12 | |
*** timcl1 has quit IRC | 16:14 | |
eharney | jgriffith: good to see | 16:15 |
*** merooney has quit IRC | 16:16 | |
*** mriedem is now known as mriedem_burgers | 16:22 | |
mtanino | burgers! | 16:31 |
*** dannywilson has quit IRC | 16:31 | |
*** dannywilson has joined #openstack-cinder | 16:32 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup metadata import missing fields https://review.openstack.org/183222 | 16:33 |
thingee | mtanino: not a bad idea, although I already have lunch/dinner planned for today | 16:33 |
mtanino | thingee: nice :) Where do you plan to go dinner? | 16:34 |
*** dannywilson has quit IRC | 16:36 | |
thingee | mtanino: staying home =/ ... but I got a baguette, cooked chorizo, brie cheese, marinated olives, and a 2010 barley wine. | 16:37 |
thingee | mtanino: living the bachelor life right now, so I might talk myself into town and hanging out at the whiskey bar.... who knows | 16:38 |
mtanino | sweet! looks good dinner for Friday. | 16:38 |
*** edmondsw has quit IRC | 16:39 | |
CaptainMorgan | thingee: I hear the blind donkey has a seat with your name on it. | 16:41 |
thingee | CaptainMorgan: haha precisely | 16:41 |
smcginnis | Blind donkey? Sounds like a fun place. | 16:42 |
mtanino | thingee: I haven't been whiskey bar. may be you can have comfortable time on there. | 16:42 |
CaptainMorgan | smcginnis: good whisky bar here in Pasadena | 16:42 |
smcginnis | CaptainMorgan: I like whisky and I like CA. I'll have to put that on my list. :) | 16:42 |
CaptainMorgan | You should. | 16:44 |
CaptainMorgan | Actually I lied. It's a whiskey bar. (Mostly bourbon and rye, but some Irish and scotch) | 16:45 |
vilobhmm | sajeesh : thanks | 16:46 |
CaptainMorgan | smcginnis: but you should def.put it on your list. Besides it.is a reason to bug thingee and grab a drink :) | 16:47 |
smcginnis | CaptainMorgan: Hah, good plan. ;) | 16:47 |
CaptainMorgan | I may have experience in this.:P | 16:48 |
thingee | smcginnis: being walking distance from pasadena's old town makes this ideal :) | 16:48 |
smcginnis | CaptainMorgan: I just have experience in bugging thingee. :] | 16:48 |
*** timcl has joined #openstack-cinder | 16:49 | |
openstackgerrit | Dmitry Guryanov proposed openstack/os-brick: remotefs: add virtuozzo storage support https://review.openstack.org/188805 | 16:53 |
*** Apoorva has joined #openstack-cinder | 16:54 | |
*** jdurgin1 has quit IRC | 16:58 | |
*** IlyaG has joined #openstack-cinder | 17:00 | |
*** rushil has quit IRC | 17:02 | |
*** belmoreira has quit IRC | 17:03 | |
*** IanGovett has quit IRC | 17:05 | |
hodos | hi guys, a question about snapshots: so if volume is deleted, it's snapshots are supposed to be deleted with it? | 17:07 |
*** VManiac16 has joined #openstack-cinder | 17:07 | |
*** merooney has joined #openstack-cinder | 17:09 | |
eharney | hodos: they have to be deleted before the volume can be deleted | 17:10 |
openstackgerrit | Scott DAngelo proposed openstack/cinder-specs: Implement force_detach for safe cleanup https://review.openstack.org/172213 | 17:10 |
hodos | eharney: so I should return an error saying that volume has snapshots, or I remove snapshots automatically? | 17:11 |
eharney | hodos: the volume manager or API will return that error for you... your driver doesn't need to worry about it | 17:12 |
hodos | eharney: interesting | 17:12 |
*** rushil has joined #openstack-cinder | 17:14 | |
*** gouthamr has joined #openstack-cinder | 17:15 | |
*** aorourke has joined #openstack-cinder | 17:19 | |
*** mriedem_burgers is now known as mriedem | 17:22 | |
openstackgerrit | Vilobh Meshram proposed openstack/cinder: Nested Quota : Create allocated column in cinder.quotas https://review.openstack.org/185704 | 17:22 |
*** lpetrut has quit IRC | 17:23 | |
*** m3m0 has quit IRC | 17:24 | |
vilobhmm | ericsonsantos : https://review.openstack.org/185704 | 17:24 |
*** zul has quit IRC | 17:25 | |
*** zul has joined #openstack-cinder | 17:25 | |
*** annashen has joined #openstack-cinder | 17:27 | |
*** diegows has joined #openstack-cinder | 17:29 | |
*** MentalRay has quit IRC | 17:30 | |
*** rushil has quit IRC | 17:31 | |
*** rushil has joined #openstack-cinder | 17:32 | |
*** gouthamr has quit IRC | 17:37 | |
*** e0ne has joined #openstack-cinder | 17:38 | |
*** ronis has joined #openstack-cinder | 17:42 | |
*** breitz has quit IRC | 17:42 | |
*** timcl has quit IRC | 17:42 | |
ericksonsantos | vilobhmm, nice! I hope we get this code merged soon :) | 17:42 |
*** breitz has joined #openstack-cinder | 17:42 | |
*** timcl has joined #openstack-cinder | 17:45 | |
*** Longgeek has quit IRC | 17:45 | |
vilobhmm | ericksonsantos : :) | 17:45 |
vilobhmm | if you can review…it should be helpful | 17:45 |
ericksonsantos | vilobhmm, I did it already :P | 17:46 |
vilobhmm | you are too quick :P thanks | 17:46 |
ericksonsantos | haha | 17:47 |
*** gouthamr has joined #openstack-cinder | 17:48 | |
*** haomaiwa_ has joined #openstack-cinder | 17:48 | |
ericksonsantos | are you planning to go with the approach we've discussed yesterday on the next patch (set default quotas of subprojects to 0)? | 17:48 |
*** jasondotstar has quit IRC | 17:56 | |
vilobhmm | ericksonsantos : need to think more….but just checking whther its a subproject or not is not sufficient IMHO | 17:59 |
vilobhmm | will let you know | 17:59 |
*** jasondot_ has joined #openstack-cinder | 17:59 | |
ericksonsantos | vilobhmm, nice :) | 18:00 |
*** Rockyg has joined #openstack-cinder | 18:05 | |
*** jwcroppe has quit IRC | 18:07 | |
*** timcl1 has joined #openstack-cinder | 18:07 | |
mriedem | is there any concept of capabilities for volume drivers? | 18:07 |
mriedem | or are the various mixins essentially the capabilities? | 18:07 |
*** jwcroppe has joined #openstack-cinder | 18:08 | |
mriedem | like, is there an 'encryptable' or 'supports_encryption' capability/flag for volume driveres in cinder? | 18:08 |
mriedem | and if you try to create a volume from an encryption type where the volume type doesn't support encryption, the api request fails | 18:08 |
*** timcl has quit IRC | 18:08 | |
hemna | mriedem, get_volume_stats reports driver capabilities | 18:10 |
hemna | or is supposed to. we are working on standardizing on some of them | 18:10 |
*** dannywilson has joined #openstack-cinder | 18:10 | |
*** dannywilson has quit IRC | 18:10 | |
*** dannywilson has joined #openstack-cinder | 18:10 | |
mriedem | ok, like get_host_stats for virt drivers | 18:10 |
*** esker has joined #openstack-cinder | 18:12 | |
*** esker has quit IRC | 18:12 | |
mriedem | i don't even know how the encryption types work in cinder anyway, do they even do any encryption? b/c in the data model the encryption provider is some other plugin, which for tempest runs lives in nova | 18:13 |
*** esker has joined #openstack-cinder | 18:13 | |
*** jungleboyj has joined #openstack-cinder | 18:14 | |
*** gouthamr has quit IRC | 18:15 | |
*** MentalRay has joined #openstack-cinder | 18:15 | |
mriedem | ugh, also, gotta say i kind of hate the ostestr thing, unless there is a way to specify a single test case to run that i'm not figuring out | 18:17 |
mriedem | sdague: do you know anything about the ostestr stuff? | 18:18 |
*** gouthamr has joined #openstack-cinder | 18:18 | |
*** alexpilotti has quit IRC | 18:18 | |
mriedem | oh i see, -n | 18:21 |
jgriffith | kmartin: understood, just voicing my opinion publicly again that it's a bad idea. given that it's merged don't think you should care too much | 18:23 |
jgriffith | mriedem: did you get that to work ok? (ostestr -n) | 18:24 |
mriedem | jgriffith: tox -e py27 -- -n cinder.tests.unit.test_rbd.RBDTestCase.test_initialize_connection | 18:24 |
mriedem | i had to go into the os-testr code to figure out how to use it | 18:25 |
mriedem | \o/ | 18:25 |
mriedem | 0.1.0 of the library might have been premature :) | 18:25 |
mriedem | for usage i mean | 18:25 |
jgriffith | mriedem: yeah... I should find some place/way to document that | 18:25 |
jgriffith | mriedem: took me a lot longer to figure it out | 18:25 |
*** gouthamr has quit IRC | 18:25 | |
mriedem | well i think mtreinish is overseas in another limbo TZ otherwise i'd be trolling him hard :) | 18:25 |
jgriffith | LOL | 18:26 |
mriedem | also, the hard-coded --concurrency=6 is odd | 18:26 |
mriedem | testr figures out your concurrency value automatically | 18:26 |
jgriffith | mriedem: well... :) | 18:26 |
mriedem | defaults to ncpu | 18:26 |
jgriffith | mriedem: so the reason I did that was that we barfed if you did 8 | 18:26 |
jgriffith | mriedem: which is what the gate systems have | 18:27 |
jgriffith | mriedem: BUT | 18:27 |
mriedem | but your vm has 4 cpus... | 18:27 |
jgriffith | mriedem: still works fine | 18:27 |
mriedem | but if youre... | 18:27 |
mriedem | yeah, kills my cpu though :) | 18:27 |
jgriffith | mriedem: good news is I'm pushing a patch to remove that right now | 18:27 |
mriedem | and if i can't multitask and break multiple things at once, i'm sad | 18:27 |
jgriffith | mriedem: it looks like eharney 's patch fixed the remaining issues | 18:27 |
mriedem | cool :) | 18:27 |
mriedem | nice | 18:27 |
openstackgerrit | Eric Harney proposed openstack/cinder: Revert "Disable backup progress notifications for unit tests" https://review.openstack.org/193579 | 18:28 |
*** leeantho has quit IRC | 18:31 | |
*** gouthamr has joined #openstack-cinder | 18:32 | |
openstackgerrit | Matt Riedemann proposed openstack/cinder: rbd: set encrypted key in connection_info during initialize https://review.openstack.org/193673 | 18:32 |
mriedem | the ceph job should fail with that ^ | 18:32 |
thingee | mriedem: mention it in the review | 18:33 |
mriedem | thingee: wip'ed | 18:33 |
mriedem | i expect nova to blow up | 18:33 |
*** kjnelson1 has joined #openstack-cinder | 18:33 | |
*** kjnelson has quit IRC | 18:34 | |
openstackgerrit | John Griffith proposed openstack/cinder: Remove the hardcoded concurrency limit for ostestr https://review.openstack.org/193676 | 18:34 |
mriedem | jgriffith: +1 so hard :) | 18:35 |
jgriffith | mriedem: LOL | 18:37 |
*** lpetrut has joined #openstack-cinder | 18:37 | |
*** diegows has quit IRC | 18:39 | |
*** merooney has quit IRC | 18:39 | |
e0ne | jgriffith: let's wait until somebody puts -1 with words 'please, file a bug' | 18:39 |
jgriffith | e0ne: or that I spelled something wrong | 18:43 |
jgriffith | e0ne: or they didn't like what the comment said | 18:43 |
e0ne | :) | 18:43 |
smcginnis | jgriffith: You missed a comma. :P | 18:43 |
jgriffith | smcginnis: wouldn't be the first OR last time :) | 18:44 |
*** rushil has quit IRC | 18:45 | |
*** Lee1092 has quit IRC | 18:45 | |
*** Longgeek has joined #openstack-cinder | 18:46 | |
*** esker has quit IRC | 18:46 | |
smcginnis | Hey, just noticed the deadline for Tokyo presentation submissions is July 15th. That seems earlier than previously. | 18:47 |
smcginnis | If anyone is thinking of a presentation, it would be good to start working on that. | 18:47 |
e0ne | smcginnis: thanks for reminder | 18:47 |
e0ne | lightning talk: how to be borring and nerdy in reviews:) | 18:50 |
smcginnis | ;) | 18:50 |
*** diegows has joined #openstack-cinder | 18:51 | |
*** rushil has joined #openstack-cinder | 18:51 | |
*** MentalRay has quit IRC | 18:52 | |
*** diegows has quit IRC | 18:52 | |
*** merooney has joined #openstack-cinder | 18:54 | |
e0ne | thingee: hi. coul you or me try set https://review.openstack.org/#/c/145703/ as depends on https://review.openstack.org/146541? it could fix devstack jobs | 18:54 |
*** hodos|2 has joined #openstack-cinder | 18:54 | |
e0ne | thingee: at least, it works on my env but i didn't install devstack from scratch | 18:55 |
e0ne | it's about removing api v1 | 18:55 |
thingee | e0ne: sure | 18:55 |
erlon | mkoderer: ping | 18:56 |
openstackgerrit | Mike Perez proposed openstack/cinder: Remove v1 server support https://review.openstack.org/145703 | 18:57 |
*** hodos has quit IRC | 18:57 | |
*** timcl1 has quit IRC | 18:57 | |
openstackgerrit | Mike Perez proposed openstack/cinder: Remove v1 server support https://review.openstack.org/145703 | 18:58 |
*** timcl has joined #openstack-cinder | 18:58 | |
*** IlyaG_ has joined #openstack-cinder | 19:03 | |
e0ne | thingee: thanks | 19:03 |
openstackgerrit | Pavel Boldin proposed openstack/cinder: Add iscsi_target_flags configuration option https://review.openstack.org/182871 | 19:05 |
*** IlyaG has quit IRC | 19:06 | |
scottda | hemna: winston-d I put some notes & questions in Cinder - Nova API etherpad under "Future of 'nova volume-detach .....' " | 19:07 |
scottda | Please have a look when you get a chance: https://etherpad.openstack.org/p/CinderNovaAPI | 19:07 |
scottda | And anyone else interested. Nova devs are proposing changes that essentially remove/ignore checks on cinder DB state during volume detach. | 19:08 |
*** merooney has quit IRC | 19:10 | |
hemna | what do you mean "ignore checks on cinder db state" ? | 19:11 |
hemna | nova needs to expect getting VolumeIsBusy | 19:12 |
hemna | and deal with that appropriately | 19:12 |
hemna | meaning, if someone issues a volume attach to nova, and the cinder volume is busy (re: 'ing' state), then nova should catch that and back out gracefully. | 19:13 |
hemna | phew, I think I solved my last problem with the os-brick nova patch. | 19:14 |
* hemna crosses fingers | 19:14 | |
*** IanGovett has joined #openstack-cinder | 19:17 | |
*** daneyon has quit IRC | 19:22 | |
scottda | hemna: a Nova dev has proposed that we shouldn't have to implement '--force' for nova during volume detach, but that all nova volume-detach should use force_detach. In other words, Nova shouldn't care about cinder state. Seems wrong to me, I'm just trying to gather specific info on where the issue are. | 19:22 |
scottda | It seems that we had talked about moving state checks out of Nova and into Cinder.... | 19:23 |
scottda | And this probably should be thought of separately for attach and detach. Detach might be safe to ignore state? | 19:24 |
*** merooney has joined #openstack-cinder | 19:27 | |
*** rushil has quit IRC | 19:27 | |
jgriffith | xyang1: for what it's worth my test loop is now up to 160 and there have not been any additional failures | 19:28 |
xyang1 | jgriffith: Cool! | 19:28 |
jgriffith | xyang1: yeah, so def don't need to be worried about that little fluke | 19:29 |
jgriffith | xyang1: it's kinda cool... all unit tests running 12 cores; only takes around 75 seconds for a full run | 19:29 |
xyang1 | jgriffith: Ok, thanks:) | 19:29 |
*** merooney has quit IRC | 19:29 | |
xyang1 | jgriffith: That's great | 19:29 |
*** mriedem has quit IRC | 19:30 | |
jgriffith | xyang1: I'm going to leave this running all weekend and see where things end up :) | 19:30 |
xyang1 | jgriffith: Good luck:) | 19:30 |
e0ne | thingee: i'm sorry. i've got patched cinderclient in my env | 19:30 |
thingee | e0ne: propose that? :) | 19:30 |
*** marcusvrn has quit IRC | 19:31 | |
openstackgerrit | Mike Perez proposed openstack/cinder: Remove v1 server support https://review.openstack.org/145703 | 19:35 |
thingee | e0ne: rebased^ | 19:35 |
*** annashen has quit IRC | 19:35 | |
*** MentalRay has joined #openstack-cinder | 19:40 | |
*** rushil has joined #openstack-cinder | 19:41 | |
anish | hemna: preliminary code is up https://review.openstack.org/#/c/193451/ | 19:42 |
anish | no tests yet, and need to get the iser guys to sign off | 19:42 |
e0ne | thingee: looks like it should be fixed in an API, not in client: https://review.openstack.org/#/c/145703/5/cinder/api/views/versions.py | 19:44 |
e0ne | thingee: this change breaks backward compatibility. | 19:44 |
anish | hemna: I also deprecated the separate iser connector, dunno if that needs a bp/spec | 19:45 |
thingee | e0ne: how about version[:-2] and call it a day? :) | 19:50 |
thingee | not really sure why that needs to be a conditional to figure that out | 19:50 |
e0ne | thingee: i think, it's located deeper in urlmap.py | 19:52 |
*** annashen has joined #openstack-cinder | 19:53 | |
thingee | e0ne: no I'm saying http://paste.openstack.org/show/305227/ | 19:54 |
thingee | it's the same thing | 19:54 |
e0ne | thingee: will it work wtih version v2.1? | 19:55 |
thingee | e0ne: that's a good point, if we ever decided to support micro versions | 19:55 |
thingee | but we don't yet :) | 19:56 |
e0ne | thingee: :) | 19:56 |
*** timcl has quit IRC | 19:56 | |
*** belmoreira has joined #openstack-cinder | 19:57 | |
e0ne | thingee: imo, we can use something like version[:-2], but we should add a comment why it is and note about micro versions | 19:58 |
thingee | e0ne: definitely...was just doing that now | 19:58 |
e0ne | thingee: ok, let's try it | 19:58 |
e0ne | thingee: need to go offline to prepare for tomorrow python meetup | 19:59 |
thingee | thank you | 19:59 |
*** rmesta has quit IRC | 20:00 | |
e0ne | thingee: you are welcome | 20:00 |
*** rmesta has joined #openstack-cinder | 20:00 | |
*** rmesta has quit IRC | 20:00 | |
*** rmesta has joined #openstack-cinder | 20:01 | |
e0ne | thingee: somobody already implemented v2 support for openstackclient, now i'm working on rally https://review.openstack.org/#/c/193685/ | 20:01 |
*** rmesta has quit IRC | 20:01 | |
e0ne | thingee: it doesn't support cinder api v2 now:( | 20:01 |
*** rmesta has joined #openstack-cinder | 20:02 | |
*** akerr has quit IRC | 20:02 | |
*** cdelatte has quit IRC | 20:02 | |
*** primechuck has joined #openstack-cinder | 20:02 | |
*** rmesta has quit IRC | 20:02 | |
*** rmesta has joined #openstack-cinder | 20:03 | |
*** rmesta has quit IRC | 20:04 | |
*** dustins has quit IRC | 20:04 | |
*** merooney has joined #openstack-cinder | 20:04 | |
*** rmesta has joined #openstack-cinder | 20:04 | |
*** boris-42 has joined #openstack-cinder | 20:04 | |
*** rmesta has quit IRC | 20:04 | |
*** rmesta has joined #openstack-cinder | 20:05 | |
*** rmesta has quit IRC | 20:05 | |
*** rmesta has joined #openstack-cinder | 20:05 | |
*** rmesta has quit IRC | 20:06 | |
*** merooney has quit IRC | 20:06 | |
*** rmesta has joined #openstack-cinder | 20:06 | |
*** rmesta has quit IRC | 20:06 | |
*** rmesta has joined #openstack-cinder | 20:07 | |
*** rmesta has quit IRC | 20:07 | |
*** rmesta has joined #openstack-cinder | 20:07 | |
*** rmesta has quit IRC | 20:08 | |
*** rmesta has joined #openstack-cinder | 20:08 | |
*** rmesta has quit IRC | 20:08 | |
*** rmesta has joined #openstack-cinder | 20:08 | |
*** rmesta has quit IRC | 20:09 | |
*** rmesta has joined #openstack-cinder | 20:09 | |
*** jwcroppe has quit IRC | 20:09 | |
*** rmesta has quit IRC | 20:10 | |
*** rmesta has joined #openstack-cinder | 20:10 | |
*** rmesta has quit IRC | 20:10 | |
*** krtaylor has quit IRC | 20:10 | |
*** e0ne has quit IRC | 20:11 | |
*** belmoreira has quit IRC | 20:14 | |
*** Yogi1 has quit IRC | 20:15 | |
*** jwcroppe has joined #openstack-cinder | 20:15 | |
openstackgerrit | Mike Perez proposed openstack/cinder: Remove v1 server support https://review.openstack.org/145703 | 20:15 |
*** rushil has quit IRC | 20:15 | |
openstackgerrit | Scott DAngelo proposed openstack/cinder-specs: Implement force_detach for safe cleanup https://review.openstack.org/172213 | 20:17 |
*** annashen has quit IRC | 20:23 | |
*** annashen has joined #openstack-cinder | 20:25 | |
*** ronis has quit IRC | 20:28 | |
*** bswartz has quit IRC | 20:30 | |
*** esker has joined #openstack-cinder | 20:30 | |
*** julim has quit IRC | 20:31 | |
openstackgerrit | Ryan Lucio proposed openstack/cinder: Adds the Violin Memory V7000 series FC driver. https://review.openstack.org/189920 | 20:38 |
*** diemt has quit IRC | 20:40 | |
*** CaptainMorgan is now known as morgan | 20:41 | |
*** gouthamr has quit IRC | 20:42 | |
*** daneyon has joined #openstack-cinder | 20:43 | |
*** daneyon has quit IRC | 20:48 | |
*** rushil has joined #openstack-cinder | 20:56 | |
*** esker has quit IRC | 20:58 | |
*** ndipanov has quit IRC | 21:06 | |
*** rushiagr_away is now known as rushiagr | 21:08 | |
*** krtaylor has joined #openstack-cinder | 21:08 | |
openstackgerrit | Pavel Boldin proposed openstack/cinder: Add iscsi_target_flags configuration option https://review.openstack.org/182871 | 21:12 |
*** IanGovett has quit IRC | 21:13 | |
*** daneyon has joined #openstack-cinder | 21:14 | |
*** kjnelson1 has left #openstack-cinder | 21:14 | |
*** lpetrut has quit IRC | 21:15 | |
*** jwcroppe has quit IRC | 21:20 | |
*** jwcroppe has joined #openstack-cinder | 21:21 | |
*** annashen has quit IRC | 21:26 | |
*** jwcroppe has quit IRC | 21:26 | |
*** bswartz has joined #openstack-cinder | 21:28 | |
*** Rockyg has quit IRC | 21:30 | |
*** pck is now known as pckizer | 21:31 | |
openstackgerrit | Anish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 21:35 |
openstackgerrit | Mikhail Khodos proposed openstack/cinder: Re-add Nexenta driver. https://review.openstack.org/188877 | 21:38 |
*** ganso_ has quit IRC | 21:38 | |
*** crose has quit IRC | 21:40 | |
*** annashen has joined #openstack-cinder | 21:41 | |
*** esker has joined #openstack-cinder | 21:51 | |
*** esker has quit IRC | 21:54 | |
*** esker has joined #openstack-cinder | 21:57 | |
*** daneyon has quit IRC | 21:57 | |
*** erlon has quit IRC | 22:00 | |
*** xyang1 has quit IRC | 22:00 | |
*** rushiagr is now known as rushiagr_away | 22:01 | |
*** Yogi1 has joined #openstack-cinder | 22:01 | |
*** jungleboyj has quit IRC | 22:02 | |
openstackgerrit | Jon Bernard proposed openstack/cinder: Add support for generic volume migration https://review.openstack.org/187270 | 22:02 |
*** rushil has quit IRC | 22:07 | |
*** garthb has quit IRC | 22:11 | |
*** garthb_ has quit IRC | 22:11 | |
*** primechuck has quit IRC | 22:11 | |
*** dimsum__ has quit IRC | 22:19 | |
*** dimsum__ has joined #openstack-cinder | 22:20 | |
*** jecarey has quit IRC | 22:29 | |
*** anshul has quit IRC | 22:30 | |
*** jasondot_ has quit IRC | 22:32 | |
*** dimsum__ has quit IRC | 22:32 | |
*** merooney has joined #openstack-cinder | 22:38 | |
*** annashen has quit IRC | 22:50 | |
*** merooney has quit IRC | 22:54 | |
*** merooney has joined #openstack-cinder | 22:55 | |
thingee | jgriffith: https://review.openstack.org/#/c/192400/2 | 22:58 |
*** Yogi1 has quit IRC | 23:05 | |
*** zigo has quit IRC | 23:05 | |
*** ondergetekende has quit IRC | 23:06 | |
*** zigo has joined #openstack-cinder | 23:07 | |
*** ondergetekende has joined #openstack-cinder | 23:07 | |
*** merooney has quit IRC | 23:12 | |
*** MentalRay has quit IRC | 23:13 | |
*** VManiac16 has quit IRC | 23:14 | |
*** tbarron has quit IRC | 23:19 | |
*** krtaylor has quit IRC | 23:21 | |
*** eharney has quit IRC | 23:21 | |
*** anshprat has quit IRC | 23:22 | |
*** eharney has joined #openstack-cinder | 23:22 | |
*** anshprat has joined #openstack-cinder | 23:22 | |
*** krtaylor has joined #openstack-cinder | 23:23 | |
*** vilobhmm has quit IRC | 23:23 | |
*** arif-ali has quit IRC | 23:26 | |
*** merooney has joined #openstack-cinder | 23:27 | |
*** esker has quit IRC | 23:28 | |
*** esker has joined #openstack-cinder | 23:28 | |
*** arif-ali has joined #openstack-cinder | 23:29 | |
*** tbarron has joined #openstack-cinder | 23:34 | |
*** IlyaG_ has quit IRC | 23:36 | |
boris-42 | jgriffith: hi there | 23:37 |
jgriffith | boris-42: hey | 23:42 |
boris-42 | jgriffith: I am finally here=) | 23:43 |
jgriffith | boris-42: Congrats! | 23:43 |
boris-42 | jgriffith: ya better time zone=) | 23:44 |
boris-42 | jgriffith: so I see some recent issues with cinder backup functioanllity http://logs.openstack.org/36/178436/22/check/gate-rally-dsvm-rally-cinder/5d93033/rally-plot/results.html.gz#/CinderVolumes.create_and_restore_volume_backup | 23:45 |
boris-42 | jgriffith: logs are here http://logs.openstack.org/36/178436/22/check/gate-rally-dsvm-rally-cinder/5d93033/logs/ | 23:45 |
boris-42 | jgriffith: did you change recently (today) related to volume backup functionallity? | 23:46 |
*** hemna_ has joined #openstack-cinder | 23:52 | |
*** hemna has quit IRC | 23:53 | |
*** dannywilson has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!