| patrickeast | and i think its doing the right thing, http://ec2-54-67-51-189.us-west-1.compute.amazonaws.com/88/186588/19/check/PureISCSIDriver-tempest-dsvm-volume-multipath-brick/e453ab2/logs/devstacklog.txt.gz#_2015-06-10_22_09_21_252 | 00:01 |
|---|---|---|
| patrickeast | cloning os-brick from git | 00:01 |
| *** smoriya has joined #openstack-cinder | 00:02 | |
| *** xiaohui has joined #openstack-cinder | 00:06 | |
| *** lixiaoy1 has joined #openstack-cinder | 00:07 | |
| *** belmoreira has joined #openstack-cinder | 00:11 | |
| *** leeantho has quit IRC | 00:12 | |
| asselin | cool! | 00:15 |
| *** annashen has quit IRC | 00:16 | |
| tsekiyama | patrickeast: I'm reading the Cinder internal tenant" spec, and have a question.. Why wouldn't the config be just "cinder_internal_tenant_id", but tenant_user/password? | 00:16 |
| *** adurbin_ has quit IRC | 00:17 | |
| patrickeast | tsekiyama: i’ve been wondering that myself actually | 00:17 |
| *** setmason has quit IRC | 00:17 | |
| patrickeast | tsekiyama: the more i think about it, the more i don’t know why we would actually need the password | 00:17 |
| patrickeast | tsekiyama: just the id | 00:17 |
| tsekiyama | patrickeast: yeah,, inside the Cinder-volume, user/password wouldn't be used | 00:18 |
| patrickeast | tsekiyama: for now i’ll assume we need both, but i might end up putting a patch in to modify the spec if we end up not needing it | 00:18 |
| tsekiyama | patrickeast: ok, I understand the current situation. | 00:18 |
| tsekiyama | patrickeast: I believe the tenant-id is what we need though | 00:19 |
| patrickeast | yea, i agree | 00:19 |
| openstackgerrit | Alexey Khodos proposed openstack/cinder: NexentaStor 5 NFS backend driver. https://review.openstack.org/190273 | 00:20 |
| *** heyun has joined #openstack-cinder | 00:26 | |
| *** dannywil_ has joined #openstack-cinder | 00:30 | |
| *** diogogmt has quit IRC | 00:31 | |
| *** dannywilson has quit IRC | 00:33 | |
| openstackgerrit | Alexey Khodos proposed openstack/cinder: NexentaStor 5 NFS backend driver. https://review.openstack.org/190391 | 00:34 |
| *** dannywil_ has quit IRC | 00:35 | |
| *** alexpilotti has quit IRC | 00:36 | |
| *** tsekiyama has quit IRC | 00:36 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 00:41 |
| *** mtanino has quit IRC | 00:44 | |
| *** IanGovett has joined #openstack-cinder | 00:47 | |
| *** annashen has joined #openstack-cinder | 00:47 | |
| *** heyun has quit IRC | 00:48 | |
| *** heyun has joined #openstack-cinder | 00:49 | |
| *** annashen has quit IRC | 00:54 | |
| *** akl__ has quit IRC | 00:56 | |
| *** _cjones_ has quit IRC | 00:57 | |
| *** diogogmt has joined #openstack-cinder | 01:09 | |
| *** chlong has quit IRC | 01:11 | |
| *** eantyshev has quit IRC | 01:15 | |
| *** dims_ has joined #openstack-cinder | 01:18 | |
| *** lcurtis has quit IRC | 01:18 | |
| *** dims has quit IRC | 01:21 | |
| *** tobe has joined #openstack-cinder | 01:24 | |
| *** Lee1092 has joined #openstack-cinder | 01:30 | |
| *** IanGovett has quit IRC | 01:39 | |
| *** vilobhmm has quit IRC | 01:41 | |
| *** IlyaG has joined #openstack-cinder | 01:42 | |
| *** IlyaG has quit IRC | 01:46 | |
| *** IlyaG has joined #openstack-cinder | 01:47 | |
| *** belmoreira has quit IRC | 01:50 | |
| *** marcusvrn has joined #openstack-cinder | 02:01 | |
| *** gouthamr has joined #openstack-cinder | 02:03 | |
| *** Apoorva has quit IRC | 02:06 | |
| *** rmstar_ is now known as rmstar | 02:11 | |
| *** lixiaoy1 has quit IRC | 02:11 | |
| openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: Using 'fallocate' instead of 'dd' to create raw volumes https://review.openstack.org/164527 | 02:11 |
| *** jungleboyj has joined #openstack-cinder | 02:11 | |
| *** lixiaoy1 has joined #openstack-cinder | 02:12 | |
| *** IlyaG has quit IRC | 02:32 | |
| *** ian_brown has joined #openstack-cinder | 02:32 | |
| *** dims_ has quit IRC | 02:33 | |
| *** nikeshm has quit IRC | 02:36 | |
| *** tobe has quit IRC | 02:46 | |
| *** tobe has joined #openstack-cinder | 02:47 | |
| *** vilobhmm has joined #openstack-cinder | 02:47 | |
| *** BharatK has quit IRC | 02:48 | |
| *** tobe has quit IRC | 02:50 | |
| *** tobe has joined #openstack-cinder | 02:51 | |
| *** setmason has joined #openstack-cinder | 02:58 | |
| *** gouthamr has quit IRC | 02:59 | |
| *** vilobhmm has quit IRC | 02:59 | |
| *** david-lyle has joined #openstack-cinder | 03:04 | |
| *** nikeshm has joined #openstack-cinder | 03:04 | |
| *** sgotliv_ has joined #openstack-cinder | 03:05 | |
| *** nikeshm has quit IRC | 03:09 | |
| *** ronenkat_ has quit IRC | 03:09 | |
| *** nikeshm has joined #openstack-cinder | 03:12 | |
| *** dims has joined #openstack-cinder | 03:33 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for L https://review.openstack.org/186327 | 03:45 |
| *** fifieldt has joined #openstack-cinder | 03:58 | |
| *** ronenkat has joined #openstack-cinder | 03:58 | |
| *** gouthamr has joined #openstack-cinder | 04:00 | |
| *** tobe has quit IRC | 04:04 | |
| *** yamada-h has joined #openstack-cinder | 04:05 | |
| *** bill_az has quit IRC | 04:07 | |
| jgriffith | Swanson: Good question: https://review.openstack.org/#/c/190291/2/cinder/volume/targets/fibre_channel.py | 04:08 |
| *** yamada-h has quit IRC | 04:08 | |
| jgriffith | Swanson: check out my answer inline | 04:08 |
| *** yamada-h has joined #openstack-cinder | 04:10 | |
| *** yamada-h has quit IRC | 04:10 | |
| *** tobe has joined #openstack-cinder | 04:13 | |
| *** annashen has joined #openstack-cinder | 04:22 | |
| *** marcusvrn has quit IRC | 04:24 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:25 | |
| *** markvoelker has quit IRC | 04:26 | |
| *** annashen has quit IRC | 04:27 | |
| *** yamada-h has joined #openstack-cinder | 04:31 | |
| *** gouthamr has quit IRC | 04:31 | |
| jgriffith | patrickeast: what's this: http://ec2-54-67-51-189.us-west-1.compute.amazonaws.com/79/186879/3/check/PureFCDriver-tempest-dsvm-volume-multipath/6d9f6fd/logs/screen-c-api.txt.gz#_2015-05-30_02_18_18_720 ? | 04:34 |
| patrickeast | jgriffith: uhhh | 04:34 |
| patrickeast | jgriffith: good question | 04:34 |
| patrickeast | jgriffith: i don’t remember setting anything for quota in the configs anywhere | 04:35 |
| jgriffith | hehe.. yeah, I didn't see anything like that either | 04:35 |
| jgriffith | weird | 04:35 |
| patrickeast | jgriffith: i want to get something like logstash (or maybe even the actual infra logstack) for my ci logs | 04:36 |
| patrickeast | jgriffith: would be cool to search for that kind of stuff in them | 04:36 |
| patrickeast | jgriffith: see when that type of thing started showing up | 04:36 |
| jgriffith | mtreinish: BTW, the culprit in my ostestr/subunit patch is debug loggging from migrate.versioning.script.base | 04:37 |
| jgriffith | mtreinish: but I can't figure out how to *fix* it | 04:37 |
| mtreinish | jgriffith man, even when I'm on the other side of the world your always online when I am | 04:37 |
| jgriffith | patrickeast: +1 for logstash | 04:37 |
| jgriffith | mtreinish: haha! | 04:37 |
| jgriffith | mtreinish: I've been wondering where you were at! | 04:37 |
| mtreinish | jgriffith: hmm it's probably the sdout pass through. Is that's what's causing the looping though? | 04:37 |
| asselin_ | patrickeast, another thing in my backlog.... | 04:37 |
| mtreinish | jgriffith: japan until sat. then israel next week | 04:38 |
| *** diogogmt has quit IRC | 04:38 | |
| jgriffith | mtreinish: I'm not sure about the looping, but it seems it's causing the subunit failure for too many chars or whatever it was | 04:38 |
| patrickeast | asselin_: i’ve been meaning to ask in -infra what they thought about letting 3rd party ci’s put logs into the infra logstash | 04:38 |
| jgriffith | mtreinish: Wow! Racking up the frequent flyer miles | 04:38 |
| mtreinish | jgriffith: heh, yeah something like that :) | 04:38 |
| asselin_ | patrickeast, my guess is no-way-jose | 04:38 |
| jgriffith | patrickeast: that could be tricky... storage and all :) | 04:38 |
| patrickeast | well yea | 04:38 |
| mtreinish | jgriffith: hmm, I thought that was an issue with the subunit-1to2 calling at the end of the script | 04:38 |
| patrickeast | but like how much no way | 04:39 |
| jgriffith | maybe if you send them a Pure :) | 04:39 |
| patrickeast | like | 04:39 |
| patrickeast | hard to do and maybe later | 04:39 |
| mtreinish | although I pushed a change to run_tox.sh which should have fixed that a couple weeks ago | 04:39 |
| patrickeast | or never no way we would die first | 04:39 |
| jgriffith | mtreinish: maybe I should look again... | 04:39 |
| mtreinish | jgriffith: fwiw: https://review.openstack.org/186809 | 04:39 |
| jgriffith | mtreinish: hmm... so now I'm confused | 04:40 |
| jgriffith | mtreinish: I'm not seeing the message I was looking for earlier | 04:40 |
| *** gouthamr has joined #openstack-cinder | 04:41 | |
| jgriffith | mtreinish: but I've got several hundred lines of DEBUG output from migrate | 04:41 |
| mtreinish | jgriffith: heh, well my patch should have fixed that, because it was a subunit bug (which I still need to file) | 04:41 |
| jgriffith | mtreinish: and the final "Length too long: 5431171" message | 04:41 |
| mtreinish | IIRC, calling subunit-1to2 is what was generting that message | 04:41 |
| jgriffith | mtreinish: I'll update things and see what happens | 04:42 |
| jgriffith | mtreinish: thanks | 04:42 |
| *** changbl has joined #openstack-cinder | 04:42 | |
| jgriffith | and safe travels :) | 04:42 |
| jgriffith | I'm finding more little "nuggest" running tests this evening than I'd care to | 04:42 |
| *** gouthamr has quit IRC | 04:43 | |
| mtreinish | jgriffith: heh, yeah I can relate to that | 04:45 |
| jgriffith | mtreinish: so quick question :) | 04:46 |
| jgriffith | mtreinish: how can I suck in the newer subunit in my tox run? | 04:46 |
| asselin_ | patrickeast, we can look at setting up a separate one for all of 3rd party ci and see....but it would likely be for 'm' at this point | 04:46 |
| * asselin_ signs off for the night | 04:46 | |
| mtreinish | jgriffith: newer subunit? like from pypi? | 04:46 |
| jgriffith | mtreinish: because the tox install of test-reqs is just going to get pypi's version which is "old" right? | 04:47 |
| jgriffith | mtreinish: yeah.. like that | 04:47 |
| jgriffith | mtreinish: ie to get your fix you just pointed me to :) | 04:47 |
| * jgriffith thinks sometimes automation is *hard* | 04:47 | |
| mtreinish | oh, that's in the infra run script it shouldn't need any newer package version | 04:47 |
| jgriffith | Oh... crud | 04:48 |
| mtreinish | but to do that I normally just activate the tox venv and manually run the install commands | 04:48 |
| jgriffith | yeah | 04:48 |
| jgriffith | I wasn't paying attention there | 04:48 |
| jgriffith | mtreinish: OH, no kidding, I didn't know tox let ya do that | 04:48 |
| jgriffith | I learned something valuable/new tonight | 04:48 |
| mtreinish | jgriffith: yeah normally something like 'source .tox/py27/bin/activate' | 04:49 |
| mtreinish | jgriffith: all that change does is switch to using 'testr last --subunit' to just write a subunit v2 stream directly instead of running subunit-1to2 on the stream in .testrepository | 04:49 |
| jgriffith | mtreinish: hey... look at that! Way cool | 04:49 |
| *** harlowja_at_home has quit IRC | 04:51 | |
| jgriffith | hmmm | 04:52 |
| * jgriffith is apparantly not very sharp | 04:52 | |
| jgriffith | ls bin | 04:53 |
| jgriffith | ahhh | 04:53 |
| jgriffith | I can't quite figure out how we tie all this together on the Cinder side | 04:54 |
| jgriffith | think I'm going to quit for the night | 04:54 |
| *** haomaiw__ has joined #openstack-cinder | 04:55 | |
| *** haomaiwang has quit IRC | 04:56 | |
| *** tobe has quit IRC | 04:56 | |
| jgriffith | Ohh.. DERP | 04:56 |
| jgriffith | yeah, so I don't have any of that because I'm not using ostestr to call everything | 04:57 |
| jgriffith | Ohhhh neat! | 04:59 |
| mtreinish | jgriffith: heh, did you figure it out? | 05:00 |
| *** tobe has joined #openstack-cinder | 05:01 | |
| jgriffith | mtreinish: just discovering stuff I can do with ostestr directly in the venv | 05:01 |
| jgriffith | mtreinish: and I did manage to do "something" that makes the problem go away | 05:01 |
| jgriffith | but I don't know what it was :) | 05:01 |
| mtreinish | heh, ok | 05:02 |
| jgriffith | Ok.. I've got to stop, I'm just getting distracted at this pont | 05:02 |
| jgriffith | I'm out of here | 05:02 |
| jgriffith | have a good night everyone | 05:02 |
| jgriffith | cya tomorrow | 05:02 |
| *** shausy has joined #openstack-cinder | 05:09 | |
| *** patrickeast has quit IRC | 05:11 | |
| *** deepakcs has joined #openstack-cinder | 05:12 | |
| *** setmason has quit IRC | 05:14 | |
| *** yamada-h has quit IRC | 05:16 | |
| *** davechen has quit IRC | 05:17 | |
| *** yamada-h has joined #openstack-cinder | 05:18 | |
| *** belmoreira has joined #openstack-cinder | 05:22 | |
| *** yamada-h has quit IRC | 05:22 | |
| *** setmason has joined #openstack-cinder | 05:23 | |
| *** markvoelker has joined #openstack-cinder | 05:27 | |
| *** markvoelker has quit IRC | 05:32 | |
| openstackgerrit | Ph. Marek proposed openstack/cinder: Re-add DRBD driver. https://review.openstack.org/178573 | 05:34 |
| *** nkrinner has joined #openstack-cinder | 05:34 | |
| *** aswadr has joined #openstack-cinder | 05:34 | |
| *** belmoreira has quit IRC | 05:38 | |
| *** nihilifer has joined #openstack-cinder | 05:38 | |
| *** setmason has quit IRC | 05:41 | |
| *** rushiagr_away is now known as rushiagr | 05:41 | |
| *** dannywilson has joined #openstack-cinder | 05:48 | |
| *** dannywilson has quit IRC | 05:48 | |
| *** dannywilson has joined #openstack-cinder | 05:48 | |
| *** lpetrut has joined #openstack-cinder | 05:49 | |
| *** nlevinki has joined #openstack-cinder | 05:49 | |
| *** coolsvap|afk is now known as coolsvap | 05:50 | |
| *** haomaiw__ has quit IRC | 05:51 | |
| openstackgerrit | Jessy Lee proposed openstack/cinder: Add volume drivers for Infortrend Storage https://review.openstack.org/177113 | 05:51 |
| *** vilobhmm has joined #openstack-cinder | 05:57 | |
| *** haomaiwang has joined #openstack-cinder | 05:57 | |
| *** BharatK has joined #openstack-cinder | 06:00 | |
| *** mdbooth has quit IRC | 06:00 | |
| *** mdbooth has joined #openstack-cinder | 06:02 | |
| *** BharatK has quit IRC | 06:05 | |
| *** yamada-h has joined #openstack-cinder | 06:15 | |
| *** yamada-h has quit IRC | 06:15 | |
| *** BharatK has joined #openstack-cinder | 06:17 | |
| *** haomaiwang has quit IRC | 06:18 | |
| *** haomaiwa_ has joined #openstack-cinder | 06:19 | |
| *** annashen has joined #openstack-cinder | 06:24 | |
| *** markvoelker has joined #openstack-cinder | 06:28 | |
| *** annashen has quit IRC | 06:29 | |
| *** markvoelker has quit IRC | 06:32 | |
| *** ian_brown has quit IRC | 06:33 | |
| *** aswadr has quit IRC | 06:35 | |
| *** rushiagr is now known as rushiagr_away | 06:50 | |
| *** e0ne has joined #openstack-cinder | 07:08 | |
| *** markus_z has joined #openstack-cinder | 07:12 | |
| *** e0ne has quit IRC | 07:12 | |
| *** deepakcs has quit IRC | 07:13 | |
| *** yamada-h has joined #openstack-cinder | 07:13 | |
| *** anshul has joined #openstack-cinder | 07:14 | |
| *** yamada-h has quit IRC | 07:19 | |
| *** vilobhmm has quit IRC | 07:21 | |
| *** jordanP has joined #openstack-cinder | 07:22 | |
| *** vilobhmm has joined #openstack-cinder | 07:23 | |
| *** ndipanov has quit IRC | 07:26 | |
| *** e0ne has joined #openstack-cinder | 07:28 | |
| openstackgerrit | Alexey Khodos proposed openstack/cinder: NexentaStor 5 iSCSI backend driver. https://review.openstack.org/190391 | 07:29 |
| *** e0ne has quit IRC | 07:29 | |
| *** yamada-h has joined #openstack-cinder | 07:32 | |
| *** yamada-h has quit IRC | 07:32 | |
| *** jistr has joined #openstack-cinder | 07:33 | |
| *** yamada-h has joined #openstack-cinder | 07:36 | |
| *** yamada-h has quit IRC | 07:36 | |
| *** dannywilson has quit IRC | 07:38 | |
| *** takedakn has joined #openstack-cinder | 07:46 | |
| *** vilobhmm has quit IRC | 07:46 | |
| *** dannywilson has joined #openstack-cinder | 07:46 | |
| *** dannywilson has quit IRC | 07:51 | |
| *** dannywilson has joined #openstack-cinder | 07:51 | |
| *** lpetrut has quit IRC | 07:52 | |
| *** takedakn has quit IRC | 07:52 | |
| *** dannywilson has quit IRC | 07:56 | |
| openstackgerrit | Alexey Khodos proposed openstack/cinder: NexentaStor 5 NFS backend driver. https://review.openstack.org/190273 | 08:02 |
| *** yamada-h has joined #openstack-cinder | 08:11 | |
| *** yamada-h has quit IRC | 08:11 | |
| *** yamada-h has joined #openstack-cinder | 08:13 | |
| *** yamada-h has quit IRC | 08:13 | |
| *** yamada-h has joined #openstack-cinder | 08:14 | |
| *** yamada-h has quit IRC | 08:15 | |
| *** markvoelker has joined #openstack-cinder | 08:16 | |
| openstackgerrit | Alexey Khodos proposed openstack/cinder: NexentaStor 5 iSCSI backend driver. https://review.openstack.org/190391 | 08:17 |
| *** yamada-h has joined #openstack-cinder | 08:18 | |
| *** yamada-h has quit IRC | 08:18 | |
| *** markvoelker has quit IRC | 08:21 | |
| *** dulek has joined #openstack-cinder | 08:24 | |
| *** yamada-h has joined #openstack-cinder | 08:25 | |
| *** yamada-h has quit IRC | 08:25 | |
| *** annashen has joined #openstack-cinder | 08:25 | |
| *** jistr has quit IRC | 08:27 | |
| *** yamada-h has joined #openstack-cinder | 08:29 | |
| *** yamada-h has quit IRC | 08:29 | |
| *** annashen has quit IRC | 08:30 | |
| *** ronenkat has quit IRC | 08:30 | |
| *** yamada-h has joined #openstack-cinder | 08:31 | |
| *** yamada-h has quit IRC | 08:31 | |
| *** yamada-h has joined #openstack-cinder | 08:31 | |
| *** yamada-h has quit IRC | 08:31 | |
| *** yamada-h has joined #openstack-cinder | 08:33 | |
| *** yamada-h has quit IRC | 08:33 | |
| *** yamada-h has joined #openstack-cinder | 08:34 | |
| *** yamada-h has quit IRC | 08:34 | |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 08:34 |
| *** yamada-h has joined #openstack-cinder | 08:35 | |
| *** yamada-h has quit IRC | 08:35 | |
| *** ronenkat has joined #openstack-cinder | 08:36 | |
| *** yamada-h has joined #openstack-cinder | 08:36 | |
| *** yamada-h has quit IRC | 08:36 | |
| *** yamada-h has joined #openstack-cinder | 08:38 | |
| *** yamada-h has quit IRC | 08:38 | |
| *** yamada-h has joined #openstack-cinder | 08:39 | |
| *** yamada-h has quit IRC | 08:39 | |
| *** yamada-h has joined #openstack-cinder | 08:41 | |
| *** yamada-h has quit IRC | 08:41 | |
| *** afazekas has joined #openstack-cinder | 08:41 | |
| *** yamada-h has joined #openstack-cinder | 08:43 | |
| *** yamada-h has quit IRC | 08:43 | |
| *** jistr has joined #openstack-cinder | 08:43 | |
| *** yamada-h has joined #openstack-cinder | 08:45 | |
| *** yamada-h has quit IRC | 08:45 | |
| *** ndipanov has joined #openstack-cinder | 08:46 | |
| *** yamada-h has joined #openstack-cinder | 08:47 | |
| *** yamada-h has quit IRC | 08:47 | |
| *** dulek has quit IRC | 08:53 | |
| *** Liu has joined #openstack-cinder | 09:05 | |
| *** liuxg has quit IRC | 09:06 | |
| *** dulek has joined #openstack-cinder | 09:12 | |
| *** geui_ has joined #openstack-cinder | 09:24 | |
| geui_ | cinder-volume not showing in service list cinder-volume log: http://paste.openstack.org/show/180918/ | 09:26 |
| openstackgerrit | Alexey Khodos proposed openstack/cinder: NexentaStor 5 NFS backend driver. https://review.openstack.org/190273 | 09:26 |
| *** deepakcs has joined #openstack-cinder | 09:32 | |
| *** dims_ has joined #openstack-cinder | 09:44 | |
| *** dims has quit IRC | 09:47 | |
| *** e0ne has joined #openstack-cinder | 09:47 | |
| *** yamada-h has joined #openstack-cinder | 09:48 | |
| *** lixiaoy1 has quit IRC | 09:54 | |
| *** e0ne is now known as e0ne_ | 09:54 | |
| *** Daviey has quit IRC | 09:55 | |
| *** Daviey has joined #openstack-cinder | 09:55 | |
| *** jseiler__ has quit IRC | 09:55 | |
| *** jseiler__ has joined #openstack-cinder | 09:56 | |
| *** c0m0 has joined #openstack-cinder | 09:56 | |
| *** geui_ has quit IRC | 09:57 | |
| *** e0ne_ has quit IRC | 10:00 | |
| *** e0ne has joined #openstack-cinder | 10:01 | |
| *** markvoelker has joined #openstack-cinder | 10:05 | |
| *** dulek_ has joined #openstack-cinder | 10:05 | |
| flip214 | help, please. local tests tell me on snapshot delete "Invalid volume: Volume still has 1 dependent snapshots.", while the CI tests do that successfully and crash later on. | 10:06 |
| flip214 | what do I have to look for, to find out where the difference is? Is that a cinder.conf setting? | 10:06 |
| e0ne | flip214: hi. what test did fail? do you have any errors in c-vol logs? | 10:07 |
| *** dulek has quit IRC | 10:07 | |
| flip214 | e0ne: tempest.scenario.test_volume_boot_pattern.TestVolumeBootPatternV2.test_volume_boot_pattern (and same with V1) | 10:08 |
| flip214 | locally I get "Invalid volume: Volume still has 1 dependent snapshots." | 10:08 |
| flip214 | but in the -infra CI that seems to work fine, but it crashes later on. | 10:09 |
| *** dulek_ has quit IRC | 10:09 | |
| flip214 | I need to reproduce that locally, so how can I enable volume deletion while there're still snapshots? | 10:09 |
| *** zhenguo has quit IRC | 10:09 | |
| *** markvoelker has quit IRC | 10:10 | |
| *** asselin has quit IRC | 10:20 | |
| *** kmartin has quit IRC | 10:21 | |
| e0ne | flip214: you can't delete volume with snapshots. cinder doesn't allow it | 10:23 |
| flip214 | e0ne: in the -infra CI it did that, if I read the logs correctly. | 10:23 |
| flip214 | http://logs.openstack.org/68/187568/12/check/check-tempest-dsvm-full-drbd-devstack/9564a44/logs/ | 10:23 |
| e0ne | flip214: i believe, it deletes snapshot first | 10:24 |
| *** yamada-h has quit IRC | 10:25 | |
| *** annashen has joined #openstack-cinder | 10:28 | |
| *** ociuhandu has joined #openstack-cinder | 10:30 | |
| *** annashen has quit IRC | 10:32 | |
| *** e0ne is now known as e0ne_ | 10:33 | |
| *** e0ne_ is now known as e0ne | 10:34 | |
| *** haypo has joined #openstack-cinder | 10:34 | |
| *** ociuhandu has quit IRC | 10:35 | |
| shausy | Hi During my cinder auto zoning- Brocade my zone manger testing, im getting "No initiators are in the nameserver for SAN FABRIC_1 get_device_mapping_from_network " | 10:40 |
| shausy | Device map for SAN context: {'BRCD_FAB_A': {'target_port_wwn_list': ['21110002ac0093f4', '23110002ac0093f4', '22110002ac0093f4', '20110002ac0093f4'], 'initiator_port_wwn_list': []} | 10:41 |
| shausy | im getting initator port list as empty during my cinder opertaion | 10:41 |
| shausy | Can some help me on this | 10:41 |
| *** heyun has quit IRC | 10:45 | |
| *** lpetrut has joined #openstack-cinder | 10:45 | |
| *** timcl has joined #openstack-cinder | 10:49 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-cinderclient: Add test for cinder extra-specs-list command output Add test for cinder list-extension command output Fix pep8 issues "E128 continuation line under-indented for visual indent" in other functions Change-Id: I28266af7a6ff23b83899c50cc230f926a44592dc Partial https://review.openstack.org/189311 | 10:53 |
| *** timcl has quit IRC | 10:55 | |
| *** yamada-h has joined #openstack-cinder | 10:56 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Ceph driver support retries on rados_connect_timeout https://review.openstack.org/190579 | 10:57 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix lvm manage existing volume https://review.openstack.org/156939 | 10:58 |
| *** yamada-h has quit IRC | 11:01 | |
| *** kbyrne has joined #openstack-cinder | 11:03 | |
| *** aix has quit IRC | 11:07 | |
| *** openstackgerrit has quit IRC | 11:13 | |
| *** openstackgerrit has joined #openstack-cinder | 11:14 | |
| *** e0ne is now known as e0ne_ | 11:21 | |
| *** markvoelker has joined #openstack-cinder | 11:21 | |
| *** markvoelker has quit IRC | 11:25 | |
| *** e0ne_ has quit IRC | 11:32 | |
| *** e0ne has joined #openstack-cinder | 11:36 | |
| *** eantyshev has joined #openstack-cinder | 11:41 | |
| *** julim has joined #openstack-cinder | 11:45 | |
| *** deepakcs has quit IRC | 11:55 | |
| *** markvoelker has joined #openstack-cinder | 11:56 | |
| *** marcusvrn has joined #openstack-cinder | 11:57 | |
| *** alexpilotti has joined #openstack-cinder | 12:01 | |
| *** akerr has joined #openstack-cinder | 12:03 | |
| *** BharatK has quit IRC | 12:05 | |
| *** aix has joined #openstack-cinder | 12:09 | |
| *** alexpilotti has quit IRC | 12:09 | |
| *** ociuhandu has joined #openstack-cinder | 12:10 | |
| *** dulek has joined #openstack-cinder | 12:13 | |
| aarefiev | dulek: hi, are you around | 12:16 |
| *** ociuhandu has quit IRC | 12:18 | |
| aarefiev | dulek: e0ne asked me to benchmark your patches with persistence, have results | 12:21 |
| aarefiev | https://etherpad.openstack.org/p/taskflow_persistence_benchmarking | 12:22 |
| *** timcl has joined #openstack-cinder | 12:26 | |
| *** jungleboyj has quit IRC | 12:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 12:33 |
| *** ociuhandu has joined #openstack-cinder | 12:33 | |
| openstackgerrit | Jessy Lee proposed openstack/cinder: Add volume drivers for Infortrend Storage https://review.openstack.org/177113 | 12:34 |
| *** gouthamr has joined #openstack-cinder | 12:38 | |
| *** shausy has quit IRC | 12:39 | |
| *** shausy has joined #openstack-cinder | 12:39 | |
| *** gouthamr has quit IRC | 12:42 | |
| *** nihilifer has quit IRC | 12:43 | |
| *** smoriya has quit IRC | 12:45 | |
| dulek | aarefiev: Cool, I'll take a look in a moment | 12:46 |
| dulek | aarefiev: How big was the setup? What kind of a DB? It was Fuel? DevStack? | 12:47 |
| aarefiev | dulek: it was fuel, 3 contr 2 vol | 12:47 |
| aarefiev | dulek: now working on profile with devstack | 12:48 |
| aarefiev | mysql db | 12:49 |
| e0ne | aarefiev: just to convirm: did you setup env with lvm driver or ceph? | 12:50 |
| aarefiev | e0ne: lvm | 12:50 |
| e0ne | aarefiev: thanks | 12:50 |
| *** gouthamr has joined #openstack-cinder | 12:50 | |
| *** rakesh_mishra has joined #openstack-cinder | 12:51 | |
| *** fifieldt has quit IRC | 12:53 | |
| *** tobe has quit IRC | 12:55 | |
| *** tobe has joined #openstack-cinder | 12:57 | |
| dulek | aarefiev: And it was without any TaskFlow patches for deleting? | 12:58 |
| aarefiev | dulek: yep, only your patches with scheduler create | 12:59 |
| dulek | aarefiev: Cool, there's overhead and it's more than 30%... | 13:01 |
| dulek | Actually that's quite high number. | 13:01 |
| aarefiev | dulek: I'm trying to see it on devstack with osprofiler | 13:01 |
| *** tobe has quit IRC | 13:02 | |
| *** xyang1 has joined #openstack-cinder | 13:02 | |
| dulek | I wouldn't expect dramatically different results. | 13:02 |
| aarefiev | dulek: for sure, just want to see bottleneck | 13:03 |
| *** Yogi1 has joined #openstack-cinder | 13:03 | |
| *** bill_az has joined #openstack-cinder | 13:04 | |
| *** Longgeek_ has quit IRC | 13:06 | |
| *** sgotliv__ has joined #openstack-cinder | 13:18 | |
| *** salv-orlando has quit IRC | 13:18 | |
| *** salv-orlando has joined #openstack-cinder | 13:19 | |
| *** sgotliv_ has quit IRC | 13:20 | |
| *** simondodsley has joined #openstack-cinder | 13:21 | |
| *** salv-orl_ has joined #openstack-cinder | 13:24 | |
| *** salv-orlando has quit IRC | 13:25 | |
| *** annashen has joined #openstack-cinder | 13:30 | |
| *** ociuhandu has quit IRC | 13:32 | |
| haypo | ok, to stop repeating myself, i wrote a blueprint to port Cinder to Python 3: https://blueprints.launchpad.net/cinder/+spec/cinder-python3 | 13:34 |
| haypo | i hope that it will help ;) | 13:34 |
| haypo | jgriffith, ^^ | 13:34 |
| dulek | haypo: You would probably need a spec for that... | 13:35 |
| haypo | dulek, jgriffith told me that a spec is not needed | 13:35 |
| *** annashen has quit IRC | 13:35 | |
| haypo | dulek, well, the spec is the summary of the blueprint ;) | 13:35 |
| dulek | Cool, I think that's a reasonable decision | 13:35 |
| haypo | as i wrote, the nova spec can be reused since i plan to follow exactly the same path for cinder | 13:36 |
| *** eharney has quit IRC | 13:38 | |
| *** mriedem_away is now known as mriedem | 13:38 | |
| *** dustins has joined #openstack-cinder | 13:39 | |
| *** rushil has joined #openstack-cinder | 13:42 | |
| jgriffith | haypo: dulek That's correct (no spec needed) | 13:44 |
| haypo | jgriffith, so, what do you think of the plan in https://blueprints.launchpad.net/cinder/+spec/cinder-python3 ? | 13:44 |
| *** setmason has joined #openstack-cinder | 13:46 | |
| *** alexpilotti has joined #openstack-cinder | 13:46 | |
| jgriffith | haypo: looks reasonable to me | 13:47 |
| jgriffith | haypo: thanks for writing it up | 13:47 |
| haypo | i wrote it because someone else asked me for my plan this morning, the third guy ;) | 13:48 |
| *** rushil has quit IRC | 13:48 | |
| *** gouthamr has quit IRC | 13:48 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Replace basestring with six.string_types https://review.openstack.org/190153 | 13:49 |
| *** ociuhandu has joined #openstack-cinder | 13:49 | |
| xyang1 | e0ne: Ping | 13:50 |
| e0ne | xyang1: hi | 13:50 |
| xyang1 | e0ne: ScaleIO CI is reporting correctly now, can you check | 13:51 |
| e0ne | xyang1: thanks for pinging me. checking | 13:51 |
| xyang1 | e0ne: Ok, thanks | 13:51 |
| *** rushil has joined #openstack-cinder | 13:52 | |
| mriedem | jbernard: got your ceph debugging hat on? | 13:53 |
| *** gouthamr has joined #openstack-cinder | 13:54 | |
| mriedem | jbernard: race in the gate with ceph: http://logs.openstack.org/02/173802/5/check/check-tempest-dsvm-full-ceph/a72aac1/logs/screen-c-vol.txt.gz#_2015-06-11_09_01_02_096 | 13:54 |
| mriedem | delete snapshot fails since the snapshot is busy | 13:54 |
| *** asselin_ has quit IRC | 13:54 | |
| *** ociuhandu has quit IRC | 13:56 | |
| *** ociuhandu has joined #openstack-cinder | 13:57 | |
| mriedem | jdurgin: ^ since you have a spec up to nova about rbd instance snapshot, maybe you know something about rbd volume snapshots | 13:58 |
| *** asselin has joined #openstack-cinder | 13:59 | |
| *** rmesta has joined #openstack-cinder | 13:59 | |
| *** jnicklin has joined #openstack-cinder | 13:59 | |
| *** thangp has joined #openstack-cinder | 14:00 | |
| *** jdurgin1 has joined #openstack-cinder | 14:00 | |
| mriedem | looks like cinder's rbd driver just needs a retry loop like was added to nova in https://review.openstack.org/#/c/169446/ | 14:00 |
| *** zhipeng has joined #openstack-cinder | 14:00 | |
| *** rmesta has left #openstack-cinder | 14:01 | |
| *** breitz has quit IRC | 14:03 | |
| *** breitz has joined #openstack-cinder | 14:03 | |
| *** ganso_ has joined #openstack-cinder | 14:04 | |
| openstackgerrit | Ph. Marek proposed openstack/cinder: Re-add DRBD driver. https://review.openstack.org/178573 | 14:05 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Use six to fix imports on Python 3 https://review.openstack.org/185417 | 14:06 |
| Liu | Is the port of review.openstack.org still 29418? | 14:07 |
| Liu | I can not cennect to review.openstack.org var port 29418 | 14:08 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Use six.reraise() for Python 3 compatibility https://review.openstack.org/190169 | 14:09 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Use six.reraise() for Python 3 compatibility https://review.openstack.org/190169 | 14:10 |
| *** eharney has joined #openstack-cinder | 14:11 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Get StringIO from six for Python 3 compatibility https://review.openstack.org/189720 | 14:12 |
| *** ronenkat has quit IRC | 14:12 | |
| xyang1 | e0ne: Thanks! | 14:15 |
| e0ne | xyang1: you're welcome! thank you for addressing my comments | 14:15 |
| jbernard | mriedem: do you know if the volume eventually leaves the busy state? | 14:15 |
| mriedem | jbernard: would that be in the ceph logs? i didn't dig that far | 14:16 |
| mriedem | sidetracked | 14:16 |
| mriedem | the ceph logs are posted though with the job | 14:16 |
| jbernard | mriedem: not sure, ill take a look | 14:16 |
| *** BharatK has joined #openstack-cinder | 14:18 | |
| *** diogogmt has joined #openstack-cinder | 14:18 | |
| *** takedakn has joined #openstack-cinder | 14:19 | |
| *** setmason has quit IRC | 14:23 | |
| *** gouthamr has quit IRC | 14:24 | |
| *** jaypipes has quit IRC | 14:26 | |
| *** gouthamr has joined #openstack-cinder | 14:29 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Replace xrange() with six.moves.range() https://review.openstack.org/185418 | 14:30 |
| *** eharney_ has joined #openstack-cinder | 14:30 | |
| *** eharney has quit IRC | 14:32 | |
| *** e0ne is now known as e0ne_ | 14:34 | |
| *** crose has joined #openstack-cinder | 14:36 | |
| *** eharney_ is now known as eharney | 14:38 | |
| *** zhipeng has quit IRC | 14:38 | |
| *** rushil has quit IRC | 14:40 | |
| *** crose has quit IRC | 14:41 | |
| *** takedakn has quit IRC | 14:41 | |
| *** shausy has quit IRC | 14:42 | |
| *** shausy has joined #openstack-cinder | 14:43 | |
| *** nkrinner has quit IRC | 14:43 | |
| *** rushil has joined #openstack-cinder | 14:44 | |
| *** tsekiyama has joined #openstack-cinder | 14:45 | |
| *** jdurgin1 has quit IRC | 14:45 | |
| *** e0ne_ has quit IRC | 14:45 | |
| *** mtanino has joined #openstack-cinder | 14:52 | |
| *** gouthamr has quit IRC | 14:58 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Update SolidFire to use target driver model https://review.openstack.org/190677 | 14:59 |
| *** Zhongjun has joined #openstack-cinder | 15:01 | |
| *** dims_ has quit IRC | 15:02 | |
| *** dims has joined #openstack-cinder | 15:02 | |
| *** gouthamr has joined #openstack-cinder | 15:03 | |
| *** nlevinki has quit IRC | 15:04 | |
| *** setmason has joined #openstack-cinder | 15:06 | |
| *** timcl has quit IRC | 15:07 | |
| *** rushil has quit IRC | 15:08 | |
| dulek | geguileo: Hi, I've commented on 185545, can you take a look if my concerns aren't invalid? | 15:10 |
| geguileo | dulek: Ok, I'll have a look | 15:10 |
| dulek | geguileo: Thanks! :) | 15:11 |
| *** rushil has joined #openstack-cinder | 15:13 | |
| *** lcurtis has joined #openstack-cinder | 15:15 | |
| *** kmartin has joined #openstack-cinder | 15:15 | |
| *** jungleboyj has joined #openstack-cinder | 15:15 | |
| *** jistr is now known as jistr|mtg | 15:17 | |
| *** vokt has joined #openstack-cinder | 15:18 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Use six to fix imports on Python 3 https://review.openstack.org/185417 | 15:19 |
| *** markus_z has quit IRC | 15:21 | |
| *** juzuluag has joined #openstack-cinder | 15:23 | |
| *** dannywilson has joined #openstack-cinder | 15:23 | |
| *** dustins has quit IRC | 15:24 | |
| *** dannywilson has quit IRC | 15:24 | |
| *** agarciam has joined #openstack-cinder | 15:24 | |
| *** dannywilson has joined #openstack-cinder | 15:24 | |
| *** timcl has joined #openstack-cinder | 15:25 | |
| *** vivek_nandavanam has joined #openstack-cinder | 15:26 | |
| vivek_nandavanam | "A pool in Cinder has a one to one mapping with a volume on a StorageController" - Is this statement right? | 15:28 |
| geguileo | dulek: In one of the comments (the most important one) I agree with you, so I changed my vote | 15:28 |
| *** anshul has quit IRC | 15:29 | |
| *** vokt has quit IRC | 15:30 | |
| openstackgerrit | Joshua Huber proposed openstack/cinder: Add cinder volume driver for Blockbridge EPS https://review.openstack.org/178295 | 15:31 |
| *** coolsvap is now known as coolsvap|afk | 15:31 | |
| *** annashen has joined #openstack-cinder | 15:32 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Refactor API create_volume flow https://review.openstack.org/187624 | 15:32 |
| Swanson | Any cores floating around to give an +2/+Amen to this one? https://review.openstack.org/#/c/187729/ Almost entirely comment changes. (I get rid of a lot of single character vars, jgriffith!) | 15:34 |
| *** salv-orlando has joined #openstack-cinder | 15:34 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO Volume driver Consistency Groups support https://review.openstack.org/189135 | 15:34 |
| jgriffith | Swanson: :) | 15:34 |
| *** salv-orl_ has quit IRC | 15:34 | |
| openstackgerrit | rakesh mishra proposed openstack/cinder: set-size-limit-for-volume https://review.openstack.org/185906 | 15:35 |
| *** jistr|mtg is now known as jistr | 15:35 | |
| dulek | geguileo: Thanks, I need to do reviews in the morning and not in the evening. ;) Now I understand the code. | 15:35 |
| geguileo | dulek: I know what you mean XD | 15:36 |
| *** annashen has quit IRC | 15:36 | |
| *** vilobhmm has joined #openstack-cinder | 15:37 | |
| Swanson | The newish target driver stuff is still only FC or iSCSI per volume, right? Can't combine? | 15:37 |
| *** pots has joined #openstack-cinder | 15:38 | |
| *** shausy has quit IRC | 15:39 | |
| *** hemnafk is now known as hemna | 15:40 | |
| *** vilobhmm1 has joined #openstack-cinder | 15:41 | |
| *** vilobhmm has quit IRC | 15:42 | |
| *** leeantho has joined #openstack-cinder | 15:42 | |
| openstackgerrit | Danny Wilson proposed openstack/cinder: Volume manager should set filter_function and goodness_function https://review.openstack.org/190695 | 15:42 |
| *** coolsvap|afk is now known as coolsvap | 15:42 | |
| dannywilson | DuncanT: kmartin: ^^ | 15:43 |
| kmartin | dannywilson, I'll take a look | 15:44 |
| *** afazekas has quit IRC | 15:45 | |
| dannywilson | kmartin: thanks | 15:45 |
| Swanson | jgriffith: Thanks! | 15:49 |
| *** potsmaster has joined #openstack-cinder | 15:51 | |
| jdurgin | mriedem: jbernard: thanks for pointing out that failure. I'd rather not retry blindly, and fix the root cause. the main culprit would be qemu getting shutdown uncleanly (in that case there's a 30s timeout until the volume can be deleted) | 15:52 |
| *** jordanP has quit IRC | 15:53 | |
| *** hodos has joined #openstack-cinder | 15:53 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR enable multiattach capability reporting https://review.openstack.org/182745 | 15:56 |
| *** agarciam has quit IRC | 15:57 | |
| *** patrickeast has joined #openstack-cinder | 15:57 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/python-cinderclient: Add volume multi attach support https://review.openstack.org/85856 | 15:57 |
| *** coolsvap is now known as coolsvap|afk | 15:58 | |
| *** rushil has quit IRC | 16:01 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Get StringIO from six for Python 3 compatibility https://review.openstack.org/189720 | 16:02 |
| mriedem | jdurgin: works for me if you have a fix | 16:02 |
| *** dulek has quit IRC | 16:03 | |
| mriedem | jdurgin: i didn't dig into the qemu logs for the related instance | 16:03 |
| jdurgin | mriedem: do you happen to be familiar with the current nova shutdown/hard stop logic as configured in the gate? | 16:04 |
| mriedem | jdurgin: as in we're not doing graceful shutdowns in the gate | 16:06 |
| mriedem | b/c it was causing timeouts | 16:06 |
| *** asselin_ has joined #openstack-cinder | 16:09 | |
| mriedem | jdurgin: actually it looks like the default in nova is to do the clean shutdown for the stop api | 16:10 |
| jdurgin | mriedem: thanks, looking further with jbernard | 16:10 |
| mriedem | and i'm not seeing shutdown_timeout changed in nova.conf in devstack, so i'm assuming it's running in the gate | 16:12 |
| *** _cjones_ has joined #openstack-cinder | 16:15 | |
| *** vilobhmm1 has quit IRC | 16:15 | |
| jdurgin | mriedem: thanks, that should be ok then. I'm suspecting a racy testcase now since a snapshot delete would only get that error when there's still a clone | 16:15 |
| *** diemt has joined #openstack-cinder | 16:16 | |
| *** anshul has joined #openstack-cinder | 16:20 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO Volume driver requests and multipath https://review.openstack.org/184596 | 16:20 |
| *** Apoorva has joined #openstack-cinder | 16:22 | |
| *** BharatK has quit IRC | 16:22 | |
| *** krtaylor has quit IRC | 16:22 | |
| *** e0ne has joined #openstack-cinder | 16:27 | |
| *** r-daneel has joined #openstack-cinder | 16:27 | |
| Swanson | Lots of python 3 things all over openstack. Is there a plan or timeline for jumping to 3.x? | 16:29 |
| hemna | ew | 16:29 |
| Swanson | hemna: did I just cause a migraine? | 16:30 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Replace it.next() with next(it) for py3 compat https://review.openstack.org/190173 | 16:30 |
| hemna | yes | 16:30 |
| haypo | Swanson, i just decided to flood openstack with my patches ;) | 16:30 |
| hemna | py3 leads to pain, pain leads to anger, anger leads to suffering. | 16:30 |
| haypo | Swanson, someone, it was decided to port openstack. the thing is that there is no more major dependency which is incompatible with python 2 | 16:30 |
| haypo | hemna, good news. i like all these things! | 16:31 |
| angela-s | hemna: +1, i have a headache just thinking about it | 16:31 |
| jgriffith | It's really not so bad | 16:31 |
| jgriffith | haypo: has a good strategy going forward | 16:32 |
| jgriffith | and it needs to be done | 16:32 |
| Swanson | haypo: so the net of that is it can work under 3 and from other reactions not everyone will move to it. | 16:32 |
| haypo | just kidding | 16:32 |
| Swanson | I'm too stupid to know I'm in for pain before it hits. | 16:33 |
| haypo | Swanson, the plan is to add python 3 support. you're free to keep python 2 | 16:33 |
| *** BharatK has joined #openstack-cinder | 16:34 | |
| *** jaypipes has joined #openstack-cinder | 16:35 | |
| Swanson | Just trying to schedule some tasks. One of them requires python 2.7.9 which ubunutu 14.04 LTS doesn't seem to support. It we were going to py3 I'd ashcan that whole effort. | 16:35 |
| haypo | Swanson, why do you need this specific version of python 2.7? (2.7.9) | 16:36 |
| Swanson | Oddly because it has some SSL stuff pulled from 3.x. | 16:36 |
| Swanson | Attack poodles or something. | 16:37 |
| *** alau2 has joined #openstack-cinder | 16:38 | |
| haypo | Swanson, ah yes, the security backports | 16:39 |
| haypo | Swanson, it's the whole ssl module + os.urandom enhancements | 16:39 |
| openstackgerrit | John Griffith proposed openstack/cinder: Update SolidFire to use target driver model https://review.openstack.org/190677 | 16:39 |
| *** vilobhmm has joined #openstack-cinder | 16:40 | |
| *** vilobhmm1 has joined #openstack-cinder | 16:41 | |
| Swanson | haypo: Yep. We have a library in no man's land. Needs those enhancements but doesn't work under 3.x. I could push to have that moved to 3.x if Openstack is going to run under 3 anytime soon. (Like Liberty.) | 16:41 |
| *** c0m0 has quit IRC | 16:41 | |
| *** anshul has quit IRC | 16:42 | |
| haypo | Swanson, it's a pain to use a specific version of python, /usr/bin/python is part of Ubuntu! | 16:42 |
| Swanson | haypo: That would be my argument. | 16:43 |
| *** vilobhmm has quit IRC | 16:44 | |
| haypo | i remember that ubuntu has/had 3.4.0 whereas this version contains a major regression in the garbage collector :-/ | 16:44 |
| haypo | https://bugs.launchpad.net/ubuntu/+source/python3.4/+bug/1367907 | 16:44 |
| openstack | Launchpad bug 1367907 in python3.4 (Ubuntu Trusty) "Segfault in gc with cyclic trash" [Undecided,Triaged] | 16:45 |
| haypo | ah, it looks to be fixed now | 16:45 |
| *** annashen has joined #openstack-cinder | 16:45 | |
| *** crose has joined #openstack-cinder | 16:46 | |
| e0ne | jgriffith, dulek: we've got a bad news. profiler results with persistance taskflow: http://e0ne.github.io/cinder/db-profiler-results/volume/create-with-persistence.html | 16:46 |
| e0ne | thanks aarefiev to make it happen | 16:46 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Drop L suffix from long integers https://review.openstack.org/190152 | 16:47 |
| e0ne | jgriffith: look on the actions with 'wf_db' type | 16:47 |
| e0ne | harlowja: ^^ | 16:47 |
| *** haypo has quit IRC | 16:49 | |
| jgriffith | e0ne: so it's kinda cool we have the smoking gun in there :) | 16:49 |
| *** ndipanov has quit IRC | 16:50 | |
| e0ne | jgriffith: it's a good feature, but too big overhead, imo | 16:51 |
| *** salv-orl_ has joined #openstack-cinder | 16:52 | |
| *** rushil has joined #openstack-cinder | 16:53 | |
| *** ronenkat has joined #openstack-cinder | 16:54 | |
| *** salv-orlando has quit IRC | 16:54 | |
| *** rakesh_mishra has quit IRC | 16:55 | |
| *** jungleboyj has quit IRC | 16:55 | |
| *** jungleboyj has joined #openstack-cinder | 16:56 | |
| *** vokt has joined #openstack-cinder | 16:59 | |
| *** mriedem is now known as mriedem_afk | 17:00 | |
| *** gouthamr has quit IRC | 17:01 | |
| *** Yogi11 has joined #openstack-cinder | 17:02 | |
| *** gouthamr has joined #openstack-cinder | 17:02 | |
| *** rushil has quit IRC | 17:03 | |
| *** annegentle has joined #openstack-cinder | 17:03 | |
| *** Yogi1 has quit IRC | 17:03 | |
| *** angela-s is now known as angela-s|afk | 17:06 | |
| *** bnemec has joined #openstack-cinder | 17:06 | |
| *** ronenkat has quit IRC | 17:07 | |
| *** anshul has joined #openstack-cinder | 17:08 | |
| *** e0ne has quit IRC | 17:09 | |
| *** gouthamr has quit IRC | 17:11 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: LVM add multiattach flag capability https://review.openstack.org/190725 | 17:11 |
| *** breitz has quit IRC | 17:12 | |
| *** gouthamr has joined #openstack-cinder | 17:12 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: WIP: LVM add multiattach flag capability https://review.openstack.org/190725 | 17:12 |
| *** patrickeast has left #openstack-cinder | 17:15 | |
| *** patrickeast has joined #openstack-cinder | 17:15 | |
| *** ronenkat has joined #openstack-cinder | 17:15 | |
| *** gouthamr has quit IRC | 17:16 | |
| *** marcusvrn1 has joined #openstack-cinder | 17:16 | |
| *** gouthamr has joined #openstack-cinder | 17:16 | |
| *** breitz has joined #openstack-cinder | 17:18 | |
| *** Yogi11 has quit IRC | 17:22 | |
| *** anshul has quit IRC | 17:24 | |
| *** juzuluag has quit IRC | 17:25 | |
| *** gouthamr has quit IRC | 17:28 | |
| marcusvrn1 | thingee: hi! did you have a chance to take a look on the new CI? | 17:29 |
| thingee | no | 17:29 |
| marcusvrn1 | hmm ok :) | 17:30 |
| *** lpetrut has quit IRC | 17:30 | |
| harlowja | aarefiev yt, so for http://e0ne.github.io/cinder/db-profiler-results/volume/create-with-persistence.html any idea how to easily interpret that :-P looks like bunch of queries, to wf_db, perhaps can be optimized more (aka, via some taskflow changes to not store anything at all for tasks that are really 'transient') | 17:31 |
| harlowja | i do see its querying things, which is sorta expected :-P | 17:31 |
| harlowja | *perhaps also need better 'bulk' query apis in taskflow to | 17:31 |
| thingee | marcusvrn1: please stop asking me too. You will know I had a chance to view it when I either move it in the etherpad or contact you about a problem. | 17:32 |
| thingee | I realize it might be a common misconception that this is all I do. | 17:32 |
| hemna | you do things other than look at CI ? | 17:33 |
| harlowja | do u look at IC the rest of the time? | 17:33 |
| harlowja | 50% CI, 50% IC | 17:33 |
| * thingee takes a look at the taskflow ci | 17:33 | |
| thingee | tsk tsk | 17:33 |
| harlowja | ha | 17:33 |
| harlowja | i only run IC | 17:34 |
| harlowja | not that CI stuff | 17:34 |
| hemna | :) | 17:34 |
| thingee | IC = I care | 17:34 |
| marcusvrn1 | thingee: sorry, I was trying to know if it was ok, because the deadline is close | 17:34 |
| harlowja | its the caring that matters the most right | 17:34 |
| *** BharatK has quit IRC | 17:35 | |
| *** jistr has quit IRC | 17:35 | |
| thingee | marcusvrn1: imagine having 14 different vendors coming to you about the same thing. | 17:35 |
| marcusvrn1 | thingee: ok, sorry, I'll not ask again | 17:35 |
| harlowja | thingee http://i.imgur.com/MJnEm6c.jpg there u go, cheer u right up, lol | 17:35 |
| *** anshul has joined #openstack-cinder | 17:36 | |
| thingee | harlowja: this is the original pitch I gave to bswartz for his project name | 17:37 |
| harlowja | u called it the 'care bear' project? | 17:37 |
| thingee | I actually had someone at netapp asking me if I would release the idea to them after mentioning it on IRC. | 17:37 |
| bswartz | lol | 17:37 |
| * harlowja thinks that idea might be owned by matell or something :-P | 17:38 | |
| harlowja | but ya +1 to care bear project, lol | 17:38 |
| thingee | I just meant the sharing is caring part | 17:38 |
| harlowja | ah | 17:38 |
| thingee | unless they own that | 17:38 |
| thingee | wait, there was some gum commercial that used that too | 17:39 |
| thingee | oh well | 17:39 |
| harlowja | hmmm, only gum commerical i remember is the big red ones, lol | 17:39 |
| * harlowja hasn't seen those in forever | 17:39 | |
| thingee | juicy fruit. sharing is caring | 17:39 |
| bswartz | I don't see how you can trademark "sharing" and then not share it -- that's just evil | 17:40 |
| harlowja | americans be crazy, lol | 17:40 |
| thingee | harlowja: https://www.youtube.com/watch?v=nALwA2BbcQM | 17:40 |
| harlowja | thingee damn, i might have nightmares now :-/ | 17:40 |
| harlowja | lol | 17:41 |
| bswartz | rofl @ 28 seconds | 17:41 |
| * harlowja goes back to hiding under his desk | 17:41 | |
| bswartz | yeah that fish is nightmarish | 17:41 |
| thingee | jgriffith: saw your comment about blockbridge ci, I haven't looked at their other reports, but I did notice their initial report for their driver being added in, the logs seem to work https://review.openstack.org/#/c/178295/ | 17:41 |
| thingee | jgriffith: from what I saw you added here http://paste.openstack.org/show/282385/ | 17:42 |
| asselin_ | thingee, https://etherpad.openstack.org/p/cinder-thirdpartyCI-issues | 17:43 |
| harlowja | to cleanse your pallet https://www.youtube.com/watch?v=AA3LrnAqOds | 17:43 |
| harlowja | lol | 17:43 |
| harlowja | ahhh, no more nightmares now | 17:43 |
| harlowja | *palate | 17:44 |
| harlowja | whatever, i don't do english :-P | 17:44 |
| thingee | asselin_, jgriffith: I need to add my own under here. don't have the subunit stuff being added in still :( | 17:44 |
| thingee | I know jgriffith did an update to his CI stuff, I just haven't updated it yet. | 17:45 |
| *** akerr is now known as akerr_away | 17:45 | |
| *** cbader has joined #openstack-cinder | 17:51 | |
| *** dannywil_ has joined #openstack-cinder | 17:53 | |
| *** IlyaG has joined #openstack-cinder | 17:54 | |
| *** dannywilson has quit IRC | 17:57 | |
| *** gouthamr has joined #openstack-cinder | 17:57 | |
| *** lpetrut has joined #openstack-cinder | 18:00 | |
| hemna | jgriffith, posed a question in your FCTarget review. see what you think.... https://review.openstack.org/#/c/190291/ | 18:01 |
| *** hemna is now known as hemnafk | 18:01 | |
| thingee | asselin_: done | 18:01 |
| thingee | asselin_, jgriffith is there a plan with these? | 18:02 |
| *** akerr_away is now known as akerr | 18:02 | |
| *** Yogi11 has joined #openstack-cinder | 18:02 | |
| thingee | asselin_: here's an example of a blockbridge ci http://logs.openstack.blockbridge.com/95/178295/8/check/check-tempest-dsvm-volume-blockbridge/873f90c/ | 18:03 |
| thingee | asselin_: that's one that was posted for their own driver code to be merged | 18:03 |
| patrickeast | that one looks correct | 18:04 |
| patrickeast | thingee: the one in that etherpad is from may 15th? i think that was before they switched to using asselin’s scripts | 18:05 |
| thingee | patrickeast: got it | 18:05 |
| thingee | patrickeast: I'll take a look at the other reports to see if they're consistent | 18:06 |
| *** ociuhandu has quit IRC | 18:06 | |
| patrickeast | thingee: i would hazard a guess that any posted before the driver got merged are potentially bad | 18:07 |
| thingee | patrickeast: yea looks good from some recent runs | 18:07 |
| *** rushil has joined #openstack-cinder | 18:12 | |
| patrickeast | thingee: also, to answer your question on what the plan was with those systems, this was the converation about it yesterday http://eavesdrop.openstack.org/irclogs/%23openstack-cinder/%23openstack-cinder.2015-06-10.log.html#t2015-06-10T23:10:00 | 18:12 |
| patrickeast | although i’m not sure if anyone has actually signed up to start contacting operators | 18:12 |
| thingee | I think I can do that. | 18:12 |
| thingee | as long as no one else has started | 18:12 |
| *** annegentle has quit IRC | 18:14 | |
| *** jdurgin1 has joined #openstack-cinder | 18:15 | |
| thingee | smcginnisafk: lgtm here https://review.openstack.org/#/c/188725/5 | 18:16 |
| *** jdurgin has quit IRC | 18:17 | |
| *** annegentle has joined #openstack-cinder | 18:17 | |
| *** krtaylor has joined #openstack-cinder | 18:18 | |
| thingee | marcusvrn1: why is the hbsd2 CI reporting FC, if the driver you're proposing ISCSI? https://review.openstack.org/#/c/184999/ | 18:19 |
| *** salv-orl_ has quit IRC | 18:20 | |
| patrickeast | woa that storpool ci is a beast… 14 min tempest run?! | 18:20 |
| *** salv-orlando has joined #openstack-cinder | 18:21 | |
| thingee | patrickeast: am I crazy about hbsd2 CI testing FC, but the driver being proposed is iscsi? https://review.openstack.org/#/c/184999/ | 18:23 |
| patrickeast | thingee: uhh that does seem odd | 18:24 |
| thingee | kmartin, hemnafk: are we not going to see a CI for this? https://review.openstack.org/#/c/184774/ | 18:24 |
| thingee | marcusvrn1: I've updated the etherpad about this. Not exactly sure what's going on there. | 18:24 |
| patrickeast | thingee: do they already have an FC driver? | 18:25 |
| thingee | patrickeast: well they were telling me in the review this is for the proposed ISCSI driver (the review link I gave you). | 18:25 |
| *** salv-orlando has quit IRC | 18:26 | |
| patrickeast | thingee: maybe i’m missing something, i don’t actually see a new driver class added in that review | 18:26 |
| patrickeast | it does seem to be adding support for iscsi related stuff | 18:27 |
| *** smcginnisafk is now known as smcginnis | 18:28 | |
| thingee | patrickeast: yeah I don't get it | 18:28 |
| thingee | lol | 18:28 |
| patrickeast | so there is an iscsi and fc driver already in tree, which is what that ci is testing | 18:28 |
| patrickeast | maybe its just expanding the driver to support more hardware? | 18:28 |
| jgriffith | patrickeast: thingee it's just adding a product line to the driver; no? | 18:29 |
| thingee | jgriffith: yeah, anyways I was saying the proposed patches are iscsi related, but the new ci they're introducing is FC | 18:29 |
| jgriffith | thingee: oh... I dunno, I've never been able to follow any of their stuff | 18:30 |
| jgriffith | thingee: not going to try and start now | 18:30 |
| jgriffith | :) | 18:30 |
| thingee | hehe | 18:30 |
| thingee | jgriffith, asselin_ I'll start contacting ops on the CI list here, unless that's already been started | 18:30 |
| thingee | jgriffith, asselin_ specifically this list https://etherpad.openstack.org/p/cinder-thirdpartyCI-issues | 18:31 |
| jgriffith | thingee: we proposed yesterday that we should *share* that effort | 18:31 |
| jgriffith | thingee: and divie up the work | 18:32 |
| thingee | ok sure. so do I just mark the ones I'll do? | 18:32 |
| jgriffith | thingee: unless you still want to be the single point of contact | 18:32 |
| jgriffith | I know you wanted it that way before, which is fine | 18:32 |
| thingee | not opposed to it, as long as I can rely on people | 18:32 |
| thingee | I can rely on myself, which is why I just took it on | 18:32 |
| jgriffith | thingee: Your call | 18:32 |
| jgriffith | thingee: honestly I think the time for that kind of effort/attention is over | 18:32 |
| thingee | sure | 18:33 |
| jgriffith | thingee: if it were me, send one email out warning.. then things get turned off and removed | 18:33 |
| thingee | jgriffith: understood | 18:33 |
| thingee | jgriffith: I added datera to the list. still need to update sos-ci | 18:33 |
| thingee | for subunit | 18:33 |
| thingee | support | 18:33 |
| jgriffith | haha... that's the least of the problems out there right now IMHO | 18:33 |
| thingee | well, just trying to not be biased here. | 18:34 |
| jgriffith | I hear ya | 18:34 |
| thingee | yeah I agree with you on the effort. sharing it seems like a good plan | 18:34 |
| jgriffith | thingee: just seems crazy to have you spending such a large percentage of your time on it | 18:35 |
| jgriffith | or any one person for that matter | 18:35 |
| jgriffith | should be split up among cores/regular-contributors IMO | 18:35 |
| jgriffith | just so long as everybody is on the same page ;) | 18:35 |
| uberjay | asselin_: thingee I'm currently looking at our CI -- it appears to be suddenly having issues uploading the logs to our logserver, which is unfortunately failing the build :( | 18:38 |
| uberjay | uh, also... the logs linked from the thirdpartyCI-issues list are not logs from us. (misfiled? they say "openvstorage" in the url...) | 18:40 |
| *** hemnafk is now known as hemna | 18:43 | |
| hemna | thingee, we are working on CI for that one....but it's not ready yet. hence my -2 on it still. | 18:43 |
| * asselin_ reads scrollback | 18:44 | |
| *** breitz has quit IRC | 18:44 | |
| *** breitz has joined #openstack-cinder | 18:44 | |
| jgriffith | uberjay: that's me :) | 18:44 |
| jgriffith | uberjay: which CI is yours? | 18:44 |
| uberjay | jgriffith: Blockbridge CI | 18:45 |
| uberjay | jgriffith: I think one of my build failures was due to this, which appears to be a Jenkins bug: https://issues.jenkins-ci.org/browse/JENKINS-27565 | 18:45 |
| *** annegentle has quit IRC | 18:45 | |
| uberjay | maybe just don't have the fix yet, not sure | 18:45 |
| jgriffith | uberjay: cool... removed that entry. Sorry about any confusion | 18:46 |
| asselin_ | thingee, I'm helping the guy set up ci for: HP XP storage. Posting locally...should have it up later today/tomorrow | 18:46 |
| *** jdurgin has joined #openstack-cinder | 18:46 | |
| thingee | jgriffith, patrickeast, smcginnis, xyang1, winston-d, hemna, eharney, jungleboyj current driver list https://etherpad.openstack.org/p/cinder-liberty-drivers ... I will be cutting driver submissions tomorrow | 18:47 |
| asselin_ | thingee, we can add sign ups to the etherpad so we don't duplicate effort | 18:47 |
| hemna | thingee, ok | 18:47 |
| thingee | as a last attempt, I will be contacting maintainers on the urgency | 18:47 |
| thingee | asselin_: +1 | 18:47 |
| *** jdurgin1 has quit IRC | 18:48 | |
| thingee | jgriffith: patrickeast smcginnis xyang1 winston-d hemna eharney jungleboyj I'd like the current list that is marked ready to be merged before june 19th. This list might grow after today depending on if people are able to fix there stuff in time...but this might be the final list. | 18:48 |
| * thingee plays final count down by europe | 18:48 | |
| asselin_ | uberjay, could you clarify on the etherpad if not done already? | 18:48 |
| * jungleboyj hear dunah dunah ... dunah dun dun dun ... dunah dunah ... | 18:49 | |
| thingee | marcusvrn1: ^ | 18:49 |
| xyang1 | thingee: Sure | 18:49 |
| jgriffith | thingee: so about that, I've been meaning to talk to folks about my driver | 18:49 |
| *** aix has quit IRC | 18:49 | |
| asselin_ | uberjay, what's the issue with uploding log files? out of disk space maybe? | 18:49 |
| jgriffith | and FC inparticular | 18:49 |
| uberjay | asselin_: ah, it appeared to just affect one build. one sec. | 18:50 |
| jgriffith | as you can see it's not a new driver, it's target option (can be both) | 18:50 |
| jgriffith | thingee: need to know if it's going to fall under deadline or not | 18:50 |
| jgriffith | thingee: if it is, I'll drop anything else that i'm working on and finish it | 18:50 |
| thingee | target drivers are excluded. | 18:51 |
| *** e0ne has joined #openstack-cinder | 18:51 | |
| *** e0ne is now known as e0ne_ | 18:51 | |
| thingee | I think we still need to have a discussion on what is a driver since I know people disagree with my thoughts on that. | 18:51 |
| jgriffith | thingee: good to know, thought that was the case at one point, but figured I should verify so no surprises | 18:51 |
| patrickeast | is it a target driver? or adding FC support to existing volume driver? | 18:51 |
| patrickeast | they are different things… right? (might be wrong) | 18:51 |
| hemna | thingee, I signed up for HGST, since no one was on that one, and I've been reviewing the os-brick connector for HGST | 18:51 |
| jgriffith | patrickeast: well, in my case it's adding a target-driver | 18:51 |
| patrickeast | jgriffith: ah ok | 18:52 |
| jgriffith | patrickeast: but honestly they're kinda the same thing | 18:52 |
| hemna | earlephilhower, ping | 18:52 |
| earlephilhower | hemna: howdy | 18:52 |
| thingee | patrickeast: IMO, if you can use the same volume driver class, and just configure the other target driver, who cares. | 18:52 |
| patrickeast | yea that makes sense | 18:52 |
| thingee | patrickeast: if it's a whole new volume driver class, you're defining a new driver. | 18:52 |
| *** jaypipes has quit IRC | 18:52 | |
| patrickeast | where does something like that fall with ci? | 18:52 |
| hemna | earlephilhower, heyas. now that we have 3rd party CI working against os-brick patches, I think you should add testing against the HGST connector | 18:53 |
| thingee | since you are, well, inheriting from something like cinder.volume.driver.VolumeDriver... that's a new driver. | 18:53 |
| uberjay | asselin: this is what happened: http://paste.openstack.org/show/284392/ | 18:53 |
| *** e0ne_ is now known as e0ne | 18:53 | |
| hemna | earlephilhower, https://review.openstack.org/#/c/186588/ take a look at what's being reported on your patch now. You can see that HP CI is reporting as is pure storage. | 18:54 |
| thingee | jgriffith: that's pretty neat. If all you have to do is tell solidfire's existing code to just use this new target driver, I don't care. I think you're also fixing FC for everyone as well | 18:54 |
| earlephilhower | hemna: Let me see how to update. Is there a new devstack override for OS_BRICK_REPO? I'm git pulling/installing before runs. Looking at your link. | 18:54 |
| hemna | earlephilhower, I would like to see HGST reporting there as well, since your os-brick patch is specifically for adding HGST trasport :) | 18:54 |
| uberjay | asselin: *maybe* some sort of race with the one-shot nodes? i've only seen it once, so maybe not a recurring problem after all | 18:54 |
| asselin_ | uberjay, ok | 18:54 |
| hemna | earlephilhower, yes, asselin_ got it in recently. you can ping him on how to enable your CI for it. | 18:54 |
| smcginnis | jgriffith: I haven't looked, but what about the fc zone manager decorators? Is that handled in your approach? | 18:54 |
| thingee | jgriffith: did I get that right, that it's just a matter of configuring a different target driver? | 18:55 |
| jgriffith | patrickeast: just for the sake of argument I'm planning to have CI | 18:55 |
| jgriffith | just easier that way | 18:55 |
| jgriffith | thingee: and to your questions above "yes" | 18:55 |
| thingee | jgriffith: excellent, yeah no sweat. | 18:55 |
| jgriffith | thingee: it's just adding the target driver, and yes, I've added patches to make it available to everyone | 18:55 |
| earlephilhower | hemna: Oh, no problem. Will do some reading then ping asselin if I'm stuck. Can it wait until tomorrow or Mon? Lots of meetings this afternoon for /me. | 18:56 |
| hemna | earlephilhower, well, our driver deadline for Cinder is what...tomorrow ? | 18:56 |
| thingee | jgriffith: yeah you can even quote my email about this, "This does not include connector drivers in os-brick. This will be a | 18:56 |
| thingee | separate discussion." http://lists.openstack.org/pipermail/openstack-dev/2015-May/064072.html | 18:56 |
| Swanson | smcginnis: There are no decorators in the actual FC target driver. | 18:56 |
| thingee | I guess me saying connector code is a bit ambiguous | 18:57 |
| uberjay | asselin: ugh, i take that back, got another failure with the same signature. i'll update the etherpad. | 18:57 |
| jgriffith | thingee: meh.. I knew what you meant ;) | 18:57 |
| hemna | Swanson, yah there aren't currently, it's something that we've tossed around in the review. | 18:57 |
| jgriffith | hemna: oh.. yeah, BTW did you see my response on that? | 18:57 |
| thingee | jgriffith: but I think that's pretty neat. this is what I wish all drivers would do for support the different fabrics =/ | 18:58 |
| earlephilhower | hemna: Okay, will try during lunch. If devstack has new option may be plug and play | 18:58 |
| jgriffith | hemna: and my question WRT "what if I don't want zones" | 18:58 |
| hemna | jgriffith, yah. I'm still trying to wrap my head around the FC target | 18:58 |
| jgriffith | hemna: k | 18:58 |
| hemna | and really the purpose of it at this point. it's confusing. | 18:58 |
| * winston-1 | 18:58 | |
| jgriffith | thingee: me too :) That's why I started it back in Juno | 18:58 |
| jgriffith | hemna: what's confusing about it? | 18:58 |
| hemna | jgriffith, if you don't want zones, then your driver doesn't return initiator_target_map | 18:58 |
| jgriffith | hemna: maybe I can clarify | 18:58 |
| hemna | or you don't configure cinder to use the FCZM | 18:58 |
| jgriffith | hemna: right | 18:59 |
| *** dannywil_ has quit IRC | 18:59 | |
| jgriffith | hemna: but it appears that forcing your decorator means you can't do that | 18:59 |
| hemna | but if you never add the decorators, then the FCZM can never be used with that driver. | 18:59 |
| hemna | if you add the decorator | 18:59 |
| jgriffith | hemna: sure... but those classes are abc'd | 18:59 |
| jgriffith | hemna: do you understand how the abc stuff works? I can try and explain if not | 18:59 |
| hemna | you can simply not configure cinder.conf to enable the zone manager and it all works. | 18:59 |
| hemna | but at least with the decorator in place, then folks have the option of enabling the fczm | 19:00 |
| jgriffith | hemna: right.. and even if I use your decorator it just becomes a noop right? IIUC | 19:00 |
| *** e0ne is now known as e0ne_ | 19:00 | |
| jgriffith | hemna: yeah.. for sure | 19:00 |
| hemna | if you want open zoning, then don't configure cinder to use fczm. done. | 19:00 |
| hemna | yup | 19:00 |
| jgriffith | hemna: totally get it, just wanted to verify I was looking at the code correctly | 19:00 |
| hemna | it's a noop then,. | 19:00 |
| *** winston-d has quit IRC | 19:00 | |
| *** dannywilson has joined #openstack-cinder | 19:01 | |
| jgriffith | So even if you put a decorator on an abc (and I can do that if it makes people feel better), it will NEVER get called | 19:01 |
| winston-1 | ls | 19:01 |
| winston-1 | oops | 19:01 |
| jgriffith | unless there's some magic you can do there that I'm not aware of | 19:01 |
| hemna | the decorator gets called, but internally it looks to see if the fczm is enabled or not. if it's not, noop. | 19:01 |
| jgriffith | the reason being is you've defined that as an abstract base class that MUST be overridden | 19:01 |
| hemna | also, if there is no initiator_target_map in the payload it's a noop. | 19:01 |
| *** winston-1 is now known as winston-d | 19:02 | |
| *** ronenkat has quit IRC | 19:02 | |
| jgriffith | hemna: no, I'm saying that in an abc deocrated method it will enever get called if it's not in the child class... unless you're saying you know that works and I'm wrong | 19:02 |
| hemna | right | 19:02 |
| jgriffith | hemna: oh... sorry... you're back on the other subject :) | 19:02 |
| jgriffith | never mind :) Yeah.. we're good there I think | 19:02 |
| hemna | :) | 19:02 |
| jgriffith | I"m trying to clarify the base target stuff that you said you didn't quite get | 19:03 |
| hemna | so, is the intention with the FCTarget being an FC Base target class that vendors extend ? | 19:03 |
| hemna | FC3PARTarget ? | 19:03 |
| hemna | so the FCTarget decorators wouldn't be used | 19:03 |
| *** winston-d has quit IRC | 19:03 | |
| hemna | just trying to follow the chain of usage for FCTarget | 19:03 |
| *** winston-d has joined #openstack-cinder | 19:04 | |
| jgriffith | hemna: yes | 19:04 |
| hemna | from what I can tell the lvm driver can use any target | 19:04 |
| jgriffith | hemna: EXACTLY!!! | 19:04 |
| jgriffith | hemna: that's the entire point | 19:04 |
| hemna | so 3par creates FC3PARTarget | 19:04 |
| jgriffith | hemna: here's a more concrete example: https://review.openstack.org/#/c/190677/ | 19:04 |
| hemna | then cinder.conf in my 3par driver section, I specify which target to use. | 19:04 |
| *** ronenkat has joined #openstack-cinder | 19:05 | |
| jgriffith | hemna: just s/SanISCSITarget/FCTarget/ | 19:05 |
| jgriffith | hemna: if you want/need.... or like in my case just auto determine, or pick based on a volume-type | 19:05 |
| hemna | ok, so if we put the decorator on the FC3PARTarget, then our 3PAR driver would just call return self.target_driver.initialize_connection(payload) | 19:06 |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series driver: Remove caching logic https://review.openstack.org/185832 | 19:07 |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series: Refactor class structure for FC https://review.openstack.org/185833 | 19:07 |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series: Add Fibre Channel Support https://review.openstack.org/188649 | 19:07 |
| * hemna is trying to figure out what/if at all/would go in a FC3PARTarget | 19:07 | |
| *** kjnelson has joined #openstack-cinder | 19:08 | |
| *** annashen has quit IRC | 19:08 | |
| hemna | I was hoping the generic FCTarget's initialize_connection would contain the contents of the fczm_utils.AddFCZone, and then vendor FC drivers would call return self.target_driver.initialize_connection(payload), which would induce the FCZM | 19:08 |
| *** mriedem_afk is now known as mriedem | 19:08 | |
| jgriffith | hemna: like two or three methods and that's all :) | 19:08 |
| jgriffith | that's the beauty of it | 19:09 |
| hemna | maybe I'll put a gist together to demostrate what I'm thinking | 19:09 |
| hemna | and I'm sure I'm all off base here. | 19:09 |
| hemna | just trying to understand it is all. | 19:09 |
| hemna | sorry for being slow | 19:09 |
| *** gouthamr has quit IRC | 19:12 | |
| hemna | thingee, if the HGST folks don't get the CI for os-brick plugged in by tomorrow, can I request an exception for them now? | 19:12 |
| hemna | since asselin_ and I didn't get the 3rd party CI capability for os-brick until just recently | 19:13 |
| asselin_ | hemna, I think there's a different deadline for os-brick | 19:13 |
| hemna | asselin_, the problem is, that the HGST Cinder volume driver needs an os-brick Connector to land | 19:13 |
| winston-d | speaking for CI for os-brick | 19:13 |
| winston-d | s/for/of | 19:14 |
| hemna | asselin_, so the HGST driver shouldn't land until the os-brick patch lands, IMHO | 19:14 |
| hemna | asselin_, https://review.openstack.org/#/c/186588/ | 19:14 |
| asselin_ | hemna, interesting...ok...will let thingee decide | 19:14 |
| *** annashen has joined #openstack-cinder | 19:14 | |
| *** gouthamr has joined #openstack-cinder | 19:15 | |
| asselin_ | hemna, honestly, it's easy to add ci to os-brick once you have it for cinder | 19:15 |
| thingee | are we requiring ci for target drivers? | 19:16 |
| asselin_ | hemna, also they should add a cross-project depends-on to their patch if they don't already | 19:16 |
| hemna | asselin_, it should be, if you have the time to work on it. earlephilhower said he might not have time until tomorrow or Monday. | 19:16 |
| *** annegentle has joined #openstack-cinder | 19:16 | |
| hemna | asselin_, and I'm just trying to think ahead for him to help make sure he doesn't miss the deadling because of us taking so long to get the os-brick CI stuff in place. It's not that fair for him and the HGST driver. | 19:17 |
| hemna | thingee, I don't think we figured that out. | 19:17 |
| asselin_ | hemna, well the deadline for drivers is next week | 19:17 |
| asselin_ | hemna, so he has time to set it up for os-brick | 19:17 |
| asselin_ | hemna, he also needs a nova patch to land... | 19:18 |
| asselin_ | https://review.openstack.org/#/c/186580/ | 19:18 |
| hemna | oh ok L-1 is June 23 | 19:18 |
| *** Zhongjun has quit IRC | 19:18 | |
| hemna | I was worried it was tomorrow :) | 19:18 |
| hemna | phew | 19:18 |
| hemna | so maybe I'm premature asking for an exception for HGST already. | 19:19 |
| thingee | hemna: the deadline tomorrow just for us to have reasonable amount of time to work with drivers that actually have a chance | 19:19 |
| hemna | nevermind! :) | 19:19 |
| thingee | to avoid last minute submissions | 19:19 |
| winston-d | speaking of os-brick CI, the rbd connector is kind of different, 'cos my understanding is Nova won't be using it even when it lands. | 19:19 |
| hemna | thingee, ok my bad. | 19:19 |
| winston-d | the only usage for rbd connector is for volume migration | 19:19 |
| hemna | winston-d, so I had that question on the review. | 19:19 |
| winston-d | which unfortunately, it's not covered by CI | 19:19 |
| hemna | winston-d, it's quite a bit different than the libvirt volume driver code. | 19:19 |
| hemna | winston-d, but I do have a WIP in progress for nova to use os-brick | 19:20 |
| *** anshul has quit IRC | 19:20 | |
| hemna | winston-d, but not all libvirt volume drivers will be converted in the first pass though. | 19:20 |
| eharney | yeah i'm not sure i understand the thinking there on the rbd connector | 19:20 |
| hemna | eharney, +1 | 19:20 |
| hemna | I didn't get it either. | 19:20 |
| eharney | i mean... why the demand for CI now | 19:20 |
| hemna | but I'm not rbd expert | 19:20 |
| eharney | the connector is needed for generic volume migration | 19:21 |
| eharney | which isn't CI'd... | 19:21 |
| winston-d | unless we have volume migration case covered in CI, having a CI won't help validating RBD connector | 19:21 |
| hemna | I'd like to make sure that the RBC connector actuall functions the same as the nova libvirt volume driver | 19:21 |
| hemna | I'm not sure what the heck the current patch in os-brick actually does | 19:21 |
| winston-d | simply because Nova won't use that connector | 19:21 |
| eharney | right... what winston-d says | 19:21 |
| hemna | need an RBD 'guy' to look at it. | 19:21 |
| eharney | blocking the RBD connector for CI is holding things up for no benefit at this time | 19:21 |
| eharney | since it's not being used for anything but migration | 19:22 |
| hemna | winston-d, copy volume to image and image to volume can exercize os-brick in cinder | 19:22 |
| winston-d | again: the only usage for RBD connector is for volume migration | 19:22 |
| hemna | depending on the cinder volume driver of course | 19:22 |
| winston-d | hemna: let me check rbd's copy_image_to_volume | 19:22 |
| eharney | hemna: it _can_, but since those don't use brick now, i'm not sure why that's a demand | 19:22 |
| earlephilhower | asselin: I'm trying the OS_BRICK_REPO / OS_BRICK_BRANCH in my pre_test_hook, but _BRANCH never seems to apply in the scripts | 19:23 |
| asselin_ | earlephilhower, you talking about os-brick ci or cinder ci? | 19:23 |
| winston-d | hemna: no, rbd driver won't use connector to do copy_image_to_volume or upload_volume_to_image. | 19:24 |
| earlephilhower | asselin_: os-brick CI, but using just plain Cinder CI. Defined overrides just like for NOVA and CINDER, but logs don't show joy | 19:24 |
| *** marcusvrn has quit IRC | 19:24 | |
| earlephilhower | Windering if need to apply elsewhere? | 19:24 |
| asselin_ | earlephilhower, that won't work...you need to do something differently | 19:24 |
| hemna | winston-d, yah it looks like the rbd cinder volume driver doesn't currently use os-brick (which makes sense) | 19:24 |
| asselin_ | earlephilhower, you need to follow this template https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml#L179 | 19:25 |
| winston-d | hemna: right, so that means having a CI won't help unless CI has volume migration test cases | 19:25 |
| asselin_ | earlephilhower, this is the main difference: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml#L206 | 19:25 |
| hemna | winston-d, well specifically to CI rbd, but CI'ing that the RBD connector doesn't break anything else is still ok. | 19:25 |
| asselin_ | export DEVSTACK_PROJECT_FROM_GIT=os-brick | 19:25 |
| asselin_ | earlephilhower, do this is a new job | 19:26 |
| hemna | so if others can review the RBD connector that'd be great | 19:26 |
| marcusvrn1 | thingee: sry, I was out. I think there's a misunderstanding. We have one driver HBSD that has CI for FC and iSCSI (Hitachi HBSD CI). That patch (https://review.openstack.org/#/c/184999) is not a new driver, it's a patch for extending funcionality for a new hardware. Once you asked us to create a CI, I started to create a CI for each hardware/backend. We didn't create a CI for the iSCSI backend for new hardware yet | 19:26 |
| *** salv-orlando has joined #openstack-cinder | 19:26 | |
| hemna | I'm RBD challenged | 19:26 |
| hemna | https://review.openstack.org/#/c/186172 | 19:26 |
| winston-d | hemna: yeah, i will review the code | 19:26 |
| eharney | hemna: does that mean you don't want to block the connector patch for CI then? | 19:26 |
| hemna | eharney, yah I won't block it | 19:26 |
| hemna | it's not being used anywhere :) | 19:27 |
| hemna | my -1 was simply to raise the question and start the discussion | 19:27 |
| earlephilhower | asselin_: Can do, but what about the main Cinder CI? Still OK to to do a git fetch from the reviews bug and python setup.py before aything? | 19:27 |
| eharney | hemna: yeah, it's leading up to the generic migration work | 19:27 |
| marcusvrn1 | thingee: so, the driver is the same, HBSD driver, what we are doing in this patch is expanding its functionalities for new hardware | 19:27 |
| hemna | on 1) CI and 2) the difference between the patch and nova's rbd libvirt volume dirver. | 19:27 |
| hemna | because if on 2) we could make the patch work like nova's libvirt volume driver and get a release of os-brick out that had the new RBD connector, then I can update my nova WIP patch to change the existing rbd libvirt volume driver to use os-brick. | 19:28 |
| asselin_ | earlephilhower, yes...that's why I asked which ci. the job definition is different to test os-brick and cinder | 19:28 |
| hemna | but I guess we can do that later | 19:28 |
| thingee | marcusvrn1: ok hang on let me pull the page back up | 19:28 |
| asselin_ | earlephilhower, which are yourpatches? | 19:28 |
| marcusvrn1 | thingee: which etherpad have you updated? https://etherpad.openstack.org/p/cinder-thirdpartyCI-issues ? | 19:28 |
| hemna | eharney, https://review.openstack.org/#/c/175569/ my WIP in nova | 19:28 |
| hemna | eharney, there is also a glusterfs libvirt volume driver | 19:29 |
| hemna | smells like we should have a glusterfs os-brick connector as well | 19:29 |
| thingee | marcusvrn1: so hbsd_horcm is FC only? | 19:29 |
| eharney | hemna: i thought we had a remotefs connector | 19:29 |
| hemna | I'd like to eliminate all of the internals of the libvirt volume drivers | 19:29 |
| earlephilhower | asselin_: Got it. Let me try it out... | 19:30 |
| eharney | the glusterfs stuff in nova doesn't do anything other than mount and unmount, i think? | 19:30 |
| hemna | and then eventually get rid of libvirt volume drivers all together and put a shim to os-brick connectors. | 19:30 |
| eharney | right | 19:30 |
| hemna | eharney, I don't know anything about gluster | 19:30 |
| *** gouthamr has quit IRC | 19:30 | |
| eharney | hemna: what's being done for NFS? | 19:30 |
| *** ronenkat has quit IRC | 19:30 | |
| hemna | so I was confused about the remotefs vs. gluster vs. nfs, etc. stuffs | 19:30 |
| marcusvrn1 | thingee: yes, and we are adding the iSCSI support | 19:30 |
| eharney | yeah, gluster and nfs should be about the same afaik | 19:30 |
| thingee | marcusvrn1: where's the horcm iscsi ci? | 19:31 |
| hemna | eharney, ok maybe you can give me a hand understanding all of that | 19:31 |
| *** annashen has quit IRC | 19:31 | |
| eharney | hemna: well i think gluster and nfs both just go through the remotefs connector right? | 19:32 |
| marcusvrn1 | thingee: we don't have it yet | 19:32 |
| winston-d | hemna: RBD uses LibvirtNetVolumeDriver: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume.py#L242 | 19:32 |
| hemna | eharney, yah they both go through the RemoteFsConnector | 19:32 |
| winston-d | hemna: which isn't part of os-brick, at least for now. | 19:32 |
| thingee | marcusvrn1: ok, that would be where my confusion is. So we would need one for each. For our requirements we require all fabrics to be tested. https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#Third_Party_CI_Requirements | 19:32 |
| hemna | winston-d, yah that's issue #2 I had | 19:32 |
| winston-d | hemna: i mean for Nova | 19:33 |
| thingee | marcusvrn1: that's why you see some products with _iscsi or _fc | 19:33 |
| thingee | for their ci's | 19:33 |
| *** gouthamr has joined #openstack-cinder | 19:33 | |
| hemna | winston-d, I'd like the RBD connector in os-brick function the same as the nova code....but I'm confused by the LibvirtNetVolumeDriver being shared between iscsi, rbd | 19:33 |
| marcusvrn1 | thingee: hmm so we need one CI for each backend, right? | 19:33 |
| thingee | marcusvrn1: yes | 19:34 |
| hemna | winston-d, eharney https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume.py#L260-L266 | 19:34 |
| hemna | the source_protocol thing. | 19:34 |
| eharney | hemna: Nova uses the NetVolumeDriver path because RBD is handled from within qemu | 19:34 |
| eharney | hemna: so it doesn't mount anything on the compute node itself | 19:34 |
| winston-d | eharney: exactly | 19:34 |
| winston-d | hemna: the only thing NetVolumeDriver does in Nova, is to manipulate a few bunch of libvirt config (xml), that's all. | 19:34 |
| hemna | eharney, so do we need a NetVolumeConnector in os-brick ? | 19:35 |
| eharney | hemna: i guess that's kinda up to how things are divided between brick and nova... but i'm not sure it would actually do much? | 19:35 |
| *** e0ne_ is now known as e0ne | 19:35 | |
| hemna | the end goal being nuking virt/libvirt/volume.py | 19:35 |
| winston-d | hemna: personally I don't think so. | 19:35 |
| hemna | and shimming into os-brick to do all of it. | 19:35 |
| *** gouthamr has quit IRC | 19:35 | |
| eharney | that means brick has to handle libvirt internals? | 19:35 |
| winston-d | hemna: unless you want to have os-brick/virt/libivrt os-brick/virt/xenapi stuff in os-brick | 19:36 |
| hemna | eharney, I don't see where it's hacking libvirt internals | 19:36 |
| marcusvrn1 | thingee: hmmm now I understood, for each backend, even it's the same driver, we have to setup a CI | 19:36 |
| hemna | get_config just fetchies things from connection_info, and massages it into something that the caller expects. | 19:36 |
| eharney | hemna: the conf.source_* stuff is generating a libvirt XML object | 19:36 |
| eharney | so presumably if brick handled this, it would just read fields and them pass them to Nova's libvirt driver which would still do mostly the same thing it does now? | 19:37 |
| hemna | eharney, yah | 19:37 |
| jgriffith | hemna: wait... what? Nuking libvirt/volume? | 19:37 |
| hemna | the code that's inside of get_config would exist inside of connect_volume for the connector ? | 19:37 |
| thingee | right... different code paths, etc. Maybe you use zone managers... I don't know. I need to know that product works in that configuration. | 19:37 |
| thingee | marcusvrn1: ^ | 19:37 |
| hemna | heh | 19:38 |
| jgriffith | hemna: that seems out of scope from what I imagined | 19:38 |
| winston-d | still, even if we do that, it has nothing to do with rbd connector | 19:38 |
| *** anshul has joined #openstack-cinder | 19:38 | |
| eharney | hemna: i don't think so, because then os-brick has to load libvirt python | 19:38 |
| hemna | eharney, ick | 19:38 |
| jgriffith | hemna: I thought it was/should be for the host attach pieces only | 19:38 |
| *** gouthamr has joined #openstack-cinder | 19:38 | |
| eharney | which seems like not the right place | 19:38 |
| *** juzuluag has joined #openstack-cinder | 19:38 | |
| hemna | jgriffith, the libvirt/volume.py is nothing more than classes for each of the transports that do connect_volume and disconnect_volume | 19:39 |
| hemna | which is what os-brick connectors do | 19:39 |
| hemna | the dupe code | 19:39 |
| e0ne | jgriffith: hi. i'm sorry, i'd got internet connection issue yesterday. i'm ready to discuss issues and changes with 'detailed' views it you'll have a time | 19:39 |
| marcusvrn1 | thingee: nice! I got it | 19:39 |
| jgriffith | hemna: there's more in there though | 19:39 |
| winston-d | hemna: not exactly, the get_config function here: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume.py#L248 is specific to Nova. | 19:39 |
| hemna | winston-d, that's the part I'm trying to understand | 19:40 |
| hemna | winston-d, and that's specific to libvirt itself no? | 19:40 |
| winston-d | yes, i think so | 19:40 |
| hemna | so it doesn't make sense to replace that at all with anything in os-brick | 19:40 |
| hemna | dunno | 19:41 |
| hemna | I'm trying to understand it | 19:41 |
| eharney | it is, because the get_config() called on the super() generates an initial libvirt-specific config obj | 19:41 |
| eharney | at 250 | 19:41 |
| hemna | so what I'm trying to get to is can we get nova's code to a point where new connectors showing up in os-brick automatically are available to nova without adding a new libvirt volume driver. | 19:41 |
| hemna | eharney, ah ok. cool thanks | 19:41 |
| *** gouthamr has quit IRC | 19:41 | |
| winston-d | heading out for quick lunch, brb | 19:42 |
| *** gouthamr has joined #openstack-cinder | 19:42 | |
| eharney | libvirt/config.py is the translation layer between those python objects and libvirt XML | 19:42 |
| jgriffith | hemna: yeah, I get where you're going and like it, but I think statements like "replace all of libvirt/volume.py are naive and a bit unrealistic :) | 19:42 |
| hemna | :) | 19:43 |
| hemna | I'm sure they are. | 19:43 |
| jgriffith | ;) | 19:43 |
| hemna | but that was my hope | 19:43 |
| hemna | if it's impossible, that's cool | 19:43 |
| hemna | but can we get mostly there ? | 19:43 |
| jgriffith | ayway... sorry to butt in; just caught that statement and was like whhaaaaa | 19:43 |
| hemna | and how to get to that point | 19:43 |
| hemna | jgriffith, yah no worries. | 19:43 |
| jgriffith | You really want to take over libvirt xml :) | 19:43 |
| hemna | no! | 19:43 |
| hemna | haha | 19:43 |
| eharney | lol | 19:44 |
| hemna | I just want to stop hacking in nova :) | 19:44 |
| hemna | and how can I get there. | 19:44 |
| hemna | ok I need food.... | 19:44 |
| *** hemna is now known as hemnafood | 19:44 | |
| thingee | me too | 19:45 |
| thingee | bbl | 19:45 |
| e0ne | harlowja: hi. are you around? | 19:45 |
| jgriffith | sigh | 19:45 |
| *** gouthamr has quit IRC | 19:46 | |
| jgriffith | e0ne: hey | 19:47 |
| *** gouthamr has joined #openstack-cinder | 19:47 | |
| e0ne | jgriffith: hi | 19:48 |
| jgriffith | e0ne: so | 19:49 |
| jgriffith | e0ne: where were we | 19:49 |
| e0ne | jgriffith: we discussed current implementation of the list view | 19:50 |
| e0ne | jgriffith: it handles --detailed param and it doesn't make our REST API easy | 19:50 |
| xyang1 | Can anyone help +A this one? https://review.openstack.org/#/c/183762 . It has two +2's. e0ne, eharney, jgriffith ? | 19:51 |
| thingee | asselin_, jgriffith: for the contacting on correcting third party issues, lets cc the maintainers but keep it to the third-party-announce mailing list | 19:51 |
| harlowja | e0ne i'm around | 19:51 |
| harlowja | are u around :-P | 19:51 |
| thingee | just so all the convos are in the open? | 19:51 |
| harlowja | i saw that nice trace, looks like more things to optimize in taskflow | 19:51 |
| thingee | just asked in infra if this would be fine, and it looks like there were no objections | 19:51 |
| harlowja | *and yes i have idears | 19:52 |
| harlowja | but none of that CI stuff | 19:52 |
| harlowja | lol | 19:52 |
| e0ne | harlowja: i'll say only my imo | 19:52 |
| harlowja | meh, its just things that can get optimized | 19:53 |
| harlowja | tis the way of life | 19:53 |
| jgriffith | e0ne: so my feeling was I didn't like that being part of list from teh client inparticular, and my argument was that it's all the same info as "show" so use "show" | 19:53 |
| e0ne | harlowja: 20-30 requests to DB is tooo lot for our case | 19:53 |
| thingee | e0ne, harlowja: I'll say only my IMHO | 19:53 |
| *** rbradbury has joined #openstack-cinder | 19:53 | |
| harlowja | lol | 19:53 |
| harlowja | e0ne sure, so we can like fix that, in fact https://github.com/openstack/taskflow/blob/master/taskflow/persistence/base.py#L120 will reduce it to 1 call | 19:53 |
| e0ne | thingee, harlowja: what CI are you talking about? | 19:53 |
| harlowja | i just need to verify thats used where it needs to be :-P | 19:53 |
| harlowja | e0ne prior joke :) | 19:53 |
| thingee | e0ne: harlowja is just being funny... I'm talking about these https://etherpad.openstack.org/p/cinder-thirdpartyCI-issues | 19:54 |
| harlowja | CI CI CI l ol | 19:54 |
| e0ne | jgriffith: it bacames a holly war | 19:55 |
| e0ne | jgriffith: personally, i prefer api that returns the same data for list and show | 19:55 |
| *** thangp has quit IRC | 19:55 | |
| harlowja | so e0ne otherwise, useful results, good to know, the question still IMHO revolves around what is wanted, if no resumption is wanted, then meh, i could optimize it to 1 call and it won't matter :-P | 19:55 |
| harlowja | i'll optimize it to -1 call! | 19:55 |
| e0ne | jgriffith: and detailed shoe could contain more data | 19:55 |
| jgriffith | e0ne: but of course :) | 19:55 |
| jgriffith | e0ne: so I guess get other peoples feedback and do what you like | 19:56 |
| e0ne | jgriffith: and of course, your arguments are absolutely valid | 19:56 |
| jgriffith | e0ne: the --detailed option isn't the end of the world for me | 19:56 |
| e0ne | jgriffith: :) | 19:56 |
| jgriffith | e0ne: but making the default have everything I do think is awful | 19:56 |
| e0ne | jgriffith: if i found a correct english version of proverb, it sould be like: different strokes for different folks | 19:58 |
| jgriffith | e0ne: LOL... seems about right | 19:58 |
| e0ne | jgriffith: i'll ask this question in openstack-dev and operators ML's | 19:59 |
| e0ne | jgriffith: it started like "adding snapshots field to 'show' command" and now we're trying to change list and show api responces | 20:00 |
| e0ne | jgriffith: we could get more opinions in the mailing lists than in the spec's review. let's try it | 20:01 |
| e0ne | harlowja: ~30 calls to DB for a pretty simple 2 tasks it's not very good from a performance point of view | 20:04 |
| *** gouthamr has quit IRC | 20:05 | |
| e0ne | harlowja: it's about 30% slower than w/o persistance | 20:05 |
| harlowja | e0ne sure, looks like a bunch of selects that just need to be optimized | 20:05 |
| e0ne | harlowja: agree | 20:06 |
| *** juzuluag has quit IRC | 20:06 | |
| harlowja | still goes back to my orginal point though | 20:06 |
| e0ne | harlowja: did you see performance result from rally? | 20:06 |
| *** Yogi11 has quit IRC | 20:06 | |
| harlowja | http://e0ne.github.io/cinder/db-profiler-results/volume/create-with-persistence.html ? | 20:06 |
| harlowja | or is there something else, idk | 20:06 |
| e0ne | harlowja: https://etherpad.openstack.org/p/taskflow_persistence_benchmarking | 20:07 |
| e0ne | thingee, harlowja: oh.. glad to see that we care about quality of CIs | 20:08 |
| *** Yogi1 has joined #openstack-cinder | 20:09 | |
| harlowja | e0ne state persistence (and resumption) isn't magic and/or free ;) | 20:10 |
| harlowja | but def things can get optimized as we need | 20:10 |
| e0ne | harlowja: :) | 20:10 |
| harlowja | *because each task state transtion becomes a mini-checkpoint (that needs to be saved somewhere) | 20:11 |
| harlowja | *unless u reduce your ability to resume from that state (which may be fine for some workflows) | 20:11 |
| *** amoturi has joined #openstack-cinder | 20:12 | |
| harlowja | u guys should try https://review.openstack.org/#/c/189526/ (which uses that capability to distribute work to workers, and resume it using those state checkpoints when u kill -9 or ctrl-c the workers) | 20:12 |
| harlowja | there's a reason i called the review 'Add indestructible 99 bottles of beer example' | 20:13 |
| harlowja | lol | 20:13 |
| harlowja | but again, adding this capability isn't 'free' or 'magic' | 20:13 |
| *** vokt has quit IRC | 20:14 | |
| harlowja | * https://review.openstack.org/#/c/189536/ (fixes the ability to ctrl-c workers, vs needing to kill -9 them) | 20:15 |
| *** gouthamr has joined #openstack-cinder | 20:16 | |
| harlowja | basically what happens, is that the song progresses on a worker, u kill it, then the song gets picked up by another worker, resumes from last state, and continues | 20:16 |
| harlowja | aka, u can't stop the song (the indestructible part, ha) | 20:16 |
| nikeshm | thingee smcginnis hemna DuncanT : CI status for hpmsa and lenovo -CI tests are passed and showing on these patches https://review.openstack.org/#/c/187853/ https://review.openstack.org/#/c/187707/ | 20:19 |
| *** juzuluag has joined #openstack-cinder | 20:20 | |
| e0ne | harlowja: thanks. i'll take a look on your patches tomorrow morning | 20:20 |
| harlowja | k | 20:20 |
| *** kjnelson has quit IRC | 20:20 | |
| e0ne | harlowja: maybe get profiler results with zookeeper also makes sense | 20:21 |
| *** annashen has joined #openstack-cinder | 20:21 | |
| harlowja | e0ne sure, but again, the question of what is desired from cinder is also useful, if nobody wants resumption, then well the point is mute (although optimizing taskflow would be good no matter) | 20:22 |
| e0ne | harlowja: ok. i like this feature, but i'm not not sure that we'll accept it to cinder with ~30% performance impact | 20:23 |
| harlowja | e0ne well help make it better in taskflow? | 20:23 |
| harlowja | i take patches to ;) | 20:24 |
| e0ne | harlowja: :) | 20:24 |
| e0ne | harlowja: unfortunately, i've got only 24hours per day and sometime i need to sleep:( | 20:25 |
| harlowja | ya, same here | 20:25 |
| *** vokt has joined #openstack-cinder | 20:25 | |
| harlowja | more important imho is the 'i like this feature' or 'i find this feature useful', optimize things as we go, those later points are IMHO the more important ones | 20:26 |
| *** timcl has quit IRC | 20:26 | |
| *** gouthamr has quit IRC | 20:29 | |
| e0ne | harlowja: ok, i like this feature because it's useful but it's pretty slow:( | 20:30 |
| *** alexpilotti has quit IRC | 20:31 | |
| tbarron | akerr: if you would be so kind as to look at https://review.openstack.netapp.com/#/c/566/ for QA+2 I would be, even more, in your debt. | 20:34 |
| tbarron | wrong window :-) | 20:35 |
| openstackgerrit | Kurt Martin proposed openstack/cinder: Update 3PAR user config help strings https://review.openstack.org/190811 | 20:43 |
| *** rmesta has joined #openstack-cinder | 20:44 | |
| *** Lee1092 has quit IRC | 20:45 | |
| * e0ne bb in ~12 hours | 20:45 | |
| *** annegentle has quit IRC | 20:45 | |
| *** marcusvrn1 has quit IRC | 20:49 | |
| *** e0ne has quit IRC | 20:54 | |
| *** kjnelson has joined #openstack-cinder | 20:58 | |
| *** jdurgin1 has joined #openstack-cinder | 20:59 | |
| *** akerr has quit IRC | 20:59 | |
| *** dannywil_ has joined #openstack-cinder | 20:59 | |
| *** jdurgin has quit IRC | 21:00 | |
| *** patrickeast_ has joined #openstack-cinder | 21:01 | |
| *** jungleboyj has quit IRC | 21:01 | |
| *** dannywilson has quit IRC | 21:02 | |
| *** patrickeast has quit IRC | 21:02 | |
| *** patrickeast_ is now known as patrickeast | 21:02 | |
| *** juzuluag has quit IRC | 21:04 | |
| *** amoturi has quit IRC | 21:04 | |
| openstackgerrit | Sean Chen proposed openstack/cinder: Tintri driver to manage existing backend storage objects https://review.openstack.org/188535 | 21:04 |
| *** setmason has quit IRC | 21:04 | |
| *** simondodsley has quit IRC | 21:05 | |
| *** angela-s|afk is now known as angela-s | 21:06 | |
| *** setmason has joined #openstack-cinder | 21:06 | |
| -openstackstatus- NOTICE: Gerrit has been restarted to terminate a persistent looping third-party CI bot | 21:06 | |
| *** lpetrut has quit IRC | 21:09 | |
| *** annegentle has joined #openstack-cinder | 21:13 | |
| scottda | What version of cinderclient was Keystone v3 support introduced? Has it been in there a while? | 21:14 |
| openstackgerrit | Sean Chen proposed openstack/cinder: Tintri driver to manage existing backend storage objects https://review.openstack.org/188535 | 21:15 |
| *** anshul has quit IRC | 21:17 | |
| *** Yogi1 has quit IRC | 21:18 | |
| scottda | OK, looks like it merged a year ago. | 21:20 |
| *** kjnelson has quit IRC | 21:20 | |
| *** hemnafood is now known as hemna | 21:21 | |
| *** jungleboyj has joined #openstack-cinder | 21:24 | |
| *** rbradbury has quit IRC | 21:24 | |
| anteaya | thingee: does cinder have a nova liason? | 21:24 |
| anteaya | in the nova meeting today there was mention of some api changes, about not having pass through to neutron and cinder | 21:25 |
| anteaya | I talked to sc68cal the neutron/nova person but thought I would let you know too | 21:25 |
| anteaya | I don't fully know what it means but it might be worth someone from cinder looking at today's nova meeting logs | 21:25 |
| thingee | anteaya: thanks for letting me know. I'll check it out | 21:26 |
| earlephilhower | asselin_, hemna: Thx for the os-brick CI help. The HGST CI system is now reporting on os-brick patches, too. | 21:27 |
| hemna | earlephilhower, woot! | 21:27 |
| asselin_ | earlephilhower, wow that was fast! :) | 21:28 |
| earlephilhower | 1-line change, really, w/new devstack-gate scripts | 21:28 |
| earlephilhower | But my CI system takes 26 min/run, not 20 like Scality. :( | 21:28 |
| IlyaG | what do u run CI on? | 21:29 |
| anteaya | thingee: thank you | 21:30 |
| *** ociuhandu has joined #openstack-cinder | 21:32 | |
| earlephilhower | Packstack on a 2x2.5GHz. 24 core w/64G of memory for now. | 21:33 |
| hemna | earlephilhower, so your HGST volume driver doesn't use brick to do copy image <--> volume commands | 21:34 |
| *** eharney has quit IRC | 21:35 | |
| hemna | and os-brick doesn't have the HGST connector yet | 21:38 |
| hemna | so I'm curious how the tempest volume attaches are working in your CI ? | 21:38 |
| hemna | and nova doesn't have an HGST libvirt volume driver | 21:38 |
| *** annegentle has quit IRC | 21:38 | |
| *** ian_brown has joined #openstack-cinder | 21:39 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Support SMI-S provider v8.0.3 in VMAX driver https://review.openstack.org/189517 | 21:39 |
| earlephilhower | hemna: Overrides are required for the 3 patches in the CI build CINDER_REPO, NOVA_REPO, and I do a manual git pull/install for OS-BRICK | 21:40 |
| earlephilhower | hemna: Should be in localrc dump | 21:40 |
| mriedem | smcginnis: jgriffith: question in here even though it's merged, https://review.openstack.org/#/c/189967/1/cinder/cmd/volume.py | 21:41 |
| mriedem | i'm asking b/c it's proposed to stable/kilo also | 21:41 |
| *** jungleboyj has quit IRC | 21:41 | |
| hemna | earlephilhower, ok I see the nova patch now from the logs | 21:42 |
| hemna | we'll have to undo that patch after os-brick lands | 21:42 |
| smcginnis | mriedem: Thanks. I'll take a look at the patch you reference. Wasn't aware of that. | 21:43 |
| earlephilhower | hemna: So does that mean that I don't need to get into Nova L-1, then? One less thing to juggle would be most welcome. For a newcomer, getting a driver in is daunting... | 21:43 |
| hemna | I'm not sure what to do here, as we are in a strange state. | 21:43 |
| *** dims_ has joined #openstack-cinder | 21:44 | |
| hemna | I think your nova patch is fine for now | 21:44 |
| hemna | I'm still working on the nova patch to pull in os-brick | 21:44 |
| hemna | https://review.openstack.org/#/c/175569/ | 21:44 |
| hemna | but without your current nova patch, there is no way your CI will work | 21:45 |
| earlephilhower | Wow, that would be cool. Leaving the Nova patch in but not getting accepted for Nova L1, though, would be doable. When os-brick patch lands, I can just abandon it. | 21:45 |
| earlephilhower | I think | 21:45 |
| *** jwcroppe has joined #openstack-cinder | 21:45 | |
| Swanson | hemna: how's that wip coming along? | 21:46 |
| *** dims has quit IRC | 21:46 | |
| hemna | Swanson, working on it today | 21:46 |
| hemna | trying to get the unit tests working | 21:46 |
| *** jdurgin1 has quit IRC | 21:47 | |
| harlowja | e0ne guess u aren't around, but https://review.openstack.org/#/c/190833 (a low hanging fruit to reduce a few persistence interactions...) | 21:49 |
| harlowja | others likely easily doable... | 21:49 |
| *** jdurgin has joined #openstack-cinder | 21:49 | |
| *** annashen has quit IRC | 21:53 | |
| *** jungleboyj has joined #openstack-cinder | 21:53 | |
| *** crose has quit IRC | 21:54 | |
| *** rushil has quit IRC | 21:54 | |
| *** dannywil_ has quit IRC | 21:54 | |
| *** rushil has joined #openstack-cinder | 21:56 | |
| *** dannywilson has joined #openstack-cinder | 21:56 | |
| smcginnis | mriedem: Yeah, just blame it on jungleboyj. ;) | 21:56 |
| *** jdandrea_ has joined #openstack-cinder | 21:56 | |
| *** jdandrea has quit IRC | 21:57 | |
| *** jdandrea_ is now known as jdandrea | 21:57 | |
| *** akerr has joined #openstack-cinder | 21:57 | |
| mriedem | i don't understand it so i will | 21:58 |
| mriedem | i hate that every project has it's own service and wsgi modules | 21:58 |
| jungleboyj | smcginnis: Huh? What did I do now. | 21:58 |
| smcginnis | mriedem: Would be great to have consistency. I doubt that's possible now though. | 21:58 |
| smcginnis | jungleboyj: Everything. | 21:58 |
| mriedem | smcginnis: isn't that what apache is for? | 21:58 |
| jungleboyj | smcginnis: Thanks man. | 21:58 |
| smcginnis | ;) | 21:59 |
| smcginnis | jungleboyj: Just questions on the db.dispose() thing. | 21:59 |
| *** diogogmt has quit IRC | 21:59 | |
| *** mriedem is now known as mriedem_away | 21:59 | |
| jungleboyj | Yeah, my network dropped out and I had to go talk to Matt f2f. | 21:59 |
| jungleboyj | Spare me the network jokes, please. | 22:00 |
| *** annegentle has joined #openstack-cinder | 22:00 | |
| jgriffith | jungleboyj: care to update the review for historical purposes then? | 22:00 |
| smcginnis | jungleboyj: I'm not saying a thing. | 22:00 |
| jgriffith | jungleboyj: because I know that I for one had to do a good deal of reading to get a basic understanding up what you were actually trying to solve with that patch to begin with | 22:00 |
| jgriffith | jungleboyj: Tried to ping you here for more info | 22:01 |
| jgriffith | jungleboyj: but you wre "jungleboyaway" or some nonsense :) | 22:01 |
| winston-d | mriedem_away: service module is mostly just a wrapper for what's in oslo. | 22:01 |
| winston-d | mriedem_away: for deploying API-like service using apache, is there any comprehensive document for that? | 22:02 |
| jungleboyj | Sure, I can do that. | 22:02 |
| hemna | smcginnis, wifi ? | 22:02 |
| thingee | asselin_: ping | 22:03 |
| asselin_ | thingee, give me a few | 22:03 |
| jgriffith | flip214: you online, or just your machine logged in? | 22:04 |
| jungleboyj | jgriffith: Which review do you want that in?> | 22:05 |
| jungleboyj | The one in master? | 22:05 |
| jgriffith | jungleboyj: yeah | 22:05 |
| jgriffith | jungleboyj: please | 22:05 |
| jgriffith | :) | 22:05 |
| jungleboyj | Ok, late for another meeting but I will do that tonight. | 22:06 |
| jgriffith | jungleboyj: just a respone to mriedem_away 's questions, so folks know you addressed his points | 22:06 |
| jgriffith | jungleboyj: as opposed to "I went and talked to him f2f" | 22:06 |
| jungleboyj | jgriffith: Oh good point. :-) | 22:07 |
| uberjay | asselin_: good news/bad news -- re-applying the puppet config (i.e., install_master.sh) and restarting the jenkins master node appears to have fixed my issues. | 22:07 |
| uberjay | mysterious issues worry me | 22:08 |
| *** rushil has quit IRC | 22:08 | |
| patrickeast | uberjay: that seems about on par for how these ci system work | 22:10 |
| asselin_ | thingee, hi | 22:12 |
| asselin_ | uberjay, what's the bad news? | 22:12 |
| asselin_ | oh..the mysterious issues.... | 22:12 |
| uberjay | asselin_: oh, the bad news is I'm not really sure why it was failing in the first place... but i'm happy for now and will keep an eye on it | 22:13 |
| uberjay | patrickeast: yeah, but I don't have to like it! :) | 22:13 |
| *** jungleboyj has quit IRC | 22:14 | |
| thingee | asselin_: I've been looking at CI's like vmware and just trying to understand where communication is for what logs need to be given | 22:14 |
| thingee | asselin_: example this only appears to mention testr_results http://docs.openstack.org/infra/system-config/third_party.html | 22:15 |
| asselin_ | thingee, that's the correct link | 22:15 |
| asselin_ | it's under requirements | 22:15 |
| asselin_ | http://paste.openstack.org/show/284609/ | 22:16 |
| thingee | asselin_: yeah I guess that's the problem is it's not very specific about what are environment details | 22:16 |
| thingee | what are "openstack logs" | 22:16 |
| thingee | I would say by these requirements, ci's like vmware maybe fine? ... just trying to see if we can be more specific so people can find this information, instead of us having to come down on people | 22:17 |
| asselin_ | "Include a public link to all test artifacts to make debugging failed tests easier " | 22:17 |
| asselin_ | i think that high level statement is accurate | 22:17 |
| asselin_ | if a reviewer can't tell what went wrong, or why it worked, then the results are not useful | 22:18 |
| asselin_ | so may as well turn it off | 22:18 |
| thingee | asselin_: well, I'm reviewing this https://etherpad.openstack.org/p/cinder-thirdpartyCI-issues and things like make sure to include localrc local.conf etc is just not clearly explained the list. This list of CI's with problems is more clear on what they should be including than the third party ci documentation itself. | 22:19 |
| asselin_ | thingee, that's jgriffith's interpretation of the requirements | 22:20 |
| asselin_ | thingee, imho localrc and local.conf are helpful, but the requirements really state the etc/cinder/cinder.conf and related conf files are what is needed | 22:20 |
| thingee | asselin_: ok... I guess the point of me talking to you about this is seeing if it makes sense to include this information either in the general requirements, or the cinder ci requirements. | 22:20 |
| jgriffith | thingee: asselin_ feel free to delete anything/everything I wrote | 22:21 |
| thingee | haha | 22:21 |
| jgriffith | thingee: asselin_ seemed to be that things like localrc and screen-logs were kinda important | 22:21 |
| thingee | jgriffith: I just want to be inline with the rest of you in my communication to these maintainers | 22:21 |
| jgriffith | thingee: my approach was "if it's in the gate logs, it should be in *your* logs" | 22:21 |
| asselin_ | jgriffith, I'm looking here: http://docs.openstack.org/infra/system-config/third_party.html#requirements | 22:21 |
| asselin_ | and trying to match up what that says vs. reality | 22:22 |
| jgriffith | thingee: asselin_ to the point of saying: "here's the template with all the expected files and dir structure" You can have more, but not less | 22:22 |
| smcginnis | jgriffith: +1 | 22:22 |
| *** annegentle has quit IRC | 22:23 | |
| asselin_ | jgriffith, I've no objection to aligning with gate logs. that's been my personal approach since day 1 | 22:23 |
| jgriffith | asselin_: indeed, I know it has | 22:23 |
| jgriffith | asselin_: thingee I'm just unclear on the questions/statements then? | 22:24 |
| jgriffith | asselin_: thingee I mean............ http://logs.openstack.org/33/185833/4/check/check-tempest-dsvm-full/1fed34f/ | 22:24 |
| jgriffith | ^^ That's the stuff I expect to see | 22:24 |
| patrickeast | +1 for matching up with whats in the gate logs | 22:24 |
| *** amoturi has joined #openstack-cinder | 22:25 | |
| jgriffith | Now if something like rabbit, apache etc isn't there I don't care as much; but screen-xxx.txt.gz and conf files are kinda critical IMHO | 22:25 |
| asselin_ | jgriffith, yes, and that's what the requirements are trying to state | 22:25 |
| smcginnis | I think the problem is they are too general and open to interpretation. | 22:27 |
| *** akerr has quit IRC | 22:27 | |
| asselin_ | if cinder wants to clarify what those mean for cinder, then I don't see an issue. | 22:29 |
| asselin_ | That page, however, is trying to apply to all current programs/projects | 22:29 |
| smcginnis | I'd rather not see per-project requirements. Some basic, common, explicit requirements would be ideal, IMO. | 22:30 |
| asselin_ | smcginnis, is this good enough? http://docs.openstack.org/infra/system-config/third_party.html#requirements | 22:30 |
| *** jwcroppe has quit IRC | 22:31 | |
| smcginnis | asselin_: I'd like to see local.conf and screen logs called out. | 22:31 |
| *** jwcroppe has joined #openstack-cinder | 22:31 | |
| smcginnis | But I guess thinking about it, there's no requirement that they are using devstack for CIs | 22:31 |
| smcginnis | So I guess that isn't necessarily a common thing. | 22:32 |
| smcginnis | Well, common, but not required. | 22:32 |
| asselin_ | smcginnis, right | 22:32 |
| *** raunak has joined #openstack-cinder | 22:32 | |
| smcginnis | I guess we just need to get to the point where everyone is running your scripts so they are all consistent. :) | 22:32 |
| asselin_ | for the record, I use devstack-gate | 22:33 |
| hemna | smcginnis, +1 | 22:33 |
| asselin_ | whatever jenkins does, so does our ci | 22:33 |
| *** bnemec has quit IRC | 22:34 | |
| *** salv-orlando has quit IRC | 22:34 | |
| thingee | jgriffith: I'm saying there is a lack of documentation on what we expect from CIs. | 22:34 |
| thingee | http://docs.openstack.org/infra/system-config/third_party.html | 22:35 |
| thingee | this mentions testr_results.html | 22:35 |
| thingee | so I was having a discussion with you and asselin_ to see if maybe it makes sense for us to update and be inline with what we're going to communicate to people | 22:35 |
| thingee | so we can point people to that for reference. and new comers in the future | 22:35 |
| thingee | to update the documentation* | 22:36 |
| asselin_ | thingee, +1 we should stick to this reference | 22:36 |
| asselin_ | thingee, if it's helpful, we can include examples to that. I can submit a patch mentioned screen, localrc, etc. | 22:36 |
| thingee | asselin_: that would be great. | 22:36 |
| thingee | asselin_: I've already started contact some of the people on this list and realized that I don't think I even know what's correct. | 22:37 |
| jgriffith | thingee: asselin_ what about http://docs.openstack.org/infra/system-config/third_party.html#requirements | 22:38 |
| thingee | jgriffith: so that's the problem. I was saying your list is more descriptive than that. | 22:39 |
| jgriffith | thingee: honestly IMHO my list is "obvious" | 22:39 |
| jgriffith | thingee: most of the items on there are "dead link to results" | 22:39 |
| jgriffith | thingee: and the wiki says: very clearly | 22:40 |
| thingee | jgriffith: I agree it's obvious to "us", but I'm concerned with new comers. I understand there is expectancy for people to learn OpenStack, but there also has to be some explanation somewhere. | 22:40 |
| jgriffith | Environment details, Test configuration, OpenStack logs and Tempest logs | 22:40 |
| thingee | openstack logs ... can mean a lot of things. | 22:40 |
| thingee | environment details. ... same thing | 22:40 |
| jgriffith | thingee: well then update that doc :) | 22:40 |
| jgriffith | thingee: or like I said before: Open a link to a Gate run and there's the shit you need to have :) | 22:41 |
| *** amoturi has quit IRC | 22:41 | |
| thingee | jgriffith: yeah that would be the plan, again, I just wanted a discussion with you and asselin_ if we felt we should make these ideas more clear in the documentation going out to people | 22:41 |
| jgriffith | thingee: Oh... | 22:41 |
| jgriffith | thingee: well, sure... clarity never hurts | 22:41 |
| *** raunak has quit IRC | 22:42 | |
| jgriffith | thingee: but like I said, I'm of the opinion that the bulk of things are just half-assed, not because they don't know, because it requires effort | 22:42 |
| *** raunak has joined #openstack-cinder | 22:42 | |
| jgriffith | thingee: I'm not here to spoon feed CI to anyone any more... at all | 22:42 |
| jgriffith | I fell into that trap, not doing it | 22:42 |
| jgriffith | again | 22:43 |
| thingee | jgriffith: totally understand. | 22:43 |
| jgriffith | asselin_: and others have provided all the tools to duplicate the CI setups | 22:43 |
| thingee | jgriffith: I know some of what you've been through | 22:43 |
| thingee | not going to claim I know all of it ;) | 22:43 |
| thingee | but when you get to the point of a docker file and go... | 22:43 |
| thingee | hehe | 22:44 |
| jgriffith | thingee: That's what people should use. Others who are more adventerous and can pull it off can do whatever they want as long as they can "pull it off" ;) | 22:44 |
| jgriffith | thingee: yeah... :) | 22:44 |
| asselin_ | jgriffith, yes that link is exactly what we're talking about. I can submit a clarifying patch | 22:45 |
| thingee | I get it. I just want to make sure our messaging is consistent to maintainers. asselin_ is here saying "that's jgriffith 's interpretation of the requirements". and that worries me. | 22:46 |
| thingee | because we plan to share the effort of communication to maintainers. | 22:46 |
| thingee | asselin_: thank you | 22:47 |
| thingee | I'll be happy to review too | 22:47 |
| jgriffith | thingee: yeah, pehaps it would be good to just remove my etherpad | 22:48 |
| rlucio | thingee: fyi, im putting toes back in the water for VMEM CI, it should be running for existing drivers now | 22:48 |
| jgriffith | thingee: so there's no confusion | 22:48 |
| thingee | jgriffith: No I think it's fine. This is a good starting point. | 22:48 |
| rlucio | thingee: im testing the new driver's CI now, should be ready soon | 22:49 |
| thingee | rlucio: cutting it close :) | 22:49 |
| jgriffith | thingee: I think I'd disagreee. If asselin_ is calling it out as "my interpretation" and doesn't agree then there's certainly a problem there | 22:49 |
| jgriffith | asselin_: is WAY more invested and involved in the CI stuff than I am | 22:50 |
| thingee | jgriffith: ok well if most are just the logs don't show properly at all, i think we can all agree on those. | 22:50 |
| rlucio | thingee: srsly, i'll save the story for another day, but progress is progress | 22:51 |
| asselin_ | thingee, jgriffith I don't think local.rc or local.conf are requirements per the web-site. I do think the /etc/cinder/cinder.conf and other service configurations are required | 22:51 |
| *** amoturi has joined #openstack-cinder | 22:51 | |
| asselin_ | but it's definitely nice to include those localrc and local.conf files | 22:52 |
| thingee | rlucio: if I had a beer for everytime I heard something about a story :) ... well.. I'd probably cut myself from drinking for a bit. | 22:52 |
| thingee | rlucio: I guess what I'm saying is, you're not alone | 22:52 |
| *** diogogmt has joined #openstack-cinder | 22:52 | |
| jgriffith | thingee: asselin_ so I'm confused... "my interpretation" seems to align just fine | 22:53 |
| *** drjones has joined #openstack-cinder | 22:53 | |
| jgriffith | thingee: asselin_ logs must be viewable, logs must be published and publicly accessible | 22:53 |
| jgriffith | Thos are the bulk of what i listed | 22:53 |
| *** annashen has joined #openstack-cinder | 22:54 | |
| jgriffith | so I'm unclear on why asselin_ stated "that's jgriffith 's interpretation" | 22:54 |
| asselin_ | jgriffith, ok so then we agree | 22:54 |
| *** _cjones_ has quit IRC | 22:54 | |
| rlucio | thingee: yep, if there's one thing i've learned its that automation is an "adventure" | 22:54 |
| thingee | yay! my job here is done | 22:54 |
| rlucio | thingee: always nice to know im not the only one :) | 22:54 |
| *** _cjones_ has joined #openstack-cinder | 22:54 | |
| jgriffith | asselin_: so then why the comment and the long drawn out pointless conversation about it? | 22:54 |
| *** setmason has quit IRC | 22:55 | |
| *** annashen has quit IRC | 22:55 | |
| jgriffith | There, I put a disclaimer at the top of the etherpad. I'm now officially done with the CI dance | 22:55 |
| thingee | jgriffith: well my point of start discussion was just doc improvement and us being inline with what we want to see in ci reports. Whether we were aligned or not from the start, I just wanted a discussion on what we agree on. It seems we agree on the specific files to make things more clear, so that's good! | 22:55 |
| thingee | jgriffith: aww come on, I thought I was going to have some help on communication | 22:56 |
| asselin_ | jgriffith, sorry...I think this isthe line that started confusion " doesn't include all of the needed log files, no local.rc or local.conf" | 22:56 |
| asselin_ | jgriffith, but now we are on the same page..that was just your quick note about missing conf files I suppose | 22:56 |
| jgriffith | asselin_: sure | 22:57 |
| jgriffith | asselin_: so here's the thing.... | 22:57 |
| jgriffith | EVERYTHING in OpenStack any more is SOOO bogged down with beuraucracy, red-tape and bike shedding that frankly it's tiring | 22:57 |
| *** drjones has quit IRC | 22:57 | |
| *** amoturi has quit IRC | 22:58 | |
| thingee | I hear playing bruno mars - "just the way you are" works for some vendors' openstack CI's. | 22:58 |
| hemna | it seems that way, because it's been the wild west for a long time. | 22:58 |
| jgriffith | When somebody (including myself) says "hey... let's write up some notes on things that we think need fixed up" and it results in various statements like the ones here and a 30+ minute conversation it just says "ok... I have better things to do with my time" | 22:58 |
| jgriffith | hemna: fair enough | 22:59 |
| *** amoturi has joined #openstack-cinder | 22:59 | |
| hemna | I think talking about this stuff is a good thing though IMO | 22:59 |
| jgriffith | hemna: ok | 22:59 |
| thingee | jgriffith: if you like, I can just start coming up rules with discussion with you and asselin_ and things will just happen :) | 23:00 |
| asselin_ | jgriffith, for what it's worth..I'm trying to get everyone to converge into a working solution that's maintained upstream...and avoid a lot of this.... | 23:00 |
| thingee | without discussion* | 23:00 |
| hemna | General thingee | 23:00 |
| hemna | all bow | 23:00 |
| jgriffith | alright... so missing the point all the way around | 23:00 |
| openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Add Virtuozzo Storage Volume Driver https://review.openstack.org/188869 | 23:00 |
| thingee | hemna: I promise there will be a ML post about it from a month ago I will reference ;) | 23:01 |
| jgriffith | do me a favor and just ignore me at this point | 23:01 |
| hemna | :) | 23:01 |
| thingee | jgriffith: have a beer | 23:01 |
| thingee | jgriffith: or ice cream | 23:01 |
| thingee | or both | 23:01 |
| Swanson | I've yet to meet the beer that goes with ice cream. | 23:02 |
| thingee | Swanson: stone smoked porter with vanilla ice cream | 23:02 |
| thingee | done | 23:02 |
| hemna | thingee, +1 | 23:02 |
| patrickeast | thingee: +1 | 23:03 |
| Swanson | I could see a porter going with ice cream. | 23:03 |
| patrickeast | a good porter or oatmeal stout does the trick | 23:03 |
| thingee | patrickeast: yes sir | 23:03 |
| * thingee goes back to break ci's with api v1 removal patches | 23:03 | |
| *** krtaylor has quit IRC | 23:04 | |
| *** annashen has joined #openstack-cinder | 23:11 | |
| *** markvoelker has quit IRC | 23:11 | |
| openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Add Virtuozzo Storage Volume Driver https://review.openstack.org/188869 | 23:13 |
| uberjay | thingee: oh, that sounds like a great idea. (there is a cafe in Emeryville that will make you a milkshake with Guinness and chocolate ice cream, which is really delicious :) | 23:15 |
| *** zhenguo has joined #openstack-cinder | 23:15 | |
| *** amoturi has quit IRC | 23:17 | |
| *** ganso_ has quit IRC | 23:18 | |
| *** annashen has quit IRC | 23:21 | |
| *** annegentle has joined #openstack-cinder | 23:24 | |
| openstackgerrit | Earle F. Philhower, III proposed openstack/cinder: Implement Cinder Volume driver for HGST Solutions https://review.openstack.org/186580 | 23:29 |
| *** annegentle has quit IRC | 23:29 | |
| *** chlong has joined #openstack-cinder | 23:35 | |
| thingee | oh hey my 3d model from the coreos booth is available http://coreosopenstack.tumblr.com/post/119897832839 | 23:35 |
| thingee | jgriffith: ^ | 23:35 |
| thingee | I went respectfully | 23:36 |
| *** annashen has joined #openstack-cinder | 23:38 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/os-brick: optimize multipath call to identify IQN https://review.openstack.org/190864 | 23:42 |
| *** hemna is now known as hemnafk | 23:45 | |
| jgriffith | thingee: haha... fallen on your sword | 23:45 |
| asselin_ | thingee, jgriffith anteaya better? https://review.openstack.org/190866 | 23:45 |
| asselin_ | patrickeast, smcginnis ^^ | 23:46 |
| patrickeast | +1 | 23:47 |
| anteaya | asselin_: here is my problem | 23:47 |
| anteaya | if we make that a requirement then someone needs to enforce it | 23:48 |
| anteaya | I'm not saying it isn't a good structure | 23:48 |
| anteaya | the problem is that not all projects across openstack will agree with and enforce this structure | 23:48 |
| anteaya | so someone setting up a ci for neutron asks neutron about this requirement | 23:49 |
| *** salv-orlando has joined #openstack-cinder | 23:49 | |
| anteaya | neutron says I don't know | 23:49 |
| anteaya | so the ci operator thinks they can ignore all the requirements | 23:49 |
| anteaya | see what I mean? | 23:49 |
| jgriffith | anteaya: bahh... I reviewed before seeing your comments here :) | 23:50 |
| asselin_ | anteaya, it's not changing the requriments, just helping people understand what they mean | 23:50 |
| anish | uberjay: wha. what is this place ? | 23:50 |
| jgriffith | anteaya: and here inlies the problem :) | 23:50 |
| jgriffith | enforcement | 23:50 |
| uberjay | anish: rudy's can't fail cafe | 23:50 |
| patrickeast | well wait, this started because we are trying to enforce them and the requirements were apparently not clear enough… | 23:50 |
| anish | uberjay: many thanks | 23:51 |
| patrickeast | so is the suggestion that we don’t update those particular ones, and have cinder specific requirements? | 23:51 |
| anteaya | it is an addition to the requirements section | 23:51 |
| patrickeast | anteaya: ^ | 23:51 |
| anteaya | or suggestions | 23:51 |
| jgriffith | anteaya: seems to me it's just a "clarification" | 23:51 |
| anteaya | sure and for you it is | 23:51 |
| anteaya | since you care | 23:51 |
| jgriffith | anteaya: are you sure about that :) | 23:51 |
| anteaya | but neutron doesn't care anymore | 23:51 |
| jgriffith | anteaya: I see your point | 23:51 |
| anteaya | you are talking about it | 23:51 |
| anteaya | so if we make something manditory | 23:52 |
| jgriffith | anteaya: darn you and your vulcan logic! | 23:52 |
| anteaya | and tell folks to ask the project about it | 23:52 |
| anteaya | and the project doesn't care | 23:52 |
| anteaya | it isn't actually mandetory | 23:52 |
| uberjay | anish: np :) | 23:52 |
| * anteaya polishes her pointy ears | 23:52 | |
| anteaya | I so wish the situation were different | 23:53 |
| anteaya | and operators cared and became involved | 23:53 |
| asselin_ | anteaya, the point is to help reviewers understand ci results. | 23:53 |
| anteaya | like cinder ops | 23:53 |
| anteaya | oh I understand the point | 23:53 |
| anteaya | and agree with it | 23:53 |
| anteaya | the problem I have is with having it as a requirement | 23:53 |
| *** salv-orlando has quit IRC | 23:53 | |
| asselin_ | anteaya, it already is | 23:54 |
| *** angela-s has quit IRC | 23:54 | |
| anteaya | having logs is a requiremtn | 23:55 |
| anteaya | requirement | 23:55 |
| anteaya | not having them in a certain structure | 23:55 |
| asselin_ | no structure was specified, just an example was given | 23:56 |
| patrickeast | maybe i’m missing something, that patch just gives examples of what those requirements mean | 23:56 |
| patrickeast | not new requirements | 23:56 |
| patrickeast | just clarification of what the existing ones mean | 23:57 |
| anteaya | have you spent any time in nova? | 23:57 |
| patrickeast | nothing significant, no | 23:57 |
| anteaya | or talked to anyone in neutron or nova about their current third party ci situation | 23:57 |
| anteaya | okay | 23:57 |
| anteaya | well I really really appreciate what cinder is doing | 23:58 |
| anteaya | as I wish all projects were doing what you are doing | 23:58 |
| anteaya | but the other thing is I can't keep adding things | 23:58 |
| anteaya | as some folks shut down long ago | 23:58 |
| anteaya | and they don't look at what is currently there let alone new things | 23:58 |
| anteaya | I wish it were different | 23:58 |
| anteaya | and I totally support trying to communicate this information | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
