*** dannywilson has quit IRC | 00:01 | |
*** vilobhmm1 has quit IRC | 00:02 | |
*** vilobhmm1 has joined #openstack-cinder | 00:02 | |
*** setmason has quit IRC | 00:03 | |
*** IanGovett has quit IRC | 00:09 | |
*** abhiram_moturi has quit IRC | 00:11 | |
*** abhiram_moturi has joined #openstack-cinder | 00:12 | |
*** garthb has quit IRC | 00:14 | |
*** abhiram_moturi has quit IRC | 00:16 | |
*** abhiram_moturi has joined #openstack-cinder | 00:16 | |
*** ociuhandu has quit IRC | 00:18 | |
*** rmesta has quit IRC | 00:21 | |
*** takedakn has quit IRC | 00:22 | |
*** IlyaG has quit IRC | 00:23 | |
*** abhiram_moturi has quit IRC | 00:27 | |
*** abhiram_moturi has joined #openstack-cinder | 00:27 | |
*** heyun has joined #openstack-cinder | 00:29 | |
*** jamielennox|away is now known as jamielennox | 00:29 | |
*** tsekiyam_ has joined #openstack-cinder | 00:31 | |
*** tsekiyama has quit IRC | 00:34 | |
*** mtanino has quit IRC | 00:36 | |
*** tsekiyam_ has quit IRC | 00:36 | |
*** zhenguo has joined #openstack-cinder | 00:38 | |
*** MentalRay has joined #openstack-cinder | 00:44 | |
*** leeantho has quit IRC | 00:46 | |
*** abhiram_moturi has quit IRC | 00:50 | |
*** abhiram_moturi has joined #openstack-cinder | 00:51 | |
*** erlon has quit IRC | 00:51 | |
*** ebalduf has joined #openstack-cinder | 00:51 | |
*** ebalduf has quit IRC | 00:56 | |
*** IlyaG has joined #openstack-cinder | 01:00 | |
vilobhmm1 | hemna : ping | 01:01 |
---|---|---|
*** abhiram_moturi has quit IRC | 01:03 | |
*** abhiram_moturi has joined #openstack-cinder | 01:03 | |
*** mkoderer has quit IRC | 01:05 | |
*** jdurgin1 has quit IRC | 01:07 | |
*** abhiram_moturi has quit IRC | 01:12 | |
*** abhiram_moturi has joined #openstack-cinder | 01:12 | |
*** mkoderer has joined #openstack-cinder | 01:14 | |
*** abhiram_moturi has quit IRC | 01:17 | |
*** abhiram_moturi has joined #openstack-cinder | 01:17 | |
*** jwang_ has joined #openstack-cinder | 01:18 | |
openstackgerrit | John Griffith proposed openstack/cinder: Check type match on create from source/snap https://review.openstack.org/181268 | 01:20 |
*** jwang has quit IRC | 01:21 | |
*** akerr has joined #openstack-cinder | 01:23 | |
*** akerr_ has joined #openstack-cinder | 01:25 | |
*** Apoorva has quit IRC | 01:27 | |
*** akerr has quit IRC | 01:28 | |
*** aix has quit IRC | 01:31 | |
*** abhiram_moturi has quit IRC | 01:33 | |
*** abhiram_moturi has joined #openstack-cinder | 01:34 | |
*** Apoorva has joined #openstack-cinder | 01:35 | |
*** abhiram_moturi has quit IRC | 01:42 | |
*** abhiram_moturi has joined #openstack-cinder | 01:42 | |
*** kaisers has quit IRC | 01:44 | |
*** Apoorva has quit IRC | 01:44 | |
*** patrickeast has quit IRC | 01:45 | |
*** primechuck has joined #openstack-cinder | 01:48 | |
openstackgerrit | Vilobh Meshram proposed openstack/cinder: Bad link in API version details response https://review.openstack.org/180691 | 01:50 |
*** winston-d_zzZ is now known as winston-d | 01:53 | |
*** abhiram_moturi has quit IRC | 01:55 | |
*** abhiram_moturi has joined #openstack-cinder | 01:55 | |
*** kaisers has joined #openstack-cinder | 01:57 | |
winston-d | jgriffith: ping | 01:59 |
*** dims_ has quit IRC | 02:00 | |
*** dims has joined #openstack-cinder | 02:00 | |
*** dims has quit IRC | 02:01 | |
*** vilobhmm1 has quit IRC | 02:01 | |
*** jwcroppe has joined #openstack-cinder | 02:03 | |
*** abhiram_moturi has quit IRC | 02:08 | |
*** abhiram_moturi has joined #openstack-cinder | 02:08 | |
*** david-lyle has joined #openstack-cinder | 02:10 | |
*** IlyaG has quit IRC | 02:11 | |
*** abhiram_moturi has quit IRC | 02:15 | |
*** abhiram_moturi has joined #openstack-cinder | 02:15 | |
*** annegentle has joined #openstack-cinder | 02:15 | |
jgriffith | winston-d: pong | 02:16 |
*** MentalRay has quit IRC | 02:17 | |
*** abhiram_moturi has quit IRC | 02:19 | |
*** patrickeast has joined #openstack-cinder | 02:19 | |
*** abhiram_moturi has joined #openstack-cinder | 02:20 | |
*** annegentle has quit IRC | 02:20 | |
winston-d | jgriffith: I left some comment to bug #1452823 | 02:24 |
openstack | bug 1452823 in Cinder "retype ignores extra_specs info including backend_name" [Undecided,In progress] https://launchpad.net/bugs/1452823 - Assigned to John Griffith (john-griffith) | 02:24 |
winston-d | jgriffith: do those make sense to you? | 02:24 |
*** abhiram_moturi has quit IRC | 02:28 | |
*** abhiram_moturi has joined #openstack-cinder | 02:29 | |
*** abhiram_moturi has quit IRC | 02:31 | |
*** abhiram_moturi has joined #openstack-cinder | 02:31 | |
jgriffith | winston-d: no, it doesn't | 02:31 |
jgriffith | winston-d: I tested with solidfire -> lvm | 02:32 |
jgriffith | winston-d: fails every time, goes direct to driver.retype call | 02:32 |
jgriffith | winston-d: just like I stated in the LP bug | 02:32 |
winston-d | really? | 02:34 |
winston-d | how come | 02:35 |
jgriffith | winston-d: not sure | 02:36 |
jgriffith | winston-d: but maybe the filter check isn't working or getting called correclty | 02:36 |
jgriffith | winston-d: I can check it out again with a breakpoint in that scheduler portion | 02:36 |
jgriffith | winston-d: didn't realize that was there TBH | 02:36 |
jgriffith | but regardless it seems it's not working | 02:36 |
jgriffith | winston-d: I also think this is the issue that is being reported by NetApp and Dell | 02:37 |
jgriffith | winston-d: which is why I started looking at it last night | 02:37 |
*** abhiram_moturi has quit IRC | 02:39 | |
*** abhiram_moturi has joined #openstack-cinder | 02:40 | |
*** r-daneel has quit IRC | 02:40 | |
winston-d | jgriffith: can you paste the extra-spec-list output for your experiment? | 02:40 |
*** annegentle has joined #openstack-cinder | 02:40 | |
openstackgerrit | wanghao proposed openstack/cinder: Notification with volume and snaphost metadata https://review.openstack.org/180400 | 02:43 |
winston-d | jgriffith: and what's the error? | 02:44 |
*** abhiram_moturi has quit IRC | 02:46 | |
*** abhiram_moturi has joined #openstack-cinder | 02:47 | |
*** abhiram_moturi has quit IRC | 02:51 | |
*** abhiram_moturi has joined #openstack-cinder | 02:51 | |
*** abhiram_moturi has quit IRC | 02:53 | |
*** harlowja_at_home has joined #openstack-cinder | 02:53 | |
*** abhiram_moturi has joined #openstack-cinder | 02:54 | |
*** abhiram_moturi has quit IRC | 02:56 | |
*** abhiram_moturi has joined #openstack-cinder | 02:56 | |
*** smallbig has joined #openstack-cinder | 02:59 | |
*** dims has joined #openstack-cinder | 03:01 | |
*** MentalRay has joined #openstack-cinder | 03:04 | |
*** dims has quit IRC | 03:06 | |
*** smoriya has joined #openstack-cinder | 03:07 | |
*** jacksmion has joined #openstack-cinder | 03:08 | |
openstackgerrit | Edwin Wang proposed openstack/cinder: There was an issue while running initialize_connection/ terminate_connection/_map_vdisk_to_host/_unmap_vdisk_from_host in parallel threads. https://review.openstack.org/181285 | 03:09 |
jacksmion | why so quite? | 03:09 |
akerr_ | 11pm | 03:09 |
akerr_ | fyi looks like taskflow update today broke remotefs derived drivers, at least netapp nfs and gluster show the errors | 03:10 |
ameade | +1 | 03:12 |
*** bill_az has quit IRC | 03:12 | |
*** IlyaG has joined #openstack-cinder | 03:12 | |
harlowja_at_home | ya, wonder whats going on there | 03:12 |
harlowja_at_home | only commit i can think of is https://github.com/openstack/taskflow/commit/227cf52 | 03:13 |
harlowja_at_home | but i don't see how that would of caused that | 03:13 |
*** akshai has joined #openstack-cinder | 03:14 | |
ameade | yeah it's gotta be something subtle | 03:14 |
harlowja_at_home | ya | 03:15 |
*** abhiram_moturi has quit IRC | 03:18 | |
harlowja_at_home | if someone has the ability to investigate a run with changed taskflow (with https://github.com/openstack/taskflow/commit/227cf52 removed, or before this commit and after to ensure thats the cause) then we can see about reverting that until more investigation | 03:18 |
*** abhiram_moturi has joined #openstack-cinder | 03:18 | |
harlowja_at_home | and/or doing other bisections | 03:18 |
ameade | i could dig in but not until tomorrow | 03:19 |
*** akshai has quit IRC | 03:19 | |
harlowja_at_home | k, pretty weird errors | 03:20 |
ameade | yeah definitely, if the sqlalchemy version wasn't capped i would blame that | 03:20 |
harlowja_at_home | :-p | 03:21 |
*** abhiram_moturi has quit IRC | 03:24 | |
*** abhiram_moturi has joined #openstack-cinder | 03:25 | |
*** ebalduf has joined #openstack-cinder | 03:25 | |
*** r-daneel has joined #openstack-cinder | 03:26 | |
openstackgerrit | wanghao proposed openstack/cinder: Fix response when querying host detail by host name https://review.openstack.org/162601 | 03:28 |
*** r-daneel has quit IRC | 03:31 | |
*** MentalRay has quit IRC | 03:32 | |
*** abhiram_moturi has quit IRC | 03:36 | |
*** abhiram_moturi has joined #openstack-cinder | 03:36 | |
*** vmtrooper has joined #openstack-cinder | 03:38 | |
*** abhiram_moturi has quit IRC | 03:41 | |
*** fanyaohong has joined #openstack-cinder | 03:41 | |
*** vilobhmm1 has joined #openstack-cinder | 03:41 | |
*** abhiram_moturi has joined #openstack-cinder | 03:41 | |
harlowja_at_home | ameade, let me know what u find out; and we can start disecting whats happening there | 03:42 |
*** vmtroope_ has joined #openstack-cinder | 03:42 | |
ameade | yeah definitely, thanks for being readily available | 03:42 |
harlowja_at_home | think i might have an idea | 03:42 |
harlowja_at_home | but not 100% yet | 03:42 |
harlowja_at_home | np | 03:42 |
*** vmtrooper has quit IRC | 03:43 | |
ameade | this line is in the volume manager vol_ref = flow_engine.storage.fetch('volume') | 03:43 |
harlowja_at_home | if u take out that commit (and leave the rest of the 0.10 ones) and that's the cause, then can revert it, and examine why (have an idea) | 03:43 |
harlowja_at_home | ya | 03:43 |
ameade | which seems related to that commit | 03:43 |
ameade | yeah | 03:43 |
*** annegentle has quit IRC | 03:43 | |
harlowja_at_home | i'm wondering if its https://github.com/openstack/taskflow/blob/master/taskflow/persistence/logbook.py#L449 | 03:43 |
harlowja_at_home | perhaps the copy.copy there is acting odd with sqlalchemy | 03:43 |
*** annegentle has joined #openstack-cinder | 03:44 | |
*** vmtroope_ has quit IRC | 03:44 | |
harlowja_at_home | and that copy is now activated in that commit (to first copy the original, update the copy, then save the copy successfully, then update the original object) | 03:44 |
harlowja_at_home | vs the order before which was (update the original, save the original, and leave the original modified if the save failed) | 03:44 |
ameade | sounds plausible | 03:44 |
harlowja_at_home | i can easily put up a review to for that, but let's double check i guess first | 03:45 |
*** r-daneel has joined #openstack-cinder | 03:46 | |
harlowja_at_home | easy fix if we just shouldn't do that copy.copy there | 03:47 |
*** patrickeast has quit IRC | 03:50 | |
harlowja_at_home | let's see https://review.openstack.org/#/c/181288/ might be all thats needed | 03:51 |
harlowja_at_home | ^ which would be pretty durn simple :) | 03:52 |
ameade | i'll see if i can test that really quick | 03:52 |
harlowja_at_home | k, if u have time | 03:54 |
harlowja_at_home | if that works, can get a emergency fix/release out | 03:56 |
*** abhiram_moturi has quit IRC | 04:00 | |
*** abhiram_moturi has joined #openstack-cinder | 04:00 | |
*** akerr_ has quit IRC | 04:02 | |
*** annegentle has quit IRC | 04:02 | |
*** MentalRay has joined #openstack-cinder | 04:04 | |
*** xyang1 has quit IRC | 04:06 | |
ameade | harlowja_at_home: that seems to fix it from what i can tell | 04:07 |
harlowja_at_home | wooot | 04:07 |
ameade | i'll give it the old 3 times and it's good | 04:07 |
harlowja_at_home | :) | 04:07 |
harlowja_at_home | damn copy! | 04:07 |
harlowja_at_home | lol | 04:07 |
harlowja_at_home | i'll open bug and tweak that commit a little | 04:08 |
harlowja_at_home | then can get another core to look it over | 04:08 |
harlowja_at_home | annnnd profit | 04:08 |
harlowja_at_home | *if i can find another core, ha | 04:08 |
ameade | yep confirmed | 04:09 |
ameade | nice | 04:09 |
ameade | thanks a bunch | 04:09 |
harlowja_at_home | cool, will wait for jenkins to come back with +1 and will get that tweaked | 04:09 |
harlowja_at_home | thx for the help/checking | 04:09 |
ameade | awesome, do me a favor and reply on the ML | 04:10 |
ameade | i'll plus 1 that patch | 04:10 |
harlowja_at_home | will do | 04:10 |
ameade | ty sir | 04:10 |
harlowja_at_home | np | 04:11 |
ameade | time to sleep, have a good night! | 04:11 |
harlowja_at_home | u too | 04:15 |
*** abhiram_moturi has quit IRC | 04:17 | |
*** abhiram_moturi has joined #openstack-cinder | 04:18 | |
*** changbl has joined #openstack-cinder | 04:18 | |
*** abhiram_moturi has quit IRC | 04:22 | |
*** abhiram_moturi has joined #openstack-cinder | 04:22 | |
*** rushiagr_away is now known as rushiagr | 04:27 | |
*** primechuck has quit IRC | 04:29 | |
*** primechuck has joined #openstack-cinder | 04:30 | |
*** ebalduf has quit IRC | 04:31 | |
*** abhiram_moturi has quit IRC | 04:33 | |
*** abhiram_moturi has joined #openstack-cinder | 04:34 | |
*** primechuck has quit IRC | 04:34 | |
*** abhiram_moturi has quit IRC | 04:40 | |
*** abhiram_moturi has joined #openstack-cinder | 04:40 | |
*** abhiram_moturi has quit IRC | 04:43 | |
*** abhiram_moturi has joined #openstack-cinder | 04:43 | |
*** deepakcs has joined #openstack-cinder | 04:43 | |
*** abhiram_moturi has quit IRC | 04:47 | |
*** abhiram_moturi has joined #openstack-cinder | 04:47 | |
*** MentalRay has quit IRC | 04:48 | |
*** julim has quit IRC | 04:49 | |
*** abhiram_moturi has quit IRC | 04:54 | |
*** abhiram_moturi has joined #openstack-cinder | 04:54 | |
*** rhefner has joined #openstack-cinder | 04:56 | |
*** julim has joined #openstack-cinder | 04:56 | |
*** nkrinner has joined #openstack-cinder | 04:57 | |
*** abhiram_moturi has quit IRC | 04:59 | |
*** abhiram_moturi has joined #openstack-cinder | 04:59 | |
*** Longgeek has joined #openstack-cinder | 05:00 | |
*** annegentle has joined #openstack-cinder | 05:03 | |
*** dulek has joined #openstack-cinder | 05:05 | |
*** emagana has joined #openstack-cinder | 05:07 | |
*** annegentle has quit IRC | 05:09 | |
*** swamireddy1 has joined #openstack-cinder | 05:10 | |
*** anuragpalsule has joined #openstack-cinder | 05:13 | |
*** swamireddy1 has quit IRC | 05:14 | |
*** swamireddy1 has joined #openstack-cinder | 05:14 | |
*** abhiram_moturi has quit IRC | 05:16 | |
*** abhiram_moturi has joined #openstack-cinder | 05:17 | |
*** swamireddy2 has joined #openstack-cinder | 05:20 | |
*** swamireddy1 has quit IRC | 05:22 | |
*** abhiram_moturi has quit IRC | 05:25 | |
*** abhiram_moturi has joined #openstack-cinder | 05:26 | |
*** bkopilov has quit IRC | 05:27 | |
*** Longgeek has quit IRC | 05:29 | |
*** abhiram_moturi has quit IRC | 05:32 | |
*** abhiram_moturi has joined #openstack-cinder | 05:33 | |
*** abhiram_moturi has quit IRC | 05:35 | |
*** abhiram_moturi has joined #openstack-cinder | 05:35 | |
*** abhiram_moturi has quit IRC | 05:39 | |
*** abhiram_moturi has joined #openstack-cinder | 05:40 | |
*** abhiram_moturi has quit IRC | 05:42 | |
*** abhiram_moturi has joined #openstack-cinder | 05:42 | |
*** emagana has quit IRC | 05:44 | |
*** vilobhmm1 has quit IRC | 05:44 | |
*** bkopilov has joined #openstack-cinder | 05:44 | |
*** emagana has joined #openstack-cinder | 05:45 | |
*** abhiram_moturi has quit IRC | 05:47 | |
*** abhiram_moturi has joined #openstack-cinder | 05:47 | |
*** fanyaohong has quit IRC | 05:49 | |
*** emagana has quit IRC | 05:49 | |
*** elonden has joined #openstack-cinder | 05:53 | |
*** abhiram_moturi has quit IRC | 05:57 | |
*** BharatK has joined #openstack-cinder | 05:57 | |
*** abhiram_moturi has joined #openstack-cinder | 05:58 | |
*** abhiram_moturi has quit IRC | 06:02 | |
*** abhiram_moturi has joined #openstack-cinder | 06:02 | |
*** abhiram_moturi has quit IRC | 06:04 | |
*** annegentle has joined #openstack-cinder | 06:04 | |
*** belmoreira has joined #openstack-cinder | 06:04 | |
*** abhiram_moturi has joined #openstack-cinder | 06:05 | |
*** vilobhmm1 has joined #openstack-cinder | 06:05 | |
*** harlowja_at_home has quit IRC | 06:09 | |
*** annegentle has quit IRC | 06:10 | |
*** abhiram_moturi has quit IRC | 06:13 | |
*** Longgeek has joined #openstack-cinder | 06:13 | |
*** abhiram_moturi has joined #openstack-cinder | 06:13 | |
*** r-daneel has quit IRC | 06:15 | |
*** abhiram_moturi has quit IRC | 06:18 | |
*** abhiram_moturi has joined #openstack-cinder | 06:18 | |
*** abhiram_moturi has quit IRC | 06:22 | |
*** abhiram_moturi has joined #openstack-cinder | 06:22 | |
*** abhiram_moturi has quit IRC | 06:24 | |
*** abhiram_moturi has joined #openstack-cinder | 06:25 | |
*** vilobhmm1 has quit IRC | 06:30 | |
*** anshul has joined #openstack-cinder | 06:31 | |
*** markvoelker has quit IRC | 06:33 | |
*** abhiram_moturi has quit IRC | 06:37 | |
*** abhiram_moturi has joined #openstack-cinder | 06:38 | |
*** abhiram_moturi has quit IRC | 06:40 | |
*** abhiram_moturi has joined #openstack-cinder | 06:40 | |
*** abhiram_moturi has quit IRC | 06:42 | |
*** abhiram_moturi has joined #openstack-cinder | 06:43 | |
*** bkopilov has quit IRC | 06:46 | |
*** anshul has quit IRC | 06:47 | |
*** anshul has joined #openstack-cinder | 06:47 | |
*** vnagpal has joined #openstack-cinder | 06:49 | |
*** abhiram_moturi has quit IRC | 06:52 | |
*** IlyaG has quit IRC | 06:52 | |
*** IlyaG_ has joined #openstack-cinder | 06:52 | |
*** abhiram_moturi has joined #openstack-cinder | 06:52 | |
*** abhiram_moturi has quit IRC | 06:54 | |
*** abhiram_moturi has joined #openstack-cinder | 06:54 | |
*** IlyaG_ has quit IRC | 06:57 | |
*** agarciam has joined #openstack-cinder | 07:00 | |
*** ankit_ag has joined #openstack-cinder | 07:00 | |
*** jacksmion has quit IRC | 07:02 | |
*** rushiagr is now known as rushiagr_away | 07:04 | |
*** annegentle has joined #openstack-cinder | 07:05 | |
*** e0ne has joined #openstack-cinder | 07:09 | |
*** annegentle has quit IRC | 07:10 | |
*** markus_z has joined #openstack-cinder | 07:14 | |
*** elonden has quit IRC | 07:16 | |
*** sks has joined #openstack-cinder | 07:16 | |
*** ronis has joined #openstack-cinder | 07:18 | |
*** abhiram_moturi has quit IRC | 07:20 | |
*** abhiram_moturi has joined #openstack-cinder | 07:21 | |
*** junhongl has quit IRC | 07:21 | |
*** junhongl has joined #openstack-cinder | 07:21 | |
*** eglynn has quit IRC | 07:22 | |
*** abhiram_moturi has quit IRC | 07:25 | |
*** abhiram_moturi has joined #openstack-cinder | 07:26 | |
*** abhiram_moturi has quit IRC | 07:28 | |
*** abhiram_moturi has joined #openstack-cinder | 07:28 | |
*** e0ne has quit IRC | 07:29 | |
*** e0ne has joined #openstack-cinder | 07:33 | |
*** jamielennox is now known as jamielennox|away | 07:36 | |
openstackgerrit | wanghao proposed openstack/cinder: Notification with volume and snaphost metadata https://review.openstack.org/180400 | 07:38 |
*** rushiagr_away is now known as rushiagr | 07:40 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add volume status to error messages in backup create flow https://review.openstack.org/171566 | 07:45 |
*** ronis_ has joined #openstack-cinder | 07:46 | |
*** ronis has quit IRC | 07:47 | |
*** abhiram_moturi has quit IRC | 07:53 | |
*** abhiram_moturi has joined #openstack-cinder | 07:53 | |
*** abhiram_moturi has quit IRC | 07:57 | |
*** abhiram_moturi has joined #openstack-cinder | 07:58 | |
*** abhiram_moturi has quit IRC | 08:00 | |
*** abhiram_moturi has joined #openstack-cinder | 08:00 | |
*** abhiram_moturi has quit IRC | 08:04 | |
*** abhiram_moturi has joined #openstack-cinder | 08:04 | |
*** tobe has joined #openstack-cinder | 08:05 | |
openstackgerrit | wanghao proposed openstack/cinder: Fix response when querying host detail by host name https://review.openstack.org/162601 | 08:05 |
*** nikeshm has quit IRC | 08:06 | |
*** annegent_ has joined #openstack-cinder | 08:06 | |
*** markvoelker has joined #openstack-cinder | 08:07 | |
*** abhiram_moturi has quit IRC | 08:09 | |
*** abhiram_moturi has joined #openstack-cinder | 08:09 | |
*** abhiram_moturi has quit IRC | 08:11 | |
*** annegent_ has quit IRC | 08:11 | |
*** abhiram_moturi has joined #openstack-cinder | 08:12 | |
*** abhiram_moturi has quit IRC | 08:14 | |
*** abhiram_moturi has joined #openstack-cinder | 08:14 | |
wanghao | hi guys, about the bp 'import/export snapshots'(https://review.openstack.org/#/c/144590/), there has been many review for a long time. Should we begin to push it forward at L-1? | 08:15 |
wanghao | Or more review is welcome if you think there still exist problem about the code:) | 08:17 |
*** eglynn has joined #openstack-cinder | 08:20 | |
*** chlong has quit IRC | 08:20 | |
*** sks has quit IRC | 08:21 | |
*** e0ne has quit IRC | 08:23 | |
*** abhiram_moturi has quit IRC | 08:27 | |
*** abhiram_moturi has joined #openstack-cinder | 08:27 | |
*** abhiram_moturi has quit IRC | 08:29 | |
*** abhiram_moturi has joined #openstack-cinder | 08:29 | |
*** tobe has quit IRC | 08:31 | |
*** abhiram_moturi has quit IRC | 08:36 | |
*** abhiram_moturi has joined #openstack-cinder | 08:36 | |
*** markvoelker has quit IRC | 08:38 | |
*** dalgaaf has joined #openstack-cinder | 08:41 | |
*** Lee1092 has joined #openstack-cinder | 08:47 | |
*** bkopilov has joined #openstack-cinder | 08:51 | |
*** abhiram_moturi has quit IRC | 08:53 | |
*** abhiram_moturi has joined #openstack-cinder | 08:54 | |
*** jwcroppe has quit IRC | 08:58 | |
*** jwcroppe has joined #openstack-cinder | 08:58 | |
*** e0ne has joined #openstack-cinder | 08:58 | |
*** e0ne is now known as e0ne_ | 08:58 | |
*** abhiram_moturi has quit IRC | 09:02 | |
*** abhiram_moturi has joined #openstack-cinder | 09:03 | |
*** abhiram_moturi has quit IRC | 09:05 | |
*** abhiram_moturi has joined #openstack-cinder | 09:05 | |
*** annegentle has joined #openstack-cinder | 09:07 | |
*** e0ne_ is now known as e0ne | 09:07 | |
*** abhiram_moturi has quit IRC | 09:09 | |
*** abhiram_moturi has joined #openstack-cinder | 09:10 | |
*** annegentle has quit IRC | 09:12 | |
*** abhiram_moturi has quit IRC | 09:14 | |
*** abhiram_moturi has joined #openstack-cinder | 09:14 | |
*** abhiram_moturi has quit IRC | 09:18 | |
*** abhiram_moturi has joined #openstack-cinder | 09:19 | |
*** e0ne is now known as e0ne_ | 09:21 | |
*** abhiram_moturi has quit IRC | 09:25 | |
*** abhiram_moturi has joined #openstack-cinder | 09:25 | |
openstackgerrit | wanghao proposed openstack/cinder-specs: Support to import image to an existed volume https://review.openstack.org/181339 | 09:27 |
*** IanGovett has joined #openstack-cinder | 09:27 | |
*** dulek_ has joined #openstack-cinder | 09:28 | |
*** dulek has quit IRC | 09:29 | |
*** abhiram_moturi has quit IRC | 09:34 | |
*** abhiram_moturi has joined #openstack-cinder | 09:34 | |
*** ndipanov has joined #openstack-cinder | 09:35 | |
*** abhiram_moturi has quit IRC | 09:39 | |
*** abhiram_moturi has joined #openstack-cinder | 09:39 | |
*** abhiram_moturi has quit IRC | 09:41 | |
*** abhiram_moturi has joined #openstack-cinder | 09:41 | |
*** abhiram_moturi has quit IRC | 09:43 | |
*** abhiram_moturi has joined #openstack-cinder | 09:44 | |
*** alexpilotti has joined #openstack-cinder | 09:48 | |
*** abhiram_moturi has quit IRC | 09:52 | |
*** abhiram_moturi has joined #openstack-cinder | 09:52 | |
*** ociuhandu has joined #openstack-cinder | 09:54 | |
*** e0ne_ is now known as e0ne | 09:55 | |
*** abhiram_moturi has quit IRC | 10:01 | |
*** abhiram_moturi has joined #openstack-cinder | 10:01 | |
*** abhiram_moturi has quit IRC | 10:03 | |
*** coolsvap|afk is now known as coolsvap | 10:03 | |
*** abhiram_moturi has joined #openstack-cinder | 10:04 | |
*** annegent_ has joined #openstack-cinder | 10:08 | |
*** abhiram_moturi has quit IRC | 10:10 | |
*** abhiram_moturi has joined #openstack-cinder | 10:10 | |
*** abhiram_moturi has quit IRC | 10:12 | |
*** abhiram_moturi has joined #openstack-cinder | 10:13 | |
*** annegent_ has quit IRC | 10:13 | |
*** zhenguo has quit IRC | 10:17 | |
*** abhiram_moturi has quit IRC | 10:19 | |
*** abhiram_moturi has joined #openstack-cinder | 10:19 | |
*** agarciam has quit IRC | 10:20 | |
*** avishay_ has joined #openstack-cinder | 10:25 | |
*** avishay_ has quit IRC | 10:28 | |
*** agarciam has joined #openstack-cinder | 10:30 | |
*** abhiram_moturi has quit IRC | 10:32 | |
*** abhiram_moturi has joined #openstack-cinder | 10:33 | |
*** markvoelker has joined #openstack-cinder | 10:35 | |
*** ebalduf has joined #openstack-cinder | 10:39 | |
*** markvoelker has quit IRC | 10:40 | |
*** dulek___ has joined #openstack-cinder | 10:43 | |
*** dulek_ has quit IRC | 10:43 | |
*** ebalduf has quit IRC | 10:43 | |
*** abhiram_moturi has quit IRC | 10:47 | |
*** abhiram_moturi has joined #openstack-cinder | 10:47 | |
*** abhiram_moturi has quit IRC | 10:49 | |
*** abhiram_moturi has joined #openstack-cinder | 10:50 | |
*** abhiram_moturi has quit IRC | 10:54 | |
*** abhiram_moturi has joined #openstack-cinder | 10:54 | |
*** e0ne is now known as e0ne_ | 10:56 | |
*** abhiram_moturi has quit IRC | 10:58 | |
*** anuragpalsule has quit IRC | 10:59 | |
*** abhiram_moturi has joined #openstack-cinder | 10:59 | |
*** smoriya has quit IRC | 11:00 | |
*** abhiram_moturi has quit IRC | 11:03 | |
*** abhiram_moturi has joined #openstack-cinder | 11:04 | |
openstackgerrit | yogeshprasad proposed openstack/cinder: Add chap support to CloudByte cinder driver https://review.openstack.org/173733 | 11:05 |
*** annegent_ has joined #openstack-cinder | 11:08 | |
*** alecv has quit IRC | 11:12 | |
*** annegent_ has quit IRC | 11:13 | |
*** dims has joined #openstack-cinder | 11:13 | |
*** e0ne_ is now known as e0ne | 11:14 | |
*** cdelatte has joined #openstack-cinder | 11:17 | |
*** delattec has joined #openstack-cinder | 11:17 | |
*** vnagpal_ has joined #openstack-cinder | 11:21 | |
*** heyun has quit IRC | 11:22 | |
*** vnagpal has quit IRC | 11:24 | |
*** dims has quit IRC | 11:28 | |
*** e0ne is now known as e0ne_ | 11:28 | |
*** dims has joined #openstack-cinder | 11:29 | |
*** e0ne_ is now known as e0ne | 11:30 | |
*** vnagpal_ has quit IRC | 11:35 | |
*** akerr has joined #openstack-cinder | 11:36 | |
*** markvoelker has joined #openstack-cinder | 11:36 | |
*** akerr_ has joined #openstack-cinder | 11:37 | |
*** akerr has quit IRC | 11:40 | |
*** aswadr has joined #openstack-cinder | 11:42 | |
*** abhiram_moturi has quit IRC | 11:44 | |
*** abhiram_moturi has joined #openstack-cinder | 11:44 | |
*** agarciam has quit IRC | 11:45 | |
*** r-daneel has joined #openstack-cinder | 11:50 | |
*** deepakcs has quit IRC | 11:53 | |
*** abhiram_moturi has quit IRC | 11:55 | |
*** ndipanov has quit IRC | 11:56 | |
*** r-daneel has quit IRC | 11:56 | |
*** abhiram_moturi has joined #openstack-cinder | 11:56 | |
*** abhiram_moturi has quit IRC | 11:58 | |
*** abhiram_moturi has joined #openstack-cinder | 11:58 | |
*** fthiagogv has joined #openstack-cinder | 11:59 | |
*** annegentle has joined #openstack-cinder | 12:00 | |
*** akerr_ has quit IRC | 12:01 | |
*** delattec has quit IRC | 12:01 | |
*** cdelatte has quit IRC | 12:01 | |
*** abhiram_moturi has quit IRC | 12:05 | |
*** abhiram_moturi has joined #openstack-cinder | 12:05 | |
*** markvoelker has quit IRC | 12:07 | |
*** markvoelker has joined #openstack-cinder | 12:07 | |
*** dalgaaf has quit IRC | 12:07 | |
*** akerr has joined #openstack-cinder | 12:08 | |
*** agarciam has joined #openstack-cinder | 12:10 | |
*** e0ne is now known as e0ne_ | 12:10 | |
*** abhiram_moturi has quit IRC | 12:12 | |
*** abhiram_moturi has joined #openstack-cinder | 12:12 | |
*** Yogi1 has joined #openstack-cinder | 12:13 | |
*** e0ne_ is now known as e0ne | 12:13 | |
*** dulek___ is now known as dulek_ | 12:16 | |
*** dulek_ is now known as dulek | 12:16 | |
*** Yogi1 has quit IRC | 12:18 | |
dulek | DuncanT, harlowja: Hi, we were able to produce kind of PoC exchanging file locks to tooz locks. | 12:18 |
dulek | DuncanT, harlowja: https://github.com/bluex-pl/cinder/tree/locks - you may want to take a look. :) | 12:19 |
dulek | harlowja: This is doing heartbeats in green threads, so this would probably need to change. | 12:20 |
*** mtanino has joined #openstack-cinder | 12:20 | |
*** akerr has quit IRC | 12:22 | |
*** abhiram_moturi has quit IRC | 12:26 | |
*** abhiram_moturi has joined #openstack-cinder | 12:27 | |
*** IanGovett1 has joined #openstack-cinder | 12:28 | |
*** IanGovett has quit IRC | 12:30 | |
*** abhiram_moturi has quit IRC | 12:35 | |
*** abhiram_moturi has joined #openstack-cinder | 12:36 | |
*** rushiagr is now known as rushiagr_away | 12:37 | |
*** abhiram_moturi has quit IRC | 12:40 | |
*** abhiram_moturi has joined #openstack-cinder | 12:40 | |
*** ankit_ag has quit IRC | 12:42 | |
*** aswadr has quit IRC | 12:43 | |
*** juzuluag has joined #openstack-cinder | 12:43 | |
*** timcl has joined #openstack-cinder | 12:45 | |
*** chlong has joined #openstack-cinder | 12:47 | |
*** lpetrut has joined #openstack-cinder | 12:52 | |
*** e0ne is now known as e0ne_ | 12:56 | |
*** krtaylor has quit IRC | 12:56 | |
*** agarciam has quit IRC | 12:56 | |
*** e0ne_ is now known as e0ne | 12:57 | |
*** thangp has joined #openstack-cinder | 12:57 | |
*** dulek has quit IRC | 13:00 | |
*** annegentle has quit IRC | 13:00 | |
*** annegentle has joined #openstack-cinder | 13:02 | |
*** ociuhandu has quit IRC | 13:05 | |
*** nkrinner has quit IRC | 13:06 | |
*** krtaylor has joined #openstack-cinder | 13:08 | |
*** rushiagr_away is now known as rushiagr | 13:09 | |
*** abhiram_moturi has quit IRC | 13:13 | |
*** abhiram_moturi has joined #openstack-cinder | 13:14 | |
*** EmilienM|afk is now known as EmilienM | 13:15 | |
*** dims has quit IRC | 13:19 | |
*** dims has joined #openstack-cinder | 13:20 | |
*** abhiram_moturi has quit IRC | 13:23 | |
*** abhiram_moturi has joined #openstack-cinder | 13:23 | |
*** jungleboyj has quit IRC | 13:23 | |
*** abhiram_moturi has quit IRC | 13:27 | |
*** anuragpalsule has joined #openstack-cinder | 13:27 | |
*** abhiram_moturi has joined #openstack-cinder | 13:28 | |
e0ne | does anyone use FakeISCSIDriver? | 13:29 |
*** anuragpalsule1 has joined #openstack-cinder | 13:29 | |
Adriano_ | thingee | 13:31 |
Adriano_ | hello | 13:31 |
Adriano_ | thingee: https://review.openstack.org/#/c/180723/ | 13:31 |
Adriano_ | can you pls aprrove this | 13:31 |
Adriano_ | ? | 13:31 |
*** primechuck has joined #openstack-cinder | 13:32 | |
*** anuragpalsule has quit IRC | 13:32 | |
*** crose has joined #openstack-cinder | 13:34 | |
*** dansmith is now known as superdan | 13:35 | |
*** coolsvap is now known as coolsvap|afk | 13:37 | |
*** mriedem_away is now known as mriedem | 13:37 | |
*** esker has joined #openstack-cinder | 13:40 | |
*** ebalduf has joined #openstack-cinder | 13:41 | |
*** esker has quit IRC | 13:41 | |
*** esker has joined #openstack-cinder | 13:41 | |
*** esker has quit IRC | 13:41 | |
*** esker has joined #openstack-cinder | 13:41 | |
*** mtanino has quit IRC | 13:42 | |
*** erlon has joined #openstack-cinder | 13:44 | |
*** annegentle has quit IRC | 13:44 | |
*** BharatK has quit IRC | 13:44 | |
*** ebalduf has quit IRC | 13:45 | |
*** ociuhandu has joined #openstack-cinder | 13:45 | |
*** annegentle has joined #openstack-cinder | 13:45 | |
*** markus_z has quit IRC | 13:47 | |
*** coolsvap has joined #openstack-cinder | 13:47 | |
*** sgotliv has joined #openstack-cinder | 13:48 | |
*** eharney has joined #openstack-cinder | 13:49 | |
*** openstackgerrit has quit IRC | 13:51 | |
*** dims is now known as dimsum__ | 13:51 | |
*** openstackgerrit has joined #openstack-cinder | 13:51 | |
*** ociuhandu has quit IRC | 13:53 | |
*** abhiram_moturi has quit IRC | 13:57 | |
*** abhiram_moturi has joined #openstack-cinder | 13:57 | |
*** lpabon has joined #openstack-cinder | 13:59 | |
*** abhiram_moturi has quit IRC | 14:01 | |
*** BharatK has joined #openstack-cinder | 14:01 | |
*** abhiram_moturi has joined #openstack-cinder | 14:02 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Fix FakeISCSIDriver and FakeISERDriver https://review.openstack.org/181396 | 14:02 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Fix FakeISCSIDriver and FakeISERDriver https://review.openstack.org/181396 | 14:03 |
*** BharatK has quit IRC | 14:07 | |
*** swamireddy2 has quit IRC | 14:09 | |
*** bnemec has joined #openstack-cinder | 14:13 | |
e0ne | jgriffith: morning John. are you around? | 14:14 |
*** annegentle has quit IRC | 14:16 | |
*** BharatK has joined #openstack-cinder | 14:19 | |
*** lpetrut has quit IRC | 14:20 | |
*** ebalduf has joined #openstack-cinder | 14:22 | |
*** MentalRay has joined #openstack-cinder | 14:22 | |
*** ebalduf has quit IRC | 14:22 | |
*** ebalduf has joined #openstack-cinder | 14:24 | |
*** shakamunyi has joined #openstack-cinder | 14:25 | |
*** abhiram_moturi has quit IRC | 14:28 | |
*** lpetrut has joined #openstack-cinder | 14:28 | |
*** abhiram_moturi has joined #openstack-cinder | 14:29 | |
*** bnemec is now known as beekneemech | 14:32 | |
*** madskier has joined #openstack-cinder | 14:32 | |
*** emagana has joined #openstack-cinder | 14:32 | |
*** anshul has quit IRC | 14:33 | |
jgriffith | e0ne: I am now, what's up? | 14:34 |
e0ne | jgriffith: afair, we talked about fake driver last week | 14:34 |
jgriffith | e0ne: we did indeed | 14:34 |
e0ne | i've got several questions ablout it | 14:35 |
e0ne | at least, it doesn't work with shceduler :(. | 14:35 |
*** ebalduf has quit IRC | 14:35 | |
jgriffith | e0ne: well, I think I stated pretty clear that I suspected it needed updating | 14:36 |
e0ne | i've added you to review request with fix | 14:36 |
jgriffith | e0ne: ahh.. excellent | 14:36 |
jgriffith | winston-d: ping | 14:36 |
e0ne | but... we've got 2(!) FakeISCSIDriver classes.. | 14:36 |
jgriffith | Yuk | 14:37 |
e0ne | the second one in unit tests https://github.com/openstack/cinder/search?utf8=%E2%9C%93&q=FakeISCSIDriver | 14:37 |
e0ne | i'm going to remove it | 14:37 |
jgriffith | Oh boy | 14:37 |
*** tsekiyama has joined #openstack-cinder | 14:37 | |
jgriffith | e0ne: I don't remember how/why that happened. But there was a reason at one point | 14:37 |
jgriffith | e0ne: I'm fine with seeing it removed and consolidating | 14:37 |
e0ne | jgriffith: ok. thanks | 14:38 |
jgriffith | might drive more intrest/activity in it | 14:38 |
e0ne | i want to make performance tests with fake driver | 14:38 |
*** ebalduf has joined #openstack-cinder | 14:38 | |
jgriffith | e0ne: seems like a good idea | 14:38 |
e0ne | to test _only_ cinder part w/o storage impact | 14:38 |
jgriffith | e0ne: that was the whole reason for it to begin with, so I certainly agree :) | 14:39 |
e0ne | good:) | 14:39 |
e0ne | jgriffith: one more question about fakes | 14:39 |
j_king | DuncanT: I've been working on adding a call to generate a trust token in cinder-backup and I think the implied trusts thing that ayoung suggested might be nicer because generating a trust at this point seems to require at least two calls into keystone and is a fair bit of ceremony for a small amount of work | 14:39 |
e0ne | jgriffith: why do we need FakeTarget? | 14:40 |
*** Guest58709 has joined #openstack-cinder | 14:41 | |
e0ne | jgriffith: i don't see any benefits of it because it's not enough to get lvm driver wroking | 14:41 |
*** madskier has quit IRC | 14:42 | |
e0ne | i didn't test it with other drivers | 14:42 |
*** bkopilov has quit IRC | 14:46 | |
anuragpalsule1 | Need some help to get cinder to send notifications to ceilometer, i have already set notification_driver = cinder.openstack.common.notifier.rpc_notifier, control_exchange=cinder. But not getting any notifications | 14:46 |
*** sgotliv has quit IRC | 14:48 | |
*** jungleboyj has joined #openstack-cinder | 14:49 | |
*** hemnafk is now known as hemna | 14:50 | |
jgriffith | e0ne: well, it was a place holder | 14:50 |
*** sgotliv has joined #openstack-cinder | 14:51 | |
*** madskier has joined #openstack-cinder | 14:51 | |
*** madskier has quit IRC | 14:51 | |
jgriffith | e0ne: I thought it would be a good idea to have "fake" versions of each of the drivers in with the modules | 14:52 |
*** anuragpalsule has joined #openstack-cinder | 14:52 | |
*** anuragpalsule1 has quit IRC | 14:52 | |
jgriffith | e0ne: and at some point flush them out and hopefully see them as the central use for tests and such | 14:52 |
*** madskier has joined #openstack-cinder | 14:53 | |
*** e0ne is now known as e0ne_ | 14:55 | |
*** abhiram_moturi has quit IRC | 14:57 | |
*** abhiram_moturi has joined #openstack-cinder | 14:58 | |
*** mtecer has joined #openstack-cinder | 14:58 | |
*** mtanino has joined #openstack-cinder | 15:03 | |
*** juzuluag has quit IRC | 15:03 | |
*** sgotliv has quit IRC | 15:05 | |
*** e0ne_ is now known as e0ne | 15:07 | |
e0ne | jgriffith: good idea. i hope. i'll share tests results with fake driver next week | 15:08 |
openstackgerrit | Tom Swanson proposed openstack/cinder: Dell: Added support for update_migrated_volume https://review.openstack.org/181160 | 15:10 |
*** rmesta has joined #openstack-cinder | 15:12 | |
*** abhiram_moturi has quit IRC | 15:14 | |
*** ronis_ has quit IRC | 15:14 | |
*** abhiram_moturi has joined #openstack-cinder | 15:14 | |
*** abhiram_moturi has quit IRC | 15:16 | |
*** abhiram_moturi has joined #openstack-cinder | 15:17 | |
*** belmoreira has quit IRC | 15:18 | |
*** dannywilson has joined #openstack-cinder | 15:20 | |
*** dannywil_ has joined #openstack-cinder | 15:24 | |
*** dannywilson has quit IRC | 15:24 | |
*** ebalduf has quit IRC | 15:25 | |
*** shakamunyi has quit IRC | 15:25 | |
*** ebalduf has joined #openstack-cinder | 15:27 | |
*** Apoorva has joined #openstack-cinder | 15:29 | |
*** ebalduf_ has joined #openstack-cinder | 15:29 | |
*** sks has joined #openstack-cinder | 15:29 | |
*** Apoorva_ has joined #openstack-cinder | 15:31 | |
*** jdurgin1 has joined #openstack-cinder | 15:31 | |
*** ebalduf has quit IRC | 15:32 | |
e0ne | where are sources located for http://docs.openstack.org/developer/cinder/devref/filter_scheduler.html? | 15:32 |
e0ne | there are a lot of errors, copy-paste from nova | 15:32 |
e0ne | and where is new docs? | 15:34 |
hemna | what's docs? | 15:34 |
hemna | :PO | 15:34 |
hemna | heh | 15:34 |
*** Apoorva has quit IRC | 15:35 | |
*** mtecer has quit IRC | 15:35 | |
*** IlyaG has joined #openstack-cinder | 15:36 | |
*** juzuluag has joined #openstack-cinder | 15:36 | |
*** jungleboyj has quit IRC | 15:37 | |
*** garthb has joined #openstack-cinder | 15:37 | |
e0ne | i'm disapointed in openstack docs in first day in openstack world | 15:38 |
Guest58709 | e0ne: sad to hear that, you'll find those are written for cinder devs by cinder devs so I'm sure they'd like to improve them. The source for those are in the cinder repo in doc/source. | 15:39 |
*** Guest58709 is now known as annegentle | 15:40 | |
hemna | I dunno, I think the docs are pretty good, all things considered. | 15:40 |
annegentle | hemna: yeah you guys do a pretty good job really | 15:41 |
hemna | we spend a lot of time each release updating ours | 15:41 |
*** jaypipes is now known as leakypipes | 15:42 | |
e0ne | Guest58709: actually, it is docs for 2013.1 release, current docs is not so detailed:( | 15:43 |
e0ne | hemna: i agree that we have better docs than few years ago. and they become better each release | 15:46 |
annegentle | e0ne: then probably the copy/paste problem is "fixed" | 15:46 |
*** sks has quit IRC | 15:46 | |
e0ne | annegentle: agree. now there is no copy-paste but we have 'other' docs for schedulere than in the past. | 15:47 |
e0ne | i need go deeper in it to understand what is good and what should be improved | 15:48 |
e0ne | hemna: i should go offline 'early' today. if you get any feadback for cinder agent, i'll be glad to discuss it on Monday | 15:51 |
hemna | e0ne, I haven't had time to look at it yet :( | 15:51 |
hemna | e0ne, https://review.openstack.org/#/c/181102/ | 15:51 |
hemna | can I Get some love on that guy ? | 15:51 |
hemna | simple enough patch | 15:51 |
e0ne | hemna: i've tolk to zigo about packaging of os-brick - it will be next chellenge for me:) | 15:52 |
e0ne | s/tolk/told | 15:52 |
*** madskier has quit IRC | 15:53 | |
hemna | packaging ? | 15:53 |
hemna | brb | 15:53 |
e0ne | deb and rpm:) | 15:53 |
openstackgerrit | Edwin Wang proposed openstack/cinder: flashsystem reports error while running tests with multi-thread https://review.openstack.org/181285 | 15:55 |
openstackgerrit | Edwin Wang proposed openstack/cinder: FlashSystem reports error while running tests with multi-thread https://review.openstack.org/181285 | 15:55 |
openstackgerrit | Edwin Wang proposed openstack/cinder: FlashSystem reports error while running tests with multi-thread https://review.openstack.org/181285 | 15:56 |
*** openstackstatus has quit IRC | 15:56 | |
*** openstackstatus has joined #openstack-cinder | 15:56 | |
*** ChanServ sets mode: +v openstackstatus | 15:56 | |
*** patrickeast has joined #openstack-cinder | 15:57 | |
openstackgerrit | Edwin Wang proposed openstack/cinder: FlashSystem reports error while running tests with multi-thread https://review.openstack.org/181285 | 16:01 |
*** annegentle has quit IRC | 16:02 | |
*** e0ne is now known as e0ne_ | 16:04 | |
*** anuragpalsule has quit IRC | 16:06 | |
hemna | e0ne_, ah ok cool | 16:06 |
*** Yogi1 has joined #openstack-cinder | 16:08 | |
*** dencaval has joined #openstack-cinder | 16:10 | |
*** abhiram_moturi has quit IRC | 16:13 | |
*** abhiram_moturi has joined #openstack-cinder | 16:13 | |
hemna | having a problem with mock in a unit test | 16:16 |
hemna | it doesn't seem to work in some cases | 16:16 |
hemna | https://gist.github.com/WaltHP/9480fcb65938d5cb8e4b | 16:16 |
hemna | any ideas? | 16:16 |
*** shakamunyi has joined #openstack-cinder | 16:16 | |
*** vilobhmm1 has joined #openstack-cinder | 16:18 | |
openstackgerrit | John Haan proposed openstack/python-cinderclient: Adding search options to backup list https://review.openstack.org/78233 | 16:19 |
*** sgotliv has joined #openstack-cinder | 16:21 | |
*** rushil has joined #openstack-cinder | 16:21 | |
*** abhiram_moturi has quit IRC | 16:24 | |
*** abhiram_moturi has joined #openstack-cinder | 16:24 | |
openstackgerrit | John Haan proposed openstack/python-cinderclient: Adding search options to backup list https://review.openstack.org/78233 | 16:25 |
*** jungleboyj has joined #openstack-cinder | 16:28 | |
*** Apoorva_ has quit IRC | 16:29 | |
*** leeantho has joined #openstack-cinder | 16:32 | |
*** shakamunyi has quit IRC | 16:33 | |
*** btran_ has joined #openstack-cinder | 16:34 | |
jgriffith | tbarron: smcginnis Swanson Would you folks be able to review and maybe test this out against the problems you were seeing: https://goo.gl/tCnUQi | 16:34 |
jgriffith | tbarron: smcginnis Swanson It fixed things up in my env | 16:34 |
*** akerr has joined #openstack-cinder | 16:36 | |
*** hodos has joined #openstack-cinder | 16:37 | |
*** dustins has joined #openstack-cinder | 16:38 | |
*** marcusvrn has quit IRC | 16:39 | |
*** rushil has quit IRC | 16:40 | |
*** marcusvrn has joined #openstack-cinder | 16:41 | |
*** ronis_ has joined #openstack-cinder | 16:43 | |
hodos | morning. does any1 see errors like this today while running tempest? ObjectDereferencedError: Can't emit change event for attribute 'Volume.provider_location' | 16:44 |
hodos | looks like new today | 16:44 |
*** harlowja has quit IRC | 16:44 | |
*** harlowja has joined #openstack-cinder | 16:44 | |
harlowja | hodos that should no longer be happening :) | 16:44 |
harlowja | http://lists.openstack.org/pipermail/openstack-dev/2015-May/063693.html | 16:45 |
harlowja | so recheck, and it should be gone | 16:45 |
hodos | harlowja: oh, so it was fixed just recently...ok | 16:46 |
harlowja | yup | 16:46 |
harlowja | resolved last night, released this morning | 16:46 |
hodos | harlowja: hmm, it says my branch is up with master | 16:47 |
*** MentalRay has quit IRC | 16:48 | |
harlowja | yes, its not really a cinder code issue, dependent package issue | 16:48 |
*** shakamunyi has joined #openstack-cinder | 16:48 | |
harlowja | so if u have a virtualenv, do pip install 'taskflow==0.10.1' | 16:48 |
hodos | harlowja: is it icluded in devstack ? | 16:49 |
harlowja | unsure, devstack afaik just installs things | 16:50 |
harlowja | so i would assume so | 16:50 |
harlowja | either way u need to get that updated version to resolve that :) | 16:50 |
*** Yogi1 has quit IRC | 16:50 | |
*** leeantho has quit IRC | 16:51 | |
*** Apoorva has joined #openstack-cinder | 16:52 | |
*** rushiagr is now known as rushiagr_away | 16:53 | |
*** ebalduf has joined #openstack-cinder | 16:53 | |
*** crose has quit IRC | 16:57 | |
*** ebalduf has quit IRC | 16:59 | |
*** annegentle has joined #openstack-cinder | 16:59 | |
*** abhiram_moturi has quit IRC | 17:01 | |
*** abhiram_moturi has joined #openstack-cinder | 17:02 | |
zigo | e0ne_: I'll be happy to help. | 17:03 |
zigo | e0ne_: FYI, the last package for Kilo just got accepted a few minutes ago in Sid! :) | 17:03 |
zigo | python-xstatic-angular-lrdragndrop -- Angular-Lrdragndrop 1.0.2 (XStatic packaging standard) https://bugs.debian.org/784429 | 17:04 |
openstack | Debian bug 784429 in wnpp "ITP: python-xstatic-angular-lrdragndrop -- Angular-Lrdragndrop 1.0.2 (XStatic packaging standard)" [Wishlist,Fixed] | 17:04 |
*** leeantho has joined #openstack-cinder | 17:04 | |
*** bkopilov has joined #openstack-cinder | 17:06 | |
*** shakamunyi has quit IRC | 17:07 | |
*** MentalRay has joined #openstack-cinder | 17:10 | |
jgriffith | vilobhmm1: ping | 17:11 |
vilobhmm1 | jgriffith : hi | 17:11 |
jgriffith | vilobhmm1: hi there | 17:11 |
vilobhmm1 | yes sir | 17:12 |
harlowja | sir yes sir | 17:12 |
harlowja | :-P | 17:12 |
jgriffith | vilobhmm1: hey.. I had a quick question on https://goo.gl/NqdKMh | 17:12 |
jgriffith | sir no sir | 17:12 |
jgriffith | what | 17:12 |
jgriffith | wait | 17:12 |
jgriffith | :) | 17:12 |
vilobhmm1 | sure | 17:12 |
jgriffith | vilobhmm1: so I'm fine with the change, but I'm curious WTF are we pulling somethign in to docs from unit tests? | 17:12 |
jgriffith | vilobhmm1: or am I reading something wrong ? | 17:13 |
*** Yogi1 has joined #openstack-cinder | 17:13 | |
smcginnis | jgriffith: I tested out that patch. It didn't address our issues until Swanson's patch, but it looked fine. | 17:13 |
jgriffith | smcginnis: thanks.. so your driver retype gets called but returns an error? | 17:13 |
jgriffith | smcginnis: I mean, that patch does nothing for you? | 17:14 |
smcginnis | jgriffith: Our problem was we didn't implement the update call. | 17:14 |
smcginnis | jgriffith: No, it didn't seem to make a difference. | 17:14 |
jgriffith | smcginnis: well, neither did I :) but I don't need it now | 17:14 |
smcginnis | Hmm | 17:14 |
vilobhmm1 | the whole point of the change https://review.openstack.org/#/c/180691/5 is to fix the broken link the docs are generated by another repo | 17:14 |
jgriffith | smcginnis: hmm, so how are you guys handling the driver.retype call | 17:14 |
vilobhmm1 | we have a unit test which basically shoudl point the right api doscs for each version | 17:14 |
vilobhmm1 | jgriffith : ^^ | 17:14 |
Swanson | jgriffith: we dont | 17:15 |
jgriffith | vilobhmm1: Ok, so I was confused by the bug | 17:15 |
jgriffith | vilobhmm1: thanks | 17:15 |
smcginnis | jgriffith: We needed to update our volume after migrate. | 17:15 |
smcginnis | jgriffith: https://review.openstack.org/#/c/181160/3/cinder/volume/drivers/dell/dell_storagecenter_api.py | 17:15 |
jgriffith | Swanson: you don't imnplement the retype command? | 17:15 |
smcginnis | jgriffith: Your patch handles if it is across different backend types, right? | 17:15 |
vilobhmm1 | jgriffith : the docs are in the openstack/api-site repo in WADL, handwritten, maintained | 17:16 |
Swanson | nope. we don't do anything with types so we don't implement it. But when a retype migrates a volume we were losing track of it. | 17:16 |
jgriffith | smcginnis: well, among other things. Mostly it just keeps from calling the driver method if we already know the types are specificed to be different backends | 17:16 |
jgriffith | Swanson: Ahh, ok, that makes total sense then | 17:16 |
jgriffith | Swanson: anything that doesn't impl it just returns False on the call via default driver and the migrate gets picked up | 17:16 |
vilobhmm1 | we are not pulling anything but just verifying that a correct link is provided | 17:17 |
smcginnis | Sounds like there were multiple issues around retype/migrate depending on your scenario. | 17:17 |
jgriffith | Swanson: so the bug i'm addressing there is only for backends that implement retype | 17:17 |
jgriffith | smcginnis: that's an understatement :) | 17:17 |
vilobhmm1 | annegentle : ping | 17:17 |
smcginnis | ;) | 17:17 |
tbarron | jgriffith: so there was some confusion here. We don't implement retype yet either. I was | 17:17 |
jgriffith | vilobhmm1: well I get that.. but what I don't get is why an api unit test has *anything* to do with docs | 17:18 |
tbarron | looking at LVM retype to understand what we'd need to do. | 17:18 |
*** shakamunyi has joined #openstack-cinder | 17:18 | |
jgriffith | tbarron: don't look at LVM it doesn't do anything either :) | 17:18 |
smcginnis | Is the driver implemented retype for retyping within the same backend? | 17:18 |
jgriffith | smcginnis: correct | 17:18 |
jgriffith | yes | 17:18 |
tbarron | IIRC I tried retyping a volume from one of our iscsi to LVM, but I was pretty confused. | 17:18 |
jgriffith | tbarron: :( | 17:19 |
smcginnis | OK, thought so. And since we currently only do one type per backend we don't need it. | 17:19 |
jgriffith | smcginnis: exactly | 17:19 |
jgriffith | smcginnis: so *my* bug only applies to backends that support multiple types, most don't | 17:19 |
vilobhmm1 | jgriffith : hth | 17:19 |
jgriffith | vilobhmm1: hth? | 17:19 |
Swanson | If we add support for storage_types or something like that via volume type we would have to add it. | 17:19 |
vilobhmm1 | hope that helps | 17:19 |
* jgriffith googles hipster slang/acronyms | 17:19 | |
jgriffith | vilobhmm1: LOL | 17:19 |
jgriffith | thanks.. you're faster than google :) | 17:19 |
tbarron | jgriffith: your fix looks fine, except if Winston is right that the scheduler just needs a tweak then maybe the fix should just go there? | 17:19 |
vilobhmm1 | :P | 17:19 |
jgriffith | tbarron: he's not | 17:20 |
jgriffith | tbarron: if you read my response in the bug | 17:20 |
tbarron | jgriffith: ok, I'll look. I've had my head down on some other stuff. | 17:20 |
*** ronis_ has quit IRC | 17:20 | |
*** IanGovett1 has quit IRC | 17:20 | |
*** annegentle has quit IRC | 17:21 | |
Swanson | That salad had a criminal amount of bacon on it. Not even pretending to eat healthy... | 17:21 |
*** IanGovett has joined #openstack-cinder | 17:22 | |
*** shakamunyi has quit IRC | 17:23 | |
smcginnis | Swanson: Hah! | 17:24 |
*** abhiram_moturi has quit IRC | 17:26 | |
*** abhiram_moturi has joined #openstack-cinder | 17:28 | |
*** ebalduf has joined #openstack-cinder | 17:28 | |
*** daneyon has left #openstack-cinder | 17:29 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Assign the platform after declaration https://review.openstack.org/181475 | 17:31 |
hemna | "criminal amount of bacon" how is that possible ? | 17:31 |
*** ebalduf_ has quit IRC | 17:32 | |
*** e0ne_ is now known as e0ne | 17:32 | |
Swanson | When your ventricles scream out in terror you've probably gone too far with the bacon. | 17:32 |
*** anshul has joined #openstack-cinder | 17:33 | |
smcginnis | Inconceivable. | 17:34 |
*** e0ne has quit IRC | 17:34 | |
*** shakamunyi has joined #openstack-cinder | 17:34 | |
*** bkopilov has quit IRC | 17:39 | |
*** fthiagogv has quit IRC | 17:39 | |
*** merooney has joined #openstack-cinder | 17:40 | |
*** sandywalsh has left #openstack-cinder | 17:42 | |
*** anshul has quit IRC | 17:42 | |
*** IanGovett has quit IRC | 17:42 | |
*** annegentle has joined #openstack-cinder | 17:43 | |
*** IanGovett has joined #openstack-cinder | 17:45 | |
*** merooney has quit IRC | 17:45 | |
*** ronis_ has joined #openstack-cinder | 17:45 | |
*** MentalRay has quit IRC | 17:48 | |
*** juzuluag has quit IRC | 17:48 | |
*** MentalRay has joined #openstack-cinder | 17:51 | |
*** esker has quit IRC | 17:52 | |
*** merooney has joined #openstack-cinder | 17:54 | |
*** anshul has joined #openstack-cinder | 17:56 | |
*** alexpilotti has quit IRC | 18:01 | |
*** alexpilotti has joined #openstack-cinder | 18:02 | |
*** ebalduf has quit IRC | 18:03 | |
*** ebalduf has joined #openstack-cinder | 18:04 | |
*** abhiram_moturi has quit IRC | 18:06 | |
*** abhiram_moturi has joined #openstack-cinder | 18:06 | |
*** alexpilotti has quit IRC | 18:07 | |
*** abhiram_moturi has quit IRC | 18:13 | |
*** abhiram_moturi has joined #openstack-cinder | 18:13 | |
*** coolsvap has quit IRC | 18:15 | |
*** sgotliv has quit IRC | 18:17 | |
*** esker has joined #openstack-cinder | 18:18 | |
*** vilobhmm1 has quit IRC | 18:19 | |
*** vilobhmm1 has joined #openstack-cinder | 18:19 | |
thingee | lpetrut: can you please give me the blueprint link for https://review.openstack.org/#/c/161299/ | 18:20 |
thingee | lpetrut: also in the future please include it in the commit message as shown https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references | 18:21 |
thingee | lpetrut: otherwise this makes tracking difficult for me. | 18:21 |
thingee | jgriffith: ^ | 18:22 |
*** rwsu_ has quit IRC | 18:24 | |
*** timcl1 has joined #openstack-cinder | 18:26 | |
*** merooney has quit IRC | 18:26 | |
*** merooney has joined #openstack-cinder | 18:28 | |
*** timcl has quit IRC | 18:28 | |
*** MentalRay has quit IRC | 18:30 | |
*** annegentle has quit IRC | 18:31 | |
thingee | hemna: what's is Ivan raising with the os-brick being added to requirements? | 18:32 |
*** vilobhmm1 has quit IRC | 18:33 | |
thingee | hemna: couldn't really catch it from irc/gerrit | 18:33 |
*** merooney has quit IRC | 18:33 | |
*** vilobhmm1 has joined #openstack-cinder | 18:33 | |
*** Longgeek has quit IRC | 18:37 | |
*** Longgeek has joined #openstack-cinder | 18:37 | |
*** Longgeek has quit IRC | 18:39 | |
*** Longgeek has joined #openstack-cinder | 18:39 | |
*** vilobhmm1 has quit IRC | 18:40 | |
*** vilobhmm1 has joined #openstack-cinder | 18:42 | |
*** vilobhmm1 has quit IRC | 18:43 | |
*** vilobhmm1 has joined #openstack-cinder | 18:44 | |
*** lpetrut has left #openstack-cinder | 18:48 | |
*** merooney has joined #openstack-cinder | 18:51 | |
*** abhiram_moturi has quit IRC | 18:56 | |
*** abhiram_moturi has joined #openstack-cinder | 18:56 | |
hodos | jgriffith: tox -eall -- '(?!.*(volume_upload|volume_backup))volume' doesn't run tempest.cli tests on my node. maybe i'm missing some services? | 19:00 |
jgriffith | hodos: just run "tox -eall -- volume" and be done :) | 19:01 |
hodos | jgriffith: can't run upload | 19:01 |
hodos | jgriffith: it's not working for nfs and corrupting the following tests | 19:01 |
jgriffith | hodos: and tempest.cli is getting pulled into projects functional tests IIRC | 19:02 |
jgriffith | hodos: oh, so you need a skip | 19:02 |
hodos | jgriffith: yes, i do it via the string | 19:02 |
hodos | hodos: don't know the other way | 19:02 |
jgriffith | hodos: OH DERP | 19:03 |
jgriffith | hodos: I see now :) | 19:03 |
*** MentalRay has joined #openstack-cinder | 19:03 | |
jgriffith | so I think the CLI tests have been removed | 19:03 |
hodos | jgriffith: just like today? | 19:03 |
jgriffith | hodos: in fact: https://goo.gl/eNXPZ8 | 19:03 |
*** BharatK_ has joined #openstack-cinder | 19:03 | |
hodos | jgriffith: oh. nice | 19:03 |
jgriffith | hodos: so that answers that question :) | 19:04 |
hodos | jgriffith: thanks, I'm happy ) | 19:04 |
jgriffith | hodos: always nice on a Friday :) | 19:04 |
*** timcl1 has quit IRC | 19:06 | |
hodos | jgriffith: strange that I've ran list_tests today and saw cli tests there... | 19:06 |
jgriffith | hodos: I dunno dude, but it merged last night or so | 19:07 |
hodos | jgriffith: just making sure. anyways I have stuff running successfully so thanks is a real Friday! And thanks a lot for providing sos-ci project; it's been a huge help | 19:09 |
jgriffith | hodos: great to hear | 19:09 |
*** abhiram_moturi has quit IRC | 19:11 | |
*** abhiram_moturi has joined #openstack-cinder | 19:12 | |
*** timcl has joined #openstack-cinder | 19:13 | |
*** lpabon has quit IRC | 19:14 | |
*** merooney has quit IRC | 19:18 | |
thingee | jungleboyj: was going to merge this, but can you answer the 1 comment question here https://review.openstack.org/#/c/152735/5 ? | 19:20 |
thingee | eharney: any update with https://review.openstack.org/#/c/165393/1 ? | 19:21 |
jungleboyj | thingee: Sure. Just a minute. | 19:21 |
thingee | eharney: this would be great ;) | 19:21 |
eharney | thingee: deepakcs is working on this one now. he's got some WIP code up for it no win association with one of the other blueprints in this area | 19:22 |
eharney | now in* | 19:22 |
thingee | eharney: cool, I'll look for deepak's update to the spec | 19:23 |
*** merooney has joined #openstack-cinder | 19:24 | |
jungleboyj | thingee: responded. I can update the spec to be more explicit if you feel it is necessary. | 19:24 |
*** nikeshm has joined #openstack-cinder | 19:25 | |
*** eglynn has quit IRC | 19:25 | |
thingee | jungleboyj: nah it's fine, thank you | 19:25 |
*** lpetrut has joined #openstack-cinder | 19:26 | |
jungleboyj | thingee: Thank you. | 19:27 |
thingee | hemna: https://review.openstack.org/#/c/152740/ | 19:30 |
nikeshm | asselin: http://paste.openstack.org/show/217335/ | 19:30 |
*** Lee1092 has quit IRC | 19:31 | |
nikeshm | asselin : now nodepool image-build dpc and nodepool image-upload dpc is working fine, but when uploading patch in sandbox,its saying noop-check not registered | 19:31 |
*** marcusvrn has quit IRC | 19:33 | |
*** dustins has quit IRC | 19:33 | |
openstackgerrit | Tom Barron proposed openstack/cinder: Use nfs_oversub_ratio when reporting pool capacity https://review.openstack.org/180199 | 19:33 |
*** merooney has quit IRC | 19:35 | |
*** hodos has quit IRC | 19:35 | |
*** abhiram_moturi has quit IRC | 19:39 | |
*** abhiram_moturi has joined #openstack-cinder | 19:39 | |
*** abhiram_moturi has quit IRC | 19:41 | |
*** abhiram_moturi has joined #openstack-cinder | 19:42 | |
*** abhiram_moturi has quit IRC | 19:44 | |
*** abhiram_moturi has joined #openstack-cinder | 19:44 | |
*** abhiram_moturi has quit IRC | 19:46 | |
*** abhiram_moturi has joined #openstack-cinder | 19:46 | |
*** abhiram_moturi has quit IRC | 19:48 | |
*** bkopilov has joined #openstack-cinder | 19:49 | |
*** abhiram_moturi has joined #openstack-cinder | 19:49 | |
*** abhiram_moturi has quit IRC | 19:51 | |
*** abhiram_moturi has joined #openstack-cinder | 19:51 | |
*** abhiram_moturi has quit IRC | 19:53 | |
*** abhiram_moturi has joined #openstack-cinder | 19:54 | |
*** geguileo has quit IRC | 19:55 | |
*** MentalRay has quit IRC | 20:00 | |
*** abhiram_moturi has quit IRC | 20:04 | |
*** abhiram_moturi has joined #openstack-cinder | 20:05 | |
*** geguileo has joined #openstack-cinder | 20:07 | |
asselin | nikeshm, telnet localhost 4730 | 20:07 |
asselin | nikeshm, type 'status' to see what jobs are registered | 20:07 |
*** vmtrooper has joined #openstack-cinder | 20:10 | |
*** abhiram_moturi has quit IRC | 20:12 | |
*** abhiram_moturi has joined #openstack-cinder | 20:12 | |
*** abhiram_moturi has quit IRC | 20:14 | |
*** abhiram_moturi has joined #openstack-cinder | 20:14 | |
*** shakamunyi has quit IRC | 20:17 | |
*** breitz has quit IRC | 20:17 | |
*** Apoorva has quit IRC | 20:17 | |
*** breitz has joined #openstack-cinder | 20:17 | |
*** abhiram_moturi has quit IRC | 20:21 | |
*** abhiram_moturi has joined #openstack-cinder | 20:21 | |
*** abhiram_moturi has quit IRC | 20:23 | |
*** abhiram_moturi has joined #openstack-cinder | 20:24 | |
*** annegentle has joined #openstack-cinder | 20:25 | |
*** annegentle has quit IRC | 20:26 | |
*** annegentle has joined #openstack-cinder | 20:26 | |
Swanson | jgriffith: thanks! | 20:27 |
*** akerr has quit IRC | 20:28 | |
*** hodos|2 has joined #openstack-cinder | 20:28 | |
*** IlyaG has quit IRC | 20:29 | |
*** abhiram_moturi has quit IRC | 20:32 | |
*** abhiram_moturi has joined #openstack-cinder | 20:32 | |
*** e0ne has joined #openstack-cinder | 20:33 | |
*** mriedem is now known as mriderman | 20:35 | |
*** julim has quit IRC | 20:37 | |
*** esker has quit IRC | 20:38 | |
*** kaisers has quit IRC | 20:38 | |
e0ne | jgriffith: thanks for +2/A https://review.openstack.org/#/c/171566/. added question to your comment. just want to not miss it | 20:39 |
*** thangp has quit IRC | 20:40 | |
e0ne | sometimes all of us don't read questions/comments for merged/approved reviews:( | 20:40 |
*** kaisers has joined #openstack-cinder | 20:41 | |
*** Apoorva has joined #openstack-cinder | 20:41 | |
*** Yogi1 has quit IRC | 20:41 | |
jgriffith | e0ne: https://goo.gl/pYKaIc | 20:41 |
jgriffith | e0ne: note the log cleanup and use of "resource=volume" | 20:42 |
jgriffith | e0ne: I added a tag mechanism to oslo.log that reads that in and formats the log messages consistently | 20:42 |
e0ne | jgriffith: yes, but in my patch i don't change logging. i changed exception message | 20:42 |
jgriffith | e0ne: by putting a volume-id in each | 20:42 |
jgriffith | e0ne: exception message counts :) | 20:43 |
nikeshm | asselin : http://paste.openstack.org/show/217389/ | 20:43 |
jgriffith | e0ne: fair enough | 20:43 |
e0ne | jgriffith: looks like i missed it | 20:43 |
*** dencaval has quit IRC | 20:43 | |
jgriffith | e0ne: no, I ment would've been cool to fix "everything" up in one shot | 20:43 |
jgriffith | e0ne: I know your patch wasn't related | 20:43 |
asselin | nikeshm, so nothing's registered. Did you start nodepool? did it create any jenkins slaves? | 20:43 |
jgriffith | e0ne: I was just saying that if we do a message cleanup/modify it would be cool to do the whole file, logs, exceptiosn and all | 20:44 |
jgriffith | e0ne: are logs are pretty inconsistenet | 20:44 |
jgriffith | e0ne: but anyway... no worries | 20:44 |
e0ne | jgriffith: agree, +1 | 20:44 |
jgriffith | e0ne: I'll pick back up on working on udpating them in chunks again next week maybe | 20:44 |
e0ne | jgriffith: i just fixed only my issue:( | 20:44 |
*** MentalRay has joined #openstack-cinder | 20:45 | |
e0ne | jgriffith: how should i add tagging for exception messages in a future patches? | 20:45 |
nikeshm | asselin: started long time ago,bot i am getting this in debug.log of nodepool http://paste.openstack.org/show/217390/ | 20:46 |
asselin | local_01 does not have image d-p-c for label d-p-c. | 20:47 |
asselin | nikeshm, nodepool image-upload all d-p-c | 20:47 |
jgriffith | e0ne: for the "reason=msg" stuff there's nothing to do | 20:48 |
*** shakamunyi has joined #openstack-cinder | 20:48 | |
jgriffith | e0ne: I was just saying that "instead of a one line change, change everything and update the whole file" :) | 20:48 |
jgriffith | e0ne: logging and exception messages are related in my opinion | 20:49 |
jgriffith | e0ne: sorryfor the confusion | 20:49 |
nikeshm | asselin: hmm nodepool image-build dpc;nodepool image-upload dpc now i have to run nodepool image-upload all d-p-c | 20:50 |
*** IlyaG has joined #openstack-cinder | 20:50 | |
e0ne | jgriffith: thanks for clarification. i appreciate your help | 20:50 |
nikeshm | asselin : http://paste.openstack.org/show/217391/ | 20:50 |
nikeshm | why so | 20:50 |
asselin | nikeshm, so the dib-image-list are the local ones built on the box. | 20:51 |
e0ne | jgriffith: i'll try fix all related staff next time instead fixing only one thing | 20:51 |
asselin | nikeshm, the upload is to sync up the provider | 20:51 |
asselin | nikeshm, now double-check the name of the image expected with the provider with the one uploaded. I think ther'es a dpc vs d-p-c | 20:51 |
jgriffith | e0ne: no worries, sometimes we just flat don't have time | 20:51 |
asselin | mismatch | 20:51 |
e0ne | jgriffith: tbh, this patch was made on my devstack env to debug something. and was posted to gerrit 'as is' | 20:51 |
jgriffith | e0ne: I just raised it to point it out otherwise nobody will ever remember :) | 20:52 |
jgriffith | e0ne: go to sleep! | 20:52 |
jgriffith | :) | 20:52 |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Added a unit test for masking iscsiadm passwords https://review.openstack.org/181553 | 20:52 |
*** primechuck has quit IRC | 20:52 | |
e0ne | jgriffith: i'll definatly do it:) | 20:52 |
e0ne | jgriffith: thanks for review and help understanding how make a code better | 20:53 |
*** ebalduf has quit IRC | 20:54 | |
*** mriderman has quit IRC | 20:54 | |
*** ebalduf has joined #openstack-cinder | 20:55 | |
nikeshm | asselin: http://paste.openstack.org/show/217394/ is the yaml of nodepool which i am using | 20:55 |
e0ne | have a good Friday and weekend. bb on Monday | 20:57 |
*** BharatK_ has quit IRC | 20:57 | |
asselin | nikeshm, change line 13 to dpc | 20:58 |
*** mriedem has joined #openstack-cinder | 20:59 | |
nikeshm | asselin: one doubt more,since we have to give private github repo while running install_master.sh,now if change in repo,then which steps in https://github.com/rasselin/os-ext-testing do i have to run again | 21:01 |
nikeshm | so like i am changing line number 13 | 21:01 |
nikeshm | to dpc | 21:01 |
nikeshm | in my private git data repo | 21:02 |
*** e0ne has quit IRC | 21:02 | |
asselin | nikeshm, ./install_master.sh from ~ | 21:02 |
nikeshm | ok thanks | 21:03 |
*** shakamunyi has quit IRC | 21:03 | |
*** abhiram_moturi has quit IRC | 21:06 | |
*** abhiram_moturi has joined #openstack-cinder | 21:07 | |
*** ronis_ has quit IRC | 21:08 | |
*** breitz has quit IRC | 21:08 | |
*** eharney has quit IRC | 21:09 | |
*** Longgeek has quit IRC | 21:10 | |
*** mriedem has quit IRC | 21:11 | |
*** mriedem has joined #openstack-cinder | 21:12 | |
*** timcl has quit IRC | 21:13 | |
*** emagana has quit IRC | 21:14 | |
*** abhiram_moturi has quit IRC | 21:21 | |
*** abhiram_moturi has joined #openstack-cinder | 21:22 | |
*** shakamunyi has joined #openstack-cinder | 21:22 | |
*** cbader has quit IRC | 21:25 | |
*** MentalRay has quit IRC | 21:26 | |
*** jungleboyj has quit IRC | 21:28 | |
*** rwsu has joined #openstack-cinder | 21:30 | |
*** rwsu has quit IRC | 21:30 | |
*** rwsu has joined #openstack-cinder | 21:31 | |
*** abhiram_moturi has quit IRC | 21:36 | |
jgriffith | thingee: so what do you want me to do on those backports? | 21:36 |
*** abhiram_moturi has joined #openstack-cinder | 21:36 | |
jgriffith | thingee: yours inparticular is completely safe AFAIC | 21:36 |
*** esker has joined #openstack-cinder | 21:36 | |
thingee | jgriffith: technically we shouldn't be doing non-security related to juno anymore. Saw some from dell | 21:37 |
thingee | jgriffith: though they've been sitting for a while | 21:37 |
thingee | since february | 21:37 |
jgriffith | thingee: yeah, I've been pretty much rejecting Juno | 21:38 |
jgriffith | thingee: Kilo is still "new" enough that I don't have any real issue | 21:38 |
jgriffith | thingee: but I also don't want to get called out by somebody for approving a "low" severity patch for backport | 21:39 |
jgriffith | thingee: at least not without someobdy having my back:) | 21:39 |
jgriffith | and discussing it | 21:39 |
thingee | real issue with backporting to Kilo or Kilo is still new enough that backporting to Juno should be fine? | 21:39 |
*** rwsu has quit IRC | 21:39 | |
jgriffith | thingee: yes, IMO | 21:39 |
jgriffith | thingee: and your patch is simple/safe | 21:40 |
jgriffith | thingee: but I -1'd it on principle :) | 21:40 |
thingee | Hang on I'm confused. :) I was trying to clarify if you were asking what to do with the Juno backports, or all backports? | 21:41 |
jgriffith | thingee: ok... one at a time | 21:41 |
*** erlon has quit IRC | 21:41 | |
*** MentalRay has joined #openstack-cinder | 21:41 | |
thingee | you just said backports and I wasn't sure what you were referring to. | 21:42 |
jgriffith | thingee: 1. Your backport to Kilo https://goo.gl/wq229V | 21:42 |
jgriffith | thingee: I gave a -1 becasue it's low severity and doesn't meet the criteria | 21:42 |
thingee | ah I see | 21:42 |
jgriffith | thingee: but I don't mind the patch itself, just need to have consistency with people | 21:42 |
thingee | Yeah I marked it low because it's a driver. I view my driver not as important as cinder core related. | 21:43 |
jgriffith | thingee: right, understood | 21:43 |
jgriffith | I do the same thing for the most part | 21:43 |
thingee | but it does break the datera CI. It was related to the ISCSI session issue when volume boot pattern got reenabled. Not sure if you remember me complaining about that early april. | 21:43 |
jgriffith | My point was that Kilo backports I'm not bugged about it | 21:44 |
jgriffith | well, in that case... | 21:44 |
jgriffith | wait | 21:44 |
jgriffith | irrelevant :) | 21:44 |
jgriffith | do you run Kilo CI and publish? | 21:44 |
jgriffith | anyway... | 21:45 |
jgriffith | thingee: do you want that patch backported or not? | 21:45 |
jgriffith | thingee: yes or now? | 21:45 |
jgriffith | no? | 21:45 |
thingee | not yet. Internally there is a juno one. | 21:45 |
thingee | I would, that's why I proposed it :). | 21:45 |
thingee | I haven't really denied a backport to previous release. I have denied backports to security only supported branches though. | 21:45 |
jgriffith | Ok | 21:46 |
jgriffith | done | 21:46 |
thingee | if you have examples where we haven't been consistent on the rules for stable branches, please let me know and I'll try to correct those or work with people who aren't following things. | 21:46 |
jgriffith | thingee: not "you" or "us" per-say | 21:46 |
jgriffith | thingee: keep in mind there's a stable maintenance core that's not just Cinder memebers | 21:46 |
jgriffith | anyway.... | 21:46 |
jgriffith | moving on... | 21:47 |
jgriffith | Juno | 21:47 |
jgriffith | I'm inclined to be pretty darn picky | 21:47 |
thingee | I'm fine with denying for Juno at this point if it's not security related. Not even my stuff can get backported. | 21:47 |
jgriffith | k | 21:47 |
jgriffith | simple | 21:47 |
jgriffith | done | 21:47 |
jgriffith | moving on :) | 21:47 |
thingee | the reexport issue is not a security fix for example. I can't overrule the stable core folks, unless I really want to raise a particular case that needs to be backported for good reason. | 21:48 |
*** mriedem has quit IRC | 21:49 | |
*** abhiram_moturi has quit IRC | 21:56 | |
*** Adriano_ has quit IRC | 21:56 | |
*** abhiram_moturi has joined #openstack-cinder | 21:56 | |
Swanson | Is a Crewman Newman (ie me) supposed to be able to do a "cherry pick" or is that a core thing? | 21:57 |
*** lpetrut has quit IRC | 21:58 | |
*** rwsu has joined #openstack-cinder | 21:58 | |
thingee | Swanson: you can do it | 21:58 |
Swanson | thanks! | 21:59 |
*** madskier has joined #openstack-cinder | 21:59 | |
*** MentalRay has quit IRC | 22:01 | |
*** madskier has quit IRC | 22:04 | |
*** shakamunyi has quit IRC | 22:04 | |
*** barra204 has joined #openstack-cinder | 22:05 | |
*** EmilienM is now known as EmilienM|afk | 22:08 | |
kmartin | thingee or jgriffith, Do either of you have the ability to change the owner\assignee of a few BPs? | 22:10 |
thingee | kmartin: yes | 22:10 |
thingee | kmartin: what do you need? | 22:11 |
kmartin | could you move https://blueprints.launchpad.net/cinder/+spec/brocade-zone-driver-friendly-zone-names and https://blueprints.launchpad.net/cinder/+spec/brocade-zone-driver-virtualfabrics-support to aallen-m | 22:11 |
kmartin | aallen-m is Angela Smith | 22:12 |
thingee | kmartin: done. do you know what brcoades plans are for CI? | 22:12 |
kmartin | yes, they are working on it currently the plan is to start giving you updates | 22:13 |
kmartin | suppose to be in place by L1 | 22:13 |
thingee | nice | 22:13 |
kmartin | thingee, thank you | 22:14 |
*** alexpilotti has joined #openstack-cinder | 22:14 | |
openstackgerrit | Mike Perez proposed openstack/cinder: Convert mox to mock: tests/compute/test_service.py https://review.openstack.org/143281 | 22:15 |
*** rmesta has quit IRC | 22:17 | |
*** beekneemech has quit IRC | 22:18 | |
*** dimsum__ has quit IRC | 22:21 | |
openstackgerrit | Mike Perez proposed openstack/cinder: rtstool: Update credentials when target(LIO) already exists https://review.openstack.org/146333 | 22:21 |
*** garthb has quit IRC | 22:22 | |
*** annegentle has quit IRC | 22:23 | |
openstackgerrit | Mike Perez proposed openstack/cinder: Handle incorrect '--config-dir' param https://review.openstack.org/157749 | 22:26 |
*** dalgaaf has joined #openstack-cinder | 22:29 | |
*** madskier has joined #openstack-cinder | 22:33 | |
*** jlk has joined #openstack-cinder | 22:36 | |
*** annegentle has joined #openstack-cinder | 22:36 | |
jlk | thingee: hey there, I think I just found a bug in python-cinderclient, or cinder, in the way it does the new endpoint discovery, and I think it has to do with how we are terminating SSL at haproxy before cinder | 22:36 |
jlk | thingee: are you around to chat through this with me? | 22:37 |
*** ebalduf has quit IRC | 22:37 | |
*** madskier has quit IRC | 22:38 | |
thingee | jlk: I have a little bit of time to chat. going into a meeting soon | 22:38 |
jlk | thingee: so client is htting the unversioned endpoint to get a list of what versions are there | 22:38 |
jlk | server is giving back the v1 and v2 end versions | 22:38 |
jlk | but it's giving an href of an http:// url | 22:39 |
jlk | however, client access to these goes via https, through haproxy | 22:39 |
jlk | haproxy terminates ssl and hands off to cinder | 22:39 |
jlk | so what happens is client gets back an http url, tries to hit it, and can't. | 22:39 |
thingee | jlk: going to bring morganfainberg in. We're just using keystone client to to hit the base url | 22:39 |
thingee | jlk: also, you rservice catalog should be set for https no? | 22:40 |
jlk | it is | 22:41 |
thingee | your* | 22:41 |
thingee | that's where keystone client is getting the endpoints from | 22:41 |
thingee | your service catalog | 22:41 |
morganfainberg | thingee: yeah I'd expect the catalog to say https. | 22:41 |
jlk | https://gist.github.com/j2sol/e1307dcebfdf3d32f06d | 22:41 |
jlk | nah, it's hitting 8778/ to get a list of cinder versions | 22:41 |
morganfainberg | And it should use https then. jlk you don't use http for internal right? | 22:42 |
jlk | well, we send all our internal cinder requests through haproxy too | 22:42 |
morganfainberg | thingee: does cinder do something odd on discovery? Where t needs | 22:43 |
morganfainberg | To know itself is https? | 22:43 |
morganfainberg | thingee: it looks like the version discovery is saying http. | 22:44 |
jlk | cinder itself doesn't do https | 22:44 |
jlk | the https termination happens at haproxy | 22:44 |
*** julim has joined #openstack-cinder | 22:44 | |
jlk | here is the catalog https://gist.github.com/j2sol/a2b7236e834052c548de | 22:45 |
morganfainberg | Right. But everything should talk via haproxy. That tells me the version discovery should probably show the haproxy url and include https | 22:45 |
jlk | morganfainberg: I would thnik so | 22:45 |
jlk | but I don't see a way to tell cinder what it's own endpoint is | 22:46 |
morganfainberg | thingee: ^^ | 22:46 |
jlk | because I'm pretty sure it's a cinder API call to discover cinder versions | 22:46 |
morganfainberg | Right. | 22:46 |
jlk | http://developer.openstack.org/api-ref-blockstorage-v2.html | 22:46 |
jlk | GET on / | 22:46 |
tbarron | thingee: jgriffith: just drove home from work, opened IRC, saw the backlog and the backport to Juno discussion. | 22:47 |
tbarron | so w.r.t. https://review.openstack.org/#/c/180199/ | 22:48 |
tbarron | may aim was to get it into liberty, backport to kilo, backport to Juno. | 22:48 |
tbarron | The reason is that we have a large enterprise customer with private cloud with this bug in juno. | 22:49 |
tbarron | They can't move to kilo for several months, or to liberty for a long time. | 22:49 |
tbarron | The whole point is the bacport. I want to do xing/winston style overprovisioning in liberty which will obsolete this fix there. | 22:50 |
morganfainberg | jlk: https://github.com/openstack/cinder/blob/master/cinder/api/views/versions.py#L23 | 22:50 |
jlk | ah hah! | 22:50 |
jlk | there is a new config | 22:50 |
jgriffith | tbarron: common problem. Give them a patch, "git diff", "git apply" :) | 22:50 |
morganfainberg | Looks like! :) | 22:50 |
jlk | good deal | 22:50 |
tbarron | the reason I bring this up is not so much to make special appeal at the moment, but to suggest that we really think this through and have a very clear policy. | 22:50 |
morganfainberg | jlk: at least it's straight forward to change :) | 22:51 |
thingee | morganfainberg: thanks...yeah looks like that was added for liberty | 22:51 |
jgriffith | tbarron: we already do: https://wiki.openstack.org/wiki/StableBranch | 22:51 |
tbarron | which you guys may already have done. I'm not questioning that, | 22:51 |
tbarron | (concurrent chat there) | 22:51 |
morganfainberg | Yep. And makes sense for version discovery purposes. | 22:51 |
jgriffith | tbarron: particularly check "Support phases" and "Appropriate Fixes" | 22:52 |
thingee | tbarron: we haven't. This is with us conforming to stable core. | 22:52 |
jlk | arg | 22:52 |
jlk | doesn't look like it is in the config reference | 22:52 |
morganfainberg | jlk: that I can't speak to. | 22:52 |
jlk | what block does that go into | 22:52 |
tbarron | kk, looking. | 22:52 |
morganfainberg | Hmm sec | 22:52 |
thingee | jlk: I see it https://wiki.openstack.org/wiki/Cinder#Sample_cinder.conf | 22:52 |
jgriffith | tbarron: it's pretty much an exact summary of what we were discussing | 22:53 |
thingee | oh in the docs? | 22:53 |
thingee | jlk: ^ | 22:53 |
morganfainberg | jlk: looks like in [default] | 22:53 |
jgriffith | tbarron: incuding Kilo being a sort of "open book" still | 22:53 |
jlk | yeah, the config reference doc | 22:53 |
thingee | jlk: I'll check it out :) | 22:53 |
jlk | somewhere in http://docs.openstack.org/juno/config-reference/content/ch_configuring-openstack-block-storage.html | 22:53 |
morganfainberg | jlk: it gets registered without a group afaict | 22:53 |
jlk | is that in juno? | 22:53 |
thingee | gotta disappear for my meeting | 22:53 |
thingee | jlk: no liberty | 22:53 |
jlk | ARGHGH | 22:54 |
tbarron | so if i understand, my best bet is to keep on track & get this backport to kilo and give them a private patch till they can upgrade to kilo | 22:54 |
morganfainberg | Oh Juno sorry | 22:54 |
morganfainberg | Missed that part of the convo. | 22:54 |
jgriffith | tbarron: that should be met without any question yes | 22:54 |
jlk | so juno has the problem, but the fix is in liberty. | 22:54 |
jlk | wheeeee. | 22:54 |
jgriffith | tbarron: the juno thing is borderline, frankly I don't care. It's isolated to your driver so doesn't bother me. | 22:55 |
*** hodos|2 has quit IRC | 22:55 | |
thingee | jlk: I think juno is pinned to not have this version discovery in the client. Doesn't help with people doing pip install python-cinderclient though. | 22:55 |
jlk | yeah | 22:55 |
jlk | the disconnect there is a bit odd | 22:55 |
jgriffith | tbarron: but like I said to thingee I'll be kinda picky going forward | 22:55 |
jgriffith | tbarron: and the Icehouse patches that keep popping up are pretty lame | 22:56 |
*** rwsu has quit IRC | 22:56 | |
morganfainberg | Jlk: so there would need to be a backport to make that work with Juno. It just takes the base url of the req | 22:56 |
tbarron | jgriffith: kk, I should get this to land in liberty real soon now, so expect some cherry pick reviews coming towards stable core | 22:56 |
thingee | fwiw, I'm not picky with driver changes in Kilo. If we're talking about branches that are marked with security only fixes..I have a problem with that. | 22:56 |
thingee | :) | 22:56 |
jlk | yeah I may do a backport in our own fork | 22:56 |
jlk | because we can't necessarily control what our customers install for cinder clients | 22:56 |
morganfainberg | It might be possible to fix it with forcing it to use the req_for (same url thing we did for shibboleth) | 22:56 |
thingee | jlk: whoops sorry, that was directed to tbarron and jgriffith | 22:56 |
morganfainberg | Whatever header that is. | 22:57 |
tbarron | we won't ask anything for icehouse, and will understand if you say we have to do a private patch for the customer for juno, though of course we'd prefer a backport if that's possible, it being | 22:57 |
morganfainberg | If it exists. Then fall back to current workflow | 22:57 |
morganfainberg | If it doesn't exist. | 22:57 |
tbarron | a big issue for our customer and limited in scope to our driver | 22:57 |
tbarron | thanks jgriffith and thingee for your time on this. I won't belabor the issue now. | 22:57 |
morganfainberg | jlk: that should be backportable if you are looking to get code in that isn't carried by you. | 22:58 |
morganfainberg | And would also be a generally good way to avoid this issue with haproxy for the same kinds of reasons shibboleth needs it. | 22:58 |
*** vilobhmm1 has quit IRC | 23:04 | |
*** vilobhmm1 has joined #openstack-cinder | 23:04 | |
morganfainberg | jlk: worth opening a bug report on imo against stable. | 23:04 |
morganfainberg | But I defer to thingee on that overall. (Since you know... Not my project ;) | 23:05 |
thingee | jlk: my mistake, this is in Kilo. | 23:05 |
thingee | jlk: I would raise the bug against Juno though. | 23:05 |
thingee | see what stable folks think | 23:05 |
*** geguileo has quit IRC | 23:06 | |
morganfainberg | thingee: we should probably use the ha-proxy header that indicates the original req in Juno, kilo, and Liberty in either case and fall back to original workflow (this config or Juno code) if the header isn't present. | 23:06 |
morganfainberg | This is a bit below where you should care what cinder's url is. | 23:07 |
morganfainberg | S/should care/should have to care | 23:07 |
thingee | morganfainberg: this fix might be a bit hard to convince people on though because of it introducing a config opt | 23:07 |
morganfainberg | Nah. Just use the haproxy header that is passed down first. | 23:07 |
morganfainberg | Then fall back to the logic in Juno or the config if it does t exist. | 23:08 |
thingee | I see what you mean I think | 23:08 |
morganfainberg | I'll dig up the specifics of what I mean (x-forwarded for? I don't know what that header is...) | 23:08 |
morganfainberg | But you know, what proxies do when adhering to http spec ;) | 23:09 |
* morganfainberg is on phone or would be easier to lookup the info. | 23:09 | |
*** annegentle has quit IRC | 23:09 | |
*** abhiram_moturi has quit IRC | 23:13 | |
* thingee disappears to meeting | 23:13 | |
*** abhiram_moturi has joined #openstack-cinder | 23:13 | |
*** patrickeast has quit IRC | 23:15 | |
*** daneyon_ has joined #openstack-cinder | 23:16 | |
*** abhiram_moturi has quit IRC | 23:19 | |
*** abhiram_moturi has joined #openstack-cinder | 23:20 | |
*** rwsu has joined #openstack-cinder | 23:22 | |
*** geguileo has joined #openstack-cinder | 23:22 | |
*** barra204 has quit IRC | 23:22 | |
*** IanGovett1 has joined #openstack-cinder | 23:24 | |
*** IanGovett has quit IRC | 23:27 | |
*** abhiram_moturi has quit IRC | 23:33 | |
*** abhiram_moturi has joined #openstack-cinder | 23:33 | |
*** abhiram_moturi has quit IRC | 23:37 | |
*** abhiram_moturi has joined #openstack-cinder | 23:37 | |
*** abhiram_moturi has quit IRC | 23:39 | |
*** abhiram_moturi has joined #openstack-cinder | 23:40 | |
*** abhiram_moturi has quit IRC | 23:44 | |
*** abhiram_moturi has joined #openstack-cinder | 23:44 | |
*** alexpilotti_ has joined #openstack-cinder | 23:45 | |
*** hemna is now known as hemnafk | 23:46 | |
*** alexpilotti has quit IRC | 23:47 | |
*** alexpilotti_ is now known as alexpilotti | 23:47 | |
*** abhiram_moturi has quit IRC | 23:49 | |
*** abhiram_moturi has joined #openstack-cinder | 23:49 | |
*** anshul has quit IRC | 23:50 | |
*** tsekiyama has quit IRC | 23:51 | |
*** drjones has quit IRC | 23:52 | |
*** _cjones_ has joined #openstack-cinder | 23:53 | |
*** abhiram_moturi has quit IRC | 23:56 | |
*** abhiram_moturi has joined #openstack-cinder | 23:56 | |
*** rwsu has quit IRC | 23:56 | |
*** _cjones_ has quit IRC | 23:58 | |
*** dannywil_ has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!