| *** haomaiwang has joined #openstack-cinder | 00:03 | |
| *** haomaiw__ has quit IRC | 00:03 | |
| *** haomaiwang has quit IRC | 00:13 | |
| *** dguryanov|2 has quit IRC | 00:18 | |
| *** smoriya has joined #openstack-cinder | 00:19 | |
| *** zhenguo has joined #openstack-cinder | 00:19 | |
| *** Longgeek has quit IRC | 00:29 | |
| *** merooney has quit IRC | 00:41 | |
| *** merooney has joined #openstack-cinder | 00:47 | |
| *** laughterwym has joined #openstack-cinder | 00:48 | |
| openstackgerrit | Yuiko Takada proposed openstack/cinder: Pass environment variables of proxy to tox https://review.openstack.org/191603 | 01:15 |
|---|---|---|
| *** julim has joined #openstack-cinder | 01:25 | |
| *** heyun has joined #openstack-cinder | 01:28 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:31 | |
| *** lixiaoy1 has left #openstack-cinder | 01:32 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:32 | |
| *** lixiaoy11 has joined #openstack-cinder | 01:33 | |
| *** merooney has quit IRC | 01:34 | |
| *** pboldin has quit IRC | 01:34 | |
| *** lixiaoy1 has quit IRC | 01:36 | |
| *** merooney has joined #openstack-cinder | 01:37 | |
| *** lixiaoy11 has left #openstack-cinder | 01:40 | |
| *** lixiaoy11 has joined #openstack-cinder | 01:40 | |
| *** Longgeek has joined #openstack-cinder | 01:47 | |
| *** Lee1092 has joined #openstack-cinder | 01:57 | |
| *** IlyaG has joined #openstack-cinder | 02:02 | |
| *** merooney has quit IRC | 02:04 | |
| *** changbl has joined #openstack-cinder | 02:07 | |
| *** merooney has joined #openstack-cinder | 02:10 | |
| *** Longgeek has quit IRC | 02:20 | |
| *** dimsum__ has quit IRC | 02:22 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Update SolidFire to use target driver model https://review.openstack.org/190677 | 02:33 |
| *** dimsum__ has joined #openstack-cinder | 02:36 | |
| *** bkopilov has quit IRC | 02:40 | |
| *** IlyaG has quit IRC | 02:47 | |
| *** vilobhmm has joined #openstack-cinder | 02:55 | |
| *** dimsum__ has quit IRC | 02:58 | |
| *** merooney has quit IRC | 02:59 | |
| *** yamada-h has joined #openstack-cinder | 03:03 | |
| *** julim has quit IRC | 03:06 | |
| *** yamada-h has quit IRC | 03:07 | |
| *** merooney has joined #openstack-cinder | 03:10 | |
| *** Longgeek has joined #openstack-cinder | 03:18 | |
| *** tobe has joined #openstack-cinder | 03:20 | |
| *** ronenkat has quit IRC | 03:25 | |
| *** daneyon has quit IRC | 03:29 | |
| *** coolsvap|afk is now known as coolsvap | 03:32 | |
| *** logan2 has quit IRC | 03:35 | |
| openstackgerrit | Edward Balduf proposed openstack/python-cinderclient: The is_public filter in VolumeTypeManager.list broke the find in VolumeTypeAccessManager, which caused all commands which should have taken a volume_type name to fail. I have choosen to fix this by effectively removing the filter in the client (and the - https://review.openstack.org/191621 | 03:41 |
| *** Longgeek has quit IRC | 03:51 | |
| *** yamada-h has joined #openstack-cinder | 03:52 | |
| *** yamada-h has quit IRC | 03:52 | |
| *** salv-orl_ has quit IRC | 03:55 | |
| *** yamada-h has joined #openstack-cinder | 03:55 | |
| *** yamada-h has quit IRC | 03:55 | |
| *** vilobhmm has quit IRC | 03:55 | |
| *** bkopilov has joined #openstack-cinder | 03:55 | |
| *** dimsum__ has joined #openstack-cinder | 03:58 | |
| *** dimsum__ has quit IRC | 04:03 | |
| *** yamada-h has joined #openstack-cinder | 04:06 | |
| *** bkopilov has quit IRC | 04:06 | |
| *** bkopilov has joined #openstack-cinder | 04:18 | |
| *** heyun has quit IRC | 04:23 | |
| *** heyun has joined #openstack-cinder | 04:23 | |
| *** sgotliv has quit IRC | 04:26 | |
| *** takedakn has joined #openstack-cinder | 04:35 | |
| *** takedakn has quit IRC | 04:39 | |
| *** yamada-h has quit IRC | 04:41 | |
| openstackgerrit | rakesh mishra proposed openstack/cinder: set-size-limit-for-volume https://review.openstack.org/185906 | 04:45 |
| *** yamada-h has joined #openstack-cinder | 04:45 | |
| *** yamada-h has quit IRC | 04:50 | |
| *** pradipta has joined #openstack-cinder | 05:00 | |
| *** Longgeek has joined #openstack-cinder | 05:09 | |
| *** merooney has quit IRC | 05:11 | |
| *** nihilifer has joined #openstack-cinder | 05:18 | |
| *** nkrinner has joined #openstack-cinder | 05:20 | |
| *** yamada-h has joined #openstack-cinder | 05:28 | |
| *** yamada-h has quit IRC | 05:28 | |
| *** ronenkat has joined #openstack-cinder | 05:31 | |
| *** edsh has quit IRC | 05:31 | |
| *** davechen_afk is now known as davechen | 05:32 | |
| *** deepakcs has joined #openstack-cinder | 05:34 | |
| *** Longgeek has quit IRC | 05:35 | |
| *** shausy has joined #openstack-cinder | 05:36 | |
| *** BharatK has joined #openstack-cinder | 05:37 | |
| *** edsh has joined #openstack-cinder | 05:40 | |
| *** BharatK has quit IRC | 05:42 | |
| *** unknown__ has joined #openstack-cinder | 05:44 | |
| unknown__ | where will be the tempest.conf?? | 05:44 |
| *** unknown__ has quit IRC | 05:45 | |
| *** belmoreira has joined #openstack-cinder | 05:47 | |
| *** BharatK has joined #openstack-cinder | 05:54 | |
| *** vilobhmm has joined #openstack-cinder | 06:12 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Volume driver for HP XP storage https://review.openstack.org/184774 | 06:13 |
| *** vilobhmm has quit IRC | 06:16 | |
| *** vilobhmm has joined #openstack-cinder | 06:16 | |
| *** Longgeek has joined #openstack-cinder | 06:17 | |
| *** lpetrut has joined #openstack-cinder | 06:21 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO Volume driver requests, multipath https://review.openstack.org/184596 | 06:22 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add migration progress indication https://review.openstack.org/189614 | 06:27 |
| *** takedakn has joined #openstack-cinder | 06:39 | |
| *** ian_brown has quit IRC | 06:43 | |
| *** tobe has quit IRC | 06:44 | |
| *** takedakn has quit IRC | 06:46 | |
| openstackgerrit | Jessy Lee proposed openstack/cinder: Add volume drivers for Infortrend Storage https://review.openstack.org/177113 | 06:49 |
| *** bkopilov has quit IRC | 06:56 | |
| *** kaisers has joined #openstack-cinder | 07:05 | |
| *** tobe has joined #openstack-cinder | 07:07 | |
| *** yamada-h has joined #openstack-cinder | 07:10 | |
| *** markus_z has joined #openstack-cinder | 07:19 | |
| *** dulek has joined #openstack-cinder | 07:21 | |
| openstackgerrit | Rajesh Tailor proposed openstack/cinder: Validate bool value using strutils.bool_from_string method https://review.openstack.org/187470 | 07:24 |
| *** chlong has quit IRC | 07:34 | |
| *** ndipanov has joined #openstack-cinder | 07:41 | |
| *** anshul has joined #openstack-cinder | 07:46 | |
| *** salv-orlando has joined #openstack-cinder | 07:48 | |
| *** yamada-h has quit IRC | 07:50 | |
| *** nikeshm has joined #openstack-cinder | 07:51 | |
| *** lpetrut has quit IRC | 07:54 | |
| *** yamada-h has joined #openstack-cinder | 07:56 | |
| *** yamada-h has quit IRC | 08:01 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add migration progress indication https://review.openstack.org/189614 | 08:06 |
| *** bkopilov has joined #openstack-cinder | 08:06 | |
| *** ronis has joined #openstack-cinder | 08:07 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add migration progress indication https://review.openstack.org/189614 | 08:08 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add migration progress indication https://review.openstack.org/189614 | 08:11 |
| *** vilobhmm has quit IRC | 08:11 | |
| *** shyama has joined #openstack-cinder | 08:12 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add support for force-delete backups https://review.openstack.org/166127 | 08:15 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Drop L suffix from long integers https://review.openstack.org/190152 | 08:21 |
| *** belmoreira has quit IRC | 08:22 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add migration progress indication https://review.openstack.org/189614 | 08:25 |
| *** Maike has joined #openstack-cinder | 08:26 | |
| *** c0m0 has joined #openstack-cinder | 08:28 | |
| *** haypo has joined #openstack-cinder | 08:30 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Fix Python 3 issues in utils https://review.openstack.org/191019 | 08:32 |
| *** lpetrut has joined #openstack-cinder | 08:33 | |
| openstackgerrit | Ph. Marek proposed openstack/cinder: Re-add DRBD driver. https://review.openstack.org/178573 | 08:37 |
| *** laughterwym has quit IRC | 08:39 | |
| *** takedakn has joined #openstack-cinder | 08:42 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 08:42 |
| *** takedakn has quit IRC | 08:43 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 08:49 |
| *** lennyb has quit IRC | 08:58 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Replace xrange() with six.moves.range() https://review.openstack.org/185418 | 08:59 |
| openstackgerrit | chenzongliang proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 09:00 |
| *** lennyb has joined #openstack-cinder | 09:03 | |
| *** yamada-h has joined #openstack-cinder | 09:09 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 09:15 |
| openstackgerrit | chenzongliang proposed openstack/cinder: Huawei Storage Multiple Pools Support https://review.openstack.org/188251 | 09:15 |
| *** m3m0 has joined #openstack-cinder | 09:22 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Drop L suffix from long integers https://review.openstack.org/190152 | 09:24 |
| *** e0ne has joined #openstack-cinder | 09:30 | |
| *** rushiagr_away is now known as rushiagr | 09:34 | |
| *** haomaiwang has joined #openstack-cinder | 09:38 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Get StringIO from six for Python 3 compatibility https://review.openstack.org/189720 | 09:39 |
| *** afazekas has joined #openstack-cinder | 09:39 | |
| *** e0ne has quit IRC | 09:40 | |
| *** jordanP has joined #openstack-cinder | 09:43 | |
| *** lixiaoy11 has quit IRC | 09:46 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Add CHAP support for Huawei Storage https://review.openstack.org/188360 | 09:47 |
| *** e0ne has joined #openstack-cinder | 09:47 | |
| *** salv-orl_ has joined #openstack-cinder | 09:51 | |
| *** salv-orlando has quit IRC | 09:52 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Support SmartX for Huawei volume driver https://review.openstack.org/188732 | 09:52 |
| *** zhenguo has quit IRC | 10:02 | |
| *** yamada-h has quit IRC | 10:05 | |
| *** ianbrown has joined #openstack-cinder | 10:07 | |
| openstackgerrit | Igor Pugovkin proposed openstack/cinder: Remove WritableLogger wrapper https://review.openstack.org/191725 | 10:07 |
| *** dimsum__ has joined #openstack-cinder | 10:09 | |
| *** aix has joined #openstack-cinder | 10:10 | |
| *** rakesh_mishra has joined #openstack-cinder | 10:12 | |
| *** rakesh_mishra_ has joined #openstack-cinder | 10:19 | |
| *** rakesh_mishra has quit IRC | 10:23 | |
| openstackgerrit | Igor Pugovkin proposed openstack/cinder: Remove WritableLogger wrapper https://review.openstack.org/191725 | 10:23 |
| *** rakesh_mishra_ has quit IRC | 10:26 | |
| *** rakesh_mishra has joined #openstack-cinder | 10:26 | |
| *** e0ne is now known as e0ne_ | 10:32 | |
| *** ianbrown has quit IRC | 10:33 | |
| *** ankit_ag has joined #openstack-cinder | 10:34 | |
| *** heyun has quit IRC | 10:35 | |
| *** e0ne_ has quit IRC | 10:39 | |
| *** lan has quit IRC | 10:41 | |
| *** e0ne has joined #openstack-cinder | 10:41 | |
| *** lan has joined #openstack-cinder | 10:42 | |
| *** jaypipes has joined #openstack-cinder | 11:00 | |
| *** timcl has joined #openstack-cinder | 11:01 | |
| *** rakesh_mishra has quit IRC | 11:02 | |
| *** rakesh_mishra has joined #openstack-cinder | 11:03 | |
| *** yamada-h has joined #openstack-cinder | 11:05 | |
| *** Longgeek_ has joined #openstack-cinder | 11:06 | |
| *** pboldin has joined #openstack-cinder | 11:08 | |
| *** edsh has quit IRC | 11:08 | |
| *** Longgeek has quit IRC | 11:09 | |
| *** yamada-h has quit IRC | 11:10 | |
| *** rakesh_mishra has quit IRC | 11:10 | |
| *** rakesh_mishra has joined #openstack-cinder | 11:15 | |
| *** rushiagr is now known as rushiagr_away | 11:15 | |
| *** edsh has joined #openstack-cinder | 11:18 | |
| *** edsh has quit IRC | 11:20 | |
| *** tobe has quit IRC | 11:25 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add support for force-delete backups https://review.openstack.org/166127 | 11:26 |
| *** rushiagr_away is now known as rushiagr | 11:27 | |
| *** aix has quit IRC | 11:28 | |
| *** Longgeek_ has quit IRC | 11:34 | |
| *** Longgeek has joined #openstack-cinder | 11:35 | |
| *** e0ne is now known as e0ne_ | 11:36 | |
| *** BharatK has quit IRC | 11:37 | |
| *** marcusvrn has joined #openstack-cinder | 11:37 | |
| flip214 | DuncanT: jgriffith: do you know off-hand whether iSCSI is configured to do direct access? In test_volume_boot_pattern (http://logs.openstack.org/68/187568/19/check/check-tempest-dsvm-full-drbd-devstack/17f8285/) I can see that the snapshot is missing the /tmp/text file... | 11:37 |
| flip214 | If tgt would use the block device in buffered mode, it might happen that the snapshot only takes "old" data. | 11:38 |
| *** aarefiev has quit IRC | 11:38 | |
| *** e0ne_ is now known as e0ne | 11:39 | |
| *** aarefiev has joined #openstack-cinder | 11:39 | |
| *** edsh has joined #openstack-cinder | 11:41 | |
| *** edsh has quit IRC | 11:43 | |
| *** pradipta has quit IRC | 11:50 | |
| *** ganso_ has joined #openstack-cinder | 11:53 | |
| *** rakesh_mishra has quit IRC | 12:00 | |
| *** Yogi1 has joined #openstack-cinder | 12:04 | |
| *** Yogi1 has quit IRC | 12:10 | |
| *** Yogi1 has joined #openstack-cinder | 12:11 | |
| *** Yogi11 has joined #openstack-cinder | 12:13 | |
| *** aix has joined #openstack-cinder | 12:15 | |
| *** Yogi11 has quit IRC | 12:16 | |
| *** Yogi1 has quit IRC | 12:16 | |
| *** diogogmt has quit IRC | 12:20 | |
| *** chlong has joined #openstack-cinder | 12:20 | |
| *** belmoreira has joined #openstack-cinder | 12:21 | |
| pboldin | jgriffith: what about https://review.openstack.org/182871/ ? Should I address Gorka's comments or is it too late? | 12:27 |
| *** e0ne is now known as e0ne_ | 12:28 | |
| *** e0ne_ is now known as e0ne | 12:28 | |
| *** edmondsw has joined #openstack-cinder | 12:29 | |
| *** bswartz has quit IRC | 12:29 | |
| *** akerr has joined #openstack-cinder | 12:34 | |
| *** kjnelson has joined #openstack-cinder | 12:37 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Ceph driver support retries on rados_connect_timeout https://review.openstack.org/190579 | 12:37 |
| *** ronenkat has quit IRC | 12:46 | |
| *** zerda has joined #openstack-cinder | 12:46 | |
| *** dimsum__ has quit IRC | 12:52 | |
| *** dimsum__ has joined #openstack-cinder | 12:54 | |
| *** dimsum__ is now known as dims | 12:55 | |
| *** zerda has quit IRC | 12:55 | |
| *** dims has quit IRC | 12:55 | |
| *** dims has joined #openstack-cinder | 12:56 | |
| *** smoriya has quit IRC | 12:56 | |
| *** julim has joined #openstack-cinder | 12:58 | |
| *** Yogi1 has joined #openstack-cinder | 12:59 | |
| *** cbader has joined #openstack-cinder | 13:00 | |
| openstackgerrit | Tristan Cacqueray proposed openstack/cinder: Disallow backing files when uploading volumes to image https://review.openstack.org/191785 | 13:01 |
| *** bill_az has joined #openstack-cinder | 13:03 | |
| *** bswartz has joined #openstack-cinder | 13:04 | |
| *** nihilifer has quit IRC | 13:06 | |
| *** deepakcs has quit IRC | 13:10 | |
| *** merooney has joined #openstack-cinder | 13:11 | |
| *** rushiagr is now known as rushiagr_away | 13:14 | |
| *** merooney has quit IRC | 13:14 | |
| *** jwcroppe has quit IRC | 13:15 | |
| flip214 | jgriffith: DuncanT: thingee: DRBD driver at https://review.openstack.org/#/c/178573/ matched CI in devstack plugin run at https://review.openstack.org/#/c/187568/ | 13:18 |
| openstackgerrit | Ph. Marek proposed openstack/cinder: Re-add DRBD driver. https://review.openstack.org/178573 | 13:21 |
| flip214 | ^^ was because pep8 complained that I use "+" to concatenate two strings on two lines... | 13:22 |
| *** merooney has joined #openstack-cinder | 13:22 | |
| *** pboldin has quit IRC | 13:23 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Switch to oslo.policy 0.3.0 https://review.openstack.org/165688 | 13:23 |
| *** jwcroppe has joined #openstack-cinder | 13:24 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 13:25 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Ceph driver support retries on rados_connect_timeout https://review.openstack.org/190579 | 13:26 |
| *** mriedem has joined #openstack-cinder | 13:27 | |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/cinder: Cinder part of volume snapshot improvements https://review.openstack.org/172808 | 13:29 |
| *** Liu has joined #openstack-cinder | 13:33 | |
| Liu | ping thingee | 13:34 |
| *** edmondsw has quit IRC | 13:34 | |
| *** Yogi1 has quit IRC | 13:36 | |
| *** pboldin has joined #openstack-cinder | 13:37 | |
| Liu | thingee: hi thingee | 13:37 |
| Liu | thingee: Is driver bp have chance to be merged in L-2? | 13:37 |
| *** edmondsw has joined #openstack-cinder | 13:38 | |
| smcginnis | Liu: Driver deadlines for Liberty is L-1. | 13:40 |
| smcginnis | Liu: Here was the announcement: http://lists.openstack.org/pipermail/openstack-dev/2015-May/064072.html | 13:41 |
| flip214 | also, http://www.openstack.org/blog/2015/05/deadline-for-new-cinder-volume-drivers-in-liberty/ | 13:42 |
| *** anshul has quit IRC | 13:42 | |
| smcginnis | flip214: Thanks, that's the better one I was looking for. :) | 13:42 |
| *** Yogi1 has joined #openstack-cinder | 13:42 | |
| *** thangp has joined #openstack-cinder | 13:43 | |
| *** merooney has quit IRC | 13:43 | |
| flip214 | smcginnis: I have it in my browser history... (unfortunately? ;| ) | 13:44 |
| smcginnis | ;) | 13:44 |
| Liu | smcginnis: Is the deadlines of L-1 is for new drivers? | 13:45 |
| smcginnis | Liu: Yes, for any new drivers. | 13:45 |
| *** zhenguo has joined #openstack-cinder | 13:45 | |
| Liu | smcginis: Our driver have been merged into Kilo, so does the deadline have infulence for the drivers that have been merged in Kilo? | 13:46 |
| smcginnis | Liu: If your driver is already there then you are fine. You can still do updates. This was just for new drivers being added. | 13:47 |
| *** jungleboyj has joined #openstack-cinder | 13:48 | |
| Liu | smcginnis: thanks smcginnis | 13:49 |
| smcginnis | Liu: np | 13:49 |
| *** superdan is now known as dansmith | 13:49 | |
| Liu | smcginnis: So is there any requirement for old drivers update? | 13:49 |
| *** ankit_ag has quit IRC | 13:50 | |
| smcginnis | Liu: Just that you keep your third party CI running and reporting results. | 13:50 |
| smcginnis | Liu: Then up to you if you want to add support for any new functionality. | 13:50 |
| Liu | smcginnis: Thanks smcginnis :) | 13:51 |
| Liu | smcginnis: you do a great flaver for us | 13:51 |
| openstackgerrit | chenzongliang proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 13:52 |
| *** merooney has joined #openstack-cinder | 13:52 | |
| *** ronenkat has joined #openstack-cinder | 13:53 | |
| Liu | smcginnis: favour | 13:53 |
| *** shausy has quit IRC | 13:56 | |
| flip214 | e0ne: please, help me get the driver merged. small changes (like constant names) can be done later (and much easier, because then the devstack plugin would be merged, too). | 13:58 |
| *** timcl has quit IRC | 13:59 | |
| e0ne | flip214: hi. at least, you need to fix using six instead of str and unicode types | 14:00 |
| *** hodos has joined #openstack-cinder | 14:01 | |
| *** Yogi1 has quit IRC | 14:02 | |
| *** rushil has joined #openstack-cinder | 14:04 | |
| *** merooney has quit IRC | 14:05 | |
| *** ociuhandu has joined #openstack-cinder | 14:06 | |
| openstackgerrit | Edward Balduf proposed openstack/python-cinderclient: The is_public filter in VolumeTypeManager.list broke the find in VolumeTypeAccessManager, which caused all commands which should have taken a volume_type name to fail. I have choosen to fix this by effectively removing the filter in the client (and the - https://review.openstack.org/191621 | 14:06 |
| e0ne | flip214: i answered to your comments. | 14:06 |
| e0ne | flip214: your code is very close to be merged. just need to fix some styling issues | 14:07 |
| *** hodos|2 has joined #openstack-cinder | 14:07 | |
| openstackgerrit | Kevin Fox proposed openstack/cinder: Add Posix backup driver. Supports simple nas too. https://review.openstack.org/163647 | 14:07 |
| e0ne | flip214: imo, it's fair enough to have the same coding guidelines for all patches | 14:08 |
| *** timcl has joined #openstack-cinder | 14:09 | |
| *** marcusvrn1 has joined #openstack-cinder | 14:09 | |
| flip214 | hmmm, my "pydoc six" on the devstack machine doesn't return "str_type"... | 14:10 |
| flip214 | in dec. you said the same thing about the same line... | 14:10 |
| *** merooney has joined #openstack-cinder | 14:10 | |
| *** hodos has quit IRC | 14:11 | |
| flip214 | https://review.openstack.org/#/c/140451/6..7/cinder/volume/drivers/drbdmanagedrv.py line 183 | 14:11 |
| flip214 | and I answered "No, sorry, that doesn't work here. I have to *check* the type, not convert." | 14:11 |
| flip214 | Isn't that still valid? | 14:11 |
| e0ne | flip214: http://pythonhosted.org/six/#six.string_types and http://pythonhosted.org/six/#six.text_type | 14:12 |
| *** marcusvrn has quit IRC | 14:12 | |
| *** eharney has joined #openstack-cinder | 14:12 | |
| e0ne | flip214: "type(volume) is six.string_types" should be | 14:13 |
| e0ne | flip214: i'm very and very sorry if i confused you. i could mix up six.string_types and six.text_type:( | 14:14 |
| e0ne | flip214: of course, you need six.string_types, not six.str_types. it was my mistake | 14:15 |
| *** IlyaG has joined #openstack-cinder | 14:17 | |
| *** IanGovett has joined #openstack-cinder | 14:17 | |
| flip214 | oh, okay. | 14:17 |
| flip214 | still, can I please do that in the very next patch, right after that one is upstream? | 14:18 |
| e0ne | flip214: if other cores will merge the current one - yes. i won't change my vote for using str and unicode. | 14:19 |
| e0ne | flip214: we try to not merge py3 incompatible code | 14:20 |
| *** yuriy_n17 has joined #openstack-cinder | 14:22 | |
| flip214 | so, " if type(volume) is str or type(volume) is unicode: | 14:23 |
| flip214 | would become "if type(volume) is six.string_types:" ? | 14:23 |
| *** openstackgerrit has quit IRC | 14:24 | |
| *** bnemec has joined #openstack-cinder | 14:24 | |
| *** diogogmt has joined #openstack-cinder | 14:24 | |
| flip214 | e0ne: ^^ | 14:24 |
| *** openstackgerrit has joined #openstack-cinder | 14:24 | |
| e0ne | flip214: just checked: isinstance(volume,six.string_types) - it works as expected | 14:25 |
| e0ne | "type(volume) is six.string_types" doesn't work:( | 14:26 |
| *** Yogi1 has joined #openstack-cinder | 14:26 | |
| flip214 | yeah, that's what I just wanted to paste... | 14:27 |
| flip214 | thanks for clarifying. | 14:27 |
| e0ne | flip214: you're welcome. i'm sorry if confused you | 14:28 |
| flip214 | well, you cleared it up again, so no harm done. | 14:28 |
| openstackgerrit | Ph. Marek proposed openstack/cinder: Re-add DRBD driver. https://review.openstack.org/178573 | 14:29 |
| flip214 | e0ne: ^^ here you are | 14:29 |
| flip214 | e0ne: pep8 doesn't blame me for the "..._id" const | 14:30 |
| flip214 | so I've kept it now as-is, to keep the diff to the originally accepted version (in kilo) smaller | 14:30 |
| e0ne | flip214: it's an issue with pep8 checker. http://legacy.python.org/dev/peps/pep-0008/#constants | 14:31 |
| *** garthb has joined #openstack-cinder | 14:34 | |
| *** garthb_ has joined #openstack-cinder | 14:34 | |
| *** sgotliv has joined #openstack-cinder | 14:34 | |
| tristanC | eharney: hello sir, I pushed your proposed fix for bug 1415087... Tests seems to be failing for master and cherry-pick is not trivial for Juno and Icehouse | 14:35 |
| openstack | bug 1415087 in Cinder "Format-guessing and file disclosure in image convert (CVE-2015-1850)" [High,In progress] https://launchpad.net/bugs/1415087 - Assigned to Tristan Cacqueray (tristan-cacqueray) | 14:36 |
| eharney | tristanC: i'll take a look at the failures and backports | 14:37 |
| *** coolsvap is now known as coolsvap|afk | 14:37 | |
| *** sgotliv has quit IRC | 14:39 | |
| tristanC | eharney: awesome, thanks! | 14:39 |
| *** cFouts has joined #openstack-cinder | 14:40 | |
| *** jnicklin has joined #openstack-cinder | 14:41 | |
| *** amoturi has joined #openstack-cinder | 14:41 | |
| *** sgotliv has joined #openstack-cinder | 14:43 | |
| *** marcusvrn has joined #openstack-cinder | 14:45 | |
| *** breitz has joined #openstack-cinder | 14:52 | |
| *** tsekiyama has joined #openstack-cinder | 14:53 | |
| *** dulek has quit IRC | 14:53 | |
| *** patrickeast has joined #openstack-cinder | 14:54 | |
| *** gouthamr has joined #openstack-cinder | 14:54 | |
| *** gouthamr has quit IRC | 14:54 | |
| *** gouthamr has joined #openstack-cinder | 14:55 | |
| *** patrickeast has quit IRC | 14:55 | |
| *** patrickeast has joined #openstack-cinder | 14:56 | |
| *** merooney has quit IRC | 15:01 | |
| *** nkrinner has quit IRC | 15:01 | |
| *** haomaiw__ has joined #openstack-cinder | 15:03 | |
| *** haomaiwang has quit IRC | 15:04 | |
| *** timcl has quit IRC | 15:04 | |
| *** crose has joined #openstack-cinder | 15:05 | |
| *** potsmaster has quit IRC | 15:10 | |
| *** IlyaG has quit IRC | 15:11 | |
| *** haomaiwa_ has joined #openstack-cinder | 15:11 | |
| *** timcl has joined #openstack-cinder | 15:11 | |
| *** merooney has joined #openstack-cinder | 15:11 | |
| *** timcl has quit IRC | 15:12 | |
| *** haomaiw__ has quit IRC | 15:13 | |
| *** timcl has joined #openstack-cinder | 15:15 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Huawei Storage Multiple Pools Support https://review.openstack.org/188251 | 15:16 |
| *** ronis has quit IRC | 15:16 | |
| *** dannywilson has joined #openstack-cinder | 15:17 | |
| *** timcl has quit IRC | 15:17 | |
| *** haypo has left #openstack-cinder | 15:17 | |
| *** belmoreira has quit IRC | 15:18 | |
| *** dannywilson has quit IRC | 15:19 | |
| *** dannywilson has joined #openstack-cinder | 15:19 | |
| *** Yogi1 has quit IRC | 15:22 | |
| *** timcl has joined #openstack-cinder | 15:22 | |
| *** e0ne is now known as e0ne_ | 15:22 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Add CHAP support for Huawei Storage https://review.openstack.org/188360 | 15:26 |
| *** timcl has quit IRC | 15:28 | |
| *** Yogi1 has joined #openstack-cinder | 15:28 | |
| *** jdurgin1 has joined #openstack-cinder | 15:30 | |
| *** e0ne_ has quit IRC | 15:32 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Support SmartX for Huawei volume driver https://review.openstack.org/188732 | 15:33 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Disallow backing files when uploading volumes to image https://review.openstack.org/191785 | 15:33 |
| *** e0ne has joined #openstack-cinder | 15:35 | |
| *** jdurgin1 has quit IRC | 15:35 | |
| eharney | tristanC: updated master and kilo patches, that should fix the failures | 15:35 |
| smcginnis | 0m0Arigato! | 15:36 |
| tristanC | eharney: that's very appreciated, any chance to get a Juno and Icehouse backport proposed ? | 15:36 |
| eharney | tristanC: going to try to do that soon | 15:36 |
| smcginnis | That was weird | 15:37 |
| eharney | i'm working on Juno now | 15:37 |
| smcginnis | :) | 15:37 |
| *** jordanP has quit IRC | 15:37 | |
| thingee | Liu: http://lists.openstack.org/pipermail/openstack-dev/2015-May/064072.html | 15:38 |
| tristanC | eharney: again, thanks! | 15:38 |
| thingee | flip214: cool, I'll wait for the next ci pass before reviewing | 15:39 |
| thingee | since the current patchset is failing for some reason | 15:39 |
| *** timcl has joined #openstack-cinder | 15:40 | |
| jungleboyj | smcginnis: Was that supposed to be Domo Origato ? | 15:40 |
| jungleboyj | Mr. Robot? | 15:41 |
| jungleboyj | Roboto | 15:41 |
| smcginnis | jungleboyj: Yeah, something like that from my clipboard. :) | 15:41 |
| jungleboyj | Hi | 15:42 |
| *** IanGovett has quit IRC | 15:42 | |
| *** IanGovett has joined #openstack-cinder | 15:42 | |
| *** harlowja_at_home has joined #openstack-cinder | 15:43 | |
| scottda | guitarzan: You had mentioned you knew Nova devs who were interested in 'force-detach'. The Nova deadline for L specs is next Tuesday, so please point them to this spec: https://review.openstack.org/#/c/84048/ | 15:44 |
| guitarzan | scottda: what a coincidence... I just heard something about nova force detach this mornign | 15:44 |
| guitarzan | scottda: I will absolutely point folks in that direction | 15:45 |
| scottda | Thanks. | 15:45 |
| *** leeantho has joined #openstack-cinder | 15:45 | |
| scottda | Anyone else interested in facilitating clean up of stuck volumes, please help with Nova side spec review ^^^. It's a must to allow operators to force-detach a volume stuck in 'attaching' or 'detaching'. | 15:46 |
| *** IanGovett has quit IRC | 15:46 | |
| *** yuriy_n17 has quit IRC | 15:49 | |
| *** afazekas has quit IRC | 15:50 | |
| *** merooney has quit IRC | 15:50 | |
| *** Apoorva has joined #openstack-cinder | 15:51 | |
| *** patrickeast has quit IRC | 15:52 | |
| *** gouthamr has quit IRC | 15:52 | |
| *** rmesta has joined #openstack-cinder | 15:53 | |
| openstackgerrit | Kurt Martin proposed openstack/cinder: Update 3PAR user config help strings https://review.openstack.org/190811 | 15:59 |
| *** amoturi has quit IRC | 16:01 | |
| *** jaypipes has quit IRC | 16:03 | |
| *** jaypipes has joined #openstack-cinder | 16:05 | |
| *** c0m0 has quit IRC | 16:05 | |
| *** vokt has joined #openstack-cinder | 16:07 | |
| *** lpetrut has quit IRC | 16:09 | |
| thingee | these nexenta ns5 patches all show the wrong driver being tested according to their CI | 16:12 |
| *** salv-orlando has joined #openstack-cinder | 16:12 | |
| *** oswa has joined #openstack-cinder | 16:14 | |
| eharney | tristanC: got them all uploaded... waiting for Jenkins now: https://review.openstack.org/#/q/Iab971879,n,z | 16:15 |
| tristanC | eharney: perfect, that will do! | 16:16 |
| *** salv-orl_ has quit IRC | 16:16 | |
| thingee | jgriffith, eharney, jungleboyj , hemnafk , e0ne, smcginnis https://etherpad.openstack.org/p/cinder-liberty-drivers Last update. | 16:19 |
| e0ne | thingee: thanks! | 16:19 |
| jungleboyj | thingee: Thanks. I have time blocked out this afternoon to look at those. Sorry I haven't been more on top of those. | 16:21 |
| thingee | jungleboyj: I haven't either. I think everyone else has been merging things for me. just been coordinating | 16:21 |
| thingee | trying to do my part now though | 16:21 |
| *** Longgeek has quit IRC | 16:21 | |
| kmartin | thingee, FYI...hemnafk is off fishing for a few days, he'll be back on Wednesday. | 16:22 |
| thingee | kmartin: ah nice | 16:22 |
| jungleboyj | :-) | 16:22 |
| thingee | kmartin: well luckily the drivers left is an easy load IMO | 16:22 |
| *** Yogi1 has quit IRC | 16:22 | |
| jungleboyj | kmartin: Who said hemnafk could go relax? ;-) | 16:22 |
| kmartin | apparently some CA lakes still have water in them | 16:23 |
| jungleboyj | thingee: So the 'Not ready' ones are not going to make it for Liberty? | 16:23 |
| jungleboyj | kmartin: Meanwhile the midwest is getting new lakes all over the place. | 16:23 |
| thingee | jungleboyj: correct | 16:23 |
| jungleboyj | thingee: Ok, cool. | 16:23 |
| thingee | the nexenta ci's are a mess and testing the wrong driver according to cinder.conf | 16:23 |
| thingee | I noted it in the comments for the patches | 16:24 |
| *** ronis has joined #openstack-cinder | 16:24 | |
| thingee | virtuozzo, is blaming a tempest test on why their CI fails. Undecided on that one atm | 16:24 |
| *** sage has quit IRC | 16:25 | |
| *** daneyon has joined #openstack-cinder | 16:26 | |
| *** daneyon has quit IRC | 16:26 | |
| *** sage has joined #openstack-cinder | 16:27 | |
| thingee | also the one nexenta CI that is testing the right driver (edge), is failing very often | 16:27 |
| *** daneyon has joined #openstack-cinder | 16:27 | |
| thingee | that last 20 runs shows it pass once. | 16:27 |
| thingee | I mentioned this issue last week, and it doesn't look like it was corrected. | 16:27 |
| *** patrickeast has joined #openstack-cinder | 16:29 | |
| flip214 | thingee: the DRBD cinder driver alone won't pass. | 16:29 |
| *** timcl has quit IRC | 16:30 | |
| flip214 | it needs the devstack plugin too, which is set to depends-on the drbd driver... | 16:30 |
| *** vilobhmm has joined #openstack-cinder | 16:30 | |
| flip214 | so, the devstack plugin at https://review.openstack.org/#/c/187568/ shows that both together work fine. | 16:32 |
| thingee | flip214: http://docs.openstack.org/infra/manual/developers.html#cross-repository-dependencies | 16:33 |
| thingee | flip214: that should help | 16:33 |
| flip214 | but the drbd driver still uses the old plugin, and so won't go through. Good thing that it's non-voting ;) | 16:33 |
| flip214 | thingee: I have one of these headers. In the devstack plugin. | 16:33 |
| flip214 | I can't add another, because "These are one-way dependencies only – do not create a cycle." | 16:33 |
| thingee | ah | 16:34 |
| flip214 | but in the plugin you'll see the line "check-tempest-dsvm-full-drbd-devstack SUCCESS in 38m 26s" | 16:34 |
| flip214 | https://review.openstack.org/#/c/187568/ | 16:34 |
| *** pboldin has quit IRC | 16:35 | |
| *** Maike has quit IRC | 16:39 | |
| *** markus_z has quit IRC | 16:41 | |
| *** rakesh_mishra has joined #openstack-cinder | 16:42 | |
| *** _cjones_ has joined #openstack-cinder | 16:44 | |
| earlephilhower_ | Is there a way to make zuul look at gerrit commits only for certain branches? My CI is failing stable/kilo patches because, well, Kilo doesn't have the driver. Don't want to spam the backports/bugfixes. | 16:44 |
| *** Yogi1 has joined #openstack-cinder | 16:48 | |
| *** pboldin has joined #openstack-cinder | 16:49 | |
| patrickeast | earlephilhower_: yep, you can specify a branch regex for the triggers | 16:49 |
| *** shyama_ has joined #openstack-cinder | 16:50 | |
| patrickeast | earlephilhower_: for example, we exclude icehouse from ours since our driver was not in-tree for it http://paste.openstack.org/show/294395/ | 16:51 |
| earlephilhower_ | patrickeast: Thanks, I'll check layout.yaml. My initial thought was a regex in the Jenkins job, but by then an abort would report a FAILED to Gerrit so no dice. | 16:51 |
| *** shyama has quit IRC | 16:51 | |
| *** shyama_ is now known as shyama | 16:51 | |
| *** alau2 has joined #openstack-cinder | 16:51 | |
| patrickeast | earlephilhower_: unfortunately there isn’t a good way to do it without having to specify for every single trigger | 16:52 |
| *** harlowja_at_home has quit IRC | 16:53 | |
| thingee | patrickeast: https://etherpad.openstack.org/p/cinder-liberty-drivers last update | 16:53 |
| *** gouthamr has joined #openstack-cinder | 16:53 | |
| earlephilhower_ | patrickeast: Got it. I've only got a couple triggers so cut-n-paste's fine by me. | 16:53 |
| patrickeast | thingee: cool, so the rebranded ones got the ok? | 16:54 |
| thingee | patrickeast: yes | 16:54 |
| *** dencaval has joined #openstack-cinder | 16:55 | |
| nikeshm | thingee : thanks | 16:56 |
| *** amoturi has joined #openstack-cinder | 16:56 | |
| *** IanGovett has joined #openstack-cinder | 16:57 | |
| *** edsh has joined #openstack-cinder | 16:58 | |
| *** kvidvans has quit IRC | 16:58 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Targets tests: Clean up long lines https://review.openstack.org/191209 | 17:00 |
| *** IanGovett has quit IRC | 17:01 | |
| *** IanGovett1 has joined #openstack-cinder | 17:01 | |
| *** e0ne has quit IRC | 17:02 | |
| patrickeast | nikeshm: what errors are you running into with the CI system to make you skip (?!tempest.scenario.test_volume_boot_pattern.*)(?!tempest.scenario.test_minimum_basic.*) ? | 17:02 |
| patrickeast | nikeshm: ssh timeouts in the tests or something different? | 17:02 |
| * patrickeast has trouble with those too | 17:02 | |
| *** edsh has quit IRC | 17:03 | |
| *** haomaiw__ has joined #openstack-cinder | 17:03 | |
| *** haomaiwa_ has quit IRC | 17:04 | |
| rlucio | thingee: i see that the vmem driver isn't on the list for liberty, not sure if you saw i updated the review the other day | 17:04 |
| nikeshm | patrickeast: tempest.scenario.test_minimum_basic is solved by asselin using neutron | 17:06 |
| patrickeast | nikeshm: orly | 17:06 |
| patrickeast | thats good to know | 17:06 |
| thingee | rlucio: your CI's are not running the right tests. See https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#What_tests_do_I_use.3F | 17:07 |
| thingee | rlucio: you're only hitting the volume.api related tests. There are additional tests that involve interactions with Nova that are needed, but depending on how you're doing your CI, that should help with doing the correct discovery | 17:08 |
| thingee | rlucio: but the number is definitely higher that 249 tests. I think we're around 280 or something. | 17:08 |
| *** lcurtis has joined #openstack-cinder | 17:09 | |
| rlucio | thingee: ok i'll take a look at that right now, thanks | 17:09 |
| thingee | 279 tests actually | 17:09 |
| *** simondodsley has joined #openstack-cinder | 17:13 | |
| vilobhmm | dulek : ping | 17:14 |
| *** rakesh_mishra has quit IRC | 17:16 | |
| thingee | vilobhmm: last update on drivers for review https://etherpad.openstack.org/p/cinder-liberty-drivers in liberty | 17:20 |
| *** merooney has joined #openstack-cinder | 17:22 | |
| *** timcl has joined #openstack-cinder | 17:23 | |
| *** vokt has quit IRC | 17:24 | |
| *** shyama has quit IRC | 17:25 | |
| *** merooney has quit IRC | 17:25 | |
| *** merooney has joined #openstack-cinder | 17:27 | |
| *** dims has quit IRC | 17:27 | |
| *** dims has joined #openstack-cinder | 17:28 | |
| smcginnis | thingee: So still a chance to Virtuozza and Voilin, correct? Everything else is out for L. | 17:30 |
| thingee | smcginnis: yea | 17:31 |
| *** vokt has joined #openstack-cinder | 17:31 | |
| vilobhmm | thingee : is there something i can help with https://etherpad.openstack.org/p/cinder-liberty-drivers | 17:34 |
| vilobhmm | saw your ping on this one https://etherpad.openstack.org/p/cinder-liberty-drivers | 17:34 |
| *** marcusvrn has quit IRC | 17:34 | |
| *** salv-orlando has quit IRC | 17:37 | |
| *** merooney has quit IRC | 17:39 | |
| *** IanGovett1 has quit IRC | 17:41 | |
| *** merooney has joined #openstack-cinder | 17:45 | |
| *** shyama has joined #openstack-cinder | 17:46 | |
| *** jnicklin has quit IRC | 17:46 | |
| anteaya | asselin_: do you want that ci system to be re-enabled? | 17:50 |
| anteaya | asselin_: or are you waiting to hear from others? | 17:50 |
| asselin_ | anteaya, would be good to have another person comment | 17:51 |
| anteaya | asselin_: you tell me when you want somethign to happen on infra side | 17:51 |
| asselin_ | anteaya, sure | 17:52 |
| anteaya | thanks | 17:52 |
| *** zhenguo has quit IRC | 17:53 | |
| asselin_ | cinder folks: anyone else want to commend on allowing IBM GPFS CI to post again? patrickeast smcginnis I think you both should be familiar with the account? ref: http://lists.openstack.org/pipermail/third-party-announce/2015-June/000211.html | 17:54 |
| asselin_ | thingee, if you have an opinion too ^^ | 17:55 |
| *** shyama has quit IRC | 17:55 | |
| patrickeast | asselin_: the sandbox stuff look ok to me | 17:56 |
| patrickeast | asselin_: as long as they are going to keep an eye on it this time i’m cool with re-enabling it | 17:57 |
| *** merooney has quit IRC | 18:02 | |
| *** lpabon has joined #openstack-cinder | 18:02 | |
| smcginnis | asselin_: Yeah, that should be fine I think. | 18:02 |
| *** merooney has joined #openstack-cinder | 18:02 | |
| *** aix has quit IRC | 18:09 | |
| asselin_ | anteaya, we can enable IBM GPFS CI ^^ | 18:09 |
| anteaya | okay | 18:10 |
| anteaya | let me find their gerrit account id | 18:10 |
| anteaya | asselin_: done, thank you all | 18:14 |
| anteaya | keep an eye on them, let me know if they go sideways again | 18:14 |
| asselin_ | anteaya, yes, definitely | 18:15 |
| anteaya | thanks | 18:15 |
| *** Longgeek has joined #openstack-cinder | 18:17 | |
| ameade | patrickeast: is this yours? http://ec2-54-67-102-119.us-west-1.compute.amazonaws.com:5000/?project=openstack%2Fcinder&user=&timeframe=72 | 18:21 |
| patrickeast | ameade: yep | 18:21 |
| ameade | patrickeast: could you add netapp-fc-ci as well? | 18:21 |
| ameade | patrickeast: also, what determines the positioning of the columns? | 18:22 |
| ameade | is it # of posts? | 18:22 |
| patrickeast | ameade: the quickest way is if you put up a patch to add it here and i’ll ninja merge it | 18:22 |
| patrickeast | https://github.com/stackforge/third-party-ci-tools/blob/master/monitoring/scoreboard/users.py | 18:22 |
| patrickeast | ameade: nah its kind of arbitrary | 18:22 |
| ameade | ah ok | 18:22 |
| patrickeast | ameade: iirc the order it processes results just kind of whatever one it sees first gets in line on the columns as we build them out | 18:22 |
| patrickeast | what i really need to do is figure out how to get that list automatically… haven’t figure out the authentication piece (or more specifcally haven’t spent any time investigating it) | 18:24 |
| patrickeast | sucks to have this giant list of users checked in like that | 18:24 |
| *** lpetrut has joined #openstack-cinder | 18:28 | |
| thingee | jungleboyj: waiting on master to merge but if we can get this one in, that would be greaaaaat https://review.openstack.org/#/c/191786/2 | 18:32 |
| *** hodos|2 has quit IRC | 18:34 | |
| dguryanov | thingee: About virtuozzo volume driver - the problem is not with tempest, but with assisted_volume_snapshots implementation in nova | 18:36 |
| ameade | patrickeast: https://github.com/stackforge/third-party-ci-tools/pull/1 | 18:38 |
| ameade | patrickeast: ty, hey whatever works for now lol | 18:38 |
| dguryanov | thingee: The test shuts down an instance, waits for SHUTOFF status and then tries to delete a snapshot. The code in RemoteFSSnap driver checks volume's status, gets in-use and calls nova.assisted_volume_snapshots.delete, which is supposed to work only with running instances. | 18:39 |
| patrickeast | ameade: oh interesting, i guess i don’t have write access on github? if you do a ‘git review’ and go through gerrit i can merge it | 18:39 |
| ameade | ah that's on gerrit, hehe >.< | 18:39 |
| ameade | patrickeast: https://review.openstack.org/#/c/191927/ | 18:40 |
| jungleboyj | thingee: Why is that not a cherry pick? | 18:40 |
| *** geguileo has quit IRC | 18:42 | |
| jgriffith | patrickeast: ping | 18:45 |
| *** merooney has quit IRC | 18:46 | |
| *** geguileo has joined #openstack-cinder | 18:47 | |
| patrickeast | jgriffith: pong | 18:48 |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: RPC and object versioning https://review.openstack.org/184404 | 18:48 |
| jgriffith | patrickeast: looking at scoreboard | 18:48 |
| jgriffith | patrickeast: IIUC you're just launching a monitor that listens and updates DB real time? | 18:49 |
| jgriffith | patrickeast: wondering if you have seen or thought of something like this: https://github.com/j-griffith/lastcomment/tree/master | 18:49 |
| patrickeast | jgriffith: yea pretty much, its the same gerrit watching code zuul uses (literally) and it records events related to third party systems | 18:49 |
| jgriffith | patrickeast: just querie gerrit periodcially for completed jobs | 18:49 |
| patrickeast | jgriffith: yea i wasn’t sure the best way to go on that | 18:50 |
| patrickeast | jgriffith: i didn’t want to have to query on-demand, and wasn’t sure abouthe periodic | 18:50 |
| jgriffith | patrickeast: Ok, just thought I'd throw it out there | 18:50 |
| *** merooney has joined #openstack-cinder | 18:50 | |
| jgriffith | patrickeast: I think both have their pros/cons | 18:51 |
| jgriffith | patrickeast: don't know if one way is better than the other or not | 18:51 |
| patrickeast | jgriffith: yea def, i was thinking that the query approach makes for doing stats much easier, and it would allow for getting the whole commit messages and such, i only store like a status of the job | 18:51 |
| patrickeast | after doing some simple decision about what the result comment meant | 18:51 |
| *** e0ne has joined #openstack-cinder | 18:51 | |
| jgriffith | patrickeast: right, honestly the only thing I care about it agreement ratio with merged patches | 18:51 |
| *** Longgeek has quit IRC | 18:52 | |
| jgriffith | patrickeast: but that's just me and "my interpretation" | 18:52 |
| jgriffith | :) | 18:52 |
| patrickeast | jgriffith: yea i’ve been pondering how best to do that | 18:52 |
| patrickeast | the problem is that jenkins fails sometimes for things the ci wont | 18:52 |
| patrickeast | like unit tests | 18:52 |
| jgriffith | patrickeast: yeah | 18:52 |
| jgriffith | patrickeast: but if it's "merged" the Gate passed | 18:52 |
| patrickeast | so im thinking the code needs to be smart enough to compare dsvm-tempest-full and such with the individual ci jobs | 18:52 |
| jgriffith | patrickeast: that's why I picked that metric TBH | 18:52 |
| patrickeast | true | 18:52 |
| patrickeast | but then we lose the results of each patchset right? | 18:53 |
| jgriffith | patrickeast: so there's a TON of extraneous test runs and data that frankly I don't know what to do with, (or want to deal with) | 18:53 |
| jgriffith | so in my stuff I'm just taking "last 'n' merged" and tallying up all the votes | 18:53 |
| jgriffith | patrickeast: yes | 18:53 |
| patrickeast | jgriffith: so one thing thats a step in the right direction is https://review.openstack.org/#/c/191910/ | 18:53 |
| jgriffith | patrickeast: you don't get patchset specific info | 18:53 |
| patrickeast | jgriffith: which sort of sets the stage for calculating whatever stats we want from the results being displayed | 18:54 |
| patrickeast | we can start adding in some of these comparisons, maybe even just a ‘disagreed with jenkins’ count | 18:54 |
| patrickeast | and refine from there | 18:54 |
| jgriffith | patrickeast: yeah, so I'm assuming then this is going to be hosted up on some OpenStack resource somewhere? | 18:55 |
| jgriffith | That's going to manage and update the DB? | 18:55 |
| *** kjnelson has quit IRC | 18:55 | |
| patrickeast | jgriffith: i think so | 18:55 |
| jgriffith | patrickeast: cool | 18:55 |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: RPC and object versioning https://review.openstack.org/184404 | 18:55 |
| patrickeast | jgriffith: at the 3rd party ci working group meeting we came to the decision to drop the dashboard spec since it isn’t going anywhere and get the scoreboard up asap as a placeholder | 18:55 |
| patrickeast | while the radar/dashboard thing keeps moving along slowly | 18:56 |
| jgriffith | patrickeast: seems like a good diea | 18:56 |
| jgriffith | idea | 18:56 |
| patrickeast | jgriffith: asselin_: volunteered to put a spec up to get it hosted by infra | 18:56 |
| patrickeast | and anteaya has been supportive of it | 18:56 |
| *** thangp has quit IRC | 18:56 | |
| jgriffith | cool, I'll have to start watching those repos/specs | 18:57 |
| jgriffith | too much info is almost as bad as not enough IMHO | 18:57 |
| patrickeast | good point | 18:57 |
| smcginnis | anteaya: ping | 18:58 |
| *** Rockyg has joined #openstack-cinder | 18:59 | |
| *** thangp has joined #openstack-cinder | 18:59 | |
| *** ociuhandu has quit IRC | 19:00 | |
| *** lpabon has quit IRC | 19:01 | |
| *** timcl has quit IRC | 19:04 | |
| *** timcl has joined #openstack-cinder | 19:05 | |
| anteaya | smcginnis: hello | 19:06 |
| *** crose has quit IRC | 19:07 | |
| *** rushil has quit IRC | 19:07 | |
| *** crose has joined #openstack-cinder | 19:15 | |
| smcginnis | anteaya: Hey, sorry. Got pulled away. | 19:16 |
| smcginnis | anteaya: I think you may have explained this before, but can you tell me what determines if a comment shows up in the CI results table on a review? | 19:16 |
| anteaya | its format | 19:17 |
| anteaya | the comment puller outer regex is formatted for jenkins | 19:17 |
| anteaya | copy jenkins format and you are in the table to the right | 19:17 |
| smcginnis | anteaya: That's what I thought. Looks like one isn't working here: https://review.openstack.org/#/c/191786/ | 19:18 |
| smcginnis | anteaya: It looks to be in the proper format, but not showing up in the table. | 19:18 |
| anteaya | there is currently some js voodoo with gerrit underway | 19:18 |
| anteaya | follow the conversation in infra for a closer play by play | 19:18 |
| smcginnis | anteaya: OK, thanks! | 19:19 |
| anteaya | I would be inclined to let them finish and then ask a question if the issue is still in evidence | 19:19 |
| anteaya | thank you | 19:19 |
| smcginnis | anteaya: Will do, thanks for the sage advice. :) | 19:19 |
| anteaya | I try | 19:20 |
| anteaya | :) | 19:20 |
| *** crose has quit IRC | 19:25 | |
| *** amoturi has quit IRC | 19:26 | |
| *** mtanino has joined #openstack-cinder | 19:28 | |
| *** hodos has joined #openstack-cinder | 19:28 | |
| *** ociuhandu has joined #openstack-cinder | 19:29 | |
| theanalyst | hi can someone review this blueprint https://blueprints.launchpad.net/cinder/+spec/backup-restore-display-name | 19:30 |
| theanalyst | something small to start my openstack cinder journey :) | 19:31 |
| *** simondodsley has quit IRC | 19:34 | |
| *** gouthamr has quit IRC | 19:34 | |
| *** gouthamr has joined #openstack-cinder | 19:36 | |
| hodos | thingee: you've posted that we have wrong backend running on couple of reviews...so it's OK to run not all the backends but only 1 for new driver review? | 19:41 |
| hodos | thingee: just the driver we're posting, right | 19:41 |
| thingee | hodos: link to review? | 19:41 |
| thingee | hodos: so I have a lot of these going on to know which you're talking about | 19:42 |
| thingee | sorry* | 19:42 |
| hodos | thingee: https://review.openstack.org/#/c/190273/ https://review.openstack.org/#/c/190391/ | 19:42 |
| *** Rockyg has quit IRC | 19:42 | |
| *** Rockyg has joined #openstack-cinder | 19:43 | |
| hodos | thingee: cause we have 4 backends to test against and right now it's not 100% stable to go through all of them | 19:43 |
| hodos | thingee: we had successfull runs for all of them, but not consistent | 19:44 |
| *** afazekas has joined #openstack-cinder | 19:45 | |
| *** Lee1092 has quit IRC | 19:45 | |
| thingee | hodos: the three I know about are: ns5 nfs, ns5 iscsi and ns edge | 19:46 |
| thingee | on the patches I've commented, I've noted ns5 drivers are not being tested according to the reports | 19:47 |
| thingee | hodos: and the edge CI is very unstable. | 19:48 |
| *** alexpilotti has quit IRC | 19:48 | |
| *** afazekas has quit IRC | 19:49 | |
| hodos | thingee: agree on the edge | 19:50 |
| hodos | thingee: we have the ns4 - to re add back since it was removed from Kilo due to missing CI, right now it's running and the most stable | 19:51 |
| hodos | thingee: https://review.openstack.org/#/c/188877/ | 19:51 |
| *** boris-42 has quit IRC | 19:52 | |
| hodos | thingee: we had a couple of successfull runs for ns5 as well, if we manage to get CI stable for ns5 in the next day - would it be able to make it? | 19:52 |
| thingee | hodos: I see, you have ns_iscsi and ns5_iscsi... | 19:53 |
| thingee | hodos: I'll have to take a closer look at the CI's | 19:53 |
| hodos | thingee: NS5 right now is off due to instability, so we don't spam reviews with negative tests | 19:54 |
| thingee | hodos: lets just focus on ns_iscsi and ns_nfs then | 19:54 |
| *** dannywilson has quit IRC | 19:57 | |
| *** afazekas has joined #openstack-cinder | 19:57 | |
| *** dannywilson has joined #openstack-cinder | 19:58 | |
| e0ne | thingee: hi. we've got BlockDeviceDriver driver. do we need CI for it? | 19:59 |
| *** daneyon_ has joined #openstack-cinder | 19:59 | |
| *** kjnelson has joined #openstack-cinder | 19:59 | |
| e0ne | thingee: wrong question. what is deadline for CI for this driver? | 20:00 |
| *** rushil has joined #openstack-cinder | 20:01 | |
| *** merooney has quit IRC | 20:01 | |
| *** gouthamr has quit IRC | 20:02 | |
| *** daneyon has quit IRC | 20:02 | |
| *** gouthamr has joined #openstack-cinder | 20:03 | |
| openstackgerrit | John Griffith proposed openstack/cinder: SQL scripts should not manage transactions https://review.openstack.org/191956 | 20:04 |
| *** afazekas has quit IRC | 20:04 | |
| *** Rockyg has quit IRC | 20:04 | |
| *** merooney has joined #openstack-cinder | 20:05 | |
| *** Rockyg has joined #openstack-cinder | 20:05 | |
| *** rushil has quit IRC | 20:07 | |
| *** rushil has joined #openstack-cinder | 20:08 | |
| *** dencaval has quit IRC | 20:08 | |
| *** marcusvrn1 has quit IRC | 20:14 | |
| *** merooney has quit IRC | 20:15 | |
| *** ronenkat has quit IRC | 20:16 | |
| *** belmoreira has joined #openstack-cinder | 20:16 | |
| *** gouthamr has quit IRC | 20:17 | |
| flip214 | jgriffith: if you have a minute, please take a look at https://review.openstack.org/#/c/178573/ -- thank you! | 20:17 |
| *** Rockyg has quit IRC | 20:17 | |
| *** gouthamr has joined #openstack-cinder | 20:18 | |
| thingee | and DuncanT ^ | 20:19 |
| flip214 | thingee: DuncanT is in another timezone, I | 20:20 |
| flip214 | I'll poke him in 10 hours. | 20:20 |
| thingee | flip214: I know, but he's listed as the other reviewer for your driver | 20:20 |
| flip214 | but thatnks for trying ;) | 20:20 |
| flip214 | *thanks | 20:20 |
| flip214 | too late, good night ;/ | 20:20 |
| thingee | it's more of a reminder...I have to do that with DuncanT :) | 20:20 |
| flip214 | okay, thanks for telling. I'll continue when you're leaving, in about 10 hours or so ;) | 20:20 |
| *** lpetrut has quit IRC | 20:22 | |
| vilobhmm | DuncanT : ping | 20:24 |
| *** afazekas has joined #openstack-cinder | 20:26 | |
| *** timcl has quit IRC | 20:26 | |
| *** timcl has joined #openstack-cinder | 20:26 | |
| *** angela-s has joined #openstack-cinder | 20:28 | |
| *** afazekas has quit IRC | 20:31 | |
| *** gouthamr has quit IRC | 20:33 | |
| openstackgerrit | Marian Horban proposed openstack/cinder: Graceful shutdown of RPC server https://review.openstack.org/191964 | 20:34 |
| *** julim has quit IRC | 20:37 | |
| *** afazekas has joined #openstack-cinder | 20:38 | |
| *** dannywilson has quit IRC | 20:43 | |
| *** ronis has quit IRC | 20:43 | |
| *** pboldin has quit IRC | 20:44 | |
| *** dannywilson has joined #openstack-cinder | 20:44 | |
| *** afazekas has quit IRC | 20:44 | |
| *** vokt has quit IRC | 20:46 | |
| *** akerr has quit IRC | 20:47 | |
| *** timcl has quit IRC | 20:47 | |
| *** Longgeek has joined #openstack-cinder | 20:49 | |
| thingee | hodos: the ns_iscsi and ns_nfs CIs are not very well stable according to stats from Nexenta CI in the last 60 times it has left comments in reviews. | 20:52 |
| *** rushil has quit IRC | 20:52 | |
| hodos | thingee: we've been working on this lately and it seems like it's getting more and more stable at this point | 20:52 |
| hodos | thingee: we've added some HW: 10gb switch and SSDs | 20:53 |
| thingee | hodos: The funny thing is, this is all happening now, but couldn't have been started back in March when Nexenta was removed from not having a CI at all. | 20:53 |
| thingee | hodos: the problem is, I need to cut the amount we're going to review so it's doable. The more last minute people I get the less that's doable | 20:54 |
| thingee | hodos: do you understand? | 20:54 |
| hodos | thingee: Yes I understand, but I thought that at least NS4 drivers made the requirements | 20:55 |
| *** rushil has joined #openstack-cinder | 20:55 | |
| *** timcl has joined #openstack-cinder | 20:55 | |
| *** kjnelson has quit IRC | 20:56 | |
| thingee | hodos: the last 60 comments from your CI... success count by job: | 20:56 |
| thingee | * nexenta-dsvm-volume-ns_iscsi: 16 | 20:56 |
| thingee | * nexenta-dsvm-volume-ns_edge: 1 | 20:56 |
| thingee | * nexenta-dsvm-volume-ns5_iscsi: 9 | 20:56 |
| thingee | * nexenta-dsvm-volume-ns5_nfs: 7 | 20:56 |
| thingee | * nexenta-dsvm-volume-ns_nfs: 16 | 20:56 |
| thingee | failure count by job: | 20:56 |
| thingee | * nexenta-dsvm-volume-ns5_nfs: 25 | 20:56 |
| thingee | * nexenta-dsvm-volume-ns_nfs: 24 | 20:56 |
| thingee | * nexenta-dsvm-volume-ns5_iscsi: 23 | 20:56 |
| thingee | * nexenta-dsvm-volume-ns_edge: 1 | 20:56 |
| thingee | * nexenta-dsvm-volume-ns_iscsi: 24 | 20:56 |
| *** dims_ has joined #openstack-cinder | 20:56 | |
| *** angela-s has quit IRC | 20:57 | |
| thingee | hodos: as a middle group i think we can look past the issues with ns4 drivers. I would like to see those stabilize during the liberty dev release. | 20:57 |
| thingee | /group/ground/ | 20:57 |
| jgriffith | hodos: thingee AFAIC at this point for systems that haven't done anythign until the last week they missed the boat | 20:58 |
| jgriffith | hodos: thingee see ya when M opens up | 20:58 |
| jgriffith | The amount of time and effort by you thingee at this point is ridiculous IMHO | 20:58 |
| hodos | jgriffith: for NS4 we started earlier than last week | 20:58 |
| jgriffith | There are more important things for you to be workign on | 20:58 |
| jgriffith | hodos: ok | 20:59 |
| thingee | jgriffith: that's why we stop this after the first milestone...though CI monitors never ends :( | 20:59 |
| thingee | monitoring* | 20:59 |
| *** pboldin has joined #openstack-cinder | 20:59 | |
| *** timcl has quit IRC | 21:00 | |
| *** dims has quit IRC | 21:00 | |
| earlephilhower_ | This may be opening a can of worms, but how can some CIs report success on jobs that Jenkins reports as merge conflicts? For example (at random), https://review.openstack.org/#/c/172808/ | 21:00 |
| *** dims_ has quit IRC | 21:00 | |
| hodos | thingee jgriffith: : I understand your point, but my point is that the driver itself is pretty stable, the backend had to have more HW that's it, and I see that NS5 door is closed for L, but please let the NS4 in | 21:01 |
| *** amoturi has joined #openstack-cinder | 21:01 | |
| asselin_ | earlephilhower_, b/c the merge conflict is detected when a patch merges to master that causes a conflict on the current patch set. | 21:01 |
| jgriffith | hodos: don't worry much about what I said, I think I'm the only one that thinks that way :) | 21:02 |
| *** vokt has joined #openstack-cinder | 21:02 | |
| asselin_ | earlephilhower_, so that patch set itself is ok, but would not have a clean rebase | 21:02 |
| dguryanov | Does anybody know, if online volume snapshots are required for volume drivers in this release? | 21:02 |
| asselin_ | earlephilhower_, jenkins tries to tell you asap | 21:02 |
| earlephilhower_ | asselin_: Sure, but how could the git pull/fetch (sorry, not git expert) fail for OpenStack's Jenkins, but be OK for other folks who are using the same base? | 21:02 |
| asselin_ | earlephilhower_, it's a separate job | 21:03 |
| asselin_ | earlephilhower_, take a look here: "merge-check" pipeline http://zuul.openstack.org/ | 21:03 |
| *** Yogi1 has quit IRC | 21:04 | |
| *** xyang1 has joined #openstack-cinder | 21:05 | |
| *** vokt_ has joined #openstack-cinder | 21:05 | |
| openstackgerrit | John Griffith proposed openstack/cinder: SQL scripts should not manage transactions https://review.openstack.org/191956 | 21:09 |
| earlephilhower_ | asselin_: Thanks, but I think maybe what I'm getting at is it seems like some aren't pulling the patches at all in for some reason? See last review by hitachi-hbds-iscsi in https://review.openstack.org/#/c/172808/ . No reference to patch 172808 in devstacklog.txt, only a local 172.24.x private git repo. | 21:10 |
| asselin_ | earlephilhower_, there's no correlation with merge check and ci. To verify what you're looking for is this: | 21:11 |
| asselin_ | http://177.84.241.119:10000/08/172808/9/check/hitachi-hbsd-iscsi/b10ffa9/logs/devstack-gate-setup-workspace-new.txt.gz#_2015-06-15_17_08_30_741 | 21:11 |
| asselin_ | HEAD is now at 2c9a3f3... Cinder part of volume snapshot improvements | 21:12 |
| asselin_ | that's what cinder is set to. | 21:12 |
| earlephilhower_ | asselin_: Thanks! Just trying to figure out how things are working under the hood now that they're working for me (cross fingers)! | 21:13 |
| *** kjnelson has joined #openstack-cinder | 21:14 | |
| *** bswartz has quit IRC | 21:16 | |
| *** Longgeek has quit IRC | 21:22 | |
| *** thangp has quit IRC | 21:22 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Remove logging statements from migrations https://review.openstack.org/191983 | 21:26 |
| *** cbader has quit IRC | 21:29 | |
| asselin_ | earlephilhower_, https://review.openstack.org/#/c/191982/ | 21:29 |
| * earlephilhower_ looks at it now... | 21:33 | |
| *** dims has joined #openstack-cinder | 21:35 | |
| *** e0ne has quit IRC | 21:36 | |
| asselin_ | patrickeast, how soon do you want that fc patch merged? We have an intern and was going to have him ci it...but don't want delay too long. let me know | 21:39 |
| *** jungleboyj has quit IRC | 21:39 | |
| patrickeast | asselin_: no rush, i’ve got it in the mix for my ci so whenever we get it in the repo is fine | 21:39 |
| patrickeast | i’ve still got a manual process to get the script updated, so it doesn’t need to be merged to use | 21:40 |
| patrickeast | for me to use* | 21:40 |
| asselin_ | patrickeast, ok...then I'll leave it a bit...I think it's low hanging fruit for him, and more meaningful if it's not yet merged | 21:40 |
| patrickeast | asselin_: sounds good | 21:40 |
| *** belmoreira has quit IRC | 21:42 | |
| *** Openstack-t has joined #openstack-cinder | 21:47 | |
| *** ianbrown has joined #openstack-cinder | 21:48 | |
| nikeshm | asselin patrickeast: i am able to use two ports of same FC card,one for one CI,one for another CI,but i have to hard-cord fc_pci_device value in job for this | 21:50 |
| nikeshm | hard-code | 21:50 |
| patrickeast | nikeshm: not sure i follow, why do you have to hard code it? | 21:51 |
| nikeshm | setting up another variable in /etc/environment is not working for me | 21:51 |
| asselin_ | nikeshm, using the existing script or modified? | 21:52 |
| patrickeast | nikeshm: if you have multiple devices in the fc_pci_device variable and specify 1 device to pass through it should try them until it finds one that can work | 21:52 |
| patrickeast | nikeshm: so having a variable like: export fc_pci_device='0000:02:00.0 0000:02:00.1 0000:03:00.0 0000:03:00.1' didn’t work for you? | 21:54 |
| patrickeast | actually that reminds me… i think the setup provider script thats checked in is broken | 21:55 |
| patrickeast | https://github.com/stackforge/third-party-ci-tools/blob/master/provisioning_scripts/fibre_channel/setup_fc_for_provider.sh#L36 is wrong | 21:56 |
| patrickeast | asselin_: ^ | 21:56 |
| *** edmondsw has quit IRC | 21:56 | |
| *** chlong has quit IRC | 21:57 | |
| nikeshm | patrickeast: i thought it will be problem,but if we can use all this listed in different individual vms depending on availablity,then its fine | 21:59 |
| patrickeast | nikeshm: it should be fine | 22:00 |
| patrickeast | nikeshm: thats what i do with my ci system | 22:00 |
| patrickeast | nikeshm: the script will try each one, if they are already in use it skips them and tries the next | 22:00 |
| nikeshm | patrickeast: awesome | 22:00 |
| nikeshm | patrickeast : no need to hardcode | 22:00 |
| patrickeast | nikeshm: yep, thats the idea | 22:01 |
| *** eharney has quit IRC | 22:01 | |
| *** boris-42 has joined #openstack-cinder | 22:01 | |
| nikeshm | patrickeast : but i have to add manually in /etc/environment for my environment ubuntu 14.04, echo export $fc_pci_device >> /etc/profile.d/fc_devices.sh is nor working for my environment | 22:02 |
| patrickeast | nikeshm: yea that script has a couple of issues | 22:03 |
| nikeshm | patrickeast: which environment are yu using? | 22:03 |
| patrickeast | first and foremost, its broken as-is… it needs to do something like echo export fc_pci_device=$fc_pci_device | 22:03 |
| patrickeast | secondly its setup for working on centos | 22:04 |
| patrickeast | nikeshm: my systems are running on centos 7, where those profile.d scripts do the right thing | 22:04 |
| patrickeast | nikeshm: but for ubuntu it seems like it needs to be in /etc/environment | 22:04 |
| *** vokt has quit IRC | 22:05 | |
| *** kjnelson has quit IRC | 22:06 | |
| nikeshm | patrickeast: its might be not related,but adding another variable in /etc/environment is not working,means if you run same command via script,it will not get value of variable | 22:06 |
| nikeshm | patrickeast: but printenv is showing another variable value | 22:07 |
| mriedem | hemnafk: there is a lot of duplication in nova's nfs/glusterfs/smbfs volume driver code, i was looking at making the glusterfs/smbfs volume drivers extend LibvirtNFSVolumeDriver so the common bits are not duplicated - i don't see any of that happening in the nova using os-brick wip here: https://review.openstack.org/#/c/175569/ - hemnafk, is that worthwhile for me to poke on outside the os-brick change? | 22:07 |
| patrickeast | nikeshm: hmm, i dunno why that would be happening | 22:08 |
| *** dannywilson has quit IRC | 22:09 | |
| *** mriedem is now known as mriedem_away | 22:09 | |
| asselin_ | nikeshm, did you select which variable to use in he script? | 22:17 |
| asselin_ | export FC_PCI_VAR_NAME=your-var-name | 22:17 |
| nikeshm | asselin_ : yes | 22:23 |
| *** rushil has quit IRC | 22:24 | |
| *** rushil has joined #openstack-cinder | 22:26 | |
| *** david-lyle has quit IRC | 22:31 | |
| *** dims has quit IRC | 22:34 | |
| *** lcurtis has quit IRC | 22:36 | |
| *** oswa has quit IRC | 22:38 | |
| rlucio | patrickeast: hey man, just curious, when you switched to doing the full cinder+nova set of 280 tests or whatever, did you by chance have to alter any nova config to get everything to pass? | 22:44 |
| rlucio | patrickeast: for instance, i changed virt_type=qemu, but was wondering if any other obvious config changes i missed | 22:45 |
| patrickeast | rlucio: mmm i think the virt type was the only one i changed | 22:45 |
| patrickeast | one sec, i can check | 22:46 |
| jgriffith | nikeshm: ping | 22:46 |
| patrickeast | rlucio: yea just that one | 22:46 |
| rlucio | patrickeast: ok, thanks for looking | 22:46 |
| nikeshm | jgriffith : hi | 22:47 |
| jgriffith | nikeshm: hey there | 22:47 |
| patrickeast | rlucio: we did however start getting ssh timeouts on a few tests intermittently | 22:47 |
| jgriffith | nikeshm: I was just reviewing your dothill drivers | 22:47 |
| nikeshm | jgriffith: yes | 22:47 |
| jgriffith | nikeshm: I have one minor complaint: https://review.openstack.org/#/c/189367/7/cinder/volume/drivers/dothill/dothill_common.py | 22:47 |
| jgriffith | Line 411 | 22:48 |
| jgriffith | nikeshm: it will be rather annoying to have stack-traces for the delete failure, especially when it may be for the "not found" case that just gets ignored | 22:48 |
| rlucio | patrickeast: ah! yes i'm seeing something like that, anything you did to dance around that? | 22:48 |
| *** rushil has quit IRC | 22:49 | |
| patrickeast | rlucio: unfortunately it remains a mystery :( | 22:49 |
| jgriffith | nikeshm: you could either catch that in the caller, or just not raise for that case | 22:49 |
| *** gouthamr has joined #openstack-cinder | 22:49 | |
| jgriffith | nikeshm: if you were to change that I'd +2/A your patch :) | 22:49 |
| patrickeast | rlucio: i tried playing with a whole bunch of timeout settings | 22:49 |
| patrickeast | rlucio: but it kind of looks like it either connects in a reasonable amount of time, or never does (i tried a like 6 hour timout and still nothing) | 22:49 |
| rlucio | patrickeast: weird... | 22:50 |
| patrickeast | rlucio: the strange part is that it seems to happen when our system is under load… so we are suspecting there is maybe some kind of network problem when we stress the software switching stuff too much | 22:50 |
| patrickeast | rlucio: i limited the number of build slaves per server and it helped quite a bit to improve things overall | 22:51 |
| patrickeast | but we still have problems | 22:51 |
| patrickeast | rlucio: one of the guys on my team (hes not on irc right now) is going to be diving into it again over the next week or two to try and figure out whats going on/how to prevent the issues | 22:52 |
| openstackgerrit | John Griffith proposed openstack/cinder: Remove logging statements from migrations https://review.openstack.org/191983 | 22:53 |
| rlucio | patrickeast: interesting, i'll keep an eye on system resources, maybe I can make fewer, more powerful CI test VMs, see if that helps | 22:53 |
| *** diogogmt has quit IRC | 22:53 | |
| patrickeast | rlucio: if you find a winning combo let me know | 22:53 |
| rlucio | patrickeast: sure thing | 22:54 |
| *** jungleboyj has joined #openstack-cinder | 22:54 | |
| nikeshm | jgriffith: can i add a bug for these to resolve this in L-2? | 22:58 |
| *** alexpilotti has joined #openstack-cinder | 23:01 | |
| *** oswa has joined #openstack-cinder | 23:02 | |
| jgriffith | nikeshm: if you just address the one issue regarding the exception right now I'll merge the code | 23:02 |
| jgriffith | nikeshm: I'll ignore the urllib AGAIN | 23:02 |
| jgriffith | nikeshm: even though you received a -1 post merge the last time and said you'd file a bug and fix it | 23:02 |
| *** dims has joined #openstack-cinder | 23:03 | |
| jgriffith | nikeshm: othewise, someobdy else can merge your patch :) | 23:03 |
| jgriffith | nikeshm: I won't block it | 23:03 |
| *** alexpilotti has quit IRC | 23:03 | |
| *** dims has quit IRC | 23:03 | |
| *** dims has joined #openstack-cinder | 23:04 | |
| *** yamada-h has joined #openstack-cinder | 23:05 | |
| *** jungleboyj has quit IRC | 23:05 | |
| *** oswa has quit IRC | 23:06 | |
| *** yamada-h has quit IRC | 23:10 | |
| *** oswa has joined #openstack-cinder | 23:11 | |
| *** gouthamr has quit IRC | 23:11 | |
| *** david-lyle has joined #openstack-cinder | 23:11 | |
| *** oswa has quit IRC | 23:13 | |
| nikeshm | jgriffith : is this ok http://paste.openstack.org/show/294567/ | 23:16 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 23:17 |
| *** ganso_ has quit IRC | 23:18 | |
| *** Longgeek has joined #openstack-cinder | 23:19 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: RPC and object versioning https://review.openstack.org/184404 | 23:20 |
| nikeshm | jgriffith: actually this is problem with array,thats why LOG.exception in both case | 23:22 |
| nikeshm | jgriffith: if we create a snaphot and then instantly say delete snapshot,then only we face this issue | 23:23 |
| jgriffith | nikeshm: so what I was getting at is that it's not "unusual" to have a create fail on the backend, but you need to run the delete from Cinder's perspective | 23:24 |
| jgriffith | nikeshm: and that having exception traces in the logs would be kind of annoying | 23:24 |
| *** EmilienM|afk is now known as EmilienM | 23:24 | |
| jgriffith | nikeshm: your pasted code is what I was looking for | 23:25 |
| jgriffith | nikeshm: but if you're not comfortable with that you should leave whatyou have | 23:25 |
| nikeshm | jgriffith: sure i will upload a new patch now like yu said | 23:26 |
| nikeshm | jgriffith: i have to upload new patches for hpmsa and lenovo too | 23:27 |
| nikeshm | jgriffith: can yu comment on patch for this | 23:28 |
| *** diogogmt has joined #openstack-cinder | 23:30 | |
| *** diemt has quit IRC | 23:30 | |
| *** diemt has joined #openstack-cinder | 23:31 | |
| *** Longgeek has quit IRC | 23:35 | |
| *** IlyaG has joined #openstack-cinder | 23:39 | |
| openstackgerrit | Ryan Lucio proposed openstack/cinder: Fix export verify routines to use proper state nodes https://review.openstack.org/165655 | 23:39 |
| *** daneyon_ has quit IRC | 23:43 | |
| *** chlong has joined #openstack-cinder | 23:47 | |
| *** garthb_ has quit IRC | 23:48 | |
| *** garthb has quit IRC | 23:48 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!