| *** salv-orlando has joined #openstack-cinder | 00:01 | |
| *** dannywilson has quit IRC | 00:02 | |
| *** openstack has joined #openstack-cinder | 00:04 | |
| *** IanGovett has joined #openstack-cinder | 00:04 | |
| *** markvoelker has quit IRC | 00:04 | |
| *** vokt has quit IRC | 00:04 | |
| *** salv-orlando has quit IRC | 00:06 | |
| *** timcl has quit IRC | 00:09 | |
| *** tsekiyam_ has joined #openstack-cinder | 00:11 | |
| *** boris-42 has quit IRC | 00:12 | |
| *** akerr is now known as akerr_away | 00:12 | |
| *** sghanekar_ has joined #openstack-cinder | 00:13 | |
| *** vilobhmm has quit IRC | 00:14 | |
| *** vilobhmm has joined #openstack-cinder | 00:14 | |
| *** gouthamr_ has joined #openstack-cinder | 00:14 | |
| *** tsekiyama has quit IRC | 00:15 | |
| *** mtanino has quit IRC | 00:15 | |
| *** tsekiyam_ has quit IRC | 00:16 | |
| *** gouthamr has quit IRC | 00:17 | |
| *** rushil has joined #openstack-cinder | 00:17 | |
| *** dalgaaf has quit IRC | 00:18 | |
| *** heyun has joined #openstack-cinder | 00:23 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Fix 3PAR driver handling of existing VLUNs https://review.openstack.org/202354 | 00:25 |
|---|---|---|
| *** garthb has quit IRC | 00:29 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Fix 3PAR driver handling of existing VLUNs https://review.openstack.org/202354 | 00:32 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Fix 3PAR driver handling of existing VLUNs https://review.openstack.org/202354 | 00:34 |
| *** barra204 has quit IRC | 00:39 | |
| *** akerr_away is now known as akerr | 00:39 | |
| *** leeantho has quit IRC | 00:39 | |
| *** gouthamr_ has quit IRC | 00:40 | |
| *** gouthamr has joined #openstack-cinder | 00:41 | |
| *** cdelatte has quit IRC | 00:41 | |
| *** laughterwym has joined #openstack-cinder | 00:45 | |
| *** diogogmt has quit IRC | 00:46 | |
| *** akerr has quit IRC | 00:48 | |
| *** diogogmt has joined #openstack-cinder | 00:49 | |
| *** topshare has joined #openstack-cinder | 00:51 | |
| *** earlephilhower has quit IRC | 00:55 | |
| *** skylerberg has joined #openstack-cinder | 00:59 | |
| *** gouthamr_ has joined #openstack-cinder | 01:03 | |
| *** gouthamr has quit IRC | 01:04 | |
| *** liuxg has joined #openstack-cinder | 01:04 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: Add class for dog command executor https://review.openstack.org/194532 | 01:05 |
| *** jasondotstar has quit IRC | 01:08 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:09 | |
| *** IanGovett has quit IRC | 01:12 | |
| *** _cjones_ has quit IRC | 01:12 | |
| *** hemna has joined #openstack-cinder | 01:14 | |
| *** gouthamr_ has quit IRC | 01:18 | |
| *** Apoorva has quit IRC | 01:20 | |
| *** hemna has quit IRC | 01:21 | |
| *** sghanekar_ has quit IRC | 01:26 | |
| *** hemna has joined #openstack-cinder | 01:27 | |
| *** breitz has quit IRC | 01:30 | |
| *** breitz has joined #openstack-cinder | 01:30 | |
| *** Lee1092 has joined #openstack-cinder | 01:31 | |
| *** afazekas has quit IRC | 01:34 | |
| *** patrickeast has quit IRC | 01:38 | |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Show backup and volume info in backup_restore https://review.openstack.org/199846 | 01:44 |
| *** jdandrea has quit IRC | 01:46 | |
| *** liuxg has quit IRC | 01:48 | |
| *** tobe_ has joined #openstack-cinder | 01:49 | |
| *** vilobhmm has quit IRC | 01:57 | |
| *** HoloIRCUser2 has joined #openstack-cinder | 01:57 | |
| *** HoloIRCUser5 has joined #openstack-cinder | 02:02 | |
| *** HoloIRCUser2 has quit IRC | 02:02 | |
| *** HoloIRCUser2 has joined #openstack-cinder | 02:03 | |
| *** HoloIRCUser5 has quit IRC | 02:03 | |
| *** HoloIRCUser2 is now known as liuxg | 02:03 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Continue conversion to volume object https://review.openstack.org/201404 | 02:07 |
| *** jungleboyj has joined #openstack-cinder | 02:13 | |
| *** piyanai has joined #openstack-cinder | 02:19 | |
| *** dims_ has quit IRC | 02:31 | |
| *** zhenguo has joined #openstack-cinder | 02:32 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: improve create and delete operation https://review.openstack.org/197393 | 02:33 |
| *** chlong-afk has quit IRC | 02:37 | |
| *** chlong has joined #openstack-cinder | 02:42 | |
| *** lcurtis has joined #openstack-cinder | 02:46 | |
| openstackgerrit | Tom Jose Kalapura proposed openstack/python-cinderclient: Add response message when volume delete https://review.openstack.org/202856 | 02:46 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 02:52 |
| *** hakimo has joined #openstack-cinder | 02:53 | |
| *** hakimo_ has quit IRC | 02:55 | |
| *** piyanai has quit IRC | 02:55 | |
| *** hakimo_ has joined #openstack-cinder | 03:10 | |
| *** hakimo has quit IRC | 03:10 | |
| openstackgerrit | Yaguang Tang proposed openstack/cinder: Remove simple scheduler which is deprecated since Juno https://review.openstack.org/202858 | 03:13 |
| *** boris-42 has joined #openstack-cinder | 03:14 | |
| *** julim has quit IRC | 03:18 | |
| *** topshare has quit IRC | 03:27 | |
| *** hakimo has joined #openstack-cinder | 03:27 | |
| *** hakimo_ has quit IRC | 03:27 | |
| *** lcurtis has quit IRC | 03:31 | |
| *** dims_ has joined #openstack-cinder | 03:32 | |
| *** topshare has joined #openstack-cinder | 03:32 | |
| openstackgerrit | Seiji Aguchi proposed openstack/cinder: Remove resource lock operation for HBSD https://review.openstack.org/185003 | 03:34 |
| *** dims_ has quit IRC | 03:37 | |
| *** rushil has quit IRC | 03:38 | |
| *** annegentle has joined #openstack-cinder | 03:38 | |
| *** hakimo_ has joined #openstack-cinder | 03:44 | |
| *** hakimo has quit IRC | 03:44 | |
| openstackgerrit | Lei Li proposed openstack/cinder: Adjust to right exception for chunkeddriver https://review.openstack.org/202481 | 03:45 |
| *** mriedem1 has quit IRC | 03:46 | |
| *** HoloIRCUser1 has joined #openstack-cinder | 03:50 | |
| *** liuxg has quit IRC | 03:51 | |
| *** HoloIRCUser1 has quit IRC | 03:54 | |
| *** coolsvap|away is now known as coolsvap | 03:54 | |
| *** xyang has quit IRC | 03:55 | |
| *** hemna has quit IRC | 03:58 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 03:59 |
| *** erlon has quit IRC | 04:00 | |
| *** hakimo has joined #openstack-cinder | 04:01 | |
| *** hakimo_ has quit IRC | 04:01 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add hypermetro support for huawei volume driver https://review.openstack.org/202023 | 04:01 |
| *** hemna has joined #openstack-cinder | 04:08 | |
| *** hemna has quit IRC | 04:11 | |
| *** annegentle has quit IRC | 04:11 | |
| *** hemna has joined #openstack-cinder | 04:12 | |
| *** hakimo_ has joined #openstack-cinder | 04:18 | |
| *** hakimo has quit IRC | 04:18 | |
| *** HoloIRCUser2 has joined #openstack-cinder | 04:22 | |
| *** abhinavsgsits has joined #openstack-cinder | 04:23 | |
| *** HoloIRCUser3 has joined #openstack-cinder | 04:23 | |
| *** abhinavsgsits is now known as aagrawal | 04:24 | |
| *** lixiaoy1 has quit IRC | 04:26 | |
| *** HoloIRCUser2 has quit IRC | 04:27 | |
| HoloIRCUser3 | . | 04:27 |
| *** HoloIRCUser3 is now known as liuxg | 04:28 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: improve snapshot and clone operation https://review.openstack.org/198952 | 04:33 |
| *** harlowja_at_home has joined #openstack-cinder | 04:35 | |
| *** skylerberg has quit IRC | 04:38 | |
| openstackgerrit | wanghao proposed openstack/cinder: Support volume backup of qcow2 format volume https://review.openstack.org/182276 | 04:41 |
| *** haomaiw__ has joined #openstack-cinder | 04:42 | |
| *** zhenguo has quit IRC | 04:43 | |
| *** haomaiwa_ has quit IRC | 04:44 | |
| *** hemna has joined #openstack-cinder | 04:48 | |
| *** hemna has quit IRC | 04:49 | |
| *** hemna has joined #openstack-cinder | 04:52 | |
| *** tobe_ has quit IRC | 04:53 | |
| *** aagrawal has quit IRC | 04:58 | |
| *** shyama has joined #openstack-cinder | 04:58 | |
| *** aagrawal has joined #openstack-cinder | 04:58 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 04:58 |
| *** liuxg has quit IRC | 04:59 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 05:01 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 05:02 |
| *** dannywilson has joined #openstack-cinder | 05:06 | |
| *** lixiaoy1 has joined #openstack-cinder | 05:09 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 05:10 |
| *** boris-42_ has joined #openstack-cinder | 05:20 | |
| *** kaisers2 has joined #openstack-cinder | 05:21 | |
| *** ameade_ has joined #openstack-cinder | 05:21 | |
| *** jungleboyj_ has joined #openstack-cinder | 05:22 | |
| *** boris-42 has quit IRC | 05:23 | |
| *** kaisers1 has quit IRC | 05:23 | |
| *** jbernard has quit IRC | 05:23 | |
| *** ameade has quit IRC | 05:23 | |
| *** ondergetekende has quit IRC | 05:23 | |
| *** jungleboyj has quit IRC | 05:23 | |
| *** ameade_ is now known as ameade | 05:24 | |
| *** ondergetekende has joined #openstack-cinder | 05:24 | |
| *** boris-42_ is now known as boris-42 | 05:24 | |
| *** jbernard has joined #openstack-cinder | 05:24 | |
| *** heyun has quit IRC | 05:25 | |
| *** heyun has joined #openstack-cinder | 05:28 | |
| *** lpetrut has joined #openstack-cinder | 05:28 | |
| *** topshare has quit IRC | 05:28 | |
| *** vilobhmm has joined #openstack-cinder | 05:29 | |
| *** lpetrut has quit IRC | 05:34 | |
| *** dannywilson has quit IRC | 05:35 | |
| *** dannywilson has joined #openstack-cinder | 05:36 | |
| *** s0nea has quit IRC | 05:37 | |
| *** BharatK has joined #openstack-cinder | 05:38 | |
| *** s0nea has joined #openstack-cinder | 05:40 | |
| *** markvoelker has joined #openstack-cinder | 05:41 | |
| BharatK | thingee, Need your blessing for the patch https://review.openstack.org/#/c/163647/ | 05:41 |
| *** markvoelker_ has joined #openstack-cinder | 05:44 | |
| *** markvoelker has quit IRC | 05:45 | |
| *** tobe_ has joined #openstack-cinder | 05:46 | |
| *** topshare has joined #openstack-cinder | 05:46 | |
| *** heyun has quit IRC | 05:48 | |
| *** topshare has quit IRC | 05:48 | |
| *** heyun has joined #openstack-cinder | 05:50 | |
| *** annashen has joined #openstack-cinder | 05:51 | |
| *** topshare has joined #openstack-cinder | 05:54 | |
| *** deepakcs has joined #openstack-cinder | 05:56 | |
| *** dims_ has joined #openstack-cinder | 06:00 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: improve snapshot and clone operation https://review.openstack.org/198952 | 06:05 |
| *** dims_ has quit IRC | 06:06 | |
| *** ig0r_ has joined #openstack-cinder | 06:08 | |
| *** harlowja_at_home has quit IRC | 06:09 | |
| *** agrawala has joined #openstack-cinder | 06:13 | |
| *** ronis has joined #openstack-cinder | 06:13 | |
| openstackgerrit | Gaurang Tapase proposed openstack/cinder: Adding NFS support to the GPFS Driver https://review.openstack.org/197884 | 06:14 |
| *** aagrawal has quit IRC | 06:15 | |
| *** tobe_ has quit IRC | 06:16 | |
| *** dannywil_ has joined #openstack-cinder | 06:24 | |
| *** dannywil_ has quit IRC | 06:25 | |
| *** dannywil_ has joined #openstack-cinder | 06:26 | |
| *** agrawala is now known as aagrawal | 06:26 | |
| *** dannywilson has quit IRC | 06:27 | |
| *** gaurangt has joined #openstack-cinder | 06:30 | |
| *** tobe_ has joined #openstack-cinder | 06:31 | |
| *** ronis has quit IRC | 06:35 | |
| *** aagrawal has quit IRC | 06:36 | |
| *** aagrawal has joined #openstack-cinder | 06:36 | |
| *** hemna has quit IRC | 06:36 | |
| *** ndipanov has quit IRC | 06:42 | |
| *** ndipanov has joined #openstack-cinder | 06:43 | |
| *** ig0r_ has quit IRC | 06:45 | |
| *** ig0r_ has joined #openstack-cinder | 06:51 | |
| *** belmoreira has joined #openstack-cinder | 06:51 | |
| openstackgerrit | Lei Li proposed openstack/cinder: Adjust to right exception for chunkeddriver https://review.openstack.org/202481 | 07:00 |
| *** s0nea has quit IRC | 07:00 | |
| *** topshare has quit IRC | 07:02 | |
| *** s0nea has joined #openstack-cinder | 07:02 | |
| *** sgordon has quit IRC | 07:03 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 07:06 |
| *** afazekas has joined #openstack-cinder | 07:06 | |
| *** topshare has joined #openstack-cinder | 07:06 | |
| *** vilobhmm has quit IRC | 07:07 | |
| *** markus_z has joined #openstack-cinder | 07:10 | |
| *** boris-42 has quit IRC | 07:12 | |
| *** aagrawal has quit IRC | 07:12 | |
| *** aagrawal has joined #openstack-cinder | 07:12 | |
| *** aagrawal has quit IRC | 07:19 | |
| *** aagrawal has joined #openstack-cinder | 07:19 | |
| *** heyun1 has joined #openstack-cinder | 07:25 | |
| *** sgordon has joined #openstack-cinder | 07:25 | |
| *** heyun has quit IRC | 07:26 | |
| *** abehl has joined #openstack-cinder | 07:27 | |
| *** ronis has joined #openstack-cinder | 07:32 | |
| *** jordanP has joined #openstack-cinder | 07:38 | |
| *** salv-orlando has joined #openstack-cinder | 07:39 | |
| *** abhishekk has joined #openstack-cinder | 07:42 | |
| *** dannywil_ has quit IRC | 07:49 | |
| *** dims_ has joined #openstack-cinder | 07:49 | |
| *** coolsvap is now known as coolsvap|afk | 07:51 | |
| openstackgerrit | Li Yingjun proposed openstack/cinder: Set default policy for "volume:get" https://review.openstack.org/202913 | 07:54 |
| openstackgerrit | Lei Zhang proposed openstack/cinder: Make concurrent backups for same volume impossible https://review.openstack.org/202914 | 07:55 |
| *** dims_ has quit IRC | 07:56 | |
| *** kaisers2 has quit IRC | 07:59 | |
| *** annashen has quit IRC | 08:05 | |
| *** HoloIRCUser3 has joined #openstack-cinder | 08:07 | |
| *** HoloIRCUser3 has quit IRC | 08:11 | |
| *** ronis_ has joined #openstack-cinder | 08:12 | |
| *** ronis has quit IRC | 08:12 | |
| *** jistr has joined #openstack-cinder | 08:15 | |
| *** chenying has joined #openstack-cinder | 08:16 | |
| *** salv-orlando has quit IRC | 08:17 | |
| *** p_deore has joined #openstack-cinder | 08:21 | |
| *** laughterwym has quit IRC | 08:22 | |
| *** aagrawal has quit IRC | 08:26 | |
| *** aagrawal has joined #openstack-cinder | 08:27 | |
| *** agarciam has quit IRC | 08:33 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Validate name and description string https://review.openstack.org/182994 | 08:37 |
| *** markvoelker_ has quit IRC | 08:42 | |
| *** lpetrut has joined #openstack-cinder | 08:42 | |
| *** lpetrut has quit IRC | 08:43 | |
| *** ociuhandu has joined #openstack-cinder | 08:43 | |
| *** aagrawal has quit IRC | 08:51 | |
| *** aagrawal has joined #openstack-cinder | 08:51 | |
| *** coolsvap|afk is now known as coolsvap | 08:54 | |
| *** markus_z has quit IRC | 08:56 | |
| *** aagrawal has quit IRC | 08:56 | |
| *** aagrawal has joined #openstack-cinder | 08:56 | |
| *** ociuhandu has quit IRC | 08:56 | |
| *** markvoelker has joined #openstack-cinder | 08:57 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 08:57 |
| *** sgotliv has quit IRC | 08:57 | |
| *** lpetrut has joined #openstack-cinder | 09:00 | |
| *** lpetrut has quit IRC | 09:00 | |
| openstackgerrit | wanghao proposed openstack/cinder: Support volume backup of qcow2 format volume https://review.openstack.org/182276 | 09:00 |
| *** e0ne has joined #openstack-cinder | 09:01 | |
| *** markvoelker has quit IRC | 09:02 | |
| theanalyst | Can any of the cores please review the bp: https://blueprints.launchpad.net/cinder/+spec/backup-restore-display-name | 09:03 |
| theanalyst | DuncanT: ^^ | 09:04 |
| *** deepakcs has quit IRC | 09:04 | |
| *** aix has joined #openstack-cinder | 09:07 | |
| *** Maike has joined #openstack-cinder | 09:08 | |
| *** lpetrut has joined #openstack-cinder | 09:10 | |
| *** markvoelker has joined #openstack-cinder | 09:12 | |
| *** markus_z has joined #openstack-cinder | 09:13 | |
| *** dims_ has joined #openstack-cinder | 09:15 | |
| *** lpetrut has quit IRC | 09:16 | |
| *** markvoelker has quit IRC | 09:17 | |
| *** agarciam has joined #openstack-cinder | 09:18 | |
| *** dims_ has quit IRC | 09:20 | |
| openstackgerrit | TaoBai proposed openstack/cinder: ==============WIP============== IBM Storwize Volume-replication V2 implementation https://review.openstack.org/202940 | 09:22 |
| openstackgerrit | TaoBai proposed openstack/cinder: ==============WIP============== https://review.openstack.org/202940 | 09:22 |
| openstackgerrit | TaoBai proposed openstack/cinder: ==============WIP============== https://review.openstack.org/202940 | 09:23 |
| *** tobe_ has quit IRC | 09:25 | |
| *** markvoelker has joined #openstack-cinder | 09:26 | |
| *** topshare has quit IRC | 09:29 | |
| *** markvoelker has quit IRC | 09:31 | |
| *** rhagarty has quit IRC | 09:40 | |
| *** rhagarty_ has quit IRC | 09:40 | |
| *** sseverson has quit IRC | 09:40 | |
| *** cbader has quit IRC | 09:40 | |
| *** asselin_ has joined #openstack-cinder | 09:40 | |
| *** sseverson has joined #openstack-cinder | 09:40 | |
| *** asselin has quit IRC | 09:40 | |
| *** cbader has joined #openstack-cinder | 09:40 | |
| *** rhagarty has joined #openstack-cinder | 09:40 | |
| *** rhagarty_ has joined #openstack-cinder | 09:40 | |
| *** markvoelker has joined #openstack-cinder | 09:41 | |
| *** markvoelker has quit IRC | 09:45 | |
| *** lixiaoy1 has quit IRC | 09:50 | |
| openstackgerrit | Abhishek Lekshmanan proposed openstack/cinder: Backups: allow name to be specified during restore https://review.openstack.org/189357 | 09:51 |
| openstackgerrit | Jordan Pittier proposed openstack/cinder: Rework the Scality SOFS driver to use the RemoteFS class https://review.openstack.org/200610 | 09:53 |
| *** piyanai has joined #openstack-cinder | 09:55 | |
| *** markvoelker has joined #openstack-cinder | 09:55 | |
| *** ankit has joined #openstack-cinder | 09:57 | |
| *** ankit has quit IRC | 09:58 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: improve image operations https://review.openstack.org/202954 | 09:59 |
| *** ankit has joined #openstack-cinder | 09:59 | |
| *** ankit has quit IRC | 09:59 | |
| *** markvoelker has quit IRC | 10:00 | |
| *** ankit_ag has joined #openstack-cinder | 10:00 | |
| *** chenying has quit IRC | 10:03 | |
| *** dims_ has joined #openstack-cinder | 10:05 | |
| *** deepakcs has joined #openstack-cinder | 10:07 | |
| *** markvoelker has joined #openstack-cinder | 10:07 | |
| *** heyun1 has quit IRC | 10:10 | |
| *** markvoelker has quit IRC | 10:12 | |
| *** IanGovett has joined #openstack-cinder | 10:15 | |
| *** mdenny has quit IRC | 10:20 | |
| *** jasondotstar has joined #openstack-cinder | 10:21 | |
| *** markvoelker has joined #openstack-cinder | 10:22 | |
| *** openstackgerrit has quit IRC | 10:31 | |
| *** markvoelker has quit IRC | 10:32 | |
| *** openstackgerrit has joined #openstack-cinder | 10:32 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter cgsnapshots data on the DB side https://review.openstack.org/197163 | 10:32 |
| *** yuriy_n17 has joined #openstack-cinder | 10:35 | |
| *** markvoelker has joined #openstack-cinder | 10:37 | |
| *** aix has quit IRC | 10:41 | |
| *** markvoelker has quit IRC | 10:42 | |
| *** gaurangt has left #openstack-cinder | 10:45 | |
| *** aix has joined #openstack-cinder | 10:45 | |
| *** markvoelker has joined #openstack-cinder | 10:51 | |
| *** markvoelker has quit IRC | 10:56 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 10:56 |
| *** vlaza has joined #openstack-cinder | 11:03 | |
| *** s0nea has quit IRC | 11:04 | |
| *** dims_ has quit IRC | 11:06 | |
| *** markvoelker has joined #openstack-cinder | 11:06 | |
| *** cdelatte has joined #openstack-cinder | 11:07 | |
| *** s0nea has joined #openstack-cinder | 11:07 | |
| *** HoloIRCUser3 has joined #openstack-cinder | 11:08 | |
| *** kcaj has joined #openstack-cinder | 11:09 | |
| *** markvoelker has quit IRC | 11:11 | |
| *** coolsvap is now known as coolsvap|away | 11:14 | |
| *** piyanai has quit IRC | 11:19 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 11:19 |
| *** markvoelker has joined #openstack-cinder | 11:19 | |
| *** ganso_ has joined #openstack-cinder | 11:19 | |
| *** piyanai has joined #openstack-cinder | 11:20 | |
| *** piyanai has quit IRC | 11:22 | |
| *** markvoelker has quit IRC | 11:24 | |
| *** lcurtis has joined #openstack-cinder | 11:26 | |
| *** markvoelker has joined #openstack-cinder | 11:32 | |
| *** jordanP has quit IRC | 11:35 | |
| *** jordan__ has joined #openstack-cinder | 11:35 | |
| *** gouthamr has joined #openstack-cinder | 11:42 | |
| *** gouthamr has quit IRC | 11:42 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 11:43 |
| *** gouthamr has joined #openstack-cinder | 11:43 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 11:43 |
| *** markvoelker has quit IRC | 11:44 | |
| *** HoloIRCUser2 has joined #openstack-cinder | 11:52 | |
| *** lcurtis has quit IRC | 11:53 | |
| *** HoloIRCUser3 has quit IRC | 11:54 | |
| *** markvoelker has joined #openstack-cinder | 11:55 | |
| *** HoloIRCUser3 has joined #openstack-cinder | 11:57 | |
| *** HoloIRCUser2 has quit IRC | 11:57 | |
| *** markus_z has quit IRC | 11:57 | |
| *** markvoelker has quit IRC | 12:00 | |
| *** nkrinner_afk is now known as nkrinner | 12:05 | |
| *** gouthamr has quit IRC | 12:06 | |
| *** jasondotstar has quit IRC | 12:06 | |
| *** porrua has joined #openstack-cinder | 12:08 | |
| *** markvoelker has joined #openstack-cinder | 12:09 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 12:10 |
| *** kjelly_ has joined #openstack-cinder | 12:13 | |
| *** markvoelker has quit IRC | 12:13 | |
| *** markvoelker has joined #openstack-cinder | 12:16 | |
| *** markvoelker has quit IRC | 12:21 | |
| *** jasondotstar has joined #openstack-cinder | 12:21 | |
| *** asselin has joined #openstack-cinder | 12:23 | |
| *** edmondsw has joined #openstack-cinder | 12:23 | |
| *** markvoelker has joined #openstack-cinder | 12:24 | |
| *** dims_ has joined #openstack-cinder | 12:25 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: Cleanup for SnapshotObject https://review.openstack.org/198341 | 12:25 |
| *** lpetrut has joined #openstack-cinder | 12:28 | |
| *** BharatK has quit IRC | 12:30 | |
| *** aagrawal has quit IRC | 12:32 | |
| *** markvoelker has quit IRC | 12:32 | |
| *** aagrawal has joined #openstack-cinder | 12:32 | |
| *** deepakcs has quit IRC | 12:34 | |
| *** akerr has joined #openstack-cinder | 12:35 | |
| *** ankit_ag has quit IRC | 12:35 | |
| *** ankit_ag has joined #openstack-cinder | 12:35 | |
| *** kjnelson has joined #openstack-cinder | 12:36 | |
| *** coolsvap|away is now known as coolsvap | 12:37 | |
| *** porrua has left #openstack-cinder | 12:38 | |
| *** markvoelker has joined #openstack-cinder | 12:39 | |
| *** piyanai has joined #openstack-cinder | 12:40 | |
| *** piyanai has quit IRC | 12:40 | |
| *** eharney has quit IRC | 12:41 | |
| *** markvoelker has quit IRC | 12:43 | |
| *** HoloIRCUser7 has joined #openstack-cinder | 12:44 | |
| *** lpetrut has quit IRC | 12:46 | |
| *** abhishekk has quit IRC | 12:46 | |
| *** HoloIRCUser3 has quit IRC | 12:47 | |
| *** piyanai has joined #openstack-cinder | 12:51 | |
| *** eharney has joined #openstack-cinder | 12:51 | |
| *** julim has joined #openstack-cinder | 12:52 | |
| *** julim has quit IRC | 12:53 | |
| *** julim has joined #openstack-cinder | 12:53 | |
| *** markvoelker has joined #openstack-cinder | 12:53 | |
| *** dims_ has quit IRC | 12:54 | |
| *** Tross1 has joined #openstack-cinder | 12:54 | |
| *** Tross has quit IRC | 12:55 | |
| *** nkrinner has quit IRC | 12:55 | |
| *** piyanai has quit IRC | 12:55 | |
| *** markvoelker has quit IRC | 12:58 | |
| *** ankit_ag has quit IRC | 12:59 | |
| *** dustins has joined #openstack-cinder | 12:59 | |
| *** aagrawal has quit IRC | 13:00 | |
| *** piyanai has joined #openstack-cinder | 13:00 | |
| *** aagrawal has joined #openstack-cinder | 13:00 | |
| *** piyanai has quit IRC | 13:00 | |
| *** dalgaaf has joined #openstack-cinder | 13:03 | |
| *** Maike has quit IRC | 13:05 | |
| *** strictlyb has quit IRC | 13:05 | |
| *** markvoelker has joined #openstack-cinder | 13:05 | |
| *** piyanai has joined #openstack-cinder | 13:06 | |
| *** markvoelker_ has joined #openstack-cinder | 13:07 | |
| *** markvoelker has quit IRC | 13:09 | |
| *** bnemec has joined #openstack-cinder | 13:11 | |
| *** jasondotstar has quit IRC | 13:11 | |
| *** p_deore has quit IRC | 13:13 | |
| *** xyang1 has joined #openstack-cinder | 13:18 | |
| *** mriedem has joined #openstack-cinder | 13:24 | |
| mriedem | thingee: does os-brick 0.3.0 come out today? | 13:25 |
| openstackgerrit | Stefan Amann proposed openstack/os-brick: Perform port_rescan on s390x platforms https://review.openstack.org/203026 | 13:25 |
| *** primechuck has joined #openstack-cinder | 13:26 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add drivers list generator https://review.openstack.org/192751 | 13:30 |
| *** dansmith is now known as superdan | 13:30 | |
| *** kambiz has quit IRC | 13:31 | |
| *** BharatK has joined #openstack-cinder | 13:32 | |
| BharatK | DuncanT, Modified the patch as per your comments https://review.openstack.org/#/c/163647/ please review it. | 13:33 |
| *** crose has joined #openstack-cinder | 13:34 | |
| *** markvoelker_ has quit IRC | 13:34 | |
| *** Yogi1 has joined #openstack-cinder | 13:35 | |
| *** wyatt88 has quit IRC | 13:35 | |
| *** gouthamr has joined #openstack-cinder | 13:36 | |
| mriedem | is cinder have unit test failures with mock 1.1.4 this morning? | 13:37 |
| mriedem | *having | 13:37 |
| mriedem | ironic and nova are | 13:37 |
| *** merooney has joined #openstack-cinder | 13:37 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add drivers list generator https://review.openstack.org/192751 | 13:37 |
| mriedem | wondering about capping mock in g-r | 13:37 |
| *** anteaya has quit IRC | 13:38 | |
| *** jungleboyj_ has quit IRC | 13:40 | |
| *** kambiz has joined #openstack-cinder | 13:41 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder-specs: Add Cinder Volume A/A HA spec https://review.openstack.org/202615 | 13:42 |
| *** anteaya has joined #openstack-cinder | 13:42 | |
| *** hakimo has joined #openstack-cinder | 13:43 | |
| *** hakimo_ has quit IRC | 13:43 | |
| *** dims_ has joined #openstack-cinder | 13:45 | |
| *** diogogmt has quit IRC | 13:47 | |
| *** sgotliv has joined #openstack-cinder | 13:49 | |
| *** markvoelker has joined #openstack-cinder | 13:49 | |
| *** markvoelker has quit IRC | 13:54 | |
| *** aagrawal has quit IRC | 13:56 | |
| *** jdandrea has joined #openstack-cinder | 13:56 | |
| *** sgotliv has quit IRC | 13:57 | |
| *** merooney has quit IRC | 13:59 | |
| *** BharatK has left #openstack-cinder | 13:59 | |
| *** BharatK has quit IRC | 13:59 | |
| *** markvoelker has joined #openstack-cinder | 13:59 | |
| aarefiev | smcginnis: hi, thanks for review drivers list generator | 13:59 |
| aarefiev | smcginnis: could you also take a look at https://review.openstack.org/#/c/139071/ , please. it blocks bug fix https://review.openstack.org/#/c/156939/ | 14:01 |
| smcginnis | aarefiev: Sure, I'll get them in my queue. Thanks for your work on that. | 14:02 |
| *** aarefiev_phone has joined #openstack-cinder | 14:04 | |
| *** julim_ has joined #openstack-cinder | 14:05 | |
| *** eharney has quit IRC | 14:06 | |
| *** mdenny has joined #openstack-cinder | 14:06 | |
| *** markvoelker has quit IRC | 14:07 | |
| *** julim has quit IRC | 14:08 | |
| *** crose has quit IRC | 14:09 | |
| *** hakimo has quit IRC | 14:11 | |
| *** hakimo_ has joined #openstack-cinder | 14:11 | |
| *** abehl has quit IRC | 14:12 | |
| *** Tross1 has quit IRC | 14:14 | |
| *** markvoelker has joined #openstack-cinder | 14:14 | |
| *** Yogi1 has quit IRC | 14:15 | |
| *** markvoelker has quit IRC | 14:18 | |
| *** merooney has joined #openstack-cinder | 14:20 | |
| *** Yogi1 has joined #openstack-cinder | 14:21 | |
| e0ne | thingee, jgriffith: hi. could you please merge https://review.openstack.org/#/c/203023/? we've got broken unit tests for python-cinderclient in stable/kilo | 14:23 |
| *** mylu has joined #openstack-cinder | 14:24 | |
| *** vlaza has quit IRC | 14:26 | |
| *** erlon has joined #openstack-cinder | 14:26 | |
| jgriffith | e0ne: won't help... without a client release because the tests in gate don't use git, they use pip | 14:27 |
| jgriffith | e0ne: well, it'll help.. but you know what I mean | 14:28 |
| jgriffith | e0ne: We had planned to fix these via requirements update and pinning | 14:28 |
| e0ne | jgriffith: thanks. | 14:28 |
| jgriffith | e0ne: which I thought was "done" | 14:28 |
| *** markvoelker has joined #openstack-cinder | 14:28 | |
| e0ne | jgriffith: oh... it was my second question | 14:29 |
| jgriffith | e0ne: The update finally hit cinder, but not minderclient it would seem | 14:29 |
| jgriffith | e0ne: I tried to submit it last friday, everybody let it sit, then conveniently said on Tuesday when the automated job hit "Why are you doing this, it's already here" sigh | 14:30 |
| e0ne | jgriffith: i'm wondering how it works: https://review.openstack.org/#/c/173977/5/test-requirements.txt | 14:30 |
| *** akshai has joined #openstack-cinder | 14:30 | |
| e0ne | jgriffith: it does'nt update requirements to the latest stable! | 14:30 |
| *** breitz has quit IRC | 14:30 | |
| jgriffith | e0ne: we just need that to land and you should be good | 14:30 |
| e0ne | jgriffith: we'd hit this https://bugs.launchpad.net/python-cinderclient/+bug/1475654 issue today:( | 14:30 |
| openstack | Launchpad bug 1475654 in python-cinderclient "incorrect verstion for hacking package on stable/kilo" [Undecided,New] - Assigned to Yuriy Nesenenko (ynesenenko) | 14:30 |
| jgriffith | DuncanT: don't yell at me for pushing that through | 14:31 |
| jgriffith | https://review.openstack.org/#/c/173977/ | 14:31 |
| *** eharney has joined #openstack-cinder | 14:32 | |
| e0ne | jgriffith: now, i'm confused more | 14:32 |
| *** topshare has joined #openstack-cinder | 14:32 | |
| jgriffith | e0ne: how come? | 14:32 |
| *** topshare has quit IRC | 14:32 | |
| jgriffith | e0ne: that patch from proposal bot will fix the issue | 14:32 |
| e0ne | jgriffith: please, look at hacking version | 14:33 |
| *** markvoelker has quit IRC | 14:33 | |
| e0ne | from the my bug's description: python-cinderclient should follow global requirements but no we have hacking>=0.10.0,<0.11 in [1] and hacking>=0.8.0,<0.9 in [2] in stable/kilo | 14:33 |
| *** xyang has joined #openstack-cinder | 14:33 | |
| *** topshare has joined #openstack-cinder | 14:33 | |
| jgriffith | e0ne: not sure I follow the concern? | 14:34 |
| *** breitz has joined #openstack-cinder | 14:34 | |
| jgriffith | e0ne: in stable kilo: https://review.openstack.org/#/c/173977/5/test-requirements.txt | 14:34 |
| jgriffith | ignore master for a second | 14:34 |
| jgriffith | the global req update just fixes the syntax on hacking, doesn't change versions | 14:35 |
| e0ne | jgriffith: but global reqs have other version: https://github.com/openstack/requirements/blob/stable/kilo/global-requirements.txt#L217 | 14:35 |
| jgriffith | e0ne: ahh | 14:35 |
| jgriffith | e0ne: well that update just hasn't made it's way through to us yet | 14:36 |
| e0ne | jgriffith: and my patch fixes unit tests! | 14:36 |
| e0ne | jgriffith: it's the second problem with stable/kilo:( | 14:36 |
| *** piyanai has quit IRC | 14:36 | |
| e0ne | jgriffith: e.g. https://review.openstack.org/202952 | 14:37 |
| jgriffith | e0ne: right, but what I was getting at is I think the update of mock is going to fix the unit tests also | 14:37 |
| *** tsekiyama has joined #openstack-cinder | 14:37 | |
| e0ne | jgriffith: oh.. now i understood it | 14:37 |
| jgriffith | e0ne: and it doesn't fix them anyway according to gerrit :) | 14:37 |
| *** ronis_ has quit IRC | 14:38 | |
| jgriffith | e0ne: wait.. never mind... you've got all kinds of crazy going on | 14:38 |
| e0ne | jgriffith: :) | 14:38 |
| jgriffith | e0ne: my point was just that once this lands: https://review.openstack.org/#/c/173977/5, the issues you are addressing in your patch that changes up unit tests should be addressed | 14:39 |
| e0ne | jgriffith: now i'm agree with you | 14:39 |
| jgriffith | e0ne: "resolved", "fixed", "no longer fail" | 14:39 |
| *** harlowja_at_home has joined #openstack-cinder | 14:39 | |
| jgriffith | e0ne: I was working on these last Friday | 14:39 |
| jgriffith | e0ne: there was a collective agreement among projects and the oslo folks that the best course for stable was to just pin the reqs, rather than update tests | 14:40 |
| e0ne | jgriffith: it's a dependency hell | 14:40 |
| jgriffith | e0ne: it is!!! | 14:40 |
| e0ne | jgriffith: so... what do you thinks about hacking version? | 14:41 |
| jgriffith | e0ne: I think everybody in OpenStack became "library happy" over the last couple years, and it's actually not a good thing | 14:41 |
| jgriffith | e0ne: so I'm not sure about that... it looks like Robert's update to that just landed on the 15'th | 14:41 |
| jgriffith | e0ne: oh.. wait, it's in the queue I think | 14:42 |
| jgriffith | hmm.. not that one :) | 14:42 |
| e0ne | jgriffith: hacking (0.10.0) was landed on January, 7th | 14:43 |
| e0ne | it's a crazy deps friday! | 14:43 |
| *** mgarza_ has joined #openstack-cinder | 14:43 | |
| *** lcurtis has joined #openstack-cinder | 14:43 | |
| *** krtaylor has quit IRC | 14:45 | |
| e0ne | jgriffith: fyi. fix hacking https://review.openstack.org/#/c/203054/ | 14:45 |
| jgriffith | e0ne: hmmm | 14:45 |
| e0ne | jgriffith: the same issue is in ironic too | 14:45 |
| jgriffith | e0ne: frankly minderclient has become so F'd up I don't even know what to think about it or do any more | 14:45 |
| jgriffith | e0ne: cinderclient even | 14:45 |
| *** hemna has joined #openstack-cinder | 14:46 | |
| e0ne | jgriffith: maybe we need to fix it in global reqs | 14:48 |
| *** aarefiev_phone has quit IRC | 14:49 | |
| jgriffith | e0ne: ? | 14:49 |
| e0ne | jgriffith: i mean set older version of hacking for stable/kilo | 14:49 |
| *** mtanino has joined #openstack-cinder | 14:49 | |
| *** markvoelker has joined #openstack-cinder | 14:50 | |
| *** hemna has quit IRC | 14:52 | |
| jgriffith | e0ne: do you have contact with Yuriy? | 14:53 |
| jgriffith | e0ne: https://review.openstack.org/#/c/203054/ | 14:53 |
| jgriffith | e0ne: it's not quite right, but I don't want to wait until tomorrow to see an update and get things rolling | 14:53 |
| jgriffith | e0ne: happy to update myself, but don't want to do that if he's around and able to work on it himself | 14:54 |
| *** piyanai has joined #openstack-cinder | 14:54 | |
| aarefiev | smcginnis: thank you | 14:55 |
| *** cbader_ has joined #openstack-cinder | 14:56 | |
| *** markvoelker has quit IRC | 14:57 | |
| *** markvoelker_ has joined #openstack-cinder | 14:57 | |
| *** markvoelker_ has quit IRC | 14:57 | |
| *** markvoelker has joined #openstack-cinder | 14:57 | |
| *** cbader_ has quit IRC | 14:57 | |
| *** krtaylor has joined #openstack-cinder | 14:57 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 15:00 |
| *** jaypipes has joined #openstack-cinder | 15:01 | |
| xyang | jgriffith, e0ne, DuncanT, smcginnis, tbarron: question regarding https://review.openstack.org/#/c/193937/ | 15:01 |
| jgriffith | xyang: what's up? | 15:01 |
| xyang | I want to introduce a new status 'backing-up-in-use' for backing up an attached volume, any objections? | 15:01 |
| xyang | the reason if that I need to know the original status when cleaning up failed backups if services restarts | 15:02 |
| tbarron | xyang: this is for the cleanup, right? | 15:02 |
| xyang | yes | 15:02 |
| jgriffith | xyang: I suppose not, but it would've been cool if we did sub-states | 15:02 |
| jgriffith | xyang: I'm certainly not asking you to do that by the way :) | 15:02 |
| geguileo | xyang: I'm not too happy about it | 15:02 |
| tbarron | xyang: I think the cleanup is already unsafe if more than one backup process is running | 15:02 |
| xyang | jgriffith: we don't have that yet, right? seems to be abandoned | 15:02 |
| jgriffith | xyang: yeah :( | 15:02 |
| tbarron | xyang: I'm not sure about how we should be doing cleanup | 15:03 |
| tbarron | xyang: the current way just looks for leftover junk and assumes that no-one else is running | 15:03 |
| xyang | there is existing code that does clean up and assumes the orig status is available | 15:03 |
| geguileo | tbarron: Because right now that's true | 15:03 |
| xyang | now we allow in-use volume to be backed up | 15:03 |
| tbarron | xyang: so I'm not objecting to what you are proposing. I jst don't know the best way forward yet. | 15:03 |
| geguileo | tbarron: We don't support Active-Active | 15:03 |
| xyang | I need to know the original status | 15:03 |
| tbarron | geguileo: I understand. | 15:04 |
| xyang | it is not saved any where if the service restarts | 15:04 |
| geguileo | xyang: Why not store it in the temporary snapshot metadata? | 15:04 |
| tbarron | geguileo: xyang: I'm just wondering about the best way forward. | 15:04 |
| xyang | it could fail before the temp snap is created | 15:04 |
| geguileo | tbarron: Ok, I'm not happy about that new state XD | 15:04 |
| xyang | geguileo: if you have a better idea, I'm happy to learn it | 15:05 |
| geguileo | xyang: That way you don't need a new state | 15:05 |
| jgriffith | xyang: geguileo what if we just add a "previous-state" column to the DB | 15:05 |
| tbarron | geguileo: I just don't want to say "yeah that's cool" in off-tne-cuff IRC | 15:05 |
| jgriffith | and don't mess with the craziness in the states so much | 15:05 |
| geguileo | xyang: Mmm, I just said my not so good idea | 15:05 |
| xyang | jgriffith: that was another way | 15:05 |
| tbarron | xyang: ^^^ not trying to hold you up, but I think it needs more pondering | 15:05 |
| xyang | jgriffith: do we want to introduce another column | 15:05 |
| geguileo | xyang: Why not store it in the temporary snapshot metadata? | 15:05 |
| jgriffith | tbarron: no pondering... and answer at this point | 15:05 |
| tbarron | jgriffith: :-) | 15:06 |
| xyang | geguileo: the temp snap may not exist yet | 15:06 |
| jgriffith | s/and answer/just an answer/ | 15:06 |
| xyang | if the temp snap exists, I know for sure that is for in-use | 15:06 |
| jgriffith | geguileo: xyang the "previous state" idea has other uses, I'd prefer it I think | 15:06 |
| geguileo | xyang: You should create the temp snap in the API before you change the volume status | 15:06 |
| xyang | geguileo: I don't think so | 15:07 |
| xyang | that should be created later | 15:07 |
| geguileo | xyang: I mean the DB entry | 15:07 |
| geguileo | xyang: Not the actual snapshot | 15:07 |
| xyang | it could be either a temp snap or volume | 15:07 |
| xyang | depends on the driver | 15:07 |
| xyang | for LVM it is temp snap | 15:07 |
| xyang | for driver.py, it is a volume | 15:07 |
| xyang | https://review.openstack.org/#/c/193937/ | 15:08 |
| jgriffith | geguileo: xyang yeah, so that's one thing for me... I'd use volume-meta and do the same thing, but to be clear some won't use snapshots for this | 15:08 |
| xyang | for LVM we can read the local path of a snapshot | 15:08 |
| xyang | but not for other drivers | 15:08 |
| geguileo | xyang: I'd have to think about it, but I'm not too keen on adding that new state you mentioned | 15:09 |
| xyang | either a new state for in-use, or a new column, | 15:09 |
| xyang | since we don't know whether it is a temp volume or temp snapshot in api, I don't think we can create that early | 15:10 |
| geguileo | New column makes more sense to me, but I think we should give it a little thought | 15:10 |
| tbarron | +1 | 15:10 |
| geguileo | As not to make it too specific | 15:10 |
| xyang | if there's more use for the new column, I'm all for it | 15:11 |
| geguileo | And maybe give it more versability for other uses/cases | 15:11 |
| geguileo | I think we should give it a thought of possible uses | 15:11 |
| geguileo | I'll add it to my "I have to think about this" list :-) | 15:11 |
| jgriffith | geguileo: what's specific about "previous_state" ? | 15:12 |
| jgriffith | geguileo: I don't know how you get much "less" specific than that? | 15:12 |
| geguileo | XD | 15:12 |
| tbarron | playing my hand, I want to re-think the way we cleanup failed backups anyways since it hampers backup scale out | 15:12 |
| jgriffith | geguileo: I don't know what XD means :) | 15:13 |
| * jgriffith is lame | 15:13 | |
| xyang | may be we just need to pick a name that everyone likes | 15:13 |
| xyang | voting time:) | 15:13 |
| geguileo | It's a smily for laughing | 15:13 |
| geguileo | Like lol | 15:13 |
| tbarron | so I'm reluctant to drive new state or even new column for that problem alone as currently implemented | 15:13 |
| jgriffith | geguileo: Ohhh!! :) | 15:13 |
| jgriffith | geguileo: thanks!! | 15:13 |
| jgriffith | LOL | 15:13 |
| jgriffith | Now I have a new shorthand I can use... I like it | 15:13 |
| tbarron | XD | 15:13 |
| xyang | tbarron: I don't want to go that route either | 15:13 |
| jgriffith | XD | 15:13 |
| xyang | tbarron: just don't know what is the alternative | 15:14 |
| tbarron | this is hilarious | 15:14 |
| e0ne | jgriffith: was away. thanks for review | 15:14 |
| xyang | that's why I brought it up here | 15:14 |
| geguileo | xyang: What I meant was that the new column proposal is ok | 15:14 |
| tbarron | xyang: me either, but I want to get you smart folks thinking about it | 15:14 |
| geguileo | But maybe we want a list of states in there | 15:14 |
| tbarron | geguileo: +1 | 15:14 |
| jgriffith | xyang: tbarron how about I just submit a patch to add the column right now... then xyang can rebase against it :) | 15:14 |
| geguileo | Or maybe something different | 15:14 |
| jgriffith | errr... XD | 15:14 |
| geguileo | That's why I mean we should think about it | 15:15 |
| jgriffith | geguileo: so I've proposed that before FWIW | 15:15 |
| tbarron | jgriffith: it may well be the right thing to do, but if so, for more than just that one problem | 15:15 |
| jgriffith | geguileo: it didn't go over well, and frankly that's how we ended up with tflow | 15:15 |
| xyang | jgriffith: fine with me. what does your "errr" mean? | 15:15 |
| jgriffith | geguileo: I've always though a simple queue impl would be MUCH better for state management than anything we do today | 15:15 |
| jgriffith | but that's a whole different bike-shedding event XD | 15:15 |
| geguileo | jgriffith: A queue of jobs you mean? | 15:16 |
| jgriffith | geguileo: That was one part of the proposal... and a stack of states as an object progresses through it's jobs | 15:16 |
| geguileo | xyang: I think we can go ahead with the new column for now and if I suddenly think that we are missing something I'll ping you | 15:17 |
| jgriffith | but I digress... xyang is trying to finish here patch so I don't want to derail | 15:17 |
| jgriffith | geguileo: we can change it to a list as well later I think without much trouble | 15:17 |
| jgriffith | geguileo: if that proves to have benefit (which I think it might) | 15:17 |
| geguileo | jgriffith: The list thing was just an example | 15:17 |
| jgriffith | geguileo: well, I know... but I liked it because it's something I've played with before | 15:18 |
| jgriffith | geguileo: I wrote a prototype that stored state changes for the cycle of an operation in order in DB.. then I could do unwinding, recovery etc | 15:18 |
| jgriffith | geguileo: but it was less interesting that I thought it would be because even though I had all the info on the transition states, the actual unwinding and retry stuff was such a PITA it wasn't worth it | 15:19 |
| geguileo | jgriffith: That's sounds like an interesting idea, and it could also serve as traceability of which user did what | 15:19 |
| jgriffith | I'm of the opinion that you should instead 'fail fast' and 'start over' | 15:19 |
| jgriffith | geguileo: yeah, I didn't think of the other more useful bits until you guys started talking about it here | 15:20 |
| geguileo | jgriffith: Yep, it's usually better to just start from scratch when you fail | 15:20 |
| *** dannywilson has joined #openstack-cinder | 15:20 | |
| geguileo | jgriffith: Than actually trying to figure out in which specific state you are and what you need to do to fix it | 15:20 |
| jgriffith | geguileo: especially in a Cloud... but sadly most people developing against OpenStack these days don't seem to have a "cloudy" mentality. | 15:20 |
| geguileo | XD | 15:21 |
| geguileo | Anyway, we digress, the column idea seem fine for now. :-) | 15:21 |
| xyang | jgriffith: so are you going to add the column or I add the column in my patch? | 15:21 |
| jgriffith | geguileo: I'm also a little tired of hearing that "things always fail" because in reality, most OpenStack deployments I've worked with are MUCH more robust than people would lead you to believe | 15:21 |
| jgriffith | xyang: geguileo I'll let you two decide and delegate to me if you'd like :) | 15:21 |
| tbarron | I'd like to see the column patch and understand it better | 15:22 |
| *** diogogmt has joined #openstack-cinder | 15:22 | |
| xyang | tbarron: you want to see a separate patch for column? | 15:22 |
| geguileo | jgriffith: That's true, but we still have more failures than I would like to have :-) | 15:22 |
| tbarron | xyang: that is up to you and john | 15:22 |
| geguileo | By the way, anybody knows what is going on with the gates? | 15:22 |
| tbarron | xyang: jgriffith: I just want to see code to understand better | 15:23 |
| jgriffith | geguileo: indeed, but most aren't doing anything to address them :( | 15:23 |
| xyang | tbarron: jgriffith says it is up to geguileo and me:) So I'll add it to my patch | 15:23 |
| jgriffith | geguileo: don't take that wrong... it's certainly not directed at you... and YES it's noted the improvements you've added | 15:23 |
| jgriffith | xyang: cool! | 15:23 |
| *** coolsvap is now known as coolsvap|away | 15:23 | |
| tbarron | xyang: ++ | 15:23 |
| xyang | thanks! | 15:23 |
| geguileo | xyang: Ok | 15:23 |
| jgriffith | geguileo: you've found and fixed a good deal of issues | 15:24 |
| geguileo | jgriffith: Don't worry, I took your statement as it is, a generalization which in some part also relates to me :-) | 15:24 |
| *** pots_ has joined #openstack-cinder | 15:24 | |
| xyang | name for the column again? orig_status, previous_status? | 15:24 |
| geguileo | xyang: +1 to previous_status | 15:24 |
| jgriffith | geguileo: an to myself as well | 15:25 |
| xyang | any other votes? | 15:25 |
| geguileo | jgriffith: xyang Any idea what is going on with the 2 broken gates? (python27 and python34) | 15:25 |
| jgriffith | geguileo: example? | 15:26 |
| jgriffith | geguileo: afraid I'm not as involved in monitoring things as of late | 15:26 |
| geguileo | jgriffith: I've reviewed now like 5 patches with the same gate issue | 15:26 |
| geguileo | jgriffith: Oh, I'm not monitoring per se | 15:26 |
| xyang | what is the error | 15:26 |
| e0ne | jgriffith: please, post a link to review request | 15:27 |
| jgriffith | geguileo: well... now, but you've "noticed" them, I haven't really looked much :( | 15:27 |
| geguileo | http://logs.openstack.org/57/189357/5/check/gate-cinder-python27/1ad86d5/console.html#_2015-07-17_12_20_46_834 | 15:27 |
| geguileo | Those are the 2 erros I've seen repeatedly in the last 5 reviews I've done | 15:27 |
| *** jaypipes is now known as blockedpipes | 15:27 | |
| jgriffith | awww rap | 15:27 |
| jgriffith | crap even | 15:27 |
| geguileo | Anyway, if you are asking me about the error it means you don't know about it | 15:28 |
| *** annashen has joined #openstack-cinder | 15:28 | |
| geguileo | And probably nobody is working on it yet | 15:28 |
| *** kjelly_ has quit IRC | 15:28 | |
| jgriffith | geguileo: Oh.. I might know... just a sec | 15:28 |
| geguileo | Great! | 15:28 |
| openstackgerrit | John Griffith proposed openstack/cinder: Backups: allow name to be specified during restore https://review.openstack.org/189357 | 15:28 |
| Swanson | xyang: can you take a look at this one again? https://review.openstack.org/#/c/201685/ I coded it in a way that works but perhaps that is an unexpected use case? | 15:29 |
| jgriffith | geguileo: see what that does for us | 15:29 |
| geguileo | jgriffith: Ok, I'll monitor that one and see how it goes | 15:29 |
| geguileo | jgriffith: Thanks | 15:30 |
| *** pots has joined #openstack-cinder | 15:30 | |
| xyang | Swanson: sure | 15:30 |
| jgriffith | geguileo: I suspect they just need rebase. Given it was started in early June, I've noticed most folks don't rebase as they go along. Just a hunch, without looking closely at the logs and such | 15:30 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient https://review.openstack.org/197643 | 15:31 |
| *** daneyon has joined #openstack-cinder | 15:31 | |
| jgriffith | geguileo: but given I've merged at least 3 changes in the last couple weeks that break the world seems like a good place to start :) | 15:31 |
| geguileo | jgriffith: I think some of the patches had been rebased | 15:31 |
| geguileo | jgriffith: XD | 15:31 |
| jgriffith | geguileo: yeah, but I like to do that first... and at least eliminate it | 15:31 |
| geguileo | jgriffith: Yes, and let's hope it's just that | 15:31 |
| *** jdurgin has joined #openstack-cinder | 15:31 | |
| jgriffith | geguileo: indeed, those are nice ones to deal with :) | 15:32 |
| *** dannywilson has quit IRC | 15:34 | |
| *** annashen has quit IRC | 15:35 | |
| xyang | Swanson: so I can see how you get cgsnapshot_id in snapshot, but it is still different from snapshot_id. you always has a snapshot_id | 15:36 |
| Swanson | xyang: yes, but if the snapshot was created via a cgsnapshot that snapshot id is meaningless to me. The only way I can find the snapshot in question is to look for the cgsnapshot id. | 15:37 |
| *** Tross has joined #openstack-cinder | 15:38 | |
| *** mylu has quit IRC | 15:38 | |
| xyang | Swanson: so you are not associating snapshot_id with anything in your backend if it is part of a cgsnapshot? | 15:38 |
| *** mylu has joined #openstack-cinder | 15:39 | |
| Swanson | Can't. | 15:39 |
| xyang | Swanson: how do you know which snapshot to delete then? you still need to do that | 15:39 |
| xyang | Swanson: do you mean that is only an entry in cinder db | 15:39 |
| Swanson | xyang: Via cgsnapshot_delete or snapshot_delete? | 15:39 |
| *** earlephilhower has joined #openstack-cinder | 15:40 | |
| xyang | Swanson: when you do cgsnapshot_delete, you still need to call snapshot_delete for each one, so sounds like you are saying that is just db entry | 15:40 |
| Swanson | xyang: When I create a consistency group snapshot on my backend I only have 1 id to work with. It is associated as a group AND with each volume. | 15:40 |
| xyang | Swanson: so you won't be able to create_volume_from_snapshot if that snapshot is part of a cgsnapshot? | 15:41 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: POC: Tooz locks demo https://review.openstack.org/185646 | 15:41 |
| Swanson | xyang: The cgsnapshot is wiped out as a group and then I roll through the db marking them deleted. | 15:41 |
| theanalyst | jgriffith: hi, you commented on https://review.openstack.org/#/c/189357/ saying patchset 5 was rebased .. what does it exactly mean in the context? | 15:41 |
| *** sgotliv has joined #openstack-cinder | 15:42 | |
| Swanson | xyang: I can create_volume_from_snapshot if I have the cgsnapshot_id. | 15:42 |
| *** yuriy_n17 has quit IRC | 15:42 | |
| *** mylu has quit IRC | 15:42 | |
| smcginnis | Swanson: But then which snapshot? | 15:42 |
| jgriffith | theanalyst: that's an auto generated comment, noting that "I" triggered a rebase against it | 15:42 |
| xyang | Swanson: I don't see how that works, you don't have an independent entry in you backend | 15:42 |
| *** sgotliv has quit IRC | 15:42 | |
| jgriffith | theanalyst: rebasing means updating the patch with things that have changed in the master branch since you started working on it | 15:43 |
| theanalyst | jgriffith: ah ok! | 15:43 |
| *** sgotliv has joined #openstack-cinder | 15:43 | |
| xyang | Swanson: seems to me you need to block that case. | 15:43 |
| jgriffith | theanalyst: it pulls in all of the changes "around" you while preserving your work | 15:43 |
| *** belmoreira has quit IRC | 15:43 | |
| Swanson | xyang: I have the source volume and the snapshot id. I have everything I need. | 15:43 |
| theanalyst | jgriffith: yep I know what a git-rebase means.. use it too often.. I thought that comment is again some sort of keyword like recheck etc :) | 15:43 |
| jgriffith | theanalyst: ahh.. ok :) | 15:43 |
| xyang | Swanson: but if your cgsnapshot goes away, what happens to that volume? | 15:44 |
| Swanson | xyang: Depends on how they delete it. Either I delete the snapshots in the db or I delete the volumes. | 15:44 |
| smcginnis | Swanson: So you're saying with the cg ID you can get the set of snaps, then from the source vol you can determine which on is the desired snap. | 15:44 |
| jgriffith | theanalyst: didn't know if you've rebased that one lately, but given last update was on the 1st I knew it needed the requirements update at least, so I went ahead and did a rebase on it. | 15:45 |
| Swanson | xyang: Actually the reverse. I look for the volume and then look for the snapshot labeled with the cgsnapshot id. | 15:45 |
| *** mylu has joined #openstack-cinder | 15:45 | |
| theanalyst | jgriffith: np, while we're at it if you find the time, could you please review the bp associated iwth the patch | 15:45 |
| jgriffith | sure | 15:46 |
| theanalyst | jgriffith: https://blueprints.launchpad.net/openstack/?searchtext=backup-restore-display-name | 15:46 |
| tbarron | theanalyst: well played :) | 15:46 |
| * jgriffith always says yes, but lately never actually does it :( | 15:46 | |
| theanalyst | tbarron: heh | 15:46 |
| Swanson | smcginnis: see comment inappropriately marked xyang ^^ | 15:46 |
| *** mylu has quit IRC | 15:47 | |
| smcginnis | Swanson: Got it. At any rate you are able to isolate the snap you need. xyang. what is the concern with delete? | 15:47 |
| theanalyst | sorry I was supposed to fish out the actual link https://blueprints.launchpad.net/cinder/+spec/backup-restore-display-name | 15:47 |
| tbarron | theanalyst: I think your patch is actually useful (can't say the same for lots of the additional knobs patches I see) | 15:47 |
| *** mylu has joined #openstack-cinder | 15:48 | |
| theanalyst | tbarron: thank you | 15:48 |
| *** garthb has joined #openstack-cinder | 15:48 | |
| *** chlong has quit IRC | 15:49 | |
| openstackgerrit | j_king proposed openstack/cinder: Add get_trust_context method to BackupManager https://review.openstack.org/202669 | 15:49 |
| *** garthb_ has joined #openstack-cinder | 15:49 | |
| *** garthb_ has quit IRC | 15:49 | |
| jgriffith | theanalyst: so I've been asked to NOT approve or target BP's | 15:50 |
| jgriffith | theanalyst: but FWIW I don't have any objection to your BP | 15:50 |
| theanalyst | tbarron: I know you had some comments on my other force-backup patch.. which xing yang is currently implementing | 15:50 |
| jgriffith | theanalyst: you'll have to work with thingee on approval/targeting | 15:51 |
| theanalyst | jgriffith: ah ok | 15:51 |
| *** earlephilhower has quit IRC | 15:51 | |
| theanalyst | I'm kind of a bit new to the openstack ecosystem, but can this BP still be in target for the next release or have I missed the deadline already | 15:52 |
| theanalyst | jgriffith: ^^ | 15:52 |
| jgriffith | theanalyst: I'm not aware of any deadlines for this type of work. You should be fine. | 15:53 |
| theanalyst | jgriffith: ty | 15:54 |
| tbarron | theanalyst: on your force-backup stuff, my point was only that it should be co-ordinated with xing's initiative for non-disruptive backup | 15:54 |
| tbarron | theanalyst: seemed to me there was overlap | 15:55 |
| *** earlephilhower has joined #openstack-cinder | 15:55 | |
| xyang | smcginnis: sorry, I'll get to you later. in a meeting now | 15:55 |
| *** hemnafk is now known as hemna | 15:55 | |
| theanalyst | tbarron: yeah just caught up with that.. seems xing's patch does account for force flag as well. | 15:56 |
| smcginnis | xyang: No problem, thanks! | 15:56 |
| *** afazekas has quit IRC | 15:58 | |
| *** breitz has quit IRC | 15:59 | |
| Swanson | xyang: thanks. | 15:59 |
| *** vilobhmm has joined #openstack-cinder | 16:00 | |
| *** jordan__ has quit IRC | 16:00 | |
| *** leeantho has joined #openstack-cinder | 16:01 | |
| *** Apoorva has joined #openstack-cinder | 16:06 | |
| openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete wait loop for CloudByte ISCSI volumes. https://review.openstack.org/201059 | 16:09 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 16:11 |
| *** jdurgin has quit IRC | 16:12 | |
| *** vokt has joined #openstack-cinder | 16:14 | |
| *** annashen has joined #openstack-cinder | 16:16 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/200898 | 16:17 |
| *** harlowja_at_home has quit IRC | 16:18 | |
| anteaya | thingee: hey there | 16:23 |
| thingee | anteaya: hi | 16:23 |
| *** _cjones_ has joined #openstack-cinder | 16:23 | |
| anteaya | thingee: so I believe it is erlon's system that is currently disabled that he is trying to get enabled | 16:23 |
| anteaya | thingee: have you an opinion? | 16:24 |
| thingee | anteaya: yeah I'm fine with it. We'll monitoring it today and give notice if there's still problems. | 16:24 |
| anteaya | http://lists.openstack.org/pipermail/third-party-announce/2015-July/000245.html | 16:24 |
| thingee | anteaya: looks like some of the issues were addressed in the mailing list | 16:24 |
| anteaya | thingee: very good, thank you | 16:24 |
| thingee | anteaya: thank you | 16:24 |
| anteaya | :) | 16:25 |
| *** asselin__ has joined #openstack-cinder | 16:26 | |
| mtanino | thingee: thank you! | 16:26 |
| *** asselin has quit IRC | 16:27 | |
| theanalyst | thingee: could you please take a look at the bp https://blueprints.launchpad.net/cinder/+spec/backup-restore-display-name | 16:29 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Add abandon script to clean up orphaned reviews https://review.openstack.org/203158 | 16:30 |
| smcginnis | thingee: ^^ Would love your thoughts on that. | 16:30 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Efficient image download and upload for Glance Cinder backend https://review.openstack.org/201754 | 16:30 |
| *** jistr has quit IRC | 16:31 | |
| *** ashuk has joined #openstack-cinder | 16:32 | |
| *** asselin__ has quit IRC | 16:34 | |
| *** annashen has quit IRC | 16:36 | |
| *** aagrawal has joined #openstack-cinder | 16:36 | |
| *** dims_ has quit IRC | 16:37 | |
| *** piyanai has quit IRC | 16:38 | |
| *** Yogi1 has quit IRC | 16:38 | |
| *** annashen has joined #openstack-cinder | 16:40 | |
| *** sgotliv has quit IRC | 16:41 | |
| *** HoloIRCUser4 has joined #openstack-cinder | 16:43 | |
| *** HoloIRCUser7 has quit IRC | 16:43 | |
| thingee | jgriffith: you mind addressing the comments here and I'll push it through? https://review.openstack.org/#/c/155644/ | 16:44 |
| *** HoloIRCUser4 has quit IRC | 16:44 | |
| pots | hi, new 3rd party tester here. can anyone tell me if rasselin's os-ext-testing scripts are still the best/easiest way to get started? i'm having all kinds of problems getting those scripts to work. | 16:46 |
| pots | in particular the install_slave.sh script doesn't seem to be installing jenkins and i'm not sure how it's supposed to get installed. | 16:47 |
| aarefiev | thingee: hi, have a min? I have question about https://review.openstack.org/#/c/176233/5/specs/liberty/cinder-manage-healthchecks-api.rst | 16:47 |
| thingee | aarefiev: about to take off..I'll bbl | 16:47 |
| * thingee take off on a plane | 16:47 | |
| *** s0nea has quit IRC | 16:48 | |
| *** rushil has joined #openstack-cinder | 16:48 | |
| *** s0nea has joined #openstack-cinder | 16:50 | |
| *** Apoorva has quit IRC | 16:52 | |
| earlephilhower | pots: I set up our CI maybe 1.5 months ago, but haven't had to go back to rasselin's original archive since. What exactly is failing on install_slave? | 16:54 |
| * earlephilhower checks my setup notes... | 16:54 | |
| *** mylu has quit IRC | 16:54 | |
| earlephilhower | pots, are you trying to use static build hosts? That's a no-no...never works AFACIT | 16:54 |
| pots | i'm trying to install on a ubuntu cloud image. the main problem is that it's not installing jenkins and i can't find where it's actually supposed to be installed. | 16:55 |
| pots | i'm new to puppet also which doesn't help. | 16:55 |
| earlephilhower | Join the club. Puppet, ansible, zuul, and 10 other tools needed...Jenkins installed on the master, not part of install_slace (that's only for static slaves). | 16:56 |
| earlephilhower | Puppet does the jenkins install on the master only. Dynamic slaves get injected by nodepool via SSH PSK | 16:57 |
| pots | i ran across a bunch of minor issues also: references to files in /etc/project-config that seemed like they should be to /root/project-config, cgconfig.conf was broken for ubuntu, puppet was not set to start, etc. | 16:57 |
| *** sghanekar_ has joined #openstack-cinder | 16:57 | |
| pots | do i need to use nodepool? i'd like to keep it as simple as possible. | 16:57 |
| *** piyanai has joined #openstack-cinder | 16:58 | |
| *** mylu has joined #openstack-cinder | 16:58 | |
| earlephilhower | Nodepool is what spins up a new clean VM for jenkins after each build. W/o it you'll have to make static slaves work. It wasn't working before. | 16:58 |
| earlephilhower | Nodepool is quite simple, actually. layout.yaml, give it an OS account to spin up VMs, and it takes care of the rest... | 16:59 |
| earlephilhower | It also makes dynamic slaves, from scratch, starting w/the ubuntu cloud VM image | 16:59 |
| pots | was hoping not to spin up VMs to run the slave, i'm running all this under VMware Workstation while I wait for hardware | 16:59 |
| earlephilhower | So never need to look at install_slave | 16:59 |
| earlephilhower | Can you set up an ubu openstack cloud under VMW workstation? Things would then be much easier, I think. | 17:00 |
| pots | maybe, i'm just afraid it'll break at the last step. | 17:01 |
| *** vilobhmm has quit IRC | 17:01 | |
| pots | install_slave seems to be setting up nodepool, so i had assumed that nodepool could just manage static slaves...is that not the case? | 17:02 |
| *** Tross has quit IRC | 17:02 | |
| earlephilhower | I'll have to check on that, because I don't remember seeing install_slave doing any of that....AFAIK nodepool runs on jenkins host only and really is only useful for creating updated/cached golden images daily and spinning up/down clean VMs for each job | 17:03 |
| earlephilhower | Let me check github, see if things changed in past 6 wks... | 17:03 |
| *** jungleboyj_ has joined #openstack-cinder | 17:03 | |
| *** patrickeast has joined #openstack-cinder | 17:04 | |
| *** jungleboyj_ is now known as jungleboyj | 17:04 | |
| earlephilhower | Crud...maybe I don't know what I'm talking about anymore... "NEW 7/1/2015: This repo is being migrated to use project-config and puppet-openstackci" | 17:04 |
| tsekiyama | patrickeast: I've updated the glance cinder backend patch to address your comments. Please check it if you have a chance. https://review.openstack.org/#/c/166414/ | 17:05 |
| *** ronis has joined #openstack-cinder | 17:05 | |
| patrickeast | tsekiyama: awesome, will do | 17:05 |
| tsekiyama | patrickeast: thanks! | 17:05 |
| pots | i'm not sure what that means--i'm drinking from a firehose trying to come up to speed | 17:05 |
| earlephilhower | It gets better the second time around. Unfortunately I've not looked at project-config, But for your sanity I'd beg, borrow, or steal an OS cloud and try to get dynamic nodes up. The slave.sh script may not have been touched to work w/the OS infra based changes they're doing. | 17:08 |
| *** mylu has quit IRC | 17:10 | |
| earlephilhower | High level is you've got a jenkins VM as the master controller. Same VM runs Zuul to hook in the OS GIT review system. When a new patch comes in, Zuul tells Jenkins which connectes to a running VM and does the tempest. When completed, the VM is shutdown by Jenkins/Nodepool(also running on Jenkins master) and another one is spun up. | 17:10 |
| earlephilhower | There are assumptions that the VM JEnkins runs on is "clean" and the std. scritps modify system files which cause things to break with static servers. You can fix, of course, with the source. | 17:11 |
| earlephilhower | But it seems like a real PITA compared to going with the flow. | 17:12 |
| earlephilhower | Do you have a Master VM running yet, and what services are up? | 17:12 |
| patrickeast | pots: oo setting up a ci system? | 17:12 |
| patrickeast | definitely avoid static slaves if you can | 17:12 |
| *** topshare has quit IRC | 17:12 | |
| pots | yes, i'm trying. i was hoping to use static slaves so i could integrate with our internal CI resources. | 17:13 |
| asselin_ | pots nodepool doesn't manage static slaves, it manages single-use slaves | 17:13 |
| pots | hi asselin, do you concur that static slaves don't work with the current scripts? | 17:13 |
| asselin_ | pots yes, statis slaves don't work, unless you e.g. revert to a snapshot after each test | 17:14 |
| asselin_ | i remember someone doing something like that | 17:14 |
| *** piyanai has quit IRC | 17:14 | |
| *** Apoorva has joined #openstack-cinder | 17:14 | |
| *** mylu has joined #openstack-cinder | 17:15 | |
| patrickeast | you can clean them up to mostly work (i did that for a few months) but its not perfect | 17:15 |
| patrickeast | i had some scripts to uninstall a bunch of stuff, clean out various configs and files left behind, etc | 17:16 |
| pots | ramy, do you think that your current os-ext-testing instructions will still work if i use nodepool & single-use slaves? | 17:16 |
| patrickeast | i still found myself having to manually intervene maybe once a week to get it working again | 17:16 |
| patrickeast | pots: i dont think anyone has tried them in a while | 17:16 |
| asselin_ | pots I never got static slaves working. I don't recommend them at all for cinder dsvm testing | 17:17 |
| *** annashen has quit IRC | 17:17 | |
| *** thangp has joined #openstack-cinder | 17:18 | |
| asselin_ | they're really to do less intrusive taskss, such as publish to pypi, or run unit tests that are contained within a sandbox virtualenv | 17:18 |
| patrickeast | asselin_: might be useful to remove that info from https://github.com/rasselin/os-ext-testing#setting-up-static-jenkins-slaves or put a big warning label on it | 17:18 |
| pots | so i should bring up an openstack cloud to run the slaves? or can i just use kvm on the master node? | 17:19 |
| asselin_ | patrickeast, good idea | 17:19 |
| asselin_ | pots, setup a small cloud | 17:19 |
| pots | i assume nodepool is openstack-specific? | 17:19 |
| patrickeast | pots: you would need an openstack cloud, but a very simple one, you can do an all-in-one kind of thing | 17:19 |
| *** mylu has quit IRC | 17:19 | |
| asselin_ | pots, you can even use devstack | 17:20 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: FC discover existing devices for removal https://review.openstack.org/202826 | 17:20 |
| patrickeast | pots: i use RDO to setup my nodes, its relatively easy to get going and handles restarts and stuff better than devstack, but you can’t beat the simplicity of devstack | 17:20 |
| *** mylu has joined #openstack-cinder | 17:21 | |
| *** piyanai has joined #openstack-cinder | 17:22 | |
| pots | ok, i'll see if i can get that going. thanks all for your help! | 17:23 |
| *** daneyon has quit IRC | 17:23 | |
| earlephilhower | Good luck, pots. You'll need it. :) | 17:23 |
| thingee | aarefiev: back, looking | 17:24 |
| pots | there goes the weekend :( | 17:24 |
| patrickeast | pots: also, if no one has pointed you to them yet, there is help available for this https://wiki.openstack.org/wiki/Meetings/ThirdParty#Weekly_Third_Party_meetings | 17:24 |
| patrickeast | a couple times a week there are meetings full of other folks who have set them up and can answer questions | 17:25 |
| pots | yep, it's in my calendar. i'll have a lot of questions on Monday... | 17:25 |
| asselin_ | pots, also a small warning, there's lots of refactoring going on on my repo coupled with upstream changes. If you run into issues, ping me or others around. | 17:26 |
| *** harlowja has quit IRC | 17:26 | |
| *** harlowja has joined #openstack-cinder | 17:26 | |
| pots | will do--thank you all for your help! | 17:26 |
| asselin_ | pots and take a look at these changes, especially recent merges: https://review.openstack.org/#/q/topic:downstream-puppet,n,z | 17:26 |
| *** ashuk has quit IRC | 17:26 | |
| *** annashen has joined #openstack-cinder | 17:27 | |
| *** mylu has quit IRC | 17:27 | |
| *** mylu has joined #openstack-cinder | 17:27 | |
| *** piyanai has quit IRC | 17:27 | |
| *** mylu has quit IRC | 17:28 | |
| thingee | aarefiev: responded | 17:30 |
| *** mylu has joined #openstack-cinder | 17:30 | |
| *** Tross has joined #openstack-cinder | 17:30 | |
| earlephilhower | thingee, hemna: Howdy. Silly question, but I see you updated os-brick changelog to 0.3.0 but didn't add a tag in git. Any ETA for the new tag? | 17:32 |
| hemna | earlephilhower, I don't publish os-brick. only the PTL can. | 17:32 |
| thingee | earlephilhower: I'm working it out right now with dhellmann | 17:33 |
| earlephilhower | Cool, thanks. | 17:33 |
| * earlephilhower reads up on what PTL is...not the televangelists I imagine... :) | 17:33 | |
| *** Yogi1 has joined #openstack-cinder | 17:33 | |
| thingee | earlephilhower: I put in a request earlier this week to the release folks, but this sort of thing is not something I can just do anymore unfortunately | 17:33 |
| openstackgerrit | Doug Hellmann proposed openstack/os-brick: Use pbr's automatically generated changelog https://review.openstack.org/201643 | 17:34 |
| earlephilhower | thingee, no worries. Appreciate the info. | 17:34 |
| *** jdurgin has joined #openstack-cinder | 17:36 | |
| thingee | earlephilhower: at this point it'll probably be monday since we don't like to do releases on friday's | 17:39 |
| earlephilhower | thingee, With the new tag, would the g-r also be updated as part of the PTL's process? | 17:41 |
| thingee | g-r? | 17:42 |
| earlephilhower | global-requiements..txt | 17:42 |
| *** e0ne has quit IRC | 17:44 | |
| aarefiev | thingee: about your answer, you are looking at problem from backend side, I'll try to see if problem in messaging, when service status is up(backend is fine), but services cant communicate, and it is difficult to see it | 17:44 |
| *** boris-42 has joined #openstack-cinder | 17:44 | |
| *** vilobhmm has joined #openstack-cinder | 17:46 | |
| *** mylu has quit IRC | 17:47 | |
| thingee | aarefiev: today services communicate through cinder-manager service list | 17:47 |
| thingee | aarefiev: I might be missing what you're trying to accomplish with this | 17:48 |
| *** shyama_ has joined #openstack-cinder | 17:49 | |
| jungleboyj | Anyone else seeing unit tests fail in test_image_utils.TestFetch.test_defaults ? | 17:49 |
| jungleboyj | I am seeing on our internal code and on Master which has me baffled. | 17:50 |
| *** shyama has quit IRC | 17:50 | |
| *** shyama_ is now known as shyama | 17:50 | |
| aarefiev | thingee: I try to say that it is possible that backend is fine, but messaging has problems, and I'd like to see it with this cmd | 17:50 |
| *** Tross has quit IRC | 17:50 | |
| thingee | aarefiev: can you define messaging here? | 17:51 |
| thingee | currently it's a face smiling or a frown face | 17:51 |
| aarefiev | thingee: I mean rpc | 17:51 |
| aarefiev | e.g. rabbitmq | 17:51 |
| thingee | aarefiev: you want to know from cinder if rabbit is running correctly? | 17:52 |
| aarefiev | thingee: I want to now if problem in it, if no continue looking the problem | 17:53 |
| thingee | aarefiev: if there is a problem with rabbit, there's going to be a problem with start cinder-scheduler or cinder-vol | 17:53 |
| smcginnis | jungleboyj: Yeah, something is up. Not sure what though. Haven't had time to dig in. | 17:54 |
| aarefiev | thingee: I saw cases when connection to rpc is fine, but messages can be received to services | 17:55 |
| aarefiev | ^can't | 17:55 |
| thingee | aarefiev: and why is that? | 17:56 |
| *** mylu has joined #openstack-cinder | 17:56 | |
| aarefiev | thingee: there a lot of problems with rabbit, i don't know them, I just want to find out root cause of problem in cinder | 17:57 |
| aarefiev | thingee: as I can see some of them is already fixed, but not all | 17:58 |
| thingee | aarefiev: I would like to understand that better first. Otherwise, there's not really a way to identify the problem to communicate it. | 17:58 |
| jungleboyj | smcginnis: Ok, so not just me seeing it. That was the first thing I wanted to. | 17:59 |
| thingee | aarefiev: regardless, based on the vague information given here, it seems like this would a problem for all than just Cinder. | 17:59 |
| aarefiev | thingee: I just want identify problem in cinder or backend or in rpc | 17:59 |
| *** piyanai has joined #openstack-cinder | 18:00 | |
| *** mylu has quit IRC | 18:01 | |
| thingee | aarefiev: I definitely welcome fixing issues of cinder + rabbit. But 1) I'm not even aware of this, yet. 2) The information you're giving me is not complete for me to understand what the problem is exactly. 3) your proposal is about message on this problem, but it seems like we don't know what the problem is exactly to be able to identify and communicate it. | 18:01 |
| *** mylu has joined #openstack-cinder | 18:02 | |
| *** jungleboyj has quit IRC | 18:03 | |
| aarefiev | thingee: but if problem in rabbit, I spent a lot of time to identify this, so I've add this tool to cinder manage, btw it could not 'one' problem | 18:03 |
| aarefiev | thingee: I don't want to fix it, I try to make troubleshooting easy | 18:04 |
| thingee | aarefiev: how are you going to identify the problem if you don't know what it is? | 18:05 |
| aarefiev | thingee: I mean to see among cinder, rabbit | 18:06 |
| *** haomaiw__ has quit IRC | 18:06 | |
| *** shyama has quit IRC | 18:06 | |
| thingee | aarefiev: I'm sorry but that didn't make sense to me for my question. | 18:06 |
| *** crose has joined #openstack-cinder | 18:07 | |
| aarefiev | thingee: if problem in rabbit has appeared, now it is difficult to see it | 18:07 |
| aarefiev | thingee: service is in status up | 18:08 |
| aarefiev | thingee: we can't see e.g. which volumes don't report stats | 18:09 |
| *** vincent_hou has joined #openstack-cinder | 18:09 | |
| aarefiev | thingee: or problem on node with one of scheduler | 18:09 |
| *** dannywilson has joined #openstack-cinder | 18:10 | |
| *** ronis has quit IRC | 18:11 | |
| *** vincent_hou has quit IRC | 18:14 | |
| *** piyanai has quit IRC | 18:14 | |
| aarefiev | thingee: sorry, it's late, I have to go out, I'll try contact with you next week to continue discussion, ofcourse if you have time; thank you for your time | 18:14 |
| *** aarefiev has quit IRC | 18:16 | |
| *** jungleboyj has joined #openstack-cinder | 18:19 | |
| *** haomaiwang has joined #openstack-cinder | 18:23 | |
| *** delattec has joined #openstack-cinder | 18:24 | |
| *** raunak has joined #openstack-cinder | 18:25 | |
| *** cdelatte has quit IRC | 18:28 | |
| *** haomaiwang has quit IRC | 18:28 | |
| *** piyanai has joined #openstack-cinder | 18:28 | |
| *** e0ne has joined #openstack-cinder | 18:29 | |
| *** haomaiwa_ has joined #openstack-cinder | 18:31 | |
| *** markvoelker has quit IRC | 18:32 | |
| *** ronis has joined #openstack-cinder | 18:33 | |
| *** haomaiwa_ has quit IRC | 18:36 | |
| *** mdenny has quit IRC | 18:38 | |
| thingee | smcginnis: I think it's related to a dependency change somehow. I have an outdated venv that passes with the latest cinder | 18:39 |
| thingee | jungleboyj: ^ | 18:39 |
| *** lpetrut has joined #openstack-cinder | 18:40 | |
| *** haomaiwang has joined #openstack-cinder | 18:41 | |
| * thingee is that guy on the plane that will use wifi for getting new deps of cinder to run unit tests | 18:45 | |
| *** delattec has quit IRC | 18:46 | |
| thingee | ugh about to land. I'll check on it again when I can | 18:46 |
| thingee | smcginnis, jungleboyj thanks for the heads up | 18:46 |
| smcginnis | thingee: Safe travels. | 18:47 |
| jungleboyj | thingee: Thank you. Safe trip. | 18:47 |
| patrickeast | tsekiyama: put a few more comments on the review | 18:51 |
| patrickeast | tsekiyama: if you haven’t started looking at it maybe today i’ll try and put some code in to handle creating the raw image-volume for any image format coming in | 18:52 |
| patrickeast | rework whats in my cache patchset to do that | 18:52 |
| tsekiyama | patrickeast: Thank you, I'll check your comments | 18:52 |
| *** haomaiwang has quit IRC | 18:53 | |
| *** delattec has joined #openstack-cinder | 18:54 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Graceful shutdown WSGI/RPC server https://review.openstack.org/194929 | 18:55 |
| *** raunak has quit IRC | 18:55 | |
| *** haomaiwang has joined #openstack-cinder | 18:57 | |
| xyang | smcginnis, Swanson: I added comments in that patch. I think it is fine, just need some clarification because it looks like you are mixing the two ids | 18:58 |
| *** mylu has quit IRC | 18:58 | |
| *** mylu has joined #openstack-cinder | 18:58 | |
| Swanson | xyang: Thanks! Adding comments to the patch now. | 18:59 |
| xyang | Swanson: sounds good | 18:59 |
| *** lpetrut has quit IRC | 19:00 | |
| *** chenying has joined #openstack-cinder | 19:00 | |
| *** mylu has quit IRC | 19:02 | |
| *** mdenny has joined #openstack-cinder | 19:02 | |
| jgriffith | mgagne: you around? | 19:02 |
| mgagne | jgriffith: I am | 19:02 |
| jgriffith | mgagne: wanted to pick your brain on this bug: https://bugs.launchpad.net/manila/+bug/1475285 | 19:02 |
| openstack | Launchpad bug 1475285 in Cinder "Non-admin user can perform 'extra-specs-list'" [High,Confirmed] | 19:02 |
| jgriffith | mgagne: Looks like it was part of the private types work: https://github.com/openstack/cinder/commit/6c0f50b1ec933a61b84d806e748afd9cb74e5cd7#diff-cb41f88f083a12f8858d6b6973ff68ca | 19:03 |
| *** mylu has joined #openstack-cinder | 19:03 | |
| mgagne | jgriffith: what can I do to help? | 19:03 |
| jgriffith | mgagne: but regardless, I was going to just add a filter for non-admin and extra-specs... | 19:04 |
| jgriffith | mgagne: hmmm.. wait, that won't work anyway :( | 19:04 |
| mgagne | jgriffith: ok let me read the whole thing | 19:04 |
| jgriffith | mgagne: anyhow.. my question became... what a bout a mix of types? | 19:04 |
| jgriffith | mgagne: so an admin in one project who is not an admin in another requests extra-specs... | 19:04 |
| jgriffith | mgagne: the way we run today, we don't have a way of filtering that out, unless we check each type individually I suppose | 19:05 |
| jgriffith | mgagne: in a nut-shell the issue is that a non-admin can now do "cinder extra-specs-list" | 19:05 |
| mgagne | jgriffith: and you wonder if he should be able to list extra-specs from a private type? | 19:06 |
| jgriffith | mgagne: part of the issue IMO is that we are kinda lazy with extra_specs_list and just pull the info out of the type-list | 19:06 |
| jgriffith | mgagne: correct | 19:06 |
| mgagne | jgriffith: hmmm I haven't thought of that | 19:06 |
| openstackgerrit | Tom Barron proposed openstack/cinder-specs: Scaling backup service blueprint spec https://review.openstack.org/203215 | 19:06 |
| jgriffith | mgagne: so I can 'fix' this bug, but it sort of introduces another odd case that's questionable, so I thought I'd consult with you :) | 19:07 |
| jgriffith | mgagne: although I suppose it's fine if somebody is an "Admin" in any project then they have the ability to "see" extra-specs-list | 19:08 |
| mgagne | jgriffith: I think the reason why I haven't "thought" of that is that Nova doesn't allow you to list extra-specs of all flavors. there is no "flavor-key-list" command | 19:08 |
| jgriffith | mgagne: hehe :) that's very true | 19:08 |
| mgagne | otherwise there should already condition in place to handle such case in db.api | 19:09 |
| *** ndipanov has quit IRC | 19:09 | |
| mgagne | could it be reused for extra-specs-list? but then: what if an admin wishes to list ALL extra-specs, irregardless of ownership | 19:09 |
| jgriffith | hmm... good point | 19:10 |
| mgagne | so now we are back to that --all-tenants thing | 19:10 |
| *** haomaiwang has quit IRC | 19:11 | |
| jgriffith | mgagne: I think I can just compare the context to the returned type in volume_types:get_all | 19:12 |
| mgagne | in fact, in nova, it's --all to list private flavors. I don't see a flavor in cinder v2 cli | 19:12 |
| *** haomaiwang has joined #openstack-cinder | 19:12 | |
| mgagne | I thought I implemented it but meh | 19:12 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Fix Consistency Group issues https://review.openstack.org/201685 | 19:13 |
| mgagne | it got removed | 19:13 |
| mgagne | https://github.com/openstack/python-cinderclient/commit/289ab37c790b8848e78582926c336628ca51ca6d | 19:13 |
| mgagne | due to bug #1462104 | 19:14 |
| openstack | bug 1462104 in Cinder "access-type-add doesn't take a volume type name" [Low,Fix released] https://launchpad.net/bugs/1462104 - Assigned to Ed Balduf (ebalduf) | 19:14 |
| jgriffith | mgagne: sigh | 19:15 |
| jgriffith | mgagne: indded | 19:15 |
| jgriffith | indeed | 19:15 |
| *** mylu has quit IRC | 19:17 | |
| jgriffith | mgagne: well, thanks for the help :) | 19:17 |
| jgriffith | mgagne: I'll see what I can do to fix both issues | 19:17 |
| mgagne | this could have been fixed by updating default value of is_public in list() while keeping support for --all | 19:17 |
| *** dims_ has joined #openstack-cinder | 19:18 | |
| mgagne | and using vtypes = cs.volume_types.list(is_public=True) when --all is not used | 19:18 |
| mgagne | or not | 19:18 |
| *** haomaiwang has quit IRC | 19:18 | |
| mgagne | I'm saying silly things | 19:18 |
| jgriffith | mgagne: :) | 19:19 |
| Swanson | What is the record for most items in 'check'? | 19:19 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: LVM: Support efficient data copy for LVM driver https://review.openstack.org/200237 | 19:19 |
| *** afazekas has joined #openstack-cinder | 19:19 | |
| mgagne | no, it fact, it would have worked since it would have kept the old behavior while fixing VolumeTypeAccessManager | 19:20 |
| mgagne | ok, back to nova cells hell :-/ | 19:20 |
| jgriffith | mgagne: oh dear.. sorry to hear that :( | 19:20 |
| jgriffith | K, I'll get lunch and then try that out, thanks for the help mgagne | 19:21 |
| mgagne | yw, bon appétit ;) | 19:21 |
| *** mylu has joined #openstack-cinder | 19:21 | |
| *** dustins has quit IRC | 19:22 | |
| *** chenying has quit IRC | 19:22 | |
| *** mylu has quit IRC | 19:23 | |
| *** merooney has quit IRC | 19:23 | |
| *** edmondsw has quit IRC | 19:24 | |
| *** mylu has joined #openstack-cinder | 19:25 | |
| *** IlyaG has joined #openstack-cinder | 19:26 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: LVM: Support efficient data copy for LVM driver https://review.openstack.org/200237 | 19:28 |
| *** mylu has quit IRC | 19:28 | |
| *** mylu has joined #openstack-cinder | 19:29 | |
| *** erlon has quit IRC | 19:30 | |
| *** dims__ has joined #openstack-cinder | 19:30 | |
| *** haomaiwang has joined #openstack-cinder | 19:32 | |
| *** dims_ has quit IRC | 19:34 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: FC discover existing devices for removal https://review.openstack.org/202826 | 19:34 |
| *** akerr has quit IRC | 19:34 | |
| *** lpetrut has joined #openstack-cinder | 19:35 | |
| *** haomaiwang has quit IRC | 19:36 | |
| *** haomaiwa_ has joined #openstack-cinder | 19:37 | |
| *** mylu has quit IRC | 19:39 | |
| *** mylu has joined #openstack-cinder | 19:40 | |
| *** haomaiwang has joined #openstack-cinder | 19:40 | |
| *** haomaiwa_ has quit IRC | 19:41 | |
| *** nikeshm has joined #openstack-cinder | 19:43 | |
| *** mylu has quit IRC | 19:45 | |
| *** Lee1092 has quit IRC | 19:45 | |
| rhe00_ | thingee: ping | 19:47 |
| *** haomaiwang has quit IRC | 19:47 | |
| *** ig0r_ has quit IRC | 19:47 | |
| *** mylu has joined #openstack-cinder | 19:51 | |
| *** haomaiwa_ has joined #openstack-cinder | 19:52 | |
| *** piyanai has quit IRC | 19:53 | |
| *** haomaiwa_ has quit IRC | 19:57 | |
| *** haomaiwang has joined #openstack-cinder | 20:03 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Remove logging statements from migrations https://review.openstack.org/191983 | 20:03 |
| *** akerr has joined #openstack-cinder | 20:03 | |
| *** haomaiwa_ has joined #openstack-cinder | 20:06 | |
| *** akerr_ has joined #openstack-cinder | 20:07 | |
| *** haomaiwang has quit IRC | 20:08 | |
| *** Yogi1 has quit IRC | 20:09 | |
| *** piyanai has joined #openstack-cinder | 20:10 | |
| *** akerr has quit IRC | 20:10 | |
| thingee | rhe00_: hi | 20:12 |
| *** delattec has quit IRC | 20:12 | |
| thingee | mriedem: https://review.openstack.org/#/c/203193/ release will be monday | 20:13 |
| mriedem | ok | 20:14 |
| thingee | smcginnis: yeah blowing away my venv allowed it to be replicated | 20:14 |
| thingee | mriedem: apparently people like to have fun on fridays and not have to fix things :) | 20:14 |
| mriedem | thingee: fwiw i don't think https://review.openstack.org/#/c/203193/ would have passed jenkins anyway | 20:14 |
| mriedem | thingee: we've been trying to fix nova's gate all day.... | 20:14 |
| lifeless | and this is why ci pins are important :) | 20:15 |
| *** haomaiwa_ has quit IRC | 20:16 | |
| *** jasondotstar has joined #openstack-cinder | 20:16 | |
| *** lpetrut has quit IRC | 20:18 | |
| thingee | lifeless: I finally updated some of my openstack/requirement bump patches. dhellmann got me up to speed with upper constraints | 20:20 |
| *** thangp has quit IRC | 20:21 | |
| *** haomaiwang has joined #openstack-cinder | 20:21 | |
| lifeless | thingee: there's a patch there with full docs | 20:22 |
| *** Apoorva_ has joined #openstack-cinder | 20:24 | |
| *** Apoorva has quit IRC | 20:24 | |
| *** raunak has joined #openstack-cinder | 20:26 | |
| *** haomaiwang has quit IRC | 20:26 | |
| *** haomaiwang has joined #openstack-cinder | 20:26 | |
| *** kjnelson has left #openstack-cinder | 20:27 | |
| *** dims__ has quit IRC | 20:34 | |
| *** Apoorva_ has quit IRC | 20:34 | |
| *** haomaiwang has quit IRC | 20:34 | |
| *** haomaiwang has joined #openstack-cinder | 20:35 | |
| *** piyanai has quit IRC | 20:36 | |
| *** jungleboyj has quit IRC | 20:40 | |
| *** mylu has quit IRC | 20:42 | |
| *** mylu has joined #openstack-cinder | 20:42 | |
| *** julim_ has quit IRC | 20:43 | |
| *** e0ne has quit IRC | 20:43 | |
| *** piyanai has joined #openstack-cinder | 20:44 | |
| rhe00_ | thingee: our CI is back up and stable (if I stop experimenting with the scripts to make it faster). However I am currently 90+ changes behind. What would you like to see before the change to add back the driver https://review.openstack.org/#/c/200196/3 is accepted? | 20:44 |
| openstackgerrit | Matthew Edmonds proposed openstack/cinder: move oslo.vmware to test-requirements https://review.openstack.org/203237 | 20:46 |
| *** piyanai has quit IRC | 20:46 | |
| *** Apoorva has joined #openstack-cinder | 20:47 | |
| *** piyanai has joined #openstack-cinder | 20:48 | |
| *** haomaiwang has quit IRC | 20:48 | |
| *** haomaiwang has joined #openstack-cinder | 20:51 | |
| *** crose has quit IRC | 20:51 | |
| *** piyanai has quit IRC | 20:51 | |
| *** bnemec has quit IRC | 20:54 | |
| *** jwcroppe_ has joined #openstack-cinder | 20:54 | |
| *** jwcroppe has quit IRC | 20:56 | |
| *** haomaiwang has quit IRC | 20:56 | |
| *** eharney has quit IRC | 20:57 | |
| *** mylu has quit IRC | 20:57 | |
| openstackgerrit | Mathieu Gagné proposed openstack/cinder: Add volume_attachment to volume usage notifications https://review.openstack.org/201811 | 20:58 |
| *** edmondsw has joined #openstack-cinder | 20:59 | |
| *** Apoorva has quit IRC | 20:59 | |
| *** dims_ has joined #openstack-cinder | 21:00 | |
| *** Apoorva has joined #openstack-cinder | 21:00 | |
| *** alwaysatthenoc has joined #openstack-cinder | 21:01 | |
| *** jungleboyj has joined #openstack-cinder | 21:04 | |
| *** jasondotstar has quit IRC | 21:05 | |
| *** akerr_ is now known as akerr_away | 21:08 | |
| *** akerr_away is now known as akerr_ | 21:08 | |
| *** akshai has quit IRC | 21:10 | |
| *** haomaiwang has joined #openstack-cinder | 21:11 | |
| *** gouthamr has quit IRC | 21:12 | |
| nikeshm | hi | 21:13 |
| *** sghanekar_ has quit IRC | 21:14 | |
| *** jwcroppe has joined #openstack-cinder | 21:14 | |
| nikeshm | looks like gate is failing on gate-cinder-python27 and gate-cinder-python34 | 21:15 |
| nikeshm | https://review.openstack.org/#/c/192751/8 https://review.openstack.org/#/c/189357/4 | 21:15 |
| *** jwcroppe_ has quit IRC | 21:16 | |
| *** sgotliv has joined #openstack-cinder | 21:17 | |
| *** jwcroppe_ has joined #openstack-cinder | 21:17 | |
| *** haomaiwang has quit IRC | 21:17 | |
| *** jwcroppe has quit IRC | 21:19 | |
| *** haomaiwa_ has joined #openstack-cinder | 21:21 | |
| *** jwcroppe has joined #openstack-cinder | 21:23 | |
| *** cbader has quit IRC | 21:23 | |
| *** raunak has quit IRC | 21:23 | |
| *** skylerberg has joined #openstack-cinder | 21:23 | |
| *** jwcroppe_ has quit IRC | 21:25 | |
| *** primechuck has quit IRC | 21:25 | |
| *** rushil has quit IRC | 21:26 | |
| *** akerr_ has quit IRC | 21:27 | |
| *** haomaiwang has joined #openstack-cinder | 21:27 | |
| *** jwcroppe_ has joined #openstack-cinder | 21:28 | |
| *** haomaiwa_ has quit IRC | 21:29 | |
| *** blockedpipes has quit IRC | 21:29 | |
| *** jwcroppe has quit IRC | 21:30 | |
| *** jasondotstar has joined #openstack-cinder | 21:31 | |
| *** haomaiwang has quit IRC | 21:32 | |
| *** sghanekar_ has joined #openstack-cinder | 21:32 | |
| *** ronis has quit IRC | 21:34 | |
| *** jwcroppe has joined #openstack-cinder | 21:35 | |
| *** jwcroppe_ has quit IRC | 21:37 | |
| *** divab has quit IRC | 21:38 | |
| *** alwaysatthenoc has quit IRC | 21:38 | |
| *** haomaiw__ has joined #openstack-cinder | 21:38 | |
| *** divab has joined #openstack-cinder | 21:39 | |
| *** divab has quit IRC | 21:40 | |
| *** divab has joined #openstack-cinder | 21:40 | |
| *** jwcroppe_ has joined #openstack-cinder | 21:40 | |
| *** jwcroppe has quit IRC | 21:42 | |
| *** jwcroppe has joined #openstack-cinder | 21:43 | |
| edmondsw | jungleboyj, see my response in https://review.openstack.org/#/c/203237/ | 21:44 |
| *** jwcroppe_ has quit IRC | 21:45 | |
| jungleboyj | edmondsw: So, when the operator wants to use that driver they have to go install the requirement themselves? I don't think that is the way we have been operating in the past. | 21:46 |
| edmondsw | yes, they would need to go install that | 21:46 |
| edmondsw | I think we've been trying to do this, albeit perhaps inconsistently | 21:47 |
| *** haomaiw__ has quit IRC | 21:48 | |
| patrickeast | that matches up with some of the other drivers (like mine) that require extra libs | 21:48 |
| jungleboyj | patrickeast: You are requiring the users to install additional packages? | 21:48 |
| patrickeast | yep | 21:49 |
| patrickeast | https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/pure.py#L39-L42 | 21:49 |
| patrickeast | then https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/pure.py#L94 | 21:49 |
| edmondsw | if you want an example from another project: https://github.com/openstack/glance_store/blob/master/test-requirements.txt#L25 | 21:50 |
| patrickeast | jungleboyj: so someone who wants to use our driver needs to ‘pip install purestorage’ on their c-vol host before it will work | 21:50 |
| jungleboyj | Ok. Then I am still -1 on the patch. | 21:50 |
| *** haomaiwa_ has joined #openstack-cinder | 21:50 | |
| edmondsw | because? | 21:51 |
| jungleboyj | Updated the patch. If you want to move this out the requirements file then we need to appropriately catch when the library isn't installed. | 21:52 |
| edmondsw | patrickeast, do we do that for pure? | 21:52 |
| jungleboyj | That is what he just pasted above. | 21:52 |
| patrickeast | nope its all open source and available on pypi | 21:52 |
| jungleboyj | Other drivers do do that if they are expecting additonal code installed. | 21:52 |
| patrickeast | just that we don’t want to force it on people not using our driver | 21:52 |
| patrickeast | and don’t want to duplicate the code in cinder | 21:53 |
| edmondsw | ok, sure... tx guys | 21:53 |
| jungleboyj | edmondsw: That makes more sense. Then I would be more comfortable with it. | 21:54 |
| edmondsw | yep, I'll make that change | 21:54 |
| jungleboyj | Thanks. | 21:54 |
| *** mriedem has quit IRC | 21:54 | |
| asselin_ | patrickeast, jungleboyj It's better not adding it to the requirements | 21:56 |
| asselin_ | we used to do that for 3par and lefthand, and it took a while to get them out | 21:56 |
| jungleboyj | Yeah, I remember that. | 21:58 |
| *** ganso_ has quit IRC | 21:58 | |
| *** raunak has joined #openstack-cinder | 21:59 | |
| openstackgerrit | Tom Barron proposed openstack/cinder-specs: Scaling backup service blueprint spec https://review.openstack.org/203215 | 21:59 |
| *** jungleboyj has quit IRC | 22:02 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Fix Consistency Group issues https://review.openstack.org/201685 | 22:03 |
| *** piyanai has joined #openstack-cinder | 22:04 | |
| *** IlyaG has quit IRC | 22:07 | |
| openstackgerrit | Matthew Edmonds proposed openstack/cinder: move oslo.vmware to test-requirements https://review.openstack.org/203237 | 22:17 |
| *** dalgaaf has quit IRC | 22:18 | |
| *** haomaiw__ has joined #openstack-cinder | 22:21 | |
| *** haomaiwa_ has quit IRC | 22:22 | |
| *** piyanai has quit IRC | 22:26 | |
| *** edmondsw has quit IRC | 22:28 | |
| *** asselin has joined #openstack-cinder | 22:30 | |
| *** IanGovett has quit IRC | 22:34 | |
| *** asselin has quit IRC | 22:35 | |
| *** sgotliv has quit IRC | 22:39 | |
| *** haomaiwa_ has joined #openstack-cinder | 22:42 | |
| *** haomaiw__ has quit IRC | 22:43 | |
| *** jasondotstar has quit IRC | 22:44 | |
| *** haomaiw__ has joined #openstack-cinder | 22:45 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: WIP: remove the iscsi rescan during disconnect https://review.openstack.org/203265 | 22:46 |
| *** haomaiwa_ has quit IRC | 22:46 | |
| hemna | :) | 22:46 |
| *** mdenny has quit IRC | 22:47 | |
| *** mgarza_ has quit IRC | 22:49 | |
| *** mylu has joined #openstack-cinder | 22:54 | |
| *** mriedem has joined #openstack-cinder | 22:57 | |
| *** haomaiw__ has quit IRC | 22:59 | |
| *** haomaiwa_ has joined #openstack-cinder | 22:59 | |
| jgriffith | patrickeast: asselin_ +1... I've always been a bit opposed to EVERYTHING in the world going into requirements file | 23:04 |
| jgriffith | hemna: surely remembers my whining about it :) | 23:04 |
| *** haomaiwa_ has quit IRC | 23:06 | |
| *** annashen has quit IRC | 23:06 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Non-disruptive backup https://review.openstack.org/193937 | 23:07 |
| *** xyang has quit IRC | 23:07 | |
| jgriffith | xyang1: Whhhaat? You added a "prev state column"? | 23:08 |
| jgriffith | xyang1: Just kidding :) | 23:08 |
| xyang1 | jgriffith: :) | 23:08 |
| *** annashen has joined #openstack-cinder | 23:08 | |
| xyang1 | jgriffith: I added to volumes instead of backups | 23:09 |
| xyang1 | jgriffith: Because it is volume status | 23:09 |
| jgriffith | xyang1: That where I would've put them as well | 23:09 |
| jgriffith | xyang1: or put "it" | 23:09 |
| *** garthb has quit IRC | 23:09 | |
| jgriffith | xyang1: logging in the migration file :( | 23:09 |
| xyang1 | jgriffith: I wondered about it:) | 23:10 |
| xyang1 | jgriffith: I thought jungleboyj_ will give -1 if i don't have it | 23:10 |
| jgriffith | xyang1: I'm sure he probably would | 23:10 |
| *** mylu has quit IRC | 23:11 | |
| xyang1 | jgriffith: I can remove that and the try catch | 23:11 |
| jgriffith | xyang1: if something else comes up I would love that personally | 23:11 |
| xyang1 | jgriffith: And reference your patch:) | 23:11 |
| xyang1 | jgriffith: I can remove that, no problem | 23:12 |
| jgriffith | xyang1: so how would .... | 23:12 |
| jgriffith | NM | 23:12 |
| jgriffith | xyang1: so in "def create_backup" | 23:13 |
| *** haomaiwa_ has joined #openstack-cinder | 23:13 | |
| xyang1 | jgriffith: manager? | 23:13 |
| jgriffith | xyang1: yeah, I'll just comment in the review | 23:13 |
| jgriffith | :) | 23:13 |
| jgriffith | besides, as I walk through it I might be wrong :) | 23:14 |
| xyang1 | jgriffith: I don't see your comments | 23:14 |
| jgriffith | xyang1: I'm not done yet :) | 23:14 |
| xyang1 | jgriffith: Ok:) | 23:14 |
| patrickeast | xyang1: hey, for those temporary volumes/snapshots you could use the internal tenant :D | 23:14 |
| xyang1 | patrickeast: Yes, is yours merged yet? | 23:15 |
| patrickeast | no :( | 23:15 |
| * patrickeast finds link | 23:15 | |
| patrickeast | https://review.openstack.org/#/c/199352/ | 23:15 |
| *** haomai___ has joined #openstack-cinder | 23:17 | |
| xyang1 | patrickeast: I'll take a look. I have a couple of dependent patches myself, like to sort out other concerns first | 23:18 |
| *** haomaiwa_ has quit IRC | 23:18 | |
| *** haomai___ has quit IRC | 23:23 | |
| *** haomaiwa_ has joined #openstack-cinder | 23:25 | |
| jgriffith | xyang1: I made a number of comments | 23:27 |
| jgriffith | xyang1: overall I'm positive so gave it a +1 | 23:27 |
| jgriffith | xyang1: there are some things around safety in getting the status from the object, and exactly what we should set it to in case of failure. | 23:28 |
| jgriffith | xyang1: logs of course I already mentioned :) And a long winded random babble of nonsense that may or may not mean anything to anybody | 23:28 |
| xyang1 | jgriffith: Thanks! Let me read your comments:) | 23:30 |
| *** hemna is now known as hemnafk | 23:31 | |
| *** haomaiw__ has joined #openstack-cinder | 23:32 | |
| xyang1 | jgriffith: The rollback part, it was like that before | 23:32 |
| *** nikeshm has quit IRC | 23:32 | |
| *** lcurtis has quit IRC | 23:33 | |
| *** haomaiwa_ has quit IRC | 23:33 | |
| xyang1 | jgriffith: It sets back to available before, so I just added in-use for attached volumes | 23:33 |
| *** sghanekar_ has quit IRC | 23:33 | |
| xyang1 | jgriffith: I guess this is because we didn't change the volume itself, just trying to create a backup from it | 23:34 |
| openstackgerrit | Tom Barron proposed openstack/cinder: POC for scalable backup https://review.openstack.org/203291 | 23:34 |
| jgriffith | xyang1: Oh... well I meant the new setting of "previous_state" to None | 23:35 |
| jgriffith | xyang1: Oh.. I see your point | 23:35 |
| xyang1 | jgriffith: Oh, what should we set it? | 23:35 |
| jgriffith | xyang1: well.. I dunno :) | 23:36 |
| jgriffith | xyang1: but if we tried to do a backup of the volume and it failed, might be worth noting that... no? | 23:36 |
| xyang1 | jgriffith: Trying to set it back to original state | 23:36 |
| jgriffith | xyang1: wel.. you're not though | 23:36 |
| xyang1 | jgriffith: The backup itself is error | 23:36 |
| jgriffith | xyang1: so one problem with the previous_state stuff is that its just setting it to "None" | 23:36 |
| jgriffith | xyang1: it's not even setting ti to what it "was" at the beginning of the operation | 23:37 |
| jgriffith | see what I mean? | 23:37 |
| jgriffith | patrickeast: I had a though on your patch there | 23:37 |
| xyang1 | jgriffith: In the beginning it is None, I set it to availabe or inuse before backup | 23:37 |
| jgriffith | patrickeast: any interest/value in considering auto-creation of the internal tenant? | 23:38 |
| *** sghanekar_ has joined #openstack-cinder | 23:38 | |
| xyang1 | jgriffith: So i can set status back to previous_status after backup | 23:38 |
| xyang1 | jgriffith: So the previous state didn't have any t | 23:38 |
| xyang1 | Anthing if we just created a volume | 23:39 |
| jgriffith | xyang1: You do "previous_state = volume['previous_state']" .... | 23:39 |
| xyang1 | jgriffith: Let me check | 23:39 |
| jgriffith | xyang1: then you don't touch it until the exception block.. where you set it to None in the db.volume_update | 23:39 |
| jgriffith | xyang1: line #344 in backup.manager | 23:40 |
| xyang1 | jgriffith: I see what you are saying:) | 23:41 |
| *** vilobhmm has quit IRC | 23:41 | |
| xyang1 | jgriffith: I think I don't need that line | 23:41 |
| patrickeast | jgriffith: i was thinking about that, we could probably do it since it doesn’t really matter what the fields are as long as they are a string | 23:41 |
| xyang1 | jgriffith: Hold on | 23:41 |
| jgriffith | patrickeast: which part? | 23:42 |
| jgriffith | patrickeast: the auto-generate thing? | 23:42 |
| *** haomaiw__ has quit IRC | 23:42 | |
| jgriffith | patrickeast: what I'm saying is you don't even need it in the config | 23:42 |
| patrickeast | jgriffith: yep | 23:42 |
| jgriffith | patrickeast: you just create one.. store it in the DB | 23:42 |
| patrickeast | jgriffith: ohh interesting | 23:42 |
| jgriffith | admins can go to keystone and say "what's the internal-cinder tenant ID" | 23:42 |
| jgriffith | patrickeast: even expose an admin call in Cinder to get it if you want | 23:43 |
| xyang1 | jgriffith: Line 343 set status back to original status, line 344, previous status is no longer needed, so set to none | 23:43 |
| jgriffith | patrickeast: but I don't see much reason for exposing "Internal cinder" details to anybody | 23:43 |
| jgriffith | xyang1: :) | 23:43 |
| patrickeast | jgriffith: so right now the internal tenant doesn’t actually have to be a keystone project or need a real user | 23:43 |
| patrickeast | jgriffith: i kind of like that functionality | 23:43 |
| xyang1 | jgriffith: Sorry it is messy:) | 23:43 |
| jgriffith | but what I'm saying is ; is it not needed? Is there any reason we should not track what it's been through? | 23:43 |
| jgriffith | patrickeast: just a though | 23:44 |
| patrickeast | jgriffith: yea i mean… i like both | 23:44 |
| jgriffith | patrickeast: or even add it to setup/config in the Cinder install guide | 23:44 |
| patrickeast | just not sure how to get there | 23:44 |
| jgriffith | patrickeast: that's the EZ part :) | 23:44 |
| jgriffith | setup/config... or an admin call out to Keystone (the second option is harder) | 23:44 |
| patrickeast | jgriffith: so one thing i was wondering is why can’t it just be the cinder service tenant? | 23:45 |
| jgriffith | patrickeast: I think.. but doable | 23:45 |
| patrickeast | we already have to create one for the service to use, right? | 23:45 |
| jgriffith | patrickeast: I'd have to look at that... that actually might work | 23:45 |
| jgriffith | I just can remember if policy let's that tenant "do" anything | 23:45 |
| jgriffith | worth looking at | 23:45 |
| patrickeast | but thats the nice part of it with how it works now… it doesn’t actually have to do anything other than fill in db columns in cinder | 23:46 |
| xyang1 | jgriffith: Set volume['status'] to previous_status which is available or in use, volume['previous_status'] is no longer needed after backup is complete, so I set it to None | 23:46 |
| patrickeast | we don’t need to be able to do anything outside of that | 23:46 |
| *** haomaiwa_ has joined #openstack-cinder | 23:46 | |
| *** alwaysatthenoc has joined #openstack-cinder | 23:49 | |
| patrickeast | jgriffith: so if we go down the auto create route, are you thinking we add a new table in the db to keep track of it? or is there somewhere already we can stash it? | 23:50 |
| *** raunak has quit IRC | 23:50 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!