*** zhurong has joined #openstack-cinder | 00:03 | |
openstackgerrit | Kien Ha proposed openstack/cinder master: Update test cases https://review.openstack.org/537017 | 00:05 |
---|---|---|
*** owalsh_ has joined #openstack-cinder | 00:05 | |
*** owalsh has quit IRC | 00:06 | |
*** owalsh_ has quit IRC | 00:11 | |
openstackgerrit | lihaijing proposed openstack/cinder master: Fix cinder quota-usage error https://review.openstack.org/535167 | 00:12 |
*** owalsh has joined #openstack-cinder | 00:16 | |
*** zhurong has quit IRC | 00:20 | |
*** yangyapeng has quit IRC | 00:25 | |
*** yangyapeng has joined #openstack-cinder | 00:25 | |
*** wanghao has quit IRC | 00:28 | |
*** wanghao has joined #openstack-cinder | 00:29 | |
*** yangyapeng has quit IRC | 00:30 | |
openstackgerrit | Kien Ha proposed openstack/cinder master: Update python version number to 3.6 for test case https://review.openstack.org/537021 | 00:33 |
openstackgerrit | Kien Ha proposed openstack/cinder master: Update python version number to 3.6 for test case https://review.openstack.org/537021 | 00:35 |
*** Dinesh_Bhor has joined #openstack-cinder | 00:38 | |
*** Brin has joined #openstack-cinder | 00:44 | |
*** Dinesh_Bhor has quit IRC | 00:49 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/537031 | 00:50 |
*** Dinesh_Bhor has joined #openstack-cinder | 00:50 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder-tempest-plugin master: Updated from global requirements https://review.openstack.org/533957 | 00:50 |
*** wanghao_ has joined #openstack-cinder | 00:55 | |
*** wanghao has quit IRC | 00:55 | |
*** zhurong has joined #openstack-cinder | 00:58 | |
*** nicolasbock has quit IRC | 00:58 | |
*** kien-ha has quit IRC | 01:01 | |
*** r-daneel has quit IRC | 01:01 | |
*** wanghao has joined #openstack-cinder | 01:07 | |
*** zhaochao has joined #openstack-cinder | 01:10 | |
*** wanghao_ has quit IRC | 01:11 | |
*** stakeda has joined #openstack-cinder | 01:14 | |
*** crushil has joined #openstack-cinder | 01:15 | |
*** Dinesh_Bhor has quit IRC | 01:15 | |
*** salv-orlando has joined #openstack-cinder | 01:15 | |
*** yangyapeng has joined #openstack-cinder | 01:17 | |
openstackgerrit | lihaijing proposed openstack/cinder master: Fix cinder quota-usage error https://review.openstack.org/535167 | 01:18 |
*** Dinesh_Bhor has joined #openstack-cinder | 01:19 | |
*** salv-orlando has quit IRC | 01:20 | |
*** david-lyle has quit IRC | 01:22 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/537135 | 01:23 |
*** yangyapeng has quit IRC | 01:26 | |
*** yangyapeng has joined #openstack-cinder | 01:26 | |
*** dobson has quit IRC | 01:27 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext master: Updated from global requirements https://review.openstack.org/535113 | 01:27 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/537158 | 01:27 |
*** dobson has joined #openstack-cinder | 01:31 | |
*** dalgaaf has quit IRC | 01:41 | |
*** dalgaaf has joined #openstack-cinder | 01:42 | |
*** s-shiono has joined #openstack-cinder | 01:48 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder-tempest-plugin master: Updated from global requirements https://review.openstack.org/533957 | 01:50 |
*** markstur has quit IRC | 02:00 | |
*** dikonoor has joined #openstack-cinder | 02:03 | |
*** gcb has joined #openstack-cinder | 02:09 | |
*** zhaochao has quit IRC | 02:11 | |
*** Dinesh_Bhor has quit IRC | 02:15 | |
*** Dinesh__Bhor has joined #openstack-cinder | 02:15 | |
*** salv-orlando has joined #openstack-cinder | 02:17 | |
*** harlowja has quit IRC | 02:20 | |
*** salv-orlando has quit IRC | 02:21 | |
*** dikonoo has joined #openstack-cinder | 02:22 | |
*** dikonoor has quit IRC | 02:22 | |
*** zhaochao has joined #openstack-cinder | 02:25 | |
*** kaisers__ has joined #openstack-cinder | 02:25 | |
*** kaisers_ has quit IRC | 02:29 | |
*** wxy has joined #openstack-cinder | 02:33 | |
openstackgerrit | liuyamin proposed openstack/cinder master: Use constants for cinder-backup and cinder-schedule https://review.openstack.org/534181 | 02:38 |
openstackgerrit | Merged openstack/cinder master: Add contributor notes on how to use pdb with tests https://review.openstack.org/533813 | 02:40 |
*** zhurong has quit IRC | 02:57 | |
*** crushil has quit IRC | 03:09 | |
*** salv-orlando has joined #openstack-cinder | 03:17 | |
*** salv-orlando has quit IRC | 03:22 | |
*** ykarel has joined #openstack-cinder | 03:23 | |
*** david-lyle has joined #openstack-cinder | 03:31 | |
*** david-lyle has quit IRC | 03:32 | |
*** david-lyle has joined #openstack-cinder | 03:33 | |
*** david-lyle has quit IRC | 03:39 | |
*** felipemonteiro_ has joined #openstack-cinder | 03:41 | |
*** felipemonteiro__ has joined #openstack-cinder | 03:42 | |
*** felipemonteiro_ has quit IRC | 03:45 | |
*** abhishekk has joined #openstack-cinder | 03:47 | |
*** ganso has quit IRC | 03:48 | |
*** psachin` has joined #openstack-cinder | 03:52 | |
*** psachin has joined #openstack-cinder | 03:52 | |
*** markstur has joined #openstack-cinder | 03:55 | |
*** liverpooler has quit IRC | 04:11 | |
*** salv-orlando has joined #openstack-cinder | 04:18 | |
*** salv-orlando has quit IRC | 04:22 | |
*** dave-mccowan has quit IRC | 04:27 | |
*** markstur has quit IRC | 04:27 | |
*** markstur has joined #openstack-cinder | 04:27 | |
*** felipemonteiro__ has quit IRC | 04:31 | |
*** markstur has quit IRC | 04:32 | |
*** Dinesh__Bhor has quit IRC | 04:40 | |
*** Dinesh__Bhor has joined #openstack-cinder | 04:40 | |
*** masber has joined #openstack-cinder | 04:42 | |
*** dikonoo has quit IRC | 05:12 | |
*** salv-orlando has joined #openstack-cinder | 05:19 | |
*** harlowja has joined #openstack-cinder | 05:19 | |
openstackgerrit | Merged openstack/cinder master: Run doc8 first https://review.openstack.org/533814 | 05:19 |
*** salv-orlando has quit IRC | 05:23 | |
*** lkuchlan has joined #openstack-cinder | 05:27 | |
*** dikonoor has joined #openstack-cinder | 05:29 | |
*** links has joined #openstack-cinder | 05:32 | |
openstackgerrit | Gao Fei proposed openstack/cinder-specs master: Replace Chinese punctuation with English punctuation https://review.openstack.org/537258 | 05:33 |
*** lkuchlan has quit IRC | 05:33 | |
*** wanghao_ has joined #openstack-cinder | 05:37 | |
*** wanghao has quit IRC | 05:39 | |
*** salv-orlando has joined #openstack-cinder | 05:45 | |
*** lkuchlan has joined #openstack-cinder | 05:49 | |
*** dikonoor has quit IRC | 05:51 | |
*** pgadiya has joined #openstack-cinder | 05:51 | |
*** armaan has quit IRC | 05:51 | |
*** dikonoor has joined #openstack-cinder | 05:52 | |
*** armaan has joined #openstack-cinder | 05:59 | |
*** Dinesh__Bhor has quit IRC | 05:59 | |
*** kaisers__ has quit IRC | 06:02 | |
*** rcernin_ has joined #openstack-cinder | 06:03 | |
*** rcernin has quit IRC | 06:03 | |
*** kaisers_ has joined #openstack-cinder | 06:05 | |
*** lkuchlan has quit IRC | 06:05 | |
*** dikonoor has quit IRC | 06:06 | |
*** dikonoor has joined #openstack-cinder | 06:06 | |
*** Dinesh__Bhor has joined #openstack-cinder | 06:08 | |
*** kaisers_ has quit IRC | 06:10 | |
*** dikonoor has quit IRC | 06:15 | |
*** lkuchlan has joined #openstack-cinder | 06:17 | |
*** kaisers_ has joined #openstack-cinder | 06:21 | |
*** gcb has quit IRC | 06:22 | |
*** kaisers_ has quit IRC | 06:26 | |
*** zhangbailin_ has joined #openstack-cinder | 06:28 | |
*** dikonoor has joined #openstack-cinder | 06:30 | |
*** Brin has quit IRC | 06:31 | |
*** kaisers_ has joined #openstack-cinder | 06:37 | |
*** threestrands has joined #openstack-cinder | 06:39 | |
*** threestrands has joined #openstack-cinder | 06:39 | |
*** Dinesh__Bhor has quit IRC | 06:46 | |
*** threestrands has quit IRC | 06:49 | |
*** Dinesh__Bhor has joined #openstack-cinder | 06:50 | |
*** Dinesh__Bhor has quit IRC | 06:52 | |
*** Dinesh__Bhor has joined #openstack-cinder | 06:52 | |
*** ykarel is now known as ykarel|away | 07:09 | |
*** pcaruana has joined #openstack-cinder | 07:10 | |
*** ykarel|away has quit IRC | 07:11 | |
*** rcernin has joined #openstack-cinder | 07:12 | |
*** rcernin_ has quit IRC | 07:12 | |
*** Dinesh__Bhor has quit IRC | 07:17 | |
*** pcaruana has quit IRC | 07:21 | |
*** pcaruana has joined #openstack-cinder | 07:22 | |
*** markvoelker has quit IRC | 07:25 | |
*** lkuchlan has quit IRC | 07:25 | |
openstackgerrit | yixuan zhang proposed openstack/cinder master: Storwize: avoid ascii code error for ssh response stderr https://review.openstack.org/537286 | 07:26 |
*** lkuchlan has joined #openstack-cinder | 07:26 | |
*** markvoelker has joined #openstack-cinder | 07:27 | |
*** jdurgin has quit IRC | 07:28 | |
*** e0ne has joined #openstack-cinder | 07:30 | |
*** markvoelker has quit IRC | 07:32 | |
openstackgerrit | yixuan zhang proposed openstack/cinder master: Storwize: avoid ascii code error for ssh response stderr https://review.openstack.org/537286 | 07:35 |
*** armaan has quit IRC | 07:41 | |
*** armaan has joined #openstack-cinder | 07:41 | |
*** lkuchlan has quit IRC | 07:44 | |
*** lkuchlan has joined #openstack-cinder | 07:44 | |
*** jdurgin has joined #openstack-cinder | 07:45 | |
*** itlinux has joined #openstack-cinder | 07:56 | |
*** e0ne has quit IRC | 08:01 | |
*** pcaruana has quit IRC | 08:01 | |
*** e0ne has joined #openstack-cinder | 08:01 | |
*** alexchadin has joined #openstack-cinder | 08:10 | |
*** AlexeyAbashkin has joined #openstack-cinder | 08:23 | |
*** itlinux has quit IRC | 08:24 | |
*** lpetrut has joined #openstack-cinder | 08:25 | |
*** tesseract has joined #openstack-cinder | 08:27 | |
*** alexchadin has quit IRC | 08:28 | |
*** alexchadin has joined #openstack-cinder | 08:29 | |
*** lpetrut has quit IRC | 08:32 | |
openstackgerrit | wanghao proposed openstack/cinder master: Transfer snapshots with volumes https://review.openstack.org/533564 | 08:34 |
*** alexchadin has quit IRC | 08:36 | |
*** alexchadin has joined #openstack-cinder | 08:36 | |
*** lpetrut has joined #openstack-cinder | 08:38 | |
*** pcaruana has joined #openstack-cinder | 08:39 | |
*** alexchad_ has joined #openstack-cinder | 08:40 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: [1/2] Fix mock for sleep mocks in volume drivers tests https://review.openstack.org/531363 | 08:41 |
*** diegows has quit IRC | 08:41 | |
*** dikonoo has joined #openstack-cinder | 08:41 | |
*** dikonoor has quit IRC | 08:42 | |
*** alexchadin has quit IRC | 08:42 | |
*** rcernin has quit IRC | 08:44 | |
openstackgerrit | wanghao proposed openstack/cinder master: Transfer snapshots with volumes https://review.openstack.org/533564 | 08:44 |
*** s-shiono has quit IRC | 08:45 | |
wanghao_ | eharney: hi, Eric, are you around? | 08:45 |
wanghao_ | eharney: need your help to review this patch: Transfer snapshots with volumes https://review.openstack.org/533564. I have addressed your concern and need to check if we can remove the -2 and push this forward this week. | 08:46 |
*** diegows has joined #openstack-cinder | 08:46 | |
*** kaisers_ has quit IRC | 08:48 | |
*** kaisers_ has joined #openstack-cinder | 08:48 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: [2/2] Fix mock for sleep mocks in volume drivers tests https://review.openstack.org/537303 | 08:49 |
*** tesseract has quit IRC | 08:54 | |
openstackgerrit | wanghao proposed openstack/cinder master: Support to query volume filter by changes since/before https://review.openstack.org/537305 | 08:58 |
*** tesseract has joined #openstack-cinder | 09:03 | |
*** harlowja has quit IRC | 09:07 | |
*** freerunner has quit IRC | 09:08 | |
*** lpetrut_ has joined #openstack-cinder | 09:08 | |
*** gkadam has joined #openstack-cinder | 09:10 | |
*** lpetrut has quit IRC | 09:10 | |
*** itlinux has joined #openstack-cinder | 09:12 | |
*** freerunner has joined #openstack-cinder | 09:14 | |
*** kaisers_ has quit IRC | 09:15 | |
*** alexchad_ is now known as alexchadin | 09:15 | |
*** kaisers_ has joined #openstack-cinder | 09:15 | |
*** kaisers_ has quit IRC | 09:20 | |
*** lkuchlan has quit IRC | 09:24 | |
*** psachin` has quit IRC | 09:24 | |
*** lpetrut_ has quit IRC | 09:25 | |
*** psachin has quit IRC | 09:25 | |
*** lkuchlan has joined #openstack-cinder | 09:25 | |
openstackgerrit | liuyamin proposed openstack/cinder master: Use constants for cinder-backup and cinder-schedule https://review.openstack.org/534181 | 09:27 |
*** markvoelker has joined #openstack-cinder | 09:28 | |
*** wanghao_ has quit IRC | 09:30 | |
*** lkuchlan has quit IRC | 09:30 | |
*** wanghao has joined #openstack-cinder | 09:30 | |
*** lkuchlan has joined #openstack-cinder | 09:30 | |
*** wanghao has quit IRC | 09:31 | |
*** lkuchlan has quit IRC | 09:31 | |
*** wanghao has joined #openstack-cinder | 09:31 | |
*** lkuchlan has joined #openstack-cinder | 09:31 | |
*** wanghao has quit IRC | 09:31 | |
*** wanghao has joined #openstack-cinder | 09:32 | |
*** chrisyang_0660 has joined #openstack-cinder | 09:32 | |
*** wanghao has quit IRC | 09:32 | |
*** wanghao has joined #openstack-cinder | 09:33 | |
*** psachin has joined #openstack-cinder | 09:33 | |
*** wanghao has quit IRC | 09:33 | |
*** wanghao has joined #openstack-cinder | 09:33 | |
*** wanghao has quit IRC | 09:34 | |
*** wanghao has joined #openstack-cinder | 09:34 | |
*** wanghao has quit IRC | 09:35 | |
*** lkuchlan has quit IRC | 09:35 | |
*** wanghao has joined #openstack-cinder | 09:35 | |
*** lkuchlan has joined #openstack-cinder | 09:35 | |
*** wanghao has quit IRC | 09:35 | |
*** wanghao has joined #openstack-cinder | 09:36 | |
*** wanghao has quit IRC | 09:36 | |
*** wanghao has joined #openstack-cinder | 09:36 | |
*** wanghao has quit IRC | 09:37 | |
*** wanghao has joined #openstack-cinder | 09:37 | |
*** wanghao has quit IRC | 09:38 | |
*** wanghao has joined #openstack-cinder | 09:38 | |
*** stakeda has quit IRC | 09:39 | |
*** kaisers_ has joined #openstack-cinder | 09:46 | |
*** yangyapeng has quit IRC | 09:50 | |
Roamer` | hm, can somebody take a look at http://logs.openstack.org/33/140733/19/check/tempest-full/efcf44f/job-output.txt.gz#_2018-01-24_07_33_00_269858 - tempest.api.volume.admin.test_volumes_backup.VolumesBackupsAdminTest.test_volume_backup_reset_status failed twice in a row for a completely unrelated Nova change | 09:50 |
*** kaisers_ has quit IRC | 09:50 | |
*** yangyapeng has joined #openstack-cinder | 09:51 | |
*** yangyapeng has quit IRC | 09:55 | |
*** kaisers_ has joined #openstack-cinder | 09:59 | |
*** markvoelker has quit IRC | 10:02 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: Fix mocks for ZFS Storage Appliance unit tests https://review.openstack.org/537332 | 10:04 |
*** jcosmao has left #openstack-cinder | 10:14 | |
*** jcosmao has joined #openstack-cinder | 10:17 | |
*** lkuchlan has quit IRC | 10:20 | |
*** dikonoo has quit IRC | 10:21 | |
*** pgadiya has quit IRC | 10:22 | |
*** pgadiya has joined #openstack-cinder | 10:23 | |
*** zhangbailin__ has joined #openstack-cinder | 10:26 | |
*** zhangbailin_ has quit IRC | 10:29 | |
*** dcdamien has joined #openstack-cinder | 10:33 | |
*** dikonoo has joined #openstack-cinder | 10:33 | |
*** abhishekk has quit IRC | 10:34 | |
*** rcernin has joined #openstack-cinder | 10:34 | |
*** yangyapeng has joined #openstack-cinder | 10:39 | |
*** dikonoo has quit IRC | 10:48 | |
*** adisky__ has joined #openstack-cinder | 10:52 | |
*** AlexeyAbashkin has quit IRC | 10:53 | |
*** AlexeyAbashkin has joined #openstack-cinder | 10:54 | |
*** alexchadin has quit IRC | 10:56 | |
*** yangyapeng has quit IRC | 10:58 | |
*** markvoelker has joined #openstack-cinder | 10:58 | |
chrisyang_0660 | Hello guys, I have a question about the volume driver. Please give me some advices. | 10:59 |
chrisyang_0660 | As we know, vendors keep fixing bugs and adding features to their drivers. | 10:59 |
chrisyang_0660 | But there are some patches are not backward-compatible. | 10:59 |
chrisyang_0660 | For example, https://review.openstack.org/#/c/453654/ | 10:59 |
chrisyang_0660 | "configuration.SHARED_CONF_GROUP" is not supported in the Ocata version. | 10:59 |
chrisyang_0660 | If users try to use the newer version of driver which contains some backward-incompatible patches in their stable version of cinder, driver may not work as expected. | 10:59 |
chrisyang_0660 | So, if vendors want to ensure backward-compatibility, how should they do? | 10:59 |
chrisyang_0660 | (1)Also contribute patches to the older (stable) cinder branch? | 10:59 |
chrisyang_0660 | (2)Handle situations in the driver code depends on different cinder versions. | 10:59 |
chrisyang_0660 | (3)Just ask users upgrade to compatible cinder version ? | 10:59 |
chrisyang_0660 | (4)Some other way? | 10:59 |
*** yangyapeng has joined #openstack-cinder | 10:59 | |
*** alexchadin has joined #openstack-cinder | 10:59 | |
*** ganso has joined #openstack-cinder | 11:03 | |
*** gaoyan has joined #openstack-cinder | 11:04 | |
*** gaoyan has quit IRC | 11:06 | |
*** gaoyan_ has joined #openstack-cinder | 11:06 | |
e0ne | chrisyang_0660: hi. #3 is the best one. we don't allow to backport features to the stable branches | 11:17 |
e0ne | chrisyang_0660: btw, it's up to distros vendors to backport new drivers to older version on their own risk | 11:18 |
*** sapd_ has quit IRC | 11:19 | |
*** sapd_ has joined #openstack-cinder | 11:19 | |
*** psachin has quit IRC | 11:19 | |
*** itlinux has quit IRC | 11:23 | |
*** gaoyan_ has quit IRC | 11:23 | |
*** gaoyan_ has joined #openstack-cinder | 11:23 | |
*** armaan has quit IRC | 11:23 | |
Roamer` | e0ne, I suspect chrisyang_0660 is not even talking about a distro, but a real hardware/software vendor that has written their own OpenStack driver and has possibly deployed it in customer installations | 11:24 |
*** armaan has joined #openstack-cinder | 11:24 | |
*** markvoelker has quit IRC | 11:32 | |
*** itlinux has joined #openstack-cinder | 11:38 | |
*** mvenesio has joined #openstack-cinder | 11:45 | |
chrisyang_0660 | eOne: Do you mean that vendors should always only fix bugs in the stable branches, and new features are only allowed in the next stable branch? | 11:46 |
*** zhangbailin__ has quit IRC | 11:54 | |
*** zhangbailin__ has joined #openstack-cinder | 11:55 | |
*** pgadiya has quit IRC | 12:02 | |
*** dave-mccowan has joined #openstack-cinder | 12:09 | |
*** zhurong has joined #openstack-cinder | 12:10 | |
*** armaan has quit IRC | 12:13 | |
*** pgadiya has joined #openstack-cinder | 12:14 | |
*** mvk has quit IRC | 12:19 | |
*** pgadiya_ has joined #openstack-cinder | 12:19 | |
*** pgadiya has quit IRC | 12:23 | |
*** lpetrut_ has joined #openstack-cinder | 12:29 | |
*** markvoelker has joined #openstack-cinder | 12:29 | |
*** mvk has joined #openstack-cinder | 12:31 | |
*** dcdamien has quit IRC | 12:34 | |
*** dcdamien has joined #openstack-cinder | 12:36 | |
*** stvnoyes has joined #openstack-cinder | 12:36 | |
*** dcdamien has quit IRC | 12:42 | |
*** dcdamien has joined #openstack-cinder | 12:42 | |
*** tpsilva has joined #openstack-cinder | 12:45 | |
Roamer` | ok, this is stupid... the same problem - test_volume_backup_reset_status failing to create a backup - failed on a different job this time, not tempest-full, but one of the legacy cells jobs... This time tempest-full passed, so this is apparently a transient problem | 12:52 |
*** mvk has quit IRC | 12:56 | |
*** dcdamien has quit IRC | 12:57 | |
*** dcdamien has joined #openstack-cinder | 12:58 | |
*** markvoelker has quit IRC | 13:03 | |
*** abishop has joined #openstack-cinder | 13:04 | |
*** lpetrut_ has quit IRC | 13:05 | |
*** lpetrut_ has joined #openstack-cinder | 13:06 | |
*** abishop has quit IRC | 13:06 | |
*** abishop has joined #openstack-cinder | 13:06 | |
zhangbailin__ | Is it possible to sync the latest version of the patch to the lower version? Some colleagues today asked if the latest version of the patch could be updated to a lower version. | 13:06 |
*** lpetrut_ has quit IRC | 13:07 | |
*** lpetrut_ has joined #openstack-cinder | 13:08 | |
zhangbailin__ | Because most users are still using stable versions, such as Ocata and Pike, if the problem found in Queens and the solution exist in previous releases, is it possible to update? | 13:08 |
zhangbailin__ | this is my idea.:) | 13:09 |
Roamer` | zhangbailin__, this is generally done by checking out the stable/pike branch, making the same changes (or cherry-picking the patch from the master branch), then pushing a new change with git-review | 13:09 |
Roamer` | Gerrit will figure out that this is a different change, even though it has the same change-id, because it's based on a different branch (stable/pike, not master) | 13:09 |
*** lpetrut_ has quit IRC | 13:09 | |
Roamer` | but you have to make sure first that the patch actually works for the older release, and (most probably) change it a bit so that it does, since some things have changed between releases | 13:10 |
*** lpetrut_ has joined #openstack-cinder | 13:10 | |
*** lpetrut_ has quit IRC | 13:11 | |
Roamer` | and if you want the patch to be accepted, it will have to be accepted in the master branch first, and only then for the older releases, and only if the releases still accept that kind of patches (some are in maintenance mode where only critical bugfixes are accepted, and some are out of maintenance, so that nothing is officially changed there) | 13:11 |
*** lpetrut_ has joined #openstack-cinder | 13:11 | |
zhangbailin__ | Yeah, there is a need for a way to test for differences between two large versions.If we can find this way, and curing this update and change, the maintenance of subsequent versions is a big improvement. | 13:13 |
Roamer` | or maybe I misunderstood exactly what you meant... that happens to me sometimes | 13:13 |
zhangbailin__ | Roamer`: I agree with you that there really will be a big problem. | 13:14 |
*** liverpooler has joined #openstack-cinder | 13:20 | |
zhangbailin__ | but still hope that the follow-up version can add a mechanism for automatic detection and synchronization. | 13:21 |
*** edmondsw has joined #openstack-cinder | 13:21 | |
*** edmondsw_ has joined #openstack-cinder | 13:22 | |
openstackgerrit | Erlon R. Cruz proposed openstack/cinder master: Overprovisioning Improvements https://review.openstack.org/534854 | 13:24 |
openstackgerrit | Erlon R. Cruz proposed openstack/cinder master: Netapp Ontap: Adds support for auto-max-over-subscription https://review.openstack.org/534855 | 13:24 |
*** pgadiya_ has quit IRC | 13:25 | |
*** zhangbailin__ has quit IRC | 13:26 | |
*** edmondsw has quit IRC | 13:26 | |
openstackgerrit | Erlon R. Cruz proposed openstack/cinder master: Overprovisioning Improvements https://review.openstack.org/534854 | 13:28 |
openstackgerrit | Erlon R. Cruz proposed openstack/cinder master: Netapp Ontap: Adds support for auto-max-over-subscription https://review.openstack.org/534855 | 13:28 |
*** lpetrut_ has quit IRC | 13:30 | |
*** alexchadin has quit IRC | 13:31 | |
*** lhx_ has joined #openstack-cinder | 13:33 | |
*** alexchadin has joined #openstack-cinder | 13:40 | |
*** alexchadin has quit IRC | 13:41 | |
*** pchavva has joined #openstack-cinder | 13:46 | |
*** AlexeyAbashkin has quit IRC | 13:47 | |
*** rcernin has quit IRC | 13:47 | |
*** daidv has quit IRC | 13:50 | |
*** mvk has joined #openstack-cinder | 13:56 | |
*** alexchadin has joined #openstack-cinder | 13:57 | |
openstackgerrit | Hamdy Khader proposed openstack/cinder master: Deprecate Cinder target_protocol values (iscsi, iser) https://review.openstack.org/537390 | 13:58 |
*** eharney has quit IRC | 13:58 | |
*** lpetrut_ has joined #openstack-cinder | 13:59 | |
*** markvoelker has joined #openstack-cinder | 14:00 | |
*** AlexeyAbashkin has joined #openstack-cinder | 14:00 | |
mriedem | ildikov: jgriffith: the novaclient 2.60 change merged yesterday, will be in the final release for queens this week | 14:01 |
*** mchlumsky has joined #openstack-cinder | 14:01 | |
ildikov | mriedem: I saw it this morning | 14:01 |
ildikov | mriedem: thank you! | 14:02 |
ildikov | mriedem: the cinderclient patch is landed too for the Cinder microversion (3.50) that's needed for the volume_type/multiattach changes | 14:02 |
mriedem | ok | 14:03 |
ildikov | mriedem: I will look into adding a test to that bug fix patch and look into the other bug when I can finally spend more than 5 minutes with my laptop, prolly tomorrow | 14:03 |
*** kaisers_ has quit IRC | 14:07 | |
*** mchlumsky has quit IRC | 14:09 | |
*** mchlumsky has joined #openstack-cinder | 14:11 | |
*** sapd has joined #openstack-cinder | 14:13 | |
jgriffith | mriedem: perfect, tahnks! | 14:14 |
*** mchlumsky has quit IRC | 14:14 | |
*** links has quit IRC | 14:15 | |
*** mchlumsky has joined #openstack-cinder | 14:15 | |
*** zhurong has quit IRC | 14:18 | |
*** markvoelker has quit IRC | 14:27 | |
jgriffith | mriedem: BTW, you know anything about this: http://paste.openstack.org/show/652153/ | 14:27 |
*** markvoelker has joined #openstack-cinder | 14:27 | |
jgriffith | mriedem: after stack.sh unable to boot nova servers; *usually* clears up after a while, but this time it's not | 14:27 |
jgriffith | could've sworn I opened a bug on this the other week, but not finding it | 14:27 |
*** crushil has joined #openstack-cinder | 14:27 | |
*** kaisers_ has joined #openstack-cinder | 14:28 | |
jgriffith | in fact, just about every bug I've ever logged against Nova is closed as "opinion" or "won't fix" now that I search on thing a bit | 14:28 |
mriedem | jgriffith: haven't seen that | 14:28 |
jgriffith | mriedem: hmm | 14:29 |
jgriffith | ok | 14:29 |
mriedem | ubuntu 16.04 LTS? | 14:29 |
jgriffith | Yes | 14:29 |
jgriffith | 16.04.3 to be precise | 14:29 |
jgriffith | It does seem like maybe running stack.sh additional times on a system is the culprit, but haven't verified | 14:31 |
jgriffith | I'll confirm if that's true or not | 14:31 |
*** AlexeyAbashkin has quit IRC | 14:33 | |
*** crushil has quit IRC | 14:42 | |
*** eharney has joined #openstack-cinder | 14:44 | |
*** zhaochao has quit IRC | 14:45 | |
*** armaan has joined #openstack-cinder | 14:47 | |
*** dcdamien has quit IRC | 14:52 | |
*** dcdamien has joined #openstack-cinder | 14:52 | |
*** markvoelker has quit IRC | 14:55 | |
*** markvoelker has joined #openstack-cinder | 14:59 | |
*** AlexeyAbashkin has joined #openstack-cinder | 15:00 | |
jgriffith | mriedem: Looks like it may be related to ./unstack.sh; stack.sh sequence. If I see it again I'll see if I can gather enough enough for a reasonable bug report | 15:01 |
*** dcdamien has quit IRC | 15:02 | |
*** dcdamien has joined #openstack-cinder | 15:02 | |
jgriffith | ildikov: what am I missing here: ERROR (Conflict): Volume b051181d-fa54-47ba-82a8-1096261714fb has 'multiattach' set, which is not supported for this instance. | 15:03 |
mriedem | i don't try to use unstack | 15:03 |
mriedem | jgriffith: using the libvirt driver with nova and lvm? | 15:03 |
jgriffith | mriedem: yes; I do a reboot inbetween to fix up lvm and the loopback files | 15:03 |
mriedem | is the instance shelved offloaded? | 15:04 |
jgriffith | mriedem: wait, you're talking about my last question :) | 15:04 |
mriedem | i'm talking about all of the questions yes | 15:04 |
mriedem | for that conflict error, | 15:04 |
jgriffith | Yes, libvirt, nova and lvm, latest novaclient from git | 15:04 |
mriedem | are you just trying to attach a multiattach volume to an active instance? | 15:04 |
jgriffith | and issued an `export OS_COMPUTE_API_VERSION=2.60` | 15:04 |
jgriffith | mriedem: yes | 15:04 |
jgriffith | but I suspect my mv setting is not right maybe? | 15:05 |
mriedem | novaclient always uses the latest available between the server and client so that env var is unnecessary, but doesn't hurt | 15:05 |
jgriffith | mriedem: ahh, I tried that when it failed the first time thinking that must be it | 15:05 |
mriedem | what's your nova git hash? | 15:05 |
jgriffith | we should make cinder do that | 15:05 |
mriedem | the nova api change just landed like 2 days ago | 15:05 |
mriedem | tbc, nova *CLI* does version negotiation for the latest, the python API bindings are opt-in and default to 2.1 | 15:06 |
jgriffith | last commit commit 87ea686f9f2cc706205d188922bb14272625e7be | 15:06 |
mriedem | we don't consider the CLI an API | 15:06 |
jgriffith | from this morning | 15:06 |
jgriffith | mriedem: that philosophy would solve a multitude of problems | 15:06 |
mriedem | assuming you restarted the nova-api service? | 15:06 |
jgriffith | no, I just ran a fresh stack.sh on a fresh Instance | 15:07 |
mriedem | "we don't consider the API an API" yes that would be easier :) | 15:07 |
jgriffith | hehe | 15:07 |
mriedem | hmm, the instance is ACTIVE yes? | 15:07 |
mriedem | not shelved offloaded? | 15:07 |
jgriffith | correct | 15:07 |
*** kwathore has quit IRC | 15:07 | |
*** kwathore has joined #openstack-cinder | 15:08 | |
*** felipemonteiro has joined #openstack-cinder | 15:08 | |
jgriffith | hmmm, my libvirt version is 3.6.0 | 15:09 |
jgriffith | :( | 15:09 |
mriedem | that's fine, that's the version from the Pike UCA | 15:09 |
mriedem | which devstack sets up if you're using xenial | 15:09 |
*** felipemonteiro_ has joined #openstack-cinder | 15:10 | |
jgriffith | well, the checks in virt driver require >=3.10 | 15:10 |
mriedem | OOOOOOOOO | 15:10 |
mriedem | yeah...you can't use the Pike UCA | 15:10 |
mriedem | forgot about that one | 15:11 |
mriedem | you need qemu<2.10 | 15:11 |
jgriffith | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L304 | 15:11 |
mriedem | yup | 15:11 |
*** yangyapeng has quit IRC | 15:11 | |
mriedem | you need the vars from the local.conf in the nova-multiattach job, sec | 15:11 |
mriedem | https://review.openstack.org/#/c/532689/5/playbooks/legacy/nova-multiattach/run.yaml@32 | 15:11 |
jgriffith | excellent | 15:11 |
jgriffith | yaml is going to eat the world | 15:12 |
*** yangyapeng has joined #openstack-cinder | 15:12 | |
jgriffith | mriedem: and so since you say "don't" unstack/stack that means start over, spawn a new instance, and wait the 45 minutes for devstack? | 15:13 |
jgriffith | :( | 15:13 |
mriedem | well, you can try it, but | 15:13 |
jgriffith | yeah | 15:13 |
*** felipemonteiro has quit IRC | 15:13 | |
mriedem | if you're going from pike UCA to non-UCA, you'll probably still have the UCA packages | 15:14 |
mriedem | i do'nt think unstack is smart enough to roll that all back | 15:14 |
jgriffith | mriedem: i'll just save myself the pain and run a new playbook | 15:14 |
ildikov | jgriffith: I'm mostly not with wifi for the day, but I can see mriedem takes good care of you already :) | 15:15 |
jgriffith | ildikov: how convenient :) | 15:16 |
jgriffith | I want to be without wifi for the day :) | 15:16 |
*** itlinux has quit IRC | 15:16 | |
ildikov | jgriffith: you just need to go up to the mountains :) | 15:16 |
*** yangyapeng has quit IRC | 15:17 | |
ildikov | jgriffith: not even far from you :) | 15:17 |
mriedem | playbook - who's loving ansible and yaml now | 15:17 |
mriedem | i am woefully behind knowing ansible, or containers, or anything that's new and sexy in the last 5 years | 15:17 |
jgriffith | mriedem: meh, you're not missing much | 15:18 |
jgriffith | mriedem: same old stuff, new vocabulary | 15:18 |
mriedem | i know enough to find a thing, copy and change it | 15:18 |
mriedem | that's called professional software development | 15:18 |
*** itlinux has joined #openstack-cinder | 15:18 | |
*** e0ne has quit IRC | 15:18 | |
jgriffith | +1 | 15:18 |
jgriffith | I find yaml to be horribly over used | 15:18 |
jgriffith | anyway... I'm old and grumpy | 15:19 |
*** e0ne has joined #openstack-cinder | 15:19 | |
ildikov | jgriffith: I think you're just grumpy :) | 15:19 |
*** alexchadin has quit IRC | 15:19 | |
jgriffith | ildikov: hmmm... you could be right | 15:19 |
smcginnis | (he is getting kind of old) | 15:20 |
smcginnis | :P | 15:20 |
*** e0ne has quit IRC | 15:20 | |
*** prometheanfire has joined #openstack-cinder | 15:21 | |
prometheanfire | http://logs.openstack.org/50/536150/1/check/cross-cinder-py27/057dda2/testr_results.html.gz | 15:21 |
prometheanfire | looks like the test needs updating for the new keystoneauth1 bump | 15:21 |
*** dave-mccowan has quit IRC | 15:22 | |
smcginnis | prometheanfire: Thanks, I'll take a look at it. | 15:22 |
prometheanfire | smcginnis: it's just adding system_scope to the list of options passed | 15:22 |
*** stvnoyes has left #openstack-cinder | 15:23 | |
-openstackstatus- NOTICE: gerrit has been suffering from a full disk, some mails may have been lost in the last couple of hours. we will now restart gerrit to address ongoing slowness, too | 15:23 | |
openstackgerrit | Jay Bryant proposed openstack/cinder-specs master: Move Generic Backup Spec to Rocky https://review.openstack.org/537416 | 15:24 |
smcginnis | prometheanfire: Do you recall - did we raise the minimum on keystoneauth1? | 15:25 |
smcginnis | prometheanfire: Or this failure is on the patch to raise that? | 15:25 |
prometheanfire | smcginnis: >=3.3.0 | 15:25 |
prometheanfire | that's current, 2 months ago | 15:26 |
smcginnis | So that's just u-c, so we might still end up running with a lib version that does not include that additional value. | 15:26 |
jgriffith | smcginnis: careful, IIRC you and are close to the same age! :) | 15:27 |
smcginnis | jgriffith: You've got a year or two on me. :) | 15:27 |
jgriffith | darn it! | 15:28 |
smcginnis | I didn't say I wasn't getting old too though. :D | 15:28 |
*** mvenesio has quit IRC | 15:29 | |
jgriffith | Well that's inevitable (almost), but the one alternative doesn't really do much good :) | 15:29 |
smcginnis | Very true | 15:29 |
prometheanfire | smcginnis: ya, if something REQUIRES that option then I agree, a gr bump would be needed | 15:30 |
smcginnis | prometheanfire: Just makes it a bit tricky. I can add that value so it passes with the new lib, but then it will fail where the new lib isn't being used. | 15:30 |
smcginnis | Which right now will be everywhere other than that requirements patch. | 15:31 |
smcginnis | So just have to look at how to redo this test a bit. | 15:31 |
prometheanfire | smcginnis: ya, that's what I hate about this hard locking on libs :P | 15:32 |
prometheanfire | imo you should be testing lib behavior with your project, only caring about the options you use | 15:33 |
*** Guest28399 is now known as mgagne | 15:33 | |
*** mgagne has joined #openstack-cinder | 15:33 | |
jungleboyj | Anyone else seeing r.o.o as down? | 15:33 |
prometheanfire | I think that would be just as valid (or maybe more valid) coverage | 15:33 |
prometheanfire | jungleboyj: related to the gerrit restarts? | 15:33 |
jungleboyj | Guess so. *Sadness* | 15:34 |
smcginnis | prometheanfire: Yeah, I agree. I think this assertion is a little too strict. | 15:34 |
*** edmondsw_ is now known as edmondsw | 15:34 | |
*** armaan has quit IRC | 15:34 | |
Swanson | What's the current state of multi attach? | 15:38 |
Swanson | Drivers need to do something? | 15:38 |
jungleboyj | Swanson: Booo yah! Landed. | 15:38 |
jungleboyj | Swanson: Oh, yeah, I believe that drivers need to indicate if they are capable. | 15:39 |
prometheanfire | oh... that's new in queens I guess? | 15:39 |
Swanson | consumer side, too? | 15:39 |
mriedem | volume backup tests seem to be timing out at a pretty high rate in the last 24 hours http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22backup%5C%22%20AND%20message%3A%5C%22failed%20to%20reach%20available%20status%20(current%20creating)%20within%20the%20required%20time%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d | 15:39 |
jungleboyj | Swanson: There are policies to be set depending on how the consumer wants to use it. | 15:40 |
prometheanfire | jungleboyj: the normal iscsi driver/provider not good enough? | 15:40 |
jungleboyj | prometheanfire: What do you mean? | 15:40 |
*** itlinux has quit IRC | 15:40 | |
mriedem | likely related to https://github.com/openstack/cinder/commit/dd556fa755adca195e7df82477ae6400f693af14 | 15:40 |
prometheanfire | jungleboyj: I'm probably just confused, ignore me :D | 15:40 |
jungleboyj | prometheanfire: Ok. Drivers that have a backend that can export volumes to be attached to multiple hosts need to indicate the support. We don't assume. | 15:41 |
jungleboyj | mriedem: Oy. Ok. Thank you. | 15:42 |
jungleboyj | geguileo: ^^ Thoughts? | 15:42 |
mriedem | jungleboyj: https://bugs.launchpad.net/cinder/+bug/1745168 | 15:42 |
openstack | Launchpad bug 1745168 in Cinder "volume backup tests timing out since 1/23" [Undecided,New] | 15:42 |
prometheanfire | jungleboyj: of course, you happen to have a list (in docs or something)? | 15:42 |
Roamer` | hm, but wasn't that "backup in native thread" change supposed to *improve* performance? But yeah, I've seen three of these failures just on a single patch today | 15:42 |
mriedem | smcginnis: let's hold off on the backport for that change also | 15:42 |
jungleboyj | Maybe we should revert that? | 15:42 |
jungleboyj | Roamer`: Yeah .... | 15:43 |
geguileo | jungleboyj: I don't know much about the specific backends | 15:43 |
jungleboyj | prometheanfire: Don't have that listed right now. | 15:43 |
geguileo | jungleboyj: I think all backends support multiple connections, right? I mean via iSCSI you can connect multiple initiators | 15:43 |
jungleboyj | mriedem: Adding to our meeting discussion. | 15:43 |
ildikov | jgriffith: smcginnis: I don't understand why you guys want to be old so much :) | 15:43 |
jungleboyj | geguileo: I was talking about the backup/native thread patch. | 15:44 |
geguileo | lol | 15:44 |
geguileo | OK, let me check | 15:44 |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Simply keystone password assertions https://review.openstack.org/537427 | 15:44 |
Swanson | smcginnis, "Simplify" | 15:45 |
*** dave-mccowan has joined #openstack-cinder | 15:45 | |
Swanson | Or is it simply? | 15:45 |
Swanson | I should look. | 15:45 |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Simplify keystone password assertions https://review.openstack.org/537427 | 15:46 |
* jungleboyj laughs | 15:46 | |
smcginnis | Swanson: Simply simplify. | 15:46 |
jungleboyj | Simply simplify simplest simpletons. | 15:47 |
*** dustins has joined #openstack-cinder | 15:47 | |
tommylikehu | hey geguileo can we deploy multiple scheduler instances within the feature of HA support? | 15:47 |
geguileo | jungleboyj: I located the error | 15:47 |
jgriffith | mriedem: FYI those settings in the local.conf on a fresh stack worked like a charm | 15:48 |
geguileo | jungleboyj: it shouldn't be happening... | 15:48 |
jungleboyj | geguileo: Ok. | 15:48 |
geguileo | jungleboyj: It's an eventlet bug | 15:48 |
mriedem | jgriffith: great | 15:48 |
geguileo | Jan 24 15:04:16.031978 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]: Traceback (most recent call last): | 15:48 |
geguileo | Jan 24 15:04:16.032126 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]: File "/usr/local/lib/python2.7/dist-packages/eventlet/hubs/hub.py", line 457, in fire_timers | 15:48 |
geguileo | Jan 24 15:04:16.032267 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]: timer() | 15:48 |
geguileo | Jan 24 15:04:16.032404 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]: File "/usr/local/lib/python2.7/dist-packages/eventlet/hubs/timer.py", line 58, in __call__ | 15:48 |
geguileo | Jan 24 15:04:16.032559 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]: cb(*args, **kw) | 15:48 |
geguileo | Jan 24 15:04:16.032703 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]: File "/usr/local/lib/python2.7/dist-packages/eventlet/semaphore.py", line 145, in _do_acquire | 15:48 |
geguileo | Jan 24 15:04:16.032840 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]: waiter.switch() | 15:48 |
geguileo | Jan 24 15:04:16.032990 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]: error: cannot switch to a different thread | 15:48 |
geguileo | jungleboyj: ^ that's happening because a native thread withing a greenthread is logging | 15:48 |
geguileo | jungleboyj: I'll try to figure out now where is this happening | 15:48 |
openstackgerrit | Aseel Awwad proposed openstack/cinder master: Add policy check for complete attachment API action https://review.openstack.org/536119 | 15:49 |
mriedem | geguileo: nice spot, because this merged the same day https://github.com/openstack/cinder/commit/1a569e15c35d772a8ca236d75cd1840a415983d3 | 15:49 |
mriedem | geguileo: something is logging where it shouldn't be | 15:49 |
Swanson | Simple simplification of the simplest simpleton simple keystone password assertions. | 15:49 |
geguileo | mriedem: that other is different, that one actually fixes the req-id, so now is easier to follow the logs | 15:50 |
geguileo | mriedem: both patches are mine XD | 15:50 |
*** mriedem has quit IRC | 15:50 | |
*** david-lyle has joined #openstack-cinder | 15:50 | |
jungleboyj | He he he. | 15:51 |
*** jose-phi_ has joined #openstack-cinder | 15:51 | |
*** mriedem has joined #openstack-cinder | 15:51 | |
*** e0ne has joined #openstack-cinder | 15:52 | |
*** yangyapeng has joined #openstack-cinder | 15:53 | |
*** jose-phillips has quit IRC | 15:54 | |
jgriffith | mriedem: I'll move to the test next; but manual run of swap seems to work as expected: http://paste.openstack.org/show/652274/ | 15:55 |
geguileo | mriedem: I believe the problem is the swift object writer, that is logging stuff on close | 15:57 |
geguileo | mriedem: that fails because of https://github.com/eventlet/eventlet/issues/432 | 15:57 |
mriedem | yeah was just looking at the swift backup driver | 15:58 |
geguileo | mriedem: I'll submit a patch right away to remove the logging there | 15:58 |
mriedem | i knew about the logging thing https://review.openstack.org/#/c/526748/ | 15:58 |
mriedem | similar issue in nova with guestfs | 15:58 |
mriedem | looks like the other chunkeddrivers (google and posix) are clean | 16:01 |
*** AlexeyAbashkin has quit IRC | 16:05 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Remove logging on Swift backup obj writer https://review.openstack.org/537437 | 16:06 |
geguileo | mriedem: ^ | 16:09 |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Simplify keystone password assertions https://review.openstack.org/537427 | 16:10 |
mriedem | geguileo: lgtm but left a comment on doc'ing this | 16:10 |
mriedem | "don't log in specific methods" is generally forgotten if there isn't a warning | 16:11 |
geguileo | mriedem: I mentioned something in https://review.openstack.org/#/c/518316/14/cinder/backup/manager.py@87 but it's true that I was not clear about the logging, I'll add it to this patch | 16:12 |
mriedem | geguileo: oh and add bug 1745168 to the commit message | 16:14 |
openstack | bug 1745168 in Cinder "volume backup tests timing out since 1/23" [High,Confirmed] https://launchpad.net/bugs/1745168 - Assigned to Gorka Eguileor (gorka) | 16:14 |
*** lpetrut_ has quit IRC | 16:17 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Remove logging on Swift backup obj writer https://review.openstack.org/537437 | 16:25 |
mriedem | geguileo: nice lgtm | 16:25 |
mriedem | smcginnis: jungleboyj: ^ | 16:26 |
geguileo | mriedem: thanks | 16:26 |
openstackgerrit | John Griffith proposed openstack/cinder master: Add debug logging around multiattach status changes https://review.openstack.org/537444 | 16:27 |
jgriffith | mriedem: maybe run with that added and see if it helps give us some leads ^^ | 16:28 |
mriedem | jgriffith: will do in a bit | 16:28 |
*** felipemonteiro_ has quit IRC | 16:33 | |
*** felipemonteiro_ has joined #openstack-cinder | 16:34 | |
*** pcaruana has quit IRC | 16:35 | |
openstackgerrit | Merged openstack/cinder-specs master: Move update cheesecake promotion to Rocky https://review.openstack.org/535467 | 16:36 |
*** kaisers_ has quit IRC | 16:38 | |
*** e0ne has quit IRC | 16:40 | |
*** e0ne has joined #openstack-cinder | 16:40 | |
openstackgerrit | Merged openstack/python-brick-cinderclient-ext master: Updated from global requirements https://review.openstack.org/535113 | 16:41 |
*** markstur has joined #openstack-cinder | 16:45 | |
erlon | e0ne, hey, do you know if this failure is related to anything the merged recently? http://logs.openstack.org/54/534854/6/check/tempest-full/4cfe7fe/job-output.txt.gz | 16:48 |
*** _ix has quit IRC | 16:49 | |
e0ne | erlon: nothing I'm familiar with | 16:50 |
*** kaisers_ has joined #openstack-cinder | 16:54 | |
*** crose has joined #openstack-cinder | 16:55 | |
*** kien-ha has joined #openstack-cinder | 16:57 | |
*** kaisers_ has quit IRC | 16:59 | |
*** prometheanfire has left #openstack-cinder | 16:59 | |
*** _ix has joined #openstack-cinder | 17:00 | |
openstackgerrit | Merged openstack/cinder-specs master: Move Generic Backup Spec to Rocky https://review.openstack.org/537416 | 17:00 |
*** luizbag has joined #openstack-cinder | 17:02 | |
jungleboyj | erlon: Which file are you talking about? | 17:02 |
*** kaisers_ has joined #openstack-cinder | 17:07 | |
kien-ha | jungleboyj: I took a deeper look into tox and trying to get it working with py36. Was going to have you take a look at the test case changes I made and see if I'm on the right track for it but Sean brought up a good point about not everything is ready for py36. Should I leave it until another time | 17:10 |
*** kaisers_ has quit IRC | 17:12 | |
*** kaisers_ has joined #openstack-cinder | 17:15 | |
jungleboyj | kien-ha: Right. That isn't a current goal for OpenStack so I don't think that something to focus on right now. | 17:18 |
openstackgerrit | Jay Bryant proposed openstack/cinder-specs master: Move Update Backup's Size Spec to Rocky https://review.openstack.org/537460 | 17:22 |
*** sapd has quit IRC | 17:24 | |
*** gkadam has quit IRC | 17:26 | |
*** felipemonteiro_ has quit IRC | 17:29 | |
openstackgerrit | Alan Bishop proposed openstack/cinder master: Fix how backups handle encryption key IDs https://review.openstack.org/537462 | 17:30 |
*** lhx_ has quit IRC | 17:30 | |
eharney | jungleboyj: this is a pretty significant backup fix that we should get on the queens list ^ | 17:33 |
*** mvk has quit IRC | 17:35 | |
jungleboyj | Ok. | 17:35 |
jungleboyj | eharney: Good idea actually. Should start a list of important bugs to address. | 17:36 |
*** gouthamr has joined #openstack-cinder | 17:49 | |
*** gouthamr has quit IRC | 17:49 | |
*** mvenesio has joined #openstack-cinder | 17:49 | |
*** Tahvok has quit IRC | 17:51 | |
*** Tahvok has joined #openstack-cinder | 17:52 | |
jungleboyj | Added a section here: https://etherpad.openstack.org/p/cinder-spec-review-tracking | 17:55 |
jungleboyj | Release python-cinderclient: https://review.openstack.org/537468 | 17:56 |
*** gouthamr has joined #openstack-cinder | 17:56 | |
jungleboyj | Release python-brick-cinderclient-ext: https://review.openstack.org/537470 | 17:56 |
*** kien-ha has quit IRC | 17:59 | |
*** david-lyle has quit IRC | 18:04 | |
*** tesseract has quit IRC | 18:08 | |
erlon | jungleboyj, this file: https://github.com/openstack/cinder/blob/master/doc/source/configuration/tables/cinder-common.inc | 18:12 |
jungleboyj | erlon: Interesting. | 18:13 |
erlon | @all, jungleboyj: are you familiar with this error ( http://logs.openstack.org/54/534854/6/check/tempest-full/4cfe7fe/job-output.txt.gz)? tempest-full job is failing on some of my patches: https://review.openstack.org/#/c/534855/, https://review.openstack.org/#/c/534854/ | 18:13 |
geguileo | erlon: I'm familiar | 18:15 |
geguileo | erlon: there's already a fix | 18:15 |
erlon | geguileo, hmmm | 18:16 |
erlon | geguileo, do you have the link? | 18:16 |
geguileo | erlon: https://review.openstack.org/#/c/537437/ | 18:16 |
geguileo | it's trying ot merge | 18:17 |
*** stvnoyes has joined #openstack-cinder | 18:20 | |
Roamer` | geguileo, actually there are six failed tests in erlon's log, and I'm not sure if the sixth one is directly related to that bug (and THANKS for the fix, BTW!); but still it might be related to "the system was in a weird state after the other five failed", so, yeah. | 18:22 |
erlon | geguileo, nice, but, now how that change will fix the bug? The error is blowing in n-cpu, on qemu-img trying to open an invalid file: http://logs.openstack.org/54/534854/6/check/tempest-full/4cfe7fe/controller/logs/screen-n-cpu.txt.gz?level=ERROR | 18:22 |
geguileo | erlon: I saw some backup errors, and those were the ones the patch is fixing, if you have more, I don't know... | 18:23 |
geguileo | erlon: I don't think those nova errors are my fault | 18:23 |
*** markstur has quit IRC | 18:23 | |
geguileo | lol | 18:23 |
erlon | geguileo, haha | 18:24 |
*** markstur has joined #openstack-cinder | 18:24 | |
erlon | geguileo, so in the error you fix there where ERRORs on c-bak? | 18:24 |
Roamer` | erlon, the first five failed tests, the ones that mention backups, should probably all be fixed by geguileo's patch | 18:25 |
*** imacdonn has joined #openstack-cinder | 18:26 | |
*** imacdonn has quit IRC | 18:26 | |
*** imacdonn has joined #openstack-cinder | 18:26 | |
geguileo | Roamer`: +1 | 18:26 |
*** e0ne has quit IRC | 18:28 | |
*** _ix has quit IRC | 18:32 | |
erlon | jungleboyj, hey, https://review.openstack.org/#/c/534855/ is passing NetApp CI now, just need to pass zuul | 18:34 |
imacdonn | any CI gurus around? I updated Jenkins for Oracle ZFSSA CI, and the Gerrit Plugin config got lost. I thought I had a backup, but can't find it now. I have it sort of working, but need to refine the triggers. At the bottom of https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers there is a reference to a layout.yaml file, but I can't find that | 18:35 |
imacdonn | I guess layout.yaml is a zuul thing, not a jenkins thing | 18:47 |
openstackgerrit | Merged openstack/cinder-specs master: Move Update Backup's Size Spec to Rocky https://review.openstack.org/537460 | 19:00 |
*** felipemonteiro has joined #openstack-cinder | 19:00 | |
*** felipemonteiro_ has joined #openstack-cinder | 19:02 | |
*** felipemonteiro has quit IRC | 19:06 | |
*** adisky__ has quit IRC | 19:07 | |
erlon | eharney, not sure if the encryption patches has something to do with this but, there are some NFS tests failing on the encryption test: http://logs.openstack.org/37/537437/2/check/legacy-tempest-dsvm-full-devstack-plugin-nfs/9fc23b9/logs/screen-c-vol.txt.gz?level=ERROR | 19:13 |
*** izombie has joined #openstack-cinder | 19:15 | |
*** david-lyle has joined #openstack-cinder | 19:17 | |
*** luizbag has quit IRC | 19:17 | |
eharney | erlon: the NFS driver doesn't support encryption. need to finish https://review.openstack.org/#/c/523138/ . | 19:24 |
*** e0ne has joined #openstack-cinder | 19:28 | |
*** felipemonteiro_ has quit IRC | 19:32 | |
*** _ix has joined #openstack-cinder | 19:32 | |
*** muttley has joined #openstack-cinder | 19:32 | |
*** AlexeyAbashkin has joined #openstack-cinder | 19:34 | |
*** e0ne_ has joined #openstack-cinder | 19:35 | |
*** e0ne has quit IRC | 19:38 | |
*** harlowja has joined #openstack-cinder | 19:38 | |
*** AlexeyAbashkin has quit IRC | 19:38 | |
*** _ix has quit IRC | 19:39 | |
*** kaisers_ has quit IRC | 19:40 | |
erlon | eharney, ow, good, just noticed the failure and was curious about it | 19:43 |
stvnoyes | jgriffith or ildikov: Can you take a look at this bug when you get a chance? - https://bugs.launchpad.net/cinder/+bug/1745209 thanks... | 19:47 |
openstack | Launchpad bug 1745209 in Cinder "multiattach fails with context object has no attribute 'authorize'" [Undecided,New] | 19:47 |
*** muttley has quit IRC | 19:48 | |
*** dave-mccowan has quit IRC | 19:58 | |
*** erlon has quit IRC | 20:03 | |
jgriffith | stvnoyes: are you running current master? | 20:15 |
stvnoyes | yes | 20:15 |
stvnoyes | with ildoko's patch for setting multiattach on the volume | 20:16 |
stvnoyes | ildilko... | 20:16 |
stvnoyes | which is why I may be hitting it as I have multiattach set on the volume (maybe?) | 20:16 |
*** kaisers_ has joined #openstack-cinder | 20:16 | |
stvnoyes | I am posting a review with a fix in a minute or 2 | 20:16 |
stvnoyes | do you agree that this is a bug? | 20:17 |
stvnoyes | with the fix it is now working for me | 20:17 |
*** izombie has quit IRC | 20:19 | |
*** e0ne_ has quit IRC | 20:19 | |
mriedem | stvnoyes: she's out basically i think | 20:19 |
openstackgerrit | Matt Riedemann proposed openstack/cinder master: Add back support for the multiattach flag for volume create https://review.openstack.org/535863 | 20:20 |
mriedem | ildikov: ^ added the bug link and a test | 20:20 |
openstackgerrit | Matt Riedemann proposed openstack/cinder master: Add back support for the multiattach flag for volume create https://review.openstack.org/535863 | 20:21 |
mriedem | stvnoyes: yeah _attachment_reserve is definitely not using the correct variable | 20:23 |
mriedem | should be ctxt.authorize | 20:23 |
mriedem | stvnoyes: push a patch | 20:23 |
*** kaisers_ has quit IRC | 20:24 | |
openstackgerrit | Steve Noyes proposed openstack/cinder master: Fix mis-named context object https://review.openstack.org/537643 | 20:25 |
*** kaisers_ has joined #openstack-cinder | 20:26 | |
*** gouthamr has quit IRC | 20:26 | |
jgriffith | stvnoyes: Well, considering I just ran it without issue I'm not sure. | 20:26 |
stvnoyes | yeah, it's seems strange that I seem to be the only hitting it. | 20:27 |
stvnoyes | most likely because I made the volume bootable... You wouldn't see this otherwise. | 20:28 |
jgriffith | stvnoyes: oh! | 20:29 |
jgriffith | yup | 20:29 |
jgriffith | That's exactly why | 20:29 |
stvnoyes | kk I added you to the review | 20:29 |
jgriffith | approved, thanks! | 20:30 |
jgriffith | so much for the unit test | 20:30 |
stvnoyes | :-) | 20:30 |
*** crose has quit IRC | 20:31 | |
*** crose has joined #openstack-cinder | 20:33 | |
mriedem | given this was broken b/c of a lack of unit test, why not add a unit test now? | 20:33 |
*** crose has quit IRC | 20:33 | |
jgriffith | mriedem: my point was there *is* a unit test | 20:33 |
mriedem | there can't be | 20:34 |
mriedem | otherwise you'd have an AttributeError | 20:34 |
jgriffith | ok | 20:34 |
mriedem | or the test is mocking out the context call | 20:34 |
mriedem | either way there is no test, or the test is broken | 20:34 |
mriedem | if no one wants to write a test for it, i'll pull it down and add that quick | 20:34 |
jgriffith | that was my point, the test is not working properly | 20:35 |
mriedem | ok so let's fix that in the same change | 20:35 |
jgriffith | fine with me; and it is a missing test for the BFV case | 20:36 |
mriedem | i'm actually not sure how this didn't fail in tempest, | 20:36 |
mriedem | need to look but i thought we had a bfv+multiattach test, | 20:36 |
mriedem | but maybe we don't try to attach that volume to a 2nd server | 20:36 |
mriedem | yeah that's what it is, it doesn't try to attach that volume to a 2nd server https://review.openstack.org/#/c/266605/32/tempest/api/compute/volumes/test_attach_volume.py@396 | 20:37 |
mriedem | stvnoyes: are you going to fix the unit test in cinder for your patch or want me to? | 20:37 |
stvnoyes | i'll do thta. I finally got my new env working again. I am moving everything to a laptop as I will be traveling next week but will still be working | 20:38 |
mriedem | ok | 20:39 |
mriedem | you didn't have a laptop all these years? | 20:39 |
jgriffith | stvnoyes: just let me know; I'm happy to add it on since i missed it in the first place | 20:39 |
stvnoyes | i'll get it... | 20:39 |
*** dave-mccowan has joined #openstack-cinder | 20:41 | |
*** crose has joined #openstack-cinder | 20:43 | |
*** itlinux has joined #openstack-cinder | 20:44 | |
openstackgerrit | Matt Riedemann proposed openstack/cinder master: Fix multiattach policy names in release note https://review.openstack.org/537652 | 20:47 |
mriedem | jgriffith: smcginnis: ^ | 20:48 |
openstackgerrit | Merged openstack/cinder master: Remove logging on Swift backup obj writer https://review.openstack.org/537437 | 20:50 |
mriedem | woot ^ | 20:50 |
smcginnis | mriedem: There's the logging/threading fix. | 20:50 |
mriedem | way ahead of you :) | 20:50 |
openstackgerrit | John Griffith proposed openstack/cinder master: Add debug logging around multiattach status changes https://review.openstack.org/537444 | 20:51 |
smcginnis | Then I shall woot your woot. | 20:51 |
smcginnis | Hmm, sounds kind of naughty. Maybe I won't do that. | 20:51 |
*** belmoreira has joined #openstack-cinder | 20:51 | |
mriedem | sloppy woots? | 20:51 |
* smcginnis is not going to continue this conversation | 20:51 | |
jungleboyj | Woot woot woot! | 20:51 |
Swanson | Eww. | 20:51 |
jungleboyj | mriedem: He he he. | 20:52 |
mriedem | and....i'm spent | 20:53 |
* jungleboyj hands mriedem a cigarette | 20:53 | |
jungleboyj | Was it good for you? | 20:53 |
mriedem | you know, | 20:53 |
mriedem | i've had better | 20:53 |
jungleboyj | bwah ha ha. | 20:54 |
*** e0ne has joined #openstack-cinder | 21:00 | |
*** liverpooler has quit IRC | 21:00 | |
*** crose has quit IRC | 21:07 | |
openstackgerrit | Eric Harney proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest https://review.openstack.org/531206 | 21:17 |
openstackgerrit | Eric Harney proposed openstack/cinder-tempest-plugin master: Add LVM+tgt tempest job https://review.openstack.org/537658 | 21:18 |
*** crose has joined #openstack-cinder | 21:19 | |
*** e0ne has quit IRC | 21:20 | |
openstackgerrit | Eric Harney proposed openstack/cinder-tempest-plugin master: Add LVM+tgt tempest job https://review.openstack.org/537658 | 21:21 |
eharney | we need some kind of tox zuul syntax checker there ^ | 21:22 |
eharney | not sure how to see if the patch is correct w/o just throwing it at the gate :) | 21:22 |
smcginnis | eharney: I think there is an infra zuul linter job. | 21:23 |
smcginnis | eharney: I wonder how hard it would be to reuse that. | 21:23 |
eharney | smcginnis: there should be one somewhere, it's done for project-config etc | 21:23 |
*** dave-mccowan has quit IRC | 21:25 | |
eharney | smcginnis: if that runs, it should address your -1 on the patch above it ^ | 21:26 |
*** pchavva has quit IRC | 21:33 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder-tempest-plugin master: Correct plugin test location reporting https://review.openstack.org/537664 | 21:38 |
smcginnis | eharney: So now we're going to run two tempests against it? | 21:39 |
eharney | smcginnis: should we not? | 21:39 |
smcginnis | eharney: Well, like I commented on there, LIO is currently non-voting on cinder, so it seems odd to me to have that a voting job on cinder-tempest-plugin. | 21:39 |
smcginnis | eharney: But maybe that should change. | 21:40 |
eharney | smcginnis: the LIO job should be voting on cinder, since it's the only thing testing barbican | 21:40 |
smcginnis | eharney: Do we expect there to be functional differences that we need to verify in the tempest plugin? | 21:40 |
smcginnis | I guess I would rather just have LIO then, and not both unless there is significant differences between the two when it comes to these tests. (which I didn't think there were) | 21:41 |
*** threestrands has joined #openstack-cinder | 21:41 | |
eharney | me too | 21:42 |
*** threestrands_ has joined #openstack-cinder | 21:44 | |
*** threestrands_ has quit IRC | 21:44 | |
*** threestrands_ has joined #openstack-cinder | 21:44 | |
*** threestrands has quit IRC | 21:47 | |
openstackgerrit | OpenStack Release Bot proposed openstack/python-brick-cinderclient-ext master: Update reno for stable/queens https://review.openstack.org/537674 | 21:52 |
*** eharney has quit IRC | 21:54 | |
openstackgerrit | Sean McGinnis proposed openstack/python-brick-cinderclient-ext master: Update reno for stable/queens https://review.openstack.org/537674 | 21:55 |
*** kien-ha has joined #openstack-cinder | 21:55 | |
*** eharney has joined #openstack-cinder | 21:56 | |
*** dustins has quit IRC | 21:56 | |
*** eharney has quit IRC | 21:59 | |
*** belmoreira has quit IRC | 22:00 | |
openstackgerrit | OpenStack Release Bot proposed openstack/python-cinderclient master: Update reno for stable/queens https://review.openstack.org/537683 | 22:02 |
*** mvenesio has quit IRC | 22:05 | |
*** tpsilva has quit IRC | 22:07 | |
openstackgerrit | Alan Bishop proposed openstack/cinder master: Fix how backups handle encryption key IDs https://review.openstack.org/537462 | 22:08 |
*** rmcall has quit IRC | 22:09 | |
*** dustins has joined #openstack-cinder | 22:10 | |
*** dustins_ has joined #openstack-cinder | 22:12 | |
*** abishop has quit IRC | 22:13 | |
*** rcernin has joined #openstack-cinder | 22:14 | |
*** dustins has quit IRC | 22:15 | |
*** kien-ha has quit IRC | 22:15 | |
*** yangyapeng has quit IRC | 22:16 | |
*** crose has quit IRC | 22:20 | |
*** rcernin has quit IRC | 22:29 | |
*** chkumar246 has joined #openstack-cinder | 22:30 | |
*** rcernin has joined #openstack-cinder | 22:31 | |
*** chandankumar has quit IRC | 22:32 | |
*** chkumar246 has quit IRC | 22:35 | |
*** chkumar246 has joined #openstack-cinder | 22:36 | |
*** erlon has joined #openstack-cinder | 22:43 | |
*** edmondsw has quit IRC | 22:47 | |
*** abishop has joined #openstack-cinder | 22:49 | |
openstackgerrit | Steve Noyes proposed openstack/cinder master: Fix mis-named context object https://review.openstack.org/537643 | 22:50 |
ildikov | stvnoyes: sorry, I don't have wifi/data most of the day :/ | 22:51 |
*** itlinux has quit IRC | 22:51 | |
stvnoyes | no problem. Hopefully that will make things less stressful for you :-) | 22:52 |
ildikov | stvnoyes: well, let's not talk about my stress level... :) | 22:53 |
ildikov | stvnoyes: jgriffith can fill you in how many times I freak out like on a weekly sometimes daily basis... :/ :) | 22:54 |
*** dustins_ has quit IRC | 22:57 | |
openstackgerrit | Matt Riedemann proposed openstack/cinder master: Fix mis-named context object https://review.openstack.org/537643 | 22:59 |
*** yangyapeng has joined #openstack-cinder | 23:01 | |
*** itlinux has joined #openstack-cinder | 23:03 | |
*** eharney has joined #openstack-cinder | 23:08 | |
*** itlinux has quit IRC | 23:23 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!