*** dkingshott has quit IRC | 00:04 | |
*** vilobhmm has quit IRC | 00:05 | |
*** vilobhmm has joined #openstack-cinder | 00:05 | |
*** eharney has quit IRC | 00:06 | |
*** annashen has joined #openstack-cinder | 00:07 | |
*** markvoelker has quit IRC | 00:07 | |
*** xyang has quit IRC | 00:08 | |
*** rmesta has quit IRC | 00:09 | |
*** xyang has joined #openstack-cinder | 00:09 | |
*** alexpilotti has quit IRC | 00:10 | |
*** bswartz has joined #openstack-cinder | 00:10 | |
thingee | hemna, xyang: I really don't think we can do brocade in k-3. | 00:13 |
---|---|---|
thingee | kmartin: ^ | 00:14 |
hemna | thingee, do they have a code patch up yet, or just the cinder-specs ? | 00:15 |
thingee | both, but it's 5K. | 00:15 |
xyang | hemna: they have one patch up, wip | 00:15 |
thingee | I want that tested well by folks | 00:15 |
hemna | ugh 5k | 00:15 |
xyang | thingee: there is a 3000 line cinder.conf.sample got submitted:) | 00:15 |
thingee | brocade folks aren't going to be able to tell me anything if it works. | 00:15 |
hemna | ! | 00:15 |
thingee | also here's k-3 currently https://launchpad.net/cinder/+milestone/kilo-3 | 00:16 |
hemna | well besides the fact that the brocade folks are active in here or the ML | 00:16 |
*** jasondotstar has joined #openstack-cinder | 00:16 | |
thingee | jnrao thought my name was Mark for starters | 00:16 |
hemna | goodh 23 BPs for K3 | 00:16 |
hemna | doh | 00:16 |
xyang | thingee: you got a new name?:) | 00:17 |
thingee | I think we're oversubscribed, and k-2 has been an eye opener for me | 00:17 |
thingee | I've had to punt way too much | 00:17 |
*** vilobhmm has quit IRC | 00:17 | |
hemna | yah we are oversubscribed at this point | 00:17 |
hemna | looking for the brocade cinder reviews | 00:18 |
xyang | https://review.openstack.org/#/c/149804/ | 00:18 |
thingee | this is why at the midcycle meet up I was mentioning we need to only look at reviews with bps that are approved...even ones that don't require a spec. Also we need to stop accept bps at some point | 00:18 |
xyang | hemna: he was pinging me today. | 00:18 |
thingee | of course bugs too | 00:18 |
hemna | so the 5k lines patch is 3k lines of cinder.conf.sample | 00:18 |
hemna | which goes away | 00:18 |
hemna | so it's really 2k lines of code to review | 00:19 |
xyang | 800 tests | 00:19 |
xyang | 800 line test | 00:19 |
thingee | it doesn't matter. we still all these other patches from vendors. They were submit way earlier. | 00:20 |
*** vilobhmm has joined #openstack-cinder | 00:20 | |
thingee | as far as I'm concerned those get priority...and we don't have time for the brocade patches at this time | 00:20 |
thingee | submitted* | 00:20 |
*** vilobhmm has quit IRC | 00:20 | |
*** vilobhmm has joined #openstack-cinder | 00:21 | |
hemna | ugh | 00:21 |
hemna | brocade has 2 cinder-specs for the same feature | 00:21 |
hemna | https://review.openstack.org/#/c/140266/ and https://review.openstack.org/#/c/144384/ | 00:22 |
openstackgerrit | Anish Bhatt proposed openstack/cinder-specs: Add open-iscsi transport support to brick https://review.openstack.org/152740 | 00:22 |
anish | hemna: done ^ | 00:22 |
hemna | I can't -2 cinder-specs or I would right now. | 00:23 |
anish | I hope you're talking about brocade :) | 00:24 |
xyang | hemna: thingee fixed it:) | 00:25 |
hemna | anish, I am | 00:25 |
*** oomichi has joined #openstack-cinder | 00:26 | |
*** jnrao has quit IRC | 00:26 | |
anish | I guess they thought a new approach needed a new spec ? | 00:26 |
hemna | If they were actively involved here working with us on getting their stuff in, then I'd say give them an exception. | 00:26 |
hemna | I'd like to see their stuff land, but I haven't heard from them in here at all. | 00:27 |
hemna | anish, so I'd mark yours as WIP | 00:27 |
openstackgerrit | Anish Bhatt proposed openstack/cinder-specs: [WIP] Add open-iscsi transport support to brick https://review.openstack.org/152740 | 00:28 |
anish | done and done | 00:29 |
anish | it'd be nice if the WIP tag automagically set workflow to -1 | 00:29 |
*** jcru has joined #openstack-cinder | 00:32 | |
*** primechu_ has quit IRC | 00:33 | |
openstackgerrit | abhiram moturi proposed openstack/cinder: Enhance unit tests for zfssa drivers https://review.openstack.org/152676 | 00:33 |
*** jcru has quit IRC | 00:34 | |
*** theanalyst has quit IRC | 00:36 | |
*** theanalyst has joined #openstack-cinder | 00:40 | |
*** jasondotstar has quit IRC | 00:41 | |
tsekiyama | hemna: ok, i have added 'enforce_multipath' arg to specify it should raise or failover to single path when multipathd is not running. https://review.openstack.org/#/c/134681/ | 00:42 |
tsekiyama | hemna: The default in cinder is failover to single path. | 00:42 |
hemna | tsekiyama, ok cool | 00:42 |
*** markvoelker has joined #openstack-cinder | 00:52 | |
*** annashen has quit IRC | 00:53 | |
*** david-lyle is now known as david-lyle_afk | 00:57 | |
thingee | jungleboyj: I hope you're not being serious about this https://review.openstack.org/#/c/144627/ ... this is coming off as a backdoor of what we all already agreed on. | 00:59 |
*** markvoelker has quit IRC | 01:00 | |
*** primechuck has joined #openstack-cinder | 01:02 | |
*** dannywilson has quit IRC | 01:02 | |
*** nellysmitt has joined #openstack-cinder | 01:11 | |
*** nellysmitt has quit IRC | 01:16 | |
*** tsekiyam_ has joined #openstack-cinder | 01:16 | |
*** Mandell has quit IRC | 01:16 | |
*** markvoelker has joined #openstack-cinder | 01:19 | |
*** mtanino has quit IRC | 01:19 | |
*** tsekiyama has quit IRC | 01:20 | |
*** tsekiyam_ has quit IRC | 01:21 | |
*** jasondotstar has joined #openstack-cinder | 01:24 | |
hemna | bbl | 01:28 |
*** hemna is now known as hemnafk | 01:28 | |
*** _cjones_ has quit IRC | 01:29 | |
*** MentalRay has joined #openstack-cinder | 01:29 | |
*** _cjones_ has joined #openstack-cinder | 01:29 | |
*** jamielennox is now known as jamielennox|away | 01:33 | |
*** _cjones_ has quit IRC | 01:34 | |
*** Lee1092 has joined #openstack-cinder | 01:37 | |
*** mriedem has quit IRC | 01:38 | |
openstackgerrit | Vilobh Meshram proposed openstack/cinder: Introducing micro_states for create workflow https://review.openstack.org/124205 | 01:44 |
*** theanalyst has quit IRC | 01:48 | |
*** jasondotstar has quit IRC | 01:50 | |
*** leeantho has quit IRC | 01:50 | |
*** theanalyst has joined #openstack-cinder | 01:51 | |
*** scottda_ has joined #openstack-cinder | 01:52 | |
*** scottda_ has quit IRC | 02:09 | |
*** kaisers has joined #openstack-cinder | 02:11 | |
*** patrickeast has quit IRC | 02:11 | |
*** rwsu is now known as rwsu-afk | 02:11 | |
*** mberlin1 has joined #openstack-cinder | 02:11 | |
*** kaisers1 has quit IRC | 02:12 | |
*** mberlin has quit IRC | 02:13 | |
openstackgerrit | wanghao proposed openstack/cinder: Implement function of import/export snapshots https://review.openstack.org/144590 | 02:19 |
*** haomaiwang has joined #openstack-cinder | 02:20 | |
*** tsekiyama has joined #openstack-cinder | 02:27 | |
*** vilobhmm has quit IRC | 02:29 | |
*** tsekiyama has quit IRC | 02:31 | |
*** diegows has quit IRC | 02:33 | |
*** Yogi1 has joined #openstack-cinder | 02:39 | |
*** rushiagr_away is now known as rushiagr | 02:48 | |
*** theanalyst has quit IRC | 02:53 | |
*** Yogi1 has quit IRC | 02:56 | |
*** theanalyst has joined #openstack-cinder | 02:56 | |
*** tellesnobrega_ has joined #openstack-cinder | 03:03 | |
*** jcru has joined #openstack-cinder | 03:06 | |
*** nellysmitt has joined #openstack-cinder | 03:12 | |
jungleboyj | thingee: Not trying to backdoor. Just surprised we are still allowing huge updates to drivers but won't take something like that. I am going to have them split it up to make it as small as possible and carry the protocol internally then. | 03:13 |
*** hemna_ has joined #openstack-cinder | 03:14 | |
*** nellysmitt has quit IRC | 03:16 | |
rhe00_ | thingee: would it make sense to talk about Cinder presentations in Vancouver at the meeting tomorrow? (submission deadline is February 9th). I remember Kurt bringing it up the idea of cross company presentations at the meetup last week but I don't think we came to a conclusion. | 03:17 |
*** hemna_ is now known as hemna | 03:17 | |
*** rushiagr is now known as rushiagr_away | 03:19 | |
thingee | jungleboyj: I agree IMO driver array updates are questionable | 03:23 |
thingee | xyang: ^ | 03:23 |
thingee | the whole reason to stop drivers after k-1 was to allow time for other things. | 03:24 |
xyang | thingee, jungleboyj: so those blueprints were approved a while back. I am sorry that we gave everyone a headache:( | 03:24 |
xyang | thingee, jungleboyj: Cinder has been allowing driver update with multiple changes in it. we planned based on that assumption | 03:25 |
xyang | thingee, jungleboyj: also our internal review process takes 2 months | 03:25 |
xyang | thingee, jundleboyj: so we are working on one big patch while waiting for it to be approved. | 03:26 |
xyang | thingee, jungleboyj: we'll definitely have to change this in L. no more big driver update like this | 03:26 |
thingee | really I think we just have to learn as we go. New protocols in drivers is a new driver imo. it's exactly why people introduce a new volume driver class..because of another protocol | 03:26 |
thingee | xyang: +1 | 03:27 |
thingee | I think we learned from this, and we just have to be explicit about what k-1 means. | 03:27 |
thingee | even more explicit* | 03:27 |
*** harlowja is now known as harlowja_away | 03:28 | |
xyang | thingee: agreed. | 03:29 |
*** annashen has joined #openstack-cinder | 03:29 | |
thingee | but I'm really curious to get thoughts in tomorrow's meeting about what to do with all these proposed patches that are new features in drivers. we can't do them all in k-3. We need to target what was submitted earlier and ready and just do those for now | 03:31 |
thingee | core stuff of course gets priority in this milestone. | 03:31 |
xyang | thingee: do we have to cut things off so early? I mean we have allowed driver features in the 3rd milestone in the past. anyway, we can discuss tomorrow | 03:33 |
thingee | there's 22 things targeted. that doesn't cover everything in status: open project: openstack/cinder | 03:33 |
thingee | https://launchpad.net/cinder/+milestone/kilo-3 | 03:34 |
thingee | xyang: we only did maybe 14 things for k-2 | 03:34 |
hemna | howdy | 03:34 |
thingee | xyang: if I accept anymore things in, we won't be productive and nothing will get merged because people will just be focused on separate things | 03:35 |
xyang | thingee: we merged about 28 blueprints in juno, mostly in j-3 | 03:35 |
hemna | thingee, agreed about the adding of a protocol. It's a new driver. In particular the IBM driver I -1'd was an FC driver that was being patched to support iSCSI | 03:36 |
hemna | and the FC driver didn't extend from the base FibreChannelDriver base class | 03:36 |
hemna | also a bad deal | 03:36 |
xyang | thingee: I remember 0 bp merged in J-1 | 03:37 |
thingee | lol | 03:37 |
hemna | I agree that we need to have a cutoff. | 03:37 |
xyang | thingee: so we are in good shape:) | 03:37 |
hemna | thingee, I thought you mentioned in the ML post that the cutoff was Feb. 15th for specs/blueprints | 03:37 |
thingee | cut off is the 15th, I'm just saying we should be saying at some point we have enough and we can't accept anymore. | 03:38 |
hemna | thingee, +1 | 03:38 |
thingee | and take a look at what's proposed, what's a priority at this time | 03:38 |
anish | thingee: there are expected outputs for fake_excecute, hence the redefinition. | 03:38 |
anish | err | 03:38 |
hemna | Nova's cutoff is tomorrow AFAIK | 03:38 |
anish | I meant there are multiple expected outputs | 03:38 |
thingee | you can't exactly figure that out with most of them being driver specific things | 03:38 |
hemna | anything after the 5th needs to have a core sponsor | 03:38 |
anish | thingee: okay if I use mock in place instead ? | 03:39 |
thingee | anish: that's what the return_value is for | 03:39 |
thingee | anish: see my examples | 03:39 |
anish | just to make sure I'm not misunderstanding you, I'm talking about keeping fake_execute out of setUp(self) and replacing stubs.Set with mock | 03:41 |
anish | or are you saying I can have multiple (and different) return values even if it is setup in setUp(self) ? | 03:42 |
jungleboyj | thingee: I am going to ask the Edwin to shrink the patch down to just the iSCSI change and submit patches to fix the other stuff separately. Not going to push that issue. | 03:45 |
*** rushiagr_away is now known as rushiagr | 03:45 | |
hemna | jungleboyj, are you talking about https://review.openstack.org/#/c/144627/ ? | 03:48 |
jungleboyj | hemna: Yes. | 03:49 |
hemna | so that patch has other issues that I commented on | 03:49 |
hemna | I think it's a really bad idea to mix FC and iSCSI in the same driver | 03:50 |
hemna | it's what caused me to have to create a decorator for FCZM zoning | 03:50 |
hemna | which has to get added to all FC drivers to do zoning. | 03:50 |
hemna | besides the fact that the existing code is an FC driver that doesn't extend from FibreChannelDriver | 03:51 |
xyang | hemna: I remember that. you initially was going to fix it differently but have to redo it completely because of the storwize driver | 03:51 |
*** Mandell has joined #openstack-cinder | 03:51 | |
hemna | xyang, yup | 03:52 |
jungleboyj | hemna: Ok, the problem is that they wrote that based on the Storwize driver so they inherited that badness. | 03:52 |
hemna | ouch | 03:52 |
hemna | yah, so lets not make it worse IMO | 03:52 |
thingee | anish: hang on, I'll give you another example of what I think you're trying to do | 03:53 |
jungleboyj | hemna: Yes dear. ;-) | 03:53 |
hemna | hehe | 03:53 |
*** MentalRay has quit IRC | 03:54 | |
jungleboyj | hemna: I will shoot them a note and work with them to make that change. | 03:55 |
hemna | ok sounds good | 03:56 |
*** mitz_ has quit IRC | 03:56 | |
hemna | I think jdg will eventually create an FC target object. We'll need to look into refactoring FC drivers at that point | 03:56 |
*** mitz has joined #openstack-cinder | 03:56 | |
hemna | and hopefully centralize the FCZM usage at that point to 1 place. | 03:56 |
thingee | anish: alright another comment posted https://review.openstack.org/#/c/147758/9/cinder/tests/targets/test_cxt_driver.py | 03:57 |
jungleboyj | hemna: On a totally unrelated note ... I was amazed today that my dad was able to load the driver for an HP printer and get it working by himself today. That was nearly a miracle! | 03:57 |
thingee | that would allow you to change the value of a method in each test case. | 03:57 |
thingee | with very little code. | 03:57 |
hemna | jungleboyj, nice! just don't tell him about the ink :P | 03:57 |
openstackgerrit | Jay Bryant proposed openstack/cinder-specs: Fix the volume replication spec https://review.openstack.org/152735 | 03:57 |
jungleboyj | hemna: I bought him the first few rounds. NOt as bad as some of ink. | 03:58 |
anish | thingee: aah, much clearer. I did not know you could overload the return value like that | 03:58 |
thingee | anish: whoops that last line in setUp() should be... | 04:00 |
thingee | self.addCleanup(self.execute_patcher.stop()) | 04:00 |
lifeless | thingee: .stop) | 04:01 |
lifeless | thingee: not .stop()) | 04:01 |
lifeless | thingee: you want to call stop later, not immediately | 04:01 |
thingee | lifeless: thanks :) | 04:01 |
anish | lifeless: thingee thanks ! | 04:02 |
* anish secretly misses C | 04:02 | |
thingee | seriously? | 04:02 |
thingee | after college, I was so happy to get back into scripting | 04:02 |
anish | once you learn the 40 or so functions, there is nothing new :P | 04:03 |
anish | not even that many I think | 04:03 |
anish | I do kernel dev, so this is a nice break for me | 04:03 |
thingee | anish: nice, I just started in that space last year | 04:04 |
anish | are you enjoying the complete lack of everything python is known for | 04:05 |
thingee | when I had to deal with xml in c, I was so pissed | 04:06 |
hemna | thingee, you had me at "when I had to deal with xml..." | 04:06 |
thingee | actually that was in libvirt...yeah kernel is fine actually. Took more time to wrap my head around things. | 04:07 |
anish | ^ | 04:07 |
anish | I'm having the same problem with stuff like mock | 04:07 |
anish | it takes a bit of reading, but then it's easey | 04:08 |
thingee | I have some good mentors though. | 04:08 |
*** Yogi1 has joined #openstack-cinder | 04:08 | |
anish | though, nothing is as terrible as parsing iscsi text response packets in C | 04:08 |
anish | so there's that | 04:08 |
thingee | 100000000+ | 04:09 |
lifeless | yacc FTW | 04:09 |
anish | in control utilities, yes | 04:12 |
anish | sometimes you do need some parsing in the kernel | 04:12 |
*** annashen has quit IRC | 04:12 | |
* anish is pushing to change control utilities to python | 04:12 | |
thingee | anish: so after those fix ups, I'm good with the target driver patch | 04:13 |
anish | ok, i'll get it done tonight | 04:13 |
anish | everything but the mock tests is done | 04:13 |
thingee | anish: cool, I'll +2 it then and hopefully jdg will have a chance to look at it soon | 04:14 |
anish | sounds | 04:14 |
anish | good | 04:14 |
anish | I wanted to check if "except exception.CinderException as TargetException" is okay | 04:14 |
*** annashen has joined #openstack-cinder | 04:15 | |
thingee | anish: should be...everything else that is explicitly raised in your code inerherits from it | 04:15 |
thingee | at least in that code path | 04:16 |
anish | aite ! | 04:16 |
openstackgerrit | Zhiyuan Cai proposed openstack/cinder-specs: Change-since filter for resource retrieving https://review.openstack.org/152770 | 04:16 |
anish | aargh. I keep looking at scst code with is 145778, while mine is 147758 | 04:17 |
anish | odd coincidence | 04:17 |
thingee | heh | 04:17 |
thingee | oh scst...so truth is my company and them were racing to be the target subsystem in the kernel | 04:18 |
thingee | I think that was around 2012 | 04:18 |
anish | thingee: mix of decorator & context manager is okay ? (in different functions) | 04:18 |
thingee | anish: so truth is with that, I could send you a youtube of what openstack prefers. I think ameade has an opinion on it, but I don't. | 04:19 |
anish | thingee: are you working for lio or something ? | 04:19 |
thingee | anish: :) | 04:19 |
thingee | yeah my company was formerly rising tide systems | 04:19 |
thingee | now datera | 04:19 |
anish | aah, we're trying to hook into lio here as well | 04:20 |
anish | you know I can see your office from my window | 04:21 |
thingee | so is scst | 04:21 |
thingee | I thought you were in NY? | 04:21 |
anish | nope, right here in mountain view | 04:22 |
thingee | ha alright | 04:22 |
thingee | I hate that building by the way | 04:22 |
thingee | ugly thing next to this trader joes and walmart parking lot | 04:22 |
anish | I'm surprised they aren't tearing that down | 04:23 |
anish | from what I hear, walmart refused to move | 04:23 |
anish | any opinions on atlantis computing ? | 04:24 |
*** Yogi1 has quit IRC | 04:24 | |
anish | thingee: out of curiosity, do send me that video | 04:25 |
thingee | haha | 04:25 |
thingee | https://wiki.openstack.org/wiki/BootstrappingHour/Mock_Best_Practices | 04:25 |
anish | I'm assuming it's the video by sdague | 04:26 |
thingee | yup | 04:26 |
*** julim has quit IRC | 04:27 | |
anish | the first 2 min of this are "see who's reviewed your code so far" for me | 04:27 |
*** _cjones_ has joined #openstack-cinder | 04:30 | |
thingee | anish: atlantis computing...not sure sorry. | 04:30 |
thingee | series d though in 2013 | 04:30 |
anish | they're in your building | 04:31 |
thingee | I barely come in :) | 04:31 |
anish | aah | 04:31 |
*** annashen has quit IRC | 04:31 | |
thingee | they had me relocate from LA, but I'm mostly traveling anyways. | 04:32 |
*** _cjones_ has quit IRC | 04:32 | |
thingee | I'll probably move back to LA after my lease though. I haven't really been able to get comfortable here =/ | 04:32 |
thingee | in sunnyvale though | 04:32 |
*** _cjones_ has joined #openstack-cinder | 04:33 | |
thingee | where my backyard is tech wasteland of a big abandon hp building, symatec, synopsys, linkedin, etc. | 04:33 |
thingee | oh and murphy st...where six people thought they were unique in opening an irish pub there. | 04:34 |
anish | that is not an irish pub | 04:34 |
anish | they have a neon dance floor | 04:34 |
thingee | heh | 04:34 |
hemna | ok dinner time..... | 04:35 |
anish | their sister establishment in MV has a neon floor | 04:35 |
anish | thingee: though dish dash, can't beat taht | 04:35 |
*** _cjones_ has quit IRC | 04:37 | |
thingee | everyone talks about dish dash. still haven't been there | 04:38 |
anish | you really should | 04:38 |
anish | it is the one good place in sunnyvale | 04:38 |
rushiagr | jordanP: I recall netapp has 4 drivers, and they test all of them at once | 04:39 |
rushiagr | woops, that was wayy back up in my scrollback :0 | 04:40 |
openstackgerrit | yatin proposed openstack/cinder: Added more options while uploading volume to glance https://review.openstack.org/152475 | 04:41 |
*** BharatK has joined #openstack-cinder | 04:45 | |
thingee | a 2PB ceph cluster http://www.dreamhoststatus.com/2015/01/28/scheduled-dreamobjects-maintenance-february-3rd-10am-pst-48-hrs/ | 04:50 |
thingee | apparently expansion causes down time | 04:51 |
openstackgerrit | Chao Zheng Li proposed openstack/cinder: Add support and Unittest of consistency group for XIV/DS8K driver proxy https://review.openstack.org/150281 | 04:55 |
*** Ilja has joined #openstack-cinder | 04:57 | |
*** changbl has joined #openstack-cinder | 04:57 | |
*** avishay has joined #openstack-cinder | 04:58 | |
rushiagr | thingee: why didn't they just add two nodes at a time? Or Ceph will have perf degradation then too? | 04:59 |
thingee | beats me, not my problem anymore | 05:00 |
thingee | :) | 05:00 |
rushiagr | thingee: heh | 05:00 |
thingee | bbl | 05:01 |
rushiagr | sure | 05:01 |
openstackgerrit | Chao Zheng Li proposed openstack/cinder: Add support and Unittest of consistency group for XIV/DS8K cinder driver proxy layer https://review.openstack.org/150281 | 05:04 |
*** oomichi has quit IRC | 05:04 | |
*** mwichmann has quit IRC | 05:08 | |
*** annashen has joined #openstack-cinder | 05:09 | |
*** david-lyle_afk has quit IRC | 05:11 | |
*** david-lyle_afk has joined #openstack-cinder | 05:11 | |
avishay | we should have a contest to see who can replace the xiv/ds8k driver with the fewest lines of code | 05:12 |
*** nellysmitt has joined #openstack-cinder | 05:13 | |
openstackgerrit | Chao Zheng Li proposed openstack/cinder: Add support and Unittest of consistency group for XIV/DS8K cinder driver proxy https://review.openstack.org/150281 | 05:13 |
rushiagr | avishay: I'll also nominate vmware's and emc's drivers | 05:14 |
avishay | rushiagr: look at the driver :) | 05:15 |
rushiagr | avishay: oh my | 05:16 |
rushiagr | avishay: this is in a different league altogether :) | 05:17 |
avishay | rushiagr: :) | 05:17 |
*** nellysmitt has quit IRC | 05:17 | |
*** harlowja_away has quit IRC | 05:19 | |
*** test2222 has joined #openstack-cinder | 05:23 | |
*** test2222 has quit IRC | 05:27 | |
thingee | avishay: can you review https://review.openstack.org/#/c/146766/ ? | 05:28 |
avishay | thingee: i did... | 05:28 |
thingee | ah right! | 05:28 |
avishay | thingee: :) | 05:29 |
avishay | thingee: you probably confused me with the other avishay :) | 05:30 |
*** sgotliv has joined #openstack-cinder | 05:30 | |
openstackgerrit | Chao Zheng Li proposed openstack/cinder: Add support and Unittest of consistency group for XIV/DS8K cinder driver proxy layer https://review.openstack.org/150281 | 05:30 |
*** pradipta_away is now known as pradipta | 05:35 | |
*** madhuri_ has joined #openstack-cinder | 05:38 | |
*** BharatK has quit IRC | 05:39 | |
*** mtreinish has quit IRC | 05:40 | |
madhuri_ | Hi, I am getting this error while creating volumes. http://paste.openstack.org/show/166588/ | 05:40 |
madhuri_ | I can't figure out the issue. Can anyone please help? | 05:41 |
thingee | madhuri_: that means the cinder-volume service is not able to talk to the backend you're trying to create volumes onto | 05:41 |
thingee | madhuri_: can you restart the cinder-volume service and paste the output when it first starts up? | 05:42 |
madhuri_ | Is there any issue while the volume-group? | 05:42 |
thingee | madhuri_: dunno, need the start up information to know what's going on | 05:43 |
madhuri_ | thingee: Here http://paste.openstack.org/show/166589/ | 05:44 |
*** lpetrut has joined #openstack-cinder | 05:48 | |
thingee | madhuri_: according to the error, something is bad with the rootwrap config file | 05:48 |
madhuri_ | thingee: Yes, after correcting it. It is working fine | 05:49 |
*** Mandell has quit IRC | 05:49 | |
madhuri_ | But one question here. | 05:49 |
thingee | madhuri_: great! | 05:49 |
thingee | second question you mean :) | 05:49 |
madhuri_ | I followed the openstack installation guide. But I din't find any rootwrap configuration. | 05:50 |
madhuri_ | thingee: Yes :) | 05:50 |
madhuri_ | So how this file was wrong. | 05:50 |
openstackgerrit | Abel Lopez proposed openstack/cinder: Purge deleted rows https://review.openstack.org/146766 | 05:50 |
thingee | diving into the code, it means the code that would parse the config wasn't able to. I believe that /etc/cinder/rootwrap.conf comes from installing cinder with whatever package you used by whatever distro | 05:51 |
thingee | if that file never got created, I dunno...bad package maybe? =/ | 05:52 |
madhuri_ | thingee: Ok. Thank you :) | 05:52 |
thingee | madhuri_: you're welcome | 05:52 |
openstackgerrit | Tina Tang proposed openstack/cinder: Pool-aware scheduler support in EMC VNX Cinder driver https://review.openstack.org/147408 | 05:53 |
*** madhuri_ has quit IRC | 05:53 | |
*** nshaikh has joined #openstack-cinder | 05:55 | |
*** hemna has quit IRC | 05:55 | |
*** BharatK has joined #openstack-cinder | 05:55 | |
*** mtreinish has joined #openstack-cinder | 05:56 | |
*** coolsvap_ is now known as coolsvap | 05:57 | |
*** Mandell has joined #openstack-cinder | 05:57 | |
*** jcru has quit IRC | 05:58 | |
*** annashen has quit IRC | 05:59 | |
*** annashen has joined #openstack-cinder | 06:01 | |
*** xyang has quit IRC | 06:02 | |
openstackgerrit | Chao Zheng Li proposed openstack/cinder: Add support and Unittest of consistency group for XIV/DS8K cinder driver proxy https://review.openstack.org/150281 | 06:02 |
*** abehl has joined #openstack-cinder | 06:06 | |
*** markvoelker has quit IRC | 06:08 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/152401 | 06:09 |
*** nlevinki has joined #openstack-cinder | 06:13 | |
*** nlevinki has quit IRC | 06:14 | |
*** nlevinki has joined #openstack-cinder | 06:15 | |
*** annashen has quit IRC | 06:17 | |
*** sgotliv has quit IRC | 06:22 | |
*** eharney has joined #openstack-cinder | 06:29 | |
*** tellesnobrega_ has quit IRC | 06:32 | |
*** tellesnobrega__ has joined #openstack-cinder | 06:32 | |
*** primechuck has quit IRC | 06:38 | |
*** markvoelker has joined #openstack-cinder | 06:39 | |
*** sgotliv has joined #openstack-cinder | 06:40 | |
*** markvoelker has quit IRC | 06:43 | |
*** nshaikh has quit IRC | 06:50 | |
*** nshaikh has joined #openstack-cinder | 06:51 | |
*** sgotliv has quit IRC | 06:52 | |
*** Mandell has quit IRC | 07:01 | |
*** Mandell has joined #openstack-cinder | 07:04 | |
*** chlong has joined #openstack-cinder | 07:07 | |
*** TobiasE has joined #openstack-cinder | 07:07 | |
*** BharatK has quit IRC | 07:08 | |
*** BharatK has joined #openstack-cinder | 07:09 | |
*** harlowja_at_home has joined #openstack-cinder | 07:10 | |
*** nellysmitt has joined #openstack-cinder | 07:13 | |
*** nellysmitt has quit IRC | 07:18 | |
*** chlong has quit IRC | 07:28 | |
*** eharney has quit IRC | 07:31 | |
*** nkrinner has joined #openstack-cinder | 07:32 | |
openstackgerrit | Chao Zheng Li proposed openstack/cinder: Add support and Unittest of consistency group for XIV/DS8K cinder driver proxy Follow the pep8 code style to refactor code https://review.openstack.org/150281 | 07:35 |
*** Miouge has joined #openstack-cinder | 07:35 | |
*** markvoelker has joined #openstack-cinder | 07:39 | |
*** lpetrut has quit IRC | 07:42 | |
*** nellysmitt has joined #openstack-cinder | 07:42 | |
*** markvoelker has quit IRC | 07:44 | |
*** ronenkat has joined #openstack-cinder | 07:47 | |
*** ronenkat_ has joined #openstack-cinder | 07:47 | |
*** ronenkat has quit IRC | 07:51 | |
*** e0ne has joined #openstack-cinder | 07:52 | |
*** dulek has joined #openstack-cinder | 07:57 | |
*** harlowja_at_home has quit IRC | 08:02 | |
avishay | e0ne: hey! want to put this one through? https://review.openstack.org/#/c/146766/ | 08:02 |
*** ronis has joined #openstack-cinder | 08:02 | |
e0ne | avishay: hi! i'll take a look on it | 08:03 |
*** Miouge_ has joined #openstack-cinder | 08:09 | |
*** Miouge has quit IRC | 08:11 | |
*** Miouge_ is now known as Miouge | 08:11 | |
e0ne | avishay: unfortunately, i'll -1 it. i'm totally disagree with raising exception in sqlalchemy/api in this way | 08:11 |
*** lpetrut has joined #openstack-cinder | 08:13 | |
*** tshefi has joined #openstack-cinder | 08:14 | |
avishay | e0ne: yes i agree with you | 08:22 |
e0ne | thanks. i hope, we'll get it in K-2 | 08:25 |
avishay | e0ne: he is responsive, should get merged soon anyway | 08:26 |
e0ne | avishay: thanks for review https://review.openstack.org/#/c/144739/. i'll fix it | 08:27 |
avishay | e0ne: i'm sorry but i don't agree that this should be implemented at all | 08:27 |
avishay | e0ne: i think 99% of the time the user cares only about name, status, and size | 08:28 |
*** eharney has joined #openstack-cinder | 08:28 | |
avishay | e0ne: for 1% that he wants to see snapshots, he can do another call instead of slowing down all other operations | 08:28 |
*** jistr has joined #openstack-cinder | 08:29 | |
e0ne | avishay: not sure about statistics. our custemers ask it to make aotomation more clear | 08:29 |
*** sgotliv has joined #openstack-cinder | 08:29 | |
e0ne | avishay: i'm going to do it only for API | 08:29 |
e0ne | for 'volume get' action | 08:30 |
avishay | e0ne: i still disagree, but if others over-ride my decision, that's ok | 08:30 |
e0ne | avishay: ok. lets see how community will vote on it. | 08:33 |
e0ne | avishay: anyway, thanks for review | 08:33 |
avishay | e0ne: sure! | 08:36 |
avishay | e0ne: it can be a flag, but not sure we want flags like that on volume list/get | 08:37 |
* e0ne bb in 2hours | 08:38 | |
*** markvoelker has joined #openstack-cinder | 08:40 | |
*** Mandell has quit IRC | 08:40 | |
e0ne | avishay: something like 'cinder show --detailed vol' could be an option | 08:41 |
avishay | e0ne: agreed | 08:41 |
*** markvoelker has quit IRC | 08:45 | |
openstackgerrit | Jeegn Chen proposed openstack/cinder: EMC VNX Cinder Driver Update https://review.openstack.org/141629 | 08:49 |
*** cinderella has joined #openstack-cinder | 08:50 | |
cinderella | good morning to everyone | 08:50 |
cinderella | am having odd problems with cinder, can anybody help me? | 08:50 |
cinderella | I'm getting a "failed to copy image to volume: permission denied" | 08:50 |
cinderella | whenever I try to create a volume with an image attached | 08:51 |
*** jordanP has joined #openstack-cinder | 08:53 | |
*** PaulCuzner has joined #openstack-cinder | 08:53 | |
*** jpich has joined #openstack-cinder | 08:55 | |
cinderella | seems like I solved, I have a new problem now | 09:01 |
cinderella | anybody? | 09:02 |
*** PaulCuzner has quit IRC | 09:03 | |
*** karimb has joined #openstack-cinder | 09:04 | |
openstackgerrit | git-harry proposed openstack/cinder: Remove use of len for testing truth https://review.openstack.org/150482 | 09:09 |
*** PaulCuzner has joined #openstack-cinder | 09:09 | |
*** takedakn has joined #openstack-cinder | 09:12 | |
*** e0ne has quit IRC | 09:12 | |
openstackgerrit | Chao Zheng Li proposed openstack/cinder: Add consistency group support for XIV/DS8K cinder driver proxy https://review.openstack.org/150281 | 09:15 |
*** jordanP has quit IRC | 09:18 | |
cinderella | guys, now cinder list shows nothing, while in horizon I have 2 volumes created. What is happening there? | 09:19 |
rushiagr | cinderella: hi | 09:20 |
rushiagr | cinderella: what is 'creating a volume with image attached'? | 09:22 |
*** Miouge has quit IRC | 09:25 | |
*** afazekas has joined #openstack-cinder | 09:25 | |
*** jordanP has joined #openstack-cinder | 09:25 | |
*** jordanP has quit IRC | 09:25 | |
*** mdbooth has quit IRC | 09:28 | |
*** takedakn has quit IRC | 09:29 | |
openstackgerrit | wanghao proposed openstack/cinder: query volume detail support volume_glance_metadata https://review.openstack.org/147738 | 09:30 |
*** Miouge has joined #openstack-cinder | 09:32 | |
*** PaulCuzner has left #openstack-cinder | 09:32 | |
*** PaulCuzner has joined #openstack-cinder | 09:33 | |
*** mdbooth has joined #openstack-cinder | 09:33 | |
openstackgerrit | lvdongbing proposed openstack/python-cinderclient: Support cinder list filter by tenant https://review.openstack.org/107886 | 09:34 |
eharney | rushiagr: please see comments/question on https://review.openstack.org/#/c/147185/ | 09:37 |
rushiagr | eharney: looking | 09:38 |
*** boris-42 has joined #openstack-cinder | 09:39 | |
*** markvoelker has joined #openstack-cinder | 09:41 | |
rushiagr | eharney: the two defs were unbound before (not belonging to any class). Can we not move them directly to the remotefs.py file, instead of making them class methods in remotefs.py and then making them unbound again in glusterfs.py? | 09:42 |
rushiagr | i.e., just move those two unbound functions from glusterfs.py to make them unbound methods in remotefs.py | 09:43 |
rushiagr | s/methods/functions/ | 09:44 |
eharney | rushiagr: it would work, but i'm not sure why it would be better to have them not belonging to a class, when they are decorators only used for methods related to that class | 09:44 |
rushiagr | eharney: then I don't see a point in them being unbound again in remotefs.py file.. | 09:45 |
eharney | rushiagr: they're in the RemoteFSSnapDriver class? | 09:46 |
eharney | so they aren't unbound | 09:46 |
rushiagr | eharney: oh, sorry, glusterfs.py | 09:46 |
eharney | rushiagr: the point there is so you don't have to repeat a ridiculously long decorator name on each function | 09:47 |
eharney | we do this fairly often i think... | 09:47 |
*** markvoelker has quit IRC | 09:47 | |
*** avishay has quit IRC | 09:50 | |
cinderella | rushiagr: brb | 09:50 |
rushiagr | eharney: hmm | 09:50 |
rushiagr | Can we not use those methods directly in glusterfs class as they're already part of the parent class? I'm not sure how staticmethods work in inherited class | 09:51 |
rushiagr | let me find that out | 09:52 |
*** alecv has joined #openstack-cinder | 09:52 | |
erlon | eharney: ping | 09:52 |
eharney | rushiagr: i was just wondering the same thing actually, not sure | 09:52 |
eharney | erlon: hi | 09:52 |
erlon | eharney: good morning | 09:52 |
eharney | good morning | 09:53 |
erlon | eharney: forget to send the comment, weird, it left the review -1 but not the inline comment | 09:53 |
erlon | eharney: fixed that | 09:53 |
*** enterprisedc has quit IRC | 09:54 | |
eharney | erlon: ok, looking now, thanks | 09:54 |
openstackgerrit | Anish Bhatt proposed openstack/cinder: Add support for chiscsi iscsi helper https://review.openstack.org/147758 | 09:55 |
*** e0ne has joined #openstack-cinder | 09:57 | |
*** avishay has joined #openstack-cinder | 09:58 | |
eharney | erlon: replied there for posterity... but basically, the system from the spec is implemented there, it just uses different (pre-existing) names for the options rather than the ones shown in the example in the spec | 09:58 |
*** jordanP has joined #openstack-cinder | 09:58 | |
eharney | just to avoid introducing more options than needed | 09:58 |
rushiagr | eharney: A staticmethod can simply be accessed in inherited class using a self.<staticmethodname>. I'm just not sure how the decorator syntax will be. Maybe @self.<staticmethodasdecorator>? | 10:00 |
openstackgerrit | Tina Tang proposed openstack/cinder: Pool-aware scheduler support in EMC VNX Cinder driver https://review.openstack.org/147408 | 10:00 |
eharney | rushiagr: yeah i think i'll try that... seems like a nice way to go | 10:00 |
rushiagr | or maybe that 'self' is silly here and remove | 10:00 |
rushiagr | eharney: okay | 10:00 |
eharney | well | 10:00 |
eharney | i think it'd be some super() thing since self is for instances and it's a staticmethod | 10:00 |
eharney | i dunno, i'll go learn python i guess :) | 10:01 |
rushiagr | heh | 10:02 |
eharney | self doesn't work for decorators because there is no "self" var... | 10:02 |
erlon | eharney: hmm, Im not familiar with the option parsing, so, when you access a variable like 'self.configuration.nas_ip', cinder will get it from a section? | 10:02 |
eharney | erlon: no, the sections are just sections from how you configure multi-backend | 10:02 |
rushiagr | eharney: I just did this: http://paste.openstack.org/show/166691/ | 10:03 |
eharney | erlon: if not using multi-backend they are just in default | 10:03 |
rushiagr | eharney: yeah, how this will work with decorators is unknown to me | 10:03 |
eharney | rushiagr: you have to bind it to a class namespace instead of an instance (which is what i was doing in the patch) | 10:03 |
*** e0ne is now known as e0ne_ | 10:04 | |
erlon | eharney: there's no section for each share, like in the specs [nfs1] nas_ip = x.x.x.x \n nas_share_path = /yyy | 10:04 |
erlon | eharney: the way described in the spec gives that impression | 10:05 |
eharney | erlon: the sections are just whatever you define in enabled_backends | 10:05 |
eharney | erlon: it will work that way | 10:05 |
eharney | erlon: when using this type of config: https://wiki.openstack.org/wiki/Cinder-multi-backend | 10:05 |
erlon | eharney: hmm, ok | 10:05 |
erlon | got it | 10:05 |
erlon | eharney: [nfs1] is a backend name | 10:05 |
rushiagr | eharney: what about making them classmethods? | 10:05 |
eharney | rushiagr: i think that would just result in the same thing in this case, since they don't need class information (which is why they are static) | 10:07 |
rushiagr | eharney: hmm..that's right | 10:08 |
openstackgerrit | Anish Bhatt proposed openstack/cinder: Add support for chiscsi iscsi helper https://review.openstack.org/147758 | 10:08 |
eharney | rushiagr: so i dunno, i'm not really seeing a better way to do it | 10:11 |
*** e0ne_ is now known as e0ne | 10:12 | |
openstackgerrit | Ph. Marek proposed openstack/cinder: DRBD: Use correct function object after DBus disconnect. https://review.openstack.org/150740 | 10:16 |
rushiagr | eharney: so other remotefs drivers won't be able to use the two staticmethods you created | 10:19 |
eharney | rushiagr: sure they will...why not? | 10:20 |
eharney | rushiagr: the NFS snaps patch which depends on this one does... | 10:20 |
rushiagr | eharney: I meant the remotefssnapdriver | 10:21 |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Group test files in the right package in cinder.tests https://review.openstack.org/151232 | 10:22 |
eharney | rushiagr: remotefssnapdriver is an interface that actual drivers inherit from... | 10:22 |
cinderella | rushiagr: I'm back | 10:22 |
cinderella | you there? | 10:22 |
eharney | rushiagr: if your driver is just a remotefsdriver w/o snaps then it doesn't need these | 10:22 |
rushiagr | eharney: I think making those decorators unbound functions is cleaner, but I think the current way works well too, as remotefssnapdriver is just the base (sort of abstract) class | 10:22 |
rushiagr | eharney: yes, got that | 10:23 |
eharney | rushiagr: i disagree that having methods floating outside of classes is likely to be cleaner i guess | 10:23 |
rushiagr | eharney: I'm actually not aware of pros vs cons of these two approaches | 10:26 |
rushiagr | removed my -1 | 10:26 |
eharney | it's mostly just about code organization in this case | 10:26 |
cinderella | rushiagr: I'm quite in a mess with volumes. Seems like my cinder is in an inconsistent state | 10:26 |
eharney | rushiagr: ok thanks... i'll look around and see if there are some useful ideas in this area in general for how to handle this in Cinder | 10:27 |
rushiagr | eharney: I'll just comment on the patch saying that there is an alternate approach, to get opinions from others as well | 10:28 |
rushiagr | cinderella: hey | 10:28 |
rushiagr | cinderella: so what's the issue? | 10:28 |
cinderella | I simply can't attach volumes (that are correctly created) to instances. | 10:28 |
rushiagr | cinderella: can you tell me more about your environment? | 10:29 |
rushiagr | cinderella: also, can you see any exceptions/errors in the logs? | 10:29 |
cinderella | fine rushiagr | 10:30 |
cinderella | I have my openstack installed in one single machine | 10:30 |
cinderella | with all of the components (nova, cinder, glance, etc.) | 10:30 |
cinderella | I followed this guide: https://github.com/fornyx/OpenStack-Havana-Install-Guide/blob/master/OpenStack-Havana-Install-Guide.rst | 10:30 |
cinderella | everything works fine, and now I was fighting with cinder | 10:31 |
cinderella | I can create volumes and no problems | 10:31 |
cinderella | but I can't attach them to machine (actually, I'm trying to attach volumes to a CentOS VMs) | 10:31 |
cinderella | machines = instances | 10:32 |
cinderella | and now, when I try to delete volumes, they go on "error_deleting" and can't remove them | 10:32 |
openstackgerrit | Michal Dulko proposed openstack/cinder-specs: Integrate persistence support of TaskFlow https://review.openstack.org/147879 | 10:32 |
cinderella | also, when I try to run "cinder list", nothing shows: but I can see in horizon some volumes I created | 10:32 |
cinderella | it's all clear, rushiagr? need more info? | 10:32 |
rushiagr | cinderella: wow, that's weird | 10:32 |
*** Ilja has quit IRC | 10:32 | |
rushiagr | how did you attach those volumes? | 10:33 |
cinderella | rushiagr: via horizon. status "attaching" hangs for a while, then nothing happens | 10:33 |
cinderella | where can I paste my logs? | 10:33 |
cinderella | (sorry for my english in advance :) ) | 10:34 |
rushiagr | cinderella: you can use paste.openstack.org | 10:34 |
rushiagr | cinderella: no issues | 10:34 |
rushiagr | cinderella: can you refresh and see if volumes are present or not? | 10:34 |
rushiagr | refresh = refresh that page in horizon | 10:34 |
cinderella | rushiagr: yes: they are there and are stuck in "error_deleting" | 10:35 |
cinderella | rushiagr: cinder list shows absolutely nothing | 10:35 |
rushiagr | cinderella: oh, they're maybe coz you tried to delete them | 10:35 |
rushiagr | cinderella: maybe you're logged into horizon as a different user/tenant | 10:35 |
rushiagr | cinderella: check that too | 10:35 |
cinderella | it's fine | 10:36 |
cinderella | I had also this issue once | 10:36 |
cinderella | and solved via mysql workaroudn | 10:36 |
cinderella | workaround* | 10:36 |
cinderella | my biggest problems is the attachment | 10:36 |
rushiagr | cinderella: you were trying attachment via horizon or CLI? | 10:37 |
*** navid__ has joined #openstack-cinder | 10:39 | |
cinderella | rushiagr: horizon | 10:39 |
*** nshaikh has quit IRC | 10:40 | |
DuncanT | If horizon shows volumes but cinder list doesn't, you /must/ be using different creds or have a serious setup problem - they both use the same rest interface | 10:40 |
cinderella | rushiagr: I resolved the volume "error_deleting" problem, | 10:41 |
cinderella | DuncanT: solved it | 10:42 |
cinderella | now I'd like to understand why I can't attach volumes | 10:42 |
DuncanT | What was your solution? | 10:42 |
cinderella | DuncanT: we went into the mysql DB | 10:43 |
*** markvoelker has joined #openstack-cinder | 10:43 | |
DuncanT | That doesn't solve anything, and may have caused more problems | 10:43 |
DuncanT | You should never need to go into the database | 10:43 |
cinderella | DuncanT: I know, but cinder delete wasn't working, cinder force-delete wasn't working... not much choices | 10:44 |
DuncanT | If you delete the volumes out of the cinder db and they haven't been deleted on the backend, you now have no (easy) way of cleaning up | 10:44 |
DuncanT | You need to work through /why/ cinder delete wasn't' working - it is probably a similar reason as to why attach doesn't work | 10:45 |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Group test files in the right package in cinder.tests https://review.openstack.org/151232 | 10:45 |
cinderella | DuncanT: the weird thing is that deleting and creating volumes was working like a charm and then - quite suddenly - deleting stopped working | 10:45 |
cinderella | while attaching never worked | 10:45 |
*** jasondotstar has joined #openstack-cinder | 10:46 | |
DuncanT | Can you run create then delete, and post of the cinder-volume log for both operations, please? | 10:46 |
cinderella | btw you were probably right - now I created a volume via horizon and cinder-list is showing nothing | 10:46 |
*** alexpilotti has joined #openstack-cinder | 10:47 | |
cinderella | log incoming | 10:47 |
cinderella | DuncanT: http://paste.openstack.org/show/166712/ | 10:47 |
*** markvoelker has quit IRC | 10:48 | |
DuncanT | Are you certain you are using the same user in cinder and the cli? Because it really shouldn't be possible for volumes to show in one and not the other | 10:48 |
DuncanT | 2015-02-04 11:45:04.150 22912 WARNING cinder.volume.manager [-] Unable to update stats, driver is uninitialized | 10:48 |
DuncanT | So are your volumes ever showing as 'available' or do they get stuck in 'creating'? | 10:49 |
*** MentalRay has joined #openstack-cinder | 10:50 | |
cinderella | DuncanT: they are available | 10:50 |
DuncanT | Do you only have one cinder-volume service? | 10:50 |
cinderella | DuncanT: yes, just one | 10:50 |
DuncanT | Then you have something weird going on, because with the driver uninitialised, there should be no way for a volume to end up available, it should never get past the scheduler | 10:51 |
*** haomaiwang has quit IRC | 10:51 | |
*** rushiagr has quit IRC | 10:52 | |
cinderella | DuncanT: actually you were right, I had a user issue :) | 10:52 |
cinderella | I sourced the right creds and now I see correctly the volumes | 10:52 |
DuncanT | Ok, that is one issue solved. | 10:53 |
cinderella | (sorry for the dumb issue) | 10:53 |
DuncanT | Can you restart cinder-volume and post the logs from the restart? Hopefully that will tell us why the driver can't initialise | 10:53 |
cinderella | DuncanT: ok, they create and delete with no problems? | 10:53 |
cinderella | no ?, they create and delete with no problems. | 10:53 |
cinderella | ok DuncanT, gonna do that | 10:53 |
cinderella | logs incoming | 10:55 |
openstackgerrit | Michal Dulko proposed openstack/cinder-specs: Integrate TaskFlow's persistence support into volume https://review.openstack.org/147884 | 10:55 |
cinderella | DuncanT: http://paste.openstack.org/show/166713/ | 10:55 |
openstackgerrit | Michal Dulko proposed openstack/cinder-specs: Integrate TaskFlow's persistence into scheduler https://review.openstack.org/147883 | 10:55 |
*** Longgeek has joined #openstack-cinder | 10:55 | |
openstackgerrit | Michal Dulko proposed openstack/cinder-specs: Integrate TaskFlow's persistence support into API https://review.openstack.org/147881 | 10:55 |
*** enterprisedc has joined #openstack-cinder | 10:55 | |
DuncanT | Hopefully there are some more logs after that? | 10:56 |
cinderella | DuncanT: no - I didn't do any operation after the restart of the service | 10:56 |
DuncanT | There should still be something from driver init, and the stats periodic task every 30 sexconds | 10:57 |
*** MentalRay has quit IRC | 10:57 | |
cinderella | DuncanT: I'm "catting" the log continuosly - no new lines, just "2015-02-04 11:57:42.563 26296 INFO cinder.volume.manager [-] Updating volume status" each minute | 10:58 |
cinderella | DuncanT: maybe the driver warning was caused by the inconsistent situation? | 10:59 |
DuncanT | Do you have debug and verbose turned on in cider.conf? If not, please turn them on and restart | 10:59 |
cinderella | I try to create a volume and paste logs | 10:59 |
cinderella | DuncanT: I have verbose, but not debug | 11:00 |
DuncanT | I generally recommend turninng on debug when you have any issues, it makes tracing things far easier | 11:00 |
cinderella | DuncanT: new logs with debug = True -> http://paste.openstack.org/show/166716/ | 11:01 |
cinderella | this is cinder-volume.log | 11:02 |
*** rushiagr has joined #openstack-cinder | 11:02 | |
cinderella | DuncanT: any weird stuff? | 11:02 |
*** navid__ is now known as nshaikh | 11:03 | |
DuncanT | No, that looks fine | 11:03 |
cinderella | DuncanT: great! can you help me with attaching? | 11:05 |
DuncanT | Sure, run create then attach and post the cinder-volume log again | 11:05 |
cinderella | DuncanT: want me to do that via CLI or horizon? | 11:06 |
DuncanT | Either, it is all the same to cinder | 11:06 |
cinderella | DuncanT: ok, created VM and THEN started attaching | 11:08 |
cinderella | now, horizon is hanging with "attaching" | 11:08 |
cinderella | I'm waiting... still "attaching" | 11:09 |
cinderella | DuncanT: what logs should I paste? | 11:10 |
cinderella | just cinder-volume? | 11:10 |
DuncanT | Start with cinder-vol | 11:10 |
cinderella | (because I noticed in nova logs I have an error) | 11:10 |
cinderella | DuncanT: this is cinder-volume.log -> http://paste.openstack.org/show/166717/ | 11:11 |
cinderella | DuncanT: here you are nova logs where I found an error | 11:12 |
cinderella | DuncanT: this is the error I found in nova-compute.log -> http://paste.openstack.org/show/166718/ | 11:12 |
cinderella | (tried to attach to /dev/htc, if this can help) | 11:13 |
DuncanT | Ok, so nova can't find the export | 11:13 |
cinderella | DuncanT: what should I do? | 11:14 |
DuncanT | Give me a sec, I'm just trying to remember how target side iscsi commands work to get some more debug | 11:15 |
DuncanT | It has been a while since I've do anything target-side, I've forgotten far too much | 11:15 |
cinderella | it's fine DuncanT | 11:16 |
cinderella | if it can help, the VMs is a CentOS-6.6-x86_64-minimal.iso | 11:17 |
cinderella | maybe I should do something "Into the VM" so that the attaching may happen? | 11:17 |
DuncanT | No, this problem is before the VM | 11:18 |
DuncanT | Nova is trying to find the target on the cinder node and failing | 11:19 |
DuncanT | On the node running nova-volume, can you ping 10.0.0.230 ? | 11:20 |
cinderella | DuncanT: it's all in one server, all the components are running on the same machine | 11:20 |
cinderella | (this is our first time with openstack - waiting budget for more machines) | 11:21 |
DuncanT | Ok. Can you check that the server address is 10.0.0.230 please? | 11:21 |
*** e0ne is now known as e0ne_ | 11:21 | |
cinderella | DuncanT: yes, I'm pretty sure it is (SSH-ing it right now) | 11:21 |
DuncanT | I'm guessing at a network problem of some sort, since the cinder-vol log looks good | 11:21 |
cinderella | DuncanT: what do you think about this workaround? seems similar to mine: http://ubuntuforums.org/showthread.php?t=2045883 | 11:22 |
cinderella | and yes, I have tgt and iscsitarget running together | 11:23 |
DuncanT | Ah, then yes, that could well be your problem | 11:23 |
cinderella | DuncanT: what do you suggest to do? | 11:23 |
cinderella | uninstall one of the two? | 11:23 |
DuncanT | Try uninstalling iscsitarget and rebooting | 11:24 |
cinderella | you mean hard reboot my server? fine, gonna do that | 11:24 |
DuncanT | Yeah, you'll need a hard reboot after uninstalling | 11:24 |
DuncanT | If this works, please file a bug, we should at least detect the situation and print a suitable warning | 11:25 |
cinderella | ok DuncanT - doing it right now | 11:25 |
cinderella | DuncanT: I never "filed a bug", maybe I'll need help in that :) | 11:26 |
DuncanT | Sure, go to https://bugs.launchpad.net/cinder/ and use the 'report a bug' button in the top right | 11:26 |
*** avishay has quit IRC | 11:27 | |
cinderella | DuncanT: rebooted, now I'll try to attach | 11:29 |
cinderella | iscsitarget uninstalled | 11:29 |
cinderella | DuncanT: "attaching"... | 11:30 |
DuncanT | You uninstalled before rebooting, right? | 11:31 |
cinderella | DuncanT: naturally | 11:31 |
DuncanT | Good. Just wanted to be sure :-) | 11:31 |
*** e0ne_ has quit IRC | 11:31 | |
cinderella | DuncanT: have you got experience in using a NAS with cinder? because maybe this is our next step, when we get cinder working :) | 11:32 |
DuncanT | I've not really played with the filesystem based cinder drivers much, other than a bit of security work a while ago. Sorry. | 11:33 |
*** Longgeek has quit IRC | 11:33 | |
cinderella | DuncanT: no problems | 11:33 |
DuncanT | Does you attach work now? | 11:34 |
cinderella | DuncanT: boot is quite long, just a sec :) | 11:34 |
DuncanT | No rush | 11:34 |
cinderella | DuncanT: I'm attaching via horizon | 11:36 |
cinderella | "attaching" is hanging | 11:36 |
cinderella | hope it isn't taking too long... | 11:37 |
cinderella | DuncanT: error :( | 11:38 |
DuncanT | Anything in the nova log? | 11:38 |
cinderella | yep, gonna post | 11:38 |
cinderella | DuncanT: http://paste.openstack.org/show/166738/ | 11:39 |
DuncanT | Hmmm, same | 11:41 |
DuncanT | Can you run 'sudo iscsiadm -m session -P3' please? | 11:41 |
cinderella | immediately | 11:41 |
cinderella | iSCSI Transport Class version 2.0-870 version 2.0-871 iscsiadm: No active sessions. | 11:41 |
*** pschaef has joined #openstack-cinder | 11:42 | |
DuncanT | sudo ietadm --op show --tid=1 | 11:43 |
*** tellesnobrega__ has quit IRC | 11:43 | |
cinderella | "iscsitarget" not installed | 11:43 |
cinderella | you can install it via apt-get etc etc | 11:43 |
cinderella | sorry - "ietadm" not installed | 11:44 |
*** markvoelker has joined #openstack-cinder | 11:44 | |
cinderella | (not pasting it since it is italian) | 11:44 |
DuncanT | Ok, that's odd, since that is the command cinder was supposed to run. Can you post up the cinder-vol log again please? | 11:45 |
cinderella | DuncanT: here you are: http://paste.openstack.org/show/166749/ | 11:46 |
*** BharatK has quit IRC | 11:48 | |
*** markvoelker has quit IRC | 11:49 | |
DuncanT | So 12:35:53.611 is the initialise connection | 11:49 |
DuncanT | 12:37:55.820 there's a terminate connection | 11:50 |
DuncanT | I can't see where the iscsi target is being set up anymore :-( | 11:51 |
*** Ilja has joined #openstack-cinder | 11:54 | |
DuncanT | Sorry, I'm out of ideas on that one. I suspect it is something simple and I'm missing it | 11:55 |
*** diegows has joined #openstack-cinder | 11:57 | |
cinderella | DuncanT: no problems, thanks anyway :) | 12:00 |
cinderella | I completely don't know where to investigate now | 12:00 |
*** cinderella is now known as cinderella_away | 12:00 | |
DuncanT | Finding out whether there is actually an iscsi target exported would be the first thing | 12:01 |
cinderella_away | DuncanT: you there later? | 12:01 |
DuncanT | I'm completely forgotten how though | 12:01 |
*** david-lyle_afk is now known as david-lyle | 12:01 | |
DuncanT | I'll be here for a while, yes. Hopefully one of the US crowd will be on soon and they'll know more than me | 12:01 |
DuncanT | I honestly think I'm going senile or something | 12:01 |
cinderella_away | ahahah DuncanT don't worry | 12:03 |
cinderella_away | I'm 25 and can't figure this, so :D | 12:03 |
*** cinderella_away has quit IRC | 12:03 | |
DuncanT | I *should* be able to, my brain is just not working | 12:03 |
*** MIDENN_ has joined #openstack-cinder | 12:13 | |
*** hemna_ has joined #openstack-cinder | 12:13 | |
*** guitarza1 has joined #openstack-cinder | 12:13 | |
*** alexpilotti has quit IRC | 12:13 | |
*** Anticime1 has joined #openstack-cinder | 12:15 | |
*** rodrigod` has joined #openstack-cinder | 12:18 | |
*** jasondotstar has quit IRC | 12:20 | |
*** afazekas has quit IRC | 12:20 | |
*** mdenny has quit IRC | 12:20 | |
*** Anticimex has quit IRC | 12:20 | |
*** hemnafk has quit IRC | 12:20 | |
*** rodrigods has quit IRC | 12:20 | |
*** svasheka has quit IRC | 12:20 | |
*** sbadia has quit IRC | 12:20 | |
*** guitarzan has quit IRC | 12:20 | |
*** rodrigod` is now known as rodrigods | 12:22 | |
*** rodrigods has quit IRC | 12:22 | |
*** rodrigods has joined #openstack-cinder | 12:22 | |
*** svasheka has joined #openstack-cinder | 12:24 | |
*** sbadia has joined #openstack-cinder | 12:25 | |
*** afazekas has joined #openstack-cinder | 12:27 | |
*** EmilienM|afk is now known as EmilienM | 12:27 | |
*** david-lyle is now known as david-lyle_afk | 12:36 | |
*** david-lyle_afk is now known as david-lyle | 12:36 | |
*** Mandell has joined #openstack-cinder | 12:41 | |
*** david-lyle is now known as david-lyle_afk | 12:44 | |
*** david-lyle_afk is now known as david-lyle | 12:45 | |
*** markvoelker has joined #openstack-cinder | 12:45 | |
*** Mandell has quit IRC | 12:46 | |
*** takedakn has joined #openstack-cinder | 12:47 | |
*** markvoelker has quit IRC | 12:49 | |
*** IanGovett has joined #openstack-cinder | 12:52 | |
*** e0ne has joined #openstack-cinder | 12:53 | |
*** aix has joined #openstack-cinder | 13:01 | |
*** e0ne is now known as e0ne_ | 13:04 | |
*** e0ne_ is now known as e0ne | 13:05 | |
*** bswartz has quit IRC | 13:05 | |
*** Miouge has quit IRC | 13:09 | |
*** pradipta has quit IRC | 13:13 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Group test files in the right package in cinder.tests https://review.openstack.org/151232 | 13:14 |
*** akerr has joined #openstack-cinder | 13:17 | |
*** markvoelker has joined #openstack-cinder | 13:20 | |
*** tshefi has quit IRC | 13:22 | |
*** xyang has joined #openstack-cinder | 13:22 | |
*** tshefi has joined #openstack-cinder | 13:22 | |
*** e0ne is now known as e0ne_ | 13:29 | |
*** karimb has quit IRC | 13:29 | |
*** nshaikh has quit IRC | 13:34 | |
*** nshaikh has joined #openstack-cinder | 13:35 | |
*** e0ne_ is now known as e0ne | 13:38 | |
*** nshaikh has quit IRC | 13:41 | |
*** karimb has joined #openstack-cinder | 13:43 | |
*** bswartz has joined #openstack-cinder | 13:45 | |
*** rushiagr is now known as rushiagr_away | 13:45 | |
*** bswartz has quit IRC | 14:00 | |
*** ronenkat has joined #openstack-cinder | 14:02 | |
*** bswartz has joined #openstack-cinder | 14:03 | |
*** ronenkat_ has quit IRC | 14:05 | |
*** mwichmann has joined #openstack-cinder | 14:05 | |
*** dulek has quit IRC | 14:06 | |
*** takedakn has quit IRC | 14:07 | |
*** dustins has joined #openstack-cinder | 14:07 | |
*** dulek has joined #openstack-cinder | 14:07 | |
openstackgerrit | Navneet Singh proposed openstack/cinder: Implement Cinder Manage/Unmanage APIs in NFS driver https://review.openstack.org/138125 | 14:11 |
*** kaufer has joined #openstack-cinder | 14:12 | |
openstackgerrit | xing-yang proposed openstack/cinder: Support over subscription in thin provisioning https://review.openstack.org/142171 | 14:16 |
*** xyang has quit IRC | 14:16 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: WIP: Add persistence to scheduler workflow https://review.openstack.org/152200 | 14:16 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Common changes for TaskFlow's persistence support https://review.openstack.org/152198 | 14:16 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Serialize data passed to scheduler workflow https://review.openstack.org/152199 | 14:16 |
*** rushiagr_away is now known as rushiagr | 14:18 | |
*** julim has joined #openstack-cinder | 14:19 | |
*** Yogi1 has joined #openstack-cinder | 14:20 | |
*** julim has quit IRC | 14:21 | |
*** jcru has joined #openstack-cinder | 14:21 | |
*** julim has joined #openstack-cinder | 14:24 | |
*** erlon has quit IRC | 14:26 | |
*** Mandell has joined #openstack-cinder | 14:28 | |
*** jecarey has quit IRC | 14:28 | |
*** chenleji has joined #openstack-cinder | 14:29 | |
*** jungleboyj has quit IRC | 14:30 | |
*** Longgeek has joined #openstack-cinder | 14:33 | |
*** Longgeek has quit IRC | 14:35 | |
openstackgerrit | nikeshmahalka proposed openstack/cinder: Add iSCSI SCST Target support to cinder https://review.openstack.org/145778 | 14:37 |
*** brut has joined #openstack-cinder | 14:38 | |
*** Mandell has quit IRC | 14:41 | |
nikesh_vedams_ | thingee jdb : now in new patch no db dependency and also not inheriting volume driver class in target driver class(scst.py),looking forward for further reviews in k-3 | 14:41 |
*** e0ne is now known as e0ne_ | 14:44 | |
*** mriedem has joined #openstack-cinder | 14:45 | |
*** mdbooth has quit IRC | 14:45 | |
*** eharney has quit IRC | 14:45 | |
*** dulek has quit IRC | 14:46 | |
*** dulek has joined #openstack-cinder | 14:47 | |
*** e0ne_ is now known as e0ne | 14:47 | |
*** sgotliv has quit IRC | 14:47 | |
*** mdbooth has joined #openstack-cinder | 14:50 | |
*** EduardBiceriMate has joined #openstack-cinder | 14:52 | |
*** rushil has joined #openstack-cinder | 14:54 | |
*** brut has quit IRC | 14:55 | |
openstackgerrit | Abel Lopez proposed openstack/cinder: Purge deleted rows https://review.openstack.org/146766 | 14:57 |
*** hemna has joined #openstack-cinder | 14:57 | |
*** stefan_amann has joined #openstack-cinder | 14:57 | |
*** Yogi11 has joined #openstack-cinder | 14:58 | |
*** dustins_ has joined #openstack-cinder | 15:00 | |
*** Yogi1 has quit IRC | 15:01 | |
*** dustins has quit IRC | 15:03 | |
*** rushil has quit IRC | 15:04 | |
*** thangp has joined #openstack-cinder | 15:04 | |
*** primechuck has joined #openstack-cinder | 15:07 | |
*** BharatK has joined #openstack-cinder | 15:09 | |
*** ronenkat_ has joined #openstack-cinder | 15:10 | |
*** ronenkat has quit IRC | 15:14 | |
*** EmilienM has quit IRC | 15:14 | |
*** julim has quit IRC | 15:15 | |
*** EmilienM has joined #openstack-cinder | 15:16 | |
*** tbarron has joined #openstack-cinder | 15:17 | |
*** julim has joined #openstack-cinder | 15:18 | |
*** jungleboyj has joined #openstack-cinder | 15:21 | |
*** mtanino has joined #openstack-cinder | 15:22 | |
openstackgerrit | Stefan Amann proposed openstack/cinder: Adjust Cinder to support FCP on System z systems https://review.openstack.org/149256 | 15:22 |
*** IanGovett has quit IRC | 15:23 | |
*** IanGovett has joined #openstack-cinder | 15:24 | |
*** MentalRay has joined #openstack-cinder | 15:24 | |
*** Yogi11 has quit IRC | 15:26 | |
*** ronenkat_ is now known as ronenkat | 15:26 | |
*** rmesta has joined #openstack-cinder | 15:27 | |
*** Yogi1 has joined #openstack-cinder | 15:27 | |
*** jecarey has joined #openstack-cinder | 15:34 | |
*** nlevinki has quit IRC | 15:34 | |
*** sgotliv has joined #openstack-cinder | 15:35 | |
*** EmilienM is now known as EmilienM|afk | 15:35 | |
*** e0ne is now known as e0ne_ | 15:37 | |
*** BharatK has quit IRC | 15:39 | |
*** newbism has joined #openstack-cinder | 15:42 | |
*** diegows has quit IRC | 15:44 | |
rushiagr | Can somebody get this out of the way? https://review.openstack.org/#/c/150876/ (Fix up help message for reset-state call). Too many +1's | 15:45 |
rushiagr | :) | 15:45 |
*** e0ne_ is now known as e0ne | 15:47 | |
*** lpetrut has quit IRC | 15:47 | |
*** BharatK has joined #openstack-cinder | 15:51 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: WIP: Add persistence to scheduler workflow https://review.openstack.org/152200 | 15:55 |
*** xyang has joined #openstack-cinder | 15:55 | |
*** erlon has joined #openstack-cinder | 15:56 | |
thingee | rushiagr: I'm assuming it hasn't been pushed forward yet because of the k-2 priorities | 15:56 |
flip214 | /join #openstack-meeting | 15:57 |
*** dannywilson has joined #openstack-cinder | 15:57 | |
rushiagr | thingee: yeah, quite possible | 15:57 |
flip214 | meeting now? | 16:00 |
*** tshefi has quit IRC | 16:00 | |
rushiagr | meeting time? | 16:02 |
flip214 | thingee: ^^ | 16:02 |
*** patrickeast has joined #openstack-cinder | 16:04 | |
*** ronis has quit IRC | 16:04 | |
*** avishay has joined #openstack-cinder | 16:04 | |
*** diemt has joined #openstack-cinder | 16:04 | |
*** tsekiyama has joined #openstack-cinder | 16:04 | |
*** diegows has joined #openstack-cinder | 16:04 | |
*** abehl has quit IRC | 16:05 | |
*** sgotliv has quit IRC | 16:06 | |
*** dannywilson has quit IRC | 16:13 | |
*** dannywilson has joined #openstack-cinder | 16:13 | |
*** ebalduf has joined #openstack-cinder | 16:16 | |
*** abehl has joined #openstack-cinder | 16:16 | |
*** dustins_ has quit IRC | 16:17 | |
*** EmilienM|afk is now known as EmilienM | 16:18 | |
*** dustins has joined #openstack-cinder | 16:19 | |
*** lpetrut has joined #openstack-cinder | 16:22 | |
*** IanGovett has quit IRC | 16:24 | |
*** coolsvap is now known as coolsvap_ | 16:28 | |
*** Anticime1 is now known as Anticimex | 16:29 | |
xyang | winston-d: hi, I updated the over subscription patch. addressed your comments. can you take a look again? thanks! | 16:29 |
*** dannywilson has quit IRC | 16:30 | |
*** Mandell has joined #openstack-cinder | 16:30 | |
winston-d | xyang: sure | 16:31 |
*** IanGovett has joined #openstack-cinder | 16:33 | |
xyang | winston-d: I'd like it to make k-2 if possible. thanks! | 16:34 |
*** IanGovett1 has joined #openstack-cinder | 16:35 | |
*** MentalRay has quit IRC | 16:36 | |
*** abehl has quit IRC | 16:36 | |
*** IanGovett has quit IRC | 16:37 | |
openstackgerrit | Shay Halsband proposed openstack/cinder: XtreamIO version 4.0 support https://review.openstack.org/152973 | 16:38 |
*** rwsu-afk is now known as rwsu | 16:40 | |
*** lan_ has quit IRC | 16:40 | |
*** diegows has quit IRC | 16:41 | |
*** IanGovett has joined #openstack-cinder | 16:41 | |
*** lan has joined #openstack-cinder | 16:42 | |
*** primechuck has quit IRC | 16:43 | |
*** IanGovett1 has quit IRC | 16:44 | |
*** MentalRay has joined #openstack-cinder | 16:48 | |
*** e0ne is now known as e0ne_ | 16:54 | |
flip214 | thingee: just for clarification, please: If I understand #meeting correctly, changing the DRBD driver to support the DRBD transport to Nova nodes would classify it as "new driver", and it's too late for that? | 16:56 |
openstackgerrit | Abel Lopez proposed openstack/cinder: Purge deleted rows https://review.openstack.org/146766 | 16:56 |
flip214 | I'm waiting for jgriffiths feedback since, oh, about 3 weeks on that ... | 16:56 |
*** avishay has quit IRC | 16:57 | |
*** diegows has joined #openstack-cinder | 16:57 | |
*** annegent_ has joined #openstack-cinder | 16:58 | |
*** ronenkat has quit IRC | 16:59 | |
flip214 | hmmm, all we can do, is sit and wait ... | 17:00 |
*** patrickeast has quit IRC | 17:00 | |
smcginnis | kmartin: We have a FC driver and would potentially be interested in a joint preso. | 17:00 |
rhe00_ | kmartin: the same for us. I also have other ideas for presentation topics. | 17:01 |
xyang | kmartin: I am interested in the FC session too | 17:01 |
*** dustins has quit IRC | 17:01 | |
*** rajinir_r has joined #openstack-cinder | 17:01 | |
rhe00_ | could we set up a meeting in a meeting channel so we don't have to fill up this channel with the discussion? | 17:02 |
nikesh_vedams_ | is 14 feb deadline confirmed? | 17:02 |
smcginnis | Even if we don't all have a lot to say, having a large group from different companies up on stage could help show that FC really is an option in cinder. | 17:02 |
smcginnis | rhe00_: Just start an etherpad? | 17:02 |
kmartin | smcginnis, rhe00_ xyang i think it ok to talk in here, it's cinder related | 17:02 |
*** vilobhmm has joined #openstack-cinder | 17:03 | |
kmartin | I have started on an abstract for the Past, present and future of Fibre Channel in OpenStack | 17:03 |
*** atan8 has joined #openstack-cinder | 17:04 | |
*** primechuck has joined #openstack-cinder | 17:04 | |
smcginnis | Notes and brainstorm here? https://etherpad.openstack.org/p/cinder-cross-company-presos | 17:04 |
*** vilobhmm_ has joined #openstack-cinder | 17:04 | |
*** nkrinner has quit IRC | 17:05 | |
*** Yogi1 has quit IRC | 17:05 | |
*** chenleji has quit IRC | 17:05 | |
vilobhmm_ | are we meeting today can’t see any activity in openstack-meeting room | 17:06 |
rhe00_ | vilobhmm: meeting started an hour ago, already over | 17:07 |
*** vilobhmm has quit IRC | 17:07 | |
*** vilobhmm_ is now known as vilobhmm | 17:07 | |
*** Adriano_ has joined #openstack-cinder | 17:07 | |
vilobhmm | oops | 17:07 |
nikesh_vedams_ | thingee:in meeting you mentioned 15 feb deadline,is it regarding merging of code in kilo? | 17:09 |
*** Yogi1 has joined #openstack-cinder | 17:09 | |
dulek | nikesh_vedams_: Merging specs and getting BPs approved | 17:10 |
*** amann has joined #openstack-cinder | 17:10 | |
dulek | Guys, anyone to share some info for the mid-cycle about bug 1408763? | 17:11 |
hemna | thingee, sorry I missed the meeting this morning. I was presenting to the FC standards body at the same time. | 17:11 |
*** Adriano__ has joined #openstack-cinder | 17:11 | |
*** pschaef has quit IRC | 17:12 | |
hemna | thingee, also FWIW, I posted on the ML last night to nova and cinder about brick and our plans. | 17:12 |
vilobhmm | thingee, sorry me too missed the meeting…would like to know about the timeline for the code cutoff for kilo release | 17:13 |
nikesh_vedams_ | dulek: ok so for already approved bps which are still under code review and targeted to k-3,deadline would be expected 19th march | 17:14 |
dulek | nikesh_vedams_: That's true. | 17:14 |
nikesh_vedams_ | sounds great,looking forward for https://review.openstack.org/#/c/145778/ in k-3 | 17:15 |
dulek | Anyone knows Anyone knows if jgriffith will be around today? | 17:16 |
*** amann has quit IRC | 17:18 | |
*** stefan_amann has quit IRC | 17:18 | |
*** alecv has quit IRC | 17:20 | |
*** atan8 has quit IRC | 17:22 | |
thingee | vilobhmm: http://lists.openstack.org/pipermail/openstack-dev/2015-January/055719.html | 17:22 |
thingee | nikesh_vedams_: http://lists.openstack.org/pipermail/openstack-dev/2015-January/055719.html | 17:22 |
vilobhmm | thingee : thanks | 17:22 |
thingee | flip214: how does drbd work currently? | 17:23 |
*** leeantho has joined #openstack-cinder | 17:25 | |
*** jistr has quit IRC | 17:25 | |
nikesh_vedams_ | thingee: thanks | 17:25 |
*** gtt116__ has joined #openstack-cinder | 17:25 | |
nikesh_vedams_ | dulex: march 10 for code freeze :) | 17:26 |
*** ronis has joined #openstack-cinder | 17:26 | |
*** annashen has joined #openstack-cinder | 17:27 | |
dulek | nikesh_vedams_: I knew it was in march. ;) | 17:27 |
nikesh_vedams_ | ok sorry | 17:28 |
*** bswartz has quit IRC | 17:28 | |
*** gtt116_ has quit IRC | 17:28 | |
*** MentalRay has quit IRC | 17:29 | |
*** bswartz has joined #openstack-cinder | 17:30 | |
*** e0ne_ is now known as e0ne | 17:30 | |
thingee | xyang: https://review.openstack.org/#/c/152973/ | 17:31 |
*** jpich has quit IRC | 17:31 | |
thingee | xyang: I'm guessing this posted now for your legal reasons :) | 17:31 |
*** BharatK has quit IRC | 17:33 | |
thingee | kmartin, hemna: is it possible for tempest to be useful today with brocade setup to a fc driver? | 17:33 |
kmartin | thingee, is should have a CI IMO | 17:34 |
thingee | kmartin: but if a fc driver is setup with brocade, would that indirectly test it still with tempest? | 17:35 |
thingee | I imagine it would, but confirming | 17:35 |
xyang | thingee: I'll talk to them about breaking the manage/unmanage out. This is not a new driver though | 17:36 |
xyang | thingee: they are going to use this updated driver to continue CI testing | 17:37 |
hemna | thingee, so yes, if you have CI setup to use FCZM, then tempest indirectly tests the FCZM | 17:37 |
thingee | xyang: ibm is introducing another protocol as well | 17:37 |
thingee | we just discussed this | 17:37 |
xyang | thingee: it has a FC and an iscsi driver | 17:37 |
kmartin | thingee, we prezone the fabric for our CI currently | 17:37 |
thingee | keyword, protocol | 17:37 |
xyang | thingee: that is not the transport protocol we are talking about. | 17:37 |
thingee | this protocol vX and vX | 17:37 |
thingee | no | 17:37 |
thingee | it's too late | 17:37 |
xyang | thingee: that is their internal protocol | 17:37 |
hemna | thingee, but yes, it's possible. | 17:38 |
thingee | doesn't matter. keyword protocol. I can't make it anymore clear | 17:38 |
thingee | this is ridiculous. You didn't have a problem when I said no to IBM, but you have a problem when I say no to xtreme io | 17:39 |
xyang | thingee: ok, I'll talk to them and understand more about the change. I'll ask them to fix the CI issue in the existing driver first | 17:39 |
xyang | thingee: not ridiculous | 17:39 |
xyang | thingee: I think that is the XML parsing protocol, but because it is confusing, I'll find out | 17:40 |
xyang | thingee: I'll ask them to fix the CI first | 17:40 |
thingee | xyang: I wouldn't argue about this honestly. I have half the mind to say no your other 5K plus patches for driver updates. You're already getting enough review resources as it is. | 17:41 |
thingee | I don't care if they are mostly doc strings | 17:41 |
xyang | thingee: ok | 17:41 |
kmartin | thingee, Is this a bad time to have my two driver BPs added to K3? https://blueprints.launchpad.net/cinder/+spec/3par-dedup and https://blueprints.launchpad.net/cinder/+spec/3par-flash-cache | 17:43 |
* jungleboyj pictures kmartin spraying gasoline on a roaring fire | 17:44 | |
openstackgerrit | Abel Lopez proposed openstack/cinder: Purge deleted rows https://review.openstack.org/146766 | 17:45 |
thingee | I'm not concerned about drivers introducing new features. My mind is just blown about this because we just talked about this and no one raised objections. | 17:45 |
*** p01s0n has joined #openstack-cinder | 17:45 | |
kmartin | thingee, I quickly added these at the meetup, it was not clear that they were required...the 1st already has code posted(sorry) the second is started | 17:45 |
winston-d | xyang: i'm afraid you need another version of overprovision. i have two more nits for you to handle. | 17:46 |
*** dulek has quit IRC | 17:47 | |
*** dulek has joined #openstack-cinder | 17:47 | |
thingee | kmartin: done | 17:49 |
thingee | whoops | 17:49 |
e0ne | Abel Lopez: are you around? | 17:49 |
thingee | kmartin: were you able to see the proposed for kilo in series goal? | 17:50 |
thingee | kmartin: I might be able to use that for people to get my attention | 17:50 |
kmartin | thingee, yes | 17:50 |
kmartin | is that new? | 17:51 |
thingee | kmartin: ok they're targeted | 17:51 |
*** _cjones_ has joined #openstack-cinder | 17:51 | |
thingee | kmartin: I don't think so...just not sure anymore what I'm able to do versus other people in launch pad...was just confirming | 17:51 |
kmartin | thingee, thanks...were you notified when I set that | 17:52 |
p01s0n | i am facing some issue with attaching volume to instance with emc vnx as backend ,its showing this error "Unable to fetch connection information from backend: Bad or unexpected response from the storage volume backend API: Failed to add volume-0c4be426-39e3-4321-aa54-14e30ec7c7d5 into SG" | 17:52 |
xyang | winston-d: sure, I'll change that. | 17:52 |
*** annegent_ has quit IRC | 17:53 | |
winston-d | xyang: thx, i'll review that first thing tomorrow. need to get some sleep now. | 17:53 |
*** winston-d is now known as winston-d_zZZ | 17:53 | |
xyang | p01s0n: hi, please send me logs and I'll ask the VNX team to take a look. thanks | 17:53 |
xyang | winston-d: sure. good night. Thanks for help! | 17:54 |
*** coolsvap_ is now known as coolsvap | 17:57 | |
rhe00_ | patrickeast: ping | 17:58 |
p01s0n | xyang: cinder-volume log http://pastebin.com/CN1fcczD , configuration : http://pastebin.com/J6aer5xU | 17:58 |
p01s0n | xyang: can you have a look | 17:58 |
xyang | p01s0n: It's not clear to me what is the problem. what version is this? | 18:00 |
thingee | How to contribute a driver has been updated https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver#Deadline_for_Kilo | 18:00 |
p01s0n | xyang : icehouse | 18:01 |
thingee | bswartz: is anyone at netapp able to pick this up...looks like it's failing one unit test and I'm sure Navneet is gone for the day https://review.openstack.org/#/c/138125/12 | 18:02 |
bswartz | thingee: I'll round someone up | 18:02 |
bswartz | thingee: you can also shout about this stuff in #openstack-netapp -- most of our team hangs out there when they're online | 18:03 |
thingee | otherwise no worries, I can bump it to k-3, but it's close to merge | 18:03 |
thingee | good to know, thanks | 18:04 |
xyang | p01s0n: ok, I'll ask someone to take a look and get back to you. Did you send me an email or is this a new problem? | 18:04 |
p01s0n | xyang : i havent sent any mail,i will send a detailed mail | 18:04 |
xyang | p01s0n: sure. you know my email, right | 18:05 |
p01s0n | xyang :yes | 18:05 |
*** vilobhmm has quit IRC | 18:05 | |
xyang | p01s0n: ok | 18:05 |
*** aix has quit IRC | 18:05 | |
*** karimb has quit IRC | 18:06 | |
thingee | kmartin: I get emails about all blueprint activity. It's not useful | 18:06 |
thingee | e0ne: is there going to be a problem with this just doing a raise in a couple of places instead of giving an actually exception to raise? https://review.openstack.org/#/c/146766/17/cinder/db/sqlalchemy/api.py | 18:07 |
thingee | e0ne: line 3302 and 3334 | 18:08 |
e0ne | thingee: not sure that i've got your point. in just re-raise catched exception | 18:08 |
rhe00_ | thingee: what's the process to update http://docs.openstack.org/developer/cinder/devref/drivers.html? would be nice if it referenced Kilo instead of Havana and Icehouse. | 18:08 |
*** rushil has joined #openstack-cinder | 18:09 | |
*** dustins has joined #openstack-cinder | 18:09 | |
thingee | rhe00_: that's the doc directory in the cinder tree | 18:09 |
e0ne | thingee: do you think that we need to raise more specific excepitons? | 18:09 |
flip214 | thingee: the currently merged DRBD driver uses jgriffiths iscsi target driver. | 18:09 |
thingee | rhe00_: so same as code | 18:10 |
jungleboyj | thingee: I have one blueprint that we have code up for but the BP hasn't been approved yet. https://blueprints.launchpad.net/cinder/+spec/linux-systemz | 18:10 |
rhe00_ | thingee: ok, got it. | 18:10 |
jungleboyj | thingee: Would appreciate feedback on that one. | 18:10 |
thingee | e0ne: my point is it's just doing 'raise' instead 'raise SomeException' | 18:10 |
flip214 | using the drbd protocol up to the nova nodes should be a) much faster b) more resilient against network interruptions. | 18:10 |
*** ebalduf has quit IRC | 18:10 | |
thingee | e0ne: I'm asking if that's going to be a problem | 18:10 |
thingee | flip214: might be a problem then yes. | 18:10 |
*** harlowja has joined #openstack-cinder | 18:11 | |
e0ne | thingee: what kind of problem are you talking about? | 18:11 |
*** jasondotstar has joined #openstack-cinder | 18:11 | |
thingee | flip214: are you going to be able to provide a ci for iscsi and your own connector? | 18:11 |
thingee | by march 19th? | 18:11 |
thingee | also is the nova part going to land? | 18:11 |
*** dkingshott has joined #openstack-cinder | 18:11 | |
flip214 | thingee: we're planning to. sadly the available hardware isn't good enough, and the replacement has not yet arrived. | 18:11 |
flip214 | it should be here in the next 2 weeks, though ... | 18:12 |
flip214 | and I hope that I can transfer the CI that I had up to now over... | 18:12 |
thingee | flip214: a little advice, I would worry about making sure what you already got in tree stays in tree. | 18:12 |
thingee | not some new connector. I know you got pressure from your company about it, but I would convey that to them | 18:13 |
*** jasondotstar has quit IRC | 18:14 | |
thingee | jungleboyj: is the nova side going to happen in K? | 18:15 |
thingee | ah looks like some stuff has been merged on that side | 18:15 |
thingee | and you're doing it in small patches for different features, nice | 18:15 |
jungleboyj | It is approved and under review. | 18:16 |
jungleboyj | Met with these guys in Paris. They seem to get the community. I think. | 18:16 |
thingee | jungleboyj: so I do have an outstanding question on this | 18:17 |
thingee | https://review.openstack.org/#/c/141092/6 | 18:17 |
thingee | it's in the comments | 18:17 |
e0ne | thingee: it's in a API, maybe raising InvalidParameterValue will be better | 18:18 |
thingee | e0ne: I agree | 18:18 |
e0ne | ok, i'll add it to a review | 18:18 |
jungleboyj | hemna: ^^ | 18:18 |
thingee | hemna, jungleboyj: this might just be a problem with brick though. but I really don't want a bunch of if system this, do this all over the place | 18:19 |
*** dannywilson has joined #openstack-cinder | 18:20 | |
hemna | sup | 18:20 |
*** coolsvap is now known as coolsvap_ | 18:20 | |
jungleboyj | Talking about https://blueprints.launchpad.net/cinder/+spec/linux-systemz | 18:21 |
thingee | anish: comments posted for https://review.openstack.org/#/c/147758/11,publish | 18:21 |
jungleboyj | hemna: thingee has questions for you in the spec review. | 18:21 |
thingee | hemna: this touches brick and is introducing a problem imo. | 18:22 |
hemna | yah the conditionals are a bit of a PITA | 18:22 |
e0ne | thingee, anish: i've added comments to https://review.openstack.org/#/c/146766/. i hope, next patch will be a final | 18:22 |
hemna | I'm not sure how else he'd do it at the moment | 18:22 |
jungleboyj | hemna: I had the same thought and same reaction. | 18:23 |
hemna | actually | 18:23 |
hemna | if this is for volume discovery | 18:23 |
thingee | anish, bswartz, xyang: I want to do a cut of k-2 by feb 5th 2:00 UTC | 18:23 |
*** patrickeast has joined #openstack-cinder | 18:24 | |
hemna | hrmm yah I dunno | 18:24 |
anish | thingee: so on the first comment, I actually mention this in the reply | 18:24 |
anish | I _absolutely_ could not figure out how to mix configuration.safe_get & self.fake_volumes_dir | 18:25 |
anish | any pointers ? | 18:26 |
jungleboyj | hemna: thingee So, given this discussion, can we target the BP and finish working the details through the review? | 18:26 |
jungleboyj | I guess Spec review. | 18:26 |
thingee | jungleboyj: IMO figure this is the point of the spec, no? | 18:26 |
thingee | figuring* | 18:27 |
thingee | you're more than welcome to work on the code, which has already happened...so why not discuss this now? | 18:27 |
jungleboyj | thingee: Agreed. | 18:28 |
* e0ne need to go offline. bb in ~2hours | 18:28 | |
jungleboyj | Oh wait, this is approved and targetted. | 18:28 |
jungleboyj | I swore it wasn't when I looked yesterday. | 18:29 |
openstackgerrit | Patrick East proposed openstack/cinder: Make PureISCSIDriver iSCSI port discovery more flexible https://review.openstack.org/152693 | 18:29 |
thingee | anish: https://github.com/openstack/cinder/blob/master/cinder/tests/volume/drivers/datera.py#L33 | 18:29 |
thingee | anish: not sure if that helps | 18:29 |
thingee | anish: as an example | 18:29 |
thingee | jungleboyj: no you're right I just targeted it, but I forgot about this issue in the spec | 18:30 |
thingee | jungleboyj: which is why I want it discussed | 18:30 |
hemna | yah looks like it's targeted for k3 already | 18:30 |
jungleboyj | thingee: Gotcha. | 18:30 |
anish | thingee: looking, thanks | 18:31 |
jungleboyj | hemna: How else can we do this without checking out the platform? | 18:32 |
hemna | I don't know really | 18:32 |
hemna | the paths are very different for that platform vs x86 | 18:32 |
*** p01s0n has quit IRC | 18:32 | |
rhe00_ | patrickeast: any chance you can take a moment and point me to your nodepool yaml. Would be super to have a working real life example while trying to figure this out. | 18:32 |
jungleboyj | The bigger question is how big an issue is this going to be? Are we really going to have a bunch of different platforms doing 'if' in brick? | 18:33 |
thingee | jungleboyj: I don't know. that's the problem | 18:34 |
thingee | but it's definitely making the main connector code in brick more complicated | 18:34 |
thingee | jungleboyj, hemna: https://review.openstack.org/#/c/149256/9/cinder/brick/initiator/connector.py | 18:34 |
thingee | 0x00$s02x000000000000 | 18:34 |
thingee | :) | 18:35 |
thingee | if hemna doesn't care, then ok with me. | 18:37 |
thingee | I don't have an answer for this. | 18:37 |
hemna | ugh yah that code is a pita | 18:37 |
hemna | the alternative is to not even check for the system | 18:37 |
hemna | so that loop is all about creating a possible list of entries that might show up | 18:38 |
*** rushiagr is now known as rushiagr_away | 18:38 | |
hemna | the search looks until it finds the first one. | 18:39 |
hemna | so we could just remove the ifs and create s390x possible entries as well. | 18:39 |
hemna | bleh | 18:39 |
hemna | not idea either | 18:39 |
thingee | can I get some cores on these quick ones https://review.openstack.org/151710 https://review.openstack.org/151959 | 18:39 |
thingee | driver removals ^ | 18:39 |
hemna | ok | 18:39 |
openstackgerrit | Abel Lopez proposed openstack/cinder: Purge deleted rows https://review.openstack.org/146766 | 18:40 |
*** e0ne is now known as e0ne_ | 18:41 | |
*** jordanP has quit IRC | 18:41 | |
*** MentalRay has joined #openstack-cinder | 18:41 | |
thingee | hemna: ok well I already have a +1 from you on the spec. | 18:42 |
*** p01s0n has joined #openstack-cinder | 18:42 | |
thingee | hemna: let me know if you're fine with it as is and we can continue....but honestly I don't know how many more of these we'll see in brick | 18:42 |
hemna | so, if we start to see more of these, then a refactor will be needed IMHO | 18:43 |
hemna | to make it more manageable | 18:43 |
*** alop has joined #openstack-cinder | 18:43 | |
p01s0n | xyang: i think the issue is with vnx itself,each time i need to map LUN to storage group after that attach to instance is working | 18:43 |
thingee | hemna: not disagreeing with you, but if we look on the target side, it was a pita for some time. Thank goodness jdg refactored things. | 18:44 |
thingee | hemna: what we found was things just didn't work unless you were tgt or lio. | 18:44 |
thingee | iet, iser didn't work, etc | 18:44 |
alop | e0ne_: Thanks for your feedback, | 18:44 |
*** e0ne_ is now known as e0ne | 18:45 | |
hemna | so the other side is | 18:45 |
hemna | where is the CI to test this ? | 18:45 |
thingee | alop: I'm fine with it after that...assuming you're the person working on db purge | 18:45 |
alop | ya | 18:45 |
thingee | hemna: haha | 18:45 |
*** e0ne is now known as e0ne_ | 18:45 | |
hemna | :P | 18:45 |
alop | thingee: I'm that happy bald guy who keeps talking to you like he knows you :) | 18:45 |
thingee | hemna, jungleboyj: you know I was going to bring that up, but honestly, I think a cert test now is fine...like we did with target drivers. CI requirements need to be updated in L IMO | 18:46 |
thingee | alop: gotta be more specific :P | 18:46 |
*** e0ne_ is now known as e0ne | 18:46 | |
*** e0ne is now known as e0ne_ | 18:47 | |
hemna | yah sounds good | 18:47 |
*** diemt has quit IRC | 18:47 | |
thingee | jungleboyj: is that fair? | 18:47 |
*** e0ne_ is now known as e0ne | 18:48 | |
*** e0ne has quit IRC | 18:48 | |
patrickeast | rhe00_: sry for the delay, just got in and had a scrum meeting… here is the nodepool.yaml i am currently running http://paste.openstack.org/show/166943/ | 18:49 |
rhe00_ | patrickeast: thanks! | 18:51 |
patrickeast | rhe00_: let me know if you have questions about the fields, happy to explain what its doing | 18:51 |
*** afazekas has quit IRC | 18:51 | |
tbarron | thingee: i'm gonna look at https://review.openstack.org/#/c/138125/12 and see if I can figure why that unit test is failing | 18:53 |
tbarron | thingee: off to a meeting in 7 min, but just wanted to let you know we're not ignoring the issue | 18:53 |
*** annegent_ has joined #openstack-cinder | 18:54 | |
rhe00_ | patrickeast: this will help fill in some of the missing pieces in my understanding of nodepool configuration. thanks again! | 18:54 |
thingee | tbarron: thanks | 18:54 |
tbarron | thingee: and appreciate the opportunity to correct it before K-2 cut | 18:54 |
alop | thingee: looks like Ivan just went offline, did I finally get the exception piece right? | 18:58 |
*** annegent_ has quit IRC | 18:59 | |
*** Yogi11 has joined #openstack-cinder | 18:59 | |
*** annegent_ has joined #openstack-cinder | 19:00 | |
*** bswartz has quit IRC | 19:01 | |
*** dustins_ has joined #openstack-cinder | 19:01 | |
*** Yogi1 has quit IRC | 19:01 | |
*** bswartz has joined #openstack-cinder | 19:03 | |
*** dustins has quit IRC | 19:03 | |
nikesh_vedams_ | jdg: there? | 19:04 |
*** rushil has quit IRC | 19:04 | |
*** vilobhmm has joined #openstack-cinder | 19:05 | |
*** ks_ne has quit IRC | 19:06 | |
jungleboyj | thingee: So, the answer is that we are ok with the if's for now. Need to readdress if we have a lot of platforms show up. | 19:07 |
jungleboyj | thingee: Need to get a cert test result. Which I think is fair. | 19:07 |
*** bill_az_ has joined #openstack-cinder | 19:07 | |
xyang | p01s0n: so you get it resolved? anyway, if you still have issues, shoot me an email with logs and we'll take a look | 19:09 |
xyang | thingee: ok, I'm updating the over subscription patch now. will see if winston-d can take a look before the cut off time. need another core too. Is jdg around? | 19:10 |
*** ebalduf has joined #openstack-cinder | 19:11 | |
*** sgotliv has joined #openstack-cinder | 19:15 | |
*** ebalduf has quit IRC | 19:15 | |
*** Yogi11 has quit IRC | 19:20 | |
anish | thingee: ok, last stubs is gone too | 19:21 |
*** e0ne has joined #openstack-cinder | 19:24 | |
anish | regarding your other comment, that test is just checking the format is correct | 19:25 |
anish | if I understand your comment correctly, you want me to mock the read here, is that correct ? | 19:26 |
nikesh_vedams_ | going to bed :) will wait for futher reviews "https://review.openstack.org/#/c/145778/" | 19:29 |
nikesh_vedams_ | also tomorrow try to upload logs too | 19:30 |
*** asmith_ has joined #openstack-cinder | 19:37 | |
*** annashen has quit IRC | 19:37 | |
Swanson | thingee: do we need a blueprint to add to our unit tests? We have an additional module we want to add to our "suite" of tests. | 19:37 |
*** annashen has joined #openstack-cinder | 19:38 | |
*** diegows has quit IRC | 19:41 | |
thingee | jungleboyj: yup! | 19:41 |
thingee | xyang: I don't know where jdg is =/... but yeah if you can round up another core on it. I'm kind of confused with it atm. | 19:42 |
thingee | xyang: I'm assuming there is a spec update waiting for me on it? | 19:42 |
jungleboyj | thingee: Works for me. I will shoot a note to those guys about how to proceed. | 19:42 |
thingee | anish: if possible yes. | 19:43 |
anish | well | 19:43 |
thingee | jungleboyj: ok, I'll push the spec through. thanks | 19:43 |
anish | thingee: well it's just a test of grep then | 19:43 |
anish | can i just remove it ? | 19:43 |
anish | *re.search | 19:43 |
jungleboyj | thingee: Thank you and hemna for talking this through. | 19:44 |
thingee | Swanson: yes please...it's just for tracking purposes. let me know the url and I'll target it | 19:44 |
xyang | thingee: I'm talking about the over subscription patch. are we talking about the same thing? | 19:44 |
thingee | xyang: yes | 19:44 |
thingee | xyang: at the midcycle meetup, we agreed on spec updates for any changes with the code | 19:45 |
xyang | thingee: oh, that. yes, but I can't update the spec when the code is still changing:) | 19:45 |
thingee | xyang: at this point with winston-d_zZZ's comments, I'm sure the spec is outdated. I once understood the proposal, but have been lost since the midcycle meetup | 19:45 |
xyang | thingee: the design keeps changing | 19:45 |
xyang | thingee: I can't update the spec until the code is merged | 19:46 |
thingee | xyang: haha ok understandable. I'm just lost at this point | 19:46 |
thingee | xyang: so it makes reviewing things hard | 19:46 |
e0ne | alop: thanks for contibution | 19:46 |
xyang | thingee: yes, winston-d made some new suggestions, so I changed code accordingly | 19:46 |
xyang | thingee: we are getting close. | 19:47 |
xyang | thingee: I'll need to log off now to submit code now. will be back | 19:47 |
thingee | xyang: so I would recommend, if there are suggestions to changes, don't change the code first. change the spec first, get agreement, and then change the code. | 19:47 |
thingee | you can change the code if you want in parallel to make sure the suggestions would actually work, but if we don't understand what we're reviewing then we can't review it | 19:48 |
*** xyang has quit IRC | 19:48 | |
*** xyang has joined #openstack-cinder | 19:48 | |
openstackgerrit | xing-yang proposed openstack/cinder: Support over subscription in thin provisioning https://review.openstack.org/142171 | 19:48 |
e0ne | alop: https://review.openstack.org/#/c/146766/18/cinder/db/sqlalchemy/api.py, line #3300: not sure that it work correct:( | 19:49 |
alop | let me test | 19:49 |
*** harlowja is now known as harlowja_away | 19:50 | |
alop | e0ne: I'm just going to drop the interpolation then | 19:50 |
*** dustins_ has quit IRC | 19:51 | |
*** ebalduf has joined #openstack-cinder | 19:52 | |
*** DaveChen has quit IRC | 19:52 | |
e0ne | alop: something like http://paste.openstack.org/show/166975/ shoul works | 19:52 |
*** dustins has joined #openstack-cinder | 19:52 | |
*** bswartz has quit IRC | 19:53 | |
alop | e0ne: ah, even better | 19:53 |
*** EmilienM is now known as EmilienM|afk | 19:53 | |
e0ne | :) | 19:54 |
vilobhmm | e0ne : ping | 19:55 |
*** xyang has quit IRC | 19:55 | |
e0ne | vilobhmm: hi | 19:55 |
*** xyang has joined #openstack-cinder | 19:55 | |
vilobhmm | not sure why the tempest tests are failing http://logs.openstack.org/05/124205/30/check/check-tempest-dsvm-full/4d8aa83/console.html | 19:55 |
vilobhmm | I can’t see anything specific to the changes I have made here | 19:55 |
xyang | thingee: just submitted another patch to address winston-d's comments. I'll update the spec too. | 19:56 |
e0ne | vilobhmm: is it work on your env when you do it manually? | 19:57 |
*** dkingshott has quit IRC | 19:58 | |
vilobhmm | yes i dont see any of these | 19:58 |
openstackgerrit | Abel Lopez proposed openstack/cinder: Purge deleted rows https://review.openstack.org/146766 | 19:58 |
*** dulek has quit IRC | 19:58 | |
*** annegent_ has quit IRC | 19:58 | |
*** diegows has joined #openstack-cinder | 19:58 | |
e0ne | vilobhmm: may be it'a an infra issue. try recheck | 19:59 |
*** ronenkat has joined #openstack-cinder | 20:00 | |
anish | thingee: can I just drop than one test case then ? it's testing open & a re.seach ? everything else is done | 20:01 |
patrickeast | vilobhmm: in the volume service logs there are errors http://logs.openstack.org/05/124205/30/check/check-tempest-dsvm-full/4d8aa83/logs/screen-c-vol.txt.gz?level=TRACE | 20:01 |
*** annashen has quit IRC | 20:02 | |
vilobhmm | patrickeast : i see but when i run run_test in my virtual enn they dont seem to be highlighted..thanks for catching this one | 20:03 |
*** annegent_ has joined #openstack-cinder | 20:04 | |
e0ne | vilobhmm: run_tests runs only unit-tests | 20:04 |
vilobhmm | true.. | 20:04 |
Swanson | thingee:https://blueprints.launchpad.net/cinder/+spec/dell-sc-unittests-update-kilo | 20:05 |
*** Ilja has quit IRC | 20:06 | |
*** dkingshott has joined #openstack-cinder | 20:06 | |
openstackgerrit | Abel Lopez proposed openstack/cinder: Purge deleted rows https://review.openstack.org/146766 | 20:09 |
thingee | Swanson: done...I think if I remember correctly, the code is in review? | 20:10 |
*** rhe00_ has quit IRC | 20:13 | |
*** xyang has quit IRC | 20:14 | |
*** xyang1 has quit IRC | 20:15 | |
*** newbism has quit IRC | 20:15 | |
Swanson | thingee: I think smcginnis put something in review. This is separate from that. I need to get my ducks in a row on this before I push this up. Specifically we want more test time on it. | 20:17 |
*** Yogi11 has joined #openstack-cinder | 20:19 | |
jungleboyj | Do we still have documentation on how to just run the certification test anywhere? | 20:19 |
*** dustins has quit IRC | 20:20 | |
*** bswartz has joined #openstack-cinder | 20:20 | |
*** dustins has joined #openstack-cinder | 20:23 | |
*** Mandell has quit IRC | 20:24 | |
openstackgerrit | Anish Bhatt proposed openstack/cinder: Add support for chiscsi iscsi helper https://review.openstack.org/147758 | 20:25 |
openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: reset-state should warn that it is DB only https://review.openstack.org/152278 | 20:25 |
thingee | jungleboyj: no | 20:25 |
*** rushil has joined #openstack-cinder | 20:28 | |
jungleboyj | thingee: Ok. Wanted to provide the info back to the z Series guys. How do you run it? | 20:30 |
*** PaulCuzner has quit IRC | 20:31 | |
thingee | jbernard: we still aiming for k-3 with this https://review.openstack.org/#/c/135002/ ? | 20:33 |
*** harlowja_away is now known as harlowja | 20:33 | |
thingee | jungleboyj: setup devstack, run https://github.com/openstack-dev/devstack/blob/master/driver_certs/cinder_driver_cert.sh | 20:33 |
thingee | jungleboyj: have you really never done this before with any of your drivers? | 20:34 |
*** Mandell has joined #openstack-cinder | 20:34 | |
jungleboyj | thingee: I really have not. The driver developers have been doing that. | 20:34 |
hemna | doh | 20:35 |
jungleboyj | I get stuck doing a lot more management stuff here than you guys realize. | 20:35 |
thingee | jungleboyj: well, it's a good thing you have internal folks working on all these ci's that can help, right? | 20:35 |
* jungleboyj is afraid I am now being looked at as the bastard child. | 20:35 | |
thingee | jungleboyj: instead of running tempest directly, you would run that script | 20:35 |
jungleboyj | thingee: Yeah. Ok. Thanks. | 20:36 |
*** lcurtis has joined #openstack-cinder | 20:36 | |
jungleboyj | Sorry for the dumb quesiton. | 20:36 |
scottda | xyang: Let me know if you have time to talk about force_detach API | 20:36 |
thingee | jungleboyj: it's not dumb, I just assumed you were it on all those cert tests. My mistake | 20:36 |
* hemna is currently waiting on a tempest run | 20:36 | |
jungleboyj | thingee: No, I don't have access to all the different backends. | 20:36 |
thingee | alright I'll bb, lunch time | 20:36 |
jungleboyj | Have a good lunch. | 20:36 |
thingee | just got my burning man tickets, so I'm in a good mood right now. | 20:37 |
hemna | nap time for me. | 20:37 |
jungleboyj | :-) | 20:37 |
* hemna is home w/ the flu | 20:37 | |
jungleboyj | hemna: Bummer! | 20:37 |
jungleboyj | hemna: Hope you feel better. | 20:37 |
*** rhe00_ has joined #openstack-cinder | 20:38 | |
*** p01s0n has quit IRC | 20:39 | |
markstur_ | thingee, representing cinder at burning man | 20:40 |
jungleboyj | He he, burning block heads. | 20:40 |
*** annashen has joined #openstack-cinder | 20:43 | |
kaufer | jungleboyj: Process question for you, if you have a minute? | 20:44 |
jungleboyj | kaufer: Shoot. | 20:44 |
kaufer | jungleboyj: Spec https://github.com/openstack/cinder-specs/blob/master/specs/kilo/volume-sorting.rst has been approved for kilo and is under review -- the API working group sorting guidelines just merged (https://review.openstack.org/#/c/145579/) so I need to update the code to reflect the new guidelines | 20:44 |
akerr | anyone else's ci system getting caught by mirrors.kernel.org being down off and on all day today? | 20:44 |
patrickeast | akerr: yep, mine is | 20:45 |
kaufer | jungleboyj: My question is, do I need to update the spec to reflect the new sorting syntax and get that re-approved? The spec is no longer aligned with the sorting guidelines. | 20:45 |
akerr | patrickeast: whew! not us :) | 20:46 |
jungleboyj | kaufer: How timely. | 20:46 |
patrickeast | akerr: i asked in -infra and they mentioned that the gate ci’s actually cache the images on the nodes which is why gate doesn’t seem to be broken right now, i’m looking into how to make that work on my ci too | 20:47 |
akerr | patrickeast: we do some cacheing on ours as well, but I guess not enough | 20:47 |
jungleboyj | kaufer: So you are only the second person to kind-of hit this. | 20:47 |
openstackgerrit | Dan Smith proposed openstack/cinder: Add support for chiscsi iscsi helper https://review.openstack.org/147758 | 20:48 |
jungleboyj | kaufer: So, if the spec is no longer accurate, what I think makes the most sense is to push a new patch for your code to the community to meet the new guidelines. Then create a patch for your spec that updates it. | 20:48 |
jungleboyj | kaufer: Push that up for review and reference it from your review that has the latest pagination code included. | 20:49 |
jungleboyj | That way we know you are getting the spec updated. | 20:49 |
jungleboyj | Lets see how that works. Then we will need to formalize a process around how to handle this in the future. | 20:50 |
kaufer | jungleboyj: So I can do both in parallel? | 20:50 |
jungleboyj | thingee: ^^^ | 20:50 |
jungleboyj | I would say so given that this is a change that is coming from the API working group's changes. | 20:50 |
jungleboyj | Just need to make sure they both get merged about the same time. | 20:50 |
*** kvidvans has joined #openstack-cinder | 20:51 | |
kaufer | jungleboyj: Ok, I'll start with the spec update (should be simple) ... I just wanted to make sure that it was still acceptable to make spec changes | 20:51 |
jungleboyj | Yes, if your spec merged you will need to do it as a new patch though. | 20:53 |
jungleboyj | There is no freeze, that I can imagine on making specs accurate. We will need to talk about that in the future though. | 20:54 |
kaufer | jungleboyj: So a patch to the existing file in specs/kilo, right? | 20:54 |
jungleboyj | Yes. | 20:54 |
jungleboyj | Need to drop. Back in a bit. | 20:54 |
*** upenrc has joined #openstack-cinder | 20:55 | |
upenrc | Hi, I am unable to either attach a new cinder volume to a nova instance OR BOOT from Image(Create a new volume).... However, creating an empty volume on my V7000 backend works. | 20:55 |
upenrc | Here's more details abt the issue: https://ask.openstack.org/en/question/60136/volume-attach-fails-error-no-initiators-or-wwpns-supplied/ | 20:56 |
*** nellysmitt has quit IRC | 20:56 | |
smcginnis | jungleboyj: ^^ | 20:56 |
upenrc | pl. check | 20:57 |
*** ronenkat has quit IRC | 20:57 | |
vilobhmm | can we run the tempest test in our dev envoirnment so as to be on the safer side before jenkins complains about tempest errors harlowja pointed me to http://www.rushiagr.com/blog/2013/03/20/openstack-tempest-quick-start-with-devstack/ …looks quite old is there something new that can be tried | 20:58 |
vilobhmm | although this is a excellent writeup just that it was written in 2013 | 20:58 |
*** jungleboyj has quit IRC | 20:59 | |
upenrc | vilobhmm - responding to me? | 21:02 |
vilobhmm | upenrc : sorry didnt got you | 21:02 |
upenrc | sorry i tht u were responding to me | 21:03 |
upenrc | Here's more details abt the issue: https://ask.openstack.org/en/question/60136/volume-attach-fails-error-no-initiators-or-wwpns-supplied/ | 21:03 |
*** jnrao has joined #openstack-cinder | 21:05 | |
*** kvidvans has quit IRC | 21:06 | |
*** kvidvans has joined #openstack-cinder | 21:06 | |
*** kvidvans has quit IRC | 21:06 | |
*** kvidvans has joined #openstack-cinder | 21:06 | |
thingee | kaufer: yes, spec update would be great. | 21:07 |
*** kvidvans has quit IRC | 21:07 | |
thingee | kaufer: did things get approved in openstack-specs and api-wg? | 21:07 |
thingee | kaufer: juggling a bunch of things atm sorry | 21:07 |
*** kvidvans has joined #openstack-cinder | 21:07 | |
vilobhmm | upenrc : what do you see in your cinder-api logs ? I havent tried with lvm but played around with other storage i can see this when nova-api can’t talk to cinder-api | 21:07 |
kaufer | thingee: Approved in the API-WG for the server sorting guidelines, the CLI openstack-spec is still under review | 21:07 |
*** annegent_ has quit IRC | 21:10 | |
upenrc | vilobhmm i only have cinder volume log posted in the linkk | 21:10 |
upenrc | pl. be specific | 21:10 |
*** EduardBiceriMate is now known as EduardBiceri_afk | 21:11 | |
upenrc | oh checking api.log | 21:11 |
vilobhmm | yup 2015-02-04 14:12:20.962 21245 ERROR cinder.volume.driver [req-a9780d29-e87f-46af-9f28-7f27a55c7b17 2c37f6bd55b8402bae2ab04402562e4f 4cf6d19846284f4bb5cb3731395de51b - - -] Unable to fetch connection information from backend: create_host: No initiators or wwpns supplied. | 21:12 |
upenrc | vilobhmm - here: 402bae2ab04402562e4f 4cf6d19846284f4bb5cb3731395de51b - - -] Connected to AMQP server on 10.6.113.223:5671 2015-02-02 13:00:49.523 3786 ERROR cinder.api.middleware.fault [req-a2ab13c4-2b92-40df-95e3-41118f5dc8a2 2c37f6bd55b8402bae2ab04402562e4f 4cf6d19846284f4bb5cb3731395de51b - - -] Caught error: Volume driver not ready. | 21:12 |
upenrc | ^^error above | 21:12 |
vilobhmm | have you made proper conf changes to your nova.conf, cinder.conf to use LVM ? | 21:12 |
vilobhmm | some config is missing | 21:13 |
upenrc | cinder.conf - yes - empty volume creation works | 21:13 |
upenrc | havent touched nova.conf | 21:13 |
upenrc | what properties to edit in nova.conf | 21:14 |
*** Yogi11 has quit IRC | 21:14 | |
*** dustins has quit IRC | 21:14 | |
upenrc | ##### VOLUMES ##### # iscsi target user-land tool to use iscsi_helper=ietadm volume_api_class=nova.volume.cinder.API # Region name of this node (string value) os_region_name=RegionOne | 21:15 |
*** karimb has joined #openstack-cinder | 21:19 | |
*** jcru has quit IRC | 21:19 | |
upenrc | ^ error | 21:20 |
*** Adriano_ has quit IRC | 21:22 | |
*** Adriano__ has quit IRC | 21:22 | |
openstackgerrit | Vilobh Meshram proposed openstack/cinder: Introducing micro_states for create workflow https://review.openstack.org/124205 | 21:23 |
*** foobloo2 has joined #openstack-cinder | 21:24 | |
*** Yogi1 has joined #openstack-cinder | 21:24 | |
foobloo2 | is there a blueprint for cinder change block tracking? | 21:24 |
*** dustins has joined #openstack-cinder | 21:24 | |
*** dustins has quit IRC | 21:25 | |
*** Yogi1 has quit IRC | 21:25 | |
*** markstur_ has quit IRC | 21:26 | |
openstackgerrit | Vilobh Meshram proposed openstack/cinder: Introducing micro_states for create workflow https://review.openstack.org/124205 | 21:27 |
harlowja | vilobhmm u might also have luck in #openstack-infra about the tempest question | 21:28 |
vilobhmm | sure | 21:28 |
thingee | foobloo2: what? | 21:29 |
*** markstur has joined #openstack-cinder | 21:29 | |
*** dustins has joined #openstack-cinder | 21:31 | |
foobloo2 | thingee, a certain storage company is saying something to the effect that there is some upstream effort re change block tracking. Not sure if they are implying cinder of libvirt itself. | 21:34 |
foobloo2 | or libvirt** | 21:35 |
*** jungleboyj has joined #openstack-cinder | 21:35 | |
*** ronis has quit IRC | 21:41 | |
*** sgotliv has quit IRC | 21:41 | |
foobloo2 | they're saying something about leveraging the cinder backup api to track changed blocks to swift | 21:41 |
foobloo2 | err track changed blocks and push them to swift | 21:44 |
*** julim has quit IRC | 21:45 | |
*** EmilienM|afk is now known as EmilienM | 21:45 | |
guitarza1 | it's been hand waved about, but is there a serious proposal somewhere? | 21:45 |
foobloo2 | that's what I am trying to figure out...there is vague reference to it in raksha, but that's all i've been able to find | 21:46 |
thingee | tbarron, DuncanT ^ | 21:46 |
thingee | foobloo2: those are the people you'll want to talk to | 21:46 |
foobloo2 | thingee, thanks | 21:46 |
*** foobloo2 is now known as foobloo | 21:47 | |
openstackgerrit | Anish Bhatt proposed openstack/cinder: Add support for chiscsi iscsi helper https://review.openstack.org/147758 | 21:54 |
anish | thingee: ^ final | 21:54 |
*** thangp has quit IRC | 21:57 | |
openstackgerrit | Nagendra Rao Jaladanki proposed openstack/cinder-specs: Spec for Brocade Zone Driver Friendly Zone names https://review.openstack.org/144384 | 21:58 |
openstackgerrit | Steven Kaufer proposed openstack/cinder-specs: Cinder Pagination Sorting Enhancements https://review.openstack.org/153061 | 21:59 |
*** upenrc has quit IRC | 21:59 | |
*** enterprisedc has quit IRC | 22:00 | |
jnrao | Hi all, we posted any update to BP: https://review.openstack.org/#/c/144384/ | 22:00 |
jnrao | Can you please take a look at it? | 22:00 |
*** karimb has quit IRC | 22:02 | |
*** annegent_ has joined #openstack-cinder | 22:04 | |
*** enterprisedc has joined #openstack-cinder | 22:04 | |
*** annegent_ has quit IRC | 22:05 | |
kaufer | jungleboyj: Thx for the guidance on updating the spec, it's up: https://review.openstack.org/153061 | 22:06 |
jungleboyj | kaufer: You are welcome! | 22:08 |
*** bswartz has quit IRC | 22:13 | |
*** dustins has quit IRC | 22:15 | |
*** akerr has quit IRC | 22:17 | |
*** jecarey has quit IRC | 22:20 | |
*** lcurtis has quit IRC | 22:20 | |
*** Lee1092 has quit IRC | 22:20 | |
*** kaufer has quit IRC | 22:20 | |
openstackgerrit | Tom Barron proposed openstack/cinder: Implement Cinder Manage/Unmanage APIs in NFS driver https://review.openstack.org/138125 | 22:21 |
tbarron | thingee: ^^^^^^ fixed that unit test | 22:21 |
tbarron | thingee: but let's see what jenkins says | 22:22 |
thingee | tbarron: thanks | 22:22 |
thingee | anish: thanks | 22:22 |
*** dulek has joined #openstack-cinder | 22:24 | |
*** jcru has joined #openstack-cinder | 22:25 | |
*** tellesnobrega_ has joined #openstack-cinder | 22:27 | |
e0ne | thingee: just fyi: i tested dulek's patches for persistance workflow. it work with some limitation. it resume flow on a new host works too | 22:27 |
*** jcru has joined #openstack-cinder | 22:28 | |
*** ebalduf has quit IRC | 22:28 | |
*** dulek has quit IRC | 22:29 | |
e0ne | thingee, dulek: i tested scheduler and manager flows | 22:29 |
*** tbarron has quit IRC | 22:36 | |
*** mriedem has quit IRC | 22:41 | |
*** vilobhmm has quit IRC | 22:44 | |
thingee | e0ne: excellent! | 22:44 |
thingee | excited to check things out | 22:44 |
thingee | I gotta do an interview in a few, but I'll bb | 22:44 |
*** vilobhmm has joined #openstack-cinder | 22:45 | |
*** MentalRay has quit IRC | 22:45 | |
e0ne | thingee: i'll publish comments to dulek, there some broken code:( | 22:45 |
e0ne | thingee: and will send review-request for volume flow with some notes tomorrow before your morning | 22:46 |
e0ne | *manager flow | 22:46 |
*** lpetrut has quit IRC | 22:53 | |
*** nellysmitt has joined #openstack-cinder | 22:57 | |
* e0ne tried to work with not latest patchset from gerrit :( | 22:57 | |
*** lpetrut has joined #openstack-cinder | 22:58 | |
*** lpetrut has quit IRC | 22:59 | |
*** jaypipes has quit IRC | 23:00 | |
*** casusbelli has joined #openstack-cinder | 23:01 | |
*** nellysmitt has quit IRC | 23:01 | |
*** Mandell has quit IRC | 23:06 | |
vilobhmm | e0ne, thingee : can you please review this https://review.openstack.org/#/c/124205/ | 23:07 |
vilobhmm | passed jenkins | 23:07 |
vilobhmm | xyang : ^^ | 23:08 |
*** Mandell has joined #openstack-cinder | 23:15 | |
*** VManiac16 has quit IRC | 23:19 | |
*** annashen has quit IRC | 23:22 | |
*** e0ne has quit IRC | 23:24 | |
*** primechuck has quit IRC | 23:26 | |
*** primechuck has joined #openstack-cinder | 23:27 | |
*** primechuck has quit IRC | 23:27 | |
*** ebalduf has joined #openstack-cinder | 23:29 | |
*** ebalduf has quit IRC | 23:34 | |
*** dulek has joined #openstack-cinder | 23:39 | |
*** rmesta has quit IRC | 23:40 | |
vilobhmm | thingee : ping | 23:46 |
*** bswartz has joined #openstack-cinder | 23:47 | |
anish | smcginnis: re : https://review.openstack.org/#/c/152687/ can you confirm setup.cfg ? | 23:52 |
*** annegent_ has joined #openstack-cinder | 23:55 | |
*** markvoelker has quit IRC | 23:56 | |
*** lan has quit IRC | 23:57 | |
smcginnis | anish: You should probably check with Captian Oslo. :) | 23:57 |
smcginnis | jungleboyj: ^^ | 23:57 |
*** lan has joined #openstack-cinder | 23:57 | |
jungleboyj | Did someone way Captain Oslo !?! | 23:58 |
anish | I was gonna say jay of the jungle | 23:58 |
anish | dropped all openstack/common file re : https://review.openstack.org/#/c/152687/ , but I am not sure if setup.cfg is supposed to be modified | 23:58 |
anish | *files | 23:58 |
*** lcurtis has joined #openstack-cinder | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!