| *** haomaiwang has quit IRC | 00:01 | |
| *** haomaiwang has joined #openstack-cinder | 00:01 | |
| *** mss-4 has joined #openstack-cinder | 00:05 | |
| *** EinstCrazy has joined #openstack-cinder | 00:10 | |
| *** EinstCrazy has quit IRC | 00:14 | |
| openstackgerrit | Angus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 00:14 |
|---|---|---|
| *** markvoelker has quit IRC | 00:15 | |
| *** edtubill has joined #openstack-cinder | 00:15 | |
| *** jgregor has quit IRC | 00:17 | |
| *** fthiagogv_ has joined #openstack-cinder | 00:26 | |
| *** ntpttr has quit IRC | 00:29 | |
| *** fthiagogv has quit IRC | 00:29 | |
| *** laughterwym has joined #openstack-cinder | 00:31 | |
| *** jgregor has joined #openstack-cinder | 00:31 | |
| *** mss-4 has quit IRC | 00:31 | |
| *** laughterwym has quit IRC | 00:32 | |
| *** laughterwym has joined #openstack-cinder | 00:32 | |
| *** laughterwym has quit IRC | 00:33 | |
| *** laughterwym has joined #openstack-cinder | 00:34 | |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add release notes for return-request-id-to-caller https://review.openstack.org/273930 | 00:37 |
| *** sheel has joined #openstack-cinder | 00:38 | |
| *** jgregor has quit IRC | 00:40 | |
| *** diogogmt has joined #openstack-cinder | 00:42 | |
| *** markvoelker has joined #openstack-cinder | 00:46 | |
| openstackgerrit | John Griffith proposed openstack/python-cinderclient: Add replication v2.1 (cheesecake) calls https://review.openstack.org/281597 | 00:46 |
| openstackgerrit | John Griffith proposed openstack/python-cinderclient: Add replication v2.1 (cheesecake) calls https://review.openstack.org/281597 | 00:47 |
| openstackgerrit | Richard Hedlind proposed openstack/cinder: Force target_lun to be int type to make os-brick happy https://review.openstack.org/283889 | 00:53 |
| openstackgerrit | Richard Hedlind proposed openstack/cinder: Force target_lun to be int type to make os-brick happy https://review.openstack.org/283889 | 00:56 |
| *** akerr_away has quit IRC | 00:56 | |
| openstackgerrit | Richard Hedlind proposed openstack/cinder: Force target_lun to be int type to make os-brick happy https://review.openstack.org/283889 | 00:59 |
| *** haomaiwang has quit IRC | 01:01 | |
| *** haomaiwang has joined #openstack-cinder | 01:01 | |
| *** Guest51662 has quit IRC | 01:01 | |
| *** garthb_ has quit IRC | 01:02 | |
| *** Guest56878 has joined #openstack-cinder | 01:05 | |
| *** Guest56878 has quit IRC | 01:05 | |
| *** wilson_liu has joined #openstack-cinder | 01:06 | |
| *** cheneydc has joined #openstack-cinder | 01:07 | |
| *** kfarr has quit IRC | 01:08 | |
| *** yangyapeng has joined #openstack-cinder | 01:09 | |
| *** yangyapeng has quit IRC | 01:10 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 (Cheesecake) https://review.openstack.org/275797 | 01:10 |
| *** akerr has joined #openstack-cinder | 01:14 | |
| *** mylu has quit IRC | 01:17 | |
| openstackgerrit | liuke proposed openstack/cinder: Huawei: Create snapshot have a log error https://review.openstack.org/280622 | 01:23 |
| *** Suyash has quit IRC | 01:28 | |
| *** akerr has quit IRC | 01:30 | |
| *** [1]Thelo has joined #openstack-cinder | 01:32 | |
| *** Thelo has quit IRC | 01:34 | |
| *** [1]Thelo is now known as Thelo | 01:34 | |
| *** lpetrut has quit IRC | 01:36 | |
| *** haomaiwang has quit IRC | 01:36 | |
| *** leeantho has quit IRC | 01:37 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/282768 | 01:39 |
| *** kenhui has quit IRC | 01:44 | |
| *** erlon has quit IRC | 01:46 | |
| *** jwcroppe has joined #openstack-cinder | 01:54 | |
| *** dims has quit IRC | 01:59 | |
| *** akerr has joined #openstack-cinder | 02:04 | |
| *** jdurgin1 has quit IRC | 02:07 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Support huawei consistency group https://review.openstack.org/260930 | 02:07 |
| *** akerr has quit IRC | 02:10 | |
| *** akerr has joined #openstack-cinder | 02:12 | |
| sheel | ameade: hi | 02:12 |
| *** haomaiwang has joined #openstack-cinder | 02:16 | |
| smcginnis | scottda: Didn't you have some test scripts for testing microversions? Can you point me at that? | 02:17 |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Add backup list sorted by data_timestamp https://review.openstack.org/270067 | 02:19 |
| *** apoorvad has quit IRC | 02:19 | |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Add backup list sorted by data_timestamp https://review.openstack.org/270067 | 02:20 |
| scottda | smcginnis: https://github.com/scottdangelo/TestCinderAPImicroversions | 02:21 |
| *** akerr has quit IRC | 02:21 | |
| *** apoorvad has joined #openstack-cinder | 02:22 | |
| lixiaoy1_ | smcginnis: Sorry for disturb. May I ask you to help review https://review.openstack.org/#/c/263179/ ? It has got several +1 already. | 02:25 |
| *** Lee1092 has joined #openstack-cinder | 02:27 | |
| smcginnis | scottda: Thanks! | 02:29 |
| smcginnis | lixiaoy1_: Sure, I'll take a look before I sign off for the night. | 02:29 |
| lixiaoy1_ | smcginnis: Thank you! | 02:29 |
| smcginnis | np | 02:29 |
| *** houming has joined #openstack-cinder | 02:38 | |
| *** lprice has joined #openstack-cinder | 02:40 | |
| *** lprice1 has quit IRC | 02:40 | |
| *** kenhui has joined #openstack-cinder | 02:41 | |
| gus | hemna: the os-brick/privsep change passes dsvm tempest and is ready for you to review. | 02:43 |
| gus | https://review.openstack.org/#/c/277224/ | 02:43 |
| openstackgerrit | Lallana proposed openstack/cinder: Modify the number of params of hypermetro in HuaweiDriver https://review.openstack.org/280997 | 02:43 |
| smcginnis | gus: Did you figure out the nova hang? | 02:44 |
| smcginnis | ? | 02:44 |
| smcginnis | Oops, sorry, delay on my client there. | 02:44 |
| gus | yes: https://review.openstack.org/#/c/280471/ | 02:44 |
| *** apoorvad has quit IRC | 02:45 | |
| gus | smcginnis: My combination of os.pipe() and io.open() wasn't using greenio at all, so would cause eventlet greenthread to do a hard blocking read(). | 02:45 |
| smcginnis | gus: Fun | 02:46 |
| gus | (The fix was to detect/use a suitable eventlet/greenlet equivalent) | 02:46 |
| smcginnis | gus: I'm sure that was a joy to track down. ;) | 02:46 |
| smcginnis | gus: So probably a hard question, but how comfortable are you with these changes. | 02:47 |
| gus | smcginnis: yes indeed, particularly because it doesn't produce any log signature, and the first version of my fix did nothing because nova doesn't monkey_patch `os` :P | 02:47 |
| smcginnis | gus: Because, TBH, I know this is highly desired, but I'm a little leary having this go in so late. | 02:47 |
| gus | smcginnis: at this point, very sure - there's really no change beyond a slightly different codepath to arrive at the same putils.execute call. | 02:48 |
| gus | Except: the commands are now run with (only) CAP_SYS_ADMIN | 02:48 |
| gus | (still run with uid=0/gid=0) | 02:48 |
| openstackgerrit | Merged openstack/python-cinderclient: Add release notes for return-request-id-to-caller https://review.openstack.org/273930 | 02:48 |
| gus | It's conceivable that some scsi/whatever driver out there requires some additional capability. | 02:48 |
| gus | smcginnis: but I understand the hesitation - particularly because no-one can easily test against the range of exotic hardware that os-brick/cinder supports. | 02:50 |
| smcginnis | gus: Thanks, that does help. | 02:51 |
| gus | As written, there's no actual *requirement* for cinder to upgrade - it's an API-compatible os-brick internal change. It should be fine to upgrade/downgrade os-brick versions across this change. | 02:51 |
| gus | (but I do need those two cinder-related changes submitted - one adds a new command to cinder's rootwrap filters, and the other sets a cinder.conf option in devstack) | 02:52 |
| smcginnis | gus: OK, I think I follow. | 02:52 |
| smcginnis | gus: I'll take a look yet tonight or tomorrow morning. | 02:53 |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Add backup-rename https://review.openstack.org/263184 | 02:53 |
| gus | smcginnis: thanks. | 02:53 |
| smcginnis | gus: Hmm, Jenkins -1 on it. | 02:54 |
| gus | smcginnis: see the experimental job. | 02:54 |
| smcginnis | gus: So what needs to be done to get this mergable? | 02:54 |
| gus | smcginnis: the regular dsvm job tests against released oslo versions - and I need a merged-but-not-yet-released oslo.concurrency change. | 02:55 |
| gus | ^ that, and the resulting requirements.txt bump. | 02:55 |
| smcginnis | gus: OK, gotcha. | 02:55 |
| smcginnis | gus: Thanks for all the details. | 02:56 |
| gus | sorry, wrong library - the oslo.concurrency one *has* been released (3.5.0). I need a new oslo.privsep release (containing https://review.openstack.org/#/c/282083/) | 02:57 |
| smcginnis | gus: Cool, I'll watch for the requirements update. | 02:57 |
| gus | I should put this in an email .. | 02:58 |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:01 | |
| *** mss-4_ has joined #openstack-cinder | 03:06 | |
| *** ChubYann has quit IRC | 03:08 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Re-enable -1 child limits for nested quotas https://review.openstack.org/282068 | 03:09 |
| *** dims has joined #openstack-cinder | 03:13 | |
| *** bardia has quit IRC | 03:19 | |
| *** haomaiwang has quit IRC | 03:20 | |
| *** yangyapeng has joined #openstack-cinder | 03:25 | |
| *** salv-orlando has joined #openstack-cinder | 03:26 | |
| *** ChubYann has joined #openstack-cinder | 03:28 | |
| *** salv-orlando has quit IRC | 03:28 | |
| *** martyturner has joined #openstack-cinder | 03:29 | |
| *** salv-orl_ has quit IRC | 03:29 | |
| openstackgerrit | Merged openstack/cinder: Include allocated quota value in the quota reserve https://review.openstack.org/240228 | 03:30 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Check before delete host https://review.openstack.org/282950 | 03:34 |
| *** mylu has joined #openstack-cinder | 03:36 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Check before delete host https://review.openstack.org/282950 | 03:37 |
| *** arch-nemesis has quit IRC | 03:39 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Match the ip more accurately in Huawei driver https://review.openstack.org/280481 | 03:40 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Don't fail when port group does not exist https://review.openstack.org/271599 | 03:42 |
| *** martyturner has quit IRC | 03:43 | |
| *** links has joined #openstack-cinder | 03:46 | |
| *** gouthamr has quit IRC | 03:48 | |
| *** boris-42 has quit IRC | 03:54 | |
| *** shausy has joined #openstack-cinder | 03:57 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/282768 | 04:00 |
| *** mylu has quit IRC | 04:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/283923 | 04:03 |
| *** wanghao has quit IRC | 04:04 | |
| *** wanghao has joined #openstack-cinder | 04:05 | |
| *** mylu has joined #openstack-cinder | 04:06 | |
| *** wN has quit IRC | 04:10 | |
| *** mylu has quit IRC | 04:12 | |
| *** wN has joined #openstack-cinder | 04:15 | |
| *** wN has joined #openstack-cinder | 04:15 | |
| *** mylu has joined #openstack-cinder | 04:16 | |
| *** mylu has quit IRC | 04:25 | |
| *** sheel has quit IRC | 04:27 | |
| *** mylu has joined #openstack-cinder | 04:38 | |
| *** laughter_ has joined #openstack-cinder | 04:48 | |
| *** laughterwym has quit IRC | 04:52 | |
| *** jamielennox is now known as jamielennox|away | 04:54 | |
| *** Poornima has joined #openstack-cinder | 05:00 | |
| *** haomaiwang has joined #openstack-cinder | 05:04 | |
| *** diablo_rojo has quit IRC | 05:05 | |
| *** jwcroppe has quit IRC | 05:10 | |
| *** zhangjn has quit IRC | 05:14 | |
| *** sgotliv has joined #openstack-cinder | 05:20 | |
| *** zhangjn has joined #openstack-cinder | 05:20 | |
| patrickeast | jgriffith: just saw your messages from earlier (been out most of the day) adding the targets to the capabilities makes sense to me | 05:21 |
| * patrickeast goes to look at the latest reviews | 05:22 | |
| *** liverpooler has quit IRC | 05:27 | |
| *** laughter_ has quit IRC | 05:28 | |
| *** laughterwym has joined #openstack-cinder | 05:29 | |
| *** cknight1 has quit IRC | 05:31 | |
| *** salv-orlando has joined #openstack-cinder | 05:36 | |
| *** gcb has quit IRC | 05:37 | |
| *** mylu has quit IRC | 05:37 | |
| *** mylu has joined #openstack-cinder | 05:39 | |
| *** chhavi has joined #openstack-cinder | 05:40 | |
| *** laughterwym has quit IRC | 05:41 | |
| *** laughterwym has joined #openstack-cinder | 05:42 | |
| *** laughterwym has quit IRC | 05:42 | |
| *** laughterwym has joined #openstack-cinder | 05:42 | |
| *** qeelee has joined #openstack-cinder | 05:42 | |
| *** mylu has quit IRC | 05:45 | |
| *** dims has quit IRC | 05:48 | |
| *** salv-orlando has quit IRC | 05:49 | |
| *** gcb has joined #openstack-cinder | 05:58 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-cinder | 06:01 | |
| *** avishay has joined #openstack-cinder | 06:15 | |
| *** nkrinner has joined #openstack-cinder | 06:16 | |
| *** harlowja_at_home has joined #openstack-cinder | 06:17 | |
| *** openstack has joined #openstack-cinder | 13:22 | |
| *** subscope has quit IRC | 13:24 | |
| *** edmondsw has joined #openstack-cinder | 13:27 | |
| *** qeelee has quit IRC | 13:29 | |
| *** yangyapeng has quit IRC | 13:33 | |
| *** ildikov has quit IRC | 13:33 | |
| yuriy_n17 | jgriffith: Hi! Very sorry for disturbing. Please review https://review.openstack.org/#/c/263257/ whenever you have free time. Blocking tempest test is merged. Thanks a lot for advance. | 13:33 |
| *** zhangjn has quit IRC | 13:33 | |
| *** timcl has joined #openstack-cinder | 13:34 | |
| *** diablo_rojo has joined #openstack-cinder | 13:35 | |
| *** shyama has joined #openstack-cinder | 13:38 | |
| *** zhangjn has joined #openstack-cinder | 13:38 | |
| jgriffith | yuriy_n17: hehe... I just worked around that bug finally | 13:39 |
| jgriffith | yuriy_n17: will review now, thanks! | 13:39 |
| *** dims has joined #openstack-cinder | 13:40 | |
| jgriffith | yuriy_n17: although I am angry because you've now given me yet another merge conflict to deal with in the Cheescake patch (just kidding, not really angry) | 13:40 |
| dulek | jgriffith: There's my comment on cheesecake to address anyway. ;) | 13:41 |
| yuriy_n17 | jgriffith: Such is life. | 13:41 |
| jgriffith | dulek: another one? | 13:41 |
| jgriffith | dulek: ahh | 13:42 |
| jgriffith | dulek: thanks, I'll fix that after yuriy_n17 patch merges then. Also I did update the hash, thanks for the reminder though | 13:42 |
| dulek | jgriffith: My intention wasn't to make you angry, really. | 13:42 |
| jgriffith | dulek: haha! | 13:43 |
| jgriffith | dulek: Nahhh, not at all | 13:43 |
| dulek | jgriffith: Yeah, but change in enum will trigger object hash change, so you'll need to update it again. | 13:43 |
| yuriy_n17 | jgriffith: Thank you ever so much. | 13:43 |
| *** zhipeng has quit IRC | 13:44 | |
| jgriffith | dulek: Oh, that's what you meant. Yeah, no problem | 13:44 |
| jgriffith | dulek: thanks! | 13:44 |
| *** EinstCrazy has joined #openstack-cinder | 13:47 | |
| *** porunov has joined #openstack-cinder | 13:47 | |
| *** akerr has joined #openstack-cinder | 13:48 | |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: cinder-api-microversions code https://review.openstack.org/224910 | 13:51 |
| *** eharney has joined #openstack-cinder | 13:57 | |
| *** ildikov has joined #openstack-cinder | 13:58 | |
| *** jgregor has joined #openstack-cinder | 13:58 | |
| *** haomaiwa_ has quit IRC | 14:01 | |
| *** salv-orlando has joined #openstack-cinder | 14:01 | |
| *** haomaiwang has joined #openstack-cinder | 14:01 | |
| *** xyang1 has joined #openstack-cinder | 14:02 | |
| *** merooney has joined #openstack-cinder | 14:06 | |
| *** edtubill has joined #openstack-cinder | 14:06 | |
| *** salv-orlando has quit IRC | 14:08 | |
| *** mriedem has joined #openstack-cinder | 14:14 | |
| *** julim has joined #openstack-cinder | 14:15 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Trying latest oslo.* from master https://review.openstack.org/284150 | 14:15 |
| *** rlrossit_ has joined #openstack-cinder | 14:17 | |
| *** merooeny_ has joined #openstack-cinder | 14:20 | |
| openstackgerrit | Dinesh Bhor proposed openstack/cinder: Provide user friendly messages for db sync https://review.openstack.org/284156 | 14:21 |
| *** merooney has quit IRC | 14:23 | |
| *** ociuhandu has quit IRC | 14:24 | |
| *** ociuhandu has joined #openstack-cinder | 14:25 | |
| *** cknight has joined #openstack-cinder | 14:29 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Mask chap password in log https://review.openstack.org/271595 | 14:32 |
| *** dustins has joined #openstack-cinder | 14:34 | |
| *** shyama has quit IRC | 14:35 | |
| *** EinstCrazy has quit IRC | 14:35 | |
| *** esker has quit IRC | 14:36 | |
| *** diogogmt has quit IRC | 14:42 | |
| *** diogogmt has joined #openstack-cinder | 14:44 | |
| openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: TESTING: Do Not Merge https://review.openstack.org/284165 | 14:47 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Trying latest oslo.* from master https://review.openstack.org/284150 | 14:49 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: Avoid hardcoding value from oslo library https://review.openstack.org/284167 | 14:49 |
| haypo | smcginnis: thanks for the infortend change ;) | 14:50 |
| haypo | smcginnis: FYI i lost the merge battle twice, my NetApp https://review.openstack.org/#/c/280331/ got a second conflict because a second NetApp was merged in the meanwhile :-p | 14:51 |
| haypo | smcginnis: (i updated my patch again ;-)) | 14:51 |
| haypo | it should be the *last* patch to port cinder tests to py3 | 14:51 |
| *** gabriel has joined #openstack-cinder | 14:52 | |
| *** ntpttr has joined #openstack-cinder | 14:52 | |
| akerr | quick, we need to merge another netapp patch | 14:52 |
| *** gabriel has left #openstack-cinder | 14:53 | |
| *** rlrossit_ has quit IRC | 14:53 | |
| *** rlrossit has joined #openstack-cinder | 14:53 | |
| smcginnis | akerr: Hah! | 14:54 |
| smcginnis | haypo: Thanks, I saw the update there. Just waiting for CIs to churn through. | 14:54 |
| smcginnis | haypo: IIRC, that doesn't remove the py3 whitelist, right? So we'll just need to do one more patch to have it turned on by default for everything? | 14:55 |
| *** rhagarty_ has quit IRC | 14:55 | |
| haypo | smcginnis: last time i wrote new patches, it was like 2 or 3 weeks ago, and i had something like 10 patches. i prefer to have independent patches rather than a patch serie | 14:55 |
| haypo | smcginnis: yes, the whitelist is still there | 14:56 |
| haypo | smcginnis: i can write a patch removing the whitelist on top of the NetApp change, once the infortrend patch is merged | 14:56 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Code cleanup https://review.openstack.org/282938 | 14:56 |
| openstackgerrit | Maxim Nestratov proposed openstack/cinder: Use versionedobjects in smbfs volume driver https://review.openstack.org/261551 | 14:56 |
| openstackgerrit | Maxim Nestratov proposed openstack/cinder: Use versionedobjects in vzstorage volume driver https://review.openstack.org/261550 | 14:56 |
| openstackgerrit | Maxim Nestratov proposed openstack/cinder: vzstorage: add ploop volume format support https://review.openstack.org/262246 | 14:56 |
| openstackgerrit | Maxim Nestratov proposed openstack/cinder: vzstorage: fix creating and deleting snapshots of in-use volumes https://review.openstack.org/276465 | 14:56 |
| openstackgerrit | Maxim Nestratov proposed openstack/cinder: Use versionedobjects in netapp_nfs unit test https://review.openstack.org/261557 | 14:56 |
| openstackgerrit | Maxim Nestratov proposed openstack/cinder: Use versionedobjects in glusterfs volume driver https://review.openstack.org/261552 | 14:57 |
| openstackgerrit | Maxim Nestratov proposed openstack/cinder: Use versionedobjects in scality volume driver https://review.openstack.org/261553 | 14:57 |
| openstackgerrit | Maxim Nestratov proposed openstack/cinder: Use versionedobjects in quobyte volume driver https://review.openstack.org/261554 | 14:57 |
| openstackgerrit | Maxim Nestratov proposed openstack/cinder: Use versionedobjects in nfs volume driver https://review.openstack.org/261555 | 14:57 |
| *** mss-4_ has quit IRC | 14:58 | |
| *** mtanino has joined #openstack-cinder | 14:59 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-cinder | 15:01 | |
| *** diogogmt has quit IRC | 15:01 | |
| *** jdillaman has quit IRC | 15:02 | |
| openstackgerrit | Merged openstack/cinder: Fix service-list filter https://review.openstack.org/263257 | 15:02 |
| *** krtaylor has quit IRC | 15:02 | |
| *** lprice has quit IRC | 15:04 | |
| *** salv-orlando has joined #openstack-cinder | 15:08 | |
| *** Julien-z_ has joined #openstack-cinder | 15:10 | |
| *** Julien-zte has quit IRC | 15:13 | |
| openstackgerrit | Mike Rooney proposed openstack/cinder: Implement Consistency Groups for ONTAP Drivers https://review.openstack.org/283194 | 15:13 |
| ameade | sheel: hey, my spec is intended to be a predecessor to the more advanced architecture your spec calls for | 15:14 |
| *** krtaylor has joined #openstack-cinder | 15:14 | |
| *** boichev2 has quit IRC | 15:14 | |
| *** boichev has joined #openstack-cinder | 15:15 | |
| ameade | speaking of which, smcginnis: I originally slated this for Mitaka but I believe it's too late in the game to impl a major feature, thoughts? | 15:17 |
| smcginnis | ameade: I think you are right, unfortunately. I was going to bring that up. | 15:18 |
| ameade | i have been spending most of my time doing replication in manila | 15:19 |
| smcginnis | ameade: I opened the Newton spec folder. If you want to move that over, we can get it queued up and ready for the next release. | 15:19 |
| smcginnis | Darn replication! :) | 15:19 |
| ameade | replication is so darn complicated | 15:19 |
| *** rhagarty has joined #openstack-cinder | 15:19 | |
| ameade | i will try to get a user messages patch up sooner than later | 15:19 |
| ameade | i'll just impl it as non-experimental until that mess is figured out | 15:20 |
| smcginnis | ameade: +1 Thanks! | 15:20 |
| *** Julien-z_ has quit IRC | 15:21 | |
| *** BharatK has quit IRC | 15:25 | |
| sheel | ameade:may i merge both specs in newton | 15:26 |
| sheel | to eliminate duplicate work | 15:26 |
| ameade | sheel: into a single spec you mean? | 15:26 |
| sheel | yes | 15:26 |
| sheel | ameade: yes | 15:27 |
| openstackgerrit | Alex Meade proposed openstack/cinder-specs: User facing error messages https://review.openstack.org/274958 | 15:27 |
| ameade | sheel: yeah, we can add additional work items | 15:28 |
| *** dims has quit IRC | 15:29 | |
| *** salv-orlando has quit IRC | 15:29 | |
| *** dims has joined #openstack-cinder | 15:30 | |
| *** annasort has quit IRC | 15:31 | |
| openstackgerrit | Eric Harney proposed openstack/python-cinderclient: Add --cascade to volume delete https://review.openstack.org/264983 | 15:33 |
| Guest60450 | https://review.openstack.org/#/c/251925/ Would appreciate if a core can review and workflow this patch. Thanks! | 15:33 |
| *** Guest60450 is now known as cfouts | 15:34 | |
| *** dims has quit IRC | 15:34 | |
| *** krtaylor has quit IRC | 15:34 | |
| *** alkhodos has joined #openstack-cinder | 15:35 | |
| *** rhagarty_ has joined #openstack-cinder | 15:36 | |
| *** pradipm has joined #openstack-cinder | 15:36 | |
| *** lprice has joined #openstack-cinder | 15:36 | |
| alkhodos | Hi guys. We have this page for all previous releases http://docs.openstack.org/liberty/config-reference/content/config_overview.html, will there be a similar page in Mitaka and if so, how can I add documentation for our drivers there? (they are merged) | 15:37 |
| *** rhagarty has quit IRC | 15:37 | |
| smcginnis | alkhodos: When your driver merged there should have been a DocImpact tag in the commit message. | 15:43 |
| smcginnis | alkhodos: That should have created a bug in openstack-manuals to add something about your driver. | 15:43 |
| smcginnis | alkhodos: I would recommend writing your own though if you want more than just a list of the config options. | 15:44 |
| *** nkrinner has quit IRC | 15:44 | |
| alkhodos | smcginnis: yes, that's what happened, I just can't understand how should I add my own? | 15:44 |
| yuriy_n17 | hemna: Hi! Very sorry for disturbing. Please review https://review.openstack.org/#/c/263744/12 to check your test whenever you have free time. Thanks a lot for advance. | 15:45 |
| alkhodos | smcginnis: create a patch with doc file or something? | 15:45 |
| smcginnis | alkhodos: You can create a patch directly. | 15:45 |
| smcginnis | alkhodos: https://github.com/openstack/openstack-manuals | 15:45 |
| smcginnis | alkhodos: It would be under here: https://github.com/openstack/openstack-manuals/tree/master/doc/config-reference/source/block-storage/drivers | 15:46 |
| *** krtaylor has joined #openstack-cinder | 15:47 | |
| openstackgerrit | Merged openstack/cinder: Port infortrend driver to Python 3 https://review.openstack.org/280389 | 15:47 |
| alkhodos | smcginnis: so I can just create a file and push it directly to the repo ? | 15:48 |
| *** nikeshm has joined #openstack-cinder | 15:48 | |
| alkhodos | smcginnis: nvm, I see now, thank you! | 15:50 |
| *** merooney has joined #openstack-cinder | 15:50 | |
| jgriffith | dulek: surprisingly that didn't change the hash | 15:50 |
| jgriffith | dulek: it looks like that file isn't included | 15:51 |
| jgriffith | dulek: which seems reasonable to me, I'd hate to bump every time we add a new status | 15:52 |
| *** merooeny_ has quit IRC | 15:52 | |
| *** haplo37 has joined #openstack-cinder | 15:52 | |
| rhedlind | I would love to get this change merged so that our CI can come back online. https://review.openstack.org/#/c/283889/3 | 15:52 |
| jgriffith | rhedlind: done | 15:52 |
| jgriffith | rhedlind: kinda surprised os-brick wouldn't just "handle" that | 15:53 |
| *** arecknag has quit IRC | 15:54 | |
| smcginnis | jgriffith: So one more serving of cheesecake yet? :) | 15:56 |
| *** garthb_ has joined #openstack-cinder | 15:56 | |
| *** jseiler has joined #openstack-cinder | 15:57 | |
| dulek | jgriffith: It probably should - when sending an enum over RPC with more statuses it may cause that older service will not understand a new one. I'll try to look into it. | 15:57 |
| jgriffith | dulek: I'm wading through the fixture a bit... | 15:57 |
| dulek | jgriffith: Anyway in this case you shouldn't care much, you don't send that anywhere. | 15:57 |
| Swanson | smcginnis, I presume I need to make a bug for repl 2.1 support to keep this in the hopper after m3? | 15:57 |
| jgriffith | dulek: it appears to only check changes in class-methods, not what they return here | 15:58 |
| *** pots has joined #openstack-cinder | 15:58 | |
| jgriffith | Not sure, but first glance anyway | 15:58 |
| jgriffith | dulek: I'll look later, but now back to yet another merge conflict :( | 15:58 |
| dulek | jgriffith: I'm basing my knowledge on what someone told me, so it's possible I'm wrong. | 15:58 |
| mc_nair | dulek: thanks for confirming the upgrade in https://review.openstack.org/#/c/282068/ isn't an issue | 15:58 |
| smcginnis | Swanson: No, we don't need an actual bug filed. | 15:59 |
| smcginnis | Swanson: We can for tracking, but not strictly required. | 15:59 |
| dulek | mc_nair: No problem, I'm watching all the db migration files in Gerrit. But it would be probably beneficial to explain how it should be judged if ALTER is allowed in devref. | 16:00 |
| mc_nair | definitely | 16:00 |
| *** haomaiwang has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-cinder | 16:01 | |
| *** erlon has joined #openstack-cinder | 16:02 | |
| *** dims has joined #openstack-cinder | 16:03 | |
| jgriffith | dulek: disregard my earlier statement :) | 16:04 |
| jgriffith | dulek: I ran the test in the wrong branch, it did in fact catch the change | 16:04 |
| *** chris_morrell has joined #openstack-cinder | 16:05 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 (Cheesecake) https://review.openstack.org/275797 | 16:06 |
| *** merooeny_ has joined #openstack-cinder | 16:06 | |
| rhedlind | jgriffith: Thanks! | 16:07 |
| *** salv-orlando has joined #openstack-cinder | 16:08 | |
| *** mss-4_ has joined #openstack-cinder | 16:09 | |
| *** merooney has quit IRC | 16:10 | |
| dulek | jgriffith: Take a look on my comment on latest patchset. Is this passing pep8 check? | 16:12 |
| *** pradipm has quit IRC | 16:12 | |
| *** annasort has joined #openstack-cinder | 16:14 | |
| *** diogogmt has joined #openstack-cinder | 16:14 | |
| *** annasort_ has joined #openstack-cinder | 16:15 | |
| *** mss-4_ has quit IRC | 16:18 | |
| *** annasort has quit IRC | 16:19 | |
| *** annasort_ is now known as annasort | 16:19 | |
| *** rajinir has joined #openstack-cinder | 16:19 | |
| *** vincent_hou has joined #openstack-cinder | 16:21 | |
| *** liverpooler has quit IRC | 16:22 | |
| jgriffith | dulek: garrrrr | 16:23 |
| mriedem | smcginnis: hemna: if no one has pushed up a request for os-brick 1.0.1 release, and you're cool with that, i was going to do it | 16:24 |
| * jgriffith said "nahh.. don't run pep8, only changed one line"... :( | 16:24 | |
| dulek | jgriffith: Your patch is still queued in zuul, so no harm in fixing that… | 16:24 |
| jgriffith | dulek: actually it passed locally | 16:24 |
| jgriffith | dulek: http://goo.gl/Kk633H | 16:25 |
| bswartz | jgriffith: I've made that mistake before | 16:26 |
| jgriffith | bswartz: :) | 16:26 |
| dulek | jgriffith: +1'ing it then. :) | 16:26 |
| jgriffith | so it does pass pep8 even though it's obviously wrong | 16:26 |
| hemna | mriedem, yah I think so, just to get that keyError fix in | 16:26 |
| jgriffith | dulek: LOL | 16:26 |
| jgriffith | dulek: I'm fixingit | 16:26 |
| *** laughterwym has quit IRC | 16:26 | |
| mriedem | hemna: ok, i'll put up the request | 16:28 |
| smcginnis | mriedem: Yep, was just waiting for the key error fix. Should be good to go now. | 16:29 |
| *** belmoreira has quit IRC | 16:31 | |
| cfouts | smcginnis, DuncanT: https://review.openstack.org/#/c/251925/ Edited the commit message just now. Ready for +2 and workflow. Thanks! | 16:31 |
| openstackgerrit | Eric Harney proposed openstack/python-cinderclient: Add --cascade to volume delete https://review.openstack.org/264983 | 16:31 |
| DuncanT | cfouts: Thanks, and sorry about that. I don't catch all the commit message issues, but I try | 16:32 |
| *** arch-nemesis has joined #openstack-cinder | 16:33 | |
| *** garthb has joined #openstack-cinder | 16:33 | |
| cfouts | DuncanT: no worries. I should have caught it myself. | 16:34 |
| openstackgerrit | Michael Rowden proposed openstack/cinder: Correcting thin provisioning behavior https://review.openstack.org/275408 | 16:36 |
| *** Poornima has joined #openstack-cinder | 16:37 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 (Cheesecake) https://review.openstack.org/275797 | 16:37 |
| merooeny_ | xyang1 I addressed your comments on the cg implementation patch https://review.openstack.org/#/c/283194/3 Please let me know if you have any other recommendations. Thanks again. | 16:38 |
| Swanson | go for 30! | 16:38 |
| xyang1 | merooeny_: sure | 16:39 |
| jgriffith | Swanson: :) | 16:40 |
| jgriffith | Swanson: I abuse gerrit and use it like a regular github repo | 16:41 |
| Swanson | :) | 16:41 |
| jgriffith | Swanson: ideally people follow changes as you go in increments... making it easier to review. As opposed to some that just dump a 3K line patch all in one go :( | 16:41 |
| jgriffith | Swanson: I need to change my workflow and do dependency chains | 16:42 |
| mc_nair | jgriffith, smcginnis, DuncanT - if you get a chance, the adding back of -1 support for nested quotas is ready for review (https://review.openstack.org/#/c/282068/) | 16:43 |
| bswartz | nooooooo | 16:43 |
| bswartz | there are times when multiple small patches are easier to review than 1 giant patch, but if it's not done just right, it can be even harder to review them | 16:44 |
| *** apoorvad has joined #openstack-cinder | 16:44 | |
| *** merooney has joined #openstack-cinder | 16:44 | |
| hemna | smcginnis, what time frame are we thinking for the midcycle ? | 16:44 |
| Swanson | jgriffith, dependency chains are great until they get deep. | 16:44 |
| hemna | smcginnis, I think we did it in August in the past, but that's awfully close to school starting for some. | 16:45 |
| *** ntpttr has quit IRC | 16:45 | |
| *** jistr has quit IRC | 16:45 | |
| hemna | smcginnis, would be nice if it was a bit earlier, July | 16:45 |
| smcginnis | hemna: Yeah, me too. | 16:45 |
| bswartz | we made that mistake with manila CGs in liberty -- we split it into 5 patches and it was a nightmare | 16:45 |
| smcginnis | hemna: I think late July or early August would probably be better. | 16:45 |
| hemna | smcginnis, we did early August last year. it conflicted with my kids starting school | 16:46 |
| hemna | :( | 16:46 |
| smcginnis | Early August? When do they start? | 16:46 |
| hemna | 1st week | 16:46 |
| hemna | mid-end July would be cool. | 16:46 |
| smcginnis | Of August?! Wow. | 16:46 |
| hemna | my $0.02 | 16:47 |
| hemna | smcginnis, yah | 16:47 |
| *** merooeny_ has quit IRC | 16:47 | |
| Swanson | jgriffith, that list_replication_targets thing is still down in driver.py. Not being called? | 16:47 |
| smcginnis | hemna: Yeah, let's see what ends up being available. | 16:47 |
| *** yuriy_n17 has quit IRC | 16:47 | |
| scottda | Getting dates for the mid-cycle are an important first step for the hosts. We need that to reserve rooms. | 16:47 |
| hemna | scottda, +1 | 16:47 |
| smcginnis | I would like to shoot closer to end of July than later in August like the past. | 16:48 |
| scottda | smcginnis: Do we have an etherpad yet? | 16:48 |
| hemna | it would be nice to have it before the 2nd milestone as well | 16:48 |
| hemna | which we always seem to do it after | 16:48 |
| smcginnis | hemna: +1 | 16:48 |
| bswartz | hemna: the problem is that hte design summit is so late in the cycle - it's usally more than halfway through milestone 1, so if you do the midcycle before milestone 2, it's only 8-10 weeks after the design summit | 16:49 |
| scottda | smcginnis: https://etherpad.openstack.org/p/cinder-newton-midcycle OK? | 16:49 |
| smcginnis | scottda: Sure! Thanks! | 16:49 |
| bswartz | ttx's proposal should help with that problem | 16:49 |
| openstackgerrit | Merged openstack/cinder: Scalable backup service - Liberty compatibility https://review.openstack.org/269412 | 16:49 |
| hemna | bswartz, sure. it's not ideal. | 16:49 |
| smcginnis | bswartz: I guess we'll find out with Ocata | 16:50 |
| hemna | having it after the 2nd milestone doesn't give us any time to try to get new things in though. | 16:50 |
| openstackgerrit | Merged openstack/cinder: Port netapp dataontap driver to Python 3 https://review.openstack.org/280331 | 16:50 |
| DuncanT | mc_nair: Queued. I probably won't get to look at it until morning though, I've replication to do tonight | 16:50 |
| hemna | puts us immediately behind :( | 16:50 |
| mc_nair | DuncanT: sounds more than fair | 16:50 |
| smcginnis | hemna: What do you mean. We completely rewrote replication at the midcycle. :D | 16:50 |
| smcginnis | kidding | 16:50 |
| hemna | :P | 16:50 |
| *** bardia has joined #openstack-cinder | 16:52 | |
| *** Guest51435 is now known as mgagne | 16:56 | |
| *** mgagne has quit IRC | 16:56 | |
| *** mgagne has joined #openstack-cinder | 16:56 | |
| ameade | eharney: https://review.openstack.org/#/c/275755 | 16:58 |
| ameade | eharney: I think your concern is addressed since he's creating an all new logger and not mocking log | 16:59 |
| *** dims has quit IRC | 17:00 | |
| *** haomaiwang has quit IRC | 17:01 | |
| openstackgerrit | Merged openstack/cinder-specs: Add pagination support to other resources https://review.openstack.org/247284 | 17:01 |
| eharney | ameade: well, the main point was about the effect of writing to and using self.logger from multiple tests | 17:01 |
| *** haomaiwang has joined #openstack-cinder | 17:01 | |
| eharney | ameade: i'm not 100% sure it breaks but it sure seems like it could | 17:01 |
| ameade | ah, lemme think about that | 17:02 |
| eharney | ameade: but it's fine if in a separate class where they won't share that variable | 17:02 |
| ameade | ah got it | 17:03 |
| ameade | yeah | 17:03 |
| *** dims has joined #openstack-cinder | 17:03 | |
| *** leeantho has joined #openstack-cinder | 17:05 | |
| *** belmoreira has joined #openstack-cinder | 17:06 | |
| *** sheel has quit IRC | 17:15 | |
| *** sheel has joined #openstack-cinder | 17:16 | |
| openstackgerrit | Jacob Gregor proposed openstack/cinder: WIP: Storwize SVC multiple management IPs https://review.openstack.org/284267 | 17:16 |
| dims | smcginnis : heads up, next set of oslo releases likely to break cinder - fix is here - https://review.openstack.org/#/c/284167/ | 17:19 |
| *** ntpttr has joined #openstack-cinder | 17:19 | |
| vincent_hou | scottda: Hi. | 17:19 |
| scottda | vincent_hou: Hi | 17:20 |
| vincent_hou | scottda: During one cinder meeting before, you said you were going to pay more attention to functional test, if I am correct. | 17:20 |
| vincent_hou | scottda: Is it something you are working on? | 17:21 |
| scottda | That sound right. Although perhaps I should have said "I'd like to pay more attention to functional tests"... | 17:21 |
| vincent_hou | yes, kinda of. | 17:21 |
| scottda | No, I am working on api-microversions ATM. Specifically the cinderclient partt. | 17:21 |
| scottda | vincent_hou: and then I'd like to add functional tests for microversions... | 17:22 |
| *** mvk has quit IRC | 17:22 | |
| vincent_hou | I see. | 17:22 |
| scottda | vincent_hou: I'd like to spend more time on fucntional tests in general, perhaps in a few weeks when we've put Mitaka to bed. | 17:22 |
| vincent_hou | I am very interested in functional tests. I guess I can start to submit BPs. | 17:23 |
| *** edmondsw has quit IRC | 17:23 | |
| *** baumann has quit IRC | 17:24 | |
| scottda | vincent_hou: OK, keep me informed, as I am interested as well. I'm just fully scheduled right now. | 17:24 |
| vincent_hou | sure. | 17:25 |
| openstackgerrit | Vivek Dhayaal proposed openstack/cinder: Support ZeroMQ messaging in cinder multibackend https://review.openstack.org/277113 | 17:29 |
| *** IlyaG has joined #openstack-cinder | 17:33 | |
| bapalm | smcginnis: Restacking worked. thanks! | 17:34 |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Fix HNAS iSCSI driver attachment https://review.openstack.org/284295 | 17:40 |
| *** vincent_hou has quit IRC | 17:40 | |
| openstackgerrit | Yucong Feng proposed openstack/cinder: Mark oslo.vmware as optional dependency https://review.openstack.org/242257 | 17:42 |
| lprice | xyang1: I made the changes you requested on https://review.openstack.org/#/c/282388, would you mind taking another look when you get a chance? | 17:43 |
| xyang1 | lprice: that is not me:), same last name but forst name is different | 17:44 |
| xyang1 | lprice: I was planning to look at it but have not got chance to | 17:45 |
| xyang1 | s/forst/first | 17:45 |
| *** ociuhandu has quit IRC | 17:46 | |
| *** edmondsw has joined #openstack-cinder | 17:47 | |
| lprice | xyang1: Whoops, sorry about that. | 17:47 |
| openstackgerrit | Yucong Feng proposed openstack/cinder: Mark oslo.vmware as optional dependency https://review.openstack.org/242257 | 17:47 |
| xyang1 | lprice: no problem. you are not the only one who thought we are the same person:) | 17:48 |
| lprice | xyang1: Thanks, I'd definitely appreciate your review on that. | 17:48 |
| *** chris_mo_ has joined #openstack-cinder | 17:50 | |
| *** chris_morrell has quit IRC | 17:50 | |
| *** knikolla has joined #openstack-cinder | 17:52 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Enable all unit tests on Python 3.4 https://review.openstack.org/284308 | 17:52 |
| haypo | smcginnis, ^^ enjoy ;) | 17:52 |
| *** anshul has quit IRC | 17:54 | |
| openstackgerrit | Merged openstack/cinder-specs: Remove Cinder API races https://review.openstack.org/207101 | 17:55 |
| openstackgerrit | Merged openstack/cinder-specs: remove python 2.6 trove classifier https://review.openstack.org/260769 | 17:56 |
| *** jordanP has quit IRC | 17:56 | |
| *** vincent_hou has joined #openstack-cinder | 17:56 | |
| openstackgerrit | Merged openstack/cinder-specs: Replace deprecated library function os.popen() with subprocess https://review.openstack.org/267504 | 17:58 |
| *** bardia has quit IRC | 17:58 | |
| *** jdillaman has joined #openstack-cinder | 17:58 | |
| *** cknight has quit IRC | 17:59 | |
| *** haomaiwang has quit IRC | 18:01 | |
| *** vincent_hou has quit IRC | 18:01 | |
| *** haomaiwang has joined #openstack-cinder | 18:01 | |
| openstackgerrit | Merged openstack/cinder-specs: Moving capacity-headroom spec to mitaka cycle https://review.openstack.org/249932 | 18:01 |
| *** ociuhandu has joined #openstack-cinder | 18:02 | |
| *** cknight has joined #openstack-cinder | 18:02 | |
| *** julim has quit IRC | 18:03 | |
| smcginnis | haypo: Thanks! | 18:03 |
| *** leeantho has quit IRC | 18:03 | |
| *** porunov has quit IRC | 18:07 | |
| *** permalac has quit IRC | 18:07 | |
| *** julim has joined #openstack-cinder | 18:07 | |
| *** leeantho has joined #openstack-cinder | 18:08 | |
| dulek | haypo: Python 3 tempest time? :D | 18:13 |
| openstackgerrit | Merged openstack/cinder: Force target_lun to be int type to make os-brick happy https://review.openstack.org/283889 | 18:14 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add backup RPC API v2.0 https://review.openstack.org/283072 | 18:16 |
| *** belmoreira has quit IRC | 18:23 | |
| *** vincent_hou has joined #openstack-cinder | 18:24 | |
| *** sgotliv has quit IRC | 18:25 | |
| *** jwcroppe has quit IRC | 18:26 | |
| *** lpetrut has quit IRC | 18:29 | |
| *** Poornima has quit IRC | 18:30 | |
| openstackgerrit | Merged openstack/python-cinderclient: Extra 'u' in output of cinder cli commands https://review.openstack.org/281475 | 18:32 |
| *** angela-s has joined #openstack-cinder | 18:35 | |
| *** zhangjn has quit IRC | 18:35 | |
| *** zhangjn has joined #openstack-cinder | 18:36 | |
| *** liverpooler has joined #openstack-cinder | 18:41 | |
| *** anshul has joined #openstack-cinder | 18:43 | |
| *** mylu has joined #openstack-cinder | 18:44 | |
| *** bardia has joined #openstack-cinder | 18:52 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Delete volumes with snapshots https://review.openstack.org/264982 | 18:53 |
| DuncanT | So I'm hitting -2 on cheese cake due to a missing space in a comment, ok? | 18:54 |
| smcginnis | LOL | 18:54 |
| *** rajinir has quit IRC | 18:55 | |
| *** vincent_hou has quit IRC | 18:57 | |
| jgriffith | DuncanT: sure go for it :) | 19:00 |
| *** nikeshm has quit IRC | 19:00 | |
| jgriffith | DuncanT: just remember, I'll *see* you in Austin | 19:00 |
| jgriffith | :) | 19:00 |
| DuncanT | +2 | 19:00 |
| *** haomaiwang has quit IRC | 19:01 | |
| DuncanT | I think the devref could benefit from another set of eyes, but the details of what extra is needed will come as people implement this stuff | 19:01 |
| *** 16WAAEDST has joined #openstack-cinder | 19:01 | |
| DuncanT | The only way to find those is to get it merged, let people implement drivers against it, then hate on John | 19:03 |
| jgriffith | DuncanT: haters gonna hate :) | 19:04 |
| aorourke | jgriffith, ping | 19:05 |
| jgriffith | aorourke: pong | 19:05 |
| aorourke | jgriffith, so i still have this is_admin issue the failing over. If instead of passing context.elevated from the manager to the Service object I just pass context, it seems to work fine. Printing out the context, is_admin is already True | 19:06 |
| jgriffith | aorourke: ok, lemme look at the code again. I don't quite understand why you'd see it and I wouldn't | 19:07 |
| aorourke | jgriffith, same here. When I LOG context.elevated, it is simply {} | 19:07 |
| aorourke | jgriffith, but LOG context, it is actually populated | 19:08 |
| *** lpetrut has joined #openstack-cinder | 19:08 | |
| aorourke | jgriffith, also, a minor thing, when you fail-over, the service is disabled but the Frozen field is still set to False | 19:10 |
| jgriffith | aorourke: yea, frozen is an "option" to failover | 19:11 |
| aorourke | jgriffith, right, but doesn't it freeze the host by default? | 19:12 |
| aorourke | jgriffith, upon fail-over the service is disabled | 19:12 |
| *** jwcroppe has joined #openstack-cinder | 19:13 | |
| *** jbernard_ is now known as jbernard | 19:13 | |
| jgriffith | aorourke: SOB... | 19:15 |
| jgriffith | reproduced | 19:15 |
| *** martyturner has joined #openstack-cinder | 19:16 | |
| aorourke | jgriffith, i'm not crazy?? | 19:16 |
| jgriffith | aorourke: well, let's not get carried away | 19:19 |
| aorourke | jgriffith, :) | 19:19 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Check for service existance in capabilities API https://review.openstack.org/271696 | 19:20 |
| *** belmoreira has joined #openstack-cinder | 19:20 | |
| *** Suyash has joined #openstack-cinder | 19:20 | |
| jgriffith | aorourke: so FWIW, you're absolutely correct about .elevated | 19:20 |
| *** Suyash has quit IRC | 19:21 | |
| jgriffith | aorourke: what's confusing is I removed that at the same time I removed it from teh freeze/thaw calls | 19:21 |
| openstackgerrit | Angela Smith proposed openstack/cinder: Fix for HTTP sessions left open in Brocade zone driver HTTP connector https://review.openstack.org/284362 | 19:21 |
| jgriffith | so I'm really confused | 19:21 |
| *** Suyash has joined #openstack-cinder | 19:21 | |
| jgriffith | I mean... obviously I didn't merge it | 19:21 |
| *** ChubYann has joined #openstack-cinder | 19:22 | |
| aorourke | jgriffith, it does work for you though? | 19:23 |
| smcginnis | Patchset 30? | 19:23 |
| jgriffith | smcginnis: just under principal I'm abandoning that series and putting it all up in one new one :) | 19:24 |
| smcginnis | Hah! | 19:24 |
| *** jwcroppe has quit IRC | 19:25 | |
| *** merooney has quit IRC | 19:26 | |
| *** merooney has joined #openstack-cinder | 19:29 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 (Cheesecake) https://review.openstack.org/275797 | 19:31 |
| jgriffith | 30 it is | 19:32 |
| smcginnis | Let's hope that's it. | 19:32 |
| Swanson | eeeeeeeeeeee | 19:32 |
| *** crose has joined #openstack-cinder | 19:32 | |
| smcginnis | Should be. I was about to hit +A. :) | 19:32 |
| Swanson | Die list_repl_targets, die! | 19:33 |
| jgriffith | :) | 19:34 |
| bswartz | Swanson: do you speak German? | 19:34 |
| openstackgerrit | Merged openstack/cinder: Avoid hardcoding value from oslo library https://review.openstack.org/284167 | 19:34 |
| Swanson | bswartz, no one who speaks german could be an evil man. | 19:34 |
| *** dustins_ has joined #openstack-cinder | 19:34 | |
| *** dustins has quit IRC | 19:35 | |
| bswartz | ... a cryptic answer | 19:35 |
| *** merooney has quit IRC | 19:36 | |
| smcginnis | bswartz: You gotta know Swanson's sense of humor. :) | 19:36 |
| smcginnis | bswartz: It's still cryptic, but makes slightly more sense. | 19:36 |
| *** baumann has joined #openstack-cinder | 19:36 | |
| *** dustins_ is now known as dustins | 19:38 | |
| *** mss-4 has joined #openstack-cinder | 19:39 | |
| *** mylu has quit IRC | 19:44 | |
| *** mylu has joined #openstack-cinder | 19:44 | |
| *** mylu has quit IRC | 19:46 | |
| openstackgerrit | Brandon Palm proposed openstack/python-cinderclient: Use instanceof instead of type https://review.openstack.org/284377 | 19:47 |
| *** esker has joined #openstack-cinder | 19:47 | |
| *** mylu has joined #openstack-cinder | 19:48 | |
| *** dustins has quit IRC | 19:50 | |
| *** sgotliv has joined #openstack-cinder | 19:52 | |
| openstackgerrit | Merged openstack/cinder: NetApp: Support iSCSI CHAP Uni-directional Auth https://review.openstack.org/251925 | 19:54 |
| openstackgerrit | Sheel Rana proposed openstack/cinder-specs: User facing error Summary Messages https://review.openstack.org/273938 | 19:55 |
| *** jgregor has quit IRC | 19:55 | |
| *** IlyaG has quit IRC | 19:56 | |
| *** jgregor has joined #openstack-cinder | 19:57 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder-specs: User facing error Summary Messages https://review.openstack.org/273938 | 19:59 |
| *** IlyaG has joined #openstack-cinder | 19:59 | |
| jgriffith | smcginnis: do you know what the status of multi-attach in Nova is currently? | 20:00 |
| *** 16WAAEDST has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-cinder | 20:01 | |
| jgriffith | smcginnis: ahh.. never mind | 20:03 |
| *** IlyaG has quit IRC | 20:04 | |
| *** ociuhandu has quit IRC | 20:04 | |
| *** jgregor has quit IRC | 20:04 | |
| Swanson | bswartz, smcginnis a simpsons reference. | 20:06 |
| *** leeantho has quit IRC | 20:08 | |
| *** jungleboyj has joined #openstack-cinder | 20:08 | |
| smcginnis | jgriffith: Figure it out? | 20:11 |
| *** jgregor has joined #openstack-cinder | 20:13 | |
| bswartz | ah | 20:14 |
| *** rhagarty_ has quit IRC | 20:16 | |
| *** rhagarty_ has joined #openstack-cinder | 20:16 | |
| *** dustins has joined #openstack-cinder | 20:17 | |
| Swanson | bswartz, but, no, I don't speak German. | 20:18 |
| *** mylu has quit IRC | 20:20 | |
| jgriffith | smcginnis: yeah, it hasn't merged still so I don't need to solve that problem "yet" | 20:21 |
| *** mylu has joined #openstack-cinder | 20:22 | |
| jgriffith | smcginnis: I was going to add a ref-counter to the volume_attachment table and start trying to work out a solution there for races etc | 20:22 |
| jgriffith | smcginnis: but I can kick the can down the road for now :) | 20:22 |
| haypo | "dulek> haypo: Python 3 tempest time? :D" yep, it's the obivous next step :) | 20:22 |
| haypo | dulek: i will probably work on that during the next Newton cycle | 20:23 |
| *** IlyaG has joined #openstack-cinder | 20:23 | |
| haypo | https://review.openstack.org/284308 nice, jenkins confirmed that all unit tests pass on python 3 | 20:24 |
| openstackgerrit | David Rosales proposed openstack/cinder: Change Fail to Failed in error messages https://review.openstack.org/284395 | 20:24 |
| *** darosale has joined #openstack-cinder | 20:28 | |
| *** anshul has quit IRC | 20:30 | |
| openstackgerrit | Mike Rooney proposed openstack/cinder: NetApp: Implement CGs for ONTAP Drivers https://review.openstack.org/283194 | 20:31 |
| *** jungleboyj has quit IRC | 20:35 | |
| openstackgerrit | Yucong Feng proposed openstack/cinder: Return all target_wwpns for FC storwize https://review.openstack.org/282541 | 20:52 |
| *** boris-42 has joined #openstack-cinder | 20:53 | |
| *** belmoreira has quit IRC | 20:53 | |
| *** jwcroppe has joined #openstack-cinder | 20:54 | |
| *** leeantho has joined #openstack-cinder | 20:57 | |
| *** rlrossit has quit IRC | 20:59 | |
| *** diablo_rojo has quit IRC | 20:59 | |
| *** haomaiwang has quit IRC | 21:01 | |
| *** diablo_rojo has joined #openstack-cinder | 21:01 | |
| *** haomaiwang has joined #openstack-cinder | 21:01 | |
| *** xyang1 has quit IRC | 21:02 | |
| *** timcl has quit IRC | 21:03 | |
| *** cknight has quit IRC | 21:04 | |
| *** martyturner has quit IRC | 21:04 | |
| *** akerr has quit IRC | 21:05 | |
| *** annasort has quit IRC | 21:08 | |
| *** mylu has quit IRC | 21:09 | |
| *** mylu has joined #openstack-cinder | 21:09 | |
| Swanson | jgriffith, 30 not the charm. ceph fail | 21:14 |
| *** rlrossit_ has joined #openstack-cinder | 21:19 | |
| *** lpetrut has quit IRC | 21:19 | |
| smcginnis | I think ceph has been failing. | 21:21 |
| jgriffith | That's nothing new for Ceph | 21:23 |
| hemna | :) | 21:23 |
| jbernard | hey now ;) | 21:23 |
| smcginnis | jgriffith: You've seen that pop up before. Or are you just talking in general? :) | 21:23 |
| jgriffith | jbernard: I didn't mean it that way :) | 21:23 |
| jbernard | jgriffith: i know ;) no worries | 21:23 |
| jgriffith | jbernard: just speaking statistically here :) | 21:23 |
| jbernard | jgriffith: yeah, turns out ci is not trivial | 21:24 |
| *** mylu has quit IRC | 21:24 | |
| jgriffith | last batch I looked at weren't even functionality related but deploy issues | 21:24 |
| jgriffith | jbernard: yeah.. who would've thought!! | 21:24 |
| jbernard | this one err's in the scheduler, 'no valid host was found' | 21:24 |
| *** baumann has left #openstack-cinder | 21:24 | |
| jbernard | i recheck *might* work | 21:24 |
| openstackgerrit | Yucong Feng proposed openstack/cinder: Return all target_wwpns for FC storwize https://review.openstack.org/282541 | 21:25 |
| jgriffith | jbernard: is it "recheck-ceph" ? | 21:26 |
| jgriffith | jbernard: or did you guys implement "run ceph" | 21:26 |
| *** salv-orl_ has joined #openstack-cinder | 21:27 | |
| jbernard | um… | 21:27 |
| jgriffith | or should I just push another change set :) | 21:27 |
| * jbernard checks | 21:27 | |
| eharney | you just run "recheck", but i think you have to wait for jenkins to finish and report first | 21:27 |
| smcginnis | Yep | 21:27 |
| smcginnis | Would be nice to be able to run individual gate tests. | 21:28 |
| jgriffith | eharney: yuk, so recheck *everything* not just Ceph :( | 21:28 |
| jgriffith | guess that makes sense since it's part of infra checks | 21:28 |
| smcginnis | Seems such as waste of resources to have to run through everything just for one issue. | 21:28 |
| jbernard | if there is a specific test feature, i haven't known about it | 21:28 |
| jgriffith | jbernard: yeah, makes sense given it's part of voting infra gate | 21:28 |
| jgriffith | I blame Swanson | 21:29 |
| jbernard | since we're not experimental anymore, i think it's just 'recheck' | 21:29 |
| jbernard | jgriffith: yep | 21:29 |
| mtanino | smcginnis: Thanks. I understand that we should remove Cinder from the component of manual bug. | 21:29 |
| smcginnis | jgriffith: Thats what I do. | 21:29 |
| jgriffith | smcginnis: smart man | 21:29 |
| *** salv-orlando has quit IRC | 21:29 | |
| smcginnis | mtanino: Yeah, usually you can just switch the project, but what you did works too. | 21:29 |
| mtanino | smcginnis: Ah, that's more smart... | 21:30 |
| mtanino | smarter. | 21:30 |
| smcginnis | mtanino: I'm assuming you're planning on submitting a doc patch since you didn't add notes and kept it assigned to you. | 21:30 |
| mtanino | smcginnis: That's is counted my closed bug :) | 21:31 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: LeftHand: Updating minimum client version https://review.openstack.org/267780 | 21:31 |
| smcginnis | mtanino: ;) | 21:32 |
| *** cknight has joined #openstack-cinder | 21:33 | |
| *** [1]Thelo has joined #openstack-cinder | 21:33 | |
| *** ekarlso- has quit IRC | 21:34 | |
| *** ekarlso- has joined #openstack-cinder | 21:34 | |
| *** Thelo has quit IRC | 21:35 | |
| *** [1]Thelo is now known as Thelo | 21:35 | |
| *** dustins has quit IRC | 21:36 | |
| *** darosale has quit IRC | 21:37 | |
| *** cknight1 has joined #openstack-cinder | 21:37 | |
| *** dustins has joined #openstack-cinder | 21:38 | |
| *** diablo_rojo has quit IRC | 21:39 | |
| *** cknight has quit IRC | 21:39 | |
| *** krtaylor has quit IRC | 21:41 | |
| *** mtanino__ has joined #openstack-cinder | 21:41 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/283923 | 21:43 |
| * DuncanT finds his giant set of notes on which drivers hit the db directly and why.... I wonder how many of those are still valid? | 21:44 | |
| smcginnis | DuncanT: I do think a few have gotten off your naughtly list. | 21:44 |
| *** jgregor has quit IRC | 21:44 | |
| *** ndipanov has quit IRC | 21:45 | |
| *** jgregor has joined #openstack-cinder | 21:45 | |
| DuncanT | Looks like I did this about six months ago. I'll update it and post it as bugs | 21:45 |
| smcginnis | +1 | 21:47 |
| *** diablo_rojo has joined #openstack-cinder | 21:47 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/282768 | 21:49 |
| *** darosale has joined #openstack-cinder | 21:49 | |
| *** mylu has joined #openstack-cinder | 21:53 | |
| *** krtaylor has joined #openstack-cinder | 21:55 | |
| *** gouthamr has quit IRC | 21:57 | |
| *** porrua has quit IRC | 21:57 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Trying latest oslo.* from master https://review.openstack.org/284150 | 21:57 |
| *** jwcroppe has quit IRC | 21:57 | |
| *** haomaiwang has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-cinder | 22:01 | |
| openstackgerrit | Jacob Gregor proposed openstack/cinder: WIP: Storwize SVC multiple management IPs https://review.openstack.org/284267 | 22:05 |
| *** knikolla has quit IRC | 22:05 | |
| openstackgerrit | Michael Price proposed openstack/cinder: Implement CHAP Authentication for E-Series Driver https://review.openstack.org/246990 | 22:07 |
| *** thingee has quit IRC | 22:09 | |
| *** jgregor has quit IRC | 22:12 | |
| *** dims has quit IRC | 22:14 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 (Cheesecake) https://review.openstack.org/275797 | 22:15 |
| jgriffith | Swanson: and that's the LAST one!!! | 22:15 |
| smcginnis | I've heard that before. | 22:16 |
| jgriffith | smcginnis: hehe.. yeah, like 20 times :) | 22:16 |
| smcginnis | jgriffith: Did you find an issue that wasn't just a need for a recheck? | 22:16 |
| *** dustins has quit IRC | 22:17 | |
| *** raildo is now known as raildo-afk | 22:19 | |
| *** adrianofr_ has quit IRC | 22:20 | |
| Swanson | jgriffith, 31 flavors of cheesecake? | 22:23 |
| *** gouthamr has joined #openstack-cinder | 22:23 | |
| jgriffith | smcginnis: yeah :( | 22:24 |
| jgriffith | smcginnis: pylint non-voting found a bogus exception I was raising | 22:24 |
| jgriffith | smcginnis: so I changed it and made it a valid/existing one | 22:24 |
| *** crose has quit IRC | 22:25 | |
| guitarzan | is that name cheesecake just there to make us way more happy about this? | 22:25 |
| guitarzan | because honestly, cheesecake is pretty amazing | 22:25 |
| *** gouthamr_ has joined #openstack-cinder | 22:26 | |
| *** tpsilva has quit IRC | 22:27 | |
| *** gouthamr has quit IRC | 22:28 | |
| smcginnis | jgriffith: Nice! | 22:28 |
| smcginnis | guitarzan: Tasty! ;) | 22:29 |
| jgriffith | Next up.. "chicken and waffles" | 22:29 |
| *** haplo37 has quit IRC | 22:29 | |
| *** Thelo has quit IRC | 22:30 | |
| ntpttr | If anyone has a moment, mind giving this patch a quick look? It's pretty old and has one +2 already https://review.openstack.org/#/c/252476/ | 22:32 |
| *** julim has quit IRC | 22:33 | |
| *** delattec has joined #openstack-cinder | 22:34 | |
| *** mylu has quit IRC | 22:36 | |
| *** cdelatte has quit IRC | 22:37 | |
| *** mylu has joined #openstack-cinder | 22:38 | |
| smcginnis | jgriffith: Or maybe "shrimp and grits"! ;) | 22:40 |
| jgriffith | smcginnis: I"m working on that one right now! | 22:41 |
| smcginnis | haha | 22:41 |
| jgriffith | smcginnis: so it's funny, since the meetup, there seems to be a southern theme at every restaraunt... turns out the best shrimp and grits I've ever had was up in Steamboat the other week-end | 22:42 |
| jgriffith | AND.. I had the best beer I've ever had | 22:42 |
| *** lprice has quit IRC | 22:42 | |
| jgriffith | Coffee stout aged in a whiskey barrel :) | 22:42 |
| smcginnis | Wow, I would not expect CO to be the place to go for that. | 22:42 |
| jgriffith | Trifecta!!! | 22:42 |
| smcginnis | Oooh, the beer I can believe. That sounds goood. | 22:42 |
| jgriffith | I"m moving there | 22:42 |
| smcginnis | How far is that from Ft Collins/Boulder area? | 22:43 |
| jgriffith | between the ranch land, skiing and now the food and beer.. I'm sold | 22:43 |
| jgriffith | smcginnis: it's about 4 hours west of Fort Collins | 22:43 |
| smcginnis | Nice. | 22:43 |
| jgriffith | http://www.steamboatchamber.com/signature-events/winter-carnival | 22:44 |
| smcginnis | I'd be good with the mountains, skiing, and the food and beer, but I wouldn't turn down a ranch. | 22:44 |
| smcginnis | Oh, nice. | 22:44 |
| *** aorourke has quit IRC | 22:44 | |
| jgriffith | oh.. and then there's the fly fishing and mtn biking all summer (albeit summer is kinda short) | 22:44 |
| smcginnis | My friend in Lyon's sent me some pics from something like that there. | 22:44 |
| smcginnis | Pfft, don't talk to me about summer. | 22:45 |
| smcginnis | I got about 5 mtn bike rides in last year. | 22:45 |
| jgriffith | smcginnis: haha | 22:45 |
| smcginnis | This is MN, so I should probably put quotes around "mtn". | 22:45 |
| jgriffith | smcginnis: Lyons is a great place.. just up the road from here | 22:45 |
| jgriffith | lol | 22:45 |
| *** aorourke has joined #openstack-cinder | 22:45 | |
| smcginnis | jgriffith: Yeah, the Ft Collins midcycles work great for me. I get to visit them after. | 22:46 |
| smcginnis | Good excuse to get out there. | 22:46 |
| jgriffith | smcginnis: nice! | 22:47 |
| *** jdillaman has quit IRC | 22:48 | |
| *** arch-nemesis has quit IRC | 22:51 | |
| *** mss-4 has quit IRC | 22:54 | |
| *** bardia_ has joined #openstack-cinder | 22:57 | |
| *** bardia has quit IRC | 22:57 | |
| *** sheel has quit IRC | 22:57 | |
| *** mriedem has quit IRC | 23:00 | |
| *** jdillaman has joined #openstack-cinder | 23:00 | |
| hemna | I'd love Ft. Collins again :) | 23:00 |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwang has joined #openstack-cinder | 23:01 | |
| *** IlyaG has quit IRC | 23:01 | |
| smcginnis | It's nice and central at least. | 23:02 |
| hemna | yup, and there is good beer, and great scenery and fishing | 23:02 |
| hemna | :P | 23:02 |
| patrickeast | jgriffith: hey, is that latest replication patch good to go? | 23:03 |
| * patrickeast is finally actually getting around to testing | 23:03 | |
| smcginnis | patrickeast: And the one before it. And the one before that. | 23:03 |
| smcginnis | :P | 23:03 |
| patrickeast | lol | 23:03 |
| jgriffith | patrickeast: too late.. merge it :) | 23:03 |
| patrickeast | jgriffith: haha, can always backport any fixes right? :D | 23:04 |
| smcginnis | Looking a couple hours out yet with how things are going with Jenkins. | 23:04 |
| jgriffith | patrickeast: seriously though... yes, I've at least finished everything I had planned to | 23:04 |
| jgriffith | patrickeast: yup!! | 23:04 |
| jgriffith | patrickeast: there's probably going to be one more version to bump rpc | 23:04 |
| jgriffith | patrickeast: but other than that that should be about it | 23:04 |
| jgriffith | unless somebody finds something else | 23:04 |
| patrickeast | jgriffith: sweet | 23:04 |
| patrickeast | jgriffith: i've gotta update my driver code a tad with the replication targets stuff and have the rest of my afternoon dedicated to it | 23:05 |
| patrickeast | jgriffith: although i anticipate that its going to be great | 23:06 |
| jgriffith | patrickeast: cool.. should be easy.. just two lines in your volume_stats | 23:06 |
| *** sgotliv has quit IRC | 23:06 | |
| patrickeast | yep yep | 23:06 |
| jgriffith | patrickeast: I hope so | 23:06 |
| jgriffith | patrickeast: I'm pretty happy with it, hope others are | 23:06 |
| *** angela-s has quit IRC | 23:07 | |
| patrickeast | jgriffith: have you had any time to get the solidfire driver updated to use it? | 23:07 |
| *** dims has joined #openstack-cinder | 23:08 | |
| *** diablo_rojo has quit IRC | 23:08 | |
| jgriffith | patrickeast: haha.. there's that whole thing again :) | 23:10 |
| *** rlrossit_ has quit IRC | 23:10 | |
| jgriffith | patrickeast: I have various hacks here and there just to test out what I wrote, but no... I don't plan to release it this go around | 23:10 |
| jgriffith | we'll see... maybe I won't be able to sleep tonight :) | 23:10 |
| jgriffith | patrickeast: It's easy enough to post a backport in a private repo if I have customers that want it | 23:11 |
| patrickeast | jgriffith: dang, well, theres still some time since we had said we were treating those updates like bug fixes too | 23:11 |
| jgriffith | patrickeast: oh.. I didn't catch that. | 23:11 |
| patrickeast | jgriffith: true, but thats always a pain | 23:11 |
| jgriffith | patrickeast: maybe I'll finish this attach stuff up and get to it next | 23:12 |
| jgriffith | patrickeast: but I PROMISE I'm not object to anyone else merging their versions | 23:12 |
| patrickeast | jgriffith: haha | 23:12 |
| jgriffith | :) | 23:12 |
| jgriffith | You know what's awesome... is the way we persist data in at least 3 places for some things | 23:13 |
| jgriffith | metadata, and two different tables in the DB | 23:13 |
| *** jgregor has joined #openstack-cinder | 23:13 | |
| jgriffith | for when you really, really, really want to persist some data! | 23:14 |
| openstackgerrit | Angus Lees proposed openstack/cinder: Add os-brick rootwrap filter for privsep https://review.openstack.org/280030 | 23:15 |
| openstackgerrit | Angus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 23:15 |
| patrickeast | hah, guess we're shooting for some sort of robust db table protection by mirroring columns on other tables :p | 23:16 |
| jgriffith | patrickeast: ha!!! | 23:16 |
| *** edtubill has quit IRC | 23:16 | |
| jgriffith | patrickeast: just shows NONE of us really know all the nooks and crannies in here any more | 23:17 |
| * jgriffith included | 23:17 | |
| hemna | jgriffith, +1 | 23:17 |
| patrickeast | jgriffith: yea, every time i feel like i understand how things are working I wander into a new section of cinder and realize its a complete mystery | 23:17 |
| patrickeast | i really enjoyed reviewing the microversion stuff for that, beforehand only had a vague idea of how our api's work | 23:17 |
| jgriffith | wOOt!! New attach code is all working | 23:18 |
| jgriffith | hemna: you now have a connector to use for detach if you so desire | 23:18 |
| jgriffith | hemna: without changing any API's | 23:18 |
| hemna | oh?! | 23:18 |
| jgriffith | hemna: yeah, going to clean up a few things and push it up for review | 23:19 |
| hemna | sweet | 23:19 |
| * hemna is stoked | 23:19 | |
| jgriffith | hemna: follow up work will be going through the detach paths | 23:19 |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: api-microversion support for python-cinderclient https://review.openstack.org/248163 | 23:19 |
| jgriffith | hemna: this will be the foundation to get you what you needed/wanted though | 23:19 |
| hemna | ok cool | 23:19 |
| hemna | thanks man | 23:19 |
| jgriffith | hemna: I also have a proposal for Nova to greatly simplify the whole multi-attach thing for ALL of us | 23:19 |
| jgriffith | hemna: I'll elaborate after we get through this deal | 23:20 |
| hemna | ok, sounds good | 23:20 |
| scottda | Cursed rebase! | 23:21 |
| hemna | smcginnis, patrickeast my apt-mirror server burned up in flames | 23:21 |
| hemna | heh | 23:21 |
| hemna | before we could even used it for CI | 23:21 |
| patrickeast | :o | 23:22 |
| patrickeast | what happened? | 23:22 |
| hemna | luckily I used a volume on my array to store mirrored files | 23:22 |
| hemna | not sure really. rebooted it because of a network hiccup/oddity....it never came back up | 23:22 |
| *** darosale has quit IRC | 23:22 | |
| hemna | which was probably the root of the network issue | 23:23 |
| hemna | so, for a test, we are going to inject some lines of sed into our custom functions prior to standing up devstack | 23:23 |
| *** eharney has quit IRC | 23:23 | |
| hemna | to update the /etc/apt/sources.list to point to our mirrog | 23:23 |
| hemna | and see if that fixes some of our lag and 503 issues with ubuntu | 23:23 |
| *** jeanfabrice has joined #openstack-cinder | 23:28 | |
| *** vincent_hou has joined #openstack-cinder | 23:30 | |
| jeanfabrice | Hi. I'm in trouble (devstack/juno), trying to set osapi_volume_base_URL w/o success. Cinder-api debug says the value is correctly set, but when I curl the root of the api, I don't get this base_URL in the href links | 23:31 |
| jeanfabrice | any hints ? | 23:31 |
| *** IlyaG has joined #openstack-cinder | 23:36 | |
| *** hemna is now known as hemnafk | 23:41 | |
| *** mylu has quit IRC | 23:44 | |
| *** IlyaG has quit IRC | 23:45 | |
| *** mylu has joined #openstack-cinder | 23:48 | |
| *** smoriya_ has joined #openstack-cinder | 23:50 | |
| *** mylu has quit IRC | 23:50 | |
| openstackgerrit | Angus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 23:52 |
| *** yhayashi has joined #openstack-cinder | 23:55 | |
| *** merooney has joined #openstack-cinder | 23:55 | |
| *** vincent_hou has quit IRC | 23:58 | |
| *** merooney_ has joined #openstack-cinder | 23:58 | |
| *** vincent_hou has joined #openstack-cinder | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!