*** jwcroppe has quit IRC | 00:00 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Update the documentation for os-brick https://review.openstack.org/157202 | 00:05 |
---|---|---|
*** lcurtis has quit IRC | 00:08 | |
hemna | thingee, jaypipes ping ? | 00:09 |
*** jordandh has quit IRC | 00:09 | |
*** patrickeast has quit IRC | 00:10 | |
openstackgerrit | Jay Bryant proposed openstack/cinder-specs: Fix the volume replication spec https://review.openstack.org/152735 | 00:14 |
*** Sam-I-Am has left #openstack-cinder | 00:14 | |
*** jordandh has joined #openstack-cinder | 00:15 | |
thingee | hemna: i'm here | 00:16 |
thingee | not for long though... | 00:17 |
hemna | I'm not sure jaypipes is around | 00:17 |
* thingee sitting at a bar with a laptop, wtf | 00:17 | |
hemna | but he and I had a chat this morning about multiattach | 00:17 |
hemna | basically the nova side isn't landing in K | 00:17 |
hemna | but we would still like to get the cinder side to land if possible | 00:17 |
thingee | do drivers need to do anything for it to be useful? | 00:17 |
hemna | no | 00:17 |
sigmavirus24 | thingee: sitting at a bar on a laptop is perfectly normal | 00:17 |
thingee | sigmavirus24: not in LA. maybe in the bay area, but no LA. People are looking at me weird | 00:18 |
hemna | nova's overhead/schedule/beuracracy makes it very difficult to get cross project stuff to land | 00:18 |
thingee | hemna: seems fine to me then | 00:18 |
sigmavirus24 | thingee: oh I didn't say people wouldn't give you weird looks, just that it's a normal thing to do in general | 00:18 |
sigmavirus24 | If it weren't, I'd feel pretty silly | 00:18 |
sigmavirus24 | =P | 00:18 |
thingee | :) | 00:19 |
hemna | hehe | 00:19 |
akerr | depends if you're also drinking or not | 00:19 |
*** nellysmitt has joined #openstack-cinder | 00:19 | |
thingee | akerr: I am. but want to order a second one. | 00:19 |
*** markvoelker has quit IRC | 00:19 | |
thingee | and maybe socialize a bit | 00:19 |
hemna | ok, so I'll just continue to get reviews on it then and see if we can get it in K | 00:19 |
hemna | and nova will land in L | 00:19 |
akerr | be a bit weirder if you just walked in and set up shop | 00:19 |
thingee | hemna: sounds fine to me | 00:19 |
thingee | should 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 |
akerr | in my experience 4s usually become 8s after a few drinks | 00:21 |
*** IanGovett has quit IRC | 00:22 | |
hemna | thingee, ok thanks man. | 00:22 |
thingee | btw people https://twitter.com/Thingee/status/568169188239511552 | 00:22 |
sigmavirus24 | thingee: socialize with us! =P | 00:22 |
hemna | thingee, nice! | 00:23 |
thingee | jgriffith, jungleboyj, xyang, smcginnis, xyang ^ | 00:24 |
*** nellysmitt has quit IRC | 00:24 | |
hemna | thingee, if you can review https://review.openstack.org/#/c/157202 for me, then I think we might be ready for a first drop of brick | 00:24 |
hemna | which would enable me to update requirements.txt and get the cinder brick patch out of WIP | 00:25 |
hemna | I'm not sure how the releases and versioning are done | 00:25 |
hemna | I might have gone overboard on the docs, dunno. | 00:25 |
thingee | hemna: ok, either tonight or tomorrow, is that fine? | 00:26 |
hemna | yup no worries | 00:26 |
hemna | thanks for the help | 00:26 |
hemna | I just want to try and get this in for K if possible, so we can move on. | 00:26 |
thingee | hemna: I'm a bit done for today :) ... want to have some drinks and talk to some folks here | 00:26 |
openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Backup object https://review.openstack.org/157085 | 00:26 |
thingee | and dinner soon..I've barely ate today | 00:27 |
*** fischerw has quit IRC | 00:27 | |
jungleboyj | thingee: I like it! | 00:27 |
jungleboyj | thingee: That sounds like a good plan. | 00:28 |
*** jkremer has quit IRC | 00:35 | |
hemna | jungleboyj, are you guys freezing over out there? | 00:37 |
*** thingee has quit IRC | 00:37 | |
*** hemna is now known as hemnafk | 00:38 | |
*** Mandell has joined #openstack-cinder | 00:39 | |
*** david-lyle is now known as david-lyle_afk | 00:41 | |
*** ebalduf has joined #openstack-cinder | 00:48 | |
*** markvoelker has joined #openstack-cinder | 00:52 | |
*** rmesta has quit IRC | 00:53 | |
*** ebalduf has quit IRC | 00:53 | |
*** rmesta has joined #openstack-cinder | 00:54 | |
*** rmesta has quit IRC | 00:54 | |
*** theanalyst has quit IRC | 00:55 | |
*** annashen has quit IRC | 00:56 | |
*** dannywilson has quit IRC | 00:56 | |
*** theanalyst has joined #openstack-cinder | 00:58 | |
*** markvoelker has quit IRC | 00:59 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 01:02 | |
*** vishy has quit IRC | 01:07 | |
*** vishy has joined #openstack-cinder | 01:08 | |
*** patrickeast has joined #openstack-cinder | 01:10 | |
*** tsekiyam_ has joined #openstack-cinder | 01:16 | |
*** Apoorva_ has quit IRC | 01:18 | |
*** Apoorva has joined #openstack-cinder | 01:19 | |
*** tsekiyama has quit IRC | 01:20 | |
*** tsekiyam_ has quit IRC | 01:21 | |
*** mtanino has quit IRC | 01:25 | |
*** Yogi11 has quit IRC | 01:28 | |
*** markvoelker has joined #openstack-cinder | 01:29 | |
*** jwcroppe has joined #openstack-cinder | 01:33 | |
*** jgravel has quit IRC | 01:34 | |
*** 64MABZ0GB is now known as strictlyb | 01:36 | |
*** EmilienM is now known as EmilienM|afk | 01:36 | |
*** garthb has quit IRC | 01:37 | |
*** garthb_ has quit IRC | 01:37 | |
*** Yogi1 has joined #openstack-cinder | 01:40 | |
*** IanGovett has joined #openstack-cinder | 01:47 | |
*** duyt1001 has joined #openstack-cinder | 01:50 | |
*** IanGovett has quit IRC | 01:54 | |
*** patrickeast has quit IRC | 01:54 | |
*** devlaps has quit IRC | 01:55 | |
*** jwcroppe has quit IRC | 01:57 | |
*** jwcroppe has joined #openstack-cinder | 01:57 | |
*** jwcroppe has quit IRC | 02:02 | |
*** scottda_ has joined #openstack-cinder | 02:02 | |
*** vilobhmm has quit IRC | 02:03 | |
*** vilobhmm2 has quit IRC | 02:03 | |
*** mdbooth has quit IRC | 02:06 | |
*** jwcroppe has joined #openstack-cinder | 02:07 | |
*** jwcroppe has quit IRC | 02:08 | |
*** krtaylor has joined #openstack-cinder | 02:09 | |
*** jwcroppe has joined #openstack-cinder | 02:09 | |
*** scottda_ has quit IRC | 02:09 | |
*** mdbooth has joined #openstack-cinder | 02:11 | |
*** jwcroppe has quit IRC | 02:13 | |
*** leeantho has quit IRC | 02:14 | |
*** scottda_ has joined #openstack-cinder | 02:17 | |
*** ebalduf has joined #openstack-cinder | 02:19 | |
*** nellysmitt has joined #openstack-cinder | 02:20 | |
*** ebalduf has quit IRC | 02:24 | |
*** Mandell has quit IRC | 02:24 | |
*** nellysmitt has quit IRC | 02:24 | |
*** Mandell has joined #openstack-cinder | 02:25 | |
*** duyt1001 has left #openstack-cinder | 02:27 | |
*** _cjones_ has quit IRC | 02:30 | |
openstackgerrit | Gloria Gu proposed openstack/cinder: HP 3par driver filter and evaluator function https://review.openstack.org/151353 | 02:35 |
*** Apoorva_ has joined #openstack-cinder | 02:35 | |
*** ybathia has quit IRC | 02:36 | |
*** Yogi1 has quit IRC | 02:37 | |
openstackgerrit | Gloria Gu proposed openstack/cinder: HP lefthand driver filter and evalautor function https://review.openstack.org/157237 | 02:38 |
*** Apoorva has quit IRC | 02:38 | |
*** Apoorva_ has quit IRC | 02:39 | |
*** scottda_ has quit IRC | 02:43 | |
*** Yogi1 has joined #openstack-cinder | 02:47 | |
*** annashen has joined #openstack-cinder | 02:48 | |
*** kaisers1 has joined #openstack-cinder | 02:51 | |
*** mriedem has joined #openstack-cinder | 02:53 | |
*** kaisers has quit IRC | 02:53 | |
*** jecarey has joined #openstack-cinder | 02:54 | |
*** annashen has quit IRC | 03:08 | |
*** Yogi1 has quit IRC | 03:13 | |
*** harlowja_ is now known as harlowja_away | 03:15 | |
*** jecarey has quit IRC | 03:28 | |
*** jecarey has joined #openstack-cinder | 03:28 | |
*** ebalduf has joined #openstack-cinder | 03:35 | |
*** scottda_ has joined #openstack-cinder | 03:37 | |
*** scottda_ has joined #openstack-cinder | 03:38 | |
*** ebalduf has quit IRC | 03:39 | |
*** scottda_ has quit IRC | 03:48 | |
*** kaufer has joined #openstack-cinder | 03:48 | |
*** scottda_ has joined #openstack-cinder | 03:50 | |
*** Mandell has quit IRC | 03:58 | |
openstackgerrit | Jeegn Chen proposed openstack/cinder: Attach/detach batch processing in VNX driver https://review.openstack.org/154350 | 03:59 |
*** akerr is now known as akerr_away | 04:01 | |
jungleboyj | hemnafk: It is only -4. ;-) | 04:01 |
jungleboyj | hemnafk: Still not wearing my winter coat. | 04:01 |
*** MentalRay has joined #openstack-cinder | 04:13 | |
*** annashen has joined #openstack-cinder | 04:16 | |
*** nellysmitt has joined #openstack-cinder | 04:21 | |
*** nellysmitt has quit IRC | 04:25 | |
*** setmason has quit IRC | 04:27 | |
*** coolsvap_ is now known as coolsvap | 04:28 | |
*** btran has quit IRC | 04:28 | |
xyang | thingee: That looks good! | 04:31 |
*** jwcroppe has joined #openstack-cinder | 04:32 | |
*** Mandell has joined #openstack-cinder | 04:34 | |
*** jecarey has quit IRC | 04:35 | |
*** changbl has joined #openstack-cinder | 04:38 | |
*** tbarron has quit IRC | 04:40 | |
*** annashen has quit IRC | 04:41 | |
*** markvoelker has quit IRC | 04:42 | |
*** markvoelker has joined #openstack-cinder | 04:42 | |
*** Mandell has quit IRC | 04:42 | |
*** Mandell has joined #openstack-cinder | 04:43 | |
*** annashen has joined #openstack-cinder | 04:45 | |
*** annashen has quit IRC | 04:46 | |
*** markvoelker has quit IRC | 04:46 | |
*** annashen has joined #openstack-cinder | 04:47 | |
*** BharatK has joined #openstack-cinder | 04:52 | |
openstackgerrit | Steven Kaufer proposed openstack/cinder: Generic filter support for volume queries https://review.openstack.org/155343 | 04:55 |
openstackgerrit | Steven Kaufer proposed openstack/cinder: Replication status periodic task optimization https://review.openstack.org/154673 | 04:55 |
*** _cjones_ has joined #openstack-cinder | 04:59 | |
*** kaufer has quit IRC | 04:59 | |
*** asselin_ has joined #openstack-cinder | 05:00 | |
*** scottda_ has quit IRC | 05:00 | |
*** annashen has quit IRC | 05:00 | |
*** murugan has joined #openstack-cinder | 05:02 | |
*** asselin_ has quit IRC | 05:02 | |
*** mriedem has quit IRC | 05:05 | |
*** coolsvap is now known as coolsvap_ | 05:05 | |
*** mwichmann has quit IRC | 05:07 | |
*** mriedem has joined #openstack-cinder | 05:10 | |
*** mriedem has left #openstack-cinder | 05:11 | |
*** markvoelker has joined #openstack-cinder | 05:13 | |
*** _cjones_ has quit IRC | 05:15 | |
*** coolsvap_ is now known as coolsvap | 05:15 | |
*** _cjones_ has joined #openstack-cinder | 05:15 | |
*** markvoelker has quit IRC | 05:17 | |
*** I has joined #openstack-cinder | 05:18 | |
*** I is now known as Guest98537 | 05:18 | |
*** pdeore_ has joined #openstack-cinder | 05:20 | |
*** Guest98537 has quit IRC | 05:22 | |
pdeore_ | Hello, jenkins is failing for one of my patch | 05:26 |
pdeore_ | https://review.openstack.org/#/c/156936/ | 05:26 |
pdeore_ | I have ensured that tests are completing successfully in my local environment | 05:27 |
pdeore_ | I have also verified python26 and python27 jenkins logs | 05:27 |
pdeore_ | http://logs.openstack.org/36/156936/1/check/gate-cinder-python26/1afcea7/console.html | 05: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-cinder | 05:38 | |
*** sandywalsh_ has quit IRC | 05:40 | |
*** Ilja has joined #openstack-cinder | 05:46 | |
*** deepakcs has joined #openstack-cinder | 05:47 | |
*** zigo has quit IRC | 05:54 | |
*** zigo has joined #openstack-cinder | 05:55 | |
*** pradipta has joined #openstack-cinder | 05:57 | |
*** _cjones_ has quit IRC | 06:00 | |
openstackgerrit | Joe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager https://review.openstack.org/157265 | 06:02 |
*** Mandell has quit IRC | 06:04 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/152401 | 06:09 |
*** cnesa1 has quit IRC | 06:10 | |
*** annashen has joined #openstack-cinder | 06:13 | |
*** markvoelker has joined #openstack-cinder | 06:13 | |
*** markvoelker has quit IRC | 06:18 | |
*** Mandell has joined #openstack-cinder | 06:19 | |
*** nellysmitt has joined #openstack-cinder | 06:22 | |
*** ankit_ag has joined #openstack-cinder | 06:23 | |
*** nellysmitt has quit IRC | 06:26 | |
*** ronenkat has quit IRC | 06:27 | |
*** ronenkat__ has quit IRC | 06:27 | |
*** ronenkat has joined #openstack-cinder | 06:28 | |
*** ronenkat_ has joined #openstack-cinder | 06:29 | |
*** sgotliv has joined #openstack-cinder | 06:32 | |
*** dannywilson has joined #openstack-cinder | 06:34 | |
*** pradeep_s has left #openstack-cinder | 06:41 | |
*** dannywilson has quit IRC | 06:41 | |
*** ronenkat_ has quit IRC | 06:42 | |
*** ronenkat has quit IRC | 06:42 | |
*** dannywilson has joined #openstack-cinder | 06:42 | |
*** nkrinner has joined #openstack-cinder | 06:46 | |
*** atmark has quit IRC | 06:50 | |
*** Lee1092 has joined #openstack-cinder | 06:51 | |
*** harlowja_at_home has joined #openstack-cinder | 06:52 | |
*** Mandell has quit IRC | 06:52 | |
*** Mandell has joined #openstack-cinder | 06:58 | |
*** bill_az has joined #openstack-cinder | 07:05 | |
*** bill_ibm__ has quit IRC | 07:05 | |
*** _cjones_ has joined #openstack-cinder | 07:07 | |
*** afazekas has joined #openstack-cinder | 07:08 | |
*** Ilja has quit IRC | 07:08 | |
*** Ilja has joined #openstack-cinder | 07:09 | |
*** harlowja_at_home has quit IRC | 07:13 | |
*** Ilja has quit IRC | 07:14 | |
*** markvoelker has joined #openstack-cinder | 07:14 | |
*** sgotliv has quit IRC | 07:16 | |
*** chlong has quit IRC | 07:19 | |
*** markvoelker has quit IRC | 07:19 | |
*** ho has quit IRC | 07:26 | |
*** sgotliv has joined #openstack-cinder | 07:26 | |
*** TobiasE has joined #openstack-cinder | 07:26 | |
*** nshaikh has joined #openstack-cinder | 07:27 | |
*** anshul has joined #openstack-cinder | 07:27 | |
*** annashen has quit IRC | 07:27 | |
openstackgerrit | Joe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager https://review.openstack.org/157265 | 07:29 |
*** _cjones_ has quit IRC | 07:31 | |
*** MentalRay has quit IRC | 07:36 | |
*** rluediger has joined #openstack-cinder | 07:59 | |
rluediger | good morning all together DuncanT are you already online? | 08:00 |
*** annashen has joined #openstack-cinder | 08:01 | |
*** harlowja_away has quit IRC | 08:03 | |
*** annashen has quit IRC | 08:05 | |
*** ronenkat has joined #openstack-cinder | 08:06 | |
*** ronenkat_ has joined #openstack-cinder | 08:06 | |
*** cnesa has joined #openstack-cinder | 08:09 | |
*** ronenkat has quit IRC | 08:13 | |
*** ronenkat_ has quit IRC | 08:14 | |
*** markvoelker has joined #openstack-cinder | 08:15 | |
*** markvoelker has quit IRC | 08:20 | |
*** nellysmitt has joined #openstack-cinder | 08:22 | |
*** atmark1 has joined #openstack-cinder | 08:23 | |
*** nellysmitt has quit IRC | 08:27 | |
*** Ilja has joined #openstack-cinder | 08:31 | |
*** Ilja has quit IRC | 08:31 | |
*** murugan has quit IRC | 08:34 | |
*** ronis has joined #openstack-cinder | 08:34 | |
*** alecv has joined #openstack-cinder | 08:35 | |
*** nellysmitt has joined #openstack-cinder | 08:36 | |
*** pradipta has quit IRC | 08:37 | |
*** jistr has joined #openstack-cinder | 08:42 | |
*** dulek has joined #openstack-cinder | 08:50 | |
*** rushiagr_away is now known as rushiagr | 08:52 | |
*** jordanP has joined #openstack-cinder | 08:53 | |
*** xyang1 has quit IRC | 09:03 | |
*** markvoelker has joined #openstack-cinder | 09:16 | |
*** dannywilson has quit IRC | 09:18 | |
*** ndipanov has quit IRC | 09:21 | |
*** markvoelker has quit IRC | 09:21 | |
*** abhishekk has joined #openstack-cinder | 09:25 | |
abhishekk | hi 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.c | 09:27 |
abhishekk | onf 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=direct | 09:27 |
abhishekk | Reference: https://jenkins01.openstack.org/job/gate-cinder-python26/7995/console | 09:27 |
abhishekk | can any one help me to understand the cause of failure? | 09:28 |
*** enterprisedc has quit IRC | 09:28 | |
*** ndipanov has joined #openstack-cinder | 09:29 | |
*** enterprisedc has joined #openstack-cinder | 09:30 | |
DuncanT | abhishekk: 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 this | 09:53 |
DuncanT | Oh, wait, unit tests? | 09:53 |
DuncanT | The execute call should be mocked. Not sure what is going on there, sorry | 09:54 |
DuncanT | rluediger: Good morning | 09:54 |
abhishekk | DuncanT: thank you, unittests are completing successfully in local environment, so on jenkins | 09:55 |
abhishekk | DuncanT: Not sure why it's giving error later while running the command with sudo | 09:56 |
DuncanT | abhishekk: Is the user you're running them as locally in sudoers | 09:59 |
abhishekk | DuncanT: yes | 10:00 |
DuncanT | So I suspect jenkins isn't, and one of the tests is missing a mock | 10:00 |
rluediger | DuncanT: 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 c | 10:01 |
rluediger | onsumed by cinder-scheduler but nothinh happens on the storage node | 10:01 |
DuncanT | abhishekk: Adding jenkins to sudoers is the quick fix, fixing the test would be better | 10:01 |
rluediger | DuncanT: can you advise how to get more info from the cinder-scheduler what he is doing? | 10:02 |
abhishekk | DuncanT: Thanks for the input, I will check for it | 10:02 |
DuncanT | rluediger: 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 out | 10:02 |
DuncanT | rluediger: Sorry I can't be more help, but I don't even know where to start with rabbit misbehaviour | 10:03 |
*** karimb has joined #openstack-cinder | 10:03 | |
rluediger | DuncanT: ok. thx for your help so far I'll try to get help @ the #openstack channel | 10:04 |
DuncanT | rluediger: Good luck | 10:06 |
*** Mandell has quit IRC | 10:06 | |
*** yuriy_n17 has joined #openstack-cinder | 10:07 | |
openstackgerrit | Joe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager https://review.openstack.org/157265 | 10:09 |
abhishekk | DuncanT: This patch is a backport to stable/juno, I haven't got this error on master, so I am more confused about this | 10:09 |
*** Lee1092 has quit IRC | 10:10 | |
DuncanT | abhishekk: 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 |
abhishekk | DuncanT: https://review.openstack.org/#/c/156936/1 | 10:11 |
*** markvoelker has joined #openstack-cinder | 10:17 | |
*** e0ne has joined #openstack-cinder | 10:21 | |
*** markvoelker has quit IRC | 10:22 | |
*** briancline has quit IRC | 10:22 | |
*** briancline has joined #openstack-cinder | 10:23 | |
jordanP | hi 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-cinder | 10:31 | |
*** _cjones_ has quit IRC | 10:36 | |
*** e0ne is now known as e0ne_ | 10:39 | |
*** e0ne_ is now known as e0ne | 10:39 | |
openstackgerrit | Joe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager https://review.openstack.org/157265 | 10:42 |
openstackgerrit | Joe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager https://review.openstack.org/157265 | 10:44 |
*** e0ne is now known as e0ne_ | 10:53 | |
*** e0ne_ is now known as e0ne | 10:54 | |
*** Swanson has quit IRC | 11:02 | |
*** smcginnis has quit IRC | 11:03 | |
*** jseiler__ has quit IRC | 11:03 | |
*** aix has quit IRC | 11:04 | |
*** ndipanov has quit IRC | 11:04 | |
*** aix has joined #openstack-cinder | 11:05 | |
*** jseiler has joined #openstack-cinder | 11:09 | |
*** smcginnis has joined #openstack-cinder | 11:09 | |
*** Swanson has joined #openstack-cinder | 11:10 | |
*** Ilja has joined #openstack-cinder | 11:10 | |
*** markvoelker has joined #openstack-cinder | 11:18 | |
*** ndipanov has joined #openstack-cinder | 11:18 | |
*** nlevinki has joined #openstack-cinder | 11:22 | |
*** markvoelker has quit IRC | 11:23 | |
*** dulek has quit IRC | 11:29 | |
*** dulek has joined #openstack-cinder | 11:30 | |
*** e0ne is now known as e0ne_ | 11:34 | |
*** deepakcs has quit IRC | 11:38 | |
*** pschaef has joined #openstack-cinder | 11:40 | |
*** chlong has joined #openstack-cinder | 11:41 | |
*** ganso_ has joined #openstack-cinder | 11:47 | |
*** marcusvrn1 has quit IRC | 11:49 | |
*** marcusvrn has joined #openstack-cinder | 11:50 | |
*** alexpilotti has joined #openstack-cinder | 11:51 | |
*** marcusvrn1 has joined #openstack-cinder | 11:52 | |
*** rushiagr is now known as rushiagr_away | 11:54 | |
*** marcusvrn has quit IRC | 11:56 | |
*** rluediger has quit IRC | 11:59 | |
*** rluediger has joined #openstack-cinder | 12:02 | |
*** Lee1092 has joined #openstack-cinder | 12:04 | |
*** agentle_ has quit IRC | 12:06 | |
*** diegows has joined #openstack-cinder | 12:08 | |
*** nlevinki_ has joined #openstack-cinder | 12:10 | |
*** e0ne_ is now known as e0ne | 12:10 | |
*** nlevinki has quit IRC | 12:13 | |
*** ndipanov has quit IRC | 12:13 | |
*** ndipanov has joined #openstack-cinder | 12:14 | |
*** IanGovett has joined #openstack-cinder | 12:15 | |
*** dannywilson has joined #openstack-cinder | 12:19 | |
*** markvoelker has joined #openstack-cinder | 12:19 | |
*** ndipanov has quit IRC | 12:20 | |
*** dannywilson has quit IRC | 12:23 | |
*** markvoelker has quit IRC | 12:24 | |
*** jseiler has quit IRC | 12:25 | |
*** smcginnis has quit IRC | 12:25 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: Backup object https://review.openstack.org/157085 | 12:25 |
*** Swanson has quit IRC | 12:25 | |
*** TobiasE has left #openstack-cinder | 12:30 | |
*** ndipanov has joined #openstack-cinder | 12:32 | |
*** nlevinki_ has quit IRC | 12:37 | |
*** Yogi1 has joined #openstack-cinder | 12:43 | |
openstackgerrit | nikeshmahalka proposed openstack/cinder: Add CHAP persistence to SCST target helper https://review.openstack.org/157356 | 12:47 |
*** avishay has joined #openstack-cinder | 12:57 | |
*** akerr_away is now known as akerr | 12:57 | |
*** EmilienM|afk is now known as EmilienM | 12:58 | |
*** akerr has quit IRC | 12:58 | |
*** marcusvrn has joined #openstack-cinder | 13:01 | |
*** Yogi1 has quit IRC | 13:01 | |
*** ankit_ag has quit IRC | 13:02 | |
*** markvoelker has joined #openstack-cinder | 13:02 | |
*** marcusvrn1 has quit IRC | 13:04 | |
*** alexpilotti has quit IRC | 13:05 | |
*** BharatK has quit IRC | 13:12 | |
*** Yogi1 has joined #openstack-cinder | 13:17 | |
*** cnesa has quit IRC | 13:23 | |
DuncanT | abhishekk: 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 culprit | 13:23 |
*** cnesa has joined #openstack-cinder | 13:24 | |
*** bswartz has quit IRC | 13:27 | |
*** cnesa has quit IRC | 13:28 | |
abhishekk | DuncanT: Thanks alot, I will do the same | 13:28 |
DuncanT | abhishekk: If you find something, please do let me know. Thanks. | 13:29 |
abhishekk | DuncanT: Sure | 13:29 |
*** aix has quit IRC | 13:29 | |
*** rluediger has quit IRC | 13:33 | |
*** akerr has joined #openstack-cinder | 13:35 | |
*** Yogi1 has quit IRC | 13:36 | |
*** jistr_ has joined #openstack-cinder | 13:37 | |
*** Ilja has quit IRC | 13:37 | |
*** Ilja has joined #openstack-cinder | 13:38 | |
*** jistr has quit IRC | 13:39 | |
*** bkopilov has joined #openstack-cinder | 13:40 | |
*** haomaiw__ has quit IRC | 13:41 | |
*** xyang has quit IRC | 13:41 | |
*** smcginnis has joined #openstack-cinder | 13:42 | |
*** markvoelker_ has joined #openstack-cinder | 13:42 | |
*** markvoelker has quit IRC | 13:42 | |
*** briancline has quit IRC | 13:48 | |
*** jistr_ is now known as jistr | 13:48 | |
*** briancline has joined #openstack-cinder | 13:49 | |
*** timcl has joined #openstack-cinder | 13:49 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:52 | |
*** Adriano_ has quit IRC | 13:52 | |
*** kaufer has joined #openstack-cinder | 13:54 | |
*** mkerrin has quit IRC | 13:55 | |
*** xyang has joined #openstack-cinder | 13:57 | |
*** BharatK has joined #openstack-cinder | 13:57 | |
*** pschaef has quit IRC | 13:58 | |
*** aix has joined #openstack-cinder | 14:01 | |
*** bswartz has joined #openstack-cinder | 14:01 | |
*** Yogi1 has joined #openstack-cinder | 14:03 | |
*** kaisers1 has quit IRC | 14:04 | |
*** Mandell has joined #openstack-cinder | 14:07 | |
*** BharatK has quit IRC | 14:08 | |
*** kaisers has joined #openstack-cinder | 14:11 | |
*** Mandell has quit IRC | 14:12 | |
*** nlevinki has joined #openstack-cinder | 14:14 | |
*** xyang1 has joined #openstack-cinder | 14:15 | |
*** rushil has joined #openstack-cinder | 14:18 | |
*** dustins has joined #openstack-cinder | 14:20 | |
*** BharatK has joined #openstack-cinder | 14:20 | |
*** tbarron has joined #openstack-cinder | 14:21 | |
*** jungleboyj has quit IRC | 14:27 | |
*** mriedem has joined #openstack-cinder | 14:27 | |
*** mwichmann_ has joined #openstack-cinder | 14:28 | |
*** Yogi1 has quit IRC | 14:31 | |
*** karimb has quit IRC | 14:32 | |
*** EmilienM is now known as EmilienM|afk | 14:32 | |
*** scottda has quit IRC | 14:34 | |
*** Yogi1 has joined #openstack-cinder | 14:34 | |
*** timcl has quit IRC | 14:36 | |
*** briancline has quit IRC | 14:37 | |
*** markvoelker_ has quit IRC | 14:38 | |
*** briancline has joined #openstack-cinder | 14:38 | |
*** markvoelker has joined #openstack-cinder | 14:39 | |
*** rluediger has joined #openstack-cinder | 14:39 | |
*** timcl has joined #openstack-cinder | 14:40 | |
rluediger | DuncanT: 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 IRC | 14:42 | |
DuncanT | rluediger: Well, glad it is working. It would have been nice to know the answer, it was a weird one | 14:43 |
*** dustins has quit IRC | 14:43 | |
rluediger | DuncanT: yepp I totally agree that | 14:43 |
*** bswartz has quit IRC | 14:43 | |
*** marcusvrn1 has joined #openstack-cinder | 14:45 | |
*** marcusvrn has quit IRC | 14:46 | |
*** marcusvrn has joined #openstack-cinder | 14:46 | |
openstackgerrit | Mikhail Khodos proposed openstack/cinder: Get the right LUN from mapping https://review.openstack.org/157390 | 14:46 |
openstackgerrit | Anton Arefiev proposed openstack/cinder: Import only modules: H302 https://review.openstack.org/145780 | 14:46 |
*** marcusvrn has quit IRC | 14:47 | |
*** marcusvrn has joined #openstack-cinder | 14:47 | |
*** Yogi1 has quit IRC | 14:47 | |
*** akerr has quit IRC | 14:49 | |
*** scottda has joined #openstack-cinder | 14:49 | |
*** marcusvrn1 has quit IRC | 14:49 | |
*** deepakcs has joined #openstack-cinder | 14:50 | |
openstackgerrit | Mikhail Khodos proposed openstack/cinder: NexentaStor SA: get the right LUN from mapping https://review.openstack.org/157390 | 14:55 |
jbernard | rhe00: let me know if i can help with anything | 14:55 |
*** akerr has joined #openstack-cinder | 14:58 | |
*** bswartz has joined #openstack-cinder | 14:59 | |
*** Mandell has joined #openstack-cinder | 14:59 | |
*** juzuluag has joined #openstack-cinder | 15:01 | |
*** Yogi1 has joined #openstack-cinder | 15:04 | |
*** dustins has joined #openstack-cinder | 15:04 | |
*** dannywilson has joined #openstack-cinder | 15:05 | |
*** ebalduf has joined #openstack-cinder | 15:06 | |
*** primechuck has joined #openstack-cinder | 15:09 | |
*** dannywilson has quit IRC | 15:09 | |
*** ebalduf has quit IRC | 15:10 | |
*** amoturi has joined #openstack-cinder | 15:10 | |
*** am1 has joined #openstack-cinder | 15:12 | |
*** am1 has quit IRC | 15:12 | |
*** delattec has joined #openstack-cinder | 15:14 | |
*** cdelatte has joined #openstack-cinder | 15:14 | |
*** jecarey has joined #openstack-cinder | 15:16 | |
*** lcurtis has joined #openstack-cinder | 15:18 | |
*** jcru has joined #openstack-cinder | 15:19 | |
*** pdeore_ has quit IRC | 15:21 | |
*** ebalduf has joined #openstack-cinder | 15:25 | |
*** rluediger has quit IRC | 15:26 | |
*** TobiasE has joined #openstack-cinder | 15:27 | |
*** Ilja has quit IRC | 15:28 | |
*** mtanino has joined #openstack-cinder | 15:29 | |
*** Ilja has joined #openstack-cinder | 15:30 | |
*** ebalduf has quit IRC | 15:30 | |
*** rmesta has joined #openstack-cinder | 15:34 | |
*** mwichmann_ has quit IRC | 15:35 | |
*** Mandell has quit IRC | 15:37 | |
*** ebalduf has joined #openstack-cinder | 15:40 | |
mtanino | avishay: ping | 15:41 |
*** EmilienM|afk is now known as EmilienM | 15:41 | |
*** MasterPiece has joined #openstack-cinder | 15:41 | |
openstackgerrit | Steven Kaufer proposed openstack/cinder: GET volumes API sorting REST/volume/DB updates https://review.openstack.org/141915 | 15:46 |
openstackgerrit | Steven Kaufer proposed openstack/cinder: GET volumes API sorting enhancements common utilities https://review.openstack.org/141914 | 15:46 |
*** Lee1092 has quit IRC | 15:50 | |
*** jaypipes has quit IRC | 15:51 | |
*** Ilja1 has joined #openstack-cinder | 15:52 | |
*** thangp has joined #openstack-cinder | 15:53 | |
*** hemnafk is now known as hemna | 15:53 | |
*** Ilja has quit IRC | 15:53 | |
*** mwichmann has joined #openstack-cinder | 15:55 | |
*** Ilja1 has quit IRC | 15:55 | |
*** Ilja has joined #openstack-cinder | 15:57 | |
*** nellysmitt has quit IRC | 15:58 | |
TobiasE | Hi, thingee, xyang, jgriffith would like to get your opinion/ comments on https://review.openstack.org/#/c/154377/ | 15:59 |
dulek | thangp: Hi | 16:00 |
thangp | dulek: yo | 16:01 |
*** jaypipes has joined #openstack-cinder | 16:01 | |
dulek | thangp: I've answered your comment on base objects patch, is my answer clear? | 16:01 |
*** Ilja has quit IRC | 16:01 | |
*** diegows has quit IRC | 16:01 | |
dulek | thangp: I haven't meant moving linstack changes from snapshots patch to the base. | 16:01 |
xyang | TobiasE: hi, sure, I'll take a look | 16:01 |
*** kallebe has joined #openstack-cinder | 16:02 | |
dulek | thangp: I just meant that you're overwriting base change in snapshots patch, which isn't necessary. | 16:02 |
dulek | thangp: Got to go now, so I want to make sure I'm not being misunderstood. | 16:03 |
thangp | dulek: ok...the answer didnt show up on the patch...but i understand | 16:03 |
thangp | dulek: thx | 16:03 |
*** Ilja has joined #openstack-cinder | 16:03 | |
dulek | thangp: Aww, that's because I haven't clicked submit, my bad. | 16:03 |
thangp | dulek: lol | 16:03 |
*** annegentle has joined #openstack-cinder | 16:04 | |
dulek | thangp: fixed. ;) Thanks! | 16:04 |
*** david-lyle_afk is now known as david-lyle | 16:04 | |
*** afazekas has quit IRC | 16:05 | |
*** nellysmitt has joined #openstack-cinder | 16:11 | |
TobiasE | xyang: cool | 16:11 |
*** dulek has quit IRC | 16:11 | |
*** deepakcs has quit IRC | 16:12 | |
*** PaulCuzner has joined #openstack-cinder | 16:16 | |
*** annashen has joined #openstack-cinder | 16:16 | |
e0ne | jgriffith: morning. looks like i've got too quick env:(. can't reproduce warning with long lvs run | 16:17 |
kallebe | Hello. 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 t | 16:17 |
kallebe | o anyone else here? I found a person posting in ask openstack the c-api error but no answers there | 16:17 |
e0ne | kallebe: hi! do you have any errors more? it works for me | 16:18 |
*** dannywilson has joined #openstack-cinder | 16:19 | |
kallebe | e0ne hi. This is the only error for now, because it fails the stack.sh script and so devstack cannot run properly | 16:19 |
*** dannywilson has quit IRC | 16:19 | |
kallebe | I already cleaned and everything but it did not work | 16:19 |
*** xyang has quit IRC | 16:19 | |
*** anshul has quit IRC | 16:21 | |
*** MentalRay has joined #openstack-cinder | 16:22 | |
*** marcusvrn1 has joined #openstack-cinder | 16:22 | |
e0ne | kallebe: i ran devstack on a clean vm about 1hour ago. it works for me | 16:22 |
*** dannywilson has joined #openstack-cinder | 16:23 | |
kallebe | e0ne 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 medberry | 16:24 | |
*** medberry is now known as medberry2 | 16:24 | |
*** marcusvrn has quit IRC | 16:24 | |
*** krtaylor has quit IRC | 16:25 | |
openstackgerrit | Thang Pham proposed openstack/cinder: Cinder objects base https://review.openstack.org/131873 | 16:25 |
*** ronis has quit IRC | 16:26 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support https://review.openstack.org/85847 | 16:26 |
*** timcl has quit IRC | 16:27 | |
*** alecv has quit IRC | 16:30 | |
*** delatte has joined #openstack-cinder | 16:30 | |
*** Mandell has joined #openstack-cinder | 16:31 | |
*** delattec has quit IRC | 16:32 | |
*** cdelatte has quit IRC | 16:32 | |
*** cdelatte has joined #openstack-cinder | 16:32 | |
*** nellysmitt has quit IRC | 16:33 | |
*** Ilja has quit IRC | 16:33 | |
openstackgerrit | Joe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager https://review.openstack.org/157265 | 16:34 |
*** nellysmitt has joined #openstack-cinder | 16:35 | |
*** openstackgerrit has quit IRC | 16:36 | |
*** openstackgerrit has joined #openstack-cinder | 16:36 | |
*** cnesa has joined #openstack-cinder | 16:37 | |
*** xyang has joined #openstack-cinder | 16:38 | |
*** tsekiyama has joined #openstack-cinder | 16:39 | |
*** annegentle has quit IRC | 16:39 | |
*** annegentle has joined #openstack-cinder | 16:40 | |
*** MentalRay has quit IRC | 16:42 | |
openstackgerrit | Joe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager https://review.openstack.org/157265 | 16:42 |
*** mwichmann has quit IRC | 16:42 | |
*** PaulCuzner has quit IRC | 16:42 | |
openstackgerrit | Alex Meade proposed openstack/cinder: Add tracing facility to NetApp ONTAP drivers https://review.openstack.org/157439 | 16:43 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Use oslo.log instead oslo-incubator https://review.openstack.org/157441 | 16:45 |
*** nellysmitt has quit IRC | 16:45 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator https://review.openstack.org/157441 | 16:46 |
*** krtaylor has joined #openstack-cinder | 16:47 | |
*** coolsvap is now known as coolsvap_ | 16:57 | |
jwcroppe | hemna: 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 IRC | 17:01 | |
smcginnis | jwcroppe: 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 | ^^^ thx | 17:02 |
hemna | jwcroppe, ok yah just confirmed. | 17:02 |
hemna | it's fine then | 17:02 |
smcginnis | +1 | 17:02 |
jwcroppe | Thanks for the quick review, folks. I think I tackled the comments. | 17:02 |
hemna | it's just useless code churn though | 17:03 |
jwcroppe | yeah, I only removed it since I was in there getting my hands dirty anyway | 17:03 |
smcginnis | I'm generally in favor of removing code when it's not needed. | 17:04 |
smcginnis | Not necessary, but the less code, the less chance of something changing and breaking. | 17:04 |
e0ne | smcginnis: +2 | 17:04 |
jwcroppe | Sounds like we're good with leaving the removal then? | 17:04 |
smcginnis | e0ne: :) | 17:04 |
jwcroppe | smcginnis, hemna: we'll let Jenkins do his thing and then hopefully that one is good to go. Thanks again. | 17:06 |
hemna | 1 more item | 17:06 |
*** TobiasE has quit IRC | 17:09 | |
*** TobiasE has joined #openstack-cinder | 17:11 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Allow archiving deleted rows to shadow tables, for performance https://review.openstack.org/131182 | 17:11 |
smcginnis | hemna: Good eye! I missed two things on one line. :/ | 17:11 |
*** _cjones_ has joined #openstack-cinder | 17:13 | |
*** afazekas has joined #openstack-cinder | 17:13 | |
e0ne | smcginnis: that's why we need at least two reviewer on each patch | 17:13 |
smcginnis | e0ne: Two in addition to me apparently! :) | 17:14 |
e0ne | :) | 17:14 |
hemna | I saw that line and thought....huh, maybe he knows something I don't | 17:14 |
hemna | and had to try it myself in a sample.py file :P | 17:14 |
hemna | oops...nope! | 17:14 |
openstackgerrit | Joe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager https://review.openstack.org/157265 | 17:15 |
*** Apoorva has joined #openstack-cinder | 17:15 | |
smcginnis | hemna: I usually assume people know more than me. ;) | 17:15 |
hemna | same here | 17:15 |
hemna | that's why I had to test it | 17:15 |
* e0ne thoughts that he is onle onr person who tryes some patches on a local env | 17:15 | |
jwcroppe | fixed - actually it was a quick oversight from a previous comment | 17:16 |
jwcroppe | can't use (initiator)s ... initiator is being iterated on | 17:16 |
jwcroppe | e.g., not a static entry, hence why {initiator: val} 'worked' | 17:16 |
*** kallebe has left #openstack-cinder | 17:17 | |
jwcroppe | The structure of those log message was interesting to begin with :) I just removed the dict style. | 17:17 |
*** annashen has quit IRC | 17:17 | |
*** nshaikh has quit IRC | 17:17 | |
*** MentalRay has joined #openstack-cinder | 17:19 | |
*** jecarey has quit IRC | 17:20 | |
*** nellysmitt has joined #openstack-cinder | 17:21 | |
*** annashen has joined #openstack-cinder | 17:21 | |
*** TobiasE has quit IRC | 17:21 | |
jwcroppe | I 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-cinder | 17:25 | |
*** PaulCuzner has joined #openstack-cinder | 17:26 | |
*** jistr has quit IRC | 17:28 | |
openstackgerrit | Joe Cropper proposed openstack/cinder: Fix some message nits in the ZoneManager https://review.openstack.org/157265 | 17:28 |
e0ne | jungleboyj: morning, Capitan Oslo! | 17:28 |
jungleboyj | e0ne: Good afternoon sir. What is up | 17:29 |
e0ne | jungleboyj: i'm wondered why we don't use oslo.log still | 17:29 |
jungleboyj | e0ne: Doh! I forgot I said I was going to do that. I have a quiet afternoon. Let me tackle that. | 17:30 |
e0ne | tryed to migrate to it, but we've got some outdated modules from oslo-incubator | 17:30 |
*** dansmith has quit IRC | 17:30 | |
jungleboyj | e0ne: That was the other thing I was going to finish up doing. | 17:30 |
*** dansmith has joined #openstack-cinder | 17:30 | |
jungleboyj | e0ne: I can work on getting that done today if you can do reviews on them tomorrow and try to get it through. :-) | 17:31 |
e0ne | jungleboyj: https://review.openstack.org/157441 - not sure that it should be landed in Kilo | 17:31 |
e0ne | jungleboyj: sure, i'll review them | 17:31 |
*** kaufer has quit IRC | 17:31 | |
*** rajinir_r has quit IRC | 17:32 | |
jungleboyj | e0ne: I think we should be able to get this in for Kilo still. | 17:32 |
*** rajinir_r has joined #openstack-cinder | 17:32 | |
jungleboyj | e0ne: So, is this patch just a WIP and you just need the incubator updates done? | 17:32 |
e0ne | jungleboyj: it doesn't change openstack.common and doesn't remove outdated modules | 17:33 |
jwcroppe | jungleboyj: 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 |
jungleboyj | jwcroppe: Ok. Cool. | 17:34 |
*** patrickeast has joined #openstack-cinder | 17:34 | |
jungleboyj | e0ne: Ok. Let me look at finishing up clean up of oslo-incubator today. Thanks for getting the oslo.log change proposed. | 17:34 |
e0ne | jungleboyj: so, it needs rebase and remove 2 files after oslo-inccubator updates landed | 17:35 |
*** Yogi1 has quit IRC | 17:35 | |
jungleboyj | e0ne: Cool. | 17:35 |
*** nlevinki has quit IRC | 17:36 | |
*** Yogi1 has joined #openstack-cinder | 17:37 | |
*** timcl has joined #openstack-cinder | 17:37 | |
jungleboyj | jwcroppe: Was reviewing code from my phone. Not surprised I missed something. :-) | 17:38 |
e0ne | o_0 | 17:38 |
smcginnis | jungleboyj: You're hardcore my friend. | 17:38 |
jungleboyj | smcginnis: I was donating blood. Had to pass the time somehow. | 17:39 |
jwcroppe | haha | 17:39 |
smcginnis | jungleboyj: Hah! Code reviewing on a phone while losing blood. | 17:39 |
jungleboyj | smcginnis: Hardcore! | 17:40 |
*** leeantho has joined #openstack-cinder | 17:40 | |
jungleboyj | Technically just red blood cells. Did an double unit aphoresis donation. | 17:41 |
e0ne | next level: make cinder works on a tablet | 17:41 |
*** thingee has joined #openstack-cinder | 17:41 | |
jungleboyj | :-) | 17:41 |
jwcroppe | jungleboyj: you're setting a new precedent here for dedication | 17:41 |
jungleboyj | Anyway, gotta go get Logan to daycare. Back in a bit. e0ne I will work on getting Oslo cleaned up this afternoon. | 17:42 |
e0ne | jungleboyj: thanks | 17:42 |
jungleboyj | Is review.openstack.org being really slow for anyone else? | 17:42 |
jwcroppe | anyhow, 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 today | 17:43 |
jungleboyj | e0ne: This would be one step towards getting oslo clean up if you can take a look: https://review.openstack.org/156455 | 17:43 |
jungleboyj | jwcroppe: Ok. Thanks. | 17:43 |
e0ne | jungleboyj: i'm thinking about thingee's comment. how is the right way to add implicit dependency? | 17:45 |
*** jordanP has quit IRC | 17:47 | |
ganso_ | thingee: ping | 17:48 |
thingee | ganso_: hi | 17:48 |
jgriffith | e0ne: interesting | 17:49 |
jgriffith | e0ne: what are you running on? | 17:49 |
e0ne | jgriffith: vm ubuntu 14.04, 1cpu, 2gb ram, ssd on host | 17:54 |
jgriffith | e0ne: interesting | 17:54 |
e0ne | jgriffith: i'll try it on others env to verify | 17:54 |
*** kaufer has joined #openstack-cinder | 17:55 | |
jgriffith | e0ne: so I've narrowed it down to one test BTW | 17:55 |
jgriffith | e0ne: tempest.api.compute.servers.test_server_rescue_negative.ServerRescueNegativeTestJSON.test_rescued_vm_detach_volume[gate,negative,volume] | 17:56 |
*** avishay has quit IRC | 17:57 | |
e0ne | jgriffith: thanks! | 17:57 |
jgriffith | e0ne: ahh... is it an OpenStack VM? | 17:57 |
jgriffith | e0ne: I think this is related to "OpenStack on OpenStack" model FWIW | 17:57 |
e0ne | jgriffith: no. i created it from ubuntu server iso | 17:57 |
*** jgravel has joined #openstack-cinder | 17:57 | |
e0ne | jgriffith: on kvm | 17:57 |
jgriffith | e0ne: so what happens is something like this.... | 17:57 |
e0ne | jgriffith: hm... where can i get this image? | 17:58 |
jgriffith | e0ne: the rescue basicly is a new vm with the old vm ephemeral disk attached | 17:58 |
jgriffith | e0ne: this does some things to the servers /dev/sdX that I haven't quite figured out (moves things around) | 17:58 |
jgriffith | e0ne: and what happens is an lvs or vgs call will sit there trying to open the /dev/sdX device that it was at but timeout | 17:59 |
jgriffith | because it's not actually valid any longer | 17:59 |
*** ganso_ has left #openstack-cinder | 17:59 | |
jgriffith | honestly, I'm trying to understand exactly what happens here and what rescue is doing | 17:59 |
thingee | So 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.html | 17:59 |
thingee | we did 7 fish bowl last time, but I think we would benefit more from the smaller working sessions | 18:00 |
e0ne | jgriffith: need to go home. bb online in a few hours | 18:00 |
jgriffith | e0ne: no worries, have a good evening | 18:00 |
e0ne | jgriffith: thanks. i'm calling it "breack for dinner:)" | 18:01 |
*** bswartz has quit IRC | 18:01 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Fix comments style according to Hacking rules https://review.openstack.org/157468 | 18:02 |
e0ne | thingee: imo, it depends on what topic on design sessions will be | 18:02 |
thingee | from the previous times we talked about sessions for the summit, most we just stuck in the sprint | 18:03 |
thingee | Perhaps I could start an etherpad to get topics from people | 18:04 |
thingee | but the great thing about working sessions is they're pretty dynamic | 18:04 |
*** bswartz has joined #openstack-cinder | 18:04 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Update the documentation for os-brick https://review.openstack.org/157202 | 18:04 |
*** Swanson has joined #openstack-cinder | 18:05 | |
*** Apoorva has quit IRC | 18:05 | |
*** ronis has joined #openstack-cinder | 18:06 | |
*** aix has quit IRC | 18:07 | |
*** ybathia has joined #openstack-cinder | 18:07 | |
*** thingee has quit IRC | 18:08 | |
*** e0ne has quit IRC | 18:08 | |
*** harlowja has joined #openstack-cinder | 18:10 | |
openstackgerrit | Sai Kiran proposed openstack/cinder: Fix argument order in assertEqual: tests/test_service.py https://review.openstack.org/157474 | 18:12 |
*** asmith_brcd has joined #openstack-cinder | 18:13 | |
kaufer | thingee: 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 |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/python-cinderclient: Add volume multi attach support https://review.openstack.org/85856 | 18:23 |
*** Yogi1 has quit IRC | 18:27 | |
*** jecarey has joined #openstack-cinder | 18:27 | |
*** smcginnis has quit IRC | 18:28 | |
*** MentalRay has quit IRC | 18:30 | |
*** smcginnis has joined #openstack-cinder | 18:30 | |
*** Apoorva has joined #openstack-cinder | 18:32 | |
*** diegows has joined #openstack-cinder | 18:32 | |
*** nikesh_vedams has quit IRC | 18:33 | |
*** cnesa1 has joined #openstack-cinder | 18:35 | |
*** cnesa has quit IRC | 18:35 | |
*** akerr is now known as akerr_away | 18:35 | |
smcginnis | I believe I had heard we are not supposed to put any attribution to the OpenStack Foundation in copyright headers. | 18:36 |
smcginnis | Can anyone confirm that? | 18:36 |
smcginnis | hemna: ^^ | 18:36 |
hemna | heh, I hadn't heard that | 18:37 |
hemna | thingee ? | 18:37 |
smcginnis | hemna: I could be wrong, so hopefully someone here can give a definitive answer. | 18:37 |
jgriffith | smcginnis: hemna yeah... that was a "thing" a while back | 18:37 |
smcginnis | jgriffith: Thanks, figured you would know. | 18:38 |
hemna | so....remove it then | 18:38 |
hemna | ok I'll put a different header in then. | 18:38 |
hemna | thanks | 18:38 |
smcginnis | hemna: I believe so. Cool if you want to put in HP. | 18:38 |
jgriffith | hemna: smcginnis in reality the only thing that should really be in there at all that's worth value anyway is the Apache license header | 18:38 |
*** akerr_away has quit IRC | 18:38 | |
smcginnis | Or no specific company. | 18:38 |
jgriffith | anything else is just crap noise and means nothing | 18:38 |
jgriffith | smcginnis: :) | 18:38 |
hemna | :) | 18:39 |
jgriffith | smcginnis: we allow companies to put them in, but they're useless for the most part | 18:39 |
*** akerr has joined #openstack-cinder | 18:39 | |
smcginnis | jgriffith: That was my understanding. It doesn't hurt, but it doesn't really add anything. | 18:39 |
jgriffith | smcginnis: indeed | 18:39 |
smcginnis | Personally I'd rather see nothing. | 18:39 |
jgriffith | smcginnis: lawyers have been going back and forth on that topic for years in OpenStack | 18:39 |
smcginnis | Over several thousand files, that's a space savings. :) | 18:40 |
jgriffith | smcginnis: and +2, I'd rather see nothing too | 18:40 |
smcginnis | jgriffith: Oh, I'm sure. | 18:40 |
*** EmilienM is now known as EmilienM|afk | 18:40 | |
jgriffith | HP lawyers, vs Foundation lawyers, vs EMC lawyers, vs ......... | 18:40 |
jgriffith | crazy koo koo | 18:40 |
smcginnis | Open source licensing is often more complex than the actual code IMO. | 18:40 |
jgriffith | haha! | 18:41 |
jgriffith | Funny observation | 18:41 |
jgriffith | and 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 |
jgriffith | LOL | 18:42 |
jgriffith | smcginnis: so I should note... some companies require their contributors put it in | 18:42 |
smcginnis | Swanson: :) | 18:42 |
jgriffith | smcginnis: so we just live with it | 18:42 |
* jgriffith wants to make sure smcginnis 's cube doesn't get burnt | 18:43 | |
smcginnis | jgriffith: Yeah, I can see that. | 18:43 |
smcginnis | jgriffith: Hah! | 18:43 |
*** BharatK has quit IRC | 18:43 | |
jgriffith | jungleboyj: have you looked at pulling oslo log lib at all? | 18:44 |
jgriffith | jungleboyj: I may tackle that today if you haven't | 18:44 |
jgriffith | ^^ or anyone | 18:44 |
smcginnis | jgriffith: I believe he had commented earlier today that he was going to get to it today. | 18:45 |
smcginnis | jgriffith: If he hasn't passed out from donating blood. | 18:45 |
jgriffith | smcginnis: haha | 18:45 |
jgriffith | smcginnis: cool! | 18:45 |
jgriffith | the lib part, not the passing out from giving blood part :) | 18:45 |
jungleboyj | jgriffith: Actually, e0ne has a review out but needs me to finish getting oslo-incubator cleaned up to get it merged. | 18:48 |
jungleboyj | jgriffith: So I am working on that right now. | 18:48 |
jgriffith | coolio | 18:48 |
jungleboyj | jgriffith: Here is e0ne s patch: https://review.openstack.org/#/c/157441/ | 18:48 |
jungleboyj | Sorry I didn't get to that earlier but have been busy since the meet-up. | 18:49 |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Update the documentation for os-brick https://review.openstack.org/157202 | 18:49 |
patrickeast | jgriffith: 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 thei | 18:51 |
patrickeast | tests? | 18:51 |
patrickeast | jgriffith: i’m ok making the changes, but i am hesitent to touch so many other drivers i can’t really test out | 18:53 |
jgriffith | patrickeast: smcginnis https://imgflip.com/i/huo9r | 18:53 |
jgriffith | jungleboyj: | 18:53 |
patrickeast | lol | 18:53 |
smcginnis | jgriffith: :) | 18:53 |
jungleboyj | jgriffith: :-) | 18:54 |
jgriffith | patrickeast: so can't you leave it? | 18:55 |
jgriffith | patrickeast: I mean, just override the call in your driver only | 18:56 |
jgriffith | patrickeast: so it only applies to those who want/need it for now? | 18:56 |
patrickeast | jgriffith: 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 has | 18:56 |
jgriffith | patrickeast: how come? | 18:57 |
jgriffith | patrickeast: so what I'm thinking is "don't" do anything with it in manager | 18:57 |
patrickeast | jgriffith: 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 args | 18:57 |
jgriffith | patrickeast: or... just do an "if initiator_table" "else" | 18:58 |
jgriffith | patrickeast: ahh... fin tuples! | 18:58 |
jgriffith | :) | 18:58 |
jgriffith | patrickeast: seriously... that's "ok" | 18:58 |
jgriffith | patrickeast: lemme look at your patch | 18:58 |
*** timcl1 has joined #openstack-cinder | 18:59 | |
jgriffith | patrickeast: maybe my heads somewhere it shouldn't be | 18:59 |
*** timcl has quit IRC | 19:00 | |
patrickeast | jgriffith: 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 one | 19:01 |
patrickeast | jgriffith: here is the manager code changes http://paste.openstack.org/show/177812/ | 19:02 |
patrickeast | (untested) | 19:02 |
patrickeast | but shows kind of a gist of things | 19:02 |
*** bswartz has quit IRC | 19:03 | |
*** mwichmann has joined #openstack-cinder | 19:04 | |
*** ronis has quit IRC | 19:05 | |
*** e0ne has joined #openstack-cinder | 19:06 | |
*** bkopilov has quit IRC | 19:07 | |
*** akerr has quit IRC | 19:08 | |
*** tbarron has quit IRC | 19:09 | |
*** r-daneel has joined #openstack-cinder | 19:10 | |
*** dencaval has joined #openstack-cinder | 19:10 | |
*** timcl1 has quit IRC | 19:11 | |
*** bkopilov has joined #openstack-cinder | 19:11 | |
*** tbarron has joined #openstack-cinder | 19:14 | |
*** akerr has joined #openstack-cinder | 19:14 | |
*** delatte has quit IRC | 19:15 | |
*** cdelatte has quit IRC | 19:15 | |
*** dustins has quit IRC | 19:15 | |
*** vilobhmm has joined #openstack-cinder | 19:18 | |
*** thingee has joined #openstack-cinder | 19:19 | |
*** r-daneel has quit IRC | 19:19 | |
*** vilobhmm1 has joined #openstack-cinder | 19:21 | |
*** bswartz has joined #openstack-cinder | 19:24 | |
*** vilobhmm has quit IRC | 19:24 | |
*** bkopilov has quit IRC | 19:25 | |
*** timcl has joined #openstack-cinder | 19:26 | |
*** nellysmitt has quit IRC | 19:26 | |
*** dustins has joined #openstack-cinder | 19:30 | |
*** cdelatte has joined #openstack-cinder | 19:30 | |
*** dulek has joined #openstack-cinder | 19:31 | |
dulek | thingee: hi, my patch adding backup object is ready, can it get into priorities etherpad? | 19:34 |
dulek | thingee: it's review 157085 | 19:34 |
thingee | dulek: I'll check it out and see about moving it in. Want to make sure we get enough testing on volumes/snapshots. | 19:35 |
thingee | xyang1: ^ | 19:35 |
thingee | dulek: DuncanT already signed off on the object for volumes | 19:36 |
thingee | xyang1: did you work out your devstack issues? | 19:36 |
dulek | thingee: Thanks. Patch is quite simple anyway. | 19:36 |
thingee | dulek: I like simple :) | 19:36 |
thingee | dulek: I also check some multi-attach stuff first too. Already put that off yesterday from hemna. | 19:37 |
dulek | thingee: we all do. ;) | 19:37 |
dulek | thingee: okay, i'll do. Thanks! | 19:37 |
*** delattec has joined #openstack-cinder | 19:38 | |
*** krtaylor has quit IRC | 19:38 | |
*** cdelatte has quit IRC | 19:38 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/python-cinderclient: Add volume multi attach support https://review.openstack.org/85856 | 19:42 |
openstackgerrit | Tom Barron proposed openstack/cinder: Refactor Swift backup driver and introduce chunking driver https://review.openstack.org/149725 | 19:42 |
*** Yogi1 has joined #openstack-cinder | 19:44 | |
*** jwang_ has joined #openstack-cinder | 19:48 | |
*** dulek has quit IRC | 19:48 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Update the documentation for os-brick https://review.openstack.org/157202 | 19:49 |
*** jwang has quit IRC | 19:50 | |
*** erlon has quit IRC | 19:51 | |
*** nellysmitt has joined #openstack-cinder | 19:54 | |
*** bswartz has quit IRC | 20:01 | |
*** timcl1 has joined #openstack-cinder | 20:02 | |
*** timcl has quit IRC | 20:06 | |
*** MentalRay has joined #openstack-cinder | 20:07 | |
*** PaulCuzner has quit IRC | 20:14 | |
*** krtaylor has joined #openstack-cinder | 20:16 | |
*** PaulCuzner has joined #openstack-cinder | 20:17 | |
*** bill_az has quit IRC | 20:18 | |
*** PaulCuzner has quit IRC | 20:18 | |
*** diegows has quit IRC | 20:19 | |
xyang | thingee: 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 |
xyang | thingee: I haven't got chance to test the object thing. | 20:23 |
smcginnis | Anyone know what this is all about when running tempest? | 20:29 |
smcginnis | No repository found in /opt/stack/tempest. Create one by running "testr init" | 20:29 |
Swanson | If you run it is there one? | 20:33 |
smcginnis | Swanson: Part of the automated CI spin up, so running again and going to keep it around to investigate more. | 20:33 |
smcginnis | But I've seen this in the past and couldn't find a clear reason. | 20:33 |
smcginnis | Sometimes I can clone devstack and stack and it's fine. Other times, not so much. | 20:34 |
Swanson | I just pulled down a new version and am having some issues. | 20:34 |
thingee | xyang: alright, weird | 20:36 |
smcginnis | On 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 IRC | 20:37 | |
Swanson | That has been a multibackend issue for me. | 20:38 |
smcginnis | Swanson: Oh. C-vol not starting? | 20:38 |
Swanson | smcginnis: The cinder-volume service. IF you do a cinder-manage service list you should see an unhappy cinder-volume service. | 20:39 |
smcginnis | Swanson: Only when you have multibackend configured? | 20:40 |
Swanson | smcginnis: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-cinder | 20:42 | |
smcginnis | Swanson: Seems odd. | 20:42 |
smcginnis | Anyone know anything about tempest with the "testr init" or see mutlibackend errors? | 20:43 |
jgriffith | Swanson: smcginnis cinder.conf file? | 20:44 |
smcginnis | Swanson: ^^ | 20:45 |
smcginnis | jgriffith: For my issue cinder appears happy. Something wonky with tempest though. | 20:45 |
jgriffith | smcginnis: tempest.conf :) | 20:45 |
jgriffith | smcginnis: most likely something around default volume-type maybe? | 20:46 |
Swanson | I have a cinder.conf I've been using. | 20:46 |
*** e0ne has quit IRC | 20:47 | |
smcginnis | jgriffith: Have it in a minute. | 20:47 |
Swanson | Need to get it to the right machine.... | 20:48 |
jgriffith | For the service not to start at all however is kinda odd | 20:48 |
jgriffith | But it seems there are two questions here.... | 20:49 |
jgriffith | tempest tests and vol-service not starting | 20:49 |
jgriffith | not sure if theyr'e the same, or individual? | 20:49 |
smcginnis | Two different ones. | 20:50 |
smcginnis | Just mass confusion over here. :) | 20:50 |
smcginnis | jgriffith: Here's my tempest.conf: http://paste.openstack.org/show/177884/ | 20:50 |
jgriffith | good times! | 20:50 |
jgriffith | smcginnis: I thought there was an explicit multi-backend setting that went in there | 20:52 |
smcginnis | jgriffith: Sorry, this paste is the issue with getting a "testr init" error message. No multibackend here. | 20:52 |
smcginnis | jgriffith: Sorry for being confusing as hell. | 20:52 |
Swanson | jgriffith: try this... http://paste.openstack.org/show/177885/ | 20:52 |
jgriffith | smcginnis: darn you! | 20:52 |
jgriffith | LOl | 20:52 |
jgriffith | LOL | 20:52 |
jgriffith | Ok, so don't use testr directly | 20:53 |
jgriffith | I was told I was "stupid" the other day on this very topic :) | 20:53 |
smcginnis | I'm calling tools/pretty_tox.sh volume and getting this. | 20:53 |
smcginnis | Hah | 20:53 |
jgriffith | yeah, you can't do that any more | 20:53 |
smcginnis | pretty_tox.sh? | 20:53 |
jgriffith | and the folks in another channel basicly told me I was a moron | 20:53 |
jgriffith | tox -e all -- volumes | 20:54 |
smcginnis | jgriffith: If that's something obvious, I'm missing something big here. | 20:54 |
jgriffith | smcginnis: tox -e all -- volume | tee -a console.log.out | 20:54 |
smcginnis | jgriffith: Well, that certainly looks better. At least it hasn't blown up immediatey. | 20:54 |
jgriffith | smcginnis: Swanson So the problem is... | 20:54 |
smcginnis | immediately | 20:55 |
jgriffith | tempest is no longer "installed" by devstack | 20:55 |
jgriffith | and you're expected to run in venv/tox | 20:55 |
smcginnis | jgriffith: Now no tempest for Swanson. Just stacking with multibackend config. | 20:55 |
jgriffith | but surprisingly pretty_tox doesn't use tox | 20:55 |
smcginnis | WTH | 20:55 |
smcginnis | That's false advertisement. | 20:55 |
smcginnis | ;) | 20:55 |
Swanson | jgriffith: That's the multibackend one. Where sometimes the volume service for one or both backends dies without much warning. usually on stack.sh | 20:55 |
jgriffith | Yeah, but like I said, I was the moron :) | 20:55 |
jgriffith | Swanson: local.conf? | 20:56 |
smcginnis | jgriffith: Thanks for taking one for the team. :) | 20:56 |
jgriffith | haha! | 20:57 |
smcginnis | jgriffith: Here was Swanson's: tox -e all -- volumes | 20:57 |
smcginnis | Doh | 20:57 |
smcginnis | Copy paste is so hard. | 20:57 |
jgriffith | Swanson: make sure you're local.conf has: | 20:57 |
smcginnis | http://paste.openstack.org/show/177885/ | 20:57 |
jgriffith | cinder_enabled_backends=bend1,bend2 | 20:57 |
jgriffith | and.... | 20:57 |
Swanson | http://paste.openstack.org/show/177889/ | 20:58 |
Swanson | local.conf there. | 20:58 |
jgriffith | https://gist.github.com/j-griffith/3abdb8cb93462b122eaf | 20:58 |
jgriffith | Oh... k... just a sec | 20:59 |
jgriffith | Swanson: need the Default section header | 20:59 |
smcginnis | Swanson: Compared to jgriffith's, maybe need to move under [[post-config|$CINDER_CONF]]? | 20:59 |
jgriffith | Swanson: also verify the ENABLED_BACKENDS entry | 20:59 |
Swanson | Oh. With the enabled backends bit. | 21:00 |
jgriffith | Swanson: IIRC I had issues with this because it might get added to cinder.conf but it would be added in the wrong place | 21:00 |
*** PaulCuzner has joined #openstack-cinder | 21:00 | |
jgriffith | You can even put the CINDER_ENABLED_BACKENDS setting in both sections :) | 21:01 |
*** nellysmitt has quit IRC | 21:01 | |
Swanson | I think I had to put it where it is to get it to work at all. | 21:01 |
jgriffith | haha... try it like I have it with the [DEFAULT] section added | 21:01 |
jgriffith | think you'll be good | 21:01 |
Swanson | I will give that a try. ThankS! | 21:02 |
smcginnis | jgriffith: Thanks for the help and dealing with the confusion. :) | 21:02 |
jgriffith | smcginnis: that's how I roll :) | 21:04 |
jungleboyj | hemna: 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-cinder | 21:07 | |
*** Mandell has quit IRC | 21:09 | |
*** Mandell has joined #openstack-cinder | 21:09 | |
openstackgerrit | Patrick East proposed openstack/cinder: Add DB table for driver specific data https://review.openstack.org/151837 | 21:13 |
patrickeast | jgriffith: ^ i put up part of the poc i was fiddling with | 21:14 |
patrickeast | jgriffith: it shows the model, db apis, and what it outlined in the spec as far as the manager and driver methods | 21:15 |
patrickeast | but i didn’t do the whole refactor in that patch | 21:15 |
*** annashen has quit IRC | 21:16 | |
*** annashen has joined #openstack-cinder | 21:17 | |
Swanson | jgriffith: 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-cinder | 21:29 | |
*** timcl1 has quit IRC | 21:30 | |
*** annashen has quit IRC | 21:32 | |
*** morganfainberg is now known as needscoffee | 21:32 | |
*** timcl has left #openstack-cinder | 21:32 | |
*** Mandell has quit IRC | 21:33 | |
*** theanalyst has quit IRC | 21:37 | |
*** gary-smith has quit IRC | 21:38 | |
*** theanalyst has joined #openstack-cinder | 21:39 | |
*** gary-smith has joined #openstack-cinder | 21:41 | |
hemna | https://bugs.launchpad.net/nova/+bug/1423690 | 21:42 |
*** bswartz has quit IRC | 21:42 | |
openstack | Launchpad bug 1423690 in OpenStack Compute (nova) "rebuild_instance doesn't detach cinder volumes correctly" [Undecided,New] | 21:42 |
openstackgerrit | Bob Callaway proposed openstack/cinder: Manage/unmanage impl for NetApp ONTAP iscsi driver https://review.openstack.org/138111 | 21:43 |
hemna | jungleboyj, thanks for the reviews | 21:43 |
jungleboyj | hemna: Welcome. | 21:43 |
*** gary-smith has quit IRC | 21:49 | |
*** gary-smith has joined #openstack-cinder | 21:50 | |
*** akerr has quit IRC | 21:51 | |
jgriffith | Swanson: well... cinder service-list should show two | 21:51 |
*** juzuluag has quit IRC | 21:53 | |
*** jaypipes has quit IRC | 21:53 | |
jgriffith | hemna: I don't think your bug is accurate | 21:55 |
jgriffith | hemna: what version of OpenStack are you looking at? | 21:55 |
*** Apoorva_ has joined #openstack-cinder | 21:55 | |
*** lcurtis has quit IRC | 21:56 | |
jgriffith | hemna: notice there's a "driver.detach_volume" call | 21:56 |
*** MasterPiece has quit IRC | 21:56 | |
hemna | this is juno and master | 21:56 |
jgriffith | hemna: and a driver.remove_export | 21:56 |
jgriffith | it's there as well | 21:57 |
hemna | detach_block_devices is what is defined and passed into driver.rebuild | 21:57 |
hemna | detach_block_devices only calls volume_api.detach() | 21:57 |
jgriffith | hemna: ok | 21:57 |
jgriffith | hemna: don't know what that means exactly | 21:57 |
hemna | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2873 | 21:58 |
*** Apoorva has quit IRC | 21: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 |
jungleboyj | So, going to have to think out how to approach this. | 22:00 |
smcginnis | Carefully | 22:00 |
*** patrickeast_ has joined #openstack-cinder | 22:02 | |
*** dannywil_ has joined #openstack-cinder | 22:02 | |
*** Yogi1 has quit IRC | 22:03 | |
jgriffith | hemna: Yeah, but that now calls os-detach | 22:03 |
hemna | yup | 22:03 |
jgriffith | hemna: whih calls volume.api:detach | 22:03 |
*** patrickeast has quit IRC | 22:04 | |
*** patrickeast_ is now known as patrickeast | 22:04 | |
hemna | which only calls the volume manager's detach_volume() | 22:04 |
jgriffith | hemna: which call rpcapi.detach_volume | 22:04 |
hemna | which doesn't detach | 22:04 |
*** dannywilson has quit IRC | 22:04 | |
hemna | it basically only touches the db | 22:04 |
hemna | as drivers don't detach volumes from that call | 22:04 |
hemna | terminate_connection is where the drivers remove the volume export from the array | 22:04 |
jgriffith | hemna: self.driver.remove_export? | 22:04 |
hemna | yah, doesn't unattach | 22:05 |
jungleboyj | smcginnis: Thank you captain obvious | 22:05 |
jgriffith | hemna: http://goo.gl/s4Z6UZ | 22:05 |
smcginnis | jungleboyj: Oh, a promotion from sergeant smart ass. :) | 22:05 |
jgriffith | hemna: I'm confused, care to teach me? | 22:05 |
jungleboyj | :-) | 22:05 |
jgriffith | hemna: you're saying driver.remove_export doesn't detach? | 22:06 |
jgriffith | hemna: sounds like a driver problem? | 22:06 |
jgriffith | what am I missing | 22:06 |
hemna | remove_export doesn't detach volumes. | 22:06 |
hemna | only terminate_connection does | 22:06 |
jgriffith | hemna: Ohhh... I see | 22:06 |
jgriffith | your driver doesn't do anything | 22:07 |
jgriffith | it's a pass in your driver | 22:07 |
jgriffith | hemna: https://github.com/openstack/cinder/blob/stable/juno/cinder/volume/drivers/lvm.py#L566 | 22:08 |
hemna | and in others | 22:08 |
jgriffith | reference implementation... emphasis on *reference* | 22:08 |
jgriffith | :) | 22:08 |
jgriffith | but yes, inconsistent | 22:09 |
jgriffith | I'd say fix it on Cinder side if it were me | 22:09 |
jgriffith | that's where the bug is | 22:09 |
hemna | lots of drivers in cinder don't do anything on remove_export | 22:09 |
hemna | nova's detach_volume does it correctly. | 22:10 |
jgriffith | indeed | 22:10 |
hemna | calls everything in the correct sequence | 22:10 |
hemna | I'm not sure why their rebuild_instance doesn't just call detach_volume | 22:10 |
jgriffith | hemna: fair point | 22:10 |
*** Apoorva_ has quit IRC | 22:10 | |
jgriffith | I'm not sure why we still have two different paths | 22:10 |
hemna | yah I agree | 22:10 |
*** Apoorva has joined #openstack-cinder | 22:11 | |
hemna | so not only do we have driver.remove_export | 22:11 |
hemna | but we also have driver.detach_volume | 22:11 |
hemna | along with driver.terminate_connection | 22:11 |
hemna | heh | 22:11 |
smcginnis | What is the difference between initialize_connection and create_export supposed to be? | 22:12 |
* smcginnis looks guilty | 22:12 | |
*** bswartz has joined #openstack-cinder | 22:12 | |
hemna | :P | 22:12 |
jgriffith | smcginnis: so I tried to make all of that go away a year ago | 22:12 |
jgriffith | but it's like whack'o mole | 22:12 |
hemna | the API is confusing for sure | 22:12 |
jgriffith | and as you see... one vendor wants one, the other wants another | 22:12 |
smcginnis | Seems most use initialize/terminate and not ensure/remove. | 22:12 |
jgriffith | detach IMO should implicitly terminate connections | 22:13 |
jgriffith | smcginnis: in the past we had this scheme of "never deleting" the iscsi target after the vol was created | 22:13 |
jgriffith | so we could just reuse it | 22:13 |
hemna | smcginnis, oh and don't forget ensure_export | 22:13 |
smcginnis | hemna: Yeah, that was confusing too. | 22:13 |
jgriffith | hemna: but hey... at least we have more drivers | 22:13 |
hemna | anyway I don't want to boil the ocean, but I still think the nova bug is valid | 22:14 |
jgriffith | hemna: and we're closer to making things easier in horizon | 22:14 |
jgriffith | never mind things are fundamentally a wreck | 22:14 |
hemna | :P | 22:14 |
hemna | yah I'm with you | 22:14 |
*** openstackstatus has joined #openstack-cinder | 22:14 | |
*** ChanServ sets mode: +v openstackstatus | 22:14 | |
Swanson | Hmmm.... 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 |
jgriffith | Swanson: you need to be admin | 22:16 |
smcginnis | Swanson: cd devstack; . openrc admin admin | 22:16 |
*** nkrinner has quit IRC | 22:16 | |
jgriffith | but manage should have it too | 22:17 |
hemna | jgriffith, 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 L | 22:19 |
jgriffith | hemna: well, we've done that twice now | 22:20 |
jgriffith | but sure... maybe third times a charm | 22:20 |
*** jungleboyj has quit IRC | 22:20 | |
hemna | some of those items got attention in K | 22:20 |
*** diegows has joined #openstack-cinder | 22:20 | |
*** _cjones_ has quit IRC | 22:24 | |
*** annashen has joined #openstack-cinder | 22:25 | |
*** annashen has quit IRC | 22:25 | |
*** annashen has joined #openstack-cinder | 22:26 | |
*** _cjones_ has joined #openstack-cinder | 22:28 | |
*** thingee has joined #openstack-cinder | 22:29 | |
tsekiyama | hemna: Should I remove the brick related changes from https://review.openstack.org/#/c/140877/ ? | 22:31 |
tsekiyama | hemna: Currently it has duplicated changes with os-brick patch https://review.openstack.org/#/c/157184/ | 22:31 |
hemna | yup | 22:31 |
tsekiyama | hemna: so 140877 will only have the changes for the lvm driver. Correct? | 22:31 |
hemna | you mean the target changes right ? | 22:32 |
*** thangp has quit IRC | 22:33 | |
*** mriedem has quit IRC | 22:33 | |
tsekiyama | hemna: Yes, and driver.py | 22:33 |
hemna | yah | 22:33 |
*** ebalduf has quit IRC | 22:33 | |
hemna | I 'think' I'm getting close to a first release of os-brick | 22:33 |
tsekiyama | hemna: okey. thanks | 22:34 |
hemna | so, once that happens then my remove-brick patch has a chance of getting in for cinder | 22:34 |
tsekiyama | hemna: BTW, can I fetch os-brick using pip? | 22:34 |
hemna | heh | 22:34 |
hemna | not yet | 22:34 |
hemna | that's the part I'm trying to get done asap | 22:35 |
tsekiyama | hemna: ah i see. | 22:35 |
tsekiyama | hemna: I was trying various names like os-brick, os_brick, or even just brick (and fetched another lib!) | 22:36 |
thingee | wanghao: can we get an update on this please https://review.openstack.org/#/c/144590/ | 22:36 |
thingee | wanghao: a few days without update | 22:37 |
hemna | tsekiyama, so I'm basically just waiting on https://review.openstack.org/#/c/157202/ to land | 22:37 |
hemna | and then whenever we can do a release to pypi. | 22:37 |
*** medberry2 is now known as med_ | 22:37 | |
tsekiyama | hemna: great | 22:37 |
*** med_ has quit IRC | 22:37 | |
*** med_ has joined #openstack-cinder | 22:37 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Update the documentation for os-brick https://review.openstack.org/157202 | 22:39 |
thingee | ameade: minor issues with this, but we could get this through today after a reup https://review.openstack.org/#/c/155915/1 | 22:41 |
*** ebalduf has joined #openstack-cinder | 22:44 | |
ameade | thingee: thanks I'll tweak it after dinner | 22:44 |
*** annashen has quit IRC | 22:46 | |
*** dulek has joined #openstack-cinder | 22:48 | |
openstackgerrit | Steven Kaufer proposed openstack/python-cinderclient: WIP Adopt CLI Sorting Argument Guidelines https://review.openstack.org/145913 | 22:49 |
*** jungleboyj has joined #openstack-cinder | 22:50 | |
thingee | hemna: I don't have voting rights on os-brick | 22:52 |
hemna | ?! | 22:52 |
thingee | oh now I do | 22:52 |
thingee | weird | 22:52 |
hemna | ok phew | 22:53 |
DuncanT | Walt: Approved | 22:53 |
hemna | that's odd | 22:53 |
hemna | DuncanT, thank you | 22:53 |
*** annashen has joined #openstack-cinder | 22:53 | |
jgriffith | hemna: thingee DuncanT wanted to get some thoughts around the connection stuff... | 22:53 |
thingee | DuncanT: oh well nevermind | 22:53 |
jgriffith | since you're all here :) | 22:53 |
hemna | ok | 22:54 |
DuncanT | jgriffith: ok... | 22:54 |
jgriffith | is there any value/reason in having the method that "just" flips the state in the DB? | 22:54 |
hemna | context ? | 22:54 |
jgriffith | what I'm thinking of proposing is something such that "all roads lead to detach" | 22:54 |
hemna | I'm not following sorry. | 22:54 |
jgriffith | hemna: hehe.. that's ok | 22:54 |
*** dulek has quit IRC | 22:54 | |
hemna | oh the driver API | 22:55 |
jgriffith | so detach, vs termiante_connection | 22:55 |
hemna | so, why don't we just have 2 methods | 22:55 |
hemna | attach_volume, ensure_attached | 22:55 |
hemna | and detach_volume | 22:55 |
hemna | I guess that's 3 :P | 22:55 |
DuncanT | jgriffith: I think you're spot on, though we can't fix it in K I don't think | 22:55 |
*** kaufer has quit IRC | 22:55 | |
jgriffith | yeah, what I'm getting at is for back compat we have one call the other :) | 22:55 |
DuncanT | jgriffith: Since Nova will need work | 22:55 |
jgriffith | DuncanT: that solves the K dilema, fixes hemna current bug | 22:56 |
jgriffith | and gives us time to work the nova change | 22:56 |
jgriffith | possibly mark the cruft as "deprecated" for Kilo | 22:56 |
DuncanT | jgriffith: Hmmm, if you think having detach call terminate is safe, then sure | 22:56 |
jgriffith | Well, it does for the ref driver :) | 22:57 |
DuncanT | jgriffith: Fair enough :-) | 22:57 |
jgriffith | trying to figure out what else people may be using it for | 22:57 |
jgriffith | looking in drivers it seems mostly it's confusion around "which to implement" | 22:57 |
jgriffith | most have done one or the other, not both | 22:57 |
hemna | oh so you are thinking for K, to help fix the nova issue, to redirect the call to terminate as well ? | 22:57 |
jgriffith | hemna: correct | 22:57 |
hemna | jgriffith, yup | 22:57 |
DuncanT | Nova's interactions with cinder makes me a bit sad... was discussing this with somebody else a few minutes ago | 22:57 |
hemna | that part has always confused me | 22:57 |
jgriffith | hemna: doesn't help your Juno customers though | 22:57 |
jgriffith | DuncanT: yeah, we need to really focus on that for L | 22:58 |
jgriffith | like first thing out of the gate IMO | 22:58 |
hemna | DuncanT, amen. | 22:58 |
hemna | it's been a sore spot for some time for me. | 22:58 |
jgriffith | anyway... I think we can fix a lot of things with changes on our side that make calls do some implicit things | 22:58 |
DuncanT | hemna: 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 submissions | 22:59 |
hemna | yah I think that'd be great | 22:59 |
*** PaulCuzner has quit IRC | 22:59 | |
jgriffith | Ok... 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 out | 22:59 |
DuncanT | jgriffith: 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 improving | 22:59 |
*** primechuck has quit IRC | 22:59 | |
DuncanT | jgriffith: Count me in for reviews & testing | 23:00 |
hemna | so, I guess the only thing I'd worry about jgriffith, is getting corner cases where terminate_connection gets called twice | 23:00 |
hemna | that could lead to other issues as well | 23:00 |
hemna | just have to make sure that doesn't happen | 23:00 |
jgriffith | hemna: yeah... interesting | 23:00 |
jgriffith | hemna: although a simple try/catch would get us through the turmoil at first | 23:00 |
jgriffith | maybe | 23:01 |
hemna | hrmm yah...trying to think through that for our drivers, what the side affect would be | 23:01 |
jgriffith | Ok... let me take a look at it. I think I'll start playing around with detach. And yeah, def look for double calls | 23:01 |
hemna | it might be ok, but would have to test it against our array. | 23:01 |
hemna | if only we had 3rd party CI setup.... | 23:01 |
hemna | :P | 23:01 |
*** nellysmitt has joined #openstack-cinder | 23:02 | |
jgriffith | I've got a few other things to finish up but hopefully I can get something going in the next couple of days | 23:02 |
hemna | ok, count me in for the reviews and testing. | 23:02 |
hemna | jgriffith, you rock man. | 23:03 |
jgriffith | great.. thanks fellas | 23:03 |
*** tbarron has quit IRC | 23:03 | |
thingee | jgriffith, hemna, DuncanT: kicking off proposed topics for Liberty summit https://etherpad.openstack.org/p/cinder-liberty-proposed-sessions | 23:04 |
*** dannywil_ has quit IRC | 23:04 | |
*** Apoorva_ has joined #openstack-cinder | 23:06 | |
*** nellysmitt has quit IRC | 23:07 | |
*** Apoorva has quit IRC | 23:09 | |
thingee | jgriffith, 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 |
hemna | that gives almost a month before the conf. | 23:11 |
hemna | yah I think that's good | 23:11 |
DuncanT | Sounds good to me | 23:11 |
thingee | great, I'll send out the email | 23:12 |
thingee | also I was thinking 5 fishbowl sessions, 8 working sessions and and 2 sprint slots | 23:12 |
hemna | over 2 days ? | 23:13 |
thingee | previously we've done 7 fishbowl, 0 working sessions and 1 slot for sprints | 23:13 |
*** Guest936 has joined #openstack-cinder | 23:13 | |
thingee | 2 sprints mean a whole day. last time we did half | 23:13 |
thingee | unless we feel like half is still good | 23:13 |
*** Guest936 is now known as annegentle | 23:13 | |
thingee | I think we went through topics pretty well though | 23:13 |
thingee | so maybe half a day is fine | 23:13 |
hemna | probably safer to schedule the day and if we bail early for beer....so be it! | 23:14 |
thingee | also keeping in mind people will be burnt out by this point | 23:14 |
*** PaulCuzner has joined #openstack-cinder | 23:14 | |
openstackgerrit | John Griffith proposed openstack/cinder: Remove warnings for long vgs and lvs calls https://review.openstack.org/157582 | 23:15 |
*** annashen has quit IRC | 23:21 | |
openstackgerrit | John Griffith proposed openstack/cinder: Remove retry_deactivation directive on lvremove https://review.openstack.org/157590 | 23:23 |
openstackgerrit | John Griffith proposed openstack/cinder: Remove warnings for long vgs and lvs calls https://review.openstack.org/157582 | 23:25 |
*** lcurtis has joined #openstack-cinder | 23:25 | |
*** ebalduf has quit IRC | 23:27 | |
*** annashen has joined #openstack-cinder | 23:27 | |
*** dannywilson has joined #openstack-cinder | 23:30 | |
openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Failover to alternative iSCSI portals on login failure https://review.openstack.org/140877 | 23:31 |
*** Mandell has joined #openstack-cinder | 23:31 | |
*** annashen_ has joined #openstack-cinder | 23:33 | |
*** annashen_ has quit IRC | 23:38 | |
*** thingee has quit IRC | 23:38 | |
*** ebalduf has joined #openstack-cinder | 23:44 | |
*** ebalduf has quit IRC | 23:48 | |
*** markvoelker has quit IRC | 23:49 | |
anteaya | are folks aware that there are tests on cinder from a stackforge repo called sheepdog? | 23:52 |
anteaya | https://review.openstack.org/#/c/154605/ | 23:52 |
anteaya | I don't yet see that cinder core/ptl has weighed in on these testing changes | 23:53 |
*** dustins has quit IRC | 23:53 | |
anteaya | am I wrong and you know and you want this? | 23:53 |
*** mriedem has joined #openstack-cinder | 23:53 | |
anteaya | https://review.openstack.org/#/c/153868/2 | 23:53 |
*** asmith_brcd has quit IRC | 23:54 | |
sigmavirus24 | anteaya: 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 project | 23:54 |
sigmavirus24 | Thanks for thinking of us anteaya | 23:54 |
anteaya | well I can -1 the change for now | 23:54 |
anteaya | and we can back out the other one if we need to | 23:54 |
anteaya | I think I would like to hear from others in the channel before I move to the ml | 23:55 |
anteaya | but sigmavirus24 you are saying this is news to you? | 23:55 |
sigmavirus24 | jgriffith: ^ | 23:55 |
*** annashen has quit IRC | 23:55 | |
sigmavirus24 | anteaya: 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 cores | 23:55 |
anteaya | sigmavirus24: thank you for stating your perspective and its context | 23:56 |
anteaya | I appreciate it | 23:56 |
patrickeast | this is the first time i’ve heard of it, although i’m not really in the know | 23:56 |
* patrickeast just lurks here | 23:56 | |
anteaya | well two lurkers count | 23:56 |
* sigmavirus24 suspects jgriffith might have heard of it | 23:56 | |
anteaya | so it hasen't had much chat if it is known | 23:56 |
*** annashen has joined #openstack-cinder | 23:56 | |
patrickeast | yea nothing in the cinder meetings or anything from what i can recall | 23:57 |
patrickeast | unless it was a long time ago | 23:57 |
anteaya | patrickeast: thanks | 23:57 |
anteaya | no, sheepdog is new | 23:57 |
* sigmavirus24 searches mailing list for prior mention | 23:57 | |
anteaya | this release and possibly since january | 23:57 |
sigmavirus24 | no results since June when I first subb'd | 23:57 |
hemna | do we care about sheepdog plugin for devstack ? | 23:58 |
*** dannywilson has quit IRC | 23:58 | |
anteaya | I don't | 23:58 |
anteaya | does cinder? | 23:58 |
anteaya | and it certainly would set an odd precent if a stackforge repo was voting on cinder gate | 23:59 |
anteaya | which sounds like their intent | 23:59 |
anteaya | now if cinder wants this, then okay | 23:59 |
anteaya | but it doesn't sound so far like cinder knows anything about this | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!