Friday, 2015-05-08

*** dannywilson has quit IRC00:01
*** vilobhmm1 has quit IRC00:02
*** vilobhmm1 has joined #openstack-cinder00:02
*** setmason has quit IRC00:03
*** IanGovett has quit IRC00:09
*** abhiram_moturi has quit IRC00:11
*** abhiram_moturi has joined #openstack-cinder00:12
*** garthb has quit IRC00:14
*** abhiram_moturi has quit IRC00:16
*** abhiram_moturi has joined #openstack-cinder00:16
*** ociuhandu has quit IRC00:18
*** rmesta has quit IRC00:21
*** takedakn has quit IRC00:22
*** IlyaG has quit IRC00:23
*** abhiram_moturi has quit IRC00:27
*** abhiram_moturi has joined #openstack-cinder00:27
*** heyun has joined #openstack-cinder00:29
*** jamielennox|away is now known as jamielennox00:29
*** tsekiyam_ has joined #openstack-cinder00:31
*** tsekiyama has quit IRC00:34
*** mtanino has quit IRC00:36
*** tsekiyam_ has quit IRC00:36
*** zhenguo has joined #openstack-cinder00:38
*** MentalRay has joined #openstack-cinder00:44
*** leeantho has quit IRC00:46
*** abhiram_moturi has quit IRC00:50
*** abhiram_moturi has joined #openstack-cinder00:51
*** erlon has quit IRC00:51
*** ebalduf has joined #openstack-cinder00:51
*** ebalduf has quit IRC00:56
*** IlyaG has joined #openstack-cinder01:00
vilobhmm1hemna : ping01:01
*** abhiram_moturi has quit IRC01:03
*** abhiram_moturi has joined #openstack-cinder01:03
*** mkoderer has quit IRC01:05
*** jdurgin1 has quit IRC01:07
*** abhiram_moturi has quit IRC01:12
*** abhiram_moturi has joined #openstack-cinder01:12
*** mkoderer has joined #openstack-cinder01:14
*** abhiram_moturi has quit IRC01:17
*** abhiram_moturi has joined #openstack-cinder01:17
*** jwang_ has joined #openstack-cinder01:18
openstackgerritJohn Griffith proposed openstack/cinder: Check type match on create from source/snap  https://review.openstack.org/18126801:20
*** jwang has quit IRC01:21
*** akerr has joined #openstack-cinder01:23
*** akerr_ has joined #openstack-cinder01:25
*** Apoorva has quit IRC01:27
*** akerr has quit IRC01:28
*** aix has quit IRC01:31
*** abhiram_moturi has quit IRC01:33
*** abhiram_moturi has joined #openstack-cinder01:34
*** Apoorva has joined #openstack-cinder01:35
*** abhiram_moturi has quit IRC01:42
*** abhiram_moturi has joined #openstack-cinder01:42
*** kaisers has quit IRC01:44
*** Apoorva has quit IRC01:44
*** patrickeast has quit IRC01:45
*** primechuck has joined #openstack-cinder01:48
openstackgerritVilobh Meshram proposed openstack/cinder: Bad link in API version details response  https://review.openstack.org/18069101:50
*** winston-d_zzZ is now known as winston-d01:53
*** abhiram_moturi has quit IRC01:55
*** abhiram_moturi has joined #openstack-cinder01:55
*** kaisers has joined #openstack-cinder01:57
winston-djgriffith: ping01:59
*** dims_ has quit IRC02:00
*** dims has joined #openstack-cinder02:00
*** dims has quit IRC02:01
*** vilobhmm1 has quit IRC02:01
*** jwcroppe has joined #openstack-cinder02:03
*** abhiram_moturi has quit IRC02:08
*** abhiram_moturi has joined #openstack-cinder02:08
*** david-lyle has joined #openstack-cinder02:10
*** IlyaG has quit IRC02:11
*** abhiram_moturi has quit IRC02:15
*** abhiram_moturi has joined #openstack-cinder02:15
*** annegentle has joined #openstack-cinder02:15
jgriffithwinston-d: pong02:16
*** MentalRay has quit IRC02:17
*** abhiram_moturi has quit IRC02:19
*** patrickeast has joined #openstack-cinder02:19
*** abhiram_moturi has joined #openstack-cinder02:20
*** annegentle has quit IRC02:20
winston-djgriffith: I left some comment to bug #145282302:24
openstackbug 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-djgriffith: do those make sense to you?02:24
*** abhiram_moturi has quit IRC02:28
*** abhiram_moturi has joined #openstack-cinder02:29
*** abhiram_moturi has quit IRC02:31
*** abhiram_moturi has joined #openstack-cinder02:31
jgriffithwinston-d: no, it doesn't02:31
jgriffithwinston-d: I tested with solidfire -> lvm02:32
jgriffithwinston-d: fails every time, goes direct to driver.retype call02:32
jgriffithwinston-d: just like I stated in the LP bug02:32
winston-dreally?02:34
winston-dhow come02:35
jgriffithwinston-d: not sure02:36
jgriffithwinston-d: but maybe the filter check isn't working or getting called correclty02:36
jgriffithwinston-d: I can check it out again with a breakpoint in that scheduler portion02:36
jgriffithwinston-d: didn't realize that was there TBH02:36
jgriffithbut regardless it seems it's not working02:36
jgriffithwinston-d: I also think this is the issue that is being reported by NetApp and Dell02:37
jgriffithwinston-d: which is why I started looking at it last night02:37
*** abhiram_moturi has quit IRC02:39
*** abhiram_moturi has joined #openstack-cinder02:40
*** r-daneel has quit IRC02:40
winston-djgriffith: can you paste the extra-spec-list output for your experiment?02:40
*** annegentle has joined #openstack-cinder02:40
openstackgerritwanghao proposed openstack/cinder: Notification with volume and snaphost metadata  https://review.openstack.org/18040002:43
winston-djgriffith: and what's the error?02:44
*** abhiram_moturi has quit IRC02:46
*** abhiram_moturi has joined #openstack-cinder02:47
*** abhiram_moturi has quit IRC02:51
*** abhiram_moturi has joined #openstack-cinder02:51
*** abhiram_moturi has quit IRC02:53
*** harlowja_at_home has joined #openstack-cinder02:53
*** abhiram_moturi has joined #openstack-cinder02:54
*** abhiram_moturi has quit IRC02:56
*** abhiram_moturi has joined #openstack-cinder02:56
*** smallbig has joined #openstack-cinder02:59
*** dims has joined #openstack-cinder03:01
*** MentalRay has joined #openstack-cinder03:04
*** dims has quit IRC03:06
*** smoriya has joined #openstack-cinder03:07
*** jacksmion has joined #openstack-cinder03:08
openstackgerritEdwin 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/18128503:09
jacksmionwhy so quite?03:09
akerr_11pm03:09
akerr_fyi looks like taskflow update today broke remotefs derived drivers, at least netapp nfs and gluster show the errors03:10
ameade+103:12
*** bill_az has quit IRC03:12
*** IlyaG has joined #openstack-cinder03:12
harlowja_at_homeya, wonder whats going on there03:12
harlowja_at_homeonly commit i can think of is https://github.com/openstack/taskflow/commit/227cf5203:13
harlowja_at_homebut i don't see how that would of caused that03:13
*** akshai has joined #openstack-cinder03:14
ameadeyeah it's gotta be something subtle03:14
harlowja_at_homeya03:15
*** abhiram_moturi has quit IRC03:18
harlowja_at_homeif 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 investigation03:18
*** abhiram_moturi has joined #openstack-cinder03:18
harlowja_at_homeand/or doing other bisections03:18
ameadei could dig in but not until tomorrow03:19
*** akshai has quit IRC03:19
harlowja_at_homek, pretty weird errors03:20
ameadeyeah definitely, if the sqlalchemy version wasn't capped i would blame that03:20
harlowja_at_home:-p03:21
*** abhiram_moturi has quit IRC03:24
*** abhiram_moturi has joined #openstack-cinder03:25
*** ebalduf has joined #openstack-cinder03:25
*** r-daneel has joined #openstack-cinder03:26
openstackgerritwanghao proposed openstack/cinder: Fix response when querying host detail by host name  https://review.openstack.org/16260103:28
*** r-daneel has quit IRC03:31
*** MentalRay has quit IRC03:32
*** abhiram_moturi has quit IRC03:36
*** abhiram_moturi has joined #openstack-cinder03:36
*** vmtrooper has joined #openstack-cinder03:38
*** abhiram_moturi has quit IRC03:41
*** fanyaohong has joined #openstack-cinder03:41
*** vilobhmm1 has joined #openstack-cinder03:41
*** abhiram_moturi has joined #openstack-cinder03:41
harlowja_at_homeameade, let me know what u find out; and we can start disecting whats happening there03:42
*** vmtroope_ has joined #openstack-cinder03:42
ameadeyeah definitely, thanks for being readily available03:42
harlowja_at_homethink i might have an idea03:42
harlowja_at_homebut not 100% yet03:42
harlowja_at_homenp03:42
*** vmtrooper has quit IRC03:43
ameadethis line is in the volume manager vol_ref = flow_engine.storage.fetch('volume')03:43
harlowja_at_homeif 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_homeya03:43
ameadewhich seems related to that commit03:43
ameadeyeah03:43
*** annegentle has quit IRC03:43
harlowja_at_homei'm wondering if its https://github.com/openstack/taskflow/blob/master/taskflow/persistence/logbook.py#L44903:43
harlowja_at_homeperhaps the copy.copy there is acting odd with sqlalchemy03:43
*** annegentle has joined #openstack-cinder03:44
*** vmtroope_ has quit IRC03:44
harlowja_at_homeand 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_homevs the order before which was (update the original, save the original, and leave the original modified if the save failed)03:44
ameadesounds plausible03:44
harlowja_at_homei can easily put up a review to for that, but let's double check i guess first03:45
*** r-daneel has joined #openstack-cinder03:46
harlowja_at_homeeasy  fix if we just shouldn't do that copy.copy there03:47
*** patrickeast has quit IRC03:50
harlowja_at_homelet's see https://review.openstack.org/#/c/181288/ might be all thats needed03:51
harlowja_at_home^ which would be pretty durn simple :)03:52
ameadei'll see if i can test that really quick03:52
harlowja_at_homek, if u have time03:54
harlowja_at_homeif that works, can get a emergency fix/release out03:56
*** abhiram_moturi has quit IRC04:00
*** abhiram_moturi has joined #openstack-cinder04:00
*** akerr_ has quit IRC04:02
*** annegentle has quit IRC04:02
*** MentalRay has joined #openstack-cinder04:04
*** xyang1 has quit IRC04:06
ameadeharlowja_at_home: that seems to fix it from what i can tell04:07
harlowja_at_homewooot04:07
ameadei'll give it the old 3 times and it's good04:07
harlowja_at_home:)04:07
harlowja_at_homedamn copy!04:07
harlowja_at_homelol04:07
harlowja_at_homei'll open bug and tweak that commit a little04:08
harlowja_at_homethen can get another core to look it over04:08
harlowja_at_homeannnnd profit04:08
harlowja_at_home*if i can find another core, ha04:08
ameadeyep confirmed04:09
ameadenice04:09
ameadethanks a bunch04:09
harlowja_at_homecool, will wait for jenkins to come back with +1 and will get that tweaked04:09
harlowja_at_homethx for the help/checking04:09
ameadeawesome, do me a favor and reply on the ML04:10
ameadei'll plus 1 that patch04:10
harlowja_at_homewill do04:10
ameadety sir04:10
harlowja_at_homenp04:11
ameadetime to sleep, have a good night!04:11
harlowja_at_homeu too04:15
*** abhiram_moturi has quit IRC04:17
*** abhiram_moturi has joined #openstack-cinder04:18
*** changbl has joined #openstack-cinder04:18
*** abhiram_moturi has quit IRC04:22
*** abhiram_moturi has joined #openstack-cinder04:22
*** rushiagr_away is now known as rushiagr04:27
*** primechuck has quit IRC04:29
*** primechuck has joined #openstack-cinder04:30
*** ebalduf has quit IRC04:31
*** abhiram_moturi has quit IRC04:33
*** abhiram_moturi has joined #openstack-cinder04:34
*** primechuck has quit IRC04:34
*** abhiram_moturi has quit IRC04:40
*** abhiram_moturi has joined #openstack-cinder04:40
*** abhiram_moturi has quit IRC04:43
*** abhiram_moturi has joined #openstack-cinder04:43
*** deepakcs has joined #openstack-cinder04:43
*** abhiram_moturi has quit IRC04:47
*** abhiram_moturi has joined #openstack-cinder04:47
*** MentalRay has quit IRC04:48
*** julim has quit IRC04:49
*** abhiram_moturi has quit IRC04:54
*** abhiram_moturi has joined #openstack-cinder04:54
*** rhefner has joined #openstack-cinder04:56
*** julim has joined #openstack-cinder04:56
*** nkrinner has joined #openstack-cinder04:57
*** abhiram_moturi has quit IRC04:59
*** abhiram_moturi has joined #openstack-cinder04:59
*** Longgeek has joined #openstack-cinder05:00
*** annegentle has joined #openstack-cinder05:03
*** dulek has joined #openstack-cinder05:05
*** emagana has joined #openstack-cinder05:07
*** annegentle has quit IRC05:09
*** swamireddy1 has joined #openstack-cinder05:10
*** anuragpalsule has joined #openstack-cinder05:13
*** swamireddy1 has quit IRC05:14
*** swamireddy1 has joined #openstack-cinder05:14
*** abhiram_moturi has quit IRC05:16
*** abhiram_moturi has joined #openstack-cinder05:17
*** swamireddy2 has joined #openstack-cinder05:20
*** swamireddy1 has quit IRC05:22
*** abhiram_moturi has quit IRC05:25
*** abhiram_moturi has joined #openstack-cinder05:26
*** bkopilov has quit IRC05:27
*** Longgeek has quit IRC05:29
*** abhiram_moturi has quit IRC05:32
*** abhiram_moturi has joined #openstack-cinder05:33
*** abhiram_moturi has quit IRC05:35
*** abhiram_moturi has joined #openstack-cinder05:35
*** abhiram_moturi has quit IRC05:39
*** abhiram_moturi has joined #openstack-cinder05:40
*** abhiram_moturi has quit IRC05:42
*** abhiram_moturi has joined #openstack-cinder05:42
*** emagana has quit IRC05:44
*** vilobhmm1 has quit IRC05:44
*** bkopilov has joined #openstack-cinder05:44
*** emagana has joined #openstack-cinder05:45
*** abhiram_moturi has quit IRC05:47
*** abhiram_moturi has joined #openstack-cinder05:47
*** fanyaohong has quit IRC05:49
*** emagana has quit IRC05:49
*** elonden has joined #openstack-cinder05:53
*** abhiram_moturi has quit IRC05:57
*** BharatK has joined #openstack-cinder05:57
*** abhiram_moturi has joined #openstack-cinder05:58
*** abhiram_moturi has quit IRC06:02
*** abhiram_moturi has joined #openstack-cinder06:02
*** abhiram_moturi has quit IRC06:04
*** annegentle has joined #openstack-cinder06:04
*** belmoreira has joined #openstack-cinder06:04
*** abhiram_moturi has joined #openstack-cinder06:05
*** vilobhmm1 has joined #openstack-cinder06:05
*** harlowja_at_home has quit IRC06:09
*** annegentle has quit IRC06:10
*** abhiram_moturi has quit IRC06:13
*** Longgeek has joined #openstack-cinder06:13
*** abhiram_moturi has joined #openstack-cinder06:13
*** r-daneel has quit IRC06:15
*** abhiram_moturi has quit IRC06:18
*** abhiram_moturi has joined #openstack-cinder06:18
*** abhiram_moturi has quit IRC06:22
*** abhiram_moturi has joined #openstack-cinder06:22
*** abhiram_moturi has quit IRC06:24
*** abhiram_moturi has joined #openstack-cinder06:25
*** vilobhmm1 has quit IRC06:30
*** anshul has joined #openstack-cinder06:31
*** markvoelker has quit IRC06:33
*** abhiram_moturi has quit IRC06:37
*** abhiram_moturi has joined #openstack-cinder06:38
*** abhiram_moturi has quit IRC06:40
*** abhiram_moturi has joined #openstack-cinder06:40
*** abhiram_moturi has quit IRC06:42
*** abhiram_moturi has joined #openstack-cinder06:43
*** bkopilov has quit IRC06:46
*** anshul has quit IRC06:47
*** anshul has joined #openstack-cinder06:47
*** vnagpal has joined #openstack-cinder06:49
*** abhiram_moturi has quit IRC06:52
*** IlyaG has quit IRC06:52
*** IlyaG_ has joined #openstack-cinder06:52
*** abhiram_moturi has joined #openstack-cinder06:52
*** abhiram_moturi has quit IRC06:54
*** abhiram_moturi has joined #openstack-cinder06:54
*** IlyaG_ has quit IRC06:57
*** agarciam has joined #openstack-cinder07:00
*** ankit_ag has joined #openstack-cinder07:00
*** jacksmion has quit IRC07:02
*** rushiagr is now known as rushiagr_away07:04
*** annegentle has joined #openstack-cinder07:05
*** e0ne has joined #openstack-cinder07:09
*** annegentle has quit IRC07:10
*** markus_z has joined #openstack-cinder07:14
*** elonden has quit IRC07:16
*** sks has joined #openstack-cinder07:16
*** ronis has joined #openstack-cinder07:18
*** abhiram_moturi has quit IRC07:20
*** abhiram_moturi has joined #openstack-cinder07:21
*** junhongl has quit IRC07:21
*** junhongl has joined #openstack-cinder07:21
*** eglynn has quit IRC07:22
*** abhiram_moturi has quit IRC07:25
*** abhiram_moturi has joined #openstack-cinder07:26
*** abhiram_moturi has quit IRC07:28
*** abhiram_moturi has joined #openstack-cinder07:28
*** e0ne has quit IRC07:29
*** e0ne has joined #openstack-cinder07:33
*** jamielennox is now known as jamielennox|away07:36
openstackgerritwanghao proposed openstack/cinder: Notification with volume and snaphost metadata  https://review.openstack.org/18040007:38
*** rushiagr_away is now known as rushiagr07:40
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Add volume status to error messages in backup create flow  https://review.openstack.org/17156607:45
*** ronis_ has joined #openstack-cinder07:46
*** ronis has quit IRC07:47
*** abhiram_moturi has quit IRC07:53
*** abhiram_moturi has joined #openstack-cinder07:53
*** abhiram_moturi has quit IRC07:57
*** abhiram_moturi has joined #openstack-cinder07:58
*** abhiram_moturi has quit IRC08:00
*** abhiram_moturi has joined #openstack-cinder08:00
*** abhiram_moturi has quit IRC08:04
*** abhiram_moturi has joined #openstack-cinder08:04
*** tobe has joined #openstack-cinder08:05
openstackgerritwanghao proposed openstack/cinder: Fix response when querying host detail by host name  https://review.openstack.org/16260108:05
*** nikeshm has quit IRC08:06
*** annegent_ has joined #openstack-cinder08:06
*** markvoelker has joined #openstack-cinder08:07
*** abhiram_moturi has quit IRC08:09
*** abhiram_moturi has joined #openstack-cinder08:09
*** abhiram_moturi has quit IRC08:11
*** annegent_ has quit IRC08:11
*** abhiram_moturi has joined #openstack-cinder08:12
*** abhiram_moturi has quit IRC08:14
*** abhiram_moturi has joined #openstack-cinder08:14
wanghaohi 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
wanghaoOr more review is welcome if you think there still exist problem about the code:)08:17
*** eglynn has joined #openstack-cinder08:20
*** chlong has quit IRC08:20
*** sks has quit IRC08:21
*** e0ne has quit IRC08:23
*** abhiram_moturi has quit IRC08:27
*** abhiram_moturi has joined #openstack-cinder08:27
*** abhiram_moturi has quit IRC08:29
*** abhiram_moturi has joined #openstack-cinder08:29
*** tobe has quit IRC08:31
*** abhiram_moturi has quit IRC08:36
*** abhiram_moturi has joined #openstack-cinder08:36
*** markvoelker has quit IRC08:38
*** dalgaaf has joined #openstack-cinder08:41
*** Lee1092 has joined #openstack-cinder08:47
*** bkopilov has joined #openstack-cinder08:51
*** abhiram_moturi has quit IRC08:53
*** abhiram_moturi has joined #openstack-cinder08:54
*** jwcroppe has quit IRC08:58
*** jwcroppe has joined #openstack-cinder08:58
*** e0ne has joined #openstack-cinder08:58
*** e0ne is now known as e0ne_08:58
*** abhiram_moturi has quit IRC09:02
*** abhiram_moturi has joined #openstack-cinder09:03
*** abhiram_moturi has quit IRC09:05
*** abhiram_moturi has joined #openstack-cinder09:05
*** annegentle has joined #openstack-cinder09:07
*** e0ne_ is now known as e0ne09:07
*** abhiram_moturi has quit IRC09:09
*** abhiram_moturi has joined #openstack-cinder09:10
*** annegentle has quit IRC09:12
*** abhiram_moturi has quit IRC09:14
*** abhiram_moturi has joined #openstack-cinder09:14
*** abhiram_moturi has quit IRC09:18
*** abhiram_moturi has joined #openstack-cinder09:19
*** e0ne is now known as e0ne_09:21
*** abhiram_moturi has quit IRC09:25
*** abhiram_moturi has joined #openstack-cinder09:25
openstackgerritwanghao proposed openstack/cinder-specs: Support to import image to an existed volume  https://review.openstack.org/18133909:27
*** IanGovett has joined #openstack-cinder09:27
*** dulek_ has joined #openstack-cinder09:28
*** dulek has quit IRC09:29
*** abhiram_moturi has quit IRC09:34
*** abhiram_moturi has joined #openstack-cinder09:34
*** ndipanov has joined #openstack-cinder09:35
*** abhiram_moturi has quit IRC09:39
*** abhiram_moturi has joined #openstack-cinder09:39
*** abhiram_moturi has quit IRC09:41
*** abhiram_moturi has joined #openstack-cinder09:41
*** abhiram_moturi has quit IRC09:43
*** abhiram_moturi has joined #openstack-cinder09:44
*** alexpilotti has joined #openstack-cinder09:48
*** abhiram_moturi has quit IRC09:52
*** abhiram_moturi has joined #openstack-cinder09:52
*** ociuhandu has joined #openstack-cinder09:54
*** e0ne_ is now known as e0ne09:55
*** abhiram_moturi has quit IRC10:01
*** abhiram_moturi has joined #openstack-cinder10:01
*** abhiram_moturi has quit IRC10:03
*** coolsvap|afk is now known as coolsvap10:03
*** abhiram_moturi has joined #openstack-cinder10:04
*** annegent_ has joined #openstack-cinder10:08
*** abhiram_moturi has quit IRC10:10
*** abhiram_moturi has joined #openstack-cinder10:10
*** abhiram_moturi has quit IRC10:12
*** abhiram_moturi has joined #openstack-cinder10:13
*** annegent_ has quit IRC10:13
*** zhenguo has quit IRC10:17
*** abhiram_moturi has quit IRC10:19
*** abhiram_moturi has joined #openstack-cinder10:19
*** agarciam has quit IRC10:20
*** avishay_ has joined #openstack-cinder10:25
*** avishay_ has quit IRC10:28
*** agarciam has joined #openstack-cinder10:30
*** abhiram_moturi has quit IRC10:32
*** abhiram_moturi has joined #openstack-cinder10:33
*** markvoelker has joined #openstack-cinder10:35
*** ebalduf has joined #openstack-cinder10:39
*** markvoelker has quit IRC10:40
*** dulek___ has joined #openstack-cinder10:43
*** dulek_ has quit IRC10:43
*** ebalduf has quit IRC10:43
*** abhiram_moturi has quit IRC10:47
*** abhiram_moturi has joined #openstack-cinder10:47
*** abhiram_moturi has quit IRC10:49
*** abhiram_moturi has joined #openstack-cinder10:50
*** abhiram_moturi has quit IRC10:54
*** abhiram_moturi has joined #openstack-cinder10:54
*** e0ne is now known as e0ne_10:56
*** abhiram_moturi has quit IRC10:58
*** anuragpalsule has quit IRC10:59
*** abhiram_moturi has joined #openstack-cinder10:59
*** smoriya has quit IRC11:00
*** abhiram_moturi has quit IRC11:03
*** abhiram_moturi has joined #openstack-cinder11:04
openstackgerrityogeshprasad proposed openstack/cinder: Add chap support to CloudByte cinder driver  https://review.openstack.org/17373311:05
*** annegent_ has joined #openstack-cinder11:08
*** alecv has quit IRC11:12
*** annegent_ has quit IRC11:13
*** dims has joined #openstack-cinder11:13
*** e0ne_ is now known as e0ne11:14
*** cdelatte has joined #openstack-cinder11:17
*** delattec has joined #openstack-cinder11:17
*** vnagpal_ has joined #openstack-cinder11:21
*** heyun has quit IRC11:22
*** vnagpal has quit IRC11:24
*** dims has quit IRC11:28
*** e0ne is now known as e0ne_11:28
*** dims has joined #openstack-cinder11:29
*** e0ne_ is now known as e0ne11:30
*** vnagpal_ has quit IRC11:35
*** akerr has joined #openstack-cinder11:36
*** markvoelker has joined #openstack-cinder11:36
*** akerr_ has joined #openstack-cinder11:37
*** akerr has quit IRC11:40
*** aswadr has joined #openstack-cinder11:42
*** abhiram_moturi has quit IRC11:44
*** abhiram_moturi has joined #openstack-cinder11:44
*** agarciam has quit IRC11:45
*** r-daneel has joined #openstack-cinder11:50
*** deepakcs has quit IRC11:53
*** abhiram_moturi has quit IRC11:55
*** ndipanov has quit IRC11:56
*** r-daneel has quit IRC11:56
*** abhiram_moturi has joined #openstack-cinder11:56
*** abhiram_moturi has quit IRC11:58
*** abhiram_moturi has joined #openstack-cinder11:58
*** fthiagogv has joined #openstack-cinder11:59
*** annegentle has joined #openstack-cinder12:00
*** akerr_ has quit IRC12:01
*** delattec has quit IRC12:01
*** cdelatte has quit IRC12:01
*** abhiram_moturi has quit IRC12:05
*** abhiram_moturi has joined #openstack-cinder12:05
*** markvoelker has quit IRC12:07
*** markvoelker has joined #openstack-cinder12:07
*** dalgaaf has quit IRC12:07
*** akerr has joined #openstack-cinder12:08
*** agarciam has joined #openstack-cinder12:10
*** e0ne is now known as e0ne_12:10
*** abhiram_moturi has quit IRC12:12
*** abhiram_moturi has joined #openstack-cinder12:12
*** Yogi1 has joined #openstack-cinder12:13
*** e0ne_ is now known as e0ne12:13
*** dulek___ is now known as dulek_12:16
*** dulek_ is now known as dulek12:16
*** Yogi1 has quit IRC12:18
dulekDuncanT, harlowja: Hi, we were able to produce kind of PoC exchanging file locks to tooz locks.12:18
dulekDuncanT, harlowja: https://github.com/bluex-pl/cinder/tree/locks - you may want to take a look. :)12:19
dulekharlowja: This is doing heartbeats in green threads, so this would probably need to change.12:20
*** mtanino has joined #openstack-cinder12:20
*** akerr has quit IRC12:22
*** abhiram_moturi has quit IRC12:26
*** abhiram_moturi has joined #openstack-cinder12:27
*** IanGovett1 has joined #openstack-cinder12:28
*** IanGovett has quit IRC12:30
*** abhiram_moturi has quit IRC12:35
*** abhiram_moturi has joined #openstack-cinder12:36
*** rushiagr is now known as rushiagr_away12:37
*** abhiram_moturi has quit IRC12:40
*** abhiram_moturi has joined #openstack-cinder12:40
*** ankit_ag has quit IRC12:42
*** aswadr has quit IRC12:43
*** juzuluag has joined #openstack-cinder12:43
*** timcl has joined #openstack-cinder12:45
*** chlong has joined #openstack-cinder12:47
*** lpetrut has joined #openstack-cinder12:52
*** e0ne is now known as e0ne_12:56
*** krtaylor has quit IRC12:56
*** agarciam has quit IRC12:56
*** e0ne_ is now known as e0ne12:57
*** thangp has joined #openstack-cinder12:57
*** dulek has quit IRC13:00
*** annegentle has quit IRC13:00
*** annegentle has joined #openstack-cinder13:02
*** ociuhandu has quit IRC13:05
*** nkrinner has quit IRC13:06
*** krtaylor has joined #openstack-cinder13:08
*** rushiagr_away is now known as rushiagr13:09
*** abhiram_moturi has quit IRC13:13
*** abhiram_moturi has joined #openstack-cinder13:14
*** EmilienM|afk is now known as EmilienM13:15
*** dims has quit IRC13:19
*** dims has joined #openstack-cinder13:20
*** abhiram_moturi has quit IRC13:23
*** abhiram_moturi has joined #openstack-cinder13:23
*** jungleboyj has quit IRC13:23
*** abhiram_moturi has quit IRC13:27
*** anuragpalsule has joined #openstack-cinder13:27
*** abhiram_moturi has joined #openstack-cinder13:28
e0nedoes anyone use FakeISCSIDriver?13:29
*** anuragpalsule1 has joined #openstack-cinder13:29
Adriano_thingee13:31
Adriano_hello13:31
Adriano_thingee: https://review.openstack.org/#/c/180723/13:31
Adriano_can you pls aprrove this13:31
Adriano_?13:31
*** primechuck has joined #openstack-cinder13:32
*** anuragpalsule has quit IRC13:32
*** crose has joined #openstack-cinder13:34
*** dansmith is now known as superdan13:35
*** coolsvap is now known as coolsvap|afk13:37
*** mriedem_away is now known as mriedem13:37
*** esker has joined #openstack-cinder13:40
*** ebalduf has joined #openstack-cinder13:41
*** esker has quit IRC13:41
*** esker has joined #openstack-cinder13:41
*** esker has quit IRC13:41
*** esker has joined #openstack-cinder13:41
*** mtanino has quit IRC13:42
*** erlon has joined #openstack-cinder13:44
*** annegentle has quit IRC13:44
*** BharatK has quit IRC13:44
*** ebalduf has quit IRC13:45
*** ociuhandu has joined #openstack-cinder13:45
*** annegentle has joined #openstack-cinder13:45
*** markus_z has quit IRC13:47
*** coolsvap has joined #openstack-cinder13:47
*** sgotliv has joined #openstack-cinder13:48
*** eharney has joined #openstack-cinder13:49
*** openstackgerrit has quit IRC13:51
*** dims is now known as dimsum__13:51
*** openstackgerrit has joined #openstack-cinder13:51
*** ociuhandu has quit IRC13:53
*** abhiram_moturi has quit IRC13:57
*** abhiram_moturi has joined #openstack-cinder13:57
*** lpabon has joined #openstack-cinder13:59
*** abhiram_moturi has quit IRC14:01
*** BharatK has joined #openstack-cinder14:01
*** abhiram_moturi has joined #openstack-cinder14:02
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Fix FakeISCSIDriver and FakeISERDriver  https://review.openstack.org/18139614:02
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Fix FakeISCSIDriver and FakeISERDriver  https://review.openstack.org/18139614:03
*** BharatK has quit IRC14:07
*** swamireddy2 has quit IRC14:09
*** bnemec has joined #openstack-cinder14:13
e0nejgriffith: morning John. are you around?14:14
*** annegentle has quit IRC14:16
*** BharatK has joined #openstack-cinder14:19
*** lpetrut has quit IRC14:20
*** ebalduf has joined #openstack-cinder14:22
*** MentalRay has joined #openstack-cinder14:22
*** ebalduf has quit IRC14:22
*** ebalduf has joined #openstack-cinder14:24
*** shakamunyi has joined #openstack-cinder14:25
*** abhiram_moturi has quit IRC14:28
*** lpetrut has joined #openstack-cinder14:28
*** abhiram_moturi has joined #openstack-cinder14:29
*** bnemec is now known as beekneemech14:32
*** madskier has joined #openstack-cinder14:32
*** emagana has joined #openstack-cinder14:32
*** anshul has quit IRC14:33
jgriffithe0ne: I am now, what's up?14:34
e0nejgriffith: afair, we talked about fake driver last week14:34
jgriffithe0ne: we did indeed14:34
e0nei've got several questions ablout it14:35
e0neat least, it doesn't work with shceduler :(.14:35
*** ebalduf has quit IRC14:35
jgriffithe0ne: well, I think I stated pretty clear that I suspected it needed updating14:36
e0nei've added you to review request with fix14:36
jgriffithe0ne: ahh.. excellent14:36
jgriffithwinston-d: ping14:36
e0nebut... we've got 2(!) FakeISCSIDriver classes..14:36
jgriffithYuk14:37
e0nethe second one in unit tests https://github.com/openstack/cinder/search?utf8=%E2%9C%93&q=FakeISCSIDriver14:37
e0nei'm going to remove it14:37
jgriffithOh boy14:37
*** tsekiyama has joined #openstack-cinder14:37
jgriffithe0ne: I don't remember how/why that happened.  But there was a reason at one point14:37
jgriffithe0ne: I'm fine with seeing it removed and consolidating14:37
e0nejgriffith: ok. thanks14:38
jgriffithmight drive more intrest/activity in it14:38
e0nei want to make performance tests with fake driver14:38
*** ebalduf has joined #openstack-cinder14:38
jgriffithe0ne: seems like a good idea14:38
e0neto test _only_ cinder part w/o storage impact14:38
jgriffithe0ne: that was the whole reason for it to begin with, so I certainly agree :)14:39
e0negood:)14:39
e0nejgriffith: one more question about fakes14:39
j_kingDuncanT: 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 work14:39
e0nejgriffith: why do we need FakeTarget?14:40
*** Guest58709 has joined #openstack-cinder14:41
e0nejgriffith: i don't see any benefits of it because it's not enough to get lvm driver wroking14:41
*** madskier has quit IRC14:42
e0nei didn't test it with other drivers14:42
*** bkopilov has quit IRC14:46
anuragpalsule1Need 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 notifications14:46
*** sgotliv has quit IRC14:48
*** jungleboyj has joined #openstack-cinder14:49
*** hemnafk is now known as hemna14:50
jgriffithe0ne: well, it was a place holder14:50
*** sgotliv has joined #openstack-cinder14:51
*** madskier has joined #openstack-cinder14:51
*** madskier has quit IRC14:51
jgriffithe0ne: I thought it would be a good idea to have "fake" versions of each of the drivers in with the modules14:52
*** anuragpalsule has joined #openstack-cinder14:52
*** anuragpalsule1 has quit IRC14:52
jgriffithe0ne: and at some point flush them out and hopefully see them as the central use for tests and such14:52
*** madskier has joined #openstack-cinder14:53
*** e0ne is now known as e0ne_14:55
*** abhiram_moturi has quit IRC14:57
*** abhiram_moturi has joined #openstack-cinder14:58
*** mtecer has joined #openstack-cinder14:58
*** mtanino has joined #openstack-cinder15:03
*** juzuluag has quit IRC15:03
*** sgotliv has quit IRC15:05
*** e0ne_ is now known as e0ne15:07
e0nejgriffith: good idea. i hope. i'll share tests results with fake driver next week15:08
openstackgerritTom Swanson proposed openstack/cinder: Dell: Added support for update_migrated_volume  https://review.openstack.org/18116015:10
*** rmesta has joined #openstack-cinder15:12
*** abhiram_moturi has quit IRC15:14
*** ronis_ has quit IRC15:14
*** abhiram_moturi has joined #openstack-cinder15:14
*** abhiram_moturi has quit IRC15:16
*** abhiram_moturi has joined #openstack-cinder15:17
*** belmoreira has quit IRC15:18
*** dannywilson has joined #openstack-cinder15:20
*** dannywil_ has joined #openstack-cinder15:24
*** dannywilson has quit IRC15:24
*** ebalduf has quit IRC15:25
*** shakamunyi has quit IRC15:25
*** ebalduf has joined #openstack-cinder15:27
*** Apoorva has joined #openstack-cinder15:29
*** ebalduf_ has joined #openstack-cinder15:29
*** sks has joined #openstack-cinder15:29
*** Apoorva_ has joined #openstack-cinder15:31
*** jdurgin1 has joined #openstack-cinder15:31
*** ebalduf has quit IRC15:32
e0newhere are sources located for http://docs.openstack.org/developer/cinder/devref/filter_scheduler.html?15:32
e0nethere are a lot of errors, copy-paste from nova15:32
e0neand where is new docs?15:34
hemnawhat's docs?15:34
hemna:PO15:34
hemnaheh15:34
*** Apoorva has quit IRC15:35
*** mtecer has quit IRC15:35
*** IlyaG has joined #openstack-cinder15:36
*** juzuluag has joined #openstack-cinder15:36
*** jungleboyj has quit IRC15:37
*** garthb has joined #openstack-cinder15:37
e0nei'm disapointed in openstack docs in first day in openstack world15:38
Guest58709e0ne: 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 annegentle15:40
hemnaI dunno, I think the docs are pretty good, all things considered.15:40
annegentlehemna: yeah you guys do a pretty good job really15:41
hemnawe spend a lot of time each release updating ours15:41
*** jaypipes is now known as leakypipes15:42
e0neGuest58709: actually, it is docs for 2013.1 release, current docs is not so detailed:(15:43
e0nehemna: i agree that we have better docs than few years ago. and they become better each release15:46
annegentlee0ne: then probably the copy/paste problem is "fixed"15:46
*** sks has quit IRC15:46
e0neannegentle: agree. now there is no copy-paste but we have 'other' docs for schedulere than in the past.15:47
e0nei need go deeper in it to understand what is good and what should be improved15:48
e0nehemna: i should go offline 'early' today. if you get any feadback for cinder agent, i'll be glad to discuss it on Monday15:51
hemnae0ne, I haven't had time to look at it yet :(15:51
hemnae0ne, https://review.openstack.org/#/c/181102/15:51
hemnacan I Get some love on that guy ?15:51
hemnasimple enough patch15:51
e0nehemna: i've tolk to zigo about packaging of os-brick - it will be next chellenge for me:)15:52
e0nes/tolk/told15:52
*** madskier has quit IRC15:53
hemnapackaging ?15:53
hemnabrb15:53
e0nedeb and rpm:)15:53
openstackgerritEdwin Wang proposed openstack/cinder: flashsystem reports error while running tests with multi-thread  https://review.openstack.org/18128515:55
openstackgerritEdwin Wang proposed openstack/cinder: FlashSystem reports error while running tests with multi-thread  https://review.openstack.org/18128515:55
openstackgerritEdwin Wang proposed openstack/cinder: FlashSystem reports error while running tests with multi-thread  https://review.openstack.org/18128515:56
*** openstackstatus has quit IRC15:56
*** openstackstatus has joined #openstack-cinder15:56
*** ChanServ sets mode: +v openstackstatus15:56
*** patrickeast has joined #openstack-cinder15:57
openstackgerritEdwin Wang proposed openstack/cinder: FlashSystem reports error while running tests with multi-thread  https://review.openstack.org/18128516:01
*** annegentle has quit IRC16:02
*** e0ne is now known as e0ne_16:04
*** anuragpalsule has quit IRC16:06
hemnae0ne_, ah ok cool16:06
*** Yogi1 has joined #openstack-cinder16:08
*** dencaval has joined #openstack-cinder16:10
*** abhiram_moturi has quit IRC16:13
*** abhiram_moturi has joined #openstack-cinder16:13
hemnahaving a problem with mock in a unit test16:16
hemnait doesn't seem to work in some cases16:16
hemnahttps://gist.github.com/WaltHP/9480fcb65938d5cb8e4b16:16
hemnaany ideas?16:16
*** shakamunyi has joined #openstack-cinder16:16
*** vilobhmm1 has joined #openstack-cinder16:18
openstackgerritJohn Haan proposed openstack/python-cinderclient: Adding search options to backup list  https://review.openstack.org/7823316:19
*** sgotliv has joined #openstack-cinder16:21
*** rushil has joined #openstack-cinder16:21
*** abhiram_moturi has quit IRC16:24
*** abhiram_moturi has joined #openstack-cinder16:24
openstackgerritJohn Haan proposed openstack/python-cinderclient: Adding search options to backup list  https://review.openstack.org/7823316:25
*** jungleboyj has joined #openstack-cinder16:28
*** Apoorva_ has quit IRC16:29
*** leeantho has joined #openstack-cinder16:32
*** shakamunyi has quit IRC16:33
*** btran_ has joined #openstack-cinder16:34
jgriffithtbarron: smcginnis Swanson Would you folks be able to review and maybe test this out against the problems you were seeing: https://goo.gl/tCnUQi16:34
jgriffithtbarron: smcginnis Swanson It fixed things up in my env16:34
*** akerr has joined #openstack-cinder16:36
*** hodos has joined #openstack-cinder16:37
*** dustins has joined #openstack-cinder16:38
*** marcusvrn has quit IRC16:39
*** rushil has quit IRC16:40
*** marcusvrn has joined #openstack-cinder16:41
*** ronis_ has joined #openstack-cinder16:43
hodosmorning. does any1 see errors like this today while running tempest? ObjectDereferencedError: Can't emit change event for attribute 'Volume.provider_location'16:44
hodoslooks like new today16:44
*** harlowja has quit IRC16:44
*** harlowja has joined #openstack-cinder16:44
harlowjahodos that should no longer be happening :)16:44
harlowjahttp://lists.openstack.org/pipermail/openstack-dev/2015-May/063693.html16:45
harlowjaso recheck, and it should be gone16:45
hodosharlowja: oh, so it was fixed just recently...ok16:46
harlowjayup16:46
harlowjaresolved last night, released this morning16:46
hodosharlowja: hmm, it says my branch is up with master16:47
*** MentalRay has quit IRC16:48
harlowjayes, its not really a cinder code issue, dependent package issue16:48
*** shakamunyi has joined #openstack-cinder16:48
harlowjaso if u have a virtualenv, do pip install 'taskflow==0.10.1'16:48
hodosharlowja: is it icluded in devstack ?16:49
harlowjaunsure,  devstack afaik just installs things16:50
harlowjaso i would assume so16:50
harlowjaeither way  u need to get that updated version to resolve that :)16:50
*** Yogi1 has quit IRC16:50
*** leeantho has quit IRC16:51
*** Apoorva has joined #openstack-cinder16:52
*** rushiagr is now known as rushiagr_away16:53
*** ebalduf has joined #openstack-cinder16:53
*** crose has quit IRC16:57
*** ebalduf has quit IRC16:59
*** annegentle has joined #openstack-cinder16:59
*** abhiram_moturi has quit IRC17:01
*** abhiram_moturi has joined #openstack-cinder17:02
zigoe0ne_: I'll be happy to help.17:03
zigoe0ne_: 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/78442917:04
openstackDebian 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-cinder17:04
*** bkopilov has joined #openstack-cinder17:06
*** shakamunyi has quit IRC17:07
*** MentalRay has joined #openstack-cinder17:10
jgriffithvilobhmm1: ping17:11
vilobhmm1jgriffith : hi17:11
jgriffithvilobhmm1: hi there17:11
vilobhmm1yes sir17:12
harlowjasir yes sir17:12
harlowja:-P17:12
jgriffithvilobhmm1: hey.. I had a quick question on https://goo.gl/NqdKMh17:12
jgriffithsir no sir17:12
jgriffithwhat17:12
jgriffithwait17:12
jgriffith:)17:12
vilobhmm1sure17:12
jgriffithvilobhmm1: so I'm fine with the change, but I'm curious WTF are we pulling somethign in to docs from unit tests?17:12
jgriffithvilobhmm1: or am I reading something wrong ?17:13
*** Yogi1 has joined #openstack-cinder17:13
smcginnisjgriffith: I tested out that patch. It didn't address our issues until Swanson's patch, but it looked fine.17:13
jgriffithsmcginnis: thanks.. so your driver retype gets called but returns an error?17:13
jgriffithsmcginnis: I mean, that patch does nothing for you?17:14
smcginnisjgriffith: Our problem was we didn't implement the update call.17:14
smcginnisjgriffith: No, it didn't seem to make a difference.17:14
jgriffithsmcginnis: well, neither did I :)  but I don't need it now17:14
smcginnisHmm17:14
vilobhmm1the whole point of the change https://review.openstack.org/#/c/180691/5 is to fix the broken link the docs are generated by another repo17:14
jgriffithsmcginnis: hmm, so how are you guys handling the driver.retype call17:14
vilobhmm1we have a unit test which basically shoudl point the right api doscs for each version17:14
vilobhmm1jgriffith : ^^17:14
Swansonjgriffith: we dont17:15
jgriffithvilobhmm1: Ok, so I was confused by the bug17:15
jgriffithvilobhmm1: thanks17:15
smcginnisjgriffith: We needed to update our volume after migrate.17:15
smcginnisjgriffith: https://review.openstack.org/#/c/181160/3/cinder/volume/drivers/dell/dell_storagecenter_api.py17:15
jgriffithSwanson: you don't imnplement the retype command?17:15
smcginnisjgriffith: Your patch handles if it is across different backend types, right?17:15
vilobhmm1jgriffith : the docs are in the openstack/api-site repo in WADL, handwritten, maintained17:16
Swansonnope.  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
jgriffithsmcginnis: well, among other things.  Mostly it just keeps from calling the driver method if we already know the types are specificed to be different backends17:16
jgriffithSwanson: Ahh, ok, that makes total sense then17:16
jgriffithSwanson: anything that doesn't impl it just returns False on the call via default driver and the migrate gets picked up17:16
vilobhmm1we are not pulling anything but just verifying that a correct link is provided17:17
smcginnisSounds like there were multiple issues around retype/migrate depending on your scenario.17:17
jgriffithSwanson: so the bug i'm addressing there is only for backends that implement retype17:17
jgriffithsmcginnis: that's an understatement :)17:17
vilobhmm1annegentle : ping17:17
smcginnis;)17:17
tbarronjgriffith: so there was some confusion here.  We don't implement retype yet either.  I was17:17
jgriffithvilobhmm1: well I get that.. but what I don't get is why an api unit test has *anything* to do with docs17:18
tbarronlooking at LVM retype to understand what we'd need to do.17:18
*** shakamunyi has joined #openstack-cinder17:18
jgriffithtbarron: don't look at LVM it doesn't do anything either :)17:18
smcginnisIs the driver implemented retype for retyping within the same backend?17:18
jgriffithsmcginnis: correct17:18
jgriffithyes17:18
tbarronIIRC I tried retyping a volume from one of our iscsi to LVM, but I was pretty confused.17:18
jgriffithtbarron: :(17:19
smcginnisOK, thought so. And since we currently only do one type per backend we don't need it.17:19
jgriffithsmcginnis: exactly17:19
jgriffithsmcginnis: so *my* bug only applies to backends that support multiple types, most don't17:19
vilobhmm1jgriffith : hth17:19
jgriffithvilobhmm1: hth?17:19
SwansonIf we add support for storage_types or something like that via volume type we would have to add it.17:19
vilobhmm1hope that helps17:19
* jgriffith googles hipster slang/acronyms17:19
jgriffithvilobhmm1: LOL17:19
jgriffiththanks.. you're faster than google :)17:19
tbarronjgriffith: 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:P17:19
jgriffithtbarron: he's not17:20
jgriffithtbarron: if you read my response in the bug17:20
tbarronjgriffith: ok, I'll look.  I've had my head down on some other stuff.17:20
*** ronis_ has quit IRC17:20
*** IanGovett1 has quit IRC17:20
*** annegentle has quit IRC17:21
SwansonThat salad had a criminal amount of bacon on it.  Not even pretending to eat healthy...17:21
*** IanGovett has joined #openstack-cinder17:22
*** shakamunyi has quit IRC17:23
smcginnisSwanson: Hah!17:24
*** abhiram_moturi has quit IRC17:26
*** abhiram_moturi has joined #openstack-cinder17:28
*** ebalduf has joined #openstack-cinder17:28
*** daneyon has left #openstack-cinder17:29
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Assign the platform after declaration  https://review.openstack.org/18147517:31
hemna"criminal amount of bacon"   how is that possible ?17:31
*** ebalduf_ has quit IRC17:32
*** e0ne_ is now known as e0ne17:32
SwansonWhen your ventricles scream out in terror you've probably gone too far with the bacon.17:32
*** anshul has joined #openstack-cinder17:33
smcginnisInconceivable.17:34
*** e0ne has quit IRC17:34
*** shakamunyi has joined #openstack-cinder17:34
*** bkopilov has quit IRC17:39
*** fthiagogv has quit IRC17:39
*** merooney has joined #openstack-cinder17:40
*** sandywalsh has left #openstack-cinder17:42
*** anshul has quit IRC17:42
*** IanGovett has quit IRC17:42
*** annegentle has joined #openstack-cinder17:43
*** IanGovett has joined #openstack-cinder17:45
*** merooney has quit IRC17:45
*** ronis_ has joined #openstack-cinder17:45
*** MentalRay has quit IRC17:48
*** juzuluag has quit IRC17:48
*** MentalRay has joined #openstack-cinder17:51
*** esker has quit IRC17:52
*** merooney has joined #openstack-cinder17:54
*** anshul has joined #openstack-cinder17:56
*** alexpilotti has quit IRC18:01
*** alexpilotti has joined #openstack-cinder18:02
*** ebalduf has quit IRC18:03
*** ebalduf has joined #openstack-cinder18:04
*** abhiram_moturi has quit IRC18:06
*** abhiram_moturi has joined #openstack-cinder18:06
*** alexpilotti has quit IRC18:07
*** abhiram_moturi has quit IRC18:13
*** abhiram_moturi has joined #openstack-cinder18:13
*** coolsvap has quit IRC18:15
*** sgotliv has quit IRC18:17
*** esker has joined #openstack-cinder18:18
*** vilobhmm1 has quit IRC18:19
*** vilobhmm1 has joined #openstack-cinder18:19
thingeelpetrut: can you please give me the blueprint link for https://review.openstack.org/#/c/161299/18:20
thingeelpetrut: also in the future please include it in the commit message as shown https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references18:21
thingeelpetrut: otherwise this makes tracking difficult for me.18:21
thingeejgriffith: ^18:22
*** rwsu_ has quit IRC18:24
*** timcl1 has joined #openstack-cinder18:26
*** merooney has quit IRC18:26
*** merooney has joined #openstack-cinder18:28
*** timcl has quit IRC18:28
*** MentalRay has quit IRC18:30
*** annegentle has quit IRC18:31
thingeehemna: what's is Ivan raising with the os-brick being added to requirements?18:32
*** vilobhmm1 has quit IRC18:33
thingeehemna: couldn't really catch it from irc/gerrit18:33
*** merooney has quit IRC18:33
*** vilobhmm1 has joined #openstack-cinder18:33
*** Longgeek has quit IRC18:37
*** Longgeek has joined #openstack-cinder18:37
*** Longgeek has quit IRC18:39
*** Longgeek has joined #openstack-cinder18:39
*** vilobhmm1 has quit IRC18:40
*** vilobhmm1 has joined #openstack-cinder18:42
*** vilobhmm1 has quit IRC18:43
*** vilobhmm1 has joined #openstack-cinder18:44
*** lpetrut has left #openstack-cinder18:48
*** merooney has joined #openstack-cinder18:51
*** abhiram_moturi has quit IRC18:56
*** abhiram_moturi has joined #openstack-cinder18:56
hodosjgriffith: tox -eall -- '(?!.*(volume_upload|volume_backup))volume' doesn't run tempest.cli tests on my node. maybe i'm missing some services?19:00
jgriffithhodos: just run "tox -eall -- volume" and be done :)19:01
hodosjgriffith: can't run upload19:01
hodosjgriffith: it's not working for nfs and corrupting the following tests19:01
jgriffithhodos: and tempest.cli is getting pulled into projects functional tests IIRC19:02
jgriffithhodos: oh, so you need a skip19:02
hodosjgriffith: yes, i do it via the string19:02
hodoshodos: don't know the other way19:02
jgriffithhodos: OH  DERP19:03
jgriffithhodos: I see now :)19:03
*** MentalRay has joined #openstack-cinder19:03
jgriffithso I think the CLI tests have been removed19:03
hodosjgriffith: just like today?19:03
jgriffithhodos: in fact: https://goo.gl/eNXPZ819:03
*** BharatK_ has joined #openstack-cinder19:03
hodosjgriffith: oh. nice19:03
jgriffithhodos: so that answers that question :)19:04
hodosjgriffith: thanks, I'm happy )19:04
jgriffithhodos: always nice on a Friday :)19:04
*** timcl1 has quit IRC19:06
hodosjgriffith: strange that I've ran list_tests today and saw cli tests there...19:06
jgriffithhodos: I dunno dude, but it merged last night or so19:07
hodosjgriffith: 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 help19:09
jgriffithhodos: great to hear19:09
*** abhiram_moturi has quit IRC19:11
*** abhiram_moturi has joined #openstack-cinder19:12
*** timcl has joined #openstack-cinder19:13
*** lpabon has quit IRC19:14
*** merooney has quit IRC19:18
thingeejungleboyj: was going to merge this, but can you answer the 1 comment question here https://review.openstack.org/#/c/152735/5 ?19:20
thingeeeharney: any update with https://review.openstack.org/#/c/165393/1 ?19:21
jungleboyjthingee: Sure.  Just a minute.19:21
thingeeeharney: this would be great ;)19:21
eharneythingee: 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 area19:22
eharneynow in*19:22
thingeeeharney: cool, I'll look for deepak's update to the spec19:23
*** merooney has joined #openstack-cinder19:24
jungleboyjthingee:  responded.  I can update the spec to be more explicit if you feel it is necessary.19:24
*** nikeshm has joined #openstack-cinder19:25
*** eglynn has quit IRC19:25
thingeejungleboyj: nah it's fine, thank you19:25
*** lpetrut has joined #openstack-cinder19:26
jungleboyjthingee: Thank you.19:27
thingeehemna: https://review.openstack.org/#/c/152740/19:30
nikeshmasselin: http://paste.openstack.org/show/217335/19:30
*** Lee1092 has quit IRC19:31
nikeshmasselin : now nodepool image-build dpc and nodepool image-upload dpc is working fine, but when uploading patch in sandbox,its saying noop-check not registered19:31
*** marcusvrn has quit IRC19:33
*** dustins has quit IRC19:33
openstackgerritTom Barron proposed openstack/cinder: Use nfs_oversub_ratio when reporting pool capacity  https://review.openstack.org/18019919:33
*** merooney has quit IRC19:35
*** hodos has quit IRC19:35
*** abhiram_moturi has quit IRC19:39
*** abhiram_moturi has joined #openstack-cinder19:39
*** abhiram_moturi has quit IRC19:41
*** abhiram_moturi has joined #openstack-cinder19:42
*** abhiram_moturi has quit IRC19:44
*** abhiram_moturi has joined #openstack-cinder19:44
*** abhiram_moturi has quit IRC19:46
*** abhiram_moturi has joined #openstack-cinder19:46
*** abhiram_moturi has quit IRC19:48
*** bkopilov has joined #openstack-cinder19:49
*** abhiram_moturi has joined #openstack-cinder19:49
*** abhiram_moturi has quit IRC19:51
*** abhiram_moturi has joined #openstack-cinder19:51
*** abhiram_moturi has quit IRC19:53
*** abhiram_moturi has joined #openstack-cinder19:54
*** geguileo has quit IRC19:55
*** MentalRay has quit IRC20:00
*** abhiram_moturi has quit IRC20:04
*** abhiram_moturi has joined #openstack-cinder20:05
*** geguileo has joined #openstack-cinder20:07
asselinnikeshm, telnet localhost 473020:07
asselinnikeshm, type 'status' to see what jobs are registered20:07
*** vmtrooper has joined #openstack-cinder20:10
*** abhiram_moturi has quit IRC20:12
*** abhiram_moturi has joined #openstack-cinder20:12
*** abhiram_moturi has quit IRC20:14
*** abhiram_moturi has joined #openstack-cinder20:14
*** shakamunyi has quit IRC20:17
*** breitz has quit IRC20:17
*** Apoorva has quit IRC20:17
*** breitz has joined #openstack-cinder20:17
*** abhiram_moturi has quit IRC20:21
*** abhiram_moturi has joined #openstack-cinder20:21
*** abhiram_moturi has quit IRC20:23
*** abhiram_moturi has joined #openstack-cinder20:24
*** annegentle has joined #openstack-cinder20:25
*** annegentle has quit IRC20:26
*** annegentle has joined #openstack-cinder20:26
Swansonjgriffith: thanks!20:27
*** akerr has quit IRC20:28
*** hodos|2 has joined #openstack-cinder20:28
*** IlyaG has quit IRC20:29
*** abhiram_moturi has quit IRC20:32
*** abhiram_moturi has joined #openstack-cinder20:32
*** e0ne has joined #openstack-cinder20:33
*** mriedem is now known as mriderman20:35
*** julim has quit IRC20:37
*** esker has quit IRC20:38
*** kaisers has quit IRC20:38
e0nejgriffith: thanks for +2/A https://review.openstack.org/#/c/171566/. added question to your comment. just want to not miss it20:39
*** thangp has quit IRC20:40
e0nesometimes all of us don't read questions/comments for merged/approved reviews:(20:40
*** kaisers has joined #openstack-cinder20:41
*** Apoorva has joined #openstack-cinder20:41
*** Yogi1 has quit IRC20:41
jgriffithe0ne: https://goo.gl/pYKaIc20:41
jgriffithe0ne: note the log cleanup and use of "resource=volume"20:42
jgriffithe0ne: I added a tag mechanism to oslo.log that reads that in and formats the log messages consistently20:42
e0nejgriffith: yes, but in my patch i don't change logging. i changed exception message20:42
jgriffithe0ne: by putting a volume-id in each20:42
jgriffithe0ne: exception message counts :)20:43
nikeshmasselin : http://paste.openstack.org/show/217389/20:43
jgriffithe0ne: fair enough20:43
e0nejgriffith: looks like i missed it20:43
*** dencaval has quit IRC20:43
jgriffithe0ne: no, I ment would've been cool to fix "everything" up in one shot20:43
jgriffithe0ne: I know your patch wasn't related20:43
asselinnikeshm, so nothing's registered. Did you start nodepool? did it create any jenkins slaves?20:43
jgriffithe0ne: I was just saying that if we do a message cleanup/modify it would be cool to do the whole file, logs, exceptiosn and all20:44
jgriffithe0ne: are logs are pretty inconsistenet20:44
jgriffithe0ne: but anyway... no worries20:44
e0nejgriffith: agree, +120:44
jgriffithe0ne: I'll pick back up on working on udpating them in chunks again next week maybe20:44
e0nejgriffith: i just fixed only my issue:(20:44
*** MentalRay has joined #openstack-cinder20:45
e0nejgriffith: how should i add tagging for exception messages in a future patches?20:45
nikeshmasselin: started long time ago,bot i am getting this in debug.log of nodepool http://paste.openstack.org/show/217390/20:46
asselinlocal_01 does not have image d-p-c for label d-p-c.20:47
asselinnikeshm, nodepool image-upload all d-p-c20:47
jgriffithe0ne: for the "reason=msg" stuff there's nothing to do20:48
*** shakamunyi has joined #openstack-cinder20:48
jgriffithe0ne: I was just saying that "instead of a one line change, change everything and update the whole file" :)20:48
jgriffithe0ne: logging and exception messages are related in my opinion20:49
jgriffithe0ne: sorryfor the confusion20:49
nikeshmasselin: hmm nodepool image-build dpc;nodepool image-upload dpc    now i have to run nodepool image-upload all d-p-c20:50
*** IlyaG has joined #openstack-cinder20:50
e0nejgriffith: thanks for clarification. i appreciate your help20:50
nikeshmasselin : http://paste.openstack.org/show/217391/20:50
nikeshmwhy so20:50
asselinnikeshm, so the dib-image-list are the local ones built on the box.20:51
e0nejgriffith: i'll try fix all related staff next time instead fixing only one thing20:51
asselinnikeshm, the upload is to sync up the provider20:51
asselinnikeshm, 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-c20:51
jgriffithe0ne: no worries, sometimes we just flat don't have time20:51
asselinmismatch20:51
e0nejgriffith: tbh, this patch was made on my devstack env to debug something. and was posted to gerrit 'as is'20:51
jgriffithe0ne: I just raised it to point it out otherwise nobody will ever remember :)20:52
jgriffithe0ne: go to sleep!20:52
jgriffith:)20:52
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Added a unit test for masking iscsiadm passwords  https://review.openstack.org/18155320:52
*** primechuck has quit IRC20:52
e0nejgriffith: i'll definatly do it:)20:52
e0nejgriffith: thanks for review and help understanding how make a code better20:53
*** ebalduf has quit IRC20:54
*** mriderman has quit IRC20:54
*** ebalduf has joined #openstack-cinder20:55
nikeshmasselin: http://paste.openstack.org/show/217394/ is the yaml of nodepool which i am using20:55
e0nehave a good Friday and weekend. bb on Monday20:57
*** BharatK_ has quit IRC20:57
asselinnikeshm, change line 13 to dpc20:58
*** mriedem has joined #openstack-cinder20:59
nikeshmasselin: 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 again21:01
nikeshmso like i am changing line number 1321:01
nikeshmto dpc21:01
nikeshmin my private git data repo21:02
*** e0ne has quit IRC21:02
asselinnikeshm, ./install_master.sh from ~21:02
nikeshmok thanks21:03
*** shakamunyi has quit IRC21:03
*** abhiram_moturi has quit IRC21:06
*** abhiram_moturi has joined #openstack-cinder21:07
*** ronis_ has quit IRC21:08
*** breitz has quit IRC21:08
*** eharney has quit IRC21:09
*** Longgeek has quit IRC21:10
*** mriedem has quit IRC21:11
*** mriedem has joined #openstack-cinder21:12
*** timcl has quit IRC21:13
*** emagana has quit IRC21:14
*** abhiram_moturi has quit IRC21:21
*** abhiram_moturi has joined #openstack-cinder21:22
*** shakamunyi has joined #openstack-cinder21:22
*** cbader has quit IRC21:25
*** MentalRay has quit IRC21:26
*** jungleboyj has quit IRC21:28
*** rwsu has joined #openstack-cinder21:30
*** rwsu has quit IRC21:30
*** rwsu has joined #openstack-cinder21:31
*** abhiram_moturi has quit IRC21:36
jgriffiththingee: so what do you want me to do on those backports?21:36
*** abhiram_moturi has joined #openstack-cinder21:36
jgriffiththingee: yours inparticular is completely safe AFAIC21:36
*** esker has joined #openstack-cinder21:36
thingeejgriffith: technically we shouldn't be doing non-security related to juno anymore. Saw some from dell21:37
thingeejgriffith: though they've been sitting for a while21:37
thingeesince february21:37
jgriffiththingee: yeah, I've been pretty much rejecting Juno21:38
jgriffiththingee: Kilo is still "new" enough that I don't have any real issue21:38
jgriffiththingee: but I also don't want to get called out by somebody for approving a "low" severity patch for backport21:39
jgriffiththingee: at least not without someobdy having my back:)21:39
jgriffithand discussing it21:39
thingeereal issue with backporting to Kilo or Kilo is still new enough that backporting to Juno should be fine?21:39
*** rwsu has quit IRC21:39
jgriffiththingee: yes, IMO21:39
jgriffiththingee: and your patch is simple/safe21:40
jgriffiththingee: but I -1'd it on principle :)21:40
thingeeHang 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
jgriffiththingee: ok... one at a time21:41
*** erlon has quit IRC21:41
*** MentalRay has joined #openstack-cinder21:41
thingeeyou just said backports and I wasn't sure what you were referring to.21:42
jgriffiththingee: 1.  Your backport to Kilo https://goo.gl/wq229V21:42
jgriffiththingee: I gave a -1 becasue it's low severity and doesn't meet the criteria21:42
thingeeah I see21:42
jgriffiththingee: but I don't mind the patch itself, just need to have consistency with people21:42
thingeeYeah I marked it low because it's a driver. I view my driver not as important as cinder core related.21:43
jgriffiththingee: right, understood21:43
jgriffithI do the same thing for the most part21:43
thingeebut 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
jgriffithMy point was that Kilo backports I'm not bugged about it21:44
jgriffithwell, in that case...21:44
jgriffithwait21:44
jgriffithirrelevant :)21:44
jgriffithdo you run Kilo CI and publish?21:44
jgriffithanyway...21:45
jgriffiththingee: do you want that patch backported or not?21:45
jgriffiththingee: yes or now?21:45
jgriffithno?21:45
thingeenot yet. Internally there is a juno one.21:45
thingeeI would, that's why I proposed it :).21:45
thingeeI haven't really denied a backport to previous release. I have denied backports to security only supported branches though.21:45
jgriffithOk21:46
jgriffithdone21:46
thingeeif 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
jgriffiththingee: not "you" or "us" per-say21:46
jgriffiththingee: keep in mind there's a stable maintenance core that's not just Cinder memebers21:46
jgriffithanyway....21:46
jgriffithmoving on...21:47
jgriffithJuno21:47
jgriffithI'm inclined to be pretty darn picky21:47
thingeeI'm fine with denying for Juno at this point if it's not security related. Not even my stuff can get backported.21:47
jgriffithk21:47
jgriffithsimple21:47
jgriffithdone21:47
jgriffithmoving on :)21:47
thingeethe 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 IRC21:49
*** abhiram_moturi has quit IRC21:56
*** Adriano_ has quit IRC21:56
*** abhiram_moturi has joined #openstack-cinder21:56
SwansonIs 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 IRC21:58
*** rwsu has joined #openstack-cinder21:58
thingeeSwanson: you can do it21:58
Swansonthanks!21:59
*** madskier has joined #openstack-cinder21:59
*** MentalRay has quit IRC22:01
*** madskier has quit IRC22:04
*** shakamunyi has quit IRC22:04
*** barra204 has joined #openstack-cinder22:05
*** EmilienM is now known as EmilienM|afk22:08
kmartinthingee or jgriffith, Do either of you have the ability to change the owner\assignee of a few BPs?22:10
thingeekmartin: yes22:10
thingeekmartin: what do you need?22:11
kmartincould 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-m22:11
kmartinaallen-m is Angela Smith22:12
thingeekmartin: done. do you know what brcoades plans are for CI?22:12
kmartinyes, they are working on it currently the plan is to start giving you updates22:13
kmartinsuppose to be in place by L122:13
thingeenice22:13
kmartinthingee, thank you22:14
*** alexpilotti has joined #openstack-cinder22:14
openstackgerritMike Perez proposed openstack/cinder: Convert mox to mock: tests/compute/test_service.py  https://review.openstack.org/14328122:15
*** rmesta has quit IRC22:17
*** beekneemech has quit IRC22:18
*** dimsum__ has quit IRC22:21
openstackgerritMike Perez proposed openstack/cinder: rtstool: Update credentials when target(LIO) already exists  https://review.openstack.org/14633322:21
*** garthb has quit IRC22:22
*** annegentle has quit IRC22:23
openstackgerritMike Perez proposed openstack/cinder: Handle incorrect '--config-dir' param  https://review.openstack.org/15774922:26
*** dalgaaf has joined #openstack-cinder22:29
*** madskier has joined #openstack-cinder22:33
*** jlk has joined #openstack-cinder22:36
*** annegentle has joined #openstack-cinder22:36
jlkthingee: 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 cinder22:36
jlkthingee: are you around to chat through this with me?22:37
*** ebalduf has quit IRC22:37
*** madskier has quit IRC22:38
thingeejlk: I have a little bit of time to chat. going into a meeting soon22:38
jlkthingee: so client is htting the unversioned endpoint to get a list of what versions are there22:38
jlkserver is giving back the v1 and v2 end versions22:38
jlkbut it's giving an href of an http:// url22:39
jlkhowever, client access to these goes via https, through haproxy22:39
jlkhaproxy terminates ssl and hands off to cinder22:39
jlkso what happens is client gets back an http url, tries to hit it, and can't.22:39
thingeejlk: going to bring morganfainberg in. We're just using keystone client to to hit the base url22:39
thingeejlk: also, you rservice catalog should be set for https no?22:40
jlkit is22:41
thingeeyour*22:41
thingeethat's where keystone client is getting the endpoints from22:41
thingeeyour service catalog22:41
morganfainbergthingee: yeah I'd expect the catalog to say https.22:41
jlkhttps://gist.github.com/j2sol/e1307dcebfdf3d32f06d22:41
jlknah, it's hitting 8778/ to get a list of cinder versions22:41
morganfainbergAnd it should use https then. jlk you don't use http for internal right?22:42
jlkwell, we send all our internal cinder requests through haproxy too22:42
morganfainbergthingee: does cinder do something odd on discovery? Where t needs22:43
morganfainbergTo know itself is https?22:43
morganfainbergthingee: it looks like the version discovery is saying http.22:44
jlkcinder itself doesn't do https22:44
jlkthe https termination happens at haproxy22:44
*** julim has joined #openstack-cinder22:44
jlkhere is the catalog https://gist.github.com/j2sol/a2b7236e834052c548de22:45
morganfainbergRight. But everything should talk via haproxy. That tells me the version discovery should probably show the haproxy url and include https22:45
jlkmorganfainberg: I would thnik so22:45
jlkbut I don't see a way to tell cinder what it's own endpoint is22:46
morganfainbergthingee: ^^22:46
jlkbecause I'm pretty sure it's a cinder API call to discover cinder versions22:46
morganfainbergRight.22:46
jlkhttp://developer.openstack.org/api-ref-blockstorage-v2.html22:46
jlkGET on /22:46
tbarronthingee: jgriffith: just drove home from work, opened IRC, saw the backlog and the backport to Juno discussion.22:47
tbarronso w.r.t. https://review.openstack.org/#/c/180199/22:48
tbarronmay aim was to get it into liberty, backport to kilo, backport to Juno.22:48
tbarronThe reason is that we have a large enterprise customer with private cloud with this bug in juno.22:49
tbarronThey can't move to kilo for several months, or to liberty for a long time.22:49
tbarronThe whole point is the bacport.  I want to do xing/winston style overprovisioning in liberty which will obsolete this fix there.22:50
morganfainbergjlk: https://github.com/openstack/cinder/blob/master/cinder/api/views/versions.py#L2322:50
jlkah hah!22:50
jlkthere is a new config22:50
jgriffithtbarron: common problem.  Give them a patch, "git diff", "git apply" :)22:50
morganfainbergLooks like! :)22:50
jlkgood deal22:50
tbarronthe 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
morganfainbergjlk: at least it's straight forward to change :)22:51
thingeemorganfainberg: thanks...yeah looks like that was added for liberty22:51
jgriffithtbarron: we already do: https://wiki.openstack.org/wiki/StableBranch22:51
tbarronwhich you guys may already have done. I'm not questioning that,22:51
tbarron(concurrent chat there)22:51
morganfainbergYep. And makes sense for version discovery purposes.22:51
jgriffithtbarron: particularly check "Support phases" and "Appropriate Fixes"22:52
thingeetbarron: we haven't. This is with us conforming to stable core.22:52
jlkarg22:52
jlkdoesn't look like it is in the config reference22:52
morganfainbergjlk: that I can't speak to.22:52
jlkwhat block does that go into22:52
tbarronkk, looking.22:52
morganfainbergHmm sec22:52
thingeejlk: I see it https://wiki.openstack.org/wiki/Cinder#Sample_cinder.conf22:52
jgriffithtbarron: it's pretty much an exact summary of what we were discussing22:53
thingeeoh in the docs?22:53
thingeejlk: ^22:53
morganfainbergjlk: looks like in [default]22:53
jgriffithtbarron: incuding Kilo being a sort of "open book" still22:53
jlkyeah, the config reference doc22:53
thingeejlk: I'll check it out :)22:53
jlksomewhere in http://docs.openstack.org/juno/config-reference/content/ch_configuring-openstack-block-storage.html22:53
morganfainbergjlk: it gets registered without a group afaict22:53
jlkis that in juno?22:53
thingeegotta disappear for my meeting22:53
thingeejlk: no liberty22:53
jlkARGHGH22:54
tbarronso 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 kilo22:54
morganfainbergOh Juno sorry22:54
morganfainbergMissed that part of the convo.22:54
jgriffithtbarron: that should be met without any question yes22:54
jlkso juno has the problem, but the fix is in liberty.22:54
jlkwheeeee.22:54
jgriffithtbarron: 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 IRC22:55
thingeejlk: 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
jlkyeah22:55
jlkthe disconnect there is a bit odd22:55
jgriffithtbarron: but like I said to thingee I'll be kinda picky going forward22:55
jgriffithtbarron: and the Icehouse patches that keep popping up are pretty lame22:56
*** rwsu has quit IRC22:56
morganfainbergJlk: so there would need to be a backport to make that work with Juno. It just takes the base url of the req22:56
tbarronjgriffith: kk, I should get this to land in liberty real soon now, so expect some cherry pick reviews coming towards stable core22:56
thingeefwiw, 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
jlkyeah I may do a backport in our own fork22:56
jlkbecause we can't necessarily control what our customers install for cinder clients22:56
morganfainbergIt might be possible to fix it with forcing it to use the req_for (same url thing we did for shibboleth)22:56
thingeejlk: whoops sorry, that was directed to tbarron and jgriffith22:56
morganfainbergWhatever header that is.22:57
tbarronwe 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 being22:57
morganfainbergIf it exists. Then fall back to current workflow22:57
morganfainbergIf it doesn't exist.22:57
tbarrona big issue for our customer and limited in scope to our driver22:57
tbarronthanks jgriffith and thingee for your time on this.  I won't belabor the issue now.22:57
morganfainbergjlk: that should be backportable if you are looking to get code in that isn't carried by you.22:58
morganfainbergAnd 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 IRC23:04
*** vilobhmm1 has joined #openstack-cinder23:04
morganfainbergjlk: worth opening a bug report on imo against stable.23:04
morganfainbergBut I defer to thingee on that overall. (Since you know... Not my project ;)23:05
thingeejlk: my mistake, this is in Kilo.23:05
thingeejlk: I would raise the bug against Juno though.23:05
thingeesee what stable folks think23:05
*** geguileo has quit IRC23:06
morganfainbergthingee: 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
morganfainbergThis is a bit below where you should care what cinder's url is.23:07
morganfainbergS/should care/should have to care23:07
thingeemorganfainberg: this fix might be a bit hard to convince people on though because of it introducing a config opt23:07
morganfainbergNah. Just use the haproxy header that is passed down first.23:07
morganfainbergThen fall back to the logic in Juno or the config if it does t exist.23:08
thingeeI see what you mean I think23:08
morganfainbergI'll dig up the specifics of what I mean (x-forwarded for? I don't know what that header is...)23:08
morganfainbergBut 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 IRC23:09
*** abhiram_moturi has quit IRC23:13
* thingee disappears to meeting23:13
*** abhiram_moturi has joined #openstack-cinder23:13
*** patrickeast has quit IRC23:15
*** daneyon_ has joined #openstack-cinder23:16
*** abhiram_moturi has quit IRC23:19
*** abhiram_moturi has joined #openstack-cinder23:20
*** rwsu has joined #openstack-cinder23:22
*** geguileo has joined #openstack-cinder23:22
*** barra204 has quit IRC23:22
*** IanGovett1 has joined #openstack-cinder23:24
*** IanGovett has quit IRC23:27
*** abhiram_moturi has quit IRC23:33
*** abhiram_moturi has joined #openstack-cinder23:33
*** abhiram_moturi has quit IRC23:37
*** abhiram_moturi has joined #openstack-cinder23:37
*** abhiram_moturi has quit IRC23:39
*** abhiram_moturi has joined #openstack-cinder23:40
*** abhiram_moturi has quit IRC23:44
*** abhiram_moturi has joined #openstack-cinder23:44
*** alexpilotti_ has joined #openstack-cinder23:45
*** hemna is now known as hemnafk23:46
*** alexpilotti has quit IRC23:47
*** alexpilotti_ is now known as alexpilotti23:47
*** abhiram_moturi has quit IRC23:49
*** abhiram_moturi has joined #openstack-cinder23:49
*** anshul has quit IRC23:50
*** tsekiyama has quit IRC23:51
*** drjones has quit IRC23:52
*** _cjones_ has joined #openstack-cinder23:53
*** abhiram_moturi has quit IRC23:56
*** abhiram_moturi has joined #openstack-cinder23:56
*** rwsu has quit IRC23:56
*** _cjones_ has quit IRC23:58
*** dannywil_ has quit IRC23:58

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