*** suyash has quit IRC | 00:05 | |
*** akapil has joined #openstack-cinder | 00:06 | |
*** karthikp_ has quit IRC | 00:14 | |
*** smoriya has joined #openstack-cinder | 00:17 | |
*** akapil has quit IRC | 00:20 | |
*** laughterwym has joined #openstack-cinder | 00:20 | |
*** laughterwym has quit IRC | 00:26 | |
*** laughterwym has joined #openstack-cinder | 00:26 | |
*** laughterwym has joined #openstack-cinder | 00:27 | |
*** arch-nemesis has quit IRC | 00:28 | |
*** zul has joined #openstack-cinder | 00:28 | |
*** itzdilip has joined #openstack-cinder | 00:37 | |
*** itzdilip has quit IRC | 00:42 | |
*** cknight has joined #openstack-cinder | 00:45 | |
*** cknight1 has quit IRC | 00:45 | |
*** diogogmt has joined #openstack-cinder | 00:52 | |
*** cknight1 has joined #openstack-cinder | 00:53 | |
*** cknight has quit IRC | 00:54 | |
*** sdake has joined #openstack-cinder | 00:58 | |
*** bardia has quit IRC | 01:00 | |
*** Apoorva has quit IRC | 01:01 | |
*** bardia has joined #openstack-cinder | 01:05 | |
*** diogogmt has quit IRC | 01:06 | |
*** cheneydc has joined #openstack-cinder | 01:13 | |
*** sdake has quit IRC | 01:14 | |
*** merooney has quit IRC | 01:14 | |
*** iceyao has joined #openstack-cinder | 01:15 | |
*** sdake has joined #openstack-cinder | 01:16 | |
*** EinstCrazy has joined #openstack-cinder | 01:21 | |
*** itzdilip has joined #openstack-cinder | 01:22 | |
*** sdake_ has joined #openstack-cinder | 01:22 | |
*** sdake has quit IRC | 01:24 | |
*** yangyapeng has joined #openstack-cinder | 01:25 | |
*** wN has quit IRC | 01:27 | |
jgriffith | smcginnis: so do you want to just merge your fix rather than that revert? | 01:30 |
---|---|---|
jgriffith | smcginnis: I'd suggest we abandon that revert if that's the case. I haven't had time to load both up and do a mysql-dump... but assuming you or Jay did? | 01:31 |
*** Apoorva has joined #openstack-cinder | 01:33 | |
*** Lee1092 has joined #openstack-cinder | 01:35 | |
*** geguileo_phone has quit IRC | 01:37 | |
*** itzdilip has quit IRC | 01:37 | |
*** itzdilip has joined #openstack-cinder | 01:38 | |
openstackgerrit | wuyuting proposed openstack/cinder: Fix infortrend driver bug: cloned with src vol size https://review.openstack.org/326792 | 01:41 |
*** mtanino has quit IRC | 01:45 | |
openstackgerrit | wuyuting proposed openstack/cinder: ProphetStor driver create vol from snap with wrong size https://review.openstack.org/329741 | 01:48 |
openstackgerrit | liuke proposed openstack/cinder: Huawei: Support reporting disk type of pool https://review.openstack.org/324163 | 01:50 |
mtreinish | jgriffith: any idea why cinder unit tests dump tracebacks even when they pass?: http://logs.openstack.org/70/330570/2/check/gate-cinder-python34-db/ffc9b11/console.html#_2016-06-16_15_41_42_629464 | 01:52 |
openstackgerrit | Merged openstack/cinder: SolidFire driver should return ceiling of volume size https://review.openstack.org/326776 | 01:55 |
*** itzdilip has quit IRC | 02:01 | |
*** Apoorva has quit IRC | 02:05 | |
*** garthb has joined #openstack-cinder | 02:10 | |
*** sdake has joined #openstack-cinder | 02:16 | |
*** huanan has quit IRC | 02:16 | |
*** akapil has joined #openstack-cinder | 02:18 | |
*** sdake_ has quit IRC | 02:19 | |
*** mtanino has joined #openstack-cinder | 02:22 | |
*** houming has joined #openstack-cinder | 02:24 | |
*** Suyash has joined #openstack-cinder | 02:25 | |
*** akapil has quit IRC | 02:38 | |
openstackgerrit | xiexs proposed openstack/cinder: Move _check_metadata_properties to cinder/utils https://review.openstack.org/330502 | 02:41 |
*** garthb has quit IRC | 02:52 | |
*** adisky has joined #openstack-cinder | 02:53 | |
openstackgerrit | huanan proposed openstack/cinder: Huawei: Fixed problems caused by password expiration https://review.openstack.org/305633 | 02:53 |
*** huanan has joined #openstack-cinder | 02:55 | |
*** jamielennox is now known as jamielennox|away | 03:03 | |
*** gouthamr_ has quit IRC | 03:12 | |
*** jamielennox|away is now known as jamielennox | 03:19 | |
openstackgerrit | wuyuting proposed openstack/cinder: Fix infortrend driver bug: cloned with src vol size https://review.openstack.org/326792 | 03:29 |
openstackgerrit | Merged openstack/cinder: Define context.roles with base class https://review.openstack.org/285354 | 03:34 |
*** bardia has quit IRC | 03:37 | |
*** markvoelker has quit IRC | 03:47 | |
*** mriedem has quit IRC | 03:51 | |
*** yhayashi has joined #openstack-cinder | 03:53 | |
*** cknight1 has quit IRC | 03:55 | |
*** GB21 has joined #openstack-cinder | 04:02 | |
*** lpetrut has joined #openstack-cinder | 04:03 | |
openstackgerrit | haobing proposed openstack/cinder: Allow setting CG name or description to empty value https://review.openstack.org/324465 | 04:12 |
nikeshm | hi | 04:19 |
nikeshm | do cinder supports cloning a volume from a attached volume? | 04:19 |
*** links has joined #openstack-cinder | 04:20 | |
nikeshm | e0ne: there | 04:22 |
nikeshm | ? | 04:22 |
*** sgotliv_ has joined #openstack-cinder | 04:24 | |
*** laughterwym has quit IRC | 04:24 | |
*** lpetrut has quit IRC | 04:28 | |
*** ociuhandu has quit IRC | 04:29 | |
*** sgotliv_ has quit IRC | 04:29 | |
*** karthikp has joined #openstack-cinder | 04:33 | |
*** cheneydc has quit IRC | 04:35 | |
openstackgerrit | Vijay Ladani proposed openstack/cinder: Add unit tests in cinder for HPE Lefthand driver https://review.openstack.org/317948 | 04:40 |
*** Suyash has quit IRC | 04:41 | |
*** ociuhandu has joined #openstack-cinder | 04:42 | |
*** markvoelker has joined #openstack-cinder | 04:47 | |
*** sdake_ has joined #openstack-cinder | 04:50 | |
*** markvoelker has quit IRC | 04:52 | |
*** sdake has quit IRC | 04:53 | |
*** GB21 has quit IRC | 04:53 | |
*** garthb has joined #openstack-cinder | 04:57 | |
*** garthb has quit IRC | 04:58 | |
*** lprice has joined #openstack-cinder | 05:01 | |
*** Poornima has joined #openstack-cinder | 05:03 | |
*** salv-orl_ has joined #openstack-cinder | 05:09 | |
*** ChubYann has quit IRC | 05:11 | |
*** salv-orlando has quit IRC | 05:12 | |
*** pgadiya has joined #openstack-cinder | 05:12 | |
*** lpetrut has joined #openstack-cinder | 05:14 | |
openstackgerrit | Parashuram Hallur proposed openstack/cinder: Implementation for CoprHD Cinder Drivers https://review.openstack.org/317643 | 05:14 |
*** akapil has joined #openstack-cinder | 05:21 | |
*** links has quit IRC | 05:27 | |
*** akapil has quit IRC | 05:27 | |
*** sdake has joined #openstack-cinder | 05:27 | |
*** links has joined #openstack-cinder | 05:28 | |
*** yangyape_ has joined #openstack-cinder | 05:28 | |
*** cknight has joined #openstack-cinder | 05:29 | |
*** sdake_ has quit IRC | 05:31 | |
*** yangyapeng has quit IRC | 05:31 | |
*** cknight has quit IRC | 05:33 | |
lixiaoy1 | nikeshm: Cinder supports cloning a volume from in-use volume | 05:34 |
*** GB21 has joined #openstack-cinder | 05:34 | |
nikeshm | thanks | 05:35 |
lixiaoy1 | nikeshm: as clone is done in backend storages | 05:35 |
*** laughterwym has joined #openstack-cinder | 05:36 | |
*** sdake_ has joined #openstack-cinder | 05:38 | |
*** sdake has quit IRC | 05:41 | |
*** bhavik has joined #openstack-cinder | 05:42 | |
*** bhavik has quit IRC | 05:47 | |
*** karthikp has quit IRC | 05:48 | |
*** sdake_ has quit IRC | 05:50 | |
openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Fix useless api_version of Manager class https://review.openstack.org/309283 | 05:50 |
*** lprice1 has joined #openstack-cinder | 05:58 | |
*** lprice has quit IRC | 05:59 | |
*** karthikp has joined #openstack-cinder | 06:01 | |
*** bhavik has joined #openstack-cinder | 06:03 | |
*** karthikp has quit IRC | 06:05 | |
*** belmoreira has joined #openstack-cinder | 06:06 | |
*** nkrinner has joined #openstack-cinder | 06:10 | |
*** rcernin has joined #openstack-cinder | 06:24 | |
*** mtanino has quit IRC | 06:24 | |
*** david-lyle has quit IRC | 06:26 | |
*** wanghao has quit IRC | 06:28 | |
*** lpetrut has quit IRC | 06:33 | |
*** arecknag has joined #openstack-cinder | 06:35 | |
openstackgerrit | wanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly https://review.openstack.org/232436 | 06:49 |
*** sheel has joined #openstack-cinder | 06:50 | |
openstackgerrit | wanghao proposed openstack/cinder: Fix snapshot stuck in 'deleting' after reboot c-v https://review.openstack.org/322418 | 06:52 |
*** sticker has quit IRC | 06:54 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware:Persist storage policy ID in vmdk meta-data https://review.openstack.org/330911 | 06:58 |
openstackgerrit | xiexs proposed openstack/cinder: Add validation for the metadata properties https://review.openstack.org/330913 | 06:59 |
*** itzdilip has joined #openstack-cinder | 07:01 | |
*** ihrachys has joined #openstack-cinder | 07:03 | |
*** cknight has joined #openstack-cinder | 07:04 | |
*** ihrachys has quit IRC | 07:08 | |
*** salv-orl_ has quit IRC | 07:09 | |
*** salv-orlando has joined #openstack-cinder | 07:09 | |
*** cknight has quit IRC | 07:09 | |
*** itzdilip has quit IRC | 07:10 | |
*** tesseract has joined #openstack-cinder | 07:10 | |
*** hogepodge has quit IRC | 07:12 | |
*** bauzas is now known as bauwser | 07:13 | |
*** bauwser is now known as bauzas | 07:14 | |
openstackgerrit | ningwei proposed openstack/cinder: Added backend driver for Huawei FusionStorage https://review.openstack.org/328887 | 07:14 |
*** bauzas has left #openstack-cinder | 07:18 | |
*** GB21 has quit IRC | 07:23 | |
*** salv-orlando has quit IRC | 07:35 | |
*** gcb has joined #openstack-cinder | 07:35 | |
*** EinstCra_ has joined #openstack-cinder | 07:38 | |
*** EinstCrazy has quit IRC | 07:40 | |
*** yangyape_ has quit IRC | 07:40 | |
huyang | kindly review: https://review.openstack.org/#/c/325682/ | 07:40 |
*** claudiub has joined #openstack-cinder | 07:41 | |
*** iceyao has quit IRC | 07:42 | |
*** iceyao has joined #openstack-cinder | 07:42 | |
*** EinstCra_ has quit IRC | 07:43 | |
*** EinstCrazy has joined #openstack-cinder | 07:43 | |
*** zhangjn has quit IRC | 07:45 | |
*** rcernin has quit IRC | 07:45 | |
openstackgerrit | huanan proposed openstack/cinder: Huawei: Fixed problems caused by password expiration https://review.openstack.org/305633 | 07:46 |
*** GB21 has joined #openstack-cinder | 07:49 | |
*** zhangjn has joined #openstack-cinder | 07:50 | |
*** lpetrut has joined #openstack-cinder | 07:57 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-cinder | 08:00 | |
*** cknight has joined #openstack-cinder | 08:06 | |
*** yangyapeng has joined #openstack-cinder | 08:07 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
*** rcernin has joined #openstack-cinder | 08:10 | |
*** cknight has quit IRC | 08:12 | |
*** itzdilip has joined #openstack-cinder | 08:12 | |
*** huyang_ has joined #openstack-cinder | 08:20 | |
*** ameade has quit IRC | 08:21 | |
*** thurloat has quit IRC | 08:21 | |
*** v12aml has quit IRC | 08:22 | |
*** ameade has joined #openstack-cinder | 08:22 | |
*** tyrola has joined #openstack-cinder | 08:22 | |
*** huyang has quit IRC | 08:23 | |
*** huyang_ is now known as huyang | 08:23 | |
*** thurloat has joined #openstack-cinder | 08:23 | |
*** v12aml has joined #openstack-cinder | 08:23 | |
*** tyrola has quit IRC | 08:23 | |
*** itzdilip has quit IRC | 08:23 | |
openstackgerrit | nikesh proposed openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver https://review.openstack.org/327285 | 08:33 |
*** bhagyashris has joined #openstack-cinder | 08:36 | |
*** mkrcmari__ has joined #openstack-cinder | 08:36 | |
*** rmstar has joined #openstack-cinder | 08:36 | |
*** kaisers has quit IRC | 08:37 | |
*** pgadiya_ has joined #openstack-cinder | 08:37 | |
*** dramakri has quit IRC | 08:37 | |
*** liverpoo1er has joined #openstack-cinder | 08:38 | |
*** TheBall has joined #openstack-cinder | 08:38 | |
*** kaisers has joined #openstack-cinder | 08:38 | |
*** ntpttr_ has quit IRC | 08:38 | |
*** bswartz has quit IRC | 08:39 | |
*** dramakri has joined #openstack-cinder | 08:39 | |
*** itzdilip has joined #openstack-cinder | 08:39 | |
*** bbhavsar_ has joined #openstack-cinder | 08:39 | |
*** Poornima_K has joined #openstack-cinder | 08:40 | |
*** ntpttr_ has joined #openstack-cinder | 08:40 | |
*** harlowja_ has quit IRC | 08:41 | |
*** winston-1 has quit IRC | 08:42 | |
*** anteaya has quit IRC | 08:42 | |
*** jay-mehta has quit IRC | 08:42 | |
*** mvk_ has quit IRC | 08:42 | |
*** bhagyashri has quit IRC | 08:42 | |
*** rmstar_ has quit IRC | 08:42 | |
*** ekarlso has quit IRC | 08:42 | |
*** bhavik has quit IRC | 08:42 | |
*** winston-d has joined #openstack-cinder | 08:42 | |
*** The_Ball has quit IRC | 08:42 | |
*** TheBall is now known as The_Ball | 08:43 | |
*** arecknag has quit IRC | 08:43 | |
*** mattymo has quit IRC | 08:43 | |
*** pgadiya has quit IRC | 08:43 | |
*** mdavidson has quit IRC | 08:43 | |
*** liverpooler has quit IRC | 08:43 | |
*** kaisers1 has quit IRC | 08:43 | |
*** Poornima has quit IRC | 08:43 | |
*** SergeyLukjanov2 has joined #openstack-cinder | 08:43 | |
*** asselin_ has joined #openstack-cinder | 08:43 | |
*** SergeyLukjanov has quit IRC | 08:43 | |
*** xek has quit IRC | 08:43 | |
*** arecknag has joined #openstack-cinder | 08:43 | |
*** arecknag has joined #openstack-cinder | 08:43 | |
openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX Driver - SnapVX licensing checks for VMAX3 https://review.openstack.org/322972 | 08:43 |
*** SergeyLukjanov2 is now known as SergeyLukjanov | 08:43 | |
*** xek has joined #openstack-cinder | 08:43 | |
*** asselin__ has quit IRC | 08:44 | |
*** mattymo has joined #openstack-cinder | 08:44 | |
*** mtreinish has quit IRC | 08:44 | |
*** e0ne has joined #openstack-cinder | 08:44 | |
*** ihrachys has joined #openstack-cinder | 08:44 | |
*** mdavidson has joined #openstack-cinder | 08:44 | |
*** kaisers1 has joined #openstack-cinder | 08:45 | |
*** anteaya has joined #openstack-cinder | 08:45 | |
*** mtreinish has joined #openstack-cinder | 08:46 | |
*** jay-mehta has joined #openstack-cinder | 08:46 | |
*** tpeoples has quit IRC | 08:47 | |
*** tpeoples has joined #openstack-cinder | 08:49 | |
*** ekarlso has joined #openstack-cinder | 08:51 | |
*** laughterwym has quit IRC | 08:52 | |
*** EinstCrazy has quit IRC | 08:55 | |
*** itzdilip has quit IRC | 08:58 | |
*** EinstCrazy has joined #openstack-cinder | 08:59 | |
openstackgerrit | Abhishek Kekane proposed openstack/python-cinderclient: Log request-id for each api call https://review.openstack.org/315925 | 09:03 |
*** EinstCra_ has joined #openstack-cinder | 09:06 | |
*** cknight has joined #openstack-cinder | 09:07 | |
*** EinstCrazy has quit IRC | 09:09 | |
*** cknight has quit IRC | 09:12 | |
zigo | Hi there! Running tempest on Newton b1 of Cinder, I have a small issue. The test: | 09:13 |
zigo | tempest.api.volume.admin.test_volume_services.VolumesServicesV1TestJSON.test_get_service_by_host_name | 09:13 |
zigo | fails with this return: | 09:13 |
zigo | reference = [u'cinder-backup', u'cinder-scheduler', u'cinder-volume'] | 09:13 |
zigo | actual = [u'cinder-backup', u'cinder-scheduler'] | 09:13 |
zigo | I believe the issue is that cinder-volume replies debian@lvm as a host, instead of just "debian" as the others do. How can I fix this? | 09:13 |
zigo | e0ne: ^ | 09:13 |
*** e0ne has quit IRC | 09:13 | |
*** openstackgerrit has quit IRC | 09:18 | |
*** openstackgerrit has joined #openstack-cinder | 09:19 | |
*** geguileo_phone has joined #openstack-cinder | 09:20 | |
*** nobody_ has joined #openstack-cinder | 09:27 | |
*** karthikp has joined #openstack-cinder | 09:31 | |
*** karthikp has quit IRC | 09:35 | |
*** bbhavsar_ has quit IRC | 09:42 | |
*** itzdilip has joined #openstack-cinder | 09:44 | |
*** cknight has joined #openstack-cinder | 09:54 | |
*** savihou has quit IRC | 09:55 | |
openstackgerrit | ningwei proposed openstack/cinder: Added backend driver for Huawei FusionStorage https://review.openstack.org/328887 | 09:58 |
*** cknight has quit IRC | 09:58 | |
*** gouthamr has joined #openstack-cinder | 09:59 | |
*** gouthamr_ has joined #openstack-cinder | 10:00 | |
*** yhayashi has quit IRC | 10:01 | |
*** openstackgerrit has quit IRC | 10:03 | |
*** openstackgerrit has joined #openstack-cinder | 10:04 | |
*** gouthamr has quit IRC | 10:04 | |
*** rcernin has quit IRC | 10:05 | |
*** GB21 has quit IRC | 10:09 | |
*** e0ne has joined #openstack-cinder | 10:12 | |
openstackgerrit | zhangsong proposed openstack/os-brick: Fix the bug of 'AttributeError' when connect volume https://review.openstack.org/331007 | 10:14 |
openstackgerrit | Parashuram Hallur proposed openstack/cinder: Implementation for CoprHD Cinder Drivers https://review.openstack.org/317643 | 10:16 |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix quota-usage list information with invalid tenant-id https://review.openstack.org/315673 | 10:23 |
*** ociuhandu has quit IRC | 10:23 | |
*** e0ne has quit IRC | 10:24 | |
openstackgerrit | Szymon Borkowski proposed openstack/cinder: Convert backup_device to OVO https://review.openstack.org/329332 | 10:28 |
*** e0ne has joined #openstack-cinder | 10:29 | |
*** iceyao has quit IRC | 10:40 | |
*** gouthamr_ is now known as gouthamr | 10:40 | |
*** GB21 has joined #openstack-cinder | 10:40 | |
*** ociuhandu has joined #openstack-cinder | 10:43 | |
*** huyang has quit IRC | 10:44 | |
*** huyang has joined #openstack-cinder | 10:45 | |
*** permalac has joined #openstack-cinder | 10:47 | |
openstackgerrit | Stefan Amann proposed openstack/os-brick: WIP: Support FICON-attached ECKD volumes on System z https://review.openstack.org/320536 | 10:47 |
*** e0ne has quit IRC | 10:47 | |
*** hogepodge has joined #openstack-cinder | 10:54 | |
*** cknight has joined #openstack-cinder | 10:55 | |
openstackgerrit | Stefan Amann proposed openstack/os-brick: WIP: Support FICON-attached ECKD volumes on System z https://review.openstack.org/320536 | 10:56 |
*** GB21 has quit IRC | 10:59 | |
*** cknight has quit IRC | 11:00 | |
*** smoriya has quit IRC | 11:05 | |
*** GB21 has joined #openstack-cinder | 11:06 | |
*** itzdilip has quit IRC | 11:14 | |
*** itzdilip has joined #openstack-cinder | 11:15 | |
*** itzdilip has quit IRC | 11:28 | |
*** nobody_ has quit IRC | 11:29 | |
*** itzdilip has joined #openstack-cinder | 11:30 | |
openstackgerrit | nikesh proposed openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver https://review.openstack.org/327285 | 11:30 |
*** bjornar_ has joined #openstack-cinder | 11:31 | |
*** itzdilip has quit IRC | 11:31 | |
*** itzdilip has joined #openstack-cinder | 11:31 | |
*** gcb has quit IRC | 11:33 | |
*** gouthamr has quit IRC | 11:43 | |
*** rcernin has joined #openstack-cinder | 11:48 | |
*** gcb has joined #openstack-cinder | 11:49 | |
*** houming has quit IRC | 11:49 | |
*** gouthamr has joined #openstack-cinder | 11:50 | |
*** EinstCra_ has quit IRC | 11:53 | |
*** eharney has joined #openstack-cinder | 11:55 | |
openstackgerrit | Vijay Ladani proposed openstack/cinder: Add unit tests in cinder for HPE Lefthand driver https://review.openstack.org/317948 | 11:55 |
*** cknight has joined #openstack-cinder | 11:56 | |
*** EinstCrazy has joined #openstack-cinder | 11:57 | |
*** EinstCra_ has joined #openstack-cinder | 12:00 | |
*** cknight has quit IRC | 12:00 | |
*** EinstCrazy has quit IRC | 12:02 | |
*** gouthamr_ has joined #openstack-cinder | 12:02 | |
*** itzdilip has quit IRC | 12:02 | |
*** gouthamr_ has quit IRC | 12:02 | |
*** gouthamr_ has joined #openstack-cinder | 12:03 | |
*** cdelatte has quit IRC | 12:03 | |
*** EinstCrazy has joined #openstack-cinder | 12:03 | |
*** dave-mccowan has joined #openstack-cinder | 12:04 | |
*** timcl has joined #openstack-cinder | 12:04 | |
*** gouthamr_ has quit IRC | 12:04 | |
*** markvoelker has joined #openstack-cinder | 12:04 | |
*** gouthamr_ has joined #openstack-cinder | 12:04 | |
*** gouthamr has quit IRC | 12:05 | |
*** gouthamr_ is now known as gouthamr | 12:05 | |
*** EinstCra_ has quit IRC | 12:06 | |
*** kfarr has joined #openstack-cinder | 12:07 | |
*** e0ne has joined #openstack-cinder | 12:07 | |
*** mtanino has joined #openstack-cinder | 12:07 | |
*** itzdilip has joined #openstack-cinder | 12:09 | |
*** EinstCra_ has joined #openstack-cinder | 12:11 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 12:11 |
*** laughterwym has joined #openstack-cinder | 12:11 | |
*** EinstCra_ has quit IRC | 12:11 | |
*** EinstCra_ has joined #openstack-cinder | 12:13 | |
*** iceyao has joined #openstack-cinder | 12:14 | |
*** EinstCrazy has quit IRC | 12:14 | |
*** laughterwym has quit IRC | 12:15 | |
*** itzdilip has quit IRC | 12:16 | |
*** itzdilip has joined #openstack-cinder | 12:19 | |
*** zhangjn has quit IRC | 12:19 | |
*** cdelatte has joined #openstack-cinder | 12:19 | |
*** EinstCra_ has quit IRC | 12:21 | |
*** GB21 has quit IRC | 12:23 | |
*** zhangjn has joined #openstack-cinder | 12:23 | |
*** yangyapeng has quit IRC | 12:24 | |
*** iceyao has quit IRC | 12:24 | |
jgriffith | mtreinish: No I don't... looks like CloudByte is dumping a bunch of logging . I'll have a look at it today | 12:24 |
*** cdelatte has quit IRC | 12:25 | |
e0ne | jgriffith: morning, John. one short qurestion according to your comment to my patch | 12:27 |
e0ne | jgriffith: I like the idea to have functional tests with fake driver. shoult I propose spec for it? | 12:27 |
jgriffith | e0ne: I don't think spec is necessary, but if you want to that's cool | 12:30 |
*** cdelatte has joined #openstack-cinder | 12:30 | |
e0ne | jgriffith: I'll try to bring this topic to the next testing relating meeting. it community requests spec, I'll do it | 12:31 |
*** jsheeren has joined #openstack-cinder | 12:31 | |
*** pauloewerton has joined #openstack-cinder | 12:34 | |
*** edmondsw has joined #openstack-cinder | 12:34 | |
*** timcl has quit IRC | 12:39 | |
*** cdelatte has quit IRC | 12:39 | |
*** alaski is now known as lascii | 12:40 | |
*** cdelatte has joined #openstack-cinder | 12:41 | |
*** adrianofr has joined #openstack-cinder | 12:43 | |
*** bswartz has joined #openstack-cinder | 12:43 | |
*** Srinu has quit IRC | 12:44 | |
*** timcl has joined #openstack-cinder | 12:44 | |
claudiub | hemna: smcginnis: hello. can you take a look at the os-brick Windows iSCSI / fibre channel / SMB connectors? lpetrut has posted tempest test results for each of them: https://review.openstack.org/#/c/272522/29 | 12:46 |
smcginnis | claudiub: Will try to get to it today. | 12:48 |
smcginnis | jgriffith: Saw your ping from yesterday afternoon. Yeah, I'd rather just fix things than revert that whole thing. | 12:50 |
smcginnis | jgriffith: Though if folks prefer and this ends up more difficult than it should be, I could go release by release and compact that way to make things easier. | 12:51 |
smcginnis | dulek: Saw your comments. Any reason why we would need to make sure the index names are the same? I didn't think that mattered. | 12:51 |
nikeshm | smcginnis: luckily i saw your recent merge and added in kaminario driver too | 12:55 |
nikeshm | interface | 12:55 |
nikeshm | thing | 12:55 |
smcginnis | nikeshm: Just noticed that. Thanks! | 12:56 |
jgriffith | smcginnis: yuck... no thanks | 12:57 |
smcginnis | jgriffith: Yeah... :) | 12:58 |
dulek | smcginnis: My perception is that unified names will be useful if we'll happen to remove the constraint. | 13:01 |
smcginnis | dulek: Oh, good point. | 13:01 |
jgriffith | dulek: aren't the unified names sort of what's causing issue/confusion now though? | 13:02 |
*** faiz89 has joined #openstack-cinder | 13:02 | |
*** gcb has quit IRC | 13:02 | |
*** cdelatte has quit IRC | 13:03 | |
dulek | jgriffith: I don't think I follow. | 13:03 |
jgriffith | dulek: it may be me that's not following :) | 13:03 |
jgriffith | dulek: sorry... | 13:03 |
jgriffith | dulek: I was looking at your comment on smcginnis 's compaction update | 13:03 |
*** akerr has joined #openstack-cinder | 13:04 | |
smcginnis | I can add explicit naming. That's pretty trivial. | 13:04 |
*** faiz89 has quit IRC | 13:04 | |
jgriffith | I'm trying to figure out if it matters :) | 13:04 |
smcginnis | Not sure if that's the same across all DBs anyway. If we remove an index, we wouldn't do so by name via sqlalchemy, would we? | 13:05 |
jgriffith | we've added so much crap to the DB stuff it's sort of ridic | 13:05 |
jgriffith | smcginnis: I would hope we'd always enforce ID, but who knows | 13:05 |
smcginnis | jgriffith: I didn't realize how true that was until I tried following its evolution. :) | 13:05 |
dulek | jgriffith: I think it may be DB-specific, but if we wanted to change or delete a constraint, having same names across deployments would be useful. | 13:05 |
jgriffith | smcginnis: yeah, it's kinda crazy. | 13:06 |
smcginnis | dulek: Yeah, I'll add it. Not sure it's necessary, but I don't think it will hurt anything. | 13:06 |
dulek | smcginnis: It's possible via SQLA I think. Let me dig one old commit… | 13:06 |
smcginnis | dulek: Had a minute of confusion here: http://paste.openstack.org/show/516822/ | 13:06 |
smcginnis | With the CREATE SEQUENCE bit. That must be an interim step, because a few migrations into it we've ended up switching ID over to a varchar(36) to accomodate UUIDs. | 13:07 |
dulek | smcginnis: Oh! | 13:07 |
smcginnis | dulek: So I'm pretty sure we don't want to start at 1 and increment by 1. :) | 13:07 |
dulek | smcginnis: Nice explanation. | 13:07 |
smcginnis | dulek: A lot of these little things have ended up being stuff like that. That's made it a little harder to have to follow the whole story. | 13:08 |
dulek | smcginnis: I totally believe it was incredibly hard. | 13:09 |
smcginnis | dulek: It just needed some patience. ;) | 13:09 |
smcginnis | A lot of "why the heck..." | 13:09 |
dulek | And I don't want to be overreacting, but I really cannot predict if these small differences wouldn't hurt us in the future. | 13:09 |
smcginnis | Though "heck" wasn't usually the word I used. | 13:09 |
smcginnis | dulek: Yeah, I appreciate that! | 13:10 |
*** xyang1 has joined #openstack-cinder | 13:10 | |
dulek | With all the DBs implementation specifics… It's just impossible to tell. | 13:10 |
smcginnis | dulek: I did want to get this in early for that reason. The longer we have in the cycle to shake out any gotchas the better. | 13:10 |
dulek | geguileo_phone knows something about implementation specifics of DBs. :D | 13:10 |
smcginnis | dulek: Hah! I'm sure more than I care to know. | 13:11 |
smcginnis | jgriffith: You cool if I just set the index names? Or do you still have concerns with that? | 13:12 |
*** nkrinner has quit IRC | 13:14 | |
*** laughterwym has joined #openstack-cinder | 13:16 | |
*** nkrinner has joined #openstack-cinder | 13:19 | |
*** diablo_rojo has joined #openstack-cinder | 13:19 | |
*** dustins has joined #openstack-cinder | 13:20 | |
*** laughterwym has quit IRC | 13:20 | |
zigo | e0ne: Around? | 13:22 |
e0ne | zigo: yes | 13:22 |
*** faiz89 has joined #openstack-cinder | 13:22 | |
zigo | Hi there! Running tempest on Newton b1 of Cinder, I have a small issue. The test: | 13:22 |
zigo | tempest.api.volume.admin.test_volume_services.VolumesServicesV1TestJSON.test_get_service_by_host_name | 13:22 |
zigo | fails with this return: | 13:22 |
zigo | reference = [u'cinder-backup', u'cinder-scheduler', u'cinder-volume'] | 13:22 |
zigo | actual = [u'cinder-backup', u'cinder-scheduler'] | 13:22 |
zigo | I believe the issue is that cinder-volume replies debian@lvm as a host, instead of just "debian" as the others do. How can I fix this? | 13:22 |
zigo | e0ne: ^ | 13:22 |
*** jsheeren has quit IRC | 13:23 | |
*** akshai has joined #openstack-cinder | 13:23 | |
*** nkrinner has quit IRC | 13:24 | |
e0ne | zigo: do you use latest tempest? | 13:25 |
zigo | e0ne: I use version 12.0.0, and I had a look, that piece of code hasn't changed, seemingly. | 13:25 |
*** timcl has quit IRC | 13:26 | |
zigo | I may be wrong. | 13:26 |
zigo | (if one of the functions used in the test case has changed, I didn't check for it) | 13:26 |
*** takedakn has joined #openstack-cinder | 13:26 | |
e0ne | zigo: give me few minutes to take a look on it | 13:27 |
zigo | e0ne: Oh, I found something... | 13:28 |
zigo | From tempest.conf: [volume-feature-enabled]/volume_services | 13:29 |
zigo | Comments say: Extract correct host info from host@backend (boolean value) | 13:29 |
zigo | I think I should try that. | 13:29 |
*** merooney has joined #openstack-cinder | 13:30 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls https://review.openstack.org/252458 | 13:30 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec in create volume calls to ovo https://review.openstack.org/246600 | 13:30 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO https://review.openstack.org/253033 | 13:30 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype) https://review.openstack.org/257305 | 13:30 |
*** mriedem has joined #openstack-cinder | 13:31 | |
*** timcl has joined #openstack-cinder | 13:31 | |
*** kongwei has joined #openstack-cinder | 13:31 | |
*** mtanino has quit IRC | 13:35 | |
*** dustins has quit IRC | 13:35 | |
*** kongwei has quit IRC | 13:39 | |
e0ne | zigo: ok. let me know please, if it helps you | 13:39 |
eantyshev | e0ne, jgriffith: Thank you for +2 and workflow on https://review.openstack.org/#/c/262246 . But it depends on a refactoring change: https://review.openstack.org/261558 (improvement in unitttests). could you please look at it? | 13:39 |
openstackgerrit | Stefan Amann proposed openstack/os-brick: Support FICON-attached ECKD volumes on System z https://review.openstack.org/320536 | 13:40 |
scottda | dulek: I'm thinking that we could take the same approach for multi-node gate testing as for multi-backend, i.e create an experimental job and try to get those project-config changes through. What do you think? | 13:40 |
*** dansmith is now known as superdan | 13:41 | |
*** cdelatte has joined #openstack-cinder | 13:42 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/329677 | 13:44 |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Correct DB modifications from compaction https://review.openstack.org/330756 | 13:44 |
*** porrua has joined #openstack-cinder | 13:46 | |
*** links has quit IRC | 13:46 | |
*** cdelatte has quit IRC | 13:49 | |
*** itzdilip has quit IRC | 13:49 | |
*** xinli has joined #openstack-cinder | 13:50 | |
*** ametts has joined #openstack-cinder | 13:50 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring https://review.openstack.org/285469 | 13:50 |
*** e0ne has quit IRC | 13:51 | |
*** e0ne has joined #openstack-cinder | 13:52 | |
*** sgotliv_ has joined #openstack-cinder | 13:57 | |
*** mriedem has quit IRC | 13:58 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file https://review.openstack.org/294076 | 13:58 |
xinli | geguileor: could you please help to review this patch 251989 (VersionObject) : https://review.openstack.org/#/c/251989/ | 13:58 |
*** mriedem has joined #openstack-cinder | 13:59 | |
dulek | xinli: geguileo is on vacations right now. I remember about your patch, just couldn't get to it lately due to DB migrations issues we've had. | 14:02 |
xinli | dulek: thank you very much for your response. I have resubmitted another patchset (22) due to code merging conflict. if you have time, please help to review the patch. | 14:05 |
*** porrua has quit IRC | 14:06 | |
*** cdelatte has joined #openstack-cinder | 14:06 | |
yuriy_n17 | DuncanT: Hi | 14:11 |
DuncanT | Hi | 14:11 |
yuriy_n17 | DuncanT: Very sorry for disturbing. Please take a look at https://review.openstack.org/#/c/279202/ whenever you have free time. | 14:12 |
*** sdake has joined #openstack-cinder | 14:12 | |
DuncanT | Certainly | 14:13 |
yuriy_n17 | DuncanT: Thanks. | 14:13 |
DuncanT | Ah yes, the API working group approved this, didn't they? | 14:13 |
yuriy_n17 | DuncanT: Sure, they approved and that patch was merged. | 14:14 |
*** cdelatte has quit IRC | 14:14 | |
DuncanT | Ok. I'll get to this one shortly, that was all I was waiting for, then forgot to follow up. | 14:14 |
*** nikeshm has quit IRC | 14:15 | |
*** zul has quit IRC | 14:16 | |
*** zul has joined #openstack-cinder | 14:16 | |
*** hemanthm|afk is now known as hemanthm | 14:20 | |
*** zul_ has joined #openstack-cinder | 14:21 | |
*** mtanino has joined #openstack-cinder | 14:21 | |
*** nikeshm has joined #openstack-cinder | 14:22 | |
*** zul has quit IRC | 14:23 | |
*** arecknag has quit IRC | 14:24 | |
*** cdelatte has joined #openstack-cinder | 14:31 | |
*** david-lyle has joined #openstack-cinder | 14:32 | |
*** e0ne has quit IRC | 14:34 | |
*** e0ne has joined #openstack-cinder | 14:35 | |
*** cdelatte has quit IRC | 14:37 | |
e0ne | eantyshev: did you investigate why quobyte and netapp ci failed on your patch? it doesn't look related to you changes | 14:41 |
*** ihrachys has quit IRC | 14:45 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Correct DB modifications from compaction https://review.openstack.org/330756 | 14:48 |
*** cdelatte has joined #openstack-cinder | 14:48 | |
*** krtaylor has joined #openstack-cinder | 14:53 | |
eantyshev | e0ne: honestly, no | 14:55 |
*** daneyon has joined #openstack-cinder | 14:55 | |
eantyshev | e0ne: quobyte CI, n-cpu.log: "Could not open '/opt/stack/data/nova/instances/8a395dbd-efa4-490d-b81d-52f0f3bc7e9f/console.log': Permission denied" | 14:56 |
eantyshev | e0ne: doesn't look like consequence of our refactoring | 14:57 |
*** laughterwym has joined #openstack-cinder | 15:00 | |
*** gouthamr has quit IRC | 15:01 | |
*** gouthamr has joined #openstack-cinder | 15:01 | |
*** merooney_ has joined #openstack-cinder | 15:01 | |
e0ne | eantyshev: I'm agree with you, just wanted to be sure | 15:02 |
*** diogogmt has joined #openstack-cinder | 15:02 | |
smcginnis | dulek: Thanks for all your help with checking on that patch! | 15:03 |
*** merooney has quit IRC | 15:04 | |
*** diogogmt has quit IRC | 15:04 | |
eantyshev | e0ne: Netapp CI also doesn't seem related: http://paste.openstack.org/show/516884 | 15:05 |
dulek | smcginnis: I'm happy we've got it right now (hopefully ;)). | 15:05 |
e0ne | eantyshev: thanks for the verification | 15:05 |
e0ne | eantyshev: I'll review your patch later today | 15:05 |
*** faiz89 has quit IRC | 15:05 | |
*** tesseract has quit IRC | 15:05 | |
*** rcernin has quit IRC | 15:07 | |
smcginnis | dulek: (fingers crossed) | 15:07 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/cinder: Fail to list backups of no backup services https://review.openstack.org/331217 | 15:07 |
*** faiz89 has joined #openstack-cinder | 15:08 | |
*** bjornar_ has quit IRC | 15:10 | |
*** faiz89 has quit IRC | 15:13 | |
*** jgregor has joined #openstack-cinder | 15:19 | |
*** kongwei has joined #openstack-cinder | 15:19 | |
*** sdake has quit IRC | 15:20 | |
*** jseiler__ has quit IRC | 15:21 | |
*** yangyapeng has joined #openstack-cinder | 15:22 | |
*** porrua has joined #openstack-cinder | 15:22 | |
*** yangyapeng has quit IRC | 15:22 | |
*** yangyapeng has joined #openstack-cinder | 15:22 | |
*** kongwei has quit IRC | 15:23 | |
*** cdelatte has quit IRC | 15:24 | |
*** afazekas|dentist is now known as afazekas | 15:25 | |
*** cdelatte has joined #openstack-cinder | 15:28 | |
*** timcl has quit IRC | 15:30 | |
*** sgotliv_ has quit IRC | 15:31 | |
*** pgadiya_ has quit IRC | 15:32 | |
*** rajinir has joined #openstack-cinder | 15:32 | |
*** yangyapeng has quit IRC | 15:34 | |
*** yuriy_n17 has quit IRC | 15:34 | |
*** mriedem has quit IRC | 15:35 | |
*** jdillaman has quit IRC | 15:37 | |
*** mriedem has joined #openstack-cinder | 15:38 | |
*** xinli has left #openstack-cinder | 15:39 | |
openstackgerrit | xiexs proposed openstack/cinder: Move _check_metadata_properties to cinder/utils https://review.openstack.org/330502 | 15:39 |
openstackgerrit | xiexs proposed openstack/cinder: Add validation for the metadata properties https://review.openstack.org/330913 | 15:39 |
*** dustins has joined #openstack-cinder | 15:39 | |
*** xinli has joined #openstack-cinder | 15:39 | |
*** belmoreira has quit IRC | 15:40 | |
*** arch-nemesis has joined #openstack-cinder | 15:42 | |
*** karthikp has joined #openstack-cinder | 15:50 | |
*** sdake has joined #openstack-cinder | 15:51 | |
*** yangyapeng has joined #openstack-cinder | 15:53 | |
*** karthikp has quit IRC | 15:54 | |
*** daneyon has quit IRC | 15:56 | |
*** Poornima_K has quit IRC | 15:57 | |
*** adisky has quit IRC | 16:00 | |
nikeshm | trying FC driver http://paste.openstack.org/show/516901/ http://paste.openstack.org/show/516903/ http://paste.openstack.org/show/516904/ attach volume is failing in initialize connection | 16:02 |
nikeshm | ProcessExecutionError: Unexpected error while running command.oslo.privsep.daemon privsep log: Command: /lib/udev/scsi_id --page 0x83 --whitelisted /dev/disk/by-path/pci-0000:03:00.1-fc-0x50024f40d6030101-lun-1 | 16:02 |
*** permalac has quit IRC | 16:03 | |
openstackgerrit | Merged openstack/cinder-specs: Deleting volume metadata keys with a single request https://review.openstack.org/279202 | 16:03 |
openstackgerrit | Merged openstack/cinder: Update manage_existing to use volume object https://review.openstack.org/258132 | 16:05 |
*** sdake has quit IRC | 16:07 | |
*** jungleboyj has joined #openstack-cinder | 16:09 | |
jungleboyj | smcginnis: Any reason to not merge this once CI passes? https://review.openstack.org/#/c/330756/ | 16:09 |
*** mdavidson has quit IRC | 16:11 | |
*** ivase has joined #openstack-cinder | 16:12 | |
*** timcl has joined #openstack-cinder | 16:15 | |
*** xinli has quit IRC | 16:15 | |
*** cknight has joined #openstack-cinder | 16:21 | |
*** takedakn has quit IRC | 16:25 | |
*** faiz89 has joined #openstack-cinder | 16:26 | |
*** e0ne has quit IRC | 16:27 | |
*** geguileo_phone has quit IRC | 16:27 | |
*** diablo_rojo has quit IRC | 16:28 | |
*** geguileo_phone has joined #openstack-cinder | 16:28 | |
*** chris_morrell has joined #openstack-cinder | 16:29 | |
*** mriedem is now known as mriedem_lunch | 16:30 | |
jgriffith | eharney: ping | 16:31 |
-openstackstatus- NOTICE: zuul was restarted for a software upgrade; events between 16:08 and 16:30 were missed, please recheck any changes uploaded during that time | 16:32 | |
*** ebalduf has joined #openstack-cinder | 16:33 | |
*** jungleboyj has quit IRC | 16:33 | |
adrianofr | smcginnis: ping | 16:34 |
*** Apoorva has joined #openstack-cinder | 16:36 | |
*** Apoorva has quit IRC | 16:37 | |
smcginnis | adrianofr: pong | 16:37 |
adrianofr | smcginnis: I didn't understand your comment regarding the decorator on https://review.openstack.org/#/c/285469/ | 16:38 |
adrianofr | smcginnis: it already has the decorator added by your patch | 16:38 |
adrianofr | smcginnis: anything else to be done? | 16:38 |
smcginnis | adrianofr: Oh, I could have sworn I looked and it didn't/ | 16:39 |
smcginnis | adrianofr: Sorry, I must have been looking at an older patch rev. | 16:39 |
smcginnis | adrianofr: Just ignore me. :) | 16:39 |
adrianofr | smcginnis: Np :P | 16:39 |
adrianofr | smcginnis: ops... :) | 16:40 |
smcginnis | adrianofr: ;) | 16:40 |
adrianofr | smcginnis: another question: a release note about a code refactoring should be described on "others" section? | 16:40 |
smcginnis | adrianofr: Oh, wait... | 16:42 |
smcginnis | adrianofr: Just ignore me completely. | 16:42 |
smcginnis | adrianofr: That patch was listed here as a new driver: https://etherpad.openstack.org/p/cinder-spec-review-tracking | 16:42 |
smcginnis | adrianofr: But it's not. | 16:42 |
smcginnis | adrianofr: So no need for a release note on that. | 16:42 |
*** karthikp has joined #openstack-cinder | 16:43 | |
claudiub | hemna: hello. can you take a look at the os-brick Windows iSCSI / fibre channel / SMB connectors? lpetrut has posted tempest test results for each of them: https://review.openstack.org/#/c/272522/29 | 16:44 |
*** karthikp_ has joined #openstack-cinder | 16:45 | |
*** karthikp_ has quit IRC | 16:45 | |
adrianofr | smcginnis: Oh, I see. We put it there because it's equivalent to a new driver review (trying to enforce cores to look at it) | 16:45 |
*** karthikp_ has joined #openstack-cinder | 16:45 | |
adrianofr | smcginnis: It was done during the meeting | 16:45 |
adrianofr | smcginnis: any problem with this? | 16:45 |
*** jungleboyj has joined #openstack-cinder | 16:47 | |
*** karthikp_ has quit IRC | 16:47 | |
*** karthikp has quit IRC | 16:47 | |
smcginnis | adrianofr: I'd rather just have new drivers listed there as they have a shorter timeframe. | 16:48 |
smcginnis | adrianofr: Updates to existing drivers have until n-3, but new drivers need to be added by n-2. | 16:49 |
*** bardia has joined #openstack-cinder | 16:49 | |
adrianofr | smcginnis: OK. I'm going to remove it from the list then. | 16:51 |
*** manjrem has joined #openstack-cinder | 16:51 | |
smcginnis | adrianofr: Thanks! | 16:52 |
smcginnis | adrianofr: I'll still try to review it and we can likely get it in earlier than that. | 16:53 |
*** harlowja has joined #openstack-cinder | 16:53 | |
adrianofr | smcginnis: Thanks :) | 16:56 |
openstackgerrit | Clinton Knight proposed openstack/cinder: Capability lists in Cinder scheduler https://review.openstack.org/331258 | 16:59 |
*** faiz89 has quit IRC | 17:01 | |
*** faiz89 has joined #openstack-cinder | 17:02 | |
*** Apoorva has joined #openstack-cinder | 17:02 | |
hemna | claudiub, sure | 17:02 |
openstackgerrit | Clinton Knight proposed openstack/cinder: Capability lists in Cinder scheduler https://review.openstack.org/331258 | 17:04 |
*** akerr has quit IRC | 17:05 | |
*** ccesario has quit IRC | 17:05 | |
*** sheel has quit IRC | 17:05 | |
*** xyang1 has quit IRC | 17:08 | |
*** xyang1 has joined #openstack-cinder | 17:08 | |
*** laughterwym has quit IRC | 17:08 | |
*** laughterwym has joined #openstack-cinder | 17:09 | |
*** mvk_ has joined #openstack-cinder | 17:11 | |
*** e0ne has joined #openstack-cinder | 17:12 | |
*** faiz891 has joined #openstack-cinder | 17:13 | |
*** faiz89 has quit IRC | 17:13 | |
eharney | jgriffith: pong | 17:13 |
*** laughterwym has quit IRC | 17:13 | |
jgriffith | eharney: hey... I was remembering a while back I think you cleaned up a bunch of the spewing of log info in the unit tests | 17:15 |
*** mkrcmari__ has quit IRC | 17:15 | |
*** GB21 has joined #openstack-cinder | 17:15 | |
jgriffith | eharney: mtreinish pointed out last night that cloudbyte is spewing info currently: http://logs.openstack.org/43/323343/4/check/gate-cinder-python27-db/80036ea/console.html#_2016-06-16_05_29_26_771535 | 17:15 |
jgriffith | eharney: thought I'd ping you and see if you remembered off the top of your head how to clean those things up :) | 17:16 |
jgriffith | eharney: or at least if you had some helpful tips for me :) | 17:16 |
mtreinish | jgriffith: there were other things leaking through to stdout, but I thought a stacktrace was more suspicious | 17:16 |
*** daneyon has joined #openstack-cinder | 17:16 | |
jgriffith | mtreinish: yeah, I noticed a few things. But in the case of cloudbyte it appears to be pretty much all of their logging | 17:17 |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring https://review.openstack.org/285469 | 17:17 |
jgriffith | mtreinish: that particular test is expecting exceptions... although not that particular one :) | 17:17 |
mtreinish | heh | 17:18 |
eharney | jgriffith: hmm, i'm not sure what that driver/test is doing differently from any others | 17:18 |
jgriffith | yeah, me neither | 17:19 |
*** daneyon_ has joined #openstack-cinder | 17:19 | |
*** ccesario has joined #openstack-cinder | 17:19 | |
eharney | jgriffith: oh, i see now -- it inherits from testtools directly instead of cinder's test.TestCase | 17:20 |
eharney | jgriffith: so it's not getting the right fixtures set up probably | 17:20 |
eharney | class CloudByteISCSIDriverTestCase(testtools.TestCase): | 17:20 |
*** faiz891 has left #openstack-cinder | 17:21 | |
*** faiz891 has joined #openstack-cinder | 17:21 | |
jgriffith | ahhh.. indeed | 17:21 |
*** daneyon has quit IRC | 17:22 | |
*** dustins has quit IRC | 17:23 | |
*** e0ne has quit IRC | 17:24 | |
*** dustins has joined #openstack-cinder | 17:24 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file https://review.openstack.org/294076 | 17:27 |
openstackgerrit | Merged openstack/cinder: Fix db purge for volume_types FK constraint https://review.openstack.org/330328 | 17:27 |
nikeshm | attach volume worked, i did mistake in initiator-target mapping | 17:30 |
nikeshm | one doubt, i saw multipath=False in connector properties, how can we make it True | 17:31 |
nikeshm | in initialize_connection | 17:31 |
nikeshm | for attach_volume | 17:31 |
hemna | nikeshm, that is set by 2 factors | 17:32 |
hemna | 1) if multipath is enabled in nova.conf [libvirt] section | 17:32 |
hemna | 2) multipath daemon is also running. | 17:33 |
hemna | either of those 2 aren't set, then multipath=False | 17:33 |
*** jungleboyj has quit IRC | 17:33 | |
nikeshm | hemna: is this is the option "iscsi_use_multipath" we have to use under [libvirt], what about fiber channel | 17:35 |
hemna | nikeshm, yes | 17:35 |
hemna | it's the same setting for iscsi and fc | 17:35 |
jgriffith | eharney: mtreinish that was it | 17:36 |
jgriffith | eharney: thanks for that | 17:36 |
jgriffith | eharney: mtreinish I'll post a patch for it | 17:36 |
nikeshm | hemna: thanks, shall we not use generic name for this option, may be use_multipath | 17:36 |
hemna | we'd have to change nova to do that | 17:37 |
hemna | and deprecate the existing and change the name | 17:37 |
hemna | meh | 17:37 |
openstackgerrit | Merged openstack/cinder: Add parameter info for manageable snapshots https://review.openstack.org/328668 | 17:37 |
hemna | but sure. I can put together a patch for that | 17:37 |
openstackgerrit | Merged openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs https://review.openstack.org/261549 | 17:37 |
nikeshm | hemna: thanks | 17:38 |
*** sdake has joined #openstack-cinder | 17:40 | |
openstackgerrit | Merged openstack/cinder: Use versionedobjects in remotefs.py https://review.openstack.org/261558 | 17:43 |
openstackgerrit | Merged openstack/cinder: vzstorage: add ploop volume format support https://review.openstack.org/262246 | 17:43 |
*** lpetrut has quit IRC | 17:43 | |
jgriffith | eharney: bahhh... although that exposed some other issues :( Like several hundred lock files littered about the working dir | 17:43 |
*** pauloewerton has quit IRC | 17:43 | |
openstackgerrit | Merged openstack/cinder: VMware:Persist storage policy ID in vmdk meta-data https://review.openstack.org/330911 | 17:44 |
*** GB21 has quit IRC | 17:45 | |
jgriffith | hmmmm | 17:48 |
jgriffith | anybody try running unit tests locally from master recently? | 17:48 |
smcginnis | Fairly recently. | 17:49 |
jgriffith | Oh... I bet I know what broke it | 17:49 |
*** dustins has quit IRC | 17:49 | |
jgriffith | smcginnis: so if you run currrent master you get a bunch of lock files | 17:49 |
jgriffith | I think it's that patch I just approved last night that replaces them with the tooz stuff | 17:49 |
smcginnis | jgriffith: Didn't we have that once before and fixed it? | 17:49 |
smcginnis | jgriffith: Ah, sounds like a likely culprit. | 17:49 |
*** pauloewerton has joined #openstack-cinder | 17:50 | |
*** kfarr has quit IRC | 17:50 | |
*** Suyash has joined #openstack-cinder | 17:50 | |
openstackgerrit | John Griffith proposed openstack/cinder: Revert "Replace locks in volume manager" https://review.openstack.org/331282 | 17:53 |
jgriffith | smcginnis: yeah | 17:53 |
*** manjrem has quit IRC | 17:57 | |
*** dustins has joined #openstack-cinder | 17:57 | |
*** timcl1 has joined #openstack-cinder | 17:57 | |
*** mriedem_lunch is now known as mriedem | 17:59 | |
*** timcl has quit IRC | 18:00 | |
*** harlowja has quit IRC | 18:04 | |
*** jungleboyj has joined #openstack-cinder | 18:07 | |
openstackgerrit | John Griffith proposed openstack/cinder: Use cinders test class in cloudbyte unit tests https://review.openstack.org/331287 | 18:08 |
jgriffith | patrickeast: hehe.. yeah... what's up with that! | 18:10 |
jgriffith | :) | 18:10 |
openstackgerrit | John Griffith proposed openstack/cinder: Revert "Replace locks in volume manager" https://review.openstack.org/331282 | 18:11 |
smcginnis | That's a little easier to read. :) | 18:11 |
patrickeast | haha | 18:11 |
hemna | oopsie | 18:12 |
patrickeast | i was tempted to just edit it in gerrit | 18:12 |
hemna | die gerrit die | 18:12 |
smcginnis | It's really too bad you can't update commit messages in gerrit without it needing to kick off new jobs. | 18:12 |
jgriffith | smcginnis: yeah... didn't the last version detect that? | 18:13 |
jgriffith | I could've sworn it did | 18:13 |
jgriffith | that and trivial rebase | 18:13 |
*** timcl1 has quit IRC | 18:13 | |
*** timcl has joined #openstack-cinder | 18:14 | |
smcginnis | Can't remember now. | 18:15 |
jgriffith | smcginnis: you're getting as bad as me :) | 18:15 |
smcginnis | I just miss the "f" shortcut key to switch between files while reviewing. | 18:15 |
smcginnis | jgriffith: Haha, I suppose. :) | 18:15 |
openstackgerrit | Tom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication https://review.openstack.org/329531 | 18:23 |
*** mkrcmari__ has joined #openstack-cinder | 18:25 | |
*** timcl has quit IRC | 18:26 | |
*** timcl has joined #openstack-cinder | 18:26 | |
*** ChubYann has joined #openstack-cinder | 18:27 | |
*** mvk_ has quit IRC | 18:28 | |
*** karthikp_ has joined #openstack-cinder | 18:40 | |
*** simondodsley has joined #openstack-cinder | 18:40 | |
*** dustins has quit IRC | 18:43 | |
*** timcl has quit IRC | 18:46 | |
*** porrua has quit IRC | 18:47 | |
*** timcl has joined #openstack-cinder | 18:48 | |
*** Lee1092 has quit IRC | 18:53 | |
*** locust has joined #openstack-cinder | 18:54 | |
*** locust has quit IRC | 19:00 | |
openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: [WIP] Switch service capabilities to ovo https://review.openstack.org/319040 | 19:00 |
*** e0ne has joined #openstack-cinder | 19:08 | |
*** dustins has joined #openstack-cinder | 19:08 | |
*** karthikp has joined #openstack-cinder | 19:10 | |
*** geguileo_phone has quit IRC | 19:12 | |
*** karthikp has quit IRC | 19:12 | |
*** shakamunyi has joined #openstack-cinder | 19:13 | |
*** karthikp has joined #openstack-cinder | 19:15 | |
*** faiz891 has quit IRC | 19:22 | |
*** faiz89 has joined #openstack-cinder | 19:22 | |
*** geguileo_phone has joined #openstack-cinder | 19:25 | |
*** dustins has quit IRC | 19:26 | |
*** harlowja has joined #openstack-cinder | 19:32 | |
*** gouthamr has quit IRC | 19:32 | |
*** ociuhandu has quit IRC | 19:34 | |
bluex | jgriffith: it seems volume manager locks patch was reverted but it's not yet visible in repo, should I start new patch replacing locks or create simple 2 line fix for current implementation? | 19:37 |
jgriffith | bluex: hey | 19:38 |
jgriffith | bluex: hasn't merged yet | 19:38 |
jgriffith | if you have a fix that's that easy post it | 19:38 |
jgriffith | bluex: I can revert my revert :) | 19:38 |
jgriffith | bluex: make sense? | 19:40 |
bluex | sure, I'll send fix, give me 10 min | 19:40 |
*** merooney_ has quit IRC | 19:40 | |
*** cdelatte has quit IRC | 19:40 | |
jgriffith | bluex: awesome | 19:41 |
*** rajinir has quit IRC | 19:44 | |
*** mriedem has quit IRC | 19:46 | |
openstackgerrit | Alyson proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages https://review.openstack.org/310785 | 19:47 |
smcginnis | Good, I was hoping that could just get fixed rather than starting over. | 19:48 |
*** pauloewerton has quit IRC | 19:49 | |
*** Apoorva has quit IRC | 19:49 | |
*** lpetrut has joined #openstack-cinder | 19:52 | |
*** mriedem has joined #openstack-cinder | 19:53 | |
*** e0ne has quit IRC | 19:53 | |
*** timcl has quit IRC | 19:54 | |
openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Mock tooz coordinator for tests https://review.openstack.org/331325 | 19:54 |
*** markvoelker has quit IRC | 19:55 | |
jgriffith | bluex: smcginnis cool, looks like that should do the trick! | 19:56 |
jgriffith | hmm... test fail | 19:57 |
*** raildo is now known as raildo-afk | 19:57 | |
jgriffith | bluex: oh... think we need to remove the mocks you had in test_volumes.py | 19:57 |
openstackgerrit | Alyson proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages https://review.openstack.org/310785 | 20:01 |
*** dustins has joined #openstack-cinder | 20:03 | |
jgriffith | bluex: hmm, seems to be some side efffects... let me checkout a fresh env | 20:04 |
bluex | hmm, tests passed locally, I have to check again, test_volumes mock only cinder.coordination.Coordinator.get_lock and fix I posted mock tooz.coordination.get_coordinator which should override any reference to tooz used by cinder.coordination | 20:05 |
jgriffith | bluex: let me make sure it's not just something on my side that's messed up | 20:05 |
*** daneyon_ has quit IRC | 20:06 | |
jgriffith | bluex: http://logs.openstack.org/25/331325/1/check/gate-cinder-python27-db/35b85ad/console.html#_2016-06-17_20_03_14_616073 | 20:09 |
*** markvoelker has joined #openstack-cinder | 20:10 | |
*** daneyon has joined #openstack-cinder | 20:10 | |
jgriffith | bluex: oh... actually probably failing because of the mock | 20:12 |
*** bardia has quit IRC | 20:12 | |
*** karthikp has quit IRC | 20:13 | |
jgriffith | bluex: so those two tests actually check to make sure creates fail because the lock is set | 20:13 |
*** bardia has joined #openstack-cinder | 20:14 | |
jgriffith | so the mock kinda screws those up | 20:14 |
*** karthikp has joined #openstack-cinder | 20:14 | |
*** gouthamr has joined #openstack-cinder | 20:14 | |
bluex | yeah, I mocked whole locking behavior too, locks are not created so files are not created and nothing is really locked | 20:15 |
*** akshai has quit IRC | 20:18 | |
*** Apoorva has joined #openstack-cinder | 20:18 | |
jgriffith | bluex: right, so those that were actually testing the locks get a bit grumpy :) | 20:19 |
Swanson | Jenkins should just fix errors like W391 blank line at end of file | 20:19 |
*** karthikp has quit IRC | 20:20 | |
*** karthikp has joined #openstack-cinder | 20:22 | |
openstackgerrit | Kendall Nelson proposed openstack/cinder: Add VolumeAttachStatus Enum https://review.openstack.org/290885 | 20:22 |
nikeshm | smcginnis: https://review.openstack.org/#/c/327285/ is not showing in gate | 20:26 |
nikeshm | even after reverify | 20:27 |
nikeshm | it went in check pipeline and passed | 20:27 |
nikeshm | but not going in gate pipeline | 20:27 |
nikeshm | is any issue in gate | 20:27 |
smcginnis | nikeshm: Hmm. And it's not dependent on another patch? | 20:27 |
nikeshm | it is independent | 20:29 |
*** karthikp has quit IRC | 20:29 | |
openstackgerrit | Merged openstack/cinder: Correct DB modifications from compaction https://review.openstack.org/330756 | 20:31 |
openstackgerrit | Merged openstack/cinder: Use cinders test class in cloudbyte unit tests https://review.openstack.org/331287 | 20:31 |
*** karthikp has joined #openstack-cinder | 20:32 | |
*** dustins has quit IRC | 20:33 | |
nikeshm | smcginnis: it is not dependent on any patch | 20:33 |
*** mvk_ has joined #openstack-cinder | 20:34 | |
*** karthikp has quit IRC | 20:37 | |
*** mkrcmari__ has quit IRC | 20:38 | |
nikeshm | smcginis xyang: i am going to sleep, please check when you get time that why it is not entering in gate pipeline | 20:41 |
smcginnis | nikeshm: It's going now. | 20:42 |
smcginnis | nikeshm: Should be merged by the time you wake up. ;) | 20:42 |
nikeshm | what was issue | 20:42 |
nikeshm | good night | 20:44 |
smcginnis | Gerrit irregularity. | 20:45 |
*** csky has quit IRC | 20:46 | |
*** jungleboyj has quit IRC | 20:50 | |
*** crose has joined #openstack-cinder | 20:52 | |
*** mkrcmari__ has joined #openstack-cinder | 20:54 | |
*** crose has quit IRC | 20:57 | |
*** faiz89 has quit IRC | 20:57 | |
*** mvk_ has quit IRC | 20:57 | |
hemna | die gerrit die | 20:58 |
*** karthikp has joined #openstack-cinder | 21:00 | |
*** ivase has quit IRC | 21:01 | |
*** mvk has joined #openstack-cinder | 21:01 | |
*** mkrcmari__ has quit IRC | 21:02 | |
openstackgerrit | Tom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication https://review.openstack.org/329531 | 21:02 |
*** karthikp has quit IRC | 21:06 | |
*** karthikp has joined #openstack-cinder | 21:09 | |
*** cknight has quit IRC | 21:10 | |
*** ametts has quit IRC | 21:13 | |
bluex | jgriffith: I have two solutions - question is whether we want to use threading.Lock on top of tooz locks - if yes I'll change implementation of locks in cinder.coordination if not I'll just mock tooz get_lock for tests to return named threading.Lock | 21:21 |
*** markvoelker has quit IRC | 21:21 | |
*** gouthamr has quit IRC | 21:22 | |
jgriffith | bluex: well... you're the tooz impl expert at this point :) | 21:24 |
jgriffith | bluex: I can't say as I understand the pros/cons of the coordination impl | 21:25 |
jgriffith | bluex: it does seem that not using threading.lock causes some weirdness for us | 21:25 |
*** karthikp_ has quit IRC | 21:25 | |
*** dave-mccowan has quit IRC | 21:26 | |
bluex | Previous implementation used threading.lock on top of file locks, current implementation uses only tooz locks, tests assumed only local (threading) locks are used so no files was created | 21:28 |
*** lascii is now known as alaski | 21:28 | |
bluex | I remember some previous implementation of coordination module also used threading.lock, but I have to check again what was the reason to not use that in final implementation | 21:29 |
*** erlon has quit IRC | 21:36 | |
bluex | Ok, as I see it - additional threading.lock would be useful only when connection with coordination service is dropped, can't be restored and we would want node to continue locked operation, but as we don't want this situation to ever happen - it wouldn't add anything | 21:43 |
bluex | And with this I think I answered my question :) | 21:43 |
*** xyang1 has quit IRC | 21:48 | |
jgriffith | bluex: :) | 21:50 |
openstackgerrit | Vasanthi Thirumalai proposed openstack/cinder: Violin Memory ISCSI storage for 7000 series AFA https://review.openstack.org/269796 | 21:52 |
*** alyson_ has quit IRC | 21:58 | |
openstackgerrit | Merged openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver https://review.openstack.org/327285 | 21:58 |
*** mriedem has quit IRC | 21:59 | |
*** lpetrut has quit IRC | 22:04 | |
*** karthikp has quit IRC | 22:06 | |
openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Mock tooz coordinator for tests https://review.openstack.org/331325 | 22:11 |
openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Mock tooz coordinator for tests https://review.openstack.org/331325 | 22:22 |
*** tpsilva has quit IRC | 22:27 | |
openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Fix lock files littering working dir during tests https://review.openstack.org/331325 | 22:33 |
*** sdake has quit IRC | 22:38 | |
*** mvk_ has joined #openstack-cinder | 22:39 | |
*** sdake has joined #openstack-cinder | 22:41 | |
*** adrianofr has quit IRC | 22:42 | |
*** mvk has quit IRC | 22:42 | |
*** dustins has joined #openstack-cinder | 22:58 | |
*** edmondsw has quit IRC | 22:59 | |
jgriffith | bluex: thanks! Seems to be a problem with py34 maybe though | 23:09 |
jgriffith | but at least the other problems appear to be fixed up | 23:09 |
*** IlyaG has joined #openstack-cinder | 23:13 | |
*** markvoelker has joined #openstack-cinder | 23:16 | |
openstackgerrit | Rodrigo Freire proposed openstack/os-brick: Multipath: fix race when detaching devices https://review.openstack.org/331375 | 23:18 |
*** dave-mccowan has joined #openstack-cinder | 23:22 | |
*** karthikp has joined #openstack-cinder | 23:25 | |
*** dave-mccowan has quit IRC | 23:26 | |
*** dustins has quit IRC | 23:26 | |
*** karthikp has quit IRC | 23:30 | |
*** karthikp has joined #openstack-cinder | 23:31 | |
*** claudiub has quit IRC | 23:38 | |
*** eharney has quit IRC | 23:46 | |
*** karthikp has quit IRC | 23:47 | |
*** karthikp has joined #openstack-cinder | 23:48 | |
*** asselin_ has quit IRC | 23:49 | |
*** Suyash has quit IRC | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!