| *** dannywilson has quit IRC | 00:01 | |
| *** Yogi1 has quit IRC | 00:04 | |
| asselin_ | thingee, ociuhandu alexpilotti I think it's fine | 00:07 |
|---|---|---|
| * alexpilotti scavenges the fridge looking for champagne | 00:08 | |
| thingee | asselin_: thanks | 00:09 |
| *** tsekiyam_ has joined #openstack-cinder | 00:13 | |
| *** tsekiyama has quit IRC | 00:16 | |
| *** mtanino has quit IRC | 00:17 | |
| *** tsekiyam_ has quit IRC | 00:17 | |
| *** yamada-h has joined #openstack-cinder | 00:19 | |
| *** akerr has quit IRC | 00:22 | |
| *** yamada-h has quit IRC | 00:24 | |
| *** jaypipes has quit IRC | 00:27 | |
| *** ebalduf has joined #openstack-cinder | 00:29 | |
| *** ronis_ has quit IRC | 00:30 | |
| *** IanGovett has quit IRC | 00:31 | |
| alexpilotti | asselin_: thanks for the CI results review! | 00:32 |
| *** ebalduf has quit IRC | 00:34 | |
| *** scottda_ has joined #openstack-cinder | 00:34 | |
| *** heyun has joined #openstack-cinder | 00:38 | |
| *** erlon has quit IRC | 00:41 | |
| *** emagana has quit IRC | 00:46 | |
| *** dflorea has joined #openstack-cinder | 00:47 | |
| *** annashen has joined #openstack-cinder | 00:47 | |
| *** dflorea has quit IRC | 00:49 | |
| *** dflorea has joined #openstack-cinder | 00:50 | |
| *** Longgeek has joined #openstack-cinder | 00:50 | |
| *** annashen has quit IRC | 00:52 | |
| *** vilobhmm1 has quit IRC | 00:52 | |
| *** dflorea has quit IRC | 00:54 | |
| *** akerr has joined #openstack-cinder | 00:55 | |
| thingee | asselin_: did I hear right there was an issue open for VolumeBootPattern? | 00:58 |
| *** akerr_ has joined #openstack-cinder | 00:59 | |
| *** Yogi1 has joined #openstack-cinder | 01:01 | |
| *** akerr has quit IRC | 01:02 | |
| *** dflorea has joined #openstack-cinder | 01:07 | |
| *** dflorea has quit IRC | 01:07 | |
| *** Longgeek has quit IRC | 01:07 | |
| *** _cjones_ has quit IRC | 01:08 | |
| *** Longgeek has joined #openstack-cinder | 01:08 | |
| *** dflorea has joined #openstack-cinder | 01:08 | |
| *** leeantho has quit IRC | 01:09 | |
| *** Mandell has quit IRC | 01:09 | |
| *** dflorea has quit IRC | 01:10 | |
| *** dflorea_ has joined #openstack-cinder | 01:10 | |
| *** dflorea_ has quit IRC | 01:12 | |
| *** dflorea has joined #openstack-cinder | 01:12 | |
| *** BharatK has joined #openstack-cinder | 01:13 | |
| *** dflorea has quit IRC | 01:17 | |
| *** scottda_ has quit IRC | 01:18 | |
| *** yamada-h has joined #openstack-cinder | 01:20 | |
| *** Lee1092 has joined #openstack-cinder | 01:24 | |
| *** dflorea has joined #openstack-cinder | 01:25 | |
| *** yamada-h has quit IRC | 01:26 | |
| *** scottda_ has joined #openstack-cinder | 01:29 | |
| *** dflorea has quit IRC | 01:30 | |
| *** ebalduf has joined #openstack-cinder | 01:30 | |
| *** dflorea has joined #openstack-cinder | 01:30 | |
| *** divya has joined #openstack-cinder | 01:31 | |
| divya | hi.. i am facing issues in configuring the hp3par storage in devstack | 01:32 |
| divya | getting error"Either WS API not running or invalid" | 01:33 |
| divya | can anybody please help in the configuation | 01:33 |
| *** ebalduf has quit IRC | 01:34 | |
| *** dflorea has quit IRC | 01:35 | |
| *** scottda_ has quit IRC | 01:39 | |
| *** divya has quit IRC | 01:39 | |
| *** melwitt has quit IRC | 01:40 | |
| *** thingee has quit IRC | 01:40 | |
| *** dims_ has quit IRC | 01:51 | |
| *** esker has quit IRC | 01:55 | |
| *** fanyaohong has quit IRC | 02:01 | |
| *** Trozz_ has joined #openstack-cinder | 02:01 | |
| *** boris-42 has quit IRC | 02:02 | |
| *** mgagne has quit IRC | 02:02 | |
| *** adam_g has quit IRC | 02:02 | |
| *** Trozz has quit IRC | 02:02 | |
| *** jbernard has quit IRC | 02:02 | |
| *** boris-42 has joined #openstack-cinder | 02:05 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Brick: Fix race in removing iSCSI device https://review.openstack.org/168506 | 02:06 |
| *** adam_g has joined #openstack-cinder | 02:07 | |
| *** adam_g has quit IRC | 02:07 | |
| *** adam_g has joined #openstack-cinder | 02:07 | |
| *** mgagne has joined #openstack-cinder | 02:08 | |
| *** mgagne is now known as Guest39926 | 02:08 | |
| *** akerr_ has quit IRC | 02:12 | |
| *** julim has quit IRC | 02:12 | |
| *** fanyaohong has joined #openstack-cinder | 02:14 | |
| *** patrickeast has quit IRC | 02:21 | |
| *** harlowja is now known as harlowja_away | 02:22 | |
| *** yamada-h has joined #openstack-cinder | 02:23 | |
| *** yamada-h has quit IRC | 02:27 | |
| *** ebalduf has joined #openstack-cinder | 02:31 | |
| *** Yogi1 has quit IRC | 02:34 | |
| *** ebalduf has quit IRC | 02:35 | |
| *** marcusvrn has joined #openstack-cinder | 02:38 | |
| *** dims has joined #openstack-cinder | 02:51 | |
| *** jbernard has joined #openstack-cinder | 02:52 | |
| *** dims has quit IRC | 02:57 | |
| *** mdbooth has quit IRC | 03:00 | |
| *** Mandell has joined #openstack-cinder | 03:02 | |
| *** marcusvrn has quit IRC | 03:03 | |
| *** mdbooth has joined #openstack-cinder | 03:05 | |
| *** julim has joined #openstack-cinder | 03:08 | |
| *** thingee has joined #openstack-cinder | 03:17 | |
| *** boris-42 has quit IRC | 03:18 | |
| *** Ilja has joined #openstack-cinder | 03:21 | |
| *** yamada-h has joined #openstack-cinder | 03:23 | |
| *** yamada-h has quit IRC | 03:28 | |
| *** ebalduf has joined #openstack-cinder | 03:31 | |
| *** xyang1 has quit IRC | 03:32 | |
| *** ebalduf has quit IRC | 03:36 | |
| *** enterprisedc has quit IRC | 03:36 | |
| *** yamada-h has joined #openstack-cinder | 03:41 | |
| *** yamada-h has quit IRC | 03:41 | |
| *** Mandell has quit IRC | 03:42 | |
| *** Mandell has joined #openstack-cinder | 03:43 | |
| *** patrickeast has joined #openstack-cinder | 03:44 | |
| *** coolsvap|afk is now known as coolsvap | 03:48 | |
| *** yamada-h has joined #openstack-cinder | 03:51 | |
| *** yamada-h has quit IRC | 03:51 | |
| *** rongze has joined #openstack-cinder | 03:52 | |
| *** pradipta has joined #openstack-cinder | 03:55 | |
| *** yamada-h has joined #openstack-cinder | 04:04 | |
| *** eharney has quit IRC | 04:05 | |
| *** rwsu has quit IRC | 04:07 | |
| *** eharney has joined #openstack-cinder | 04:08 | |
| *** sks has joined #openstack-cinder | 04:08 | |
| *** _cjones_ has joined #openstack-cinder | 04:19 | |
| *** _cjones_ has quit IRC | 04:19 | |
| *** _cjones_ has joined #openstack-cinder | 04:20 | |
| *** yamada-h has quit IRC | 04:20 | |
| *** aswadr has joined #openstack-cinder | 04:21 | |
| *** patrickeast has quit IRC | 04:21 | |
| *** Ilja has quit IRC | 04:23 | |
| *** Maike has quit IRC | 04:27 | |
| *** _cjones_ has quit IRC | 04:28 | |
| *** ebalduf has joined #openstack-cinder | 04:32 | |
| *** ebalduf has quit IRC | 04:37 | |
| *** BharatK has quit IRC | 04:39 | |
| *** coolsvap is now known as coolsvap|afk | 04:41 | |
| *** anshul has joined #openstack-cinder | 04:44 | |
| *** annegentle has joined #openstack-cinder | 04:49 | |
| *** annashen has joined #openstack-cinder | 04:51 | |
| *** coolsvap|afk is now known as coolsvap | 04:53 | |
| *** annegentle has quit IRC | 04:54 | |
| *** annashen has quit IRC | 04:56 | |
| *** tshefi has joined #openstack-cinder | 05:00 | |
| *** Ilja has joined #openstack-cinder | 05:06 | |
| *** BharatK has joined #openstack-cinder | 05:08 | |
| *** cdelatte has quit IRC | 05:10 | |
| *** nshaikh has joined #openstack-cinder | 05:13 | |
| *** garysmith_ has quit IRC | 05:13 | |
| *** julim has quit IRC | 05:15 | |
| *** deepakcs has joined #openstack-cinder | 05:20 | |
| *** changbl has joined #openstack-cinder | 05:31 | |
| *** tshefi has quit IRC | 05:31 | |
| *** sgotliv has joined #openstack-cinder | 05:32 | |
| *** rushiagr_away is now known as rushiagr | 05:33 | |
| *** ebalduf has joined #openstack-cinder | 05:33 | |
| *** ebalduf has quit IRC | 05:37 | |
| *** dims has joined #openstack-cinder | 05:41 | |
| *** shyama has joined #openstack-cinder | 05:43 | |
| *** tshefi has joined #openstack-cinder | 05:45 | |
| *** lpetrut has joined #openstack-cinder | 05:46 | |
| *** cdelatte has joined #openstack-cinder | 05:47 | |
| *** dims has quit IRC | 05:47 | |
| *** delattec has joined #openstack-cinder | 05:47 | |
| *** dulek has joined #openstack-cinder | 05:50 | |
| *** emagana has joined #openstack-cinder | 05:50 | |
| *** ronis_ has joined #openstack-cinder | 05:54 | |
| *** ankit_ag has joined #openstack-cinder | 05:54 | |
| *** zerda has joined #openstack-cinder | 05:58 | |
| *** nlevinki has joined #openstack-cinder | 05:59 | |
| *** yamada-h has joined #openstack-cinder | 06:01 | |
| *** HoloIRCUser1 has joined #openstack-cinder | 06:02 | |
| *** yamada-h has quit IRC | 06:06 | |
| openstackgerrit | Michal Dulko proposed openstack/python-cinderclient: Add set_management_url to cinderclient.client https://review.openstack.org/153274 | 06:10 |
| *** vbala has joined #openstack-cinder | 06:16 | |
| *** HoloIRCUser1 has quit IRC | 06:17 | |
| openstackgerrit | Michal Dulko proposed openstack/python-cinderclient: Add reset-state function for backups https://review.openstack.org/117436 | 06:21 |
| *** anshul has quit IRC | 06:31 | |
| *** anshul has joined #openstack-cinder | 06:31 | |
| *** ebalduf has joined #openstack-cinder | 06:34 | |
| *** ebalduf has quit IRC | 06:38 | |
| openstackgerrit | Bharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Returning provider location of volume from snapshot https://review.openstack.org/171482 | 06:39 |
| *** abehl has joined #openstack-cinder | 06:41 | |
| *** anshul has quit IRC | 06:41 | |
| *** dflorea has joined #openstack-cinder | 06:41 | |
| *** yamada-h has joined #openstack-cinder | 06:41 | |
| *** emagana has quit IRC | 06:42 | |
| *** emagana has joined #openstack-cinder | 06:42 | |
| openstackgerrit | Mike Perez proposed openstack/cinder: Revert "Removing Windows drivers" https://review.openstack.org/171484 | 06:44 |
| *** yamada-h has quit IRC | 06:46 | |
| *** emagana has quit IRC | 06:47 | |
| *** annashen has joined #openstack-cinder | 06:53 | |
| *** dulek has quit IRC | 06:56 | |
| *** anshul has joined #openstack-cinder | 06:56 | |
| *** annashen has quit IRC | 06:57 | |
| *** abehl has quit IRC | 06:58 | |
| *** thingee has quit IRC | 07:00 | |
| *** sks has quit IRC | 07:00 | |
| *** dulek has joined #openstack-cinder | 07:00 | |
| *** pschaef has joined #openstack-cinder | 07:02 | |
| *** markus_z has joined #openstack-cinder | 07:03 | |
| DuncanT | check-tempest-dsvm-full-ceph check-tempest-dsvm-full-glusterfs-nv check-tempest-> dsvm-full-sheepdog-nv have all been failing on every patch for a while | 07:08 |
| *** yamada-h has joined #openstack-cinder | 07:12 | |
| *** sks has joined #openstack-cinder | 07:14 | |
| *** afazekas has joined #openstack-cinder | 07:23 | |
| *** dflorea has quit IRC | 07:24 | |
| *** dflorea has joined #openstack-cinder | 07:24 | |
| *** eharney has quit IRC | 07:27 | |
| *** nshaikh has quit IRC | 07:28 | |
| *** dflorea has quit IRC | 07:29 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails https://review.openstack.org/160682 | 07:29 |
| *** anshul has quit IRC | 07:29 | |
| *** sks has quit IRC | 07:30 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails https://review.openstack.org/160682 | 07:33 |
| *** ebalduf has joined #openstack-cinder | 07:35 | |
| *** Mandell has quit IRC | 07:37 | |
| *** anshul has joined #openstack-cinder | 07:38 | |
| *** ebalduf has quit IRC | 07:39 | |
| openstackgerrit | Pradeep Sathasivam proposed openstack/cinder: Removed sleep between the time a command is sent and 'YES' is sent to confim the operation. https://review.openstack.org/171505 | 07:41 |
| *** e0ne has joined #openstack-cinder | 07:41 | |
| *** eharney has joined #openstack-cinder | 07:42 | |
| *** Ilja has quit IRC | 07:44 | |
| *** lpetrut has quit IRC | 07:45 | |
| *** sks has joined #openstack-cinder | 07:47 | |
| *** e0ne has quit IRC | 07:48 | |
| *** e0ne has joined #openstack-cinder | 07:52 | |
| *** shyama has quit IRC | 07:53 | |
| *** shyama has joined #openstack-cinder | 07:57 | |
| *** e0ne is now known as e0ne_ | 07:57 | |
| *** e0ne_ is now known as e0ne | 08:00 | |
| *** sgotliv has quit IRC | 08:01 | |
| *** jistr has joined #openstack-cinder | 08:02 | |
| * DuncanT was just about to approve https://review.openstack.org/160682 witht he suggestion we file a low-hanging-fruit for the cleanup nit. Never mind, will get back to it once jenkins has caught up | 08:08 | |
| *** lpetrut has joined #openstack-cinder | 08:10 | |
| *** e0ne has quit IRC | 08:10 | |
| DuncanT | Who is our contact for the microsoft CI? It would be nice to see it vote properly on https://review.openstack.org/#/c/171484/ | 08:11 |
| DuncanT | (re-add the windows driver) | 08:11 |
| *** e0ne has joined #openstack-cinder | 08:11 | |
| DuncanT | ociuhandu: Is that you? | 08:11 |
| *** e0ne has quit IRC | 08:13 | |
| *** jordanP has joined #openstack-cinder | 08:14 | |
| *** cppforlife_ has quit IRC | 08:15 | |
| *** cppforlife_ has joined #openstack-cinder | 08:16 | |
| *** e0ne has joined #openstack-cinder | 08:17 | |
| *** Lee1092 has quit IRC | 08:21 | |
| *** e0ne has quit IRC | 08:21 | |
| *** Lee1092 has joined #openstack-cinder | 08:22 | |
| *** e0ne has joined #openstack-cinder | 08:24 | |
| *** e0ne has quit IRC | 08:31 | |
| *** ho has quit IRC | 08:34 | |
| *** ebalduf has joined #openstack-cinder | 08:35 | |
| ociuhandu | DuncanT: I am looking into that now | 08:37 |
| DuncanT | ociuhandu: Great, thanks | 08:37 |
| *** ebalduf has quit IRC | 08:40 | |
| ociuhandu | DuncanT: I am now removing the patches we have added to work around the commit so we can use the drivers for testing. | 08:41 |
| *** xyang has quit IRC | 08:46 | |
| DuncanT | ociuhandu: Can you kick off a recheck on the driver re-add when you think the ci is ready, please? | 08:46 |
| ociuhandu | DuncanT: sure thing, should take long to check the changes are fine and I’ll issue a recheck | 08:47 |
| *** alecv has joined #openstack-cinder | 08:48 | |
| *** yamada-h has quit IRC | 08:48 | |
| alexpilotti | DuncanT: we were cherry picking the driver on top of the zuul review patchset under test, which now conflicts with the revert (as expected), easy fix :-) | 08:48 |
| *** anshul has quit IRC | 08:56 | |
| *** ronis_ has quit IRC | 08:56 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Handle concurrent inventory folder create https://review.openstack.org/167989 | 08:59 |
| *** theanalyst has quit IRC | 09:04 | |
| *** theanalyst has joined #openstack-cinder | 09:07 | |
| *** ociuhandu has quit IRC | 09:11 | |
| *** sks has quit IRC | 09:11 | |
| *** e0ne has joined #openstack-cinder | 09:16 | |
| *** anshul has joined #openstack-cinder | 09:16 | |
| *** dims has joined #openstack-cinder | 09:19 | |
| openstackgerrit | Ankit Agrawal proposed openstack/cinder: Fix missing translations for log messages https://review.openstack.org/164692 | 09:20 |
| openstackgerrit | yogeshprasad proposed openstack/cinder: Fixes snapshot creation failure in CloudByte driver. https://review.openstack.org/167978 | 09:22 |
| *** dims has quit IRC | 09:24 | |
| openstackgerrit | Ankit Agrawal proposed openstack/cinder: Fix missing translations for log messages https://review.openstack.org/164697 | 09:25 |
| *** sks has joined #openstack-cinder | 09:25 | |
| *** pradipta has quit IRC | 09:30 | |
| *** ebalduf has joined #openstack-cinder | 09:36 | |
| *** heyun has quit IRC | 09:37 | |
| *** ebalduf has quit IRC | 09:40 | |
| *** yamada-h has joined #openstack-cinder | 09:41 | |
| *** e0ne is now known as e0ne_ | 09:41 | |
| *** yamada-h has quit IRC | 09:46 | |
| *** e0ne_ is now known as e0ne | 09:49 | |
| *** bkopilov has quit IRC | 09:52 | |
| *** jordanP has quit IRC | 09:55 | |
| *** annashen has joined #openstack-cinder | 09:55 | |
| *** ronis has joined #openstack-cinder | 10:00 | |
| *** annashen has quit IRC | 10:00 | |
| *** ociuhandu has joined #openstack-cinder | 10:00 | |
| *** dims has joined #openstack-cinder | 10:08 | |
| *** sr_slack has joined #openstack-cinder | 10:11 | |
| *** rongze has quit IRC | 10:13 | |
| *** sks has quit IRC | 10:13 | |
| *** jordanP has joined #openstack-cinder | 10:20 | |
| *** IanGovett has joined #openstack-cinder | 10:21 | |
| *** sks has joined #openstack-cinder | 10:26 | |
| *** sr_slack is now known as teh_slack | 10:30 | |
| *** jordanP has quit IRC | 10:35 | |
| *** ebalduf has joined #openstack-cinder | 10:37 | |
| *** jordanP has joined #openstack-cinder | 10:38 | |
| *** ebalduf has quit IRC | 10:41 | |
| nikesh | DuncanT: any idea when L release will open for cinder blueprints | 10:46 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add volume status to error message in backup create flow https://review.openstack.org/171566 | 10:52 |
| e0ne | nikesh: afaik, it will be opened after RC-1 | 10:52 |
| *** Maike has joined #openstack-cinder | 10:52 | |
| *** annegentle has joined #openstack-cinder | 10:53 | |
| *** annegentle has quit IRC | 10:58 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add volume status to error message in backup create flow https://review.openstack.org/171566 | 11:07 |
| DuncanT | nikesh: As soon as RC1 is cut, I think | 11:08 |
| DuncanT | nikesh: Right now we'd like people focused on release testing | 11:08 |
| DuncanT | nikesh: and any blocker bugs | 11:08 |
| DuncanT | nikesh: https://etherpad.openstack.org/p/cinder-kilo-rc-priorities is a good place to start | 11:09 |
| nikesh | e0ne DuncanT : thanks https://launchpad.net/cinder/+milestone/liberty-1 | 11:12 |
| nikesh | "Yes. Drivers can target bugs and blueprints to this milestone. " | 11:13 |
| *** coolsvap is now known as coolsvap|afk | 11:15 | |
| openstackgerrit | Julien Danjou proposed openstack/cinder: Leverage timeutils, drop strtime() usage https://review.openstack.org/165798 | 11:20 |
| nikesh | e0ne DuncanT : is there any deadline finalized for blueprint approval for cinder drivers for L release because seems like blueprint approval started for L1 https://launchpad.net/cinder/+milestone/liberty-1 | 11:24 |
| *** sgotliv has joined #openstack-cinder | 11:31 | |
| *** vbala has quit IRC | 11:31 | |
| *** deepakcs has quit IRC | 11:36 | |
| *** Yogi1 has joined #openstack-cinder | 11:38 | |
| *** ebalduf has joined #openstack-cinder | 11:38 | |
| *** Mandell has joined #openstack-cinder | 11:38 | |
| DuncanT | nikesh: I don't think there is one set yet | 11:38 |
| *** Yogi11 has joined #openstack-cinder | 11:39 | |
| nikesh | ok but better to submit blueprint as soon as possible | 11:40 |
| nikesh | :) | 11:40 |
| *** delatte has joined #openstack-cinder | 11:40 | |
| *** aix has joined #openstack-cinder | 11:42 | |
| *** Yogi1 has quit IRC | 11:42 | |
| *** Mandell has quit IRC | 11:42 | |
| *** ebalduf has quit IRC | 11:42 | |
| *** cdelatte has quit IRC | 11:43 | |
| *** yamada-h has joined #openstack-cinder | 11:43 | |
| *** delattec has quit IRC | 11:43 | |
| *** ociuhandu has quit IRC | 11:43 | |
| *** yamada-h has quit IRC | 11:47 | |
| *** davechen has joined #openstack-cinder | 11:48 | |
| *** davechen has left #openstack-cinder | 11:49 | |
| *** ganso_ has joined #openstack-cinder | 11:50 | |
| *** Chilli_ has joined #openstack-cinder | 11:50 | |
| *** bkopilov has joined #openstack-cinder | 11:50 | |
| *** bkopilov has quit IRC | 11:52 | |
| *** bkopilov has joined #openstack-cinder | 11:53 | |
| *** Chilli_ has left #openstack-cinder | 11:53 | |
| *** annegentle has joined #openstack-cinder | 11:54 | |
| *** zerda has quit IRC | 11:54 | |
| *** bkopilov has quit IRC | 11:57 | |
| *** ociuhandu has joined #openstack-cinder | 11:57 | |
| *** bkopilov has joined #openstack-cinder | 11:57 | |
| *** annashen has joined #openstack-cinder | 11:58 | |
| *** annegentle has quit IRC | 11:59 | |
| *** bkopilov has quit IRC | 12:01 | |
| *** bkopilov has joined #openstack-cinder | 12:01 | |
| *** annashen has quit IRC | 12:02 | |
| *** marcusvrn has joined #openstack-cinder | 12:04 | |
| openstackgerrit | Bharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Returning provider location of volume from snapshot https://review.openstack.org/171482 | 12:05 |
| *** bkopilov has quit IRC | 12:05 | |
| *** e0ne is now known as e0ne_ | 12:05 | |
| *** nshaikh has joined #openstack-cinder | 12:08 | |
| *** IanGovett1 has joined #openstack-cinder | 12:10 | |
| *** Yogi11 has quit IRC | 12:10 | |
| *** IanGovett has quit IRC | 12:12 | |
| *** nshaikh has left #openstack-cinder | 12:12 | |
| *** bkopilov has joined #openstack-cinder | 12:12 | |
| *** bswartz has quit IRC | 12:15 | |
| *** bkopilov has quit IRC | 12:15 | |
| *** e0ne_ has quit IRC | 12:15 | |
| *** bkopilov has joined #openstack-cinder | 12:17 | |
| *** e0ne has joined #openstack-cinder | 12:17 | |
| ankit_ag | jungleboyj: I have addressed all your review comments on https://review.openstack.org/#/c/164692/ and https://review.openstack.org/#/c/164697/ to change "%" => "," | 12:18 |
| ankit_ag | could you please take a look when you get some time | 12:18 |
| *** akerr has joined #openstack-cinder | 12:19 | |
| ankit_ag | e0ne: Hi, I have addressed your review comments on https://review.openstack.org/#/c/164692/ to remove checks from hacking | 12:22 |
| *** delattec has joined #openstack-cinder | 12:22 | |
| ankit_ag | Could you please take a look when you get some time | 12:22 |
| *** bkopilov has quit IRC | 12:22 | |
| *** delatte has quit IRC | 12:22 | |
| *** ociuhandu has quit IRC | 12:22 | |
| *** boris-42 has joined #openstack-cinder | 12:23 | |
| *** jdandrea has quit IRC | 12:23 | |
| *** bkopilov has joined #openstack-cinder | 12:24 | |
| e0ne | ankit_ag: sure, i'll do. thanks for the update | 12:25 |
| ankit_ag | e0ne: Thank you :) | 12:26 |
| *** bkopilov has quit IRC | 12:28 | |
| *** BharatK has quit IRC | 12:28 | |
| *** jdandrea has joined #openstack-cinder | 12:38 | |
| *** ebalduf has joined #openstack-cinder | 12:38 | |
| *** Yogi1 has joined #openstack-cinder | 12:40 | |
| *** ebalduf has quit IRC | 12:43 | |
| *** jdandrea has quit IRC | 12:46 | |
| openstackgerrit | Kamil Rykowski proposed openstack/cinder: Leverage dict comprehension in PEP-0274 https://review.openstack.org/143851 | 12:50 |
| *** jdandrea has joined #openstack-cinder | 12:51 | |
| *** bswartz has joined #openstack-cinder | 12:51 | |
| *** tshefi has quit IRC | 12:55 | |
| *** annegentle has joined #openstack-cinder | 12:55 | |
| *** sks has quit IRC | 13:01 | |
| *** dulek has quit IRC | 13:02 | |
| openstackgerrit | Ankit Agrawal proposed openstack/cinder: Fix missing translations for log messages https://review.openstack.org/164697 | 13:03 |
| smcginnis | e0ne: I left a comment in ankit_ag's patch. I think it could still go ahead if you agree. | 13:04 |
| e0ne | smcginnis: you're quick! | 13:04 |
| smcginnis | e0ne: Not the one he just uploaded. :) | 13:04 |
| e0ne | smcginnis: agree that it should be targeted to L | 13:04 |
| smcginnis | e0ne: Yeah, at this point I'm guessing stuff like this should probably wait. | 13:05 |
| e0ne | smcginnis: i'm going to make my patch with simple log change to L too:( | 13:05 |
| smcginnis | e0ne: Seems like we're getting close to L opening up at least. | 13:05 |
| e0ne | +1 | 13:06 |
| openstackgerrit | Ankit Agrawal proposed openstack/cinder: Fix missing translations for log messages https://review.openstack.org/164692 | 13:09 |
| *** asrangne has joined #openstack-cinder | 13:09 | |
| smcginnis | I think I've found a new pet peeve. Translations being done and actual log strings being compared in unit tests. :) | 13:09 |
| smcginnis | ankit_ag: Thanks! Looks good. | 13:11 |
| *** julim has joined #openstack-cinder | 13:11 | |
| ankit_ag | smcginnis: Thank you ! | 13:11 |
| *** aswadr has quit IRC | 13:12 | |
| *** sks has joined #openstack-cinder | 13:13 | |
| *** ndipanov has quit IRC | 13:18 | |
| *** ndipanov has joined #openstack-cinder | 13:18 | |
| *** e0ne is now known as e0ne_ | 13:19 | |
| *** mriedem_away is now known as mriedem | 13:19 | |
| *** bkopilov has joined #openstack-cinder | 13:19 | |
| *** bkopilov has quit IRC | 13:19 | |
| ankit_ag | e0ne: you're going to make your patch with simple log change to L too, does that mean the patches under review isn't intended to merge or you are planning to submit further changes ? | 13:23 |
| *** Longgeek has quit IRC | 13:23 | |
| *** bkopilov has joined #openstack-cinder | 13:24 | |
| alexpilotti | DuncanT: https://review.openstack.org/#/c/171484/ The Windows SMB3 CI is positive | 13:24 |
| *** ociuhandu has joined #openstack-cinder | 13:24 | |
| *** Mandell has joined #openstack-cinder | 13:25 | |
| alexpilotti | DuncanT: the other 2 (Linux SMB3 and iSCSI) are still processing a recheck, but those are unrelated to this patch | 13:25 |
| *** bkopilov has quit IRC | 13:28 | |
| *** e0ne_ has quit IRC | 13:28 | |
| *** bkopilov has joined #openstack-cinder | 13:30 | |
| *** jungleboyj has quit IRC | 13:32 | |
| *** bkopilov has quit IRC | 13:32 | |
| *** xyang1 has joined #openstack-cinder | 13:32 | |
| *** bkopilov has joined #openstack-cinder | 13:33 | |
| *** dustins has joined #openstack-cinder | 13:35 | |
| *** bkopilov has quit IRC | 13:36 | |
| *** rushil has joined #openstack-cinder | 13:39 | |
| *** ebalduf has joined #openstack-cinder | 13:39 | |
| *** ebalduf has quit IRC | 13:43 | |
| *** e0ne has joined #openstack-cinder | 13:46 | |
| *** xyang has joined #openstack-cinder | 13:46 | |
| e0ne | ankit_ag: yes, we're going to not merge anythong except https://etherpad.openstack.org/p/cinder-kilo-rc-priorities | 13:46 |
| DuncanT | alexpilotti: That's great, thanks | 13:46 |
| *** rushiagr is now known as rushiagr_away | 13:46 | |
| *** marcusvrn has quit IRC | 13:47 | |
| *** Mandell has quit IRC | 13:48 | |
| *** jaypipes has joined #openstack-cinder | 13:48 | |
| ankit_ag | e0ne: Could you please help to know about how should we proceed with pending reviews | 13:48 |
| *** xyang has quit IRC | 13:49 | |
| *** dims has quit IRC | 13:52 | |
| *** dims has joined #openstack-cinder | 13:53 | |
| *** jistr has quit IRC | 13:54 | |
| *** jistr has joined #openstack-cinder | 13:54 | |
| *** bkopilov has joined #openstack-cinder | 13:56 | |
| smcginnis | ankit_ag: I believe we can keep reviewing and getting them ready. Then once K is cut things open up again and anything ready can be merged. | 13:56 |
| *** aix has quit IRC | 13:56 | |
| ankit_ag | smcginnis: Yeah, We need to rebase these patches untill K is cut. | 13:58 |
| *** chlong has quit IRC | 13:58 | |
| ankit_ag | Thank you for your reply :) | 13:59 |
| *** nlevinki has quit IRC | 13:59 | |
| smcginnis | NP :) | 14:01 |
| *** IanGovett1 has quit IRC | 14:09 | |
| *** jistr has quit IRC | 14:10 | |
| *** Lee1092 has quit IRC | 14:10 | |
| *** ociuhandu has quit IRC | 14:11 | |
| *** ociuhandu_ has joined #openstack-cinder | 14:11 | |
| *** jistr has joined #openstack-cinder | 14:12 | |
| *** Yogi1 has quit IRC | 14:16 | |
| *** IanGovett has joined #openstack-cinder | 14:16 | |
| *** marcusvrn has joined #openstack-cinder | 14:17 | |
| *** sks has quit IRC | 14:18 | |
| *** jungleboyj has joined #openstack-cinder | 14:18 | |
| *** asrangne__ has joined #openstack-cinder | 14:20 | |
| *** mtanino has joined #openstack-cinder | 14:20 | |
| *** social has joined #openstack-cinder | 14:20 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/cinder: Use AF_INET6 if eventlet's green dns/socket module fails https://review.openstack.org/171671 | 14:21 |
| *** asrangne has quit IRC | 14:22 | |
| *** Yogi1 has joined #openstack-cinder | 14:22 | |
| *** aix has joined #openstack-cinder | 14:24 | |
| *** shyama has quit IRC | 14:26 | |
| *** deepakcs has joined #openstack-cinder | 14:27 | |
| *** juzuluag has joined #openstack-cinder | 14:28 | |
| deepakcs | dims, ping | 14:29 |
| dims | deepakcs: pong | 14:29 |
| deepakcs | dims, https://bugs.launchpad.net/cinder/+bug/1438027 - wondering why the prio was chosen as "low" | 14:29 |
| openstack | Launchpad bug 1438027 in OpenStack Compute (nova) "backing file path isn't relative path for file-based volume drivers using qcow2 snapshots" [Low,In progress] - Assigned to Deepak C Shetty (dpkshetty) | 14:29 |
| *** pschaef has quit IRC | 14:31 | |
| dims | deepakcs: that's the nova priority, not cinder...right? also, what would you like it to be? | 14:31 |
| *** rushiagr_away is now known as rushiagr | 14:32 | |
| *** sks has joined #openstack-cinder | 14:32 | |
| teh_slack | That seems like a nova problem. | 14:32 |
| deepakcs | dims, yes, i first opened bug against cinder, since problem is seen by cinder, but its bcos of not using proper libvirt flag in nova | 14:32 |
| deepakcs | so it affects both | 14:32 |
| deepakcs | i would expect it to be med if not high for both cidner and nova | 14:33 |
| deepakcs | dims, ^ | 14:33 |
| eharney | but the fix is only in Nova, right? | 14:33 |
| deepakcs | eharney, yes, fix is in nova only | 14:33 |
| aarefiev | geguileo: -1 means unlimited quota | 14:33 |
| dims | deepakcs: please add a few comments in the bug on why and we can flip it | 14:33 |
| geguileo | aarefiev: Mmmm, but that's something different, right? | 14:34 |
| aarefiev | geguileo: e.g it's used for volume types by default | 14:34 |
| geguileo | aarefiev: That would mean you actually set an unlimited quota, right? | 14:34 |
| *** pradipta has joined #openstack-cinder | 14:34 | |
| aarefiev | geguileo: yes | 14:34 |
| geguileo | aarefiev: And you think that's what should happen when you delete the quotas? | 14:35 |
| deepakcs | dims, sure, will do | 14:35 |
| aarefiev | geguileo: I'm not quite sure, but we could discuss it | 14:36 |
| geguileo | aarefiev: I think it shouldn't go to -1 | 14:37 |
| teh_slack | Maybe it should go to image size, or available free space? -1 seems inappropriate in all but the most extreme cases. | 14:38 |
| geguileo | Because then you are actually setting quotas and entering in the DB | 14:38 |
| marcusvrn | deepakcs: hi! I saw your email about boot_pattern test issue. Did you get any progress on that? | 14:38 |
| deepakcs | marcusvrn, yeah i updated the LP but with the details | 14:39 |
| deepakcs | marcusvrn, there are 2 issues, one with the gluster cinder driver and second with the way tempest "assumes" things in the testcase | 14:39 |
| deepakcs | marcusvrn, patches for both the issues are up | 14:39 |
| geguileo | aarefiev: If you want unlimited you should set them as such, right? | 14:39 |
| teh_slack | Everyone wants unlimited, but you have to have hardware to support the quota. | 14:40 |
| deepakcs | marcusvrn, scratch that, 1 patch is up, 2nd is still in works | 14:40 |
| marcusvrn | deepakcs: hmm...nice! because one of our driver fails sometimes in this test... | 14:41 |
| geguileo | teh_slack: We were discussing the quota delete bug where it also deletes usage and reservations | 14:41 |
| deepakcs | marcusvrn, looking at the python logging that gets dumped after the failure (in console.html) can help u debug, thats how i narrowed it down | 14:41 |
| aarefiev | geguileo: it will otherwise, for volume type quotas. We set default values for volumes, snapshots, backups from conf, for other default value -1 | 14:41 |
| geguileo | aarefiev: Those are the defaults, right? | 14:42 |
| aarefiev | geguileo: defaults from conf https://github.com/openstack/cinder/blob/master/cinder/quota.py#L36, otherwise https://github.com/openstack/cinder/blob/master/cinder/quota.py#L492 | 14:44 |
| *** haypo has joined #openstack-cinder | 14:44 | |
| marcusvrn | deepakcs: I did it..it seems a SSH issue... but I couldn't find how to solve it. have you ever seen this issue? http://177.84.241.119:10000/84/152284/9/silent/hds-hnas-iscsi/c7561bb/console.html.gz | 14:44 |
| *** jistr has quit IRC | 14:45 | |
| *** emagana has joined #openstack-cinder | 14:46 | |
| *** BharatK has joined #openstack-cinder | 14:47 | |
| *** lpabon has joined #openstack-cinder | 14:47 | |
| geguileo | aarefiev: If we delete project quotas doesn't it use default ones? | 14:48 |
| *** emagana_ has joined #openstack-cinder | 14:49 | |
| *** emagana has quit IRC | 14:50 | |
| geguileo | aarefiev: At least that's what I thought would happen | 14:50 |
| aarefiev | geguileo: yes, if we delete quotas, default will be used, I just thought about set all quota -1 , I don't know it is good idea | 14:50 |
| deepakcs | marcusvrn, ur problem seems more similar to https://bugs.launchpad.net/cinder/+bug/1439371 (ceph issue) | 14:51 |
| openstack | Launchpad bug 1439371 in Cinder "Volume creation from image fails for UEC+Ceph" [Undecided,In progress] - Assigned to Jon Bernard (jbernard) | 14:51 |
| *** jistr has joined #openstack-cinder | 14:51 | |
| deepakcs | marcusvrn, same testcase but issues are different based on the backend, so check your issue with the ceph one | 14:52 |
| geguileo | aarefiev: I don't think it's a good idea... I see it as a stack of quotas configuration | 14:52 |
| geguileo | aarefiev: First is default, then you change those you want | 14:53 |
| geguileo | aarefiev: And the rest are applied from default | 14:53 |
| jbernard | marcusvrn: ssh can fail for a few reasons, for me it seems to fail most often because the instance, although in the ACTIVE state, didn't actually boot | 14:53 |
| geguileo | aarefiev: And if you delete project quotas you don't set them to -1, you remove them so defaults can be applied | 14:53 |
| jbernard | marcusvrn: i would verify the boot device and kvm params are correct to boot the instance | 14:53 |
| *** rongze has joined #openstack-cinder | 14:54 | |
| jbernard | marcusvrn: console should give you an indicator | 14:54 |
| *** erlon has joined #openstack-cinder | 14:55 | |
| aarefiev | geguileo: sure, but it seams confusing for me, 'delete quota' means 'reset quota to default' | 14:56 |
| geguileo | aarefiev: I see what you mean :) | 14:56 |
| marcusvrn | deepakcs: hm..ok! I'll check with ceph | 14:57 |
| geguileo | aarefiev: To me it means "delete specific quota rule" | 14:57 |
| *** li_zhang has joined #openstack-cinder | 14:57 | |
| marcusvrn | jbernard: yeah..I guess I saw someone with problem to use kvm and have switched to qemu on top of kvm... | 14:58 |
| geguileo | aarefiev: For me setting -1 would be set quota to unlimited and not delete quotas XD | 14:58 |
| jbernard | marcusvrn: that's actually what i mean | 14:59 |
| marcusvrn | jbernard: yep, I'll try it out! | 15:00 |
| jbernard | marcusvrn: the important part is that if a volume doesn't have the right bootable bits, it my appear to have booted when it actually did not | 15:00 |
| *** Mandell has joined #openstack-cinder | 15:00 | |
| jbernard | marcusvrn: and your first sign will be ssh times out, which can be misleading initially | 15:01 |
| *** Mandell has quit IRC | 15:01 | |
| aarefiev | geguileo: I see your point :), it is more terminology problem | 15:01 |
| geguileo | aarefiev: I agree :) | 15:01 |
| marcusvrn | jbernard: hm.. I see | 15:02 |
| marcusvrn | jbernard: deepakcs thanks! If I have some progress, I'll let you guys know. | 15:02 |
| geguileo | aarefiev: I believe we are going to have a thread in dev list regarding this, because Nova has the same issue and they don't like my patch. XD | 15:02 |
| *** jaypipes has quit IRC | 15:03 | |
| aarefiev | geguileo: good idea | 15:03 |
| geguileo | aarefiev: That way both will have same behavior | 15:03 |
| deepakcs | jbernard, np, BTW FYI the glusterfs failure in test_volume_boot_pattern is related to 2 issues, one with glusterfs driver and other with tempest testcase itself | 15:03 |
| deepakcs | jbernard, LP bug @ https://bugs.launchpad.net/cinder/+bug/1441050 has the details | 15:04 |
| openstack | Launchpad bug 1441050 in Cinder "test_volume_boot_pattern tempest test failure for glusterfs backend" [Undecided,In progress] - Assigned to Bharat Kumar Kobagana (bharat-kobagana) | 15:04 |
| jbernard | deepakcs: cool, glad you're making progress | 15:04 |
| *** kvidvans_ has joined #openstack-cinder | 15:04 | |
| *** jaypipes has joined #openstack-cinder | 15:05 | |
| *** emagana_ has quit IRC | 15:05 | |
| *** tsekiyama has joined #openstack-cinder | 15:05 | |
| *** asselin has joined #openstack-cinder | 15:05 | |
| *** emagana has joined #openstack-cinder | 15:06 | |
| *** jaypipes has quit IRC | 15:06 | |
| aarefiev | geguileo: agree, we should have same behavior to keep consistency | 15:09 |
| *** sgotliv has quit IRC | 15:09 | |
| openstackgerrit | Alex Meade proposed openstack/cinder: Add tracing facility to NetApp ONTAP drivers https://review.openstack.org/157439 | 15:10 |
| openstackgerrit | Alex Meade proposed openstack/cinder: Add tracing facility to NetApp ESeries driver https://review.openstack.org/170580 | 15:10 |
| *** e0ne is now known as e0ne_ | 15:10 | |
| *** rongze has quit IRC | 15:11 | |
| *** Zhong-jun has joined #openstack-cinder | 15:13 | |
| *** ronis has quit IRC | 15:15 | |
| *** jistr is now known as jistr|mtg | 15:16 | |
| *** e0ne_ has quit IRC | 15:16 | |
| *** _cjones_ has joined #openstack-cinder | 15:20 | |
| *** rmesta has joined #openstack-cinder | 15:21 | |
| *** rasoto has joined #openstack-cinder | 15:21 | |
| *** shyama has joined #openstack-cinder | 15:21 | |
| haypo | geguileo, i didn't follow the whole discussion, but i agree that the quota issue must be handled the same way in cinder & nova | 15:22 |
| haypo | geguileo, (fix the current API or add a new API) | 15:22 |
| geguileo | haypo: I agree | 15:22 |
| *** ebalduf has joined #openstack-cinder | 15:22 | |
| geguileo | haypo: And since this is a relevant matter affecting 2 components it's better to use dev list (I think) | 15:23 |
| rasoto | hello all, I am trying to find information on how cinder manages the volumes via netapp. I look through the documentation but it seems that is not provided there. In particular I am trying to find out what does cinder needs from netapp to create/delete volumes. Does it need the space already pre-proovisioned? Is it a one to one releationship between volume and luns? | 15:23 |
| *** _cjones_ has quit IRC | 15:24 | |
| *** yk has joined #openstack-cinder | 15:24 | |
| *** sgotliv has joined #openstack-cinder | 15:25 | |
| *** sgotliv has quit IRC | 15:27 | |
| smcginnis | bswartz, tbarron ^^ | 15:27 |
| *** ebalduf has quit IRC | 15:27 | |
| *** sgotliv has joined #openstack-cinder | 15:27 | |
| *** e0ne has joined #openstack-cinder | 15:29 | |
| *** Maike has quit IRC | 15:29 | |
| DuncanT | geguileo: I agree your interpretation makes the most sense from an operator's point of view | 15:30 |
| *** jistr|mtg is now known as jistr | 15:30 | |
| DuncanT | geguileo: I'm happy to join a thread on the dev list to that effect | 15:31 |
| geguileo | DuncanT: Great, we'll start one | 15:32 |
| *** lpetrut has quit IRC | 15:32 | |
| *** setmason has joined #openstack-cinder | 15:33 | |
| *** jdurgin1 has joined #openstack-cinder | 15:34 | |
| tbarron | smcginnis: ty | 15:34 |
| smcginnis | tbarron: Anytime. :) | 15:35 |
| tbarron | rasoto: your best bet will be to take your question to #openstack-netapp, as we'll have more folks on there | 15:36 |
| rasoto | tbarron: thank you, will do :) | 15:36 |
| tbarron | rasoto: we just implemented manage/unmanage in Kilo and our docs for Kilo aren't out yet | 15:37 |
| tbarron | rasoto: we back Cinder volumes with files (for NFS) or LUNs (for iSCSI and fibre channel) | 15:37 |
| *** rwsu has joined #openstack-cinder | 15:37 | |
| tbarron | rasoto: so manage-existing needs a reference to one of these | 15:37 |
| tbarron | rasoto: on the other channel there are folks who may have done this recently and can tell you more than I have in my head right now about how to reference the pre-existing backing-store | 15:38 |
| rasoto | tbarron : thank you, for this info it has clear some of the doubts I had, I will direct some other questions to the other channel | 15:40 |
| tbarron | rasoto: cool | 15:40 |
| *** afazekas has quit IRC | 15:43 | |
| *** Apoorva has joined #openstack-cinder | 15:44 | |
| *** ozialien has joined #openstack-cinder | 15:44 | |
| *** zongliang has joined #openstack-cinder | 15:45 | |
| *** ozialien has quit IRC | 15:47 | |
| li_zhang | tbarron:as you mentioned NFS in cinder, i have just something not sure for this. i saw netapp implemented much more code in NFS driver(cmode/7mode) compare to other drivers, is it because netapp device support file operation like snapshot/clone via outband? | 15:50 |
| openstackgerrit | Kamil Rykowski proposed openstack/cinder: Update openstack-common reference in openstack/common/README https://review.openstack.org/171705 | 15:50 |
| *** yuriy_n17 has joined #openstack-cinder | 15:51 | |
| *** asselin has quit IRC | 15:51 | |
| *** thingee has joined #openstack-cinder | 15:52 | |
| *** dannywilson has joined #openstack-cinder | 15:52 | |
| *** xyang has joined #openstack-cinder | 15:54 | |
| *** markus_z has quit IRC | 15:55 | |
| li_zhang | tbarron: we tried the base NFS driver to manage our storage device and it works, so i am not sure if it is necessary to implement a vender NFS driver | 15:55 |
| *** jistr has quit IRC | 15:56 | |
| *** patrickeast has joined #openstack-cinder | 15:56 | |
| *** _cjones_ has joined #openstack-cinder | 15:57 | |
| *** anshul has quit IRC | 15:57 | |
| *** ozialien has joined #openstack-cinder | 15:59 | |
| *** thangp has joined #openstack-cinder | 15:59 | |
| *** annashen has joined #openstack-cinder | 16:01 | |
| *** ozialien has quit IRC | 16:01 | |
| bswartz | li_zhang: the generic NFS driver is indeed generic and will work, but functionality-wise you'll get a way better experience with a vendor-specific NFS driver | 16:01 |
| *** avishay has joined #openstack-cinder | 16:01 | |
| *** sks has quit IRC | 16:02 | |
| *** juzuluag has quit IRC | 16:04 | |
| *** theanalyst has quit IRC | 16:04 | |
| *** crose has joined #openstack-cinder | 16:05 | |
| *** ozialien has joined #openstack-cinder | 16:05 | |
| tbarron | bswartz: +1 | 16:05 |
| *** esker has joined #openstack-cinder | 16:05 | |
| *** ozialien has quit IRC | 16:06 | |
| *** annashen has quit IRC | 16:06 | |
| tbarron | li_zhang: yes, we do on-filer optimizations and stuff like that | 16:06 |
| *** winston-d_zZZ is now known as winston-d | 16:07 | |
| *** theanalyst has joined #openstack-cinder | 16:07 | |
| *** ebalduf has joined #openstack-cinder | 16:08 | |
| *** ebalduf has quit IRC | 16:09 | |
| li_zhang | bswartz tbarron: got it, thanks | 16:11 |
| *** hemnafk is now known as hemna | 16:12 | |
| *** Adriano__ has joined #openstack-cinder | 16:13 | |
| *** rasoto has quit IRC | 16:13 | |
| *** julim has quit IRC | 16:14 | |
| *** Zhong-jun has quit IRC | 16:17 | |
| *** esker has quit IRC | 16:17 | |
| yuriy_n17 | #join #openstack-meeting | 16:18 |
| *** yk has quit IRC | 16:19 | |
| *** ozialien has joined #openstack-cinder | 16:19 | |
| *** chip_ has joined #openstack-cinder | 16:20 | |
| *** ebalduf has joined #openstack-cinder | 16:20 | |
| *** haypo has quit IRC | 16:21 | |
| tbarron | scottda, DuncanT, jgriffith: see https://review.openstack.org/#/c/117436/ (add reset-state function for backups) | 16:22 |
| tbarron | ^^^^ in light of the mid-cycle meetup discussion about reset-state | 16:23 |
| *** ozialien has quit IRC | 16:23 | |
| *** Longgeek has joined #openstack-cinder | 16:25 | |
| *** lpetrut has joined #openstack-cinder | 16:25 | |
| *** ebalduf has quit IRC | 16:27 | |
| *** asrangne__ has quit IRC | 16:28 | |
| *** setmason has quit IRC | 16:30 | |
| *** Longgeek has quit IRC | 16:30 | |
| *** setmason has joined #openstack-cinder | 16:31 | |
| *** ebalduf has joined #openstack-cinder | 16:31 | |
| *** ebalduf has left #openstack-cinder | 16:32 | |
| *** ebalduf has joined #openstack-cinder | 16:33 | |
| *** esker has joined #openstack-cinder | 16:33 | |
| *** Mandell has joined #openstack-cinder | 16:34 | |
| *** harlowja_at_home has joined #openstack-cinder | 16:39 | |
| *** ronis has joined #openstack-cinder | 16:39 | |
| *** dannywilson has quit IRC | 16:40 | |
| *** Tross has quit IRC | 16:41 | |
| *** Tross has joined #openstack-cinder | 16:41 | |
| *** dannywilson has joined #openstack-cinder | 16:41 | |
| *** alecv has quit IRC | 16:42 | |
| *** harlowja_at_home has quit IRC | 16:45 | |
| *** dflorea has joined #openstack-cinder | 16:45 | |
| *** leeantho has joined #openstack-cinder | 16:45 | |
| mriedem | DuncanT: please flip your vote (or at least remove) when you get a chance https://review.openstack.org/#/c/170913/ | 16:46 |
| mriedem | thingee: ^ | 16:46 |
| e0ne | mriedem: we should fix it before making voting, shouldn't we? | 16:46 |
| mriedem | e0ne: we already covered that in the meeting | 16:47 |
| thingee | mriedem: I will and link to the meeting discussion when we finish | 16:47 |
| mriedem | that change is depednent on the tempest patch to skip the failing job for ceph | 16:47 |
| mriedem | thingee: thanks | 16:47 |
| openstackgerrit | Rajini Ram proposed openstack/cinder: Fixes the retries on Network Connection Error https://review.openstack.org/171736 | 16:48 |
| DuncanT | mriedem: Done | 16:49 |
| mriedem | DuncanT: thanks | 16:50 |
| patrickeast | mriedem: so the plan is to make ceph gate too? or just vote on check? | 16:50 |
| mriedem | gate | 16:50 |
| mriedem | otherewise you can wedge the gate with other changes from other projects running that test | 16:50 |
| mriedem | e.g. nova/glance | 16:50 |
| mriedem | jogo could explain that better than i could | 16:50 |
| patrickeast | mriedem: ah ok | 16:50 |
| *** melwitt has joined #openstack-cinder | 16:51 | |
| mriedem | i think basically your cinder patch passes check, when it gets to the gate something merged in glance that now makes your patch fail, but since we don't gate on it, we merge your change that also breaks cinder | 16:51 |
| mriedem | something like that | 16:51 |
| mriedem | and thus nova | 16:51 |
| mriedem | so all 3 projects would be busted if we don't gate | 16:51 |
| patrickeast | yea that makes sense | 16:51 |
| mriedem | and then said rage gauge in ML | 16:52 |
| patrickeast | i guess its kind of a side effect of having these on the jenkins user | 16:52 |
| *** jdurgin1 has quit IRC | 16:52 | |
| DuncanT | patrickeast: More a case of other projects (glance, nova) gating on them | 16:53 |
| mriedem | yes, it's a cross-project thing | 16:53 |
| *** julim has joined #openstack-cinder | 16:54 | |
| patrickeast | ah yea, i was thinking the gate check was shared across them, but i see the issues with different sets of gating jobs | 16:54 |
| patrickeast | seems like we want to gate on these anyway | 16:55 |
| patrickeast | no point letting changes in that break widely used backends | 16:55 |
| thingee | mriedem: done! | 16:56 |
| thingee | alright bbl | 16:56 |
| thingee | DuncanT: thanks for your help | 16:56 |
| *** thingee has quit IRC | 16:57 | |
| DuncanT | thingee: NP | 16:57 |
| *** avishay has quit IRC | 16:58 | |
| *** li_zhang has quit IRC | 16:59 | |
| *** spradeepv has joined #openstack-cinder | 17:00 | |
| *** dflorea has quit IRC | 17:00 | |
| rhe00 | asselin_: doing a grep in the drivers directory, the hp_3par driver seems to be the only driver that sets the encrypted property, so it appears all other CIs are passing the encrypted volume test without testing encrypted volumes. | 17:01 |
| *** dflorea has joined #openstack-cinder | 17:01 | |
| *** dflorea_ has joined #openstack-cinder | 17:03 | |
| *** dflorea has quit IRC | 17:03 | |
| *** vilobhmm1 has joined #openstack-cinder | 17:04 | |
| *** _cjones_ has quit IRC | 17:06 | |
| *** _cjones_ has joined #openstack-cinder | 17:07 | |
| *** dannywilson has quit IRC | 17:07 | |
| *** drjones has joined #openstack-cinder | 17:09 | |
| *** annashen has joined #openstack-cinder | 17:09 | |
| *** _cjones_ has quit IRC | 17:10 | |
| *** annegentle has quit IRC | 17:10 | |
| openstackgerrit | Kamil Rykowski proposed openstack/cinder: Update openstack-common reference in openstack/common/README https://review.openstack.org/171705 | 17:15 |
| openstackgerrit | Kamil Rykowski proposed openstack/cinder: Update openstack-common reference in openstack/common/README https://review.openstack.org/171705 | 17:15 |
| DuncanT | rhe00: How can tempest test if a volume is encrypted? | 17:16 |
| scottda | tbarron: thx. I responded in the review. | 17:17 |
| rhe00 | DuncanT: I don't know the details of the test but from what I understand the test will only go through the encrypted volume test path if the 'encrypted' property in the connection_info returned from initialize_connection is set to True | 17:19 |
| *** leeantho has quit IRC | 17:19 | |
| rhe00 | DuncanT: https://bugs.launchpad.net/tempest/+bug/1440227 | 17:19 |
| openstack | Launchpad bug 1440227 in tempest "encrypted volume tests don't check if a volume is actually encrypted" [Undecided,New] | 17:19 |
| *** ebalduf has quit IRC | 17:20 | |
| *** ebalduf has joined #openstack-cinder | 17:20 | |
| *** lcurtis has joined #openstack-cinder | 17:20 | |
| *** rlucio has joined #openstack-cinder | 17:20 | |
| rhe00 | DuncanT: since almost no drivers (except hp_3par) sets that property then the test won't go down the enctyption path. | 17:21 |
| *** Mandell has quit IRC | 17:21 | |
| *** harlowja_away is now known as harlowja | 17:21 | |
| *** leeantho has joined #openstack-cinder | 17:21 | |
| asselin_ | rhe00, wow, didn't realize it's that bad | 17:21 |
| *** ozialien has joined #openstack-cinder | 17:22 | |
| *** e0ne has quit IRC | 17:22 | |
| DuncanT | rhe00: Most drivers don't support backend encryption AFAIK... perhaps we should have a test to make sure they don't get given volumes with backend encryption selected? | 17:22 |
| DuncanT | rhe00: host-based encryption still works though | 17:23 |
| rhe00 | DuncanT: this is host based | 17:23 |
| asselin_ | DuncanT, it's not back end encyption | 17:23 |
| rhe00 | not sure why it is up to the driver to set the encrypted property | 17:23 |
| DuncanT | Hmmm, ok, I've entirely mis-understood the issue then. Give me a moment to re-read then | 17:23 |
| rhe00 | since it has nothing to do with the data content | 17:23 |
| DuncanT | It shouldn't be up tot eh driver IMO, it should be entirely transparent | 17:23 |
| patrickeast | rhe00: yea that seems weird that all drivers would need to set it if its host encryption | 17:23 |
| DuncanT | *up to the | 17:24 |
| asselin_ | smcginnis, do you know what was the fix to "AttributeError: 'NoneType' object has no attribute 'url'" | 17:25 |
| *** deepakcs has quit IRC | 17:26 | |
| vilobhmm1 | asselin_ : can you point me to the code | 17:26 |
| vilobhmm1 | and the error i can help you if smcginnis is not around | 17:26 |
| vilobhmm1 | error -> logs | 17:26 |
| asselin_ | vilobhmm1, hmm...it happened a few days ago. one sec | 17:27 |
| *** Mandell has joined #openstack-cinder | 17:27 | |
| *** eharney has quit IRC | 17:27 | |
| jordanP | asselin_, it was a pip issue | 17:27 |
| *** dflorea_ has quit IRC | 17:27 | |
| jordanP | a new release of pip went out, it broke us, a change was reverted in pip | 17:27 |
| asselin_ | jordanP, that's what I thought...which version is good now? | 17:28 |
| *** dflorea has joined #openstack-cinder | 17:28 | |
| jordanP | the very latest | 17:28 |
| jordanP | asselin_, 6.1.1 | 17:28 |
| *** Guest39926 is now known as mgagne | 17:29 | |
| *** mgagne has joined #openstack-cinder | 17:29 | |
| asselin_ | jordanP, ok thanks | 17:29 |
| jordanP | np | 17:29 |
| DuncanT | asselin_: https://review.openstack.org/#/c/171060/ was the fix I think | 17:31 |
| asselin_ | rhe00, DuncanT so our lefthand driver didn't have the issue beccause the encryption flag is set here: http://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/drivers/san/hp/hp_lefthand_rest_proxy.py#n363 | 17:32 |
| *** dflorea_ has joined #openstack-cinder | 17:32 | |
| asselin_ | DuncanT, looking | 17:32 |
| *** dflorea has quit IRC | 17:33 | |
| DuncanT | asselin_: It's good that lefthand does that, given the current design, but I think the design is wrong | 17:33 |
| asselin_ | ok...if other drivers do that, then they should work | 17:33 |
| *** yuriy_n17 has quit IRC | 17:36 | |
| *** drjones has quit IRC | 17:36 | |
| DuncanT | asselin_: Yes, but they shouldn't have to. There's nothing useful a driver can do with this information, so the manager should take care of it | 17:36 |
| asselin_ | DuncanT, I agree the driver shouldn't have to take care of it | 17:36 |
| DuncanT | asselin_: The only reasona driver might want to know that host encryption is in use is if it wants to do less compression / de-dupe or something | 17:37 |
| rhe00 | asselin_: do you know where this property is looked at and acted upon? | 17:38 |
| asselin_ | rhe00, yeah, it's linked in the bug...but the line number changed. one sec... | 17:38 |
| asselin_ | rhe00, https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L1048 | 17:39 |
| *** _cjones_ has joined #openstack-cinder | 17:39 | |
| *** chip_ has quit IRC | 17:44 | |
| *** jordanP has quit IRC | 17:44 | |
| alexpilotti | DuncanT: I noticed that there are 2x +2 on the SMBFS Windows revert patch but not a +a yet: https://review.openstack.org/#/c/171484/ | 17:47 |
| kmartin | jungleboyj, one more for Kilo-RC1, this is a fix for FC drivers on RHEL7 -> https://review.openstack.org/#/c/170157/ | 17:47 |
| alexpilotti | DuncanT: I’d like to run a recheck for the iSCSI and Limux SMBFS, but at this point if gets a +a it will happen automatically | 17:48 |
| *** setmason has quit IRC | 17:48 | |
| asselin_ | alexpilotti, I'd do the recheck...looks better to see it pass before +A | 17:49 |
| *** setmason has joined #openstack-cinder | 17:49 | |
| alexpilotti | asselin_: ok, just to be clear, those two CIs are unrelated to this patch | 17:49 |
| *** dflorea_ has quit IRC | 17:49 | |
| alexpilotti | asselin_: I’d just like to have them all green :-) | 17:49 |
| alexpilotti | ociuhandu_: ^ | 17:50 |
| asselin_ | alexpilotti, ok I see...this patch is just for the one that's being reverted. | 17:50 |
| *** dflorea has joined #openstack-cinder | 17:50 | |
| alexpilotti | asselin_: yes, correct and that one is already positive | 17:51 |
| DuncanT | alexpilotti: I'd be quite happy if somebody said they'd looked at the problem with the others and understood it | 17:51 |
| *** timcl has quit IRC | 17:52 | |
| ociuhandu_ | DuncanT: hello, I have looked at both of them | 17:52 |
| *** eharney has joined #openstack-cinder | 17:52 | |
| DuncanT | ociuhandu_: +A added... if jenkins doesn't blow up then you should be back in shortly | 17:53 |
| alexpilotti | DuncanT: tx! | 17:53 |
| ociuhandu_ | DuncanT: for the iscsi one I have found an issue with the windows image, we are working now on isolating the exact problem. In random cases it will not set the admin password. As a quick workaround I have put a retry at the moment in place but we am also in contact with Microsoft to nail the exact problem with it | 17:54 |
| *** dflorea has quit IRC | 17:54 | |
| *** dflorea has joined #openstack-cinder | 17:54 | |
| ociuhandu_ | we are keeping the runs under close observations to make sure that we locate these issues and work to fix them asap | 17:55 |
| ociuhandu_ | DuncanT: thank you | 17:55 |
| *** dflorea has quit IRC | 17:56 | |
| smcginnis | asselin_: Sorry, was out for lunch. Did you get it sorted? | 17:56 |
| *** dflorea has joined #openstack-cinder | 17:56 | |
| asselin_ | smcginnis, yes | 17:57 |
| smcginnis | asselin_: Cool. | 17:57 |
| smcginnis | asselin_: Had the review link somewhere, but it would take a bit to track it down. | 17:57 |
| *** bswartz has quit IRC | 17:57 | |
| *** timcl has joined #openstack-cinder | 17:59 | |
| *** timcl has quit IRC | 17:59 | |
| *** sks has joined #openstack-cinder | 17:59 | |
| *** bswartz has joined #openstack-cinder | 18:00 | |
| *** sks has quit IRC | 18:00 | |
| *** delatte has joined #openstack-cinder | 18:00 | |
| *** annegentle has joined #openstack-cinder | 18:01 | |
| *** dflorea has quit IRC | 18:01 | |
| *** delattec has quit IRC | 18:03 | |
| *** timcl has joined #openstack-cinder | 18:03 | |
| *** timcl has left #openstack-cinder | 18:03 | |
| *** rushil_ has joined #openstack-cinder | 18:04 | |
| *** rushil has quit IRC | 18:07 | |
| *** dflorea has joined #openstack-cinder | 18:09 | |
| *** delattec has joined #openstack-cinder | 18:12 | |
| *** timcl has joined #openstack-cinder | 18:13 | |
| *** delatte has quit IRC | 18:15 | |
| *** ebalduf has quit IRC | 18:18 | |
| tbarron | scottda, DuncanT: ty for remarks in the review of https://review.openstack.org/#/c/117436/ Was in a meeting. | 18:18 |
| *** dannywilson has joined #openstack-cinder | 18:19 | |
| *** ankit_ag has quit IRC | 18:19 | |
| *** aix has quit IRC | 18:24 | |
| *** delatte has joined #openstack-cinder | 18:26 | |
| *** annegentle has quit IRC | 18:28 | |
| *** boris-42 has quit IRC | 18:28 | |
| Swanson | Soooo... what was the result of the encryption thing? I don't support it so I'm not setting anything. Should I be failing when I create the volume? | 18:28 |
| *** annegentle has joined #openstack-cinder | 18:30 | |
| *** delattec has quit IRC | 18:30 | |
| *** diemt has quit IRC | 18:31 | |
| *** diemt has joined #openstack-cinder | 18:31 | |
| *** lpabon has quit IRC | 18:34 | |
| *** lcurtis has quit IRC | 18:34 | |
| *** delattec has joined #openstack-cinder | 18:35 | |
| *** Mandell has quit IRC | 18:35 | |
| DuncanT | tbarron: scottda: replied inline on the review | 18:37 |
| hemna | so I'm looking into a nova fix | 18:38 |
| *** delatte has quit IRC | 18:38 | |
| hemna | that will fetch the volume's uuid from the device | 18:38 |
| hemna | and use that for a name instead of the iqn | 18:38 |
| hemna | about to test it | 18:38 |
| hemna | I'm not sure why the bug was marked as invalid | 18:38 |
| hemna | https://bugs.launchpad.net/nova/+bug/1432490 | 18:39 |
| openstack | Launchpad bug 1432490 in tempest "TestEncryptedCinderVolumes cryptsetup name is too long" [Undecided,Invalid] | 18:39 |
| DuncanT | hemna: feel free to re-open, I guess | 18:40 |
| marcusvrn | asselin_: ping | 18:40 |
| asselin_ | marcusvrn, hi | 18:40 |
| smcginnis | hemna: Just don't read my comment, that's confidential. :p | 18:40 |
| hemna | ok I'll skip that one. | 18:41 |
| hemna | :P | 18:41 |
| *** e0ne has joined #openstack-cinder | 18:41 | |
| *** cardinalnation has joined #openstack-cinder | 18:41 | |
| marcusvrn | asselin_: in this bug: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1413540 you said to edit local.conf to run qemu on top of kvm as workaround | 18:42 |
| openstack | Launchpad bug 1413540 in linux (Ubuntu) "Trusty soft lockup issues with nested KVM" [High,Confirmed] - Assigned to Chris J Arges (arges) | 18:42 |
| *** bswartz has quit IRC | 18:42 | |
| marcusvrn | asselin_: but it's not working in our job...the libvirt is overrided, I guess | 18:43 |
| tbarron | DuncanT: thanks! | 18:43 |
| *** bswartz has joined #openstack-cinder | 18:44 | |
| marcusvrn | asselin_: here is the job config: http://paste.openstack.org/show/200630/ and the nova.conf: http://paste.openstack.org/show/200631/ | 18:45 |
| asselin_ | [[post-config|$NOVA_CONF]] --> [[post-config|\$NOVA_CONF]] | 18:45 |
| asselin_ | missing slash | 18:45 |
| marcusvrn | asselin_: ¬¬"... | 18:46 |
| marcusvrn | asselin_: sry! hehe thanks | 18:46 |
| asselin_ | marcusvrn, np ;) | 18:46 |
| *** cardinalnation has quit IRC | 18:48 | |
| *** ho has joined #openstack-cinder | 18:50 | |
| *** ebalduf has joined #openstack-cinder | 18:50 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add volume status to error message in backup create flow https://review.openstack.org/171566 | 18:51 |
| *** ebalduf has quit IRC | 18:51 | |
| *** annegentle has quit IRC | 18:53 | |
| *** thangp has quit IRC | 18:56 | |
| *** zongliang has quit IRC | 18:56 | |
| *** dflorea has quit IRC | 19:00 | |
| *** ebalduf has joined #openstack-cinder | 19:01 | |
| *** dflorea_ has joined #openstack-cinder | 19:03 | |
| e0ne | hm... 2 3rd party ci's (vm ware & mellanox) failed with error 'Can't find package MySQL-python in | 19:05 |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Add cache to Managers' find/findall methods https://review.openstack.org/162139 | 19:07 |
| *** shyama has quit IRC | 19:07 | |
| *** ociuhandu_ has quit IRC | 19:10 | |
| *** Mandell has joined #openstack-cinder | 19:13 | |
| *** sgotliv has quit IRC | 19:13 | |
| *** annegentle has joined #openstack-cinder | 19:15 | |
| *** bswartz has quit IRC | 19:26 | |
| *** spradeepv has quit IRC | 19:27 | |
| *** BharatK has quit IRC | 19:37 | |
| *** mtanino has quit IRC | 19:40 | |
| *** e0ne has quit IRC | 19:43 | |
| *** bswartz has joined #openstack-cinder | 19:44 | |
| *** rushiagr is now known as rushiagr_away | 19:45 | |
| *** abhiram_moturi has joined #openstack-cinder | 19:46 | |
| *** e0ne has joined #openstack-cinder | 19:47 | |
| *** _cjones_ has quit IRC | 19:50 | |
| *** dflorea_ has quit IRC | 19:51 | |
| *** dflorea has joined #openstack-cinder | 19:52 | |
| *** jungleboyj has quit IRC | 19:54 | |
| *** Rockyg has joined #openstack-cinder | 19:56 | |
| *** dflorea has quit IRC | 19:57 | |
| *** ozialien has quit IRC | 19:59 | |
| *** kvidvans_ has quit IRC | 20:04 | |
| *** e0ne has quit IRC | 20:04 | |
| *** _cjones_ has joined #openstack-cinder | 20:04 | |
| *** Swanson has quit IRC | 20:04 | |
| *** dflorea has joined #openstack-cinder | 20:04 | |
| *** Swanson has joined #openstack-cinder | 20:05 | |
| *** ndipanov has quit IRC | 20:09 | |
| *** marcusvrn has quit IRC | 20:13 | |
| *** bkopilov has quit IRC | 20:14 | |
| *** annegentle has quit IRC | 20:17 | |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Add cache to Managers' find/findall methods https://review.openstack.org/162139 | 20:18 |
| *** bswartz has quit IRC | 20:20 | |
| *** annegentle has joined #openstack-cinder | 20:23 | |
| *** bkopilov has joined #openstack-cinder | 20:24 | |
| *** bswartz has joined #openstack-cinder | 20:24 | |
| *** bkopilov has quit IRC | 20:32 | |
| *** mtanino has joined #openstack-cinder | 20:32 | |
| *** timcl has quit IRC | 20:36 | |
| *** annegentle has quit IRC | 20:38 | |
| *** annegentle has joined #openstack-cinder | 20:39 | |
| *** annashen has quit IRC | 20:40 | |
| *** annashen has joined #openstack-cinder | 20:42 | |
| *** Maike has joined #openstack-cinder | 20:44 | |
| *** annashen has quit IRC | 20:44 | |
| *** annegentle has quit IRC | 20:45 | |
| *** annashen has joined #openstack-cinder | 20:46 | |
| *** Maike_ has joined #openstack-cinder | 20:50 | |
| *** annashen has quit IRC | 20:51 | |
| *** Maike has quit IRC | 20:54 | |
| *** jungleboyj has joined #openstack-cinder | 20:54 | |
| *** ebalduf has quit IRC | 20:55 | |
| *** bswartz has quit IRC | 21:00 | |
| *** annegentle has joined #openstack-cinder | 21:00 | |
| *** ebalduf has joined #openstack-cinder | 21:09 | |
| *** annegentle has quit IRC | 21:09 | |
| *** annegentle has joined #openstack-cinder | 21:10 | |
| *** akerr has quit IRC | 21:10 | |
| *** pradipta has quit IRC | 21:12 | |
| *** Rockyg has quit IRC | 21:14 | |
| *** ebalduf has quit IRC | 21:25 | |
| *** Adriano__ has quit IRC | 21:26 | |
| *** Adriano_ has quit IRC | 21:26 | |
| *** setmason has quit IRC | 21:26 | |
| *** rushil_ has quit IRC | 21:28 | |
| *** rushil has joined #openstack-cinder | 21:29 | |
| *** ronis has quit IRC | 21:30 | |
| *** dustins has quit IRC | 21:30 | |
| *** Yogi1 has quit IRC | 21:31 | |
| *** esker has quit IRC | 21:33 | |
| *** ebalduf has joined #openstack-cinder | 21:34 | |
| rhe00 | asselin_, DuncanT: I have a fix to set the encrypted property in initialize_connection in volume manager so that it doesn't have to be set by every driver. I will open a new bug and work check in the fix against that bug. Sounds good? | 21:36 |
| *** rushil has quit IRC | 21:36 | |
| *** mriedem is now known as mriedem_away | 21:42 | |
| hemna | rhe00, which bug? | 21:43 |
| *** dflorea has quit IRC | 21:44 | |
| *** dflorea has joined #openstack-cinder | 21:44 | |
| *** lpetrut has quit IRC | 21:47 | |
| *** asselin has joined #openstack-cinder | 21:47 | |
| *** mriedem_away has quit IRC | 21:48 | |
| *** dflorea has quit IRC | 21:49 | |
| rhe00 | hemna: this is related to https://bugs.launchpad.net/tempest/+bug/1440227 | 21:49 |
| openstack | Launchpad bug 1440227 in tempest "encrypted volume tests don't check if a volume is actually encrypted" [Undecided,New] | 21:49 |
| smcginnis | rhe00: So this will affect (fix) all drivers? | 21:50 |
| rhe00 | asselin fixed the hp_3par driver but DuncanT commented that the encrypted property should be set by the manager and keep it transparent for the volume driver | 21:50 |
| rhe00 | smcginnis: no, the point of the fix I am making is to avoid every driver having to set the encrypted property on their own | 21:51 |
| *** Mandell has quit IRC | 21:51 | |
| smcginnis | rhe00: Hm, guess I'll have to take a look once you have it up. | 21:51 |
| *** asselin has quit IRC | 21:52 | |
| rhe00 | it really has nothing to do with the driver since this is host based encryption. | 21:52 |
| *** reed_ has joined #openstack-cinder | 21:52 | |
| rhe00 | got to run now, will put it up for review tomorrow. | 21:52 |
| hemna | rhe00, ok so this is a related issue | 21:52 |
| hemna | so I think I'm stuck on the issue I was looking at | 21:52 |
| smcginnis | rhe00: So it fixes all drivers in that it doesn't place an unnecessary requirement on them. I'm sure it will make more sense when I see it. | 21:52 |
| hemna | the volume name for cryptsetup | 21:52 |
| hemna | https://bugs.launchpad.net/tempest/+bug/1432490/ | 21:53 |
| openstack | Launchpad bug 1432490 in tempest "TestEncryptedCinderVolumes cryptsetup name is too long" [Undecided,Invalid] | 21:53 |
| hemna | I was looking at a possible fix for that | 21:53 |
| hemna | but new block devices don't have a UUID attribute set on them | 21:53 |
| *** crose has quit IRC | 21:54 | |
| asselin_ | rhe00, ok cool | 21:58 |
| *** chlong has joined #openstack-cinder | 22:01 | |
| *** sandywalsh has quit IRC | 22:04 | |
| *** Mandell has joined #openstack-cinder | 22:04 | |
| *** sandywalsh has joined #openstack-cinder | 22:05 | |
| *** annegentle has quit IRC | 22:09 | |
| *** jamielennox is now known as jamielennox|away | 22:11 | |
| hemna | ok....so | 22:16 |
| hemna | it looks like only iSCSI volumes can work in nova for encryption now. | 22:17 |
| hemna | *sigh* | 22:17 |
| hemna | the rootwrap.d/compute.filters intentionally baked in iscsi-iqn regex for creating the /dev/mapper/* entries | 22:17 |
| hemna | so anything other than iSCSI will always fail. | 22:17 |
| *** Yogi1 has joined #openstack-cinder | 22:19 | |
| *** annegentle has joined #openstack-cinder | 22:20 | |
| *** esker has joined #openstack-cinder | 22:23 | |
| *** Yogi1 has quit IRC | 22:26 | |
| *** bkopilov has joined #openstack-cinder | 22:30 | |
| *** annegentle has quit IRC | 22:36 | |
| *** ebalduf has quit IRC | 22:38 | |
| *** alexpilotti has quit IRC | 22:39 | |
| *** annegentle has joined #openstack-cinder | 22:47 | |
| *** bswartz has joined #openstack-cinder | 22:51 | |
| *** IanGovett has quit IRC | 22:52 | |
| *** IanGovett has joined #openstack-cinder | 22:52 | |
| *** git-harry has quit IRC | 22:56 | |
| *** IanGovett has quit IRC | 22:57 | |
| *** annegentle has quit IRC | 23:03 | |
| *** annegentle has joined #openstack-cinder | 23:05 | |
| *** chlong has quit IRC | 23:05 | |
| *** Mandell has quit IRC | 23:05 | |
| *** bkopilov has quit IRC | 23:06 | |
| *** bkopilov has joined #openstack-cinder | 23:06 | |
| hemna | ok | 23:07 |
| hemna | wow | 23:07 |
| hemna | encrypted volumes is so broken | 23:07 |
| hemna | the nova code is just wrong | 23:07 |
| hemna | found 3 or 4 problems just fixing the iqn length issue | 23:07 |
| patrickeast | does that mean no one is actually using it? or just didn’t notice it was broken? | 23:07 |
| hemna | I can't see how anyone is using it and working | 23:08 |
| hemna | impossible | 23:08 |
| hemna | 1) it doesn't work for FC devices at all | 23:08 |
| hemna | 2) the rootwrap.d/compute.filter is broken | 23:08 |
| hemna | 3) the /dev/mapper/* device it creates via a ln --symbolic is backwards and could never work | 23:09 |
| hemna | 4) they "fixed" the error in #3 by using --force option on ln, which hides the failure | 23:09 |
| asselin_ | patrickeast, I think the issue is that it's hard to tell it doesn't work. The UI has a checkbox showing "encypted"...but it's not | 23:10 |
| hemna | but I do have fixes for it here....now I just have to cycle through unit test hell and nova reviews. | 23:11 |
| *** jamielennox|away is now known as jamielennox | 23:12 | |
| *** Mandell has joined #openstack-cinder | 23:13 | |
| *** git-harry has joined #openstack-cinder | 23:14 | |
| *** ebalduf has joined #openstack-cinder | 23:15 | |
| *** annegentle has quit IRC | 23:16 | |
| *** dims has quit IRC | 23:16 | |
| *** ganso_ has quit IRC | 23:17 | |
| *** ebalduf has quit IRC | 23:19 | |
| *** reed_ has quit IRC | 23:23 | |
| *** emagana has quit IRC | 23:25 | |
| hemna | rhe00, https://gist.github.com/WaltHP/2c24ea17559d658bf392 | 23:27 |
| hemna | rhe00, that's my current work on fixing the iqn length problem. | 23:27 |
| hemna | I've been able to attach a lefthand volume as encrypted with that patch in place. | 23:27 |
| hemna | I'm going to work on cleaning it up and updating unit tests, etc. | 23:27 |
| *** hemna is now known as hemnafk | 23:28 | |
| *** dims has joined #openstack-cinder | 23:34 | |
| *** Mandell has quit IRC | 23:42 | |
| *** patrickeast has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!