| *** setmason has quit IRC | 00:04 | |
| *** laughterwym has joined #openstack-cinder | 00:04 | |
| *** lixiaoy1 has joined #openstack-cinder | 00:06 | |
| *** merooney has joined #openstack-cinder | 00:07 | |
| *** topshare has quit IRC | 00:09 | |
| *** delatte has joined #openstack-cinder | 00:15 | |
| *** delattec has quit IRC | 00:17 | |
| *** dims has quit IRC | 00:21 | |
| *** dims has joined #openstack-cinder | 00:22 | |
| *** adurbin_ has quit IRC | 00:24 | |
| *** bswartz has joined #openstack-cinder | 00:24 | |
| *** _cjones_ has quit IRC | 00:26 | |
| *** _cjones_ has joined #openstack-cinder | 00:26 | |
| *** dims has quit IRC | 00:26 | |
| *** jamielennox is now known as jamielennox|away | 00:29 | |
| *** diogogmt has joined #openstack-cinder | 00:29 | |
| *** _cjones_ has quit IRC | 00:30 | |
| *** jwcroppe has quit IRC | 00:35 | |
| *** jamielennox|away is now known as jamielennox | 00:39 | |
| *** r-daneel has quit IRC | 00:40 | |
| openstackgerrit | Mike Perez proposed openstack/os-brick: Prep for 0.2.1 release https://review.openstack.org/199766 | 00:42 |
|---|---|---|
| *** skylerberg has quit IRC | 00:45 | |
| *** MentalRay has joined #openstack-cinder | 00:48 | |
| *** garthb has quit IRC | 00:49 | |
| *** tsekiyam_ has joined #openstack-cinder | 00:50 | |
| *** topshare has joined #openstack-cinder | 00:52 | |
| *** mtanino has quit IRC | 00:53 | |
| *** tsekiyama has quit IRC | 00:54 | |
| *** tsekiyam_ has quit IRC | 00:55 | |
| *** cdelatte has joined #openstack-cinder | 00:57 | |
| *** delatte has quit IRC | 00:59 | |
| *** jgriffit1 has joined #openstack-cinder | 01:00 | |
| smcginnis | Oh man, this email flood is a pain. Someone's going to have a rough morning tomorrow. | 01:00 |
| patrickeast | haha yea | 01:02 |
| patrickeast | the worst part is that i keep checking it | 01:02 |
| patrickeast | i’m like ‘ooh new review mail' | 01:02 |
| patrickeast | and it just leads to disappointment | 01:02 |
| *** laughterwym_ has joined #openstack-cinder | 01:02 | |
| *** DericHorn-HP has joined #openstack-cinder | 01:03 | |
| *** laughterwym has quit IRC | 01:05 | |
| smcginnis | patrickeast: Hah, same! :) | 01:06 |
| *** jwcroppe has joined #openstack-cinder | 01:06 | |
| *** leeantho has quit IRC | 01:06 | |
| *** garth___ has joined #openstack-cinder | 01:08 | |
| *** raunak has quit IRC | 01:09 | |
| *** merooney has quit IRC | 01:09 | |
| *** jgriffit1 has quit IRC | 01:10 | |
| *** jgriffit1 has joined #openstack-cinder | 01:10 | |
| jgriffit1 | :set theme default | 01:11 |
| *** jwcroppe has quit IRC | 01:17 | |
| asselin | xyang1, hi | 01:23 |
| xyang1 | asselin: Hi | 01:24 |
| asselin | what's up? | 01:24 |
| asselin | "I'll need to ask asselin what is the trick" | 01:24 |
| xyang1 | asselin: We have problem getting brick ci running stable | 01:24 |
| *** zhonghua-li has joined #openstack-cinder | 01:24 | |
| xyang1 | asselin: Only got it working once | 01:24 |
| asselin | ok | 01:25 |
| *** zhonghua-li has quit IRC | 01:25 | |
| xyang1 | asselin: When do cherrypick, it fail instantly | 01:25 |
| asselin | why are you cherrypicking? | 01:25 |
| *** jgriffit1 has quit IRC | 01:26 | |
| xyang1 | asselin: Pretty sure we didn't do it right | 01:26 |
| xyang1 | asselin: What is your procedure | 01:26 |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix restore point if backup base is diff-format in ceph https://review.openstack.org/198993 | 01:26 |
| asselin | we don't cherrypick anything | 01:26 |
| *** zhonghua-li has joined #openstack-cinder | 01:26 | |
| asselin | so i'd like to know more about why you do that, what you're trying to test | 01:27 |
| *** bdr7 has joined #openstack-cinder | 01:27 | |
| patrickeast | xyang1: safe to assume you are not using devstack-gate to run the tests? | 01:27 |
| xyang1 | asselin: We have compute and cinder node separate | 01:28 |
| asselin | xyang1, actually...getting late and I need to go home. Can we take a look tomorrow? patrickeast might also be able to help, we do the same thing | 01:29 |
| xyang1 | asselin: Sure | 01:29 |
| xyang1 | asselin: patrickeast you need to pull the os brick patch in nova | 01:30 |
| patrickeast | xyang1: gotcha, so you are doing a cherry-pick of the commit being reviewed onto the existing brick code on the compute/volume nodes? | 01:31 |
| xyang1 | Right | 01:32 |
| patrickeast | i’m looking at http://storage.superna.net/openstack/scaleio/EMC-os-brick-CI/build-28/console.log where it looks like it worked ok, is there a log of it failing? | 01:32 |
| xyang1 | We have a shim Scaleio libvirt volume driver dependent on the os brick patch | 01:32 |
| patrickeast | does the cherry-pick not apply cleanly? | 01:33 |
| patrickeast | or whats the error? | 01:33 |
| *** Lee1092 has joined #openstack-cinder | 01:33 | |
| xyang1 | patrickeast: It failed right away, no logs published | 01:33 |
| *** DericHorn-HP_ has joined #openstack-cinder | 01:33 | |
| *** DericHorn-HP_ has quit IRC | 01:34 | |
| *** DericHorn-HP has quit IRC | 01:34 | |
| patrickeast | xyang1: oh wait, are you cherry-picking the driver review on top of the review branch, or the review commit on top of the driver branch? | 01:34 |
| xyang1 | patrickeast: So nova side we need to use the shim libvirt driver dependent on hema's os brick patch | 01:34 |
| patrickeast | haha ouch, so you have a bunch of dependencies | 01:35 |
| patrickeast | that makes it hard | 01:35 |
| xyang1 | patrickeast: That is using the os brick connector | 01:35 |
| *** jasondotstar has joined #openstack-cinder | 01:35 | |
| xyang1 | None is merged | 01:35 |
| xyang1 | patrickeast: That is the challenge | 01:35 |
| patrickeast | is the ci blocking them from merging? | 01:36 |
| xyang1 | patrickeast: Yes | 01:36 |
| patrickeast | :o brutal | 01:36 |
| xyang1 | patrickeast: Need to talk to hemna if it is not feasible | 01:36 |
| patrickeast | any chance you can get a log of what the error is when it fails? | 01:36 |
| patrickeast | without that its hard to tell exactly what the problem is | 01:37 |
| xyang1 | patrickeast: I'll have to ask the guy tomorrow, i don't have access to it | 01:37 |
| patrickeast | fwiw i think it would be easier to just get them merged and make sure things are stable with the ci | 01:38 |
| xyang1 | hemnafk: :) | 01:38 |
| patrickeast | a lot of effort to get it working with doing a bunch of cherry-picks/merges just to pre-flight test it | 01:38 |
| patrickeast | easy enough to just revert if it breaks anything too bad, right? | 01:38 |
| xyang1 | patrickeast: Need to persuade hemnafk | 01:39 |
| openstackgerrit | Jinru Yan proposed openstack/cinder: Report capability of multiattach for FlashSystem https://review.openstack.org/199818 | 01:39 |
| xyang1 | patrickeast: It will be easier if it is merged | 01:39 |
| *** rajinir has quit IRC | 01:42 | |
| *** mriedem has quit IRC | 01:42 | |
| *** topshare has quit IRC | 01:43 | |
| *** gouthamr has joined #openstack-cinder | 01:43 | |
| *** gouthamr has quit IRC | 01:43 | |
| patrickeast | xyang1: so looking at the logs of one that worked i would hazard a guess that there will be cases where the cherry-picks will not apply cleanly, which may cause the job to fail | 01:44 |
| *** gouthamr has joined #openstack-cinder | 01:44 | |
| patrickeast | but the ones that failed did it in 3 seconds… which seems too fast to have gotten that far in the job, a lot happens before the cherry-picks | 01:44 |
| *** bitblt has quit IRC | 01:45 | |
| *** vilobhmm has quit IRC | 01:45 | |
| xyang1 | patrickeast: I don't completely understand the problem myself, he told me it cherrypick the scaleio driver. On top of whatever patch is being tested | 01:46 |
| xyang1 | patrickeast: I think I'll have ask for script, then ask you and asselin to take a look | 01:46 |
| xyang1 | patrickeast: Could be the procedure is flawed | 01:46 |
| *** topshare has joined #openstack-cinder | 01:46 | |
| xyang1 | patrickeast: He said it failed because it was the same change | 01:47 |
| *** MentalRay has quit IRC | 01:48 | |
| xyang1 | patrickeast: Which I don't understand why it has to be there before | 01:48 |
| patrickeast | xyang1: hmm, so i ran into something similar when getting our FC job working before the driver was merged, we would cherry-pick the driver on top of the zuul commit, and i do remember having issues with duplicating the commits | 01:51 |
| patrickeast | i had to do some if’s to check what review it was looking at and skip the cherry-pick sometimes | 01:51 |
| patrickeast | since the order mattered for some dependent changes | 01:51 |
| patrickeast | xyang1: but probably your best bet is to get the script and log of it failing, should hopefully be fairly easy to figure out whats going on | 01:52 |
| xyang1 | patrickeast: Sure, let me get the logs and script and ask you again | 01:53 |
| xyang1 | patrickeast: Thanks! | 01:53 |
| patrickeast | np | 01:53 |
| *** gouthamr has quit IRC | 01:53 | |
| *** gouthamr has joined #openstack-cinder | 01:55 | |
| *** MentalRay has joined #openstack-cinder | 01:56 | |
| *** laughterwym has joined #openstack-cinder | 01:59 | |
| *** zhonghua has joined #openstack-cinder | 02:06 | |
| *** zhonghua has quit IRC | 02:06 | |
| *** laughterwym has quit IRC | 02:06 | |
| *** zhonghua-li has quit IRC | 02:09 | |
| *** MentalRay has quit IRC | 02:14 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 02:14 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Fix saving tz aware datetimes in Versioned Objects https://review.openstack.org/196071 | 02:14 |
| *** mutoulbj has joined #openstack-cinder | 02:17 | |
| *** vilobhmm has joined #openstack-cinder | 02:19 | |
| *** annasort has joined #openstack-cinder | 02:20 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 02:23 |
| *** yamada-h has joined #openstack-cinder | 02:26 | |
| *** annashen has joined #openstack-cinder | 02:26 | |
| *** simondodsley has joined #openstack-cinder | 02:27 | |
| *** heyun has joined #openstack-cinder | 02:27 | |
| *** yamada-h has quit IRC | 02:30 | |
| *** xyang has joined #openstack-cinder | 02:30 | |
| *** simondodsley has quit IRC | 02:31 | |
| *** mutoulbj_ has joined #openstack-cinder | 02:33 | |
| *** vilobhmm has quit IRC | 02:33 | |
| *** vilobhmm has joined #openstack-cinder | 02:33 | |
| *** r-daneel has joined #openstack-cinder | 02:35 | |
| *** mutoulbj has quit IRC | 02:36 | |
| *** jogo has joined #openstack-cinder | 02:38 | |
| *** annashen has quit IRC | 02:42 | |
| *** annashen has joined #openstack-cinder | 02:42 | |
| *** annashen has quit IRC | 02:43 | |
| *** asselin_ has joined #openstack-cinder | 02:48 | |
| *** patrickeast has quit IRC | 02:49 | |
| *** gouthamr has quit IRC | 02:50 | |
| *** hakimo has joined #openstack-cinder | 02:52 | |
| *** hakimo_ has quit IRC | 02:54 | |
| *** zhonghua-lee has joined #openstack-cinder | 02:55 | |
| *** r-daneel has quit IRC | 02:55 | |
| *** zhonghua-li has joined #openstack-cinder | 02:55 | |
| *** zhonghua-li has quit IRC | 02:55 | |
| *** xiangfeiz has joined #openstack-cinder | 03:05 | |
| *** bdr7 has quit IRC | 03:06 | |
| *** bdr7 has joined #openstack-cinder | 03:07 | |
| *** davechen has joined #openstack-cinder | 03:07 | |
| *** xyang1 has quit IRC | 03:09 | |
| *** garth___ has quit IRC | 03:10 | |
| openstackgerrit | Kuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key https://review.openstack.org/198614 | 03:11 |
| *** jdandrea has quit IRC | 03:16 | |
| *** annashen has joined #openstack-cinder | 03:17 | |
| *** garthb has joined #openstack-cinder | 03:20 | |
| *** garthb has quit IRC | 03:20 | |
| *** garthb has joined #openstack-cinder | 03:20 | |
| *** garthb_ has joined #openstack-cinder | 03:20 | |
| *** jamielennox is now known as jamielennox|away | 03:22 | |
| *** bdr7 has quit IRC | 03:30 | |
| *** jamielennox|away is now known as jamielennox | 03:31 | |
| *** topshare has quit IRC | 03:32 | |
| *** julim has quit IRC | 03:36 | |
| *** julim has joined #openstack-cinder | 03:37 | |
| *** julim has quit IRC | 03:37 | |
| *** jasondotstar has quit IRC | 03:37 | |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Show backup and volume info in output of backup_restore command https://review.openstack.org/199846 | 03:39 |
| *** asselin__ has joined #openstack-cinder | 03:39 | |
| *** diogogmt has quit IRC | 03:40 | |
| *** rajinir has joined #openstack-cinder | 03:40 | |
| *** asselin_ has quit IRC | 03:43 | |
| *** xiangfeiz has quit IRC | 03:45 | |
| *** dims has joined #openstack-cinder | 03:45 | |
| *** dims_ has joined #openstack-cinder | 03:46 | |
| *** dims_ has quit IRC | 03:46 | |
| *** dims_ has joined #openstack-cinder | 03:46 | |
| *** dims has quit IRC | 03:49 | |
| *** jwcroppe has joined #openstack-cinder | 03:54 | |
| *** xyang has quit IRC | 03:54 | |
| *** xyang has joined #openstack-cinder | 03:55 | |
| *** markvoelker has quit IRC | 04:08 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: improve create and delete operation https://review.openstack.org/197393 | 04:10 |
| *** dims_ has quit IRC | 04:14 | |
| *** dims has joined #openstack-cinder | 04:15 | |
| *** BharatK has joined #openstack-cinder | 04:16 | |
| *** rajinir has quit IRC | 04:18 | |
| *** dims has quit IRC | 04:20 | |
| *** yusuke has joined #openstack-cinder | 04:21 | |
| *** yusuke has left #openstack-cinder | 04:22 | |
| *** sgotliv has joined #openstack-cinder | 04:25 | |
| *** davechen has quit IRC | 04:28 | |
| *** zhonghua-lee has quit IRC | 04:29 | |
| *** vilobhmm has quit IRC | 04:30 | |
| *** shyama has joined #openstack-cinder | 04:31 | |
| *** sgotliv has quit IRC | 04:33 | |
| openstackgerrit | Kuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key https://review.openstack.org/198614 | 04:34 |
| *** vilobhmm has joined #openstack-cinder | 04:34 | |
| openstackgerrit | Kuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key https://review.openstack.org/198614 | 04:39 |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Show backup and volume info in backup_restore https://review.openstack.org/199846 | 04:40 |
| *** mutoulbj_ has quit IRC | 04:46 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:49 | |
| *** annashen has quit IRC | 04:50 | |
| *** setmason has joined #openstack-cinder | 04:51 | |
| *** mutoulbj has joined #openstack-cinder | 04:52 | |
| *** dims has joined #openstack-cinder | 05:01 | |
| *** dims_ has joined #openstack-cinder | 05:02 | |
| *** shausy has joined #openstack-cinder | 05:03 | |
| *** dims has quit IRC | 05:06 | |
| *** shyama has quit IRC | 05:08 | |
| *** shyama has joined #openstack-cinder | 05:09 | |
| *** mutoulbj has quit IRC | 05:11 | |
| *** topshare has joined #openstack-cinder | 05:14 | |
| *** dims_ has quit IRC | 05:14 | |
| *** mutoulbj has joined #openstack-cinder | 05:15 | |
| *** nkrinner has joined #openstack-cinder | 05:16 | |
| *** davechen has joined #openstack-cinder | 05:17 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix restore point if backup base is diff-format in ceph https://review.openstack.org/198993 | 05:21 |
| *** BharatK has quit IRC | 05:25 | |
| *** davechen1 has joined #openstack-cinder | 05:27 | |
| *** davechen has quit IRC | 05:29 | |
| *** setmason_ has joined #openstack-cinder | 05:31 | |
| *** nlevinki has joined #openstack-cinder | 05:33 | |
| *** setmason has quit IRC | 05:34 | |
| *** setmason_ is now known as setmason | 05:34 | |
| *** BharatK has joined #openstack-cinder | 05:39 | |
| *** pots has quit IRC | 05:47 | |
| *** garthb has quit IRC | 05:48 | |
| *** garthb_ has quit IRC | 05:48 | |
| *** vilobhmm has quit IRC | 05:49 | |
| *** ig0r_ has joined #openstack-cinder | 05:52 | |
| *** ig0r__ has quit IRC | 05:55 | |
| *** topshare has quit IRC | 05:55 | |
| *** topshare has joined #openstack-cinder | 05:56 | |
| *** liewegas has quit IRC | 05:58 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Show backup and volume info in backup_restore https://review.openstack.org/199876 | 05:59 |
| *** liewegas has joined #openstack-cinder | 06:00 | |
| *** markvoelker has joined #openstack-cinder | 06:00 | |
| *** setmason has left #openstack-cinder | 06:03 | |
| *** liewegas has quit IRC | 06:03 | |
| *** markvoelker has quit IRC | 06:04 | |
| *** liewegas has joined #openstack-cinder | 06:05 | |
| *** afazekas has joined #openstack-cinder | 06:08 | |
| *** aix has joined #openstack-cinder | 06:09 | |
| *** pradipta has joined #openstack-cinder | 06:09 | |
| *** nlevinki has quit IRC | 06:12 | |
| *** dims has joined #openstack-cinder | 06:15 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: improve snapshot and clone operation https://review.openstack.org/198952 | 06:19 |
| *** dims has quit IRC | 06:20 | |
| *** annashen_ has joined #openstack-cinder | 06:27 | |
| *** setmason has joined #openstack-cinder | 06:28 | |
| *** topshare has quit IRC | 06:31 | |
| *** sgotliv has joined #openstack-cinder | 06:33 | |
| *** deepakcs has joined #openstack-cinder | 06:34 | |
| *** setmason has quit IRC | 06:40 | |
| *** hakimo has quit IRC | 06:40 | |
| *** annashen_ has quit IRC | 06:45 | |
| *** yamada-h has joined #openstack-cinder | 06:51 | |
| *** nihilifer has joined #openstack-cinder | 06:58 | |
| *** yamada-h has quit IRC | 07:05 | |
| openstackgerrit | Jinru Yan proposed openstack/cinder: Report capability of multiattach for FlashSystem https://review.openstack.org/199818 | 07:07 |
| *** Longgeek has quit IRC | 07:10 | |
| *** topshare has joined #openstack-cinder | 07:13 | |
| *** lpetrut has joined #openstack-cinder | 07:13 | |
| *** belmoreira has joined #openstack-cinder | 07:15 | |
| *** yamada-h has joined #openstack-cinder | 07:15 | |
| *** ociuhandu has joined #openstack-cinder | 07:16 | |
| openstackgerrit | wanghao proposed openstack/cinder: Validate value when user update quota https://review.openstack.org/197938 | 07:17 |
| *** tobe has joined #openstack-cinder | 07:18 | |
| *** lpetrut has quit IRC | 07:19 | |
| *** Miouge has joined #openstack-cinder | 07:20 | |
| *** belmoreira has quit IRC | 07:21 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 07:21 |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 07:23 |
| *** harlowja_at_home has quit IRC | 07:24 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Remove unused arguments from c-vol's create_volume https://review.openstack.org/199590 | 07:25 |
| *** sgotliv has quit IRC | 07:28 | |
| *** belmoreira has joined #openstack-cinder | 07:28 | |
| *** jwcroppe has quit IRC | 07:29 | |
| *** abehl has joined #openstack-cinder | 07:34 | |
| *** smoriya has joined #openstack-cinder | 07:50 | |
| *** Longgeek has joined #openstack-cinder | 07:50 | |
| *** ronis has joined #openstack-cinder | 07:51 | |
| *** bkopilov has quit IRC | 07:52 | |
| *** sgotliv has joined #openstack-cinder | 07:54 | |
| *** yamada-h has quit IRC | 07:54 | |
| *** bluex-pl has joined #openstack-cinder | 07:59 | |
| *** bluex-pl has quit IRC | 08:00 | |
| *** bluex-pl has joined #openstack-cinder | 08:00 | |
| *** markvoelker has joined #openstack-cinder | 08:01 | |
| *** bkopilov has joined #openstack-cinder | 08:04 | |
| *** jasondotstar has joined #openstack-cinder | 08:04 | |
| *** lpetrut has joined #openstack-cinder | 08:05 | |
| *** markvoelker has quit IRC | 08:06 | |
| *** jistr has joined #openstack-cinder | 08:09 | |
| *** bkopilov has quit IRC | 08:12 | |
| *** lpetrut has left #openstack-cinder | 08:14 | |
| *** dims has joined #openstack-cinder | 08:16 | |
| *** shyama_ has joined #openstack-cinder | 08:19 | |
| *** shyama has quit IRC | 08:20 | |
| *** shyama_ is now known as shyama | 08:20 | |
| *** dims has quit IRC | 08:21 | |
| *** ndipanov has joined #openstack-cinder | 08:21 | |
| *** bkopilov has joined #openstack-cinder | 08:24 | |
| *** Longgeek has quit IRC | 08:25 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add tenant validation to quota https://review.openstack.org/143645 | 08:26 |
| *** laughterwym_ has quit IRC | 08:26 | |
| winston-d_ | wanghao: hey, I left some comment to your change about quota modification. | 08:26 |
| *** jasondotstar has quit IRC | 08:29 | |
| *** Longgeek has joined #openstack-cinder | 08:34 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: [WIP] Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 08:38 |
| *** xyang has quit IRC | 08:43 | |
| *** bkopilov has quit IRC | 08:44 | |
| *** ankit has joined #openstack-cinder | 08:48 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix properties extracting from image with glance api v2 https://review.openstack.org/169430 | 08:50 |
| *** jasondotstar has joined #openstack-cinder | 08:53 | |
| aarefiev | winston-d_: DuncanT: I've removed check tenant id from show, could you look guys https://review.openstack.org/#/c/143645/19 , please ? | 08:55 |
| *** kjelly has joined #openstack-cinder | 08:55 | |
| *** haomaiwang has joined #openstack-cinder | 08:57 | |
| *** haomaiw__ has quit IRC | 09:00 | |
| *** bkopilov has joined #openstack-cinder | 09:00 | |
| openstackgerrit | Pradeep Sathasivam proposed openstack/cinder: Adds VF support with HTTPS southbound connector https://review.openstack.org/184799 | 09:05 |
| *** topshare has quit IRC | 09:11 | |
| *** nihilifer has quit IRC | 09:13 | |
| *** coolsvap|away is now known as coolsvap | 09:15 | |
| *** jordanP has joined #openstack-cinder | 09:16 | |
| *** lpetrut has joined #openstack-cinder | 09:19 | |
| *** lpetrut1 has joined #openstack-cinder | 09:27 | |
| *** e0ne has joined #openstack-cinder | 09:28 | |
| *** lpetrut has quit IRC | 09:29 | |
| deepakcs | Is something broken in cinder quota for multi-backend ? I tried to set quota for my volume_type but it doesn't seem to work. In-use is always 0 even when i create some volumes | 09:36 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Remove unused arguments from c-vol's create_volume https://review.openstack.org/199590 | 09:47 |
| *** coolsvap is now known as coolsvap|away | 09:48 | |
| *** lixiaoy1 has quit IRC | 09:52 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: IBM storwize driver configuration item default valume change https://review.openstack.org/198692 | 09:53 |
| *** davechen1 has left #openstack-cinder | 09:54 | |
| *** nihilifer has joined #openstack-cinder | 09:55 | |
| *** markvoelker has joined #openstack-cinder | 10:02 | |
| *** markvoelker has quit IRC | 10:06 | |
| *** coolsvap|away is now known as coolsvap | 10:11 | |
| *** dims has joined #openstack-cinder | 10:17 | |
| *** dims has quit IRC | 10:21 | |
| *** mutoulbj has quit IRC | 10:22 | |
| *** e0ne is now known as e0ne_ | 10:24 | |
| *** coolsvap is now known as coolsvap|away | 10:25 | |
| *** pradipta has quit IRC | 10:27 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 10:27 |
| *** heyun has quit IRC | 10:29 | |
| *** e0ne_ is now known as e0ne | 10:31 | |
| *** yuriy_n17 has joined #openstack-cinder | 10:34 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient https://review.openstack.org/197643 | 10:39 |
| *** timcl has joined #openstack-cinder | 10:43 | |
| *** IanGovett has joined #openstack-cinder | 10:45 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: Provide backup service that uses ProtecTIER as the backend https://review.openstack.org/153969 | 10:49 |
| *** bkopilov has quit IRC | 10:49 | |
| *** e0ne is now known as e0ne_ | 10:51 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: [WIP] Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 10:57 |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: Provide backup service that uses ProtecTIER as the backend https://review.openstack.org/153969 | 10:58 |
| *** Longgeek has quit IRC | 10:59 | |
| *** bkopilov has joined #openstack-cinder | 11:00 | |
| *** mutoulbj has joined #openstack-cinder | 11:07 | |
| *** Longgeek has joined #openstack-cinder | 11:14 | |
| *** cdelatte has quit IRC | 11:16 | |
| *** mutoulbj has quit IRC | 11:17 | |
| *** mutoulbj has joined #openstack-cinder | 11:18 | |
| *** coolsvap|away is now known as coolsvap | 11:22 | |
| openstackgerrit | Pradeep Sathasivam proposed openstack/cinder: Adds VF support with HTTPS southbound connector https://review.openstack.org/184799 | 11:32 |
| *** Miouge_ has joined #openstack-cinder | 11:32 | |
| *** markvoelker has joined #openstack-cinder | 11:33 | |
| *** Miouge has quit IRC | 11:35 | |
| *** Miouge_ is now known as Miouge | 11:35 | |
| *** cdelatte has joined #openstack-cinder | 11:36 | |
| *** 32NACCGVG has joined #openstack-cinder | 11:36 | |
| *** markvoelker has quit IRC | 11:37 | |
| *** mutoulbj has quit IRC | 11:44 | |
| *** jaypipes has joined #openstack-cinder | 11:45 | |
| *** tobe has quit IRC | 11:48 | |
| openstackgerrit | Victor Stinner proposed openstack/os-brick: Cleanup Python 3 changes https://review.openstack.org/197624 | 11:48 |
| haypo | lifeless, hemnafk ^^ after a good sleep, i updated my patch ;) | 11:49 |
| haypo | lifeless, hemnafk: sorry for last night, i was tired | 11:49 |
| *** zhenguo has quit IRC | 11:50 | |
| *** e0ne_ is now known as e0ne | 11:50 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 11:51 |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 11:51 |
| *** xyang has joined #openstack-cinder | 11:54 | |
| lifeless | haypo: I'd already updated it :/ | 11:55 |
| lifeless | haypo: did you incorporate my changes? | 11:55 |
| *** xyang has quit IRC | 11:56 | |
| haypo | lifeless: did you see https://review.openstack.org/#/c/197624/4/README.rst ? | 11:57 |
| lifeless | haypo: ah you did, cool. | 11:57 |
| lifeless | haypo: no, I hadn't | 11:57 |
| haypo | lifeless: my patch is based on yours | 11:58 |
| *** delattec has joined #openstack-cinder | 12:00 | |
| *** delatte has joined #openstack-cinder | 12:00 | |
| *** 32NACCGVG has quit IRC | 12:00 | |
| *** cdelatte has quit IRC | 12:01 | |
| *** lpabon has joined #openstack-cinder | 12:02 | |
| *** e0ne is now known as e0ne_ | 12:02 | |
| *** e0ne_ is now known as e0ne | 12:09 | |
| *** markvoelker has joined #openstack-cinder | 12:12 | |
| *** annegentle has joined #openstack-cinder | 12:15 | |
| *** dims has joined #openstack-cinder | 12:17 | |
| openstackgerrit | Ralf Haferkamp proposed openstack/cinder: Fix block_device driver to behave as documented https://review.openstack.org/200039 | 12:19 |
| *** Miouge has quit IRC | 12:20 | |
| *** Miouge has joined #openstack-cinder | 12:21 | |
| *** dims has quit IRC | 12:22 | |
| *** yuriy_n17 has quit IRC | 12:31 | |
| *** aarefiev has quit IRC | 12:31 | |
| *** e0ne has quit IRC | 12:32 | |
| *** kjnelson has joined #openstack-cinder | 12:33 | |
| *** edmondsw has joined #openstack-cinder | 12:37 | |
| *** annegentle has quit IRC | 12:38 | |
| *** jasondotstar has quit IRC | 12:38 | |
| *** dims has joined #openstack-cinder | 12:40 | |
| *** bswartz has quit IRC | 12:41 | |
| *** mutoulbj has joined #openstack-cinder | 12:41 | |
| *** julim has joined #openstack-cinder | 12:44 | |
| *** akerr has joined #openstack-cinder | 12:45 | |
| *** simondodsley has joined #openstack-cinder | 12:45 | |
| *** mildan has joined #openstack-cinder | 12:45 | |
| *** mutoulbj has quit IRC | 12:46 | |
| *** mutoulbj has joined #openstack-cinder | 12:46 | |
| *** pserebryakov has joined #openstack-cinder | 12:47 | |
| *** Longgeek has quit IRC | 12:49 | |
| *** ronis has quit IRC | 12:54 | |
| *** xiangfeiz has joined #openstack-cinder | 12:55 | |
| *** xiangfeiz has left #openstack-cinder | 12:56 | |
| *** jaypipes has quit IRC | 12:56 | |
| *** jasondotstar has joined #openstack-cinder | 12:58 | |
| *** merooney has joined #openstack-cinder | 12:59 | |
| *** yuriy_n17 has joined #openstack-cinder | 12:59 | |
| *** aarefiev has joined #openstack-cinder | 12:59 | |
| *** e0ne has joined #openstack-cinder | 13:00 | |
| *** diogogmt has joined #openstack-cinder | 13:02 | |
| *** abehl has quit IRC | 13:03 | |
| *** topshare has joined #openstack-cinder | 13:06 | |
| *** topshare has quit IRC | 13:06 | |
| *** topshare has joined #openstack-cinder | 13:07 | |
| *** topshare has quit IRC | 13:07 | |
| *** topshare has joined #openstack-cinder | 13:08 | |
| *** zhenguo has joined #openstack-cinder | 13:09 | |
| *** BharatK has quit IRC | 13:10 | |
| *** Miouge has quit IRC | 13:10 | |
| *** jaypipes has joined #openstack-cinder | 13:10 | |
| *** mriedem has joined #openstack-cinder | 13:12 | |
| *** Miouge has joined #openstack-cinder | 13:15 | |
| *** shausy has quit IRC | 13:16 | |
| *** Longgeek has joined #openstack-cinder | 13:17 | |
| *** Yogi1 has joined #openstack-cinder | 13:17 | |
| *** Miouge has quit IRC | 13:17 | |
| *** Miouge has joined #openstack-cinder | 13:18 | |
| *** akshai has joined #openstack-cinder | 13:23 | |
| *** primechuck has joined #openstack-cinder | 13:24 | |
| *** sgotliv has quit IRC | 13:27 | |
| *** edmondsw has quit IRC | 13:28 | |
| *** eharney has joined #openstack-cinder | 13:28 | |
| *** deepakcs has quit IRC | 13:29 | |
| *** diogogmt has quit IRC | 13:31 | |
| *** rushil has joined #openstack-cinder | 13:33 | |
| *** dustins has joined #openstack-cinder | 13:34 | |
| *** bswartz has joined #openstack-cinder | 13:36 | |
| *** jungleboyj has quit IRC | 13:40 | |
| *** gouthamr has joined #openstack-cinder | 13:43 | |
| *** jdandrea has joined #openstack-cinder | 13:43 | |
| *** mildan has quit IRC | 13:46 | |
| *** rushil has quit IRC | 13:48 | |
| *** setmason has joined #openstack-cinder | 13:50 | |
| *** MentalRay has joined #openstack-cinder | 13:54 | |
| *** xyang1 has joined #openstack-cinder | 13:55 | |
| *** mtanino has joined #openstack-cinder | 13:59 | |
| *** pserebryakov has quit IRC | 14:00 | |
| *** rushil has joined #openstack-cinder | 14:01 | |
| *** mtanino has quit IRC | 14:08 | |
| *** edmondsw has joined #openstack-cinder | 14:09 | |
| *** Tross has joined #openstack-cinder | 14:10 | |
| *** annegentle has joined #openstack-cinder | 14:12 | |
| *** mtanino has joined #openstack-cinder | 14:13 | |
| *** jwcroppe has joined #openstack-cinder | 14:15 | |
| *** merooney has quit IRC | 14:15 | |
| smcginnis | DuncanT: Related to yesterday's conversation - maybe cinder needs a tools directory: https://github.com/openstack/neutron/blob/master/tools/abandon_old_reviews.sh | 14:16 |
| *** setmason has quit IRC | 14:17 | |
| *** annasort has quit IRC | 14:17 | |
| DuncanT | smcginnis: Good idea. | 14:19 |
| jordanP | jgriffith, fyi: https://review.openstack.org/#/c/200108/ | 14:20 |
| *** setmason has joined #openstack-cinder | 14:23 | |
| *** jungleboyj has joined #openstack-cinder | 14:28 | |
| jgriffith | jordanP: nice | 14:33 |
| jgriffith | jordanP: thanks for that! | 14:33 |
| *** rushil has quit IRC | 14:34 | |
| *** bdr7 has joined #openstack-cinder | 14:34 | |
| *** jwcroppe has quit IRC | 14:34 | |
| *** rushil has joined #openstack-cinder | 14:35 | |
| *** jwcroppe has joined #openstack-cinder | 14:35 | |
| *** rushil has quit IRC | 14:36 | |
| *** nihilifer has quit IRC | 14:39 | |
| *** bluex-pl has quit IRC | 14:39 | |
| *** thangp has joined #openstack-cinder | 14:39 | |
| *** bluex-pl has joined #openstack-cinder | 14:40 | |
| *** Yogi1 has quit IRC | 14:41 | |
| *** ganso_ has joined #openstack-cinder | 14:42 | |
| *** nkrinner has quit IRC | 14:42 | |
| *** ankit has quit IRC | 14:43 | |
| *** dims has quit IRC | 14:45 | |
| *** xyang has joined #openstack-cinder | 14:48 | |
| *** bdr7 has quit IRC | 14:48 | |
| *** setmason has quit IRC | 14:48 | |
| *** bdr7 has joined #openstack-cinder | 14:48 | |
| *** shausy has joined #openstack-cinder | 14:50 | |
| *** dims has joined #openstack-cinder | 14:50 | |
| sdague | jgriffith: can you check out this revert - https://review.openstack.org/#/c/200134/ - it looks like cinder is failing to shutdown with a single kill reliably after that change merged | 14:51 |
| sdague | so the grenade jobs are failing at a pretty high rate now | 14:52 |
| jgriffith | sdague: looking... | 14:52 |
| jgriffith | DuncanT: will you finish it | 14:53 |
| xyang | hemnafk: ScaleIO os-brick CI is reporting success now: https://review.openstack.org/#/c/185835/ | 14:53 |
| jgriffith | DuncanT: rather than slam me for ninjaing the patch later, you can review and give it +A | 14:53 |
| DuncanT | jgriffith: Certainly, looking now | 14:54 |
| jgriffith | Why did we backport that anyway :( | 14:54 |
| jgriffith | I'm the dummy that approved it of course | 14:54 |
| mtreinish | oh, I shouldn't have pushed the revert because I could +A it... | 14:54 |
| mtreinish | jgriffith: heh, I was going to say :) | 14:54 |
| *** tsekiyama has joined #openstack-cinder | 14:55 | |
| *** annasort has joined #openstack-cinder | 14:55 | |
| * jgriffith plays the sad trombone music | 14:55 | |
| *** lpabon has quit IRC | 14:55 | |
| *** bdr7 has quit IRC | 14:55 | |
| asselin__ | xyang, hi, did you get your issues resolved? | 14:56 |
| *** bdr7 has joined #openstack-cinder | 14:56 | |
| *** bdr7 has quit IRC | 14:56 | |
| DuncanT | jgriffith: Actually I don't have +2 on stable, but since it is to fix the gate I wouldn't object to it being pushed through. | 14:56 |
| DuncanT | jgriffith: I should probably get +2 on stable and be done with it | 14:56 |
| sdague | oh, I do have +2 on stable, given jgriffith and DuncanT are cool with it, approving now | 14:57 |
| DuncanT | sdague: Thanks | 14:58 |
| *** annegentle has quit IRC | 14:58 | |
| xyang | asselin__: so I finally understood what is the problem | 14:59 |
| asselin__ | xyang, ok cool | 15:00 |
| xyang | asselin__: we run CI against every patch in os-brick. because our connector is not merged, we try to cherry pick the connector patch and run against every os-brick patch | 15:00 |
| *** topshare has quit IRC | 15:01 | |
| xyang | asselin__: it will fail on the scaleio os-brick connector patch because it is trying to do a merge on the same patch:) | 15:01 |
| *** topshare has joined #openstack-cinder | 15:01 | |
| xyang | asselin__: so now we just stopped the merge, that will only work on the scaleio os-brick connector patch | 15:01 |
| asselin__ | xyang, interesting...I would have thought the cherry-pick would succeed as a noop | 15:01 |
| xyang | asselin__: it will fail on other patches because the scaleio os-brick connector is not merged yet | 15:01 |
| xyang | asselin__: once it is merged, we shouldn't have this problem any more | 15:02 |
| *** lpabon has joined #openstack-cinder | 15:02 | |
| xyang | asselin__: so yesterday our CI actually reported success on a couple of other os-brick patches, but just failed on our own patch:) | 15:03 |
| xyang | asselin__: did I explain it clearly? :) | 15:03 |
| asselin__ | xyang, did you try this? It was working for us: https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#How_do_I_run_my_CI_to_test_all_cinder_patches_with_my_driver_not_yet_merged.3F | 15:03 |
| *** topshare has quit IRC | 15:03 | |
| asselin__ | xyang, yes, but still not clear why cherry-picking failed...it is supposed to work | 15:04 |
| xyang | asselin__: because it is empty | 15:04 |
| asselin__ | xyang, supposed to work on all patches | 15:04 |
| *** msno has joined #openstack-cinder | 15:04 | |
| asselin__ | what is empty? | 15:04 |
| msno | Hi guyz | 15:04 |
| xyang | asselin__: so we are running test on the scaleio patch and it is cherry picking the scaleio patch | 15:05 |
| msno | one of my cinder test case is failing without generating any logs in cinder | 15:05 |
| xyang | asselin__: there's no difference to merge | 15:05 |
| msno | volume test olume_create_get_update_delete_from_image | 15:05 |
| msno | this is the test | 15:05 |
| xyang | asselin__: I guess it is like you can't submit a patch which is exactly the same as the existing one | 15:05 |
| msno | its a Juno based setup | 15:05 |
| msno | anyone have faced this issue | 15:06 |
| asselin__ | xyang, ok...not sure what you're doing differently, but we've gotten that to work for all patches | 15:06 |
| xyang | asselin__: you don't have an os-brick connector patch un-merged | 15:07 |
| asselin__ | xyang, we've only run into issues when there are unresolvable merge conflicts | 15:07 |
| *** topshare has joined #openstack-cinder | 15:07 | |
| *** changbl has quit IRC | 15:07 | |
| asselin__ | xyang, no, but we did the same for unmerged cinder driver | 15:07 |
| xyang | asselin__: Cinder is different because it is just one separate file | 15:07 |
| xyang | asselin__: this connector.py is shared | 15:08 |
| asselin__ | xyang, ok, I see, so it fails due to merge conflict during cherrypick | 15:08 |
| xyang | asselin__: yes | 15:08 |
| asselin__ | xyang, ok, now I understand | 15:09 |
| xyang | asselin__: okay, good | 15:09 |
| xyang | asselin__: it was very confusing for me too | 15:09 |
| asselin__ | xyang, well no...I take that back | 15:09 |
| *** diogogmt has joined #openstack-cinder | 15:09 | |
| *** coolsvap is now known as coolsvap|away | 15:09 | |
| asselin__ | xyang, why would there be a merge conflict when you cherrypick the same patch on top of itself? | 15:09 |
| xyang | asselin__: https://review.openstack.org/#/c/185835/ So we run CI against this patch, in the same time, we do cherry pick | 15:09 |
| xyang | asselin__: it failed because there's nothing to merge | 15:10 |
| asselin__ | xyang, why should that fail? | 15:10 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add tenant validation to quota https://review.openstack.org/143645 | 15:10 |
| xyang | asselin__: I don't know. that is all I heard. I know that I can't submit a patch that is the same as the existing one | 15:10 |
| asselin__ | xyang, right, but this is different...this is during the setup of the os-brick repo | 15:11 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add tenant validation to quota https://review.openstack.org/143645 | 15:11 |
| asselin__ | xyang, if you checkout os-brick to pacth 185835, then cherry-pick 185835 it would be ok | 15:12 |
| asselin__ | then you run your test | 15:12 |
| xyang | asselin__: let's get the exact error and show you | 15:12 |
| xyang | asselin__: all I heard is this fails every time | 15:13 |
| asselin__ | xyang, understood...many people are using the instructions on that wiki page for unmerged patches successfully. | 15:14 |
| xyang | asselin__: we didn't have this problem with Cinder CI before our driver was merged | 15:16 |
| xyang | asselin__: the change set is empty so nothing to merge | 15:16 |
| asselin__ | xyang, so why fail? | 15:16 |
| xyang | asselin__: I'll have to get the error message and show you | 15:17 |
| asselin__ | ok :) | 15:17 |
| *** bdr768 has joined #openstack-cinder | 15:19 | |
| *** rushil has joined #openstack-cinder | 15:20 | |
| * jgriffith needs more coffee | 15:21 | |
| *** sgotliv has joined #openstack-cinder | 15:21 | |
| e0ne | jgriffith: https://igfasouza.files.wordpress.com/2014/02/programmers_coffee01.jpg ;) | 15:22 |
| *** bdr768 has quit IRC | 15:22 | |
| *** haypo has quit IRC | 15:22 | |
| jgriffith | e0ne: Nice~ | 15:22 |
| *** jamielennox is now known as jamielennox|away | 15:23 | |
| *** bkopilov has quit IRC | 15:25 | |
| *** bkopilov has joined #openstack-cinder | 15:26 | |
| *** jgriffith has left #openstack-cinder | 15:27 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: LVM Thin Provisioning auto-detect https://review.openstack.org/104653 | 15:27 |
| openstackgerrit | Ralf Haferkamp proposed openstack/cinder: Fix block_device driver to behave as documented https://review.openstack.org/200039 | 15:30 |
| *** dims has quit IRC | 15:30 | |
| *** jamielennox|away is now known as jamielennox | 15:32 | |
| *** dims has joined #openstack-cinder | 15:32 | |
| *** dims has quit IRC | 15:32 | |
| eharney | ooh we've crossed 200k reviews ^ | 15:32 |
| *** e0ne is now known as e0ne_ | 15:32 | |
| *** dims has joined #openstack-cinder | 15:33 | |
| *** Miouge has quit IRC | 15:35 | |
| smcginnis | eharney: Pop the champagne. :) | 15:35 |
| *** bdr768 has joined #openstack-cinder | 15:35 | |
| *** bdr768 is now known as jgriffith | 15:35 | |
| *** shyama has quit IRC | 15:35 | |
| *** diogogmt has quit IRC | 15:35 | |
| eharney | smcginnis: attempted as directed. champagne seems to have broken jenkins. | 15:36 |
| eharney | :) | 15:36 |
| *** crose has joined #openstack-cinder | 15:36 | |
| *** bkopilov has quit IRC | 15:36 | |
| *** shausy has quit IRC | 15:36 | |
| *** diogogmt has joined #openstack-cinder | 15:37 | |
| xyang | asselin__: git fetch https://review.openstack.org/openstack/os-brick refs/changes/35/185835/13 && git checkout FETCH_HEAD | 15:38 |
| *** garthb_ has joined #openstack-cinder | 15:38 | |
| *** garthb has joined #openstack-cinder | 15:38 | |
| xyang | asselin__: git fetch {Zuul change} && git cherry-pick FETCH_HEAD | 15:38 |
| smcginnis | :) | 15:38 |
| xyang | asselin__: this failed if Zuul change is 185835 | 15:39 |
| asselin__ | xyang, it should be the other way around | 15:39 |
| xyang | asselin__: is it? | 15:39 |
| *** rushil has quit IRC | 15:39 | |
| asselin__ | get the zuul change first, then cherry-pick your os-brick connector | 15:40 |
| xyang | asselin__: I have two script there | 15:40 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter cgsnapshots data on the DB side https://review.openstack.org/197163 | 15:40 |
| xyang | asselin__: run 1 first and then 2 | 15:40 |
| asselin__ | the zuul change should be git checkout, and the os-brick connector should be cherry-pick | 15:41 |
| *** vokt has joined #openstack-cinder | 15:41 | |
| xyang | asselin__: (1) git fetch https://review.openstack.org/openstack/os-brick refs/changes/35/185835/13 && git checkout FETCH_HEAD | 15:41 |
| xyang | asselin__: (2) git fetch {Zuul change} && git cherry-pick FETCH_HEAD | 15:41 |
| asselin__ | xyang, no | 15:42 |
| asselin__ | 1: git fetch {Zuul change} && git checkout FETCH_HEAD | 15:42 |
| asselin__ | 2: git fetch https://review.openstack.org/openstack/os-brick refs/changes/35/185835/13 && git cherry-pick FETCH_HEAD | 15:42 |
| xyang | asselin__: oh, okay | 15:42 |
| *** setmason has joined #openstack-cinder | 15:43 | |
| xyang | asselin__: we can give that a try | 15:43 |
| xyang | asselin__: thanks! | 15:43 |
| asselin__ | xyang, let me know how it goes | 15:43 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter cgsnapshots data on the DB side https://review.openstack.org/197163 | 15:47 |
| *** e0ne_ is now known as e0ne | 15:48 | |
| *** daneyon has joined #openstack-cinder | 15:48 | |
| *** belmoreira has quit IRC | 15:48 | |
| rhe00_ | is there a way to reverse a change once it is merged? I would like to add back the X-IO driver to get the driver and related changes back in so I can test that our CI is working properly with the driver in tree. The change to remove it was https://review.openstack.org/#/c/198741/ | 15:49 |
| *** ronis has joined #openstack-cinder | 15:50 | |
| smcginnis | rhe00_: I believe you need to submit a patch adding it back in, then cherry pick that change on top of whatever other patches you want to run against. | 15:51 |
| *** bkopilov has joined #openstack-cinder | 15:52 | |
| *** Miouge has joined #openstack-cinder | 15:52 | |
| rhe00_ | smcginnis: ok, I'll try that route. thanks! | 15:52 |
| *** jaypipes has quit IRC | 15:53 | |
| *** yuriy_n17 has quit IRC | 15:53 | |
| e0ne | rhe00_: afair, we don't accept any more drivers for Liberty | 15:54 |
| *** coolsvap|away is now known as coolsvap | 15:54 | |
| smcginnis | e0ne: This ones different. It's just an undo of a removal that thingee said would get pushed through once I looked good again. | 15:55 |
| smcginnis | ~ /I looked/it looked/ | 15:56 |
| e0ne | smcginnis: thanks for clarification. i din't know about it | 15:57 |
| *** shyama has joined #openstack-cinder | 15:58 | |
| *** setmason has quit IRC | 16:00 | |
| *** harlowja_at_home has joined #openstack-cinder | 16:01 | |
| mgagne | I see there is no way for Cinder to override the auth_url when using privileged_user with novaclient: https://github.com/openstack/cinder/blob/master/cinder/compute/nova.py#L107-L123 Any suggestion on how to fix that? | 16:02 |
| *** jgriffith1 has joined #openstack-cinder | 16:02 | |
| *** jgriffith1 is now known as bdr78 | 16:03 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add tenant validation to quota https://review.openstack.org/143645 | 16:03 |
| *** bitblt has joined #openstack-cinder | 16:03 | |
| *** setmason has joined #openstack-cinder | 16:03 | |
| *** rushil has joined #openstack-cinder | 16:03 | |
| *** Yogi1 has joined #openstack-cinder | 16:05 | |
| *** dims_ has joined #openstack-cinder | 16:05 | |
| *** leeantho has joined #openstack-cinder | 16:06 | |
| *** dims has quit IRC | 16:09 | |
| *** rushil has quit IRC | 16:10 | |
| *** agarwch has joined #openstack-cinder | 16:11 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add entry create and cast tasks to manage workflow https://review.openstack.org/139071 | 16:12 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix lvm manage existing volume https://review.openstack.org/156939 | 16:12 |
| *** Miouge has quit IRC | 16:13 | |
| *** _cjones_ has joined #openstack-cinder | 16:16 | |
| *** afazekas has quit IRC | 16:19 | |
| *** hemnafk is now known as hemna | 16:19 | |
| xyang | e0ne: ping | 16:20 |
| e0ne | xyang: hi | 16:20 |
| e0ne | xyang: i'm looking on your patch right now:). | 16:20 |
| xyang | e0ne: can you take a look of this patch to see if I've addressed your comments? https://review.openstack.org/#/c/193937/ | 16:20 |
| *** jamielennox is now known as jamielennox|away | 16:21 | |
| e0ne | xyang: i found new issue, but it's not releted to your patch | 16:21 |
| xyang | e0ne: what is it | 16:21 |
| e0ne | xyang: https://github.com/openstack/cinder/blob/master/cinder/objects/volume.py#L126: IMO, we missed return here | 16:21 |
| *** topshare has quit IRC | 16:22 | |
| agarwch | hemna: Need to discuss on the comments on https://review.openstack.org/#/c/198400/ | 16:22 |
| xyang | e0ne: oh | 16:22 |
| *** boris-42 has quit IRC | 16:22 | |
| e0ne | xyang: because of it, here line #833 https://review.openstack.org/#/c/193937/19/cinder/volume/driver.py you need to call db once more | 16:22 |
| *** Tross has quit IRC | 16:22 | |
| e0ne | xyang: i'm working on patch | 16:22 |
| *** Miouge has joined #openstack-cinder | 16:22 | |
| *** abehl has joined #openstack-cinder | 16:22 | |
| xyang | e0ne: ok, let me know how it goes | 16:22 |
| hemna | agarwch, ok just a minute, I'm busy | 16:23 |
| *** harlowja_at_home has quit IRC | 16:23 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix lvm manage existing volume https://review.openstack.org/156939 | 16:24 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Return object after create https://review.openstack.org/200172 | 16:25 |
| e0ne | xyang: ^^ | 16:25 |
| e0ne | xyang: continue to review your patch | 16:25 |
| xyang | e0ne: sure, thanks | 16:25 |
| xyang | e0ne: let me know if you find other issues | 16:26 |
| e0ne | xyang: sure, i'll do | 16:26 |
| *** BharatK has joined #openstack-cinder | 16:27 | |
| thangp | e0ne: I put a comment in your patch https://review.openstack.org/#/c/200172/ | 16:28 |
| thangp | e0ne: I really do not see why we need to return the object | 16:28 |
| e0ne | thangp: thanks for fast feedback | 16:28 |
| thangp | e0ne: maybe i am not seeing something :/ | 16:28 |
| *** bdr78 has quit IRC | 16:29 | |
| *** jamielennox|away is now known as jamielennox | 16:29 | |
| e0ne | thangp: we need to return it to get id in new object | 16:29 |
| e0ne | thangp: in the current implementation, we need 2(!) db requests https://review.openstack.org/#/c/193937/19/cinder/volume/driver.py (line #835) instead of 1 | 16:30 |
| e0ne | thangp: in a current implementation, there is no reason to call '_from_db_object' in create | 16:30 |
| thangp | e0ne: hmm...looking | 16:31 |
| *** garthb_ has quit IRC | 16:31 | |
| e0ne | thangp: imo, i need to fix commit message and patch nova too:) | 16:32 |
| *** BharatK has left #openstack-cinder | 16:32 | |
| thangp | e0ne: if all we need is to set the id, why not just assign it in create()? | 16:32 |
| *** jgriffith1 has joined #openstack-cinder | 16:32 | |
| *** BharatK has joined #openstack-cinder | 16:33 | |
| thangp | e0ne: self.id = db_snapshot['id'] | 16:33 |
| e0ne | thangp: because 'id' is only example. after creationd in DB other fields could changes too | 16:33 |
| thangp | e0ne: the fields should be the same | 16:33 |
| thangp | e0ne: can we do self.update(db_snapshot)? | 16:35 |
| *** annegentle has joined #openstack-cinder | 16:35 | |
| *** mgarza_ has joined #openstack-cinder | 16:35 | |
| thangp | e0ne: that would update what is returned | 16:35 |
| *** jgriffith1 has quit IRC | 16:35 | |
| *** coolsvap is now known as coolsvap|away | 16:36 | |
| thangp | e0ne: because the obj that was instantiated could be used elsewhere | 16:36 |
| thangp | e0ne: and it may not have the updated fields | 16:36 |
| *** ndipanov has quit IRC | 16:36 | |
| thangp | e0ne: by using self.update() that obj would match the db | 16:36 |
| thangp | e0ne: then we would not have to reassign | 16:37 |
| *** jistr has quit IRC | 16:37 | |
| thangp | e0ne: e.g. obj = obj.create() | 16:37 |
| *** annashen has joined #openstack-cinder | 16:38 | |
| e0ne | thangp: i need to verify it. i coulf be wrong | 16:39 |
| agarwch | Where can i find the current progress on the nova code for shared volume | 16:39 |
| e0ne | thangp: i marked patch as WIP. i'll update it and abandone or change commit message later today | 16:40 |
| thangp | e0ne: from a quick look, you might be right :) it's just another way to fix it | 16:40 |
| thangp | e0ne: just requires us to make less changes | 16:41 |
| thangp | e0ne: thanks | 16:41 |
| e0ne | thangp: we pass 'self' to '_from_db_object' in 'create'. | 16:41 |
| e0ne | thangp: so it could work | 16:41 |
| thangp | yup | 16:41 |
| *** jgriffith1 has joined #openstack-cinder | 16:41 | |
| e0ne | thangp: need to head hope. bb and test it in ~2 hours | 16:41 |
| thangp | and it is a dict that db.*_create() returns | 16:42 |
| *** e0ne has quit IRC | 16:42 | |
| thangp | e0ne: ok np | 16:42 |
| *** jgriffith has quit IRC | 16:42 | |
| *** jgriffith1 is now known as jgriffith | 16:43 | |
| *** bitblt has quit IRC | 16:45 | |
| *** annegentle has quit IRC | 16:46 | |
| *** Miouge has quit IRC | 16:47 | |
| *** Apoorva_ has joined #openstack-cinder | 16:50 | |
| *** annegentle has joined #openstack-cinder | 16:50 | |
| *** msno has quit IRC | 16:53 | |
| *** agarwch has quit IRC | 16:53 | |
| *** jordanP has quit IRC | 16:55 | |
| *** Tross has joined #openstack-cinder | 16:57 | |
| *** setmason has quit IRC | 16:57 | |
| *** Miouge has joined #openstack-cinder | 16:57 | |
| *** asselin_ has joined #openstack-cinder | 16:58 | |
| *** asselin__ has quit IRC | 17:00 | |
| *** Yogi1 has quit IRC | 17:01 | |
| mriedem | jgriffith: thingee: xyang1: jungleboyj: DuncanT: all of the dependent changes for https://review.openstack.org/#/c/193673/ are merged now | 17:01 |
| *** setmason has joined #openstack-cinder | 17:01 | |
| mriedem | looks like quobyte ci hasn't changed over yet http://176.9.127.22:8081/refs-changes-73-193673-4/console.log.out | 17:02 |
| mriedem | or hitachi http://177.84.241.119:10000/73/193673/4/check/hitachi-hbsd-iscsi/3335906/console.html | 17:02 |
| xyang1 | mriedem: Cool, so we are good to merge this? | 17:02 |
| DuncanT | mriedem: What needs changing over? Shouldn't they get it on a clone? | 17:02 |
| mriedem | or hds http://177.84.241.119:10000/73/193673/4/check/hds-hnas-nfs/2f96243/console.html | 17:02 |
| mriedem | DuncanT: they have to set a flag in their devstack | 17:02 |
| mriedem | DuncanT: like this https://review.openstack.org/#/c/199709/ | 17:03 |
| thingee | mriedem: excellent! | 17:03 |
| DuncanT | mriedem: Ah, ok, thanks | 17:03 |
| thingee | xyang1: fire away :) | 17:03 |
| mriedem | i assume the cats were herded as much as possible | 17:03 |
| mriedem | the stragglers will have to catch up | 17:03 |
| thingee | mriedem: I'm fine with this. | 17:03 |
| mriedem | cool, thanks everyone | 17:04 |
| xyang1 | thingee: +A | 17:04 |
| thingee | excellent, I'll point all ci people to xyang1 now :P | 17:04 |
| mriedem | that's dastardly | 17:05 |
| DuncanT | mriedem: Not holding up your patch or complaining at all, just wondering if this should, in the long run, be something the driver sets somewhere, rather than a config option? | 17:05 |
| xyang1 | thingee: I'll repoint them back to mriedem :) | 17:05 |
| *** cFouts has quit IRC | 17:05 | |
| DuncanT | mriedem: Since it is information the driver has | 17:05 |
| jgriffith | mriedem: dang... xyang1 beat me to it :) | 17:05 |
| *** nug has joined #openstack-cinder | 17:05 | |
| mriedem | DuncanT: yeah, i mentioned that in the ML and commit message, like a capabilities flag or something coming up from the driver that the volume manager could then puke on when asked to create a volume from an encrypted type | 17:05 |
| xyang1 | :) | 17:05 |
| *** bswartz has quit IRC | 17:06 | |
| *** afazekas has joined #openstack-cinder | 17:06 | |
| mriedem | althought that'd be a 400 and tempest shouldn't probably treat the 400 as 'oh the config doesn't support this, so let's skip' | 17:06 |
| mriedem | so you'd still need configs in tempest more or less to skip the test | 17:06 |
| *** nug has quit IRC | 17:07 | |
| *** Yogi1 has joined #openstack-cinder | 17:07 | |
| *** coolsvap|away is now known as coolsvap | 17:08 | |
| *** bswartz has joined #openstack-cinder | 17:08 | |
| *** nug has joined #openstack-cinder | 17:09 | |
| *** nug has quit IRC | 17:10 | |
| jungleboyj | mriedem: Thanks for getting that fix put together. | 17:10 |
| mriedem | np | 17:10 |
| mriedem | i enjoy venturing into cinder land once every 6 months or so | 17:11 |
| *** rushil has joined #openstack-cinder | 17:12 | |
| *** akshai has quit IRC | 17:12 | |
| *** afazekas has quit IRC | 17:12 | |
| *** nug has joined #openstack-cinder | 17:15 | |
| *** annegentle has quit IRC | 17:16 | |
| *** rushil has quit IRC | 17:19 | |
| *** boris-42 has joined #openstack-cinder | 17:20 | |
| *** salv-orl_ has quit IRC | 17:20 | |
| *** salv-orlando has joined #openstack-cinder | 17:22 | |
| shyama | hemna: hi | 17:23 |
| *** afazekas has joined #openstack-cinder | 17:24 | |
| *** rushil has joined #openstack-cinder | 17:25 | |
| xyang | thingee: regarding the XtremIO CI failures you showed me yesterday, those were caused by a network issue "unable to access 'https://git.openstack.org/openstack-infra/devstack-gate/': ". I think others have also seen this issue. I believe this is fixed now. | 17:26 |
| asselin | xyang thingee yeah, that sever went down yesterday | 17:27 |
| *** primechuck has quit IRC | 17:28 | |
| *** rushil has quit IRC | 17:29 | |
| xyang | asselin: thanks | 17:30 |
| *** rushil has joined #openstack-cinder | 17:32 | |
| *** vilobhmm has joined #openstack-cinder | 17:33 | |
| *** dims_ has quit IRC | 17:35 | |
| *** primechuck has joined #openstack-cinder | 17:36 | |
| *** dims has joined #openstack-cinder | 17:37 | |
| *** abehl has quit IRC | 17:37 | |
| *** raunak has joined #openstack-cinder | 17:39 | |
| *** annegentle has joined #openstack-cinder | 17:41 | |
| openstackgerrit | Joshua Harlow proposed openstack/cinder: Ensure the the setup.cfg version has an epoch set https://review.openstack.org/200187 | 17:42 |
| *** Yogi1 has quit IRC | 17:43 | |
| *** BharatK has quit IRC | 17:43 | |
| * harlowja_ wants to see why all the projects that changed there version to be way smaller didn't do that ^ | 17:45 | |
| harlowja_ | so ^ may just be a conversation starter ;) | 17:45 |
| harlowja_ | cause idk why | 17:45 |
| harlowja_ | *idk why they didn't | 17:45 |
| harlowja_ | i mean people actually install this stuff right, lol | 17:46 |
| harlowja_ | (and version numbers therefore mean something) | 17:46 |
| harlowja_ | (or am i batshit crazy, lol) | 17:48 |
| eharney | harlowja_: i would assume we need to do that | 17:49 |
| thingee | hemna: http://lists.openstack.org/pipermail/openstack-dev/2015-July/068769.html | 17:50 |
| thingee | jgriffith: fyi http://lists.openstack.org/pipermail/openstack-dev/2015-July/068877.html | 17:51 |
| *** e0ne has joined #openstack-cinder | 17:51 | |
| *** IlyaG has joined #openstack-cinder | 17:52 | |
| harlowja_ | eharney ya i would think so | 17:53 |
| jgriffith | thingee: thanks, been following along at home :) | 17:53 |
| thingee | jgriffith: I figured. just wanted you to know we're listening :) | 17:53 |
| *** _cjones_ has quit IRC | 17:53 | |
| *** bdr768 has joined #openstack-cinder | 17:54 | |
| *** rushil has quit IRC | 17:54 | |
| hemna | oh hey | 17:54 |
| hemna | man, when it rains it pours....wayyyyy to many things going on. | 17:55 |
| xyang | jgriffith: hi, about your comments here: https://review.openstack.org/#/c/198556/ , do you suggest that I submit a patch just for attach snapshot, without backup involved? | 17:55 |
| *** salv-orlando has quit IRC | 17:55 | |
| hemna | so, my answer to that is that we add attach/detach coordination to the cinderclient, which is what ironic is going to use anyway. | 17:55 |
| *** shyama has quit IRC | 17:56 | |
| *** Yogi1 has joined #openstack-cinder | 17:56 | |
| hemna | mail client doesn't have that ML post :( | 17:57 |
| hemna | so I can't reply. lame | 17:57 |
| jgriffith | xyang: well, I kinda suggest that there needs to be some thought discussion on that again (ie do we even allow it) | 17:58 |
| xyang | jgriffith: I thought you are okay with it when I brought it up earlier:) | 17:58 |
| xyang | jgriffith: backup from a snapshot is usually more efficient | 17:59 |
| bdr768 | xyang: well I didn't realize you were going to put it in manager and do iscsi attach of a snapshot | 18:00 |
| *** annegentle has quit IRC | 18:00 | |
| bdr768 | bahhh | 18:00 |
| thingee | hemna: is there a write up of that that I can reference and familiarize myself with again? | 18:00 |
| xyang | bdr768 ? | 18:00 |
| *** _cjones_ has joined #openstack-cinder | 18:00 | |
| bdr768 | xyang: :) bdr768 == jgriffith | 18:01 |
| bdr768 | xyang: or at least his server | 18:01 |
| xyang | bdr768: oh:) | 18:01 |
| bdr768 | LOL... sorry | 18:01 |
| xyang | bdr768: what if I just put it in driver, but I need a driver interface | 18:01 |
| bdr768 | xyang: so this has always been an ongoing question... allow snapshots to be attached or not | 18:01 |
| bdr768 | xyang: leaving it up to driver is great IMHO | 18:01 |
| xyang | bdr768: the reason I add in manager is to do it remote | 18:02 |
| bdr768 | xyang: because then the driver can figure out what they need to do (create from snap blah blah blah) | 18:02 |
| xyang | bdr768: however, we can't do remote yet with backup | 18:02 |
| bdr768 | xyang: hmm... what do you mean remote? | 18:02 |
| *** rushil has joined #openstack-cinder | 18:02 | |
| xyang | bdr768: so I can just modify the driver interface | 18:02 |
| *** jdurgin has quit IRC | 18:02 | |
| xyang | bdr768: hard to type your new nick:). I mean backup manager only talks to a local volume driver now | 18:03 |
| bdr768 | xyang: just a sec... I'll fix my nic :) | 18:03 |
| *** jgriffith has quit IRC | 18:03 | |
| *** bdr768 is now known as jgriffith | 18:03 | |
| jgriffith | and I'm back :) | 18:03 |
| xyang | jgriffith: much better:) | 18:03 |
| jgriffith | I feel better :) | 18:03 |
| *** jgriffith is now known as Guest77747 | 18:04 | |
| xyang | jgriffith: so if I can the driver interface to allow it to happen locally, that should be enough for now | 18:04 |
| Guest77747 | bahhhh | 18:04 |
| thingee | Guest77747: what's going? :P | 18:04 |
| thingee | going on* | 18:05 |
| xyang | jgriffith: manager change is needed if I go over rpc | 18:05 |
| xyang | who is that? | 18:05 |
| *** afazekas has quit IRC | 18:05 | |
| xyang | jgriffith changes again? | 18:05 |
| thingee | xyang: yea | 18:05 |
| xyang | :) | 18:05 |
| openstackgerrit | Richard Hedlind proposed openstack/cinder: Adding back X-IO driver https://review.openstack.org/200196 | 18:06 |
| *** rushil has quit IRC | 18:06 | |
| Guest77747 | aye aye aye | 18:06 |
| Guest77747 | what have I done | 18:06 |
| xyang | Guest7747 :) | 18:07 |
| xyang | Guest77747 | 18:07 |
| hemna | thingee, not yet, it's on my ever growing list of things to do. | 18:07 |
| *** Guest77747 has quit IRC | 18:07 | |
| *** bdr768 has joined #openstack-cinder | 18:08 | |
| xyang | bdr768 are you all set? | 18:08 |
| smcginnis | Would the real jgriffith please step forward? :) | 18:09 |
| xyang | :) | 18:09 |
| *** bdr768 has quit IRC | 18:09 | |
| xyang | what's going on:) | 18:10 |
| *** jgriffit1 has joined #openstack-cinder | 18:10 | |
| smcginnis | I think those irissi customizations might not have been a good idea. :) | 18:10 |
| e0ne | hemna: what mailing thread are you talking about? | 18:11 |
| xyang | smcginnis: maybe:) | 18:11 |
| hemna | e0ne, http://lists.openstack.org/pipermail/openstack-dev/2015-July/068769.html | 18:11 |
| jgriffit1 | smcginnis: apprantly not | 18:11 |
| e0ne | hemna: thanks! | 18:11 |
| hemna | e0ne, my mail client doesn't have that message thread, so I can't reply to it :( | 18:11 |
| jgriffit1 | crud | 18:11 |
| jgriffit1 | and I have now completely Juped my account | 18:11 |
| xyang | jgriffit1: back? | 18:11 |
| jgriffit1 | almost | 18:11 |
| jgriffit1 | s/h/1/ | 18:12 |
| e0ne | hemna: i definitely need to respond. i could cc you | 18:12 |
| jgriffit1 | e0ne: can you point me to the thread you guys are talking about? | 18:13 |
| hemna | my filtering might have put it in another folder | 18:13 |
| *** jgriffit1 is now known as jdg | 18:13 | |
| jdg | LOL | 18:13 |
| hemna | omg | 18:13 |
| jdg | What the heck... now I'm jdg again | 18:14 |
| hemna | how many john griffiths do we have today ? | 18:14 |
| xyang | jdg: what's next | 18:14 |
| e0ne | hemna: :) | 18:14 |
| hemna | did John figure out how to clone himself? | 18:14 |
| hemna | dude...I need that technology, like right now. | 18:14 |
| smcginnis | ]It's like agent smith in the matrix. | 18:14 |
| xyang | hemna: :) | 18:14 |
| jdg | I'm back on my AWS instance | 18:14 |
| jdg | This shoudl hopefully stay until I can figure out what happened to my normal nic | 18:14 |
| *** bluex-pl has quit IRC | 18:14 | |
| e0ne | jdg: do you mean cc you in ML? | 18:15 |
| hemna | jdg, http://lists.openstack.org/pipermail/openstack-dev/2015-July/068769.html | 18:15 |
| jdg | hemna: thanks | 18:15 |
| jdg | e0ne: nope, jus the link :) | 18:16 |
| *** rushil has joined #openstack-cinder | 18:16 | |
| jdg | hehe... I was just working on a prez for this :) | 18:16 |
| jdg | hemna: e0ne sorry, I missed the converstation around it | 18:16 |
| e0ne | jdg: you asked it 2 lines later than Walter posted it. so i misunderstood you | 18:16 |
| thingee | jdg: was just going to respond that with the ironic bit needed in cinderclient that hemna mentioned | 18:16 |
| jdg | thingee: Ohhh! | 18:17 |
| jdg | about that :) | 18:17 |
| hemna | jdg, yah I missed it as well. I'm trying to find the ML post in my mail client, so I can respond | 18:17 |
| *** lpetrut1 has quit IRC | 18:17 | |
| jdg | I have a patch | 18:17 |
| jdg | suppose I better test it | 18:17 |
| e0ne | i'm going to have working poc of cinderagent by the monday | 18:17 |
| hemna | the cinderclient needs the attach/detach orchestration. | 18:17 |
| jdg | hemna: I put it in Cinder | 18:17 |
| jdg | hemna: API extension | 18:18 |
| jdg | hemna: implements a new "attach/detach" | 18:18 |
| hemna | hrmm | 18:18 |
| hemna | but how do you detect the volume showing up on that calling host ? | 18:18 |
| jdg | hemna: calls volume/api:external_attach/detach | 18:18 |
| thingee | jdg: well if you'd like to respond with that :) | 18:18 |
| hemna | cinderclient would need brick to discover that. | 18:18 |
| jdg | hemna: I don't | 18:18 |
| jdg | hemna: I just give the caller the connection info | 18:18 |
| jdg | hemna: the rest is not my problem | 18:18 |
| hemna | but you don't know if the volume ever shows up then. | 18:19 |
| hemna | that's not really good enough IMO | 18:19 |
| jdg | hemna: nope, but the caller does | 18:19 |
| jdg | hemna: ok, well feel free to write something better | 18:19 |
| *** annegentle has joined #openstack-cinder | 18:19 | |
| hemna | isn't the caller in this case cinderclient ? | 18:19 |
| jdg | hemna: maybe... maybe not | 18:19 |
| jdg | hemna: cinderclient is just a rest wrapper | 18:19 |
| *** rushil has quit IRC | 18:19 | |
| *** mutoulbj has quit IRC | 18:19 | |
| jdg | hemna: I'm cutting out keystone too BTW | 18:20 |
| hemna | ok sure, so that part is cool though | 18:20 |
| thingee | anyone noticing problems with ips not being accessible with dnsmasq dhcp after it has ran for sometime? This is with juno and dnsmasq 2.66 | 18:20 |
| *** annegentle has quit IRC | 18:20 | |
| jdg | hemna: so it would be trivial to add another API call that notifies success or something | 18:20 |
| hemna | we could do a follow up to the cinderclient to do the volume discovery/removal via brick. I think that'd be enough. | 18:20 |
| jdg | hemna: I'm not sure I follow what you mean? | 18:20 |
| *** jdurgin has joined #openstack-cinder | 18:21 | |
| smcginnis | thingee: I've heard of issues. | 18:21 |
| smcginnis | thingee: I had something weird in mine where it was leaking outside my private network. | 18:21 |
| smcginnis | Had to use ebtables to block DHCP traffic completely. | 18:21 |
| *** dims has quit IRC | 18:21 | |
| hemna | whoever is calling cinder to attach a volume to itself, needs to know that not only did cinder export the volume, but also that the volume showed up on the host and what it's path is. | 18:22 |
| hemna | especially in the case of iSCSI, where you have to do session login, etc for the volume to ever show up. | 18:22 |
| jdg | hemna: I think we're talking about different things here | 18:22 |
| jdg | I'm talking "no brick" | 18:22 |
| jdg | I'm talking... a general Block as a service | 18:23 |
| hemna | I'm talking about an end to end solution for using cinderclient to attach a volume to a host. | 18:23 |
| smcginnis | hemna: Not necessarily. I've worked with other systems where it was the storage's job to expose the volume and that's it. It was up to a different process on the host side to login, rescan, etc. | 18:23 |
| jdg | user say has an ubuntu server.. wants a volume | 18:23 |
| hemna | w/o nova | 18:23 |
| jdg | cinder create xxxx | 18:23 |
| thingee | smcginnis: This is all of a sudden with my ci. I think it first started happening a couple of weeks ago | 18:23 |
| jdg | hemna: yes, no Nova | 18:23 |
| jdg | no other OpenStack bits at all | 18:23 |
| smcginnis | thingee: Are you getting "no IP addresses available" or something similar in syslog? | 18:23 |
| thingee | smcginnis: killing dnsmasq and restarting nova network fixes the problem for a bit | 18:24 |
| *** markvoelker has quit IRC | 18:24 | |
| thingee | smcginnis: nah, the instances get assigned ips just fine.. just not accessible | 18:24 |
| hemna | w/o collecting the initiator information on the host trying to do that attach, it won't work as well. | 18:24 |
| *** markvoelker has joined #openstack-cinder | 18:24 | |
| * thingee is also scared to upgrade his CI to the latest stable juno lololo | 18:24 | |
| *** jdg is now known as jgriffith | 18:24 | |
| *** bswartz has quit IRC | 18:24 | |
| smcginnis | thingee: Hmm, different than what I was getting. Sounds familiar though. I'll see if I can find any reference. | 18:25 |
| *** afazekas has joined #openstack-cinder | 18:25 | |
| *** jgriffith is now known as Guest43643 | 18:25 | |
| smcginnis | thingee: :) You're PTL. You should be running bleeding edge Liberty. :P | 18:25 |
| e0ne | jdg, hemna: https://blueprints.launchpad.net/cinder/+spec/use-cinder-without-nova | 18:25 |
| Guest43643 | e0ne: Oh wow! I had not seen that | 18:26 |
| Guest43643 | frik! | 18:26 |
| thingee | smcginnis: ha, I'm running juno on the ci and the lab for others here to do development on vms | 18:26 |
| *** MentalRay has quit IRC | 18:26 | |
| thingee | Guest43643: haha | 18:26 |
| e0ne | Guest43643: nice nick :) | 18:26 |
| xyang | Guest43643 ?:) | 18:26 |
| *** bswartz has joined #openstack-cinder | 18:26 | |
| *** Guest43643 is now known as jgriffith | 18:27 | |
| *** jaypipes has joined #openstack-cinder | 18:27 | |
| xyang | jgriffith: before you change your nick again, are you ok if I change initialize_connection in driver, but not in manager? | 18:28 |
| jgriffith | xyang: haha.. not sure :) | 18:28 |
| *** ganso_ has quit IRC | 18:28 | |
| thingee | smcginnis: doh, syslog is yelling about it | 18:29 |
| xyang | jgriffith: :) the backup driver needs to call a common volume driver api to do backup | 18:29 |
| smcginnis | thingee: That was my clue. Non-stop streaming of errors. | 18:29 |
| jgriffith | xyang: I get ya, I'm just tryign to figure out how to address it | 18:29 |
| xyang | jgriffith: ok | 18:30 |
| thingee | smcginnis: weird it looks like nova-network is assigning ips that dnsmasq thinks is already leased | 18:30 |
| thingee | https://www.irccloud.com/pastebin/mKFrepWi/ | 18:31 |
| *** patrickeast has joined #openstack-cinder | 18:31 | |
| *** jdurgin1 has joined #openstack-cinder | 18:31 | |
| *** Yogi1 has quit IRC | 18:32 | |
| smcginnis | thingee: That actually does look similar to what I had, IIRC. | 18:33 |
| thingee | smcginnis: ha looks like patrickeast has this problem too https://ask.openstack.org/en/question/66105/dnsmasq-dhcp-server-doesnt-lease-a-released-ip-address/ | 18:33 |
| smcginnis | thingee: That's why it sounded familiar! | 18:33 |
| patrickeast | on man, did anyone figure out how to fix that? | 18:34 |
| patrickeast | cus i still have that problem | 18:34 |
| *** simondodsley has quit IRC | 18:34 | |
| smcginnis | If it helps anyone, after I did this my problem cleared up: | 18:34 |
| smcginnis | sudo ebtables -I INPUT -i em1 --protocol ipv4 --ip-proto udp --ip-dport 67:68 -j DROP | 18:34 |
| thingee | patrickeast: I have to kill dnsmasq and restart nova-network every time | 18:34 |
| *** jdurgin has quit IRC | 18:34 | |
| patrickeast | yea same | 18:34 |
| patrickeast | i found out that the release commands sent to the dnsmasq process were ignored sometimes | 18:35 |
| thingee | patrickeast: I'm running Juno... the weird thing is this just start happening two weeks ago | 18:35 |
| xyang | jgriffith: currently I have to change almost every driver because I am trying to use initialize_connection. alternatively I can introduce a new interface, this way I don't have to change all existing drivers | 18:35 |
| patrickeast | even though im 100% sure they reached it | 18:35 |
| thingee | patrickeast: ci has been running fine before then | 18:35 |
| patrickeast | oh weird | 18:35 |
| smcginnis | thingee: Ubuntu package update? | 18:35 |
| patrickeast | maybe dnsmasq update? | 18:35 |
| thingee | smcginnis: unless it was security related...it shouldn't have been done automatically. | 18:35 |
| thingee | smcginnis, patrickeast: oh also I had to install a particlar PPA because there was a bug with openstack and dnsmasq back in juno | 18:36 |
| *** Yogi1 has joined #openstack-cinder | 18:36 | |
| * patrickeast takes notes | 18:37 | |
| patrickeast | one of these days i’m going to go back and try it again | 18:37 |
| patrickeast | for now i’m rolling with 3 all-in-one installs using neutron networking, it works but is super ineffecient | 18:38 |
| e0ne | jgriffith: this is could be interesting for you too: https://github.com/e0ne/cinder-agent-poc/ | 18:38 |
| jgriffith | e0ne: NICE!! | 18:38 |
| e0ne | i'm hitting with devstack now:( | 18:39 |
| e0ne | it's broken on my env again:( | 18:39 |
| thangp | e0ne: I put a couple of comments in https://review.openstack.org/#/c/200172...I think it should be working as is, but I am wondering which fields are not updating for you | 18:41 |
| patrickeast | oh so if anyone is interested in the image cache stuff, i’ve been playing around a little bit with rally to beat on it, as sort of an idiot check to make sure its working as expected i did a ‘best-case’ kind of test | 18:41 |
| patrickeast | without the cache http://ec2-54-67-51-189.us-west-1.compute.amazonaws.com/image-volume-cache-results/no-cache-50.html#/CinderVolumes.create_and_delete_volume | 18:42 |
| patrickeast | with it enabled http://ec2-54-67-51-189.us-west-1.compute.amazonaws.com/image-volume-cache-results/cache-50.html#/CinderVolumes.create_and_delete_volume | 18:42 |
| * patrickeast likes to have graphs to show PMs | 18:42 | |
| e0ne | thangp: thanks a lot! | 18:42 |
| smcginnis | patrickeast: Nice! | 18:43 |
| thangp | e0ne: i need to run some tests to make sure...but I think it should be working | 18:43 |
| patrickeast | those are with 8gb volumes iirc | 18:43 |
| *** afazekas has quit IRC | 18:43 | |
| e0ne | thangp: looks like you're right! | 18:43 |
| thingee | patrickeast: wow, really cool | 18:44 |
| e0ne | thangp: i'm not expert in objects yet | 18:44 |
| e0ne | i'll test and update you | 18:44 |
| thangp | e0ne: :) cool thx | 18:44 |
| *** rushil has joined #openstack-cinder | 18:44 | |
| xyang | patrickeast: that is a big difference! | 18:45 |
| hemna | e0ne, ok I replied to that thread. Thanks for the heads up thingee. | 18:46 |
| hemna | I didn't see it because it didn't have [cinder] in the title. (mail filtering) | 18:46 |
| *** lpetrut has joined #openstack-cinder | 18:47 | |
| e0ne | hemna: thanks. waiting for it in my inbox | 18:47 |
| thingee | hemna: thanks | 18:47 |
| hemna | I'd like an end to end solution that does the entire process, instead of just having cinder export a volume. | 18:47 |
| e0ne | hemna: +1 | 18:48 |
| e0ne | hemna: it will be one option | 18:48 |
| jgriffith | hemna: I guess I don't understand that ? | 18:48 |
| jgriffith | hemna: you want something that someobdy has to install brick on their system and does the attach for them? | 18:49 |
| e0ne | hemna: the other one that ironic team want is to have ability to to it manually, w/o agent inside instance | 18:49 |
| e0ne | jgriffith: yep. see my messages above | 18:49 |
| hemna | jgriffith, no | 18:50 |
| jgriffith | e0ne: yeah... so certainly make it an option, but I know many have no interest in somethign attaching for them. Think there's folks in both camps which means good option to have either or | 18:50 |
| *** lpetrut has quit IRC | 18:50 | |
| hemna | jgriffith, I want brick to be embedded into the cinderclient and the cinderclient calls brick apis to: 1) collect the initiator information and then 2) discover the volume after it's exported. | 18:50 |
| jgriffith | hemna: sorry, I don't understand. But that's ok | 18:51 |
| hemna | that gives ironic end users that ability to just use cinderclient to do attaches and it just works. | 18:51 |
| openstackgerrit | Skyler Berg proposed openstack/cinder: Port Tintri driver to ABC driver model https://review.openstack.org/200209 | 18:51 |
| jgriffith | hemna: oh... umm | 18:51 |
| hemna | cinder attach <volume uuid> | 18:51 |
| hemna | the result of that | 18:51 |
| hemna | is a volume in /dev/disk/by-path | 18:51 |
| hemna | done | 18:51 |
| *** bswartz has quit IRC | 18:51 | |
| jgriffith | doesn't that kinda step way outside of the bounds of what the clients are intended to be? | 18:51 |
| jgriffith | like WAYYY out of bounds | 18:52 |
| *** bswartz has joined #openstack-cinder | 18:52 | |
| *** Tross has quit IRC | 18:52 | |
| hemna | that's what we all pretty much agreed to in the ironic discussion in vancouver | 18:52 |
| hemna | and I'm not sure I agree with it being out of bounds. | 18:52 |
| jgriffith | hemna: oh.. odd | 18:52 |
| jgriffith | hemna: I didn't catch that | 18:52 |
| hemna | anyway, that's what I had in mind. | 18:52 |
| thingee | hemna: yeah I remember that too. We agreed it would have to be documented that this shouldn't be used by end users as well. | 18:52 |
| hemna | if that's not what others want, then that's cool. | 18:53 |
| jgriffith | hemna: but ok, if everybody agreed on it ok | 18:53 |
| e0ne | hemna: afair, we agreed to have some other client | 18:53 |
| hemna | we can write another client tool that does all the same stuff I suppose | 18:53 |
| harlowja_ | eharney well https://review.openstack.org/#/c/200187/ blew up royally, lol | 18:53 |
| patrickeast | sounds like a brick-client? | 18:53 |
| jgriffith | I'm pretty opposed ot the idea of putting intiator calls in cinderclient FWIW, but apparantly "everybody" else picked that up and wanted it so "ok" I guess | 18:53 |
| thingee | jgriffith: I agree with what you're saying. I'm not really sure we have any other ideas though to satisfy ironic's needs here. | 18:53 |
| eharney | harlowja_: that's pretty impressive landing with no green at all :) | 18:54 |
| hemna | we can write another tool | 18:54 |
| hemna | that does it all. | 18:54 |
| jgriffith | thingee: my interpreation from that session were very different | 18:54 |
| hemna | I don't care what it's called, but I'd like a single tool that does it. | 18:54 |
| *** primechuck has quit IRC | 18:54 | |
| hemna | foo attach <cinder uuid> | 18:54 |
| thingee | jgriffith: what did you get from the session? Maybe it's an idea we can think about as opposed to this one | 18:54 |
| hemna | if foo is cinderclient, cool, if foo is new tool, then cool as well. | 18:54 |
| jgriffith | thingee: so when I walked through the steps with the folks from Rax and devananda I layed it out as: "just provide a call that gives the connection info" | 18:55 |
| *** Lee1092 has quit IRC | 18:55 | |
| jgriffith | thingee: and left it up to the caller to do what they wanted with it | 18:55 |
| jgriffith | thingee: folks seemed to nod and agree | 18:55 |
| jgriffith | thingee: but maybe I completely missed something | 18:56 |
| jgriffith | thingee: IIRC the first 25 minutes were discussions about agents, embedded ops etc etc | 18:56 |
| thingee | jgriffith: by call you just mean something from the cinder-api. no client involved that we maintain | 18:56 |
| e0ne | jgriffith: it's something like minimal featureset | 18:56 |
| jgriffith | and a good deal of debating between Rax and HP | 18:56 |
| hemna | I'd be up for writing a new tool just for attach/detaches then that uses the new API that jgriffith is working on. | 18:56 |
| jgriffith | thingee: it could/would certainly be a call added to cinderclient | 18:56 |
| hemna | then folks can just pip install <new tool> && <new tool> attach <cinder uuid> | 18:57 |
| hemna | done | 18:57 |
| jgriffith | thingee: I'm just saying I wouldn't have cinderclient actually "attach" a volume somewhere | 18:57 |
| thingee | got it | 18:57 |
| jgriffith | thingee: that's the part I missed that everybody seems to have taken away | 18:57 |
| thingee | jgriffith: sounds like hemna is fine with this being outside of Cinder client | 18:57 |
| *** skylerberg has joined #openstack-cinder | 18:58 | |
| thingee | and we can just have something fetch the initialize connection info | 18:58 |
| jgriffith | hemna: so I had thought a "tool" that calls cinderclient takes the info and does "whatever" with it, rather than one tool to rule them all | 18:58 |
| hemna | yah that's ok. we'll just write a new tool that uses the new API | 18:58 |
| jgriffith | hemna: linux philosophy "tool chains" | 18:58 |
| thingee | sounds like we have consensus :) | 18:58 |
| hemna | sure, the new tool could import cinderclient and call the api to get the connection_info | 18:58 |
| jgriffith | hemna: I'm sure it will all work out fine in the end :) | 18:58 |
| hemna | and then do the rest of the work w/ brick to make sure that the volume actually shows up. | 18:59 |
| *** rushil has quit IRC | 18:59 | |
| e0ne | jgriffith, hemna: one question... what could i do for it? | 19:00 |
| *** lpetrut has joined #openstack-cinder | 19:00 | |
| e0ne | hemna: looks like that agent with rest is not needed anymore | 19:00 |
| e0ne | s/rest/rest api | 19:00 |
| *** changbl has joined #openstack-cinder | 19:00 | |
| hemna | e0ne, write the new tool :) | 19:00 |
| e0ne | hemna: ok, sounde doable:) | 19:01 |
| hemna | the new API is going to need the initiator information | 19:01 |
| jgriffith | e0ne: hemna we should put our heads together and share some things next week | 19:01 |
| hemna | ok sounds good | 19:01 |
| *** thangp has quit IRC | 19:01 | |
| e0ne | jgriffith: agree | 19:01 |
| hemna | I think we are working towards the same end goal. | 19:01 |
| hemna | just different layers of the onion so to speak. | 19:02 |
| *** rushil has joined #openstack-cinder | 19:02 | |
| *** IlyaG has quit IRC | 19:03 | |
| *** IlyaG has joined #openstack-cinder | 19:03 | |
| e0ne | hemna: i'll change my agent poc to cli tool | 19:04 |
| *** coolsvap is now known as coolsvap|away | 19:04 | |
| *** thangp has joined #openstack-cinder | 19:04 | |
| hemna | e0ne, probably a safe bet :) | 19:04 |
| e0ne | hemna: :) | 19:05 |
| e0ne | jgriffith: it would be great it you'll share your draft api changes | 19:06 |
| openstackgerrit | Ryan Lucio proposed openstack/cinder: VMEM v6000: Fix export verify routines https://review.openstack.org/165655 | 19:07 |
| *** ociuhandu has quit IRC | 19:08 | |
| *** dims has joined #openstack-cinder | 19:10 | |
| mtanino | jgriffith: ping, | 19:10 |
| *** jgriffith has quit IRC | 19:10 | |
| *** jgriffith has joined #openstack-cinder | 19:11 | |
| *** annashen has quit IRC | 19:11 | |
| *** raunak has quit IRC | 19:11 | |
| * e0ne bb in ~12 hours | 19:13 | |
| *** MentalRay has joined #openstack-cinder | 19:14 | |
| *** markvoelker has quit IRC | 19:15 | |
| *** markvoelker has joined #openstack-cinder | 19:15 | |
| boris-42 | patrickeast: nice results=) | 19:17 |
| *** mutoulbj has joined #openstack-cinder | 19:20 | |
| *** IlyaG has quit IRC | 19:21 | |
| *** lpetrut has quit IRC | 19:22 | |
| *** lpetrut has joined #openstack-cinder | 19:24 | |
| *** mutoulbj has quit IRC | 19:25 | |
| *** lpetrut has quit IRC | 19:25 | |
| *** rushil has quit IRC | 19:32 | |
| *** annashen has joined #openstack-cinder | 19:35 | |
| *** asselin_ has quit IRC | 19:40 | |
| *** Tross has joined #openstack-cinder | 19:40 | |
| *** asselin_ has joined #openstack-cinder | 19:40 | |
| *** zhenguo has quit IRC | 19:43 | |
| *** Longgeek has quit IRC | 19:49 | |
| *** annashen has quit IRC | 19:51 | |
| *** annashen has joined #openstack-cinder | 19:51 | |
| *** Lee1092 has joined #openstack-cinder | 19:53 | |
| *** annashen has quit IRC | 19:56 | |
| *** _cjones_ has quit IRC | 19:58 | |
| jgriffith | ssh bdr78 | 20:03 |
| *** annegentle has joined #openstack-cinder | 20:04 | |
| e0ne | jungleboyj: hi. i've added you to review https://review.openstack.org/#/c/198721/. we need your help with oslo:) | 20:05 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: LVM: Support efficient data copy for LVM driver https://review.openstack.org/200237 | 20:05 |
| *** _cjones_ has joined #openstack-cinder | 20:06 | |
| *** pots has joined #openstack-cinder | 20:07 | |
| *** daneyon has quit IRC | 20:14 | |
| *** timcl has quit IRC | 20:15 | |
| *** salv-orlando has joined #openstack-cinder | 20:19 | |
| *** raunak has joined #openstack-cinder | 20:21 | |
| *** jwcroppe has quit IRC | 20:25 | |
| *** lpetrut has joined #openstack-cinder | 20:25 | |
| *** xyang has quit IRC | 20:28 | |
| *** annashen has joined #openstack-cinder | 20:31 | |
| *** jwcroppe has joined #openstack-cinder | 20:31 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: WIP: Add new Connector APIs for path validation https://review.openstack.org/199764 | 20:33 |
| *** lpetrut has quit IRC | 20:34 | |
| *** xyang has joined #openstack-cinder | 20:37 | |
| *** primechuck has joined #openstack-cinder | 20:38 | |
| *** r-daneel has joined #openstack-cinder | 20:39 | |
| *** bswartz has quit IRC | 20:39 | |
| *** e0ne has quit IRC | 20:42 | |
| *** xyang has quit IRC | 20:44 | |
| eharney | what's going on with the python-cinderclient releases here? | 20:46 |
| eharney | https://launchpad.net/python-cinderclient | 20:46 |
| eharney | there's no 1.2.1, or 1.3.0.. 1.3.1 has 121 bugs fixed in it | 20:46 |
| *** delatte has quit IRC | 20:46 | |
| smcginnis | And 1.2.0 comes after 1.3.1? | 20:46 |
| *** xyang has joined #openstack-cinder | 20:47 | |
| *** delattec has quit IRC | 20:47 | |
| eharney | smcginnis: i think because 1.2.0 is a "milestone" and 1.3.1 is a "release" | 20:47 |
| eharney | anyway this looks all wrong... | 20:47 |
| eharney | thingee: ^ | 20:48 |
| smcginnis | eharney: Well that's not confusing at all. :; | 20:48 |
| *** MentalRay has quit IRC | 20:48 | |
| *** r-daneel has quit IRC | 20:49 | |
| *** Yogi1 has quit IRC | 20:50 | |
| *** Longgeek has joined #openstack-cinder | 20:50 | |
| *** jaypipes has quit IRC | 20:53 | |
| *** Longgeek has quit IRC | 20:55 | |
| *** annegentle has quit IRC | 20:56 | |
| *** MentalRay has joined #openstack-cinder | 20:56 | |
| *** julim has quit IRC | 20:57 | |
| *** rushil has joined #openstack-cinder | 20:57 | |
| *** Miouge has quit IRC | 20:58 | |
| *** akerr has quit IRC | 20:58 | |
| *** setmason has quit IRC | 20:59 | |
| openstackgerrit | Mathieu Gagné proposed openstack/cinder: Add ability to override OpenStack privileged user auth url https://review.openstack.org/200264 | 20:59 |
| *** leeantho has quit IRC | 20:59 | |
| *** patrickeast_ has joined #openstack-cinder | 21:01 | |
| openstackgerrit | Richard Hedlind proposed openstack/cinder: Adding back X-IO driver https://review.openstack.org/200196 | 21:01 |
| *** ronis has quit IRC | 21:01 | |
| *** annasort has quit IRC | 21:02 | |
| *** leeantho has joined #openstack-cinder | 21:03 | |
| *** patrickeast has quit IRC | 21:03 | |
| *** patrickeast_ is now known as patrickeast | 21:03 | |
| *** setmason has joined #openstack-cinder | 21:03 | |
| *** dustins has quit IRC | 21:03 | |
| *** dims has quit IRC | 21:05 | |
| *** dims has joined #openstack-cinder | 21:07 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Cinder replication V2 https://review.openstack.org/200269 | 21:13 |
| jgriffith | mgagne: becasue I'm somewhat 'slow', | 21:16 |
| jgriffith | can you help me better understand that | 21:16 |
| jgriffith | patch? | 21:16 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Graceful shutdown WSGI/RPC server https://review.openstack.org/194929 | 21:17 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Dump running WSGI threads before shutdown for testing https://review.openstack.org/200271 | 21:17 |
| mgagne | jgriffith: I wish to bypass the endpoint URLs contained in the token and force my own. A bit like nova_endpoint_template allows already | 21:17 |
| mgagne | the auth_url contained in the token is not reachable in our management network so we force our own everywhere we can | 21:18 |
| jgriffith | mgagne: yeah, I get that.... trying to | 21:18 |
| jgriffith | put together how I would "use it" | 21:18 |
| jgriffith | myself. | 21:18 |
| mgagne | and also because of that: http://blog.mgagne.ca/reducing-keystone-pki-token-size/ | 21:18 |
| *** jwcroppe has quit IRC | 21:18 | |
| *** cbader has quit IRC | 21:18 | |
| jgriffith | mgagne: in other words a way to test it | 21:18 |
| jgriffith | out | 21:18 |
| *** jwcroppe_ has joined #openstack-cinder | 21:18 | |
| mgagne | jgriffith: do you wish me to include an example? | 21:18 |
| * jgriffith reading | 21:18 | |
| mgagne | jgriffith: I think there is one in the test :D | 21:18 |
| jgriffith | mgagne: well, it's more for my benefit | 21:18 |
| jgriffith | :) | 21:18 |
| jgriffith | mgagne: indeed there is :) | 21:19 |
| jgriffith | mgagne: ok, never mind :) | 21:19 |
| *** jwcroppe has joined #openstack-cinder | 21:21 | |
| *** jwcroppe_ has quit IRC | 21:24 | |
| *** annashen has quit IRC | 21:26 | |
| *** annashen has joined #openstack-cinder | 21:27 | |
| *** kjnelson has quit IRC | 21:27 | |
| *** annasort has joined #openstack-cinder | 21:29 | |
| *** lpetrut has joined #openstack-cinder | 21:34 | |
| *** jamielennox is now known as jamielennox|away | 21:37 | |
| *** jwcroppe_ has joined #openstack-cinder | 21:41 | |
| *** jwcroppe has quit IRC | 21:42 | |
| *** IlyaG has joined #openstack-cinder | 21:44 | |
| openstackgerrit | David Medberry proposed openstack/cinder: Removing archaic references to hypervisors and quickstart. https://review.openstack.org/193791 | 21:45 |
| *** jwcroppe has joined #openstack-cinder | 21:46 | |
| *** daneyon has joined #openstack-cinder | 21:46 | |
| *** DTadrzak has quit IRC | 21:47 | |
| *** DTadrzak has joined #openstack-cinder | 21:47 | |
| *** jwcroppe_ has quit IRC | 21:47 | |
| *** jamielennox|away is now known as jamielennox | 21:48 | |
| *** daneyon_ has joined #openstack-cinder | 21:49 | |
| *** jungleboyj has quit IRC | 21:51 | |
| *** Longgeek has joined #openstack-cinder | 21:51 | |
| *** daneyon has quit IRC | 21:52 | |
| *** leeantho_ has joined #openstack-cinder | 21:54 | |
| *** daneyon has joined #openstack-cinder | 21:56 | |
| *** leeantho has quit IRC | 21:56 | |
| *** thangp has quit IRC | 21:56 | |
| *** MentalRay has quit IRC | 21:59 | |
| *** daneyon_ has quit IRC | 22:00 | |
| *** daneyon_ has joined #openstack-cinder | 22:01 | |
| *** changbl has quit IRC | 22:02 | |
| *** dims has quit IRC | 22:03 | |
| *** dims has joined #openstack-cinder | 22:03 | |
| *** pv_ has joined #openstack-cinder | 22:03 | |
| *** daneyon has quit IRC | 22:04 | |
| pv_ | how do i configure the devstack local.conf to use my cinder backend/driver instead of the default? | 22:05 |
| *** MentalRay has joined #openstack-cinder | 22:05 | |
| *** coolsvap|away is now known as coolsvap | 22:05 | |
| *** rajinir has joined #openstack-cinder | 22:05 | |
| *** jgriffith is now known as Guest85831 | 22:06 | |
| *** crose has quit IRC | 22:07 | |
| *** lcurtis has joined #openstack-cinder | 22:08 | |
| *** Longgeek has quit IRC | 22:10 | |
| *** mriedem is now known as mriedem_away | 22:10 | |
| openstackgerrit | Mathieu Gagné proposed openstack/cinder: Add ability to override OpenStack privileged user auth url https://review.openstack.org/200264 | 22:11 |
| *** daneyon_ has quit IRC | 22:11 | |
| *** gouthamr has quit IRC | 22:13 | |
| *** jgriffith has joined #openstack-cinder | 22:14 | |
| *** Lee1092 has quit IRC | 22:15 | |
| *** edmondsw has quit IRC | 22:18 | |
| patrickeast | hmm i guess the encrypted volume tests dont work in my ci system | 22:20 |
| patrickeast | http://i.imgur.com/amkoqfX.jpg | 22:20 |
| smcginnis | :) | 22:21 |
| smcginnis | pv_: Here's an example from my CI: http://oslogs.compellent.com/dell-sc-iscsi-200209-1/logs/local.conf.txt.gz | 22:22 |
| *** jwcroppe_ has joined #openstack-cinder | 22:22 | |
| *** lpetrut has quit IRC | 22:24 | |
| *** jwcroppe has quit IRC | 22:24 | |
| *** jasondotstar has quit IRC | 22:25 | |
| pv_ | thanks man, i have a quick question | 22:25 |
| pv_ | is DellStorageCenterISCSIDriver an arbitrary name you picked for the backend? | 22:25 |
| pv_ | i understand what you're doing when you're setting your driver code | 22:25 |
| pv_ | i also have my own specific driver code i've written that i can put in | 22:25 |
| pv_ | but im not sure what to put in the volume_backend_name field... does it really matter? or can it be anything | 22:26 |
| patrickeast | iirc the backend name needs to match the volume type setting | 22:27 |
| smcginnis | pv_: It can be anything, but for ease of scripting things in my CI I just use the driver name as the backend name. | 22:27 |
| patrickeast | devstack would create a volume type that matches that backend name | 22:27 |
| smcginnis | Oh, and that. | 22:27 |
| pv_ | cool ok, got it thanks! | 22:27 |
| patrickeast | i forget which of them have to match up | 22:27 |
| patrickeast | i usually have the backend_name the same all over, just like DellStorageCenterISCSIDriver in that one | 22:28 |
| pv_ | basically the backend_name once set will be the volume_type that appears when i create volumes right? | 22:28 |
| pv_ | so im guessing in the default devstack code, the backend name is lvm-driver-1? | 22:28 |
| smcginnis | pv_: With devstack, yes, I believe so. | 22:28 |
| smcginnis | bbl | 22:28 |
| pv_ | fasho thanks | 22:29 |
| *** sgotliv has quit IRC | 22:30 | |
| *** rushil has quit IRC | 22:30 | |
| openstackgerrit | Richard Hedlind proposed openstack/cinder: Adding back X-IO driver https://review.openstack.org/200196 | 22:31 |
| *** primechuck has quit IRC | 22:32 | |
| *** IlyaG has quit IRC | 22:34 | |
| patrickeast | hmm this doesn’t look like the existing bugs for the encryption issues http://ec2-54-67-51-189.us-west-1.compute.amazonaws.com/55/165655/6/check/PureFCDriver-tempest-dsvm-volume-multipath/307e3c5/logs/screen-n-cpu.txt.gz?level=TRACE#_2015-07-09_20_05_34_058 | 22:36 |
| *** jwcroppe has joined #openstack-cinder | 22:36 | |
| patrickeast | and it looks like my driver is not doing this step in initialize_connection… https://github.com/openstack/cinder/blob/master/cinder/volume/driver.py#L1546 | 22:36 |
| *** eharney has quit IRC | 22:37 | |
| *** jwcroppe_ has quit IRC | 22:38 | |
| tbarron | patrickeast: it looks like the 'volume is in use' bug to me | 22:42 |
| patrickeast | tbarron: different command though | 22:42 |
| patrickeast | i think | 22:42 |
| *** jungleboyj has joined #openstack-cinder | 22:43 | |
| *** jasondotstar has joined #openstack-cinder | 22:43 | |
| tbarron | Well, you are getting 'in use or mounted' early, which I don't think I saw. | 22:43 |
| *** annashen has quit IRC | 22:43 | |
| tbarron | But then you get 'Device /dev/sdaj is not a valid luks device', which we see even in the runs where this test succeeds. | 22:44 |
| tbarron | That forces a format attempt. | 22:44 |
| patrickeast | hmm interesting | 22:44 |
| *** annashen has joined #openstack-cinder | 22:44 | |
| tbarron | which then results in 'Cannot format device /dev/sdaj which is still in use.\n' | 22:44 |
| tbarron | That last part is what I reported in the 'in use' bug that e.g. Xing says she sees with her iscsi drivers as well. | 22:45 |
| tbarron | Matt thinks a retry three times (he has proposed the review) may well make this go away. | 22:45 |
| tbarron | Based on experience with a similar issue. | 22:46 |
| *** mdenny has quit IRC | 22:46 | |
| patrickeast | ah ok, cool | 22:47 |
| patrickeast | i see https://review.openstack.org/#/c/197721/ | 22:47 |
| patrickeast | i’ll try that out and see if it works better | 22:47 |
| *** annashen has quit IRC | 22:49 | |
| *** annashen has joined #openstack-cinder | 22:51 | |
| *** rushil has joined #openstack-cinder | 22:52 | |
| *** VManiac16 has joined #openstack-cinder | 22:53 | |
| *** rushil has quit IRC | 22:55 | |
| *** diogogmt has quit IRC | 22:58 | |
| *** lcurtis has quit IRC | 23:07 | |
| *** hemna is now known as hemnafk | 23:11 | |
| *** mtanino_ has joined #openstack-cinder | 23:11 | |
| *** mtanino has quit IRC | 23:11 | |
| *** setmason has quit IRC | 23:12 | |
| *** coolsvap is now known as coolsvap|afk | 23:14 | |
| *** salv-orlando has quit IRC | 23:18 | |
| *** zhenguo has joined #openstack-cinder | 23:21 | |
| *** guest774 has joined #openstack-cinder | 23:23 | |
| *** guest774 has left #openstack-cinder | 23:23 | |
| *** pv_ has quit IRC | 23:23 | |
| *** mutoulbj has joined #openstack-cinder | 23:24 | |
| *** mutoulbj has quit IRC | 23:28 | |
| *** setmason has joined #openstack-cinder | 23:31 | |
| *** annashen has quit IRC | 23:34 | |
| *** mtanino_ has quit IRC | 23:36 | |
| *** patrickeast has quit IRC | 23:37 | |
| *** mtanino has joined #openstack-cinder | 23:37 | |
| *** mgarza_ has quit IRC | 23:37 | |
| *** Longgeek has joined #openstack-cinder | 23:38 | |
| *** openstack has joined #openstack-cinder | 23:39 | |
| openstackgerrit | Skyler Berg proposed openstack/cinder: Add retries to Tintri driver API calls https://review.openstack.org/200312 | 23:40 |
| *** mdenny has joined #openstack-cinder | 23:43 | |
| *** MentalRay has quit IRC | 23:44 | |
| *** chlong has quit IRC | 23:49 | |
| *** Longgeek has quit IRC | 23:52 | |
| *** IanGovett has quit IRC | 23:53 | |
| *** kfox1111 has joined #openstack-cinder | 23:54 | |
| kfox1111 | doing a juno -> kilo upgrade. almost there. cinder volume's grumpy. complaining that "/usr/lib/python2.7/site-packages/cinder/volume/drivers/rbd.py", line 384, in _update-volume_stats | 23:55 |
| kfox1111 | pool_stats['max_avail'] / units.Gi) | 23:55 |
| kfox1111 | KeyError: 'max_avail' | 23:55 |
| kfox1111 | any ideas? | 23:55 |
| *** rajinir has quit IRC | 23:56 | |
| openstackgerrit | Kuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key https://review.openstack.org/198614 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
