Wednesday, 2017-10-25

*** rtjure has joined #openstack-cinder00:07
*** rtjure has quit IRC00:12
*** yangyapeng has quit IRC00:18
*** rcernin has joined #openstack-cinder00:19
*** rtjure has joined #openstack-cinder00:20
*** rcernin has quit IRC00:23
*** rcernin has joined #openstack-cinder00:24
*** rtjure has quit IRC00:24
*** rtjure has joined #openstack-cinder00:32
*** jmlowe has quit IRC00:32
*** jdavis has joined #openstack-cinder00:34
*** jmlowe has joined #openstack-cinder00:36
*** s-shiono has joined #openstack-cinder00:36
*** rtjure has quit IRC00:37
*** jdavis has quit IRC00:39
*** wanghao has joined #openstack-cinder00:40
*** newmember has quit IRC00:42
*** rtjure has joined #openstack-cinder00:42
*** wanghao has quit IRC00:43
*** wanghao has joined #openstack-cinder00:43
openstackgerritVipin Balachandran proposed openstack/cinder master: VMware: Add profile ID to connection info  https://review.openstack.org/51487400:44
*** rtjure has quit IRC00:47
*** adisky_ has joined #openstack-cinder00:52
openstackgerritTommyLike proposed openstack/cinder master: Support count info in List&Detail APIs  https://review.openstack.org/50510100:53
*** rtjure has joined #openstack-cinder00:54
*** mriedem has quit IRC00:56
*** Guest5155 has quit IRC00:57
*** stvnoyes has quit IRC00:58
*** rtjure has quit IRC00:59
*** stvnoyes has joined #openstack-cinder00:59
*** geguileo has joined #openstack-cinder00:59
*** Apoorva_ has joined #openstack-cinder01:00
*** geguileo is now known as Guest8238201:00
*** jdurgin has quit IRC01:01
*** Apoorva has quit IRC01:02
*** markstur_ has quit IRC01:03
*** amoralej|off has quit IRC01:04
*** Apoorva_ has quit IRC01:04
*** rtjure has joined #openstack-cinder01:05
*** jdurgin has joined #openstack-cinder01:07
tonybildikov, smcginnis: For the record it wasn't the mysql update that cause that regression.01:08
*** amoralej has joined #openstack-cinder01:08
openstackgerritJohn Griffith proposed openstack/cinder master: Make service object UUID not nullable  https://review.openstack.org/51485301:09
*** bkopilov has joined #openstack-cinder01:09
*** rtjure has quit IRC01:10
tonybSQLAlchemy was last updated in September01:11
*** amoralej has quit IRC01:13
*** mriedem has joined #openstack-cinder01:15
mriedemsmcginnis: not sure why you rechecked this https://review.openstack.org/#/c/510201/01:17
mriedemsee the -1s on there?01:17
mriedemit doesn't perform the online data migration as noted01:17
mriedemhttps://bugs.launchpad.net/cinder/+bug/172711701:18
openstackLaunchpad bug 1727117 in Cinder "Service.uuid is not migrated via online data migrations" [High,Triaged]01:18
mriedemjgriffith: ^01:18
*** markstur has joined #openstack-cinder01:19
*** rtjure has joined #openstack-cinder01:19
*** dillaman has joined #openstack-cinder01:19
*** kaisers_ has joined #openstack-cinder01:19
*** amoralej has joined #openstack-cinder01:20
*** gus has quit IRC01:21
*** gus has joined #openstack-cinder01:22
*** bkopilov_ has joined #openstack-cinder01:22
*** kaisers has quit IRC01:23
*** Guest82382 has quit IRC01:23
*** rtjure has quit IRC01:23
*** bkopilov has quit IRC01:24
*** geguileor has joined #openstack-cinder01:24
*** geguileor is now known as Guest7473001:25
*** yangyapeng has joined #openstack-cinder01:27
*** yangyapeng has quit IRC01:27
*** yangyapeng has joined #openstack-cinder01:28
*** yangyapeng has quit IRC01:28
*** mriedem has quit IRC01:28
*** yangyapeng has joined #openstack-cinder01:28
*** rtjure has joined #openstack-cinder01:28
*** salv-orlando has joined #openstack-cinder01:28
*** rcernin has quit IRC01:32
*** rcernin has joined #openstack-cinder01:32
*** salv-orlando has quit IRC01:33
*** rcernin has quit IRC01:33
*** rtjure has quit IRC01:33
*** psachin has joined #openstack-cinder01:37
*** rtjure has joined #openstack-cinder01:40
*** edmondsw has joined #openstack-cinder01:43
*** rtjure has quit IRC01:45
*** sapd has quit IRC01:46
*** edmondsw has quit IRC01:47
*** sapd has joined #openstack-cinder01:48
*** mwheckmann has joined #openstack-cinder01:49
*** wewe0901 has joined #openstack-cinder01:51
*** dhinesh_ has quit IRC01:52
*** itlinux has joined #openstack-cinder01:53
*** rtjure has joined #openstack-cinder01:53
*** rcernin has joined #openstack-cinder01:55
openstackgerritVan Hung Pham proposed openstack/cinder master: Stop using deprecated 'message' attribute in Exception  https://review.openstack.org/51488201:57
*** rtjure has quit IRC01:58
*** dikonoor has joined #openstack-cinder02:06
*** rtjure has joined #openstack-cinder02:07
*** dalgaaf has quit IRC02:11
*** rtjure has quit IRC02:12
*** dalgaaf has joined #openstack-cinder02:12
*** amoralej has quit IRC02:12
*** itlinux has quit IRC02:15
*** bkopilov_ has quit IRC02:16
*** itlinux has joined #openstack-cinder02:16
*** rtjure has joined #openstack-cinder02:18
*** rtjure has quit IRC02:23
*** zhaochao has quit IRC02:26
*** rtjure has joined #openstack-cinder02:27
*** rtjure has quit IRC02:32
*** lhx_ has joined #openstack-cinder02:34
*** rtjure has joined #openstack-cinder02:37
*** tpsilva has quit IRC02:39
*** zhaochao has joined #openstack-cinder02:40
*** rtjure has quit IRC02:42
*** stakeda has joined #openstack-cinder02:47
*** rtjure has joined #openstack-cinder02:48
*** mwheckmann has quit IRC02:48
*** rtjure has quit IRC02:53
*** rtjure has joined #openstack-cinder02:57
*** dave-mccowan has quit IRC02:59
*** namnh has joined #openstack-cinder02:59
*** itlinux has quit IRC03:00
openstackgerrityixuan zhang proposed openstack/cinder master: Storwize: add hyperswap volume support  https://review.openstack.org/49241403:01
*** rtjure has quit IRC03:02
openstackgerritTommyLike proposed openstack/python-cinderclient master: Return all resources only when required  https://review.openstack.org/51489603:02
*** rtjure has joined #openstack-cinder03:10
*** rtjure has quit IRC03:15
*** rtjure has joined #openstack-cinder03:22
*** dhinesh has joined #openstack-cinder03:23
*** rtjure has quit IRC03:26
*** dhinesh has quit IRC03:27
*** dhinesh has joined #openstack-cinder03:28
*** salv-orlando has joined #openstack-cinder03:29
*** edmondsw has joined #openstack-cinder03:31
openstackgerritKushal Wathore proposed openstack/cinder master: HPE 3PAR: ISCSI/FC drivers – code refactoring  https://review.openstack.org/44647503:32
*** rtjure has joined #openstack-cinder03:32
*** salv-orlando has quit IRC03:33
*** edmondsw has quit IRC03:35
*** bkopilov_ has joined #openstack-cinder03:36
*** rtjure has quit IRC03:37
*** gkadam has joined #openstack-cinder03:38
*** rtjure has joined #openstack-cinder03:45
*** rtjure has quit IRC03:50
openstackgerritTommyLike proposed openstack/python-cinderclient master: Return all resources only when required  https://review.openstack.org/51489603:52
*** rtjure has joined #openstack-cinder03:56
*** rtjure has quit IRC04:01
*** rtjure has joined #openstack-cinder04:08
*** rtjure has quit IRC04:12
*** rtjure has joined #openstack-cinder04:18
*** rcernin has quit IRC04:21
*** rtjure has quit IRC04:22
*** rtjure has joined #openstack-cinder04:27
*** ganso has quit IRC04:28
*** rtjure has quit IRC04:32
*** rtjure has joined #openstack-cinder04:40
*** kaisers_ has quit IRC04:43
*** rcernin has joined #openstack-cinder04:45
*** rtjure has quit IRC04:45
*** rtjure has joined #openstack-cinder04:49
*** wN has quit IRC04:52
*** rtjure has quit IRC04:54
*** felipemonteiro_ has joined #openstack-cinder04:57
*** abhitechie has joined #openstack-cinder04:58
*** lkuchlan has joined #openstack-cinder04:58
*** rtjure has joined #openstack-cinder04:59
*** felipemonteiro_ has quit IRC05:02
*** Trident has quit IRC05:03
*** rtjure has quit IRC05:04
*** rtjure has joined #openstack-cinder05:10
*** kaisers has joined #openstack-cinder05:13
*** rtjure has quit IRC05:15
*** lpetrut has joined #openstack-cinder05:18
*** salv-orlando has joined #openstack-cinder05:18
*** rtjure has joined #openstack-cinder05:20
*** ChubYann has quit IRC05:20
*** rtjure has quit IRC05:24
*** lpetrut has quit IRC05:26
*** lpetrut has joined #openstack-cinder05:28
*** rtjure has joined #openstack-cinder05:30
*** armaan has joined #openstack-cinder05:32
*** rtjure has quit IRC05:35
*** markstur has quit IRC05:39
*** rtjure has joined #openstack-cinder05:40
openstackgerritMerged openstack/cinder master: Move 'zoning_mode' back to DEFAULT section  https://review.openstack.org/51220805:46
*** wanghao has quit IRC05:51
*** wanghao has joined #openstack-cinder05:56
*** armaan_ has joined #openstack-cinder06:03
*** armaan has quit IRC06:04
*** vivsoni has quit IRC06:04
*** vivsoni has joined #openstack-cinder06:05
*** armaan has joined #openstack-cinder06:05
*** armaan_ has quit IRC06:06
*** abhitechie has quit IRC06:06
*** lhx_ has quit IRC06:08
*** lhx_ has joined #openstack-cinder06:08
*** lpetrut has quit IRC06:13
*** gouthamr has quit IRC06:14
*** abhitechie has joined #openstack-cinder06:18
*** josecastroleon has quit IRC06:20
ildikovtonyb: thanks for testing that06:20
openstackgerritTommyLike proposed openstack/python-cinderclient master: Return all resources only when required  https://review.openstack.org/51489606:23
*** lpetrut has joined #openstack-cinder06:33
*** arnewiebalck_ has joined #openstack-cinder06:34
*** josecastroleon has joined #openstack-cinder06:38
*** vivsoni has quit IRC06:39
*** pcaruana has joined #openstack-cinder06:39
*** vivsoni has joined #openstack-cinder06:39
*** dhinesh has quit IRC06:43
*** salv-orlando has quit IRC06:44
*** salv-orlando has joined #openstack-cinder06:45
*** ykarel has joined #openstack-cinder06:46
*** junboli has joined #openstack-cinder06:46
*** salv-orlando has quit IRC06:49
openstackgerritwang yong proposed openstack/cinder master: Inspur Cinder FC/iSCSI driver  https://review.openstack.org/50245506:49
*** josecastroleon has quit IRC06:49
*** salv-orlando has joined #openstack-cinder06:49
openstackgerritjunbo.li proposed openstack/cinder master: Remove doc/build before running docs job  https://review.openstack.org/51250706:53
*** tesseract has joined #openstack-cinder06:54
*** gcb has joined #openstack-cinder07:00
*** lpetrut has quit IRC07:01
*** pgadiya has joined #openstack-cinder07:02
*** josecastroleon has joined #openstack-cinder07:05
*** armax has quit IRC07:09
*** abhitechie has quit IRC07:10
openstackgerrityixuan zhang proposed openstack/cinder master: Storwize: add hyperswap volume support  https://review.openstack.org/49241407:12
*** spectr has joined #openstack-cinder07:13
*** josecastroleon has quit IRC07:13
*** abhitechie has joined #openstack-cinder07:14
*** josecastroleon has joined #openstack-cinder07:27
*** dhinesh has joined #openstack-cinder07:44
*** amoralej has joined #openstack-cinder07:45
*** AlexeyAbashkin has joined #openstack-cinder07:46
*** dhinesh has quit IRC07:48
*** salv-orlando has quit IRC07:51
*** junboli has quit IRC07:51
*** salv-orlando has joined #openstack-cinder07:52
*** ykarel is now known as ykarel|lunch07:53
*** salv-orlando has quit IRC07:56
*** lhx_ has quit IRC07:57
*** gcb has quit IRC07:57
*** armaan has quit IRC07:59
*** armaan has joined #openstack-cinder08:00
openstackgerritTommyLike proposed openstack/python-cinderclient master: Return all resources only when required  https://review.openstack.org/51489608:02
*** salv-orlando has joined #openstack-cinder08:05
*** hoonetorg has quit IRC08:07
openstackgerritVivek Soni proposed openstack/cinder master: HPE 3PAR: Handle single path in FC configuration  https://review.openstack.org/51495208:09
*** danpawlik has quit IRC08:18
*** abhitechie has quit IRC08:18
*** danpawlik has joined #openstack-cinder08:18
*** danpawlik has quit IRC08:18
*** danpawlik has joined #openstack-cinder08:19
*** namnh_ has joined #openstack-cinder08:21
openstackgerritjunbo.li proposed openstack/cinder master: Remove doc/build before running docs job  https://review.openstack.org/51250708:21
*** namnh has quit IRC08:23
*** hoonetorg has joined #openstack-cinder08:24
*** s-shiono has quit IRC08:26
*** jdavis has joined #openstack-cinder08:37
*** josecastroleon has quit IRC08:40
*** jdavis has quit IRC08:42
openstackgerritTommyLike proposed openstack/cinder master: Support count info in List&Detail APIs  https://review.openstack.org/50510108:43
openstackgerritDao Cong Tien proposed openstack/cinder master: Completely remove mox from Cinder unittest  https://review.openstack.org/51464008:44
*** dhinesh has joined #openstack-cinder08:45
*** dhinesh has quit IRC08:49
*** namnh has joined #openstack-cinder08:51
*** namnh_ has quit IRC08:53
*** ykarel|lunch is now known as ykarel08:56
*** fanzhang has joined #openstack-cinder08:56
*** rcernin has quit IRC08:59
*** lpetrut has joined #openstack-cinder09:07
*** jmlowe has quit IRC09:07
*** jmlowe_ has joined #openstack-cinder09:07
*** armaan has quit IRC09:16
*** vivsoni has quit IRC09:16
*** armaan has joined #openstack-cinder09:16
*** armaan has quit IRC09:16
*** storshoo has joined #openstack-cinder09:16
*** vivsoni has joined #openstack-cinder09:16
*** sdague has joined #openstack-cinder09:20
*** abhitechie has joined #openstack-cinder09:20
*** wanghao has quit IRC09:31
openstackgerrityixuan zhang proposed openstack/cinder master: Fix migate_volume API with replication_status ‘not-capable’  https://review.openstack.org/51498809:36
*** arnewiebalck_ has quit IRC09:39
*** vivsoni has quit IRC09:45
*** vivsoni has joined #openstack-cinder09:46
openstackgerritTommyLike proposed openstack/python-cinderclient master: Return all resources only when required  https://review.openstack.org/51489609:50
*** e0ne has joined #openstack-cinder09:53
niraj_singhi am using lvmdriver and want to test service token for cinder to nova interaction. so which API i should call to test this.09:54
*** arnewiebalck_ has joined #openstack-cinder09:56
openstackgerritAlexandru Muresan proposed openstack/cinder master: imageutils: allow passing subformat when converting  https://review.openstack.org/51465810:02
openstackgerritAlexandru Muresan proposed openstack/cinder master: SMBFS: add fixed image support  https://review.openstack.org/51465910:02
*** yangyapeng has quit IRC10:17
*** ganso has joined #openstack-cinder10:25
*** salv-orlando has quit IRC10:32
*** salv-orlando has joined #openstack-cinder10:33
*** pcaruana has quit IRC10:37
*** namnh has quit IRC10:38
*** bkopilov_ has quit IRC10:42
*** chhavi has joined #openstack-cinder10:48
*** arnewiebalck_ has quit IRC10:49
*** tpsilva has joined #openstack-cinder10:49
*** nicolasbock has joined #openstack-cinder10:55
*** armaan has joined #openstack-cinder11:02
*** pgadiya has quit IRC11:02
openstackgerritAbhishek Sharma M proposed openstack/cinder master: Adding path exists check before trying to delete  https://review.openstack.org/51502611:05
openstackgerritAbhishek Sharma M proposed openstack/cinder master: Adding path exists check before trying to delete  https://review.openstack.org/51502611:07
*** stakeda has quit IRC11:07
*** pgadiya has joined #openstack-cinder11:10
*** armaan has quit IRC11:17
*** armaan has joined #openstack-cinder11:17
*** josecastroleon has joined #openstack-cinder11:21
*** yangyapeng has joined #openstack-cinder11:25
*** dave-mccowan has joined #openstack-cinder11:30
*** belmoreira has joined #openstack-cinder11:33
openstackgerritIsaac Beckman proposed openstack/cinder master: ibm-storage: Fix create_volume_from_snapshot  https://review.openstack.org/51503111:34
*** pcaruana has joined #openstack-cinder11:43
*** mvk has quit IRC11:58
*** Guest3211 has quit IRC11:59
*** pgadiya has quit IRC12:00
*** mvk has joined #openstack-cinder12:01
*** abhitechie has quit IRC12:03
*** lpetrut_ has joined #openstack-cinder12:03
*** pcaruana has quit IRC12:03
*** pcaruana has joined #openstack-cinder12:04
*** masuberu has joined #openstack-cinder12:06
*** abishop has joined #openstack-cinder12:06
*** masber has quit IRC12:09
*** pgadiya has joined #openstack-cinder12:09
*** amrith has joined #openstack-cinder12:14
*** amrith is now known as Guest723012:15
*** pgadiya has quit IRC12:21
*** jdavis has joined #openstack-cinder12:22
*** hemna_ has joined #openstack-cinder12:23
*** sbezverk has joined #openstack-cinder12:26
*** Guest7230 has quit IRC12:31
*** edmondsw has joined #openstack-cinder12:32
*** amrith_ has joined #openstack-cinder12:35
*** amrith_ is now known as Guest3070712:35
*** pgadiya has joined #openstack-cinder12:35
*** edmondsw has quit IRC12:37
*** edmondsw has joined #openstack-cinder12:37
*** armaan has quit IRC12:41
*** armaan has joined #openstack-cinder12:41
*** pgadiya has quit IRC12:41
*** danpawlik has quit IRC12:42
*** jdavis has quit IRC12:43
*** markvoelker has joined #openstack-cinder12:43
*** danpawlik has joined #openstack-cinder12:43
*** wanghao has joined #openstack-cinder12:49
*** mriedem has joined #openstack-cinder12:50
*** pgadiya has joined #openstack-cinder12:53
*** bkopilov_ has joined #openstack-cinder12:55
*** pgadiya has quit IRC12:56
*** dustins has joined #openstack-cinder13:03
*** belmoreira has quit IRC13:10
*** ykarel has quit IRC13:15
*** wanghao has quit IRC13:18
*** wanghao has joined #openstack-cinder13:19
*** armaan has quit IRC13:20
*** lhx_ has joined #openstack-cinder13:20
*** armaan has joined #openstack-cinder13:20
*** eharney has joined #openstack-cinder13:24
*** armaan has quit IRC13:32
*** lhx__ has joined #openstack-cinder13:35
*** lhx_ has quit IRC13:35
*** vivsoni_ has joined #openstack-cinder13:41
*** vivsoni has quit IRC13:42
*** sbezverk has quit IRC13:50
*** gus has quit IRC13:51
*** gus has joined #openstack-cinder13:51
*** armax has joined #openstack-cinder13:53
*** lpetrut_ has quit IRC13:53
*** dikonoor has quit IRC13:54
*** dikonoor has joined #openstack-cinder13:55
ericyoungfor driver fixes that need to go back to ocata and newton, should they now go to driverfixes or stable? I assume driverfixes?13:56
e0neericyoung: for driverfixes branches only13:57
e0neericyoung: stable/ocata and stable/newton are open for security issues only13:58
ericyounge0ne, thanks for confirming.13:58
e0neericyoung: you're welcome!13:58
ericyoungI assume the general rule is: if the driverfixes branch exists... that's where things go :)13:59
smcginnisericyoung: For the most part. You can see the allowed changes for normal stables branches here: https://docs.openstack.org/project-team-guide/stable-branches.html#support-phases14:01
ericyoungsmcginnis, thanks. was looking for something like that as well14:02
*** liverpooler has quit IRC14:03
*** wes_dillingham has joined #openstack-cinder14:03
*** eharney has quit IRC14:04
*** Guest30707 is now known as amrith14:10
*** wanghao has quit IRC14:10
*** xyang1 has joined #openstack-cinder14:11
*** lkwan has joined #openstack-cinder14:17
*** salv-orlando has quit IRC14:21
*** salv-orlando has joined #openstack-cinder14:22
*** salv-orlando has quit IRC14:26
*** fanzhang__ has joined #openstack-cinder14:31
*** wanghao has joined #openstack-cinder14:31
*** liverpooler has joined #openstack-cinder14:32
*** fanzhang has quit IRC14:32
*** lkuchlan has quit IRC14:36
*** gouthamr has joined #openstack-cinder14:41
*** david-lyle has quit IRC14:44
*** wanghao has quit IRC14:46
*** david-lyle has joined #openstack-cinder14:53
*** felipemonteiro_ has joined #openstack-cinder14:54
*** eharney has joined #openstack-cinder14:55
*** sbezverk has joined #openstack-cinder14:56
*** felipemonteiro__ has joined #openstack-cinder14:57
*** wanghao has joined #openstack-cinder14:57
*** markstur has joined #openstack-cinder14:57
*** AlexeyAbashkin has quit IRC14:58
*** wanghao has quit IRC14:58
*** wanghao has joined #openstack-cinder14:58
*** felipemonteiro_ has quit IRC15:01
jgriffithmriedem: ?15:01
smcginnisjgriffith: I think there was confusion about the migration, non-null UUID, patch going through and the follow up for it.15:02
jgriffithmriedem: Oh15:02
jgriffithyou logged a bug to :)15:02
jgriffithhttps://bugs.launchpad.net/cinder/+bug/172709115:02
openstackLaunchpad bug 1727091 in Cinder "service object uuid should NOT be nullable" [High,In progress] - Assigned to John Griffith (john-griffith)15:02
smcginnisI duped it to yours, but then noticed you didn't link your patch to the bug, so also commented there.15:02
jgriffithsmcginnis: thanks :)15:02
*** AlexeyAbashkin has joined #openstack-cinder15:03
jgriffithsmcginnis: I didn't add that until I can figure out how to test that migration method15:03
jgriffithsorry, I just put WIP and posted it so I could go home :)15:03
mriedemjgriffith: test should be pretty straight-forward, create like 3 services in the db w/o uuids, create another one that has a uuid, run the migration routine with some limits to see it migrate like 2 of them, then run the rest and verify it migrated 3 of the 4, and the one with the existing uuid wasn't updated15:05
jgriffithmriedem: the problem is that I have a check/generate uuid in the service create15:05
mriedemthe test would create the service records in the database using the models directly, not the object15:05
mriedemjust use the db api methods15:05
jgriffithmriedem: I put some comments here:  https://review.openstack.org/#/c/514853/2/cinder/tests/unit/test_db_api.py15:05
mriedembypass the object15:05
mriedemjgriffith: this is the nova test https://github.com/openstack/nova/blob/master/nova/tests/unit/db/test_db_api.py#L380015:06
mriedemjust copy that15:06
*** wanghao has quit IRC15:06
jgriffithmriedem: well, that's the problem I think; the db.ap.service_create is creating the UUID15:06
jgriffithlooking15:06
mriedemoh i see https://review.openstack.org/#/c/510201/10/cinder/db/sqlalchemy/api.py@57215:07
mriedemyeah...i'd just leave that to the object15:07
jgriffithright15:07
mriedemor,15:07
jgriffithso I was messing with trying to "drop" the column etc15:07
mriedemyou bypass the object AND the model and do straight up sqla insert calls15:08
*** psachin has quit IRC15:08
mriedemoslodbutils.get_table('services')15:08
mriedemtable.insert().execute(values)15:08
jgriffithmriedem: cool, that's what I was looking for15:09
mriedemor just rip out that code in the db api and leave it to the object, either works15:09
jgriffithmriedem: thanks!  I'll check it out after my meeting15:09
mriedemno, thank you15:09
jgriffithmriedem: Yeah, I'm just unclear which is better?  Remove it from the db.api15:09
mriedemthe service create goes through the object and Service.create() is already handling this, so the db api doing it is redundant15:10
jgriffithI mean in terms of the actual usage of the code15:10
mriedemso i'd just remove it from the db api code myself15:10
jgriffithYeah, that's fair; I'll likely remove that from the db api, it will cause confusion later I fear if not :)15:10
jgriffithand to your point it is redundant15:10
jgriffithwith no real value15:10
mriedemcool15:11
mriedemmakes your test life easier too15:11
jgriffithnothing wrong with that :)15:11
jgriffithalrighty.. thanks for the help15:11
*** dhinesh has joined #openstack-cinder15:16
*** dhinesh has quit IRC15:21
*** Apoorva has joined #openstack-cinder15:25
openstackgerritRajini Karthik proposed openstack/cinder master: Dell EMC PS: Fix Duplicate ACL records Issue  https://review.openstack.org/51510715:29
*** hemna_ has quit IRC15:31
*** thiagolib has quit IRC15:32
openstackgerritAbhishek Sharma M proposed openstack/cinder master: Adding path exists check before trying to delete  https://review.openstack.org/51502615:44
*** jccates has joined #openstack-cinder15:47
*** pcaruana has quit IRC15:49
*** mchlumsky has joined #openstack-cinder15:49
openstackgerritJohn Griffith proposed openstack/cinder master: Make service object UUID not nullable  https://review.openstack.org/51485315:57
*** salv-orlando has joined #openstack-cinder16:00
*** itlinux has joined #openstack-cinder16:01
*** mchlumsky_ has joined #openstack-cinder16:01
e0newill we have meeting today?16:01
*** jmlowe_ has quit IRC16:02
*** mchlumsky has quit IRC16:02
*** Guest74730 is now known as geguileo16:03
openstackgerritJohn Griffith proposed openstack/cinder master: Make service object UUID not nullable  https://review.openstack.org/51485316:06
*** jmlowe has joined #openstack-cinder16:08
*** dustins has quit IRC16:10
*** jmlowe has quit IRC16:11
*** dhinesh has joined #openstack-cinder16:17
*** erlon has quit IRC16:20
*** dhinesh has quit IRC16:21
*** e0ne has quit IRC16:21
*** tesseract has quit IRC16:25
*** armaan has joined #openstack-cinder16:29
*** erlon has joined #openstack-cinder16:32
*** AlexeyAbashkin has quit IRC16:38
jgriffithanybody know if something besides the author switch from jenkins/zul took place for gerrit stream monitors?16:56
smcginnisjgriffith: That's all I'm aware of.16:56
smcginnisBut I luckily no longer need to run my own CI.16:57
jgriffithsmcginnis: Yeah, I almost asked you specifically then realized you no longer need to care :)16:57
*** lpetrut has quit IRC16:59
*** dhinesh has joined #openstack-cinder17:04
erlonsmcginnis: hey Sean, quick question, I remember have heard about Cinder being able to configure the interval in which the volume manager called _report_driver_status(). Do you know if that interval is configurable?17:14
erlon@all ^17:14
jgriffitherlon: periodic task interval17:17
jgriffithbut that's effects *other* things as well17:17
*** hemna_ has joined #openstack-cinder17:17
erlonjgriffith: ow, so its only one parameter to control all periodic tasks?17:18
*** dikonoor has quit IRC17:18
jgriffithI don't think ayone every changed that... let me look17:18
jgriffitherlon: doesn't appear as though it's been isolated.  Running genconfig incase there's something "magic" somewhere :)17:19
smcginniserlon, jgriffith: Right, I don't think anythings changed there.17:20
smcginniserlon: We're actually trying to _remove_ config options, but maybe could be something you could add.17:20
jgriffithsmcginnis: erlon yeah, the only option right now is periodic_interval (default is 60 seconds)17:21
jgriffitherlon: the only other thing that seems to use that is the scheduler message cleanup17:21
jgriffithand some *special* netapp driver code seems to be using it as well17:21
jgriffithbut I think that has it's own periodic interval17:22
smcginnisOh, that's good then.17:22
erlonjgriffith: hmm17:22
jgriffitherlon: self.SSC_UPDATE_INTERVAL17:22
jgriffithe-series17:23
erlon@smcginnis by removing config options you mean a specific set of options?  have lost the context of that17:23
jgriffith3par has some too17:23
jgriffitherlon: do a grep on periodic_task to find them all17:23
smcginniserlon: Just the discussion from the PTG about all the drivers having their own config options that basically mean the same thing, and trying to reduce the number of those and have less knobs for deployers to have to turn.17:23
jgriffiththe ones in the drivers appear to be customized at first glance17:23
erlonsmcginnis: ow yes, I remember that17:24
erlonsmcginnis: the thing in this is that the drivers would need another per-backend option to use in the backend level17:25
jgriffitherlon: why would you want to do that?17:25
jgriffitherlon: I mean, why would one backend need a different stats reporting interval than another?17:26
jgriffithThat's just adding too many knobs and dials IMHO17:26
erlonjgriffith: I don't actually, that was what we discussed in the PTG, if we want to reduce the set of repeated options in the drivers17:26
jgriffitherlon: well, that was a different context though17:27
jgriffithwe were talking about driver specific options and how many damn duplicates there are17:27
jgriffithwe *all* have (for the most part) and endpoint, login, pass17:27
jgriffithbut rather than use san_login, san_pass and san_ip17:27
jgriffitheverybody had to have "<vendor>_ip|login|pass"17:28
jgriffithstupid stuff liek that17:28
jgriffithlike17:28
erlonjgriffith: ow yeah that sucks17:28
jgriffitherlon: yeah... it's annoying17:28
*** jmlowe has joined #openstack-cinder17:29
diablo_rojo_phonTell me about it. Working on cleaning them up is quite tedious.17:29
jgriffithdiablo_rojo_phon: :)17:30
diablo_rojo_phonI have a patch just about ready just trying to get tests to pass.17:30
diablo_rojo_phonAnd my testing environment is hosed.17:30
jgriffithF*&^ S#$% D**&^17:37
*** kaisers has quit IRC17:37
*** kaisers has joined #openstack-cinder17:37
jgriffithzul, zuul  WTF is it different in different places :(17:37
jgriffiththose "places" being my code17:37
jgriffithIRADUMDUM17:39
*** lpetrut has joined #openstack-cinder17:43
*** gkadam has quit IRC17:54
*** salv-orlando has quit IRC17:55
*** salv-orlando has joined #openstack-cinder17:56
* jungleboyj keeps accidentally spelling zuul, zull . 17:56
* jungleboyj sighs17:56
SwansonZool me once...17:57
jungleboyjSwanson:  Ok ... I will admit that one was decent.17:59
*** kaisers has quit IRC17:59
*** salv-orlando has quit IRC18:00
*** kaisers has joined #openstack-cinder18:02
*** e0ne has joined #openstack-cinder18:03
*** Apoorva_ has joined #openstack-cinder18:07
*** kaisers has quit IRC18:07
*** dustins has joined #openstack-cinder18:08
*** amoralej is now known as amoralej|off18:09
*** AlexeyAbashkin has joined #openstack-cinder18:10
*** Apoorva has quit IRC18:10
*** strigazi_ has joined #openstack-cinder18:13
*** AlexeyAbashkin has quit IRC18:14
*** strigazi has quit IRC18:16
openstackgerritRajini Karthik proposed openstack/cinder master: Dell EMC: Update PS and SC CI wiki names  https://review.openstack.org/51515918:16
*** strigazi_ is now known as strigazi18:17
*** chhavi has quit IRC18:17
*** kaisers has joined #openstack-cinder18:18
*** kaisers has quit IRC18:23
openstackgerritRajini Karthik proposed openstack/cinder master: Dell EMC: Update PS and SC CI wiki names  https://review.openstack.org/51515918:26
*** kaisers has joined #openstack-cinder18:30
*** lpetrut has quit IRC18:31
*** Guest62252 has joined #openstack-cinder18:36
*** ChubYann has joined #openstack-cinder18:36
*** eharney has quit IRC18:40
*** lhx__ has quit IRC18:41
*** catintheroof has joined #openstack-cinder18:46
*** dustins has quit IRC18:48
*** catintheroof has quit IRC18:49
*** newmember has joined #openstack-cinder18:50
*** catintheroof has joined #openstack-cinder18:50
*** salv-orlando has joined #openstack-cinder18:55
*** eharney has joined #openstack-cinder18:55
*** Apoorva_ has quit IRC19:06
*** Apoorva has joined #openstack-cinder19:06
*** armax has quit IRC19:08
*** Guest62252 has quit IRC19:15
*** wN has joined #openstack-cinder19:16
*** wN has quit IRC19:16
*** wN has joined #openstack-cinder19:16
*** salv-orlando has quit IRC19:33
*** salv-orlando has joined #openstack-cinder19:35
*** swamireddy has quit IRC19:37
*** armax has joined #openstack-cinder19:42
*** wes_dillingham has quit IRC19:46
openstackgerritJon Bernard proposed openstack/cinder-specs master: Update cheesecake promotion specification  https://review.openstack.org/40883019:51
*** AlexeyAbashkin has joined #openstack-cinder19:51
*** dustins has joined #openstack-cinder19:54
*** AlexeyAbashkin has quit IRC19:55
openstackgerritEric Harney proposed openstack/cinder master: Tests: Fix coverage unit test failure  https://review.openstack.org/51519120:01
*** eharney has quit IRC20:01
*** edmondsw_ has joined #openstack-cinder20:06
*** edmondsw has quit IRC20:09
openstackgerritMerged openstack/os-brick master: Enable Python hash randomization for tests  https://review.openstack.org/50206920:18
*** jmlowe has quit IRC20:24
*** e0ne_ has joined #openstack-cinder20:27
*** e0ne has quit IRC20:29
*** e0ne has joined #openstack-cinder20:42
openstackgerritAbhishek Sharma M proposed openstack/cinder master: Adding path exists check before trying to delete  https://review.openstack.org/51502620:44
*** e0ne_ has quit IRC20:45
*** abishop has quit IRC20:51
*** AlexeyAbashkin has joined #openstack-cinder20:51
*** masuberu has quit IRC20:51
*** edmondsw_ is now known as edmondsw20:55
*** AlexeyAbashkin has quit IRC20:55
*** hemna_ has quit IRC21:03
*** crose has joined #openstack-cinder21:11
*** rcernin has joined #openstack-cinder21:16
openstackgerritMerged openstack/cinder master: Fix key_manager API call  https://review.openstack.org/51445621:17
diablo_rojoWas over in the #openstack-101 channel and this guy asked about getting his patch reviewed: https://review.openstack.org/#/c/508771/21:20
*** xyang1 has quit IRC21:23
*** e0ne has quit IRC21:25
*** sbezverk has quit IRC21:27
*** crose has quit IRC21:28
*** wes_dillingham has joined #openstack-cinder21:37
*** AlexeyAbashkin has joined #openstack-cinder21:51
*** edmondsw has quit IRC21:51
*** kaisers has quit IRC21:54
*** AlexeyAbashkin has quit IRC21:55
*** edmondsw has joined #openstack-cinder21:56
*** sdague has quit IRC21:56
*** markstur has quit IRC22:02
*** kaisers has joined #openstack-cinder22:02
*** edmondsw has quit IRC22:04
*** dustins has quit IRC22:06
*** itlinux has quit IRC22:14
*** catintheroof has quit IRC22:18
*** felipemonteiro__ has quit IRC22:24
*** edmondsw has joined #openstack-cinder22:28
*** edmondsw has quit IRC22:34
*** felipemonteiro_ has joined #openstack-cinder22:43
jungleboyjdiablo_rojo:  Done.22:45
Swanson"... it's a family act."22:49
jungleboyjSwanson:  I don't want to know.22:50
*** AlexeyAbashkin has joined #openstack-cinder22:52
*** felipemonteiro_ has quit IRC22:53
diablo_rojojungleboyj, thanks22:53
*** AlexeyAbashkin has quit IRC22:56
*** mchlumsky_ has quit IRC22:56
*** tpsilva has quit IRC22:57
*** masuberu has joined #openstack-cinder23:05
*** swamireddy has joined #openstack-cinder23:13
*** gouthamr has quit IRC23:22
*** gouthamr has joined #openstack-cinder23:22
*** jccates has quit IRC23:38
*** abhitechie has joined #openstack-cinder23:39
*** mriedem is now known as mriedem_away23:44
*** mriedem_away has quit IRC23:44
*** liverpooler has quit IRC23:47
*** markstur has joined #openstack-cinder23:49
*** abhitechie has quit IRC23:57

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