Thursday, 2015-02-19

*** jwcroppe has quit IRC00:00
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Update the documentation for os-brick  https://review.openstack.org/15720200:05
*** lcurtis has quit IRC00:08
hemnathingee, jaypipes ping ?00:09
*** jordandh has quit IRC00:09
*** patrickeast has quit IRC00:10
openstackgerritJay Bryant proposed openstack/cinder-specs: Fix the volume replication spec  https://review.openstack.org/15273500:14
*** Sam-I-Am has left #openstack-cinder00:14
*** jordandh has joined #openstack-cinder00:15
thingeehemna: i'm here00:16
thingeenot for long though...00:17
hemnaI'm not sure jaypipes is around00:17
* thingee sitting at a bar with a laptop, wtf00:17
hemnabut he and I had a chat this morning about multiattach00:17
hemnabasically the nova side isn't landing in K00:17
hemnabut we would still like to get the cinder side to land if possible00:17
thingeedo drivers need to do anything for it to be useful?00:17
hemnano00:17
sigmavirus24thingee: sitting at a bar on a laptop is perfectly normal00:17
thingeesigmavirus24: not in LA. maybe in the bay area, but no LA. People are looking at me weird00:18
hemnanova's overhead/schedule/beuracracy makes it very difficult to get cross project stuff to land00:18
thingeehemna: seems fine to me then00:18
sigmavirus24thingee: oh I didn't say people wouldn't give you weird looks, just that it's a normal thing to do in general00:18
sigmavirus24If it weren't, I'd feel pretty silly00:18
sigmavirus24=P00:18
thingee:)00:19
hemnahehe00:19
akerrdepends if you're also drinking or not00:19
*** nellysmitt has joined #openstack-cinder00:19
thingeeakerr: I am. but want to order a second one.00:19
*** markvoelker has quit IRC00:19
thingeeand maybe socialize a bit00:19
hemnaok, so I'll just continue to get reviews on it then and see if we can get it in K00:19
hemnaand nova will land in L00:19
akerrbe a bit weirder if you just walked in and set up shop00:19
thingeehemna: sounds fine to me00:19
thingeeshould do it during scrum meetings. "dunno sounds like an 8 right now, but maybe after a couple more of these that can be a 4?"00:20
akerrin my experience 4s usually become 8s after a few drinks00:21
*** IanGovett has quit IRC00:22
hemnathingee, ok thanks man.00:22
thingeebtw people https://twitter.com/Thingee/status/56816918823951155200:22
sigmavirus24thingee: socialize with us! =P00:22
hemnathingee, nice!00:23
thingeejgriffith, jungleboyj, xyang, smcginnis, xyang ^00:24
*** nellysmitt has quit IRC00:24
hemnathingee, if you can review https://review.openstack.org/#/c/157202 for me, then I think we might be ready for a first drop of brick00:24
hemnawhich would enable me to update requirements.txt and get the cinder brick patch out of WIP00:25
hemnaI'm not sure how the releases and versioning are done00:25
hemnaI might have gone overboard on the docs, dunno.00:25
thingeehemna: ok, either tonight or tomorrow, is that fine?00:26
hemnayup no worries00:26
hemnathanks for the help00:26
hemnaI just want to try and get this in for K if possible, so we can move on.00:26
thingeehemna: I'm a bit done for today :) ... want to have some drinks and talk to some folks here00:26
openstackgerritThang Pham proposed openstack/cinder: WIP: Backup object  https://review.openstack.org/15708500:26
thingeeand dinner soon..I've barely ate today00:27
*** fischerw has quit IRC00:27
jungleboyjthingee: I like it!00:27
jungleboyjthingee: That sounds like a good plan.00:28
*** jkremer has quit IRC00:35
hemnajungleboyj, are you guys freezing over out there?00:37
*** thingee has quit IRC00:37
*** hemna is now known as hemnafk00:38
*** Mandell has joined #openstack-cinder00:39
*** david-lyle is now known as david-lyle_afk00:41
*** ebalduf has joined #openstack-cinder00:48
*** markvoelker has joined #openstack-cinder00:52
*** rmesta has quit IRC00:53
*** ebalduf has quit IRC00:53
*** rmesta has joined #openstack-cinder00:54
*** rmesta has quit IRC00:54
*** theanalyst has quit IRC00:55
*** annashen has quit IRC00:56
*** dannywilson has quit IRC00:56
*** theanalyst has joined #openstack-cinder00:58
*** markvoelker has quit IRC00:59
*** sigmavirus24 is now known as sigmavirus24_awa01:02
*** vishy has quit IRC01:07
*** vishy has joined #openstack-cinder01:08
*** patrickeast has joined #openstack-cinder01:10
*** tsekiyam_ has joined #openstack-cinder01:16
*** Apoorva_ has quit IRC01:18
*** Apoorva has joined #openstack-cinder01:19
*** tsekiyama has quit IRC01:20
*** tsekiyam_ has quit IRC01:21
*** mtanino has quit IRC01:25
*** Yogi11 has quit IRC01:28
*** markvoelker has joined #openstack-cinder01:29
*** jwcroppe has joined #openstack-cinder01:33
*** jgravel has quit IRC01:34
*** 64MABZ0GB is now known as strictlyb01:36
*** EmilienM is now known as EmilienM|afk01:36
*** garthb has quit IRC01:37
*** garthb_ has quit IRC01:37
*** Yogi1 has joined #openstack-cinder01:40
*** IanGovett has joined #openstack-cinder01:47
*** duyt1001 has joined #openstack-cinder01:50
*** IanGovett has quit IRC01:54
*** patrickeast has quit IRC01:54
*** devlaps has quit IRC01:55
*** jwcroppe has quit IRC01:57
*** jwcroppe has joined #openstack-cinder01:57
*** jwcroppe has quit IRC02:02
*** scottda_ has joined #openstack-cinder02:02
*** vilobhmm has quit IRC02:03
*** vilobhmm2 has quit IRC02:03
*** mdbooth has quit IRC02:06
*** jwcroppe has joined #openstack-cinder02:07
*** jwcroppe has quit IRC02:08
*** krtaylor has joined #openstack-cinder02:09
*** jwcroppe has joined #openstack-cinder02:09
*** scottda_ has quit IRC02:09
*** mdbooth has joined #openstack-cinder02:11
*** jwcroppe has quit IRC02:13
*** leeantho has quit IRC02:14
*** scottda_ has joined #openstack-cinder02:17
*** ebalduf has joined #openstack-cinder02:19
*** nellysmitt has joined #openstack-cinder02:20
*** ebalduf has quit IRC02:24
*** Mandell has quit IRC02:24
*** nellysmitt has quit IRC02:24
*** Mandell has joined #openstack-cinder02:25
*** duyt1001 has left #openstack-cinder02:27
*** _cjones_ has quit IRC02:30
openstackgerritGloria Gu proposed openstack/cinder: HP 3par driver filter and evaluator function  https://review.openstack.org/15135302:35
*** Apoorva_ has joined #openstack-cinder02:35
*** ybathia has quit IRC02:36
*** Yogi1 has quit IRC02:37
openstackgerritGloria Gu proposed openstack/cinder: HP lefthand driver filter and evalautor function  https://review.openstack.org/15723702:38
*** Apoorva has quit IRC02:38
*** Apoorva_ has quit IRC02:39
*** scottda_ has quit IRC02:43
*** Yogi1 has joined #openstack-cinder02:47
*** annashen has joined #openstack-cinder02:48
*** kaisers1 has joined #openstack-cinder02:51
*** mriedem has joined #openstack-cinder02:53
*** kaisers has quit IRC02:53
*** jecarey has joined #openstack-cinder02:54
*** annashen has quit IRC03:08
*** Yogi1 has quit IRC03:13
*** harlowja_ is now known as harlowja_away03:15
*** jecarey has quit IRC03:28
*** jecarey has joined #openstack-cinder03:28
*** ebalduf has joined #openstack-cinder03:35
*** scottda_ has joined #openstack-cinder03:37
*** scottda_ has joined #openstack-cinder03:38
*** ebalduf has quit IRC03:39
*** scottda_ has quit IRC03:48
*** kaufer has joined #openstack-cinder03:48
*** scottda_ has joined #openstack-cinder03:50
*** Mandell has quit IRC03:58
openstackgerritJeegn Chen proposed openstack/cinder: Attach/detach batch processing in VNX driver  https://review.openstack.org/15435003:59
*** akerr is now known as akerr_away04:01
jungleboyjhemnafk: It is only -4.  ;-)04:01
jungleboyjhemnafk: Still not wearing my winter coat.04:01
*** MentalRay has joined #openstack-cinder04:13
*** annashen has joined #openstack-cinder04:16
*** nellysmitt has joined #openstack-cinder04:21
*** nellysmitt has quit IRC04:25
*** setmason has quit IRC04:27
*** coolsvap_ is now known as coolsvap04:28
*** btran has quit IRC04:28
xyangthingee: That looks good!04:31
*** jwcroppe has joined #openstack-cinder04:32
*** Mandell has joined #openstack-cinder04:34
*** jecarey has quit IRC04:35
*** changbl has joined #openstack-cinder04:38
*** tbarron has quit IRC04:40
*** annashen has quit IRC04:41
*** markvoelker has quit IRC04:42
*** markvoelker has joined #openstack-cinder04:42
*** Mandell has quit IRC04:42
*** Mandell has joined #openstack-cinder04:43
*** annashen has joined #openstack-cinder04:45
*** annashen has quit IRC04:46
*** markvoelker has quit IRC04:46
*** annashen has joined #openstack-cinder04:47
*** BharatK has joined #openstack-cinder04:52
openstackgerritSteven Kaufer proposed openstack/cinder: Generic filter support for volume queries  https://review.openstack.org/15534304:55
openstackgerritSteven Kaufer proposed openstack/cinder: Replication status periodic task optimization  https://review.openstack.org/15467304:55
*** _cjones_ has joined #openstack-cinder04:59
*** kaufer has quit IRC04:59
*** asselin_ has joined #openstack-cinder05:00
*** scottda_ has quit IRC05:00
*** annashen has quit IRC05:00
*** murugan has joined #openstack-cinder05:02
*** asselin_ has quit IRC05:02
*** mriedem has quit IRC05:05
*** coolsvap is now known as coolsvap_05:05
*** mwichmann has quit IRC05:07
*** mriedem has joined #openstack-cinder05:10
*** mriedem has left #openstack-cinder05:11
*** markvoelker has joined #openstack-cinder05:13
*** _cjones_ has quit IRC05:15
*** coolsvap_ is now known as coolsvap05:15
*** _cjones_ has joined #openstack-cinder05:15
*** markvoelker has quit IRC05:17
*** I has joined #openstack-cinder05:18
*** I is now known as Guest9853705:18
*** pdeore_ has joined #openstack-cinder05:20
*** Guest98537 has quit IRC05:22
pdeore_Hello, jenkins is failing for one of my patch05:26
pdeore_https://review.openstack.org/#/c/156936/05:26
pdeore_I have ensured that tests are completing successfully in my local environment05:27
pdeore_I have also verified python26 and python27 jenkins logs05:27
pdeore_http://logs.openstack.org/36/156936/1/check/gate-cinder-python26/1afcea7/console.html05:27
pdeore_The error is like, "This test has failed because it attempted to execute commands"05:28
pdeore_Can anyone please help me to understand the reason behind this issue?05:29
*** sandywalsh has joined #openstack-cinder05:38
*** sandywalsh_ has quit IRC05:40
*** Ilja has joined #openstack-cinder05:46
*** deepakcs has joined #openstack-cinder05:47
*** zigo has quit IRC05:54
*** zigo has joined #openstack-cinder05:55
*** pradipta has joined #openstack-cinder05:57
*** _cjones_ has quit IRC06:00
openstackgerritJoe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager  https://review.openstack.org/15726506:02
*** Mandell has quit IRC06:04
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex  https://review.openstack.org/15240106:09
*** cnesa1 has quit IRC06:10
*** annashen has joined #openstack-cinder06:13
*** markvoelker has joined #openstack-cinder06:13
*** markvoelker has quit IRC06:18
*** Mandell has joined #openstack-cinder06:19
*** nellysmitt has joined #openstack-cinder06:22
*** ankit_ag has joined #openstack-cinder06:23
*** nellysmitt has quit IRC06:26
*** ronenkat has quit IRC06:27
*** ronenkat__ has quit IRC06:27
*** ronenkat has joined #openstack-cinder06:28
*** ronenkat_ has joined #openstack-cinder06:29
*** sgotliv has joined #openstack-cinder06:32
*** dannywilson has joined #openstack-cinder06:34
*** pradeep_s has left #openstack-cinder06:41
*** dannywilson has quit IRC06:41
*** ronenkat_ has quit IRC06:42
*** ronenkat has quit IRC06:42
*** dannywilson has joined #openstack-cinder06:42
*** nkrinner has joined #openstack-cinder06:46
*** atmark has quit IRC06:50
*** Lee1092 has joined #openstack-cinder06:51
*** harlowja_at_home has joined #openstack-cinder06:52
*** Mandell has quit IRC06:52
*** Mandell has joined #openstack-cinder06:58
*** bill_az has joined #openstack-cinder07:05
*** bill_ibm__ has quit IRC07:05
*** _cjones_ has joined #openstack-cinder07:07
*** afazekas has joined #openstack-cinder07:08
*** Ilja has quit IRC07:08
*** Ilja has joined #openstack-cinder07:09
*** harlowja_at_home has quit IRC07:13
*** Ilja has quit IRC07:14
*** markvoelker has joined #openstack-cinder07:14
*** sgotliv has quit IRC07:16
*** chlong has quit IRC07:19
*** markvoelker has quit IRC07:19
*** ho has quit IRC07:26
*** sgotliv has joined #openstack-cinder07:26
*** TobiasE has joined #openstack-cinder07:26
*** nshaikh has joined #openstack-cinder07:27
*** anshul has joined #openstack-cinder07:27
*** annashen has quit IRC07:27
openstackgerritJoe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager  https://review.openstack.org/15726507:29
*** _cjones_ has quit IRC07:31
*** MentalRay has quit IRC07:36
*** rluediger has joined #openstack-cinder07:59
rluedigergood morning all together DuncanT are you already online?08:00
*** annashen has joined #openstack-cinder08:01
*** harlowja_away has quit IRC08:03
*** annashen has quit IRC08:05
*** ronenkat has joined #openstack-cinder08:06
*** ronenkat_ has joined #openstack-cinder08:06
*** cnesa has joined #openstack-cinder08:09
*** ronenkat has quit IRC08:13
*** ronenkat_ has quit IRC08:14
*** markvoelker has joined #openstack-cinder08:15
*** markvoelker has quit IRC08:20
*** nellysmitt has joined #openstack-cinder08:22
*** atmark1 has joined #openstack-cinder08:23
*** nellysmitt has quit IRC08:27
*** Ilja has joined #openstack-cinder08:31
*** Ilja has quit IRC08:31
*** murugan has quit IRC08:34
*** ronis has joined #openstack-cinder08:34
*** alecv has joined #openstack-cinder08:35
*** nellysmitt has joined #openstack-cinder08:36
*** pradipta has quit IRC08:37
*** jistr has joined #openstack-cinder08:42
*** dulek has joined #openstack-cinder08:50
*** rushiagr_away is now known as rushiagr08:52
*** jordanP has joined #openstack-cinder08:53
*** xyang1 has quit IRC09:03
*** markvoelker has joined #openstack-cinder09:16
*** dannywilson has quit IRC09:18
*** ndipanov has quit IRC09:21
*** markvoelker has quit IRC09:21
*** abhishekk has joined #openstack-cinder09:25
abhishekkhi all, jenkins is failing for python26 and 27 on stable/juno patch with error "This test has failed because it attempted to execute commands with sudo", >> 2015-02-19 08:57:14.452 | > Feb 19 08:56:31 bare-centos6-1424268955 sudo:  jenkins : command not allowed ; TTY=unknown ; PWD=/home/jenkins/workspace/gate-cinder-python26 ; USER=root ; COMMAND=/home/jenkins/workspace/gate-cinder-python26/.tox/py26/bin/cinder-rootwrap /etc/cinder/rootwrap.c09:27
abhishekkonf dd count=0 if=/dev/mapper/cinder--volumes-clone--snap--ddecf645--750a--404a--a672--d3e258dd39ab of=/dev/mapper/cinder--volumes-volume--ddecf645--750a--404a--a672--d3e258dd39ab oflag=direct09:27
abhishekkReference: https://jenkins01.openstack.org/job/gate-cinder-python26/7995/console09:27
abhishekkcan any one help me to understand the cause of failure?09:28
*** enterprisedc has quit IRC09:28
*** ndipanov has joined #openstack-cinder09:29
*** enterprisedc has joined #openstack-cinder09:30
DuncanTabhishekk: You need to add the cinder user to your /etc/sudoers file , with no password prompting. Google around to find the advised way of doing this09:53
DuncanTOh, wait, unit tests?09:53
DuncanTThe execute call should be mocked. Not sure what is going on there, sorry09:54
DuncanTrluediger: Good morning09:54
abhishekkDuncanT: thank you, unittests are completing successfully in local environment, so on jenkins09:55
abhishekkDuncanT: Not sure why it's giving error later while running the command with sudo09:56
DuncanTabhishekk: Is the user you're running them as locally in sudoers09:59
abhishekkDuncanT: yes10:00
DuncanTSo I suspect jenkins isn't, and one of the tests is missing a mock10:00
rluedigerDuncanT: I've tested yesterday evening and observed the following. If i stop all cinder services on all nodes excepting cinder-api and created one new volume i found one message for the scheduler in rabbitmq as expected. when cinder-scheduler comes up with cinder-volume down the message would be consumed and cinder-scheduler throws an error taht no wihed host can be found. if the cinder-volume is up and running the message will also be c10:01
rluedigeronsumed by cinder-scheduler but nothinh happens on the storage node10:01
DuncanTabhishekk: Adding jenkins to sudoers is the quick fix, fixing the test would be better10:01
rluedigerDuncanT: can you advise how to get more info from the cinder-scheduler what he is doing?10:02
abhishekkDuncanT: Thanks for the input, I will check for it10:02
DuncanTrluediger: I have absolutely no idea at all what could cause that! I suggest doing and asking on #openstack, where there are people with much more deploy experience. That sounds most odd to me, and I'd love to know the answer once you've figured it out10:02
DuncanTrluediger: Sorry I can't be more help, but I don't even know where to start with rabbit misbehaviour10:03
*** karimb has joined #openstack-cinder10:03
rluedigerDuncanT: ok. thx for your help so far I'll try to get help @ the #openstack channel10:04
DuncanTrluediger: Good luck10:06
*** Mandell has quit IRC10:06
*** yuriy_n17 has joined #openstack-cinder10:07
openstackgerritJoe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager  https://review.openstack.org/15726510:09
abhishekkDuncanT: This patch is a backport to stable/juno, I haven't got this error on master, so I am more confused about this10:09
*** Lee1092 has quit IRC10:10
DuncanTabhishekk: Can you link me to the patch? It is possible there are some standard mocks on master that aren't on stable, I guess?10:10
abhishekkDuncanT: https://review.openstack.org/#/c/156936/110:11
*** markvoelker has joined #openstack-cinder10:17
*** e0ne has joined #openstack-cinder10:21
*** markvoelker has quit IRC10:22
*** briancline has quit IRC10:22
*** briancline has joined #openstack-cinder10:23
jordanPhi guys. We've just unsilenced our Scality CI. It should review all new patchset pushed to openstack/cinder. We hope it will work as expected :)10:28
*** _cjones_ has joined #openstack-cinder10:31
*** _cjones_ has quit IRC10:36
*** e0ne is now known as e0ne_10:39
*** e0ne_ is now known as e0ne10:39
openstackgerritJoe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager  https://review.openstack.org/15726510:42
openstackgerritJoe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager  https://review.openstack.org/15726510:44
*** e0ne is now known as e0ne_10:53
*** e0ne_ is now known as e0ne10:54
*** Swanson has quit IRC11:02
*** smcginnis has quit IRC11:03
*** jseiler__ has quit IRC11:03
*** aix has quit IRC11:04
*** ndipanov has quit IRC11:04
*** aix has joined #openstack-cinder11:05
*** jseiler has joined #openstack-cinder11:09
*** smcginnis has joined #openstack-cinder11:09
*** Swanson has joined #openstack-cinder11:10
*** Ilja has joined #openstack-cinder11:10
*** markvoelker has joined #openstack-cinder11:18
*** ndipanov has joined #openstack-cinder11:18
*** nlevinki has joined #openstack-cinder11:22
*** markvoelker has quit IRC11:23
*** dulek has quit IRC11:29
*** dulek has joined #openstack-cinder11:30
*** e0ne is now known as e0ne_11:34
*** deepakcs has quit IRC11:38
*** pschaef has joined #openstack-cinder11:40
*** chlong has joined #openstack-cinder11:41
*** ganso_ has joined #openstack-cinder11:47
*** marcusvrn1 has quit IRC11:49
*** marcusvrn has joined #openstack-cinder11:50
*** alexpilotti has joined #openstack-cinder11:51
*** marcusvrn1 has joined #openstack-cinder11:52
*** rushiagr is now known as rushiagr_away11:54
*** marcusvrn has quit IRC11:56
*** rluediger has quit IRC11:59
*** rluediger has joined #openstack-cinder12:02
*** Lee1092 has joined #openstack-cinder12:04
*** agentle_ has quit IRC12:06
*** diegows has joined #openstack-cinder12:08
*** nlevinki_ has joined #openstack-cinder12:10
*** e0ne_ is now known as e0ne12:10
*** nlevinki has quit IRC12:13
*** ndipanov has quit IRC12:13
*** ndipanov has joined #openstack-cinder12:14
*** IanGovett has joined #openstack-cinder12:15
*** dannywilson has joined #openstack-cinder12:19
*** markvoelker has joined #openstack-cinder12:19
*** ndipanov has quit IRC12:20
*** dannywilson has quit IRC12:23
*** markvoelker has quit IRC12:24
*** jseiler has quit IRC12:25
*** smcginnis has quit IRC12:25
openstackgerritMichal Dulko proposed openstack/cinder: Backup object  https://review.openstack.org/15708512:25
*** Swanson has quit IRC12:25
*** TobiasE has left #openstack-cinder12:30
*** ndipanov has joined #openstack-cinder12:32
*** nlevinki_ has quit IRC12:37
*** Yogi1 has joined #openstack-cinder12:43
openstackgerritnikeshmahalka proposed openstack/cinder: Add CHAP persistence to SCST target helper  https://review.openstack.org/15735612:47
*** avishay has joined #openstack-cinder12:57
*** akerr_away is now known as akerr12:57
*** EmilienM|afk is now known as EmilienM12:58
*** akerr has quit IRC12:58
*** marcusvrn has joined #openstack-cinder13:01
*** Yogi1 has quit IRC13:01
*** ankit_ag has quit IRC13:02
*** markvoelker has joined #openstack-cinder13:02
*** marcusvrn1 has quit IRC13:04
*** alexpilotti has quit IRC13:05
*** BharatK has quit IRC13:12
*** Yogi1 has joined #openstack-cinder13:17
*** cnesa has quit IRC13:23
DuncanTabhishekk: Off the top of my head, I can't see what would cause an exec in that patch. I suggest running it locally as a user that doesn't have sudo, and seeing if that creates the same issue as jenkins is seeing. If so, start disabling parts of the test until you find a culprit13:23
*** cnesa has joined #openstack-cinder13:24
*** bswartz has quit IRC13:27
*** cnesa has quit IRC13:28
abhishekkDuncanT: Thanks alot, I will do the same13:28
DuncanTabhishekk: If you find something, please do let me know. Thanks.13:29
abhishekkDuncanT: Sure13:29
*** aix has quit IRC13:29
*** rluediger has quit IRC13:33
*** akerr has joined #openstack-cinder13:35
*** Yogi1 has quit IRC13:36
*** jistr_ has joined #openstack-cinder13:37
*** Ilja has quit IRC13:37
*** Ilja has joined #openstack-cinder13:38
*** jistr has quit IRC13:39
*** bkopilov has joined #openstack-cinder13:40
*** haomaiw__ has quit IRC13:41
*** xyang has quit IRC13:41
*** smcginnis has joined #openstack-cinder13:42
*** markvoelker_ has joined #openstack-cinder13:42
*** markvoelker has quit IRC13:42
*** briancline has quit IRC13:48
*** jistr_ is now known as jistr13:48
*** briancline has joined #openstack-cinder13:49
*** timcl has joined #openstack-cinder13:49
*** sigmavirus24_awa is now known as sigmavirus2413:52
*** Adriano_ has quit IRC13:52
*** kaufer has joined #openstack-cinder13:54
*** mkerrin has quit IRC13:55
*** xyang has joined #openstack-cinder13:57
*** BharatK has joined #openstack-cinder13:57
*** pschaef has quit IRC13:58
*** aix has joined #openstack-cinder14:01
*** bswartz has joined #openstack-cinder14:01
*** Yogi1 has joined #openstack-cinder14:03
*** kaisers1 has quit IRC14:04
*** Mandell has joined #openstack-cinder14:07
*** BharatK has quit IRC14:08
*** kaisers has joined #openstack-cinder14:11
*** Mandell has quit IRC14:12
*** nlevinki has joined #openstack-cinder14:14
*** xyang1 has joined #openstack-cinder14:15
*** rushil has joined #openstack-cinder14:18
*** dustins has joined #openstack-cinder14:20
*** BharatK has joined #openstack-cinder14:20
*** tbarron has joined #openstack-cinder14:21
*** jungleboyj has quit IRC14:27
*** mriedem has joined #openstack-cinder14:27
*** mwichmann_ has joined #openstack-cinder14:28
*** Yogi1 has quit IRC14:31
*** karimb has quit IRC14:32
*** EmilienM is now known as EmilienM|afk14:32
*** scottda has quit IRC14:34
*** Yogi1 has joined #openstack-cinder14:34
*** timcl has quit IRC14:36
*** briancline has quit IRC14:37
*** markvoelker_ has quit IRC14:38
*** briancline has joined #openstack-cinder14:38
*** markvoelker has joined #openstack-cinder14:39
*** rluediger has joined #openstack-cinder14:39
*** timcl has joined #openstack-cinder14:40
rluedigerDuncanT: Hey Thomas. Unfortunatally my colleague puts his healing hand on both environments and without any known changes the cinder volume creation is now working well. I'm very confused about that and I have absolutely no idea what happens here.14:42
*** abhishekk has quit IRC14:42
DuncanTrluediger: Well, glad it is working. It would have been nice to know the answer, it was a weird one14:43
*** dustins has quit IRC14:43
rluedigerDuncanT: yepp I totally agree that14:43
*** bswartz has quit IRC14:43
*** marcusvrn1 has joined #openstack-cinder14:45
*** marcusvrn has quit IRC14:46
*** marcusvrn has joined #openstack-cinder14:46
openstackgerritMikhail Khodos proposed openstack/cinder: Get the right LUN from mapping  https://review.openstack.org/15739014:46
openstackgerritAnton Arefiev proposed openstack/cinder: Import only modules: H302  https://review.openstack.org/14578014:46
*** marcusvrn has quit IRC14:47
*** marcusvrn has joined #openstack-cinder14:47
*** Yogi1 has quit IRC14:47
*** akerr has quit IRC14:49
*** scottda has joined #openstack-cinder14:49
*** marcusvrn1 has quit IRC14:49
*** deepakcs has joined #openstack-cinder14:50
openstackgerritMikhail Khodos proposed openstack/cinder: NexentaStor SA: get the right LUN from mapping  https://review.openstack.org/15739014:55
jbernardrhe00: let me know if i can help with anything14:55
*** akerr has joined #openstack-cinder14:58
*** bswartz has joined #openstack-cinder14:59
*** Mandell has joined #openstack-cinder14:59
*** juzuluag has joined #openstack-cinder15:01
*** Yogi1 has joined #openstack-cinder15:04
*** dustins has joined #openstack-cinder15:04
*** dannywilson has joined #openstack-cinder15:05
*** ebalduf has joined #openstack-cinder15:06
*** primechuck has joined #openstack-cinder15:09
*** dannywilson has quit IRC15:09
*** ebalduf has quit IRC15:10
*** amoturi has joined #openstack-cinder15:10
*** am1 has joined #openstack-cinder15:12
*** am1 has quit IRC15:12
*** delattec has joined #openstack-cinder15:14
*** cdelatte has joined #openstack-cinder15:14
*** jecarey has joined #openstack-cinder15:16
*** lcurtis has joined #openstack-cinder15:18
*** jcru has joined #openstack-cinder15:19
*** pdeore_ has quit IRC15:21
*** ebalduf has joined #openstack-cinder15:25
*** rluediger has quit IRC15:26
*** TobiasE has joined #openstack-cinder15:27
*** Ilja has quit IRC15:28
*** mtanino has joined #openstack-cinder15:29
*** Ilja has joined #openstack-cinder15:30
*** ebalduf has quit IRC15:30
*** rmesta has joined #openstack-cinder15:34
*** mwichmann_ has quit IRC15:35
*** Mandell has quit IRC15:37
*** ebalduf has joined #openstack-cinder15:40
mtaninoavishay: ping15:41
*** EmilienM|afk is now known as EmilienM15:41
*** MasterPiece has joined #openstack-cinder15:41
openstackgerritSteven Kaufer proposed openstack/cinder: GET volumes API sorting REST/volume/DB updates  https://review.openstack.org/14191515:46
openstackgerritSteven Kaufer proposed openstack/cinder: GET volumes API sorting enhancements common utilities  https://review.openstack.org/14191415:46
*** Lee1092 has quit IRC15:50
*** jaypipes has quit IRC15:51
*** Ilja1 has joined #openstack-cinder15:52
*** thangp has joined #openstack-cinder15:53
*** hemnafk is now known as hemna15:53
*** Ilja has quit IRC15:53
*** mwichmann has joined #openstack-cinder15:55
*** Ilja1 has quit IRC15:55
*** Ilja has joined #openstack-cinder15:57
*** nellysmitt has quit IRC15:58
TobiasEHi, thingee, xyang, jgriffith would like to get your opinion/ comments on https://review.openstack.org/#/c/154377/15:59
dulekthangp: Hi16:00
thangpdulek: yo16:01
*** jaypipes has joined #openstack-cinder16:01
dulekthangp: I've answered your comment on base objects patch, is my answer clear?16:01
*** Ilja has quit IRC16:01
*** diegows has quit IRC16:01
dulekthangp: I haven't meant moving linstack changes from snapshots patch to the base.16:01
xyangTobiasE: hi, sure, I'll take a look16:01
*** kallebe has joined #openstack-cinder16:02
dulekthangp: I just meant that you're overwriting base change in snapshots patch, which isn't necessary.16:02
dulekthangp: Got to go now, so I want to make sure I'm not being misunderstood.16:03
thangpdulek: ok...the answer didnt show up on the patch...but i understand16:03
thangpdulek: thx16:03
*** Ilja has joined #openstack-cinder16:03
dulekthangp: Aww, that's because I haven't clicked submit, my bad.16:03
thangpdulek: lol16:03
*** annegentle has joined #openstack-cinder16:04
dulekthangp: fixed. ;) Thanks!16:04
*** david-lyle_afk is now known as david-lyle16:04
*** afazekas has quit IRC16:05
*** nellysmitt has joined #openstack-cinder16:11
TobiasExyang: cool16:11
*** dulek has quit IRC16:11
*** deepakcs has quit IRC16:12
*** PaulCuzner has joined #openstack-cinder16:16
*** annashen has joined #openstack-cinder16:16
e0nejgriffith: morning. looks like i've got too quick env:(. can't reproduce warning with long lvs run16:17
kallebeHello. I tried to execute ./stack.sh today but there is a problem with c-api. It does not start and devstack fails. The error message is "AttributeError: 'function' object has no attribute '_periodic_name'". The line given by the log is https://github.com/openstack/cinder/blob/master/cinder/openstack/common/periodic_task.py#L117 Has this happened t16:17
kallebeo anyone else here? I found a person posting in ask openstack the c-api error but no answers there16:17
e0nekallebe: hi! do you have any errors more? it works for me16:18
*** dannywilson has joined #openstack-cinder16:19
kallebee0ne hi. This is the only error for now, because it fails the stack.sh script and so devstack cannot run properly16:19
*** dannywilson has quit IRC16:19
kallebeI already cleaned and everything but it did not work16:19
*** xyang has quit IRC16:19
*** anshul has quit IRC16:21
*** MentalRay has joined #openstack-cinder16:22
*** marcusvrn1 has joined #openstack-cinder16:22
e0nekallebe: i ran devstack on a clean vm about 1hour ago. it works for me16:22
*** dannywilson has joined #openstack-cinder16:23
kallebee0ne ok, thanks for the help. I think I will have to start a fresh one indeed. I will try now.16:23
*** med_` is now known as medberry16:24
*** medberry is now known as medberry216:24
*** marcusvrn has quit IRC16:24
*** krtaylor has quit IRC16:25
openstackgerritThang Pham proposed openstack/cinder: Cinder objects base  https://review.openstack.org/13187316:25
*** ronis has quit IRC16:26
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support  https://review.openstack.org/8584716:26
*** timcl has quit IRC16:27
*** alecv has quit IRC16:30
*** delatte has joined #openstack-cinder16:30
*** Mandell has joined #openstack-cinder16:31
*** delattec has quit IRC16:32
*** cdelatte has quit IRC16:32
*** cdelatte has joined #openstack-cinder16:32
*** nellysmitt has quit IRC16:33
*** Ilja has quit IRC16:33
openstackgerritJoe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager  https://review.openstack.org/15726516:34
*** nellysmitt has joined #openstack-cinder16:35
*** openstackgerrit has quit IRC16:36
*** openstackgerrit has joined #openstack-cinder16:36
*** cnesa has joined #openstack-cinder16:37
*** xyang has joined #openstack-cinder16:38
*** tsekiyama has joined #openstack-cinder16:39
*** annegentle has quit IRC16:39
*** annegentle has joined #openstack-cinder16:40
*** MentalRay has quit IRC16:42
openstackgerritJoe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager  https://review.openstack.org/15726516:42
*** mwichmann has quit IRC16:42
*** PaulCuzner has quit IRC16:42
openstackgerritAlex Meade proposed openstack/cinder: Add tracing facility to NetApp ONTAP drivers  https://review.openstack.org/15743916:43
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use oslo.log instead oslo-incubator  https://review.openstack.org/15744116:45
*** nellysmitt has quit IRC16:45
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator  https://review.openstack.org/15744116:46
*** krtaylor has joined #openstack-cinder16:47
*** coolsvap is now known as coolsvap_16:57
jwcroppehemna: Re: https://review.openstack.org/157265 -- my comment on removal of 'None' make sense?  I can put it back if you'd prefer, but I just thought you'd appreciate 6 fewer characters of Cinder code.  :)17:00
*** annegentle has quit IRC17:01
smcginnisjwcroppe: FWIW, I agree with your change. The default if it's not there is None anyway, so it is cleaner not explicitly setting the default to None.17:02
jwcroppe^^^ thx17:02
hemnajwcroppe, ok yah just confirmed.17:02
hemnait's fine then17:02
smcginnis+117:02
jwcroppeThanks for the quick review, folks.  I think I tackled the comments.17:02
hemnait's just useless code churn though17:03
jwcroppeyeah, I only removed it since I was in there getting my hands dirty anyway17:03
smcginnisI'm generally in favor of removing code when it's not needed.17:04
smcginnisNot necessary, but the less code, the less chance of something changing and breaking.17:04
e0nesmcginnis: +217:04
jwcroppeSounds like we're good with leaving the removal then?17:04
smcginnise0ne: :)17:04
jwcroppesmcginnis, hemna: we'll let Jenkins do his thing and then hopefully that one is good to go.  Thanks again.17:06
hemna1 more item17:06
*** TobiasE has quit IRC17:09
*** TobiasE has joined #openstack-cinder17:11
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Allow archiving deleted rows to shadow tables, for performance  https://review.openstack.org/13118217:11
smcginnishemna: Good eye! I missed two things on one line. :/17:11
*** _cjones_ has joined #openstack-cinder17:13
*** afazekas has joined #openstack-cinder17:13
e0nesmcginnis: that's why we need at least two reviewer on each patch17:13
smcginnise0ne: Two in addition to me apparently! :)17:14
e0ne:)17:14
hemnaI saw that line and thought....huh, maybe he knows something I don't17:14
hemnaand had to try it myself in a sample.py file :P17:14
hemnaoops...nope!17:14
openstackgerritJoe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager  https://review.openstack.org/15726517:15
*** Apoorva has joined #openstack-cinder17:15
smcginnishemna: I usually assume people know more than me. ;)17:15
hemnasame here17:15
hemnathat's why I had to test it17:15
* e0ne thoughts that he is onle onr person who tryes some patches on a local env17:15
jwcroppefixed - actually it was a quick oversight from a previous comment17:16
jwcroppecan't use (initiator)s ... initiator is being iterated on17:16
jwcroppee.g., not a static entry, hence why {initiator: val} 'worked'17:16
*** kallebe has left #openstack-cinder17:17
jwcroppeThe structure of those log message was interesting to begin with :)    I just removed the dict style.17:17
*** annashen has quit IRC17:17
*** nshaikh has quit IRC17:17
*** MentalRay has joined #openstack-cinder17:19
*** jecarey has quit IRC17:20
*** nellysmitt has joined #openstack-cinder17:21
*** annashen has joined #openstack-cinder17:21
*** TobiasE has quit IRC17:21
jwcroppeI caught that by pondering how the interpreter wasn't blowing up all over the place forming those dicts.  We should be ready to go now.17:24
*** jungleboyj has joined #openstack-cinder17:25
*** PaulCuzner has joined #openstack-cinder17:26
*** jistr has quit IRC17:28
openstackgerritJoe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager  https://review.openstack.org/15726517:28
e0nejungleboyj: morning, Capitan Oslo!17:28
jungleboyje0ne: Good afternoon sir.  What is up17:29
e0nejungleboyj: i'm wondered why we don't use oslo.log still17:29
jungleboyje0ne: Doh!  I forgot I said I was going to do that.  I have a quiet afternoon.  Let me tackle that.17:30
e0netryed to migrate to it, but we've got some outdated modules from oslo-incubator17:30
*** dansmith has quit IRC17:30
jungleboyje0ne: That was the other thing I was going to finish up doing.17:30
*** dansmith has joined #openstack-cinder17:30
jungleboyje0ne: I can work on getting that done today if you can do reviews on them tomorrow and try to get it through.  :-)17:31
e0nejungleboyj: https://review.openstack.org/157441 - not sure that it should be landed in Kilo17:31
e0nejungleboyj: sure, i'll review them17:31
*** kaufer has quit IRC17:31
*** rajinir_r has quit IRC17:32
jungleboyje0ne: I think we should be able to get this in for Kilo still.17:32
*** rajinir_r has joined #openstack-cinder17:32
jungleboyje0ne: So, is this patch just a WIP and you just need the incubator updates done?17:32
e0nejungleboyj: it doesn't change openstack.common and doesn't remove outdated modules17:33
jwcroppejungleboyj: please see latest patch set on 157265 -- regarding your 'initiator' comment, I actually just removed dict-style passing since it was a variable, so (initiator)s wouldn't work.  I missed that initially too -- props to hemna for catching that.17:33
jungleboyjjwcroppe: Ok.  Cool.17:34
*** patrickeast has joined #openstack-cinder17:34
jungleboyje0ne: Ok.  Let me look at finishing up clean up of oslo-incubator today.  Thanks for getting the oslo.log change proposed.17:34
e0nejungleboyj: so, it needs rebase and remove 2 files after oslo-inccubator updates landed17:35
*** Yogi1 has quit IRC17:35
jungleboyje0ne: Cool.17:35
*** nlevinki has quit IRC17:36
*** Yogi1 has joined #openstack-cinder17:37
*** timcl has joined #openstack-cinder17:37
jungleboyjjwcroppe: Was reviewing code from my phone.  Not surprised I missed something.  :-)17:38
e0neo_017:38
smcginnisjungleboyj: You're hardcore my friend.17:38
jungleboyjsmcginnis:  I was donating blood.  Had to pass the time somehow.17:39
jwcroppehaha17:39
smcginnisjungleboyj: Hah! Code reviewing on a phone while losing blood.17:39
jungleboyjsmcginnis: Hardcore!17:40
*** leeantho has joined #openstack-cinder17:40
jungleboyjTechnically just red blood cells.  Did an double unit aphoresis donation.17:41
e0nenext level: make cinder works on a tablet17:41
*** thingee has joined #openstack-cinder17:41
jungleboyj:-)17:41
jwcroppejungleboyj: you're setting a new precedent here for dedication17:41
jungleboyjAnyway, gotta go get Logan to daycare.  Back in a bit.  e0ne I will work on getting Oslo cleaned up this afternoon.17:42
e0nejungleboyj: thanks17:42
jungleboyjIs review.openstack.org being really slow for anyone else?17:42
jwcroppeanyhow, hopefully that 'nit' patch is ready to go... those nits always turn into several updates :)  Thanks again guys.  Stepping away for some grub.17:42
jwcroppe^^^ slow for me today17:43
jungleboyje0ne: This would be one step towards getting oslo clean up if you can take a look:  https://review.openstack.org/15645517:43
jungleboyjjwcroppe: Ok.  Thanks.17:43
e0nejungleboyj: i'm thinking about thingee's comment. how is the right way to add implicit dependency?17:45
*** jordanP has quit IRC17:47
ganso_thingee: ping17:48
thingeeganso_: hi17:48
jgriffithe0ne: interesting17:49
jgriffithe0ne: what are you running on?17:49
e0nejgriffith: vm ubuntu 14.04, 1cpu, 2gb ram, ssd on host17:54
jgriffithe0ne: interesting17:54
e0nejgriffith: i'll try it on others env to verify17:54
*** kaufer has joined #openstack-cinder17:55
jgriffithe0ne: so I've narrowed it down to one test BTW17:55
jgriffithe0ne: tempest.api.compute.servers.test_server_rescue_negative.ServerRescueNegativeTestJSON.test_rescued_vm_detach_volume[gate,negative,volume]17:56
*** avishay has quit IRC17:57
e0nejgriffith: thanks!17:57
jgriffithe0ne: ahh... is it an OpenStack VM?17:57
jgriffithe0ne: I think this is related to "OpenStack on OpenStack" model FWIW17:57
e0nejgriffith: no. i created it from ubuntu server iso17:57
*** jgravel has joined #openstack-cinder17:57
e0nejgriffith: on kvm17:57
jgriffithe0ne: so what happens is something like this....17:57
e0nejgriffith: hm... where can i get this image?17:58
jgriffithe0ne: the rescue basicly is a new vm with the old vm ephemeral disk attached17:58
jgriffithe0ne: this does some things to the servers /dev/sdX that I haven't quite figured out (moves things around)17:58
jgriffithe0ne: and what happens is an lvs or vgs call will sit there trying to open the /dev/sdX device that it was at but timeout17:59
jgriffithbecause it's not actually valid any longer17:59
*** ganso_ has left #openstack-cinder17:59
jgriffithhonestly, I'm trying to understand exactly what happens here and what rescue is doing17:59
thingeeSo quick question for people, does 5 fish bowl sessions, 8 working sessions and 2 sprint sessions on friday seem good for the next summit? http://lists.openstack.org/pipermail/openstack-dev/2015-January/054122.html17:59
thingeewe did 7 fish bowl last time, but I think we would benefit more from the smaller working sessions18:00
e0nejgriffith: need to go home. bb online in a few hours18:00
jgriffithe0ne: no worries, have a good evening18:00
e0nejgriffith: thanks. i'm calling it "breack for dinner:)"18:01
*** bswartz has quit IRC18:01
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Fix comments style according to Hacking rules  https://review.openstack.org/15746818:02
e0nethingee: imo, it  depends on what topic on design sessions will be18:02
thingeefrom the previous times we talked about sessions for the summit, most we just stuck in the sprint18:03
thingeePerhaps I could start an etherpad to get topics from people18:04
thingeebut the great thing about working sessions is they're pretty dynamic18:04
*** bswartz has joined #openstack-cinder18:04
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Update the documentation for os-brick  https://review.openstack.org/15720218:04
*** Swanson has joined #openstack-cinder18:05
*** Apoorva has quit IRC18:05
*** ronis has joined #openstack-cinder18:06
*** aix has quit IRC18:07
*** ybathia has joined #openstack-cinder18:07
*** thingee has quit IRC18:08
*** e0ne has quit IRC18:08
*** harlowja has joined #openstack-cinder18:10
openstackgerritSai Kiran proposed openstack/cinder: Fix argument order in assertEqual: tests/test_service.py  https://review.openstack.org/15747418:12
*** asmith_brcd has joined #openstack-cinder18:13
kauferthingee: I wanted to follow up with you on this review if you have a minute: https://review.openstack.org/#/c/148373/ ... I believe that this was discussed at the mid-cycle meetup, anything you need me to do on that?18:19
openstackgerritWalter A. Boring IV (hemna) proposed openstack/python-cinderclient: Add volume multi attach support  https://review.openstack.org/8585618:23
*** Yogi1 has quit IRC18:27
*** jecarey has joined #openstack-cinder18:27
*** smcginnis has quit IRC18:28
*** MentalRay has quit IRC18:30
*** smcginnis has joined #openstack-cinder18:30
*** Apoorva has joined #openstack-cinder18:32
*** diegows has joined #openstack-cinder18:32
*** nikesh_vedams has quit IRC18:33
*** cnesa1 has joined #openstack-cinder18:35
*** cnesa has quit IRC18:35
*** akerr is now known as akerr_away18:35
smcginnisI believe I had heard we are not supposed to put any attribution to the OpenStack Foundation in copyright headers.18:36
smcginnisCan anyone confirm that?18:36
smcginnishemna: ^^18:36
hemnaheh, I hadn't heard that18:37
hemnathingee ?18:37
smcginnishemna: I could be wrong, so hopefully someone here can give a definitive answer.18:37
jgriffithsmcginnis: hemna yeah... that was a "thing" a while back18:37
smcginnisjgriffith: Thanks, figured you would know.18:38
hemnaso....remove it then18:38
hemnaok I'll put a different header in then.18:38
hemnathanks18:38
smcginnishemna: I believe so. Cool if you want to put in HP.18:38
jgriffithhemna: smcginnis in reality the only thing that should really be in there at all that's worth value anyway is the Apache license header18:38
*** akerr_away has quit IRC18:38
smcginnisOr no specific company.18:38
jgriffithanything else is just crap noise and means nothing18:38
jgriffithsmcginnis: :)18:38
hemna:)18:39
jgriffithsmcginnis: we allow companies to put them in, but they're useless for the most part18:39
*** akerr has joined #openstack-cinder18:39
smcginnisjgriffith: That was my understanding. It doesn't hurt, but it doesn't really add anything.18:39
jgriffithsmcginnis: indeed18:39
smcginnisPersonally I'd rather see nothing.18:39
jgriffithsmcginnis: lawyers have been going back and forth on that topic for years in OpenStack18:39
smcginnisOver several thousand files, that's a space savings. :)18:40
jgriffithsmcginnis: and +2, I'd rather see nothing too18:40
smcginnisjgriffith: Oh, I'm sure.18:40
*** EmilienM is now known as EmilienM|afk18:40
jgriffithHP lawyers, vs Foundation lawyers, vs EMC lawyers, vs .........18:40
jgriffithcrazy koo koo18:40
smcginnisOpen source licensing is often more complex than the actual code IMO.18:40
jgriffithhaha!18:41
jgriffithFunny observation18:41
jgriffithand not because it's not true either :)18:41
smcginnis;)18:41
Swanson-1 me, sean, for the copyright notice and i'll burn down your cube.18:42
jgriffithLOL18:42
jgriffithsmcginnis: so I should note... some companies require their contributors put it in18:42
smcginnisSwanson: :)18:42
jgriffithsmcginnis: so we just live with it18:42
* jgriffith wants to make sure smcginnis 's cube doesn't get burnt18:43
smcginnisjgriffith: Yeah, I can see that.18:43
smcginnisjgriffith: Hah!18:43
*** BharatK has quit IRC18:43
jgriffithjungleboyj: have you looked at pulling oslo log lib at all?18:44
jgriffithjungleboyj: I may tackle that today if you haven't18:44
jgriffith^^  or anyone18:44
smcginnisjgriffith: I believe he had commented earlier today that he was going to get to it today.18:45
smcginnisjgriffith: If he hasn't passed out from donating blood.18:45
jgriffithsmcginnis: haha18:45
jgriffithsmcginnis: cool!18:45
jgriffiththe lib part, not the passing out from giving blood part :)18:45
jungleboyjjgriffith: Actually, e0ne has a review out but needs me to finish getting oslo-incubator cleaned up to get it merged.18:48
jungleboyjjgriffith: So I am working on that right now.18:48
jgriffithcoolio18:48
jungleboyjjgriffith: Here is e0ne s patch:  https://review.openstack.org/#/c/157441/18:48
jungleboyjSorry I didn't get to that earlier but have been busy since the meet-up.18:49
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Update the documentation for os-brick  https://review.openstack.org/15720218:49
patrickeastjgriffith: hey, so i spent a few min coding up a poc of the initiator meta data table spec to make sure it would work ok and I didn’t leave anything out of the spec, and realized that my proposal of modifying the signature of BaseVD::initialize_connection touches a ton of code, any thoughts on a less intrusive way to do a model update kind of thing with it? or is it going to be ok for my patch to go modify every single driver and all thei18:51
patrickeasttests?18:51
patrickeastjgriffith: i’m ok making the changes, but i am hesitent to touch so many other drivers i can’t really test out18:53
jgriffithpatrickeast: smcginnis https://imgflip.com/i/huo9r18:53
jgriffithjungleboyj:18:53
patrickeastlol18:53
smcginnisjgriffith: :)18:53
jungleboyjjgriffith: :-)18:54
jgriffithpatrickeast: so can't you leave it?18:55
jgriffithpatrickeast: I mean, just override the call in your driver only18:56
jgriffithpatrickeast: so it only applies to those who want/need it for now?18:56
patrickeastjgriffith: hmm yea, we probably could, but it makes the code in the volume manager that calls it more complex, it would need to test out which one the driver has18:56
jgriffithpatrickeast: how come?18:57
jgriffithpatrickeast: so what I'm thinking is "don't" do anything with it in manager18:57
patrickeastjgriffith: in the current spec after some advice from DuncanT it returns a tuple, maybe my understanding of python isn’t good enough but i was under the impression it throws an error if you try and return the wrong number of args18:57
jgriffithpatrickeast: or... just do an "if initiator_table" "else"18:58
jgriffithpatrickeast: ahh... fin tuples!18:58
jgriffith:)18:58
jgriffithpatrickeast: seriously... that's "ok"18:58
jgriffithpatrickeast: lemme look at your patch18:58
*** timcl1 has joined #openstack-cinder18:59
jgriffithpatrickeast: maybe my heads somewhere it shouldn't be18:59
*** timcl has quit IRC19:00
patrickeastjgriffith: i’ll post a patch in an hour or two (have a meeting right now), i’ll pull the trigger on the refactor and put up a WIP one19:01
patrickeastjgriffith: here is the manager code changes http://paste.openstack.org/show/177812/19:02
patrickeast(untested)19:02
patrickeastbut shows kind of a gist of things19:02
*** bswartz has quit IRC19:03
*** mwichmann has joined #openstack-cinder19:04
*** ronis has quit IRC19:05
*** e0ne has joined #openstack-cinder19:06
*** bkopilov has quit IRC19:07
*** akerr has quit IRC19:08
*** tbarron has quit IRC19:09
*** r-daneel has joined #openstack-cinder19:10
*** dencaval has joined #openstack-cinder19:10
*** timcl1 has quit IRC19:11
*** bkopilov has joined #openstack-cinder19:11
*** tbarron has joined #openstack-cinder19:14
*** akerr has joined #openstack-cinder19:14
*** delatte has quit IRC19:15
*** cdelatte has quit IRC19:15
*** dustins has quit IRC19:15
*** vilobhmm has joined #openstack-cinder19:18
*** thingee has joined #openstack-cinder19:19
*** r-daneel has quit IRC19:19
*** vilobhmm1 has joined #openstack-cinder19:21
*** bswartz has joined #openstack-cinder19:24
*** vilobhmm has quit IRC19:24
*** bkopilov has quit IRC19:25
*** timcl has joined #openstack-cinder19:26
*** nellysmitt has quit IRC19:26
*** dustins has joined #openstack-cinder19:30
*** cdelatte has joined #openstack-cinder19:30
*** dulek has joined #openstack-cinder19:31
dulekthingee: hi, my patch adding backup object is ready, can it get into priorities etherpad?19:34
dulekthingee: it's review 15708519:34
thingeedulek: I'll check it out and see about moving it in. Want to make sure we get enough testing on volumes/snapshots.19:35
thingeexyang1: ^19:35
thingeedulek: DuncanT already signed off on the object for volumes19:36
thingeexyang1: did you work out your devstack issues?19:36
dulekthingee: Thanks. Patch is quite simple anyway.19:36
thingeedulek: I like simple :)19:36
thingeedulek: I also check some multi-attach stuff first too. Already put that off yesterday from hemna.19:37
dulekthingee: we all do. ;)19:37
dulekthingee: okay, i'll do. Thanks!19:37
*** delattec has joined #openstack-cinder19:38
*** krtaylor has quit IRC19:38
*** cdelatte has quit IRC19:38
openstackgerritWalter A. Boring IV (hemna) proposed openstack/python-cinderclient: Add volume multi attach support  https://review.openstack.org/8585619:42
openstackgerritTom Barron proposed openstack/cinder: Refactor Swift backup driver and introduce chunking driver  https://review.openstack.org/14972519:42
*** Yogi1 has joined #openstack-cinder19:44
*** jwang_ has joined #openstack-cinder19:48
*** dulek has quit IRC19:48
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Update the documentation for os-brick  https://review.openstack.org/15720219:49
*** jwang has quit IRC19:50
*** erlon has quit IRC19:51
*** nellysmitt has joined #openstack-cinder19:54
*** bswartz has quit IRC20:01
*** timcl1 has joined #openstack-cinder20:02
*** timcl has quit IRC20:06
*** MentalRay has joined #openstack-cinder20:07
*** PaulCuzner has quit IRC20:14
*** krtaylor has joined #openstack-cinder20:16
*** PaulCuzner has joined #openstack-cinder20:17
*** bill_az has quit IRC20:18
*** PaulCuzner has quit IRC20:18
*** diegows has quit IRC20:19
xyangthingee: hi, the new devstack didn't work for me. I went back to an earlier version. got most services installed, still need to sort out a few things.20:23
xyangthingee: I haven't got chance to test the object thing.20:23
smcginnisAnyone know what this is all about when running tempest?20:29
smcginnisNo repository found in /opt/stack/tempest. Create one by running "testr init"20:29
SwansonIf you run it is there one?20:33
smcginnisSwanson: Part of the automated CI spin up, so running again and going to keep it around to investigate more.20:33
smcginnisBut I've seen this in the past and couldn't find a clear reason.20:33
smcginnisSometimes I can clone devstack and stack and it's fine. Other times, not so much.20:34
SwansonI just pulled down a new version and am having some issues.20:34
thingeexyang: alright, weird20:36
smcginnisOn another existing one I had an issue with c-vol service not starting. Think I had to nuke /opt/stack/* to clear it up.20:36
*** thingee has quit IRC20:37
SwansonThat has been a multibackend issue for me.20:38
smcginnisSwanson: Oh. C-vol not starting?20:38
Swansonsmcginnis: The cinder-volume service.  IF you do a cinder-manage service list you should see an unhappy cinder-volume service.20:39
smcginnisSwanson: Only when you have multibackend configured?20:40
Swansonsmcginnis:Only multibackend.  If it comes up wrong (something deep in sqlalchemy errors) it never works right.  If it doesn't it never fails.20:41
*** bswartz has joined #openstack-cinder20:42
smcginnisSwanson: Seems odd.20:42
smcginnisAnyone know anything about tempest with the "testr init" or see mutlibackend errors?20:43
jgriffithSwanson: smcginnis cinder.conf file?20:44
smcginnisSwanson: ^^20:45
smcginnisjgriffith: For my issue cinder appears happy. Something wonky with tempest though.20:45
jgriffithsmcginnis: tempest.conf :)20:45
jgriffithsmcginnis: most likely something around default volume-type maybe?20:46
SwansonI have a cinder.conf I've been using.20:46
*** e0ne has quit IRC20:47
smcginnisjgriffith: Have it in a minute.20:47
SwansonNeed to get it to the right machine....20:48
jgriffithFor the service not to start at all however is kinda odd20:48
jgriffithBut it seems there are two questions here....20:49
jgriffithtempest tests and vol-service not starting20:49
jgriffithnot sure if theyr'e the same, or individual?20:49
smcginnisTwo different ones.20:50
smcginnisJust mass confusion over here. :)20:50
smcginnisjgriffith: Here's my tempest.conf: http://paste.openstack.org/show/177884/20:50
jgriffithgood times!20:50
jgriffithsmcginnis: I thought there was an explicit multi-backend setting that went in there20:52
smcginnisjgriffith: Sorry, this paste is the issue with getting a "testr init" error message. No multibackend here.20:52
smcginnisjgriffith: Sorry for being confusing as hell.20:52
Swansonjgriffith: try this... http://paste.openstack.org/show/177885/20:52
jgriffithsmcginnis: darn you!20:52
jgriffithLOl20:52
jgriffithLOL20:52
jgriffithOk, so don't use testr directly20:53
jgriffithI was told I was "stupid" the other day on this very topic :)20:53
smcginnisI'm calling tools/pretty_tox.sh volume and getting this.20:53
smcginnisHah20:53
jgriffithyeah, you can't do that any more20:53
smcginnispretty_tox.sh?20:53
jgriffithand the folks in another channel basicly told me I was a moron20:53
jgriffithtox -e all -- volumes20:54
smcginnisjgriffith: If that's something obvious, I'm missing something big here.20:54
jgriffithsmcginnis: tox -e all -- volume | tee -a console.log.out20:54
smcginnisjgriffith: Well, that certainly looks better. At least it hasn't blown up immediatey.20:54
jgriffithsmcginnis: Swanson So the problem is...20:54
smcginnisimmediately20:55
jgriffithtempest is no longer "installed" by devstack20:55
jgriffithand you're expected to run in venv/tox20:55
smcginnisjgriffith: Now no tempest for Swanson. Just stacking with multibackend config.20:55
jgriffithbut surprisingly pretty_tox doesn't use tox20:55
smcginnisWTH20:55
smcginnisThat's false advertisement.20:55
smcginnis;)20:55
Swansonjgriffith: That's the multibackend one.  Where sometimes the volume service for one or both backends dies without much warning.  usually on stack.sh20:55
jgriffithYeah, but like I said, I was the moron :)20:55
jgriffithSwanson: local.conf?20:56
smcginnisjgriffith: Thanks for taking one for the team. :)20:56
jgriffithhaha!20:57
smcginnisjgriffith: Here was Swanson's: tox -e all -- volumes20:57
smcginnisDoh20:57
smcginnisCopy paste is so hard.20:57
jgriffithSwanson: make sure you're local.conf has:20:57
smcginnis http://paste.openstack.org/show/177885/20:57
jgriffithcinder_enabled_backends=bend1,bend220:57
jgriffithand....20:57
Swansonhttp://paste.openstack.org/show/177889/20:58
Swansonlocal.conf there.20:58
jgriffithhttps://gist.github.com/j-griffith/3abdb8cb93462b122eaf20:58
jgriffithOh... k... just a sec20:59
jgriffithSwanson: need the Default section header20:59
smcginnisSwanson: Compared to jgriffith's, maybe need to move under [[post-config|$CINDER_CONF]]?20:59
jgriffithSwanson: also verify the ENABLED_BACKENDS entry20:59
SwansonOh.  With the enabled backends bit.21:00
jgriffithSwanson: IIRC I had issues with this because it might get added to cinder.conf but it would be added in the wrong place21:00
*** PaulCuzner has joined #openstack-cinder21:00
jgriffithYou can even put the CINDER_ENABLED_BACKENDS setting in both sections :)21:01
*** nellysmitt has quit IRC21:01
SwansonI think I had to put it where it is to get it to work at all.21:01
jgriffithhaha... try it like I have it with the [DEFAULT] section added21:01
jgriffiththink you'll be good21:01
SwansonI will give that a try.  ThankS!21:02
smcginnisjgriffith: Thanks for the help and dealing with the confusion. :)21:02
jgriffithsmcginnis: that's how I roll :)21:04
jungleboyjhemna: I am assuming the paths in README.rst for https://review.openstack.org/#/c/157202/4 are where the documentation will be?21:04
*** fischerw has joined #openstack-cinder21:07
*** Mandell has quit IRC21:09
*** Mandell has joined #openstack-cinder21:09
openstackgerritPatrick East proposed openstack/cinder: Add DB table for driver specific data  https://review.openstack.org/15183721:13
patrickeastjgriffith: ^ i put up part of the poc i was fiddling with21:14
patrickeastjgriffith: it shows the model, db apis, and what it outlined in the spec as far as the manager and driver methods21:15
patrickeastbut i didn’t do the whole refactor in that patch21:15
*** annashen has quit IRC21:16
*** annashen has joined #openstack-cinder21:17
Swansonjgriffith: still floating around?  should I end up with a single cinder-volume service?  Or one for each of the backends?21:27
*** timcl has joined #openstack-cinder21:29
*** timcl1 has quit IRC21:30
*** annashen has quit IRC21:32
*** morganfainberg is now known as needscoffee21:32
*** timcl has left #openstack-cinder21:32
*** Mandell has quit IRC21:33
*** theanalyst has quit IRC21:37
*** gary-smith has quit IRC21:38
*** theanalyst has joined #openstack-cinder21:39
*** gary-smith has joined #openstack-cinder21:41
hemnahttps://bugs.launchpad.net/nova/+bug/142369021:42
*** bswartz has quit IRC21:42
openstackLaunchpad bug 1423690 in OpenStack Compute (nova) "rebuild_instance doesn't detach cinder volumes correctly" [Undecided,New]21:42
openstackgerritBob Callaway proposed openstack/cinder: Manage/unmanage impl for NetApp ONTAP iscsi driver  https://review.openstack.org/13811121:43
hemnajungleboyj, thanks for the reviews21:43
jungleboyjhemna: Welcome.21:43
*** gary-smith has quit IRC21:49
*** gary-smith has joined #openstack-cinder21:50
*** akerr has quit IRC21:51
jgriffithSwanson: well... cinder service-list should show two21:51
*** juzuluag has quit IRC21:53
*** jaypipes has quit IRC21:53
jgriffithhemna: I don't think your bug is accurate21:55
jgriffithhemna: what version of OpenStack are you looking at?21:55
*** Apoorva_ has joined #openstack-cinder21:55
*** lcurtis has quit IRC21:56
jgriffithhemna: notice there's a "driver.detach_volume" call21:56
*** MasterPiece has quit IRC21:56
hemnathis is juno and master21:56
jgriffithhemna: and a driver.remove_export21:56
jgriffithit's there as well21:57
hemnadetach_block_devices is what is defined and passed into driver.rebuild21:57
hemnadetach_block_devices only calls volume_api.detach()21:57
jgriffithhemna: ok21:57
jgriffithhemna: don't know what that means exactly21:57
hemnahttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L287321:58
*** Apoorva has quit IRC21:59
jungleboyj*sigh*  Going to take some work to get oslo cleaned up.  Been a bunch of changes in there to account for the namespace change.22:00
jungleboyjSo, going to have to think out how to approach this.22:00
smcginnisCarefully22:00
*** patrickeast_ has joined #openstack-cinder22:02
*** dannywil_ has joined #openstack-cinder22:02
*** Yogi1 has quit IRC22:03
jgriffithhemna: Yeah, but that now calls os-detach22:03
hemnayup22:03
jgriffithhemna: whih calls volume.api:detach22:03
*** patrickeast has quit IRC22:04
*** patrickeast_ is now known as patrickeast22:04
hemnawhich only calls the volume manager's detach_volume()22:04
jgriffithhemna: which call rpcapi.detach_volume22:04
hemnawhich doesn't detach22:04
*** dannywilson has quit IRC22:04
hemnait basically only touches the db22:04
hemnaas drivers don't detach volumes from that call22:04
hemnaterminate_connection is where the drivers remove the volume export from the array22:04
jgriffithhemna: self.driver.remove_export?22:04
hemnayah, doesn't unattach22:05
jungleboyjsmcginnis: Thank you captain obvious22:05
jgriffithhemna: http://goo.gl/s4Z6UZ22:05
smcginnisjungleboyj: Oh, a promotion from sergeant smart ass. :)22:05
jgriffithhemna: I'm confused, care to teach me?22:05
jungleboyj:-)22:05
jgriffithhemna: you're saying driver.remove_export doesn't detach?22:06
jgriffithhemna: sounds like a driver problem?22:06
jgriffithwhat am I missing22:06
hemnaremove_export doesn't detach volumes.22:06
hemnaonly terminate_connection does22:06
jgriffithhemna: Ohhh... I see22:06
jgriffithyour driver doesn't do anything22:07
jgriffithit's a pass in your driver22:07
jgriffithhemna: https://github.com/openstack/cinder/blob/stable/juno/cinder/volume/drivers/lvm.py#L56622:08
hemnaand in others22:08
jgriffithreference implementation... emphasis on *reference*22:08
jgriffith:)22:08
jgriffithbut yes, inconsistent22:09
jgriffithI'd say fix it on Cinder side if it were me22:09
jgriffiththat's where the bug is22:09
hemnalots of drivers in cinder don't do anything on remove_export22:09
hemnanova's detach_volume does it correctly.22:10
jgriffithindeed22:10
hemnacalls everything in the correct sequence22:10
hemnaI'm not sure why their rebuild_instance doesn't just call detach_volume22:10
jgriffithhemna: fair point22:10
*** Apoorva_ has quit IRC22:10
jgriffithI'm not sure why we still have two different paths22:10
hemnayah I agree22:10
*** Apoorva has joined #openstack-cinder22:11
hemnaso not only do we have driver.remove_export22:11
hemnabut we also have driver.detach_volume22:11
hemnaalong with driver.terminate_connection22:11
hemnaheh22:11
smcginnisWhat is the difference between initialize_connection and create_export supposed to be?22:12
* smcginnis looks guilty22:12
*** bswartz has joined #openstack-cinder22:12
hemna:P22:12
jgriffithsmcginnis: so I tried to make all of that go away a year ago22:12
jgriffithbut it's like whack'o mole22:12
hemnathe API is confusing for sure22:12
jgriffithand as you see... one vendor wants one, the other wants another22:12
smcginnisSeems most use initialize/terminate and not ensure/remove.22:12
jgriffithdetach IMO should implicitly terminate connections22:13
jgriffithsmcginnis: in the past we had this scheme of "never deleting" the iscsi target after the vol was created22:13
jgriffithso we could just reuse it22:13
hemnasmcginnis, oh and don't forget ensure_export22:13
smcginnishemna: Yeah, that was confusing too.22:13
jgriffithhemna: but hey... at least we have more drivers22:13
hemnaanyway I don't want to boil the ocean, but I still think the nova bug is valid22:14
jgriffithhemna: and we're closer to making things easier in horizon22:14
jgriffithnever mind things are fundamentally a wreck22:14
hemna:P22:14
hemnayah I'm with you22:14
*** openstackstatus has joined #openstack-cinder22:14
*** ChanServ sets mode: +v openstackstatus22:14
SwansonHmmm....  I have a cinder-scheduler and a cinder-volume service...  That's from cinder-manage service list.  cinder service-list gives me a 403 forbidden.  (probably wrong creds)22:16
jgriffithSwanson: you need to be admin22:16
smcginnisSwanson: cd devstack; . openrc admin admin22:16
*** nkrinner has quit IRC22:16
jgriffithbut manage should have it too22:17
hemnajgriffith, so lets work on creating another "tech debt" list in Vancouver and see if we can get some folks signed up to try and tackle some of those for L22:19
jgriffithhemna: well, we've done that twice now22:20
jgriffithbut sure... maybe third times a charm22:20
*** jungleboyj has quit IRC22:20
hemnasome of those items got attention in K22:20
*** diegows has joined #openstack-cinder22:20
*** _cjones_ has quit IRC22:24
*** annashen has joined #openstack-cinder22:25
*** annashen has quit IRC22:25
*** annashen has joined #openstack-cinder22:26
*** _cjones_ has joined #openstack-cinder22:28
*** thingee has joined #openstack-cinder22:29
tsekiyamahemna: Should I remove the brick related changes from https://review.openstack.org/#/c/140877/ ?22:31
tsekiyamahemna: Currently it has duplicated changes with os-brick patch https://review.openstack.org/#/c/157184/22:31
hemnayup22:31
tsekiyamahemna: so 140877 will only have the changes for the lvm driver. Correct?22:31
hemnayou mean the target changes right ?22:32
*** thangp has quit IRC22:33
*** mriedem has quit IRC22:33
tsekiyamahemna: Yes, and driver.py22:33
hemnayah22:33
*** ebalduf has quit IRC22:33
hemnaI 'think' I'm getting close to a first release of os-brick22:33
tsekiyamahemna: okey. thanks22:34
hemnaso, once that happens then my remove-brick patch has a chance of getting in for cinder22:34
tsekiyamahemna: BTW, can I fetch os-brick using pip?22:34
hemnaheh22:34
hemnanot yet22:34
hemnathat's the part I'm trying to get done asap22:35
tsekiyamahemna: ah i see.22:35
tsekiyamahemna: I was trying various names like os-brick, os_brick, or even just brick (and fetched another lib!)22:36
thingeewanghao: can we get an update on this please https://review.openstack.org/#/c/144590/22:36
thingeewanghao: a few days without update22:37
hemnatsekiyama, so I'm basically just waiting on https://review.openstack.org/#/c/157202/ to land22:37
hemnaand then whenever we can do a release to pypi.22:37
*** medberry2 is now known as med_22:37
tsekiyamahemna: great22:37
*** med_ has quit IRC22:37
*** med_ has joined #openstack-cinder22:37
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Update the documentation for os-brick  https://review.openstack.org/15720222:39
thingeeameade: minor issues with this, but we could get this through today after a reup https://review.openstack.org/#/c/155915/122:41
*** ebalduf has joined #openstack-cinder22:44
ameadethingee: thanks I'll tweak it after dinner22:44
*** annashen has quit IRC22:46
*** dulek has joined #openstack-cinder22:48
openstackgerritSteven Kaufer proposed openstack/python-cinderclient: WIP Adopt CLI Sorting Argument Guidelines  https://review.openstack.org/14591322:49
*** jungleboyj has joined #openstack-cinder22:50
thingeehemna: I don't have voting rights on os-brick22:52
hemna?!22:52
thingeeoh now I do22:52
thingeeweird22:52
hemnaok phew22:53
DuncanTWalt: Approved22:53
hemnathat's odd22:53
hemnaDuncanT, thank you22:53
*** annashen has joined #openstack-cinder22:53
jgriffithhemna: thingee DuncanT wanted to get some thoughts around the connection stuff...22:53
thingeeDuncanT: oh well nevermind22:53
jgriffithsince you're all here :)22:53
hemnaok22:54
DuncanTjgriffith: ok...22:54
jgriffithis there any value/reason in having the method that "just" flips the state in the DB?22:54
hemnacontext ?22:54
jgriffithwhat I'm thinking of proposing is something such that "all roads lead to detach"22:54
hemnaI'm not following sorry.22:54
jgriffithhemna: hehe.. that's ok22:54
*** dulek has quit IRC22:54
hemnaoh the driver API22:55
jgriffithso detach, vs termiante_connection22:55
hemnaso, why don't we just have 2 methods22:55
hemnaattach_volume, ensure_attached22:55
hemnaand detach_volume22:55
hemnaI guess that's 3 :P22:55
DuncanTjgriffith: I think you're spot on, though we can't fix it in K I don't think22:55
*** kaufer has quit IRC22:55
jgriffithyeah, what I'm getting at is for back compat we have one call the other :)22:55
DuncanTjgriffith: Since Nova will need work22:55
jgriffithDuncanT: that solves the K dilema, fixes hemna current bug22:56
jgriffithand gives us time to work the nova change22:56
jgriffithpossibly mark the cruft as "deprecated" for Kilo22:56
DuncanTjgriffith: Hmmm, if you think having detach call terminate is safe, then sure22:56
jgriffithWell, it does for the ref driver :)22:57
DuncanTjgriffith: Fair enough :-)22:57
jgriffithtrying to figure out what else people may be using it for22:57
jgriffithlooking in drivers it seems mostly it's confusion around "which to implement"22:57
jgriffithmost have done one or the other, not both22:57
hemnaoh so you are thinking for K, to help fix the nova issue, to redirect the call to terminate as well ?22:57
jgriffithhemna: correct22:57
hemnajgriffith, yup22:57
DuncanTNova's interactions with cinder makes me a bit sad... was discussing this with somebody else a few minutes ago22:57
hemnathat part has always confused me22:57
jgriffithhemna: doesn't help your Juno customers though22:57
jgriffithDuncanT: yeah, we need to really focus on that for L22:58
jgriffithlike first thing out of the gate IMO22:58
hemnaDuncanT, amen.22:58
hemnait's been a sore spot for some time for me.22:58
jgriffithanyway... I think we can fix a lot of things with changes on our side that make calls do some implicit things22:58
DuncanThemna: I remember you commenting on it repeatedly. We should figure it out so the patches are knocking on nova's door the day they open for submissions22:59
hemnayah I think that'd be great22:59
*** PaulCuzner has quit IRC22:59
jgriffithOk... if there's no objections I'll look and see what I can come up with and put it on github for everybody to check out22:59
DuncanTjgriffith: There's some state checks we can't do that I think we should, due to the way nova abuses the interface, so it would require some intensive testing, but it definitely needs improving22:59
*** primechuck has quit IRC22:59
DuncanTjgriffith: Count me in for reviews & testing23:00
hemnaso, I guess the only thing I'd worry about jgriffith, is getting corner cases where terminate_connection gets called twice23:00
hemnathat could lead to other issues as well23:00
hemnajust have to make sure that doesn't happen23:00
jgriffithhemna: yeah... interesting23:00
jgriffithhemna: although a simple try/catch would get us through the turmoil at first23:00
jgriffithmaybe23:01
hemnahrmm yah...trying to think through that for our drivers, what the side affect would be23:01
jgriffithOk... let me take a look at it.  I think I'll start playing around with detach.  And yeah, def look for double calls23:01
hemnait might be ok, but would have to test it against our array.23:01
hemnaif only we had 3rd party CI setup....23:01
hemna:P23:01
*** nellysmitt has joined #openstack-cinder23:02
jgriffithI've got a few other things to finish up but hopefully I can get something going in the next couple of days23:02
hemnaok, count me in for the reviews and testing.23:02
hemnajgriffith, you rock man.23:03
jgriffithgreat.. thanks fellas23:03
*** tbarron has quit IRC23:03
thingeejgriffith, hemna, DuncanT: kicking off proposed topics for Liberty summit https://etherpad.openstack.org/p/cinder-liberty-proposed-sessions23:04
*** dannywil_ has quit IRC23:04
*** Apoorva_ has joined #openstack-cinder23:06
*** nellysmitt has quit IRC23:07
*** Apoorva has quit IRC23:09
thingeejgriffith, hemna, DuncanT: does april 27th seem like a good time to meet and discuss what is proposed? in case we have extras, and to have time to get things official for the schedule?23:10
hemnathat gives almost a month before the conf.23:11
hemnayah I think that's good23:11
DuncanTSounds good to me23:11
thingeegreat, I'll send out the email23:12
thingeealso I was thinking 5 fishbowl sessions, 8 working sessions and and 2 sprint slots23:12
hemnaover 2 days ?23:13
thingeepreviously we've done 7 fishbowl, 0 working sessions and 1 slot for sprints23:13
*** Guest936 has joined #openstack-cinder23:13
thingee2 sprints mean a whole day. last time we did half23:13
thingeeunless we feel like half is still good23:13
*** Guest936 is now known as annegentle23:13
thingeeI think we went through topics pretty well though23:13
thingeeso maybe half a day is fine23:13
hemnaprobably safer to schedule the day and if we bail early for beer....so be it!23:14
thingeealso keeping in mind people will be burnt out by this point23:14
*** PaulCuzner has joined #openstack-cinder23:14
openstackgerritJohn Griffith proposed openstack/cinder: Remove warnings for long vgs and lvs calls  https://review.openstack.org/15758223:15
*** annashen has quit IRC23:21
openstackgerritJohn Griffith proposed openstack/cinder: Remove retry_deactivation directive on lvremove  https://review.openstack.org/15759023:23
openstackgerritJohn Griffith proposed openstack/cinder: Remove warnings for long vgs and lvs calls  https://review.openstack.org/15758223:25
*** lcurtis has joined #openstack-cinder23:25
*** ebalduf has quit IRC23:27
*** annashen has joined #openstack-cinder23:27
*** dannywilson has joined #openstack-cinder23:30
openstackgerritTomoki Sekiyama proposed openstack/cinder: Failover to alternative iSCSI portals on login failure  https://review.openstack.org/14087723:31
*** Mandell has joined #openstack-cinder23:31
*** annashen_ has joined #openstack-cinder23:33
*** annashen_ has quit IRC23:38
*** thingee has quit IRC23:38
*** ebalduf has joined #openstack-cinder23:44
*** ebalduf has quit IRC23:48
*** markvoelker has quit IRC23:49
anteayaare folks aware that there are tests on cinder from a stackforge repo called sheepdog?23:52
anteayahttps://review.openstack.org/#/c/154605/23:52
anteayaI don't yet see that cinder core/ptl has weighed in on these testing changes23:53
*** dustins has quit IRC23:53
anteayaam I wrong and you know and you want this?23:53
*** mriedem has joined #openstack-cinder23:53
anteayahttps://review.openstack.org/#/c/153868/223:53
*** asmith_brcd has quit IRC23:54
sigmavirus24anteaya: can you email the ML? The PTL isn't currently online and I'm not active enough to be able to speak on behalf of the project23:54
sigmavirus24Thanks for thinking of us anteaya23:54
anteayawell I can -1 the change for now23:54
anteayaand we can back out the other one if we need to23:54
anteayaI think I would like to hear from others in the channel before I move to the ml23:55
anteayabut sigmavirus24 you are saying this is news to you?23:55
sigmavirus24jgriffith: ^23:55
*** annashen has quit IRC23:55
sigmavirus24anteaya: I'm very barely involved in the project so I wouldn't take my level of awareness as an indicator of the project as a whole or its cores23:55
anteayasigmavirus24: thank you for stating your perspective and its context23:56
anteayaI appreciate it23:56
patrickeastthis is the first time i’ve heard of it, although i’m not really in the know23:56
* patrickeast just lurks here23:56
anteayawell two lurkers count23:56
* sigmavirus24 suspects jgriffith might have heard of it 23:56
anteayaso it hasen't had much chat if it is known23:56
*** annashen has joined #openstack-cinder23:56
patrickeastyea nothing in the cinder meetings or anything from what i can recall23:57
patrickeastunless it was a long time ago23:57
anteayapatrickeast: thanks23:57
anteayano, sheepdog is new23:57
* sigmavirus24 searches mailing list for prior mention23:57
anteayathis release and possibly since january23:57
sigmavirus24no results since June when I first subb'd23:57
hemnado we care about sheepdog plugin for devstack ?23:58
*** dannywilson has quit IRC23:58
anteayaI don't23:58
anteayadoes cinder?23:58
anteayaand it certainly would set an odd precent if a stackforge repo was voting on cinder gate23:59
anteayawhich sounds like their intent23:59
anteayanow if cinder wants this, then okay23:59
anteayabut it doesn't sound so far like cinder knows anything about this23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!