*** KeithMnemonic has joined #openstack-cinder | 00:17 | |
*** markvoelker has joined #openstack-cinder | 00:20 | |
openstackgerrit | Merged openstack/cinder master: NEC driver: add automatic configuration of SAN access control. https://review.openstack.org/529430 | 00:21 |
---|---|---|
openstackgerrit | Merged openstack/cinder master: Update 'force' parameter for volume delete API https://review.openstack.org/523344 | 00:21 |
openstackgerrit | Merged openstack/cinder master: V3 jsonschema validation: Volume-types https://review.openstack.org/520561 | 00:21 |
*** KeithMnemonic has quit IRC | 00:22 | |
*** yangyapeng has quit IRC | 00:22 | |
*** yangyapeng has joined #openstack-cinder | 00:23 | |
*** markvoelker has quit IRC | 00:24 | |
*** yangyapeng has quit IRC | 00:27 | |
*** itlinux has joined #openstack-cinder | 00:41 | |
openstackgerrit | Merged openstack/cinder master: Fix incorrect description in api ref https://review.openstack.org/530063 | 00:44 |
*** KeithMnemonic has joined #openstack-cinder | 00:44 | |
tommylikehu | hey smcginnis, could you take a look at this patch? https://review.openstack.org/#/c/529993/ | 00:48 |
*** KeithMnemonic has quit IRC | 00:50 | |
*** Brin has joined #openstack-cinder | 00:50 | |
*** harlowja has joined #openstack-cinder | 00:54 | |
openstackgerrit | wanghao proposed openstack/cinder master: Use constants for cinder-volume https://review.openstack.org/530162 | 01:05 |
*** psachin has joined #openstack-cinder | 01:11 | |
*** KeithMnemonic has joined #openstack-cinder | 01:13 | |
*** yangyapeng has joined #openstack-cinder | 01:13 | |
openstackgerrit | Merged openstack/os-brick master: Add the StorPool brick connector https://review.openstack.org/192639 | 01:16 |
*** KeithMnemonic has quit IRC | 01:18 | |
*** edmondsw has joined #openstack-cinder | 01:20 | |
*** markvoelker has joined #openstack-cinder | 01:21 | |
*** AlexeyAbashkin has joined #openstack-cinder | 01:22 | |
*** edmondsw has quit IRC | 01:25 | |
*** markvoelker has quit IRC | 01:25 | |
openstackgerrit | Merged openstack/cinder master: Remove some no-use code about failover https://review.openstack.org/530091 | 01:26 |
*** wanghao has joined #openstack-cinder | 01:26 | |
*** AlexeyAbashkin has quit IRC | 01:27 | |
*** daidv-xmas has quit IRC | 01:27 | |
*** daidv_ has joined #openstack-cinder | 01:27 | |
*** daidv has joined #openstack-cinder | 01:27 | |
*** Trident has quit IRC | 01:30 | |
*** KeithMnemonic has joined #openstack-cinder | 01:38 | |
*** KeithMnemonic has quit IRC | 01:42 | |
*** zhurong has joined #openstack-cinder | 01:44 | |
*** gcb has joined #openstack-cinder | 01:48 | |
*** KeithMnemonic has joined #openstack-cinder | 01:57 | |
*** namnh has joined #openstack-cinder | 02:01 | |
*** psachin has quit IRC | 02:01 | |
*** KeithMnemonic has quit IRC | 02:02 | |
*** daidv_ has quit IRC | 02:03 | |
*** psachin has joined #openstack-cinder | 02:05 | |
*** lhx_ has joined #openstack-cinder | 02:15 | |
openstackgerrit | yixuan zhang proposed openstack/cinder master: Storwize: add hyperswap volume support https://review.openstack.org/492414 | 02:19 |
*** KeithMnemonic has joined #openstack-cinder | 02:20 | |
*** markvoelker has joined #openstack-cinder | 02:21 | |
*** KeithMnemonic has quit IRC | 02:25 | |
*** markvoelker has quit IRC | 02:26 | |
Brin | Hi wanghao.Have time talk about notification for qos? | 02:33 |
wanghao | sure | 02:33 |
Brin | Did you read my reply? | 02:34 |
wanghao | yeah | 02:34 |
Brin | I think that modify is good, not needed updated_at in "associate". | 02:36 |
Brin | This is in line with usage. | 02:36 |
wanghao | I see, your point is we notify the association 'create' time, not the qos 'update' time. | 02:36 |
Brin | I mean updated_at no need to add qos time, and the code will generate updated_at time. | 02:38 |
*** harlowja has quit IRC | 02:41 | |
*** KeithMnemonic has joined #openstack-cinder | 02:41 | |
Brin | I think you need to understand the needs first, thak you. | 02:41 |
*** gcb has quit IRC | 02:42 | |
*** gcb has joined #openstack-cinder | 02:44 | |
*** KeithMnemonic has quit IRC | 02:45 | |
*** zhurong has quit IRC | 02:50 | |
*** v12aml has quit IRC | 02:51 | |
openstackgerrit | TommyLike proposed openstack/cinder master: Validate volume status again when snapshot created https://review.openstack.org/529993 | 02:57 |
wanghao | I know you want to add some log when doing notification, but since you add the updated_at and created_at at notifier_info. My point is the update_at should also be added in notification body sent to rabbitmq. | 02:58 |
*** v12aml has joined #openstack-cinder | 03:00 | |
wanghao | okay, I see what's going on here. | 03:04 |
wanghao | no need to add updated_at at notifer_info, since cinder didn't update the time in this action. | 03:05 |
*** edmondsw has joined #openstack-cinder | 03:08 | |
*** Brin has quit IRC | 03:10 | |
*** Brin has joined #openstack-cinder | 03:11 | |
*** edmondsw has quit IRC | 03:13 | |
wanghao | Brin: ping | 03:13 |
*** ericyoung has quit IRC | 03:15 | |
Brin | Yeah. A little busy. | 03:16 |
wanghao | just another question | 03:17 |
Brin | yes | 03:17 |
wanghao | you mean log information in commit message, does it mean the cinder log file? | 03:18 |
wanghao | or notification information that cinder sent out. | 03:18 |
wanghao | a little misunderstanding with that. | 03:18 |
*** ericyoung has joined #openstack-cinder | 03:20 | |
Brin | These notifications will be printed in the cinder's log. You can reference bug-1569150 | 03:21 |
openstackgerrit | Neha Alhat proposed openstack/cinder master: V2/V3 jsonschema validation: snapshots https://review.openstack.org/520991 | 03:21 |
wanghao | okay, I see this | 03:22 |
wanghao | for log side, no need to print the update_at time. But do you think we need to add the update_at time in notifier_info. That will also be sent to celimeter too. | 03:23 |
wanghao | I'm fine with your patch, just want to discuss this a little. | 03:24 |
Brin | I agree with tommylike, and update time does not make sense here. | 03:24 |
Brin | When qos is updated, update time will be printed. | 03:27 |
wanghao | yeah, that's right. I agree with tommy too. The issue here is now when we do assassociate or disassociate action, cinder will not update the qos's time, just volume_type's time. | 03:30 |
wanghao | so there is no need to add update_at time. | 03:30 |
*** abhishekk has joined #openstack-cinder | 03:30 | |
wanghao | because that will be a little confusing. | 03:30 |
wanghao | thank your reply for this. | 03:31 |
Brin | No problem. | 03:32 |
*** harlowja has joined #openstack-cinder | 03:33 | |
*** abhishekk has quit IRC | 03:33 | |
*** rmk has quit IRC | 03:41 | |
*** rmk has joined #openstack-cinder | 03:48 | |
*** rmk has quit IRC | 03:48 | |
*** rmk has joined #openstack-cinder | 03:48 | |
*** Brin has quit IRC | 04:00 | |
*** wanghao has quit IRC | 04:05 | |
*** wanghao has joined #openstack-cinder | 04:05 | |
*** wanghao has quit IRC | 04:05 | |
*** wanghao has joined #openstack-cinder | 04:06 | |
*** wanghao has quit IRC | 04:06 | |
*** wanghao has joined #openstack-cinder | 04:07 | |
*** wanghao has quit IRC | 04:07 | |
*** wanghao has joined #openstack-cinder | 04:07 | |
*** wanghao has quit IRC | 04:08 | |
*** wanghao has joined #openstack-cinder | 04:08 | |
*** wanghao has quit IRC | 04:09 | |
*** wanghao has joined #openstack-cinder | 04:09 | |
*** wanghao has quit IRC | 04:09 | |
*** Brin has joined #openstack-cinder | 04:10 | |
*** wanghao has joined #openstack-cinder | 04:10 | |
*** wanghao has quit IRC | 04:10 | |
*** wanghao has joined #openstack-cinder | 04:11 | |
*** wanghao has quit IRC | 04:11 | |
*** wanghao has joined #openstack-cinder | 04:11 | |
*** wanghao has quit IRC | 04:12 | |
*** wanghao has joined #openstack-cinder | 04:12 | |
*** wanghao has quit IRC | 04:13 | |
*** wanghao has joined #openstack-cinder | 04:13 | |
*** wanghao has quit IRC | 04:13 | |
*** KeithMnemonic has joined #openstack-cinder | 04:17 | |
*** KeithMnemonic has quit IRC | 04:21 | |
*** markvoelker has joined #openstack-cinder | 04:23 | |
*** harlowja has quit IRC | 04:24 | |
*** markvoelker has quit IRC | 04:27 | |
*** mikal has quit IRC | 04:29 | |
*** mikal has joined #openstack-cinder | 04:30 | |
*** gkadam has joined #openstack-cinder | 04:39 | |
*** KeithMnemonic has joined #openstack-cinder | 04:43 | |
*** AlexeyAbashkin has joined #openstack-cinder | 04:46 | |
*** lhx_ has quit IRC | 04:47 | |
*** KeithMnemonic has quit IRC | 04:48 | |
*** gkadam has quit IRC | 04:49 | |
*** AlexeyAbashkin has quit IRC | 04:50 | |
*** edmondsw has joined #openstack-cinder | 04:56 | |
*** gcb has quit IRC | 04:57 | |
*** lhx_ has joined #openstack-cinder | 04:58 | |
*** KeithMnemonic has joined #openstack-cinder | 04:59 | |
*** edmondsw has quit IRC | 05:01 | |
*** KeithMnemonic has quit IRC | 05:03 | |
*** itlinux has quit IRC | 05:15 | |
*** zhurong has joined #openstack-cinder | 05:21 | |
*** KeithMnemonic has joined #openstack-cinder | 05:21 | |
*** markvoelker has joined #openstack-cinder | 05:23 | |
*** KeithMnemonic has quit IRC | 05:26 | |
*** markvoelker has quit IRC | 05:28 | |
*** wanghao has joined #openstack-cinder | 05:38 | |
*** lhx_ has quit IRC | 05:40 | |
*** lhx_ has joined #openstack-cinder | 05:40 | |
*** gcb has joined #openstack-cinder | 05:40 | |
*** KeithMnemonic has joined #openstack-cinder | 05:45 | |
*** KeithMnemonic has quit IRC | 05:49 | |
*** gkadam has joined #openstack-cinder | 05:55 | |
*** KeithMnemonic has joined #openstack-cinder | 06:03 | |
*** lhx_ has quit IRC | 06:07 | |
*** KeithMnemonic has quit IRC | 06:07 | |
*** arnewiebalck_ has joined #openstack-cinder | 06:13 | |
openstackgerrit | Chhavi Agarwal proposed openstack/cinder master: Run backup-restore operations on native thread https://review.openstack.org/518316 | 06:21 |
openstackgerrit | Pooja Jadhav proposed openstack/cinder master: V2/V3 jsonschema validation: snapshots https://review.openstack.org/520991 | 06:21 |
openstackgerrit | Pooja Jadhav proposed openstack/cinder master: V3 jsonschema validation: Backups https://review.openstack.org/530098 | 06:21 |
*** psachin has quit IRC | 06:21 | |
*** markvoelker has joined #openstack-cinder | 06:24 | |
*** lhx_ has joined #openstack-cinder | 06:27 | |
*** arnewiebalck_ has quit IRC | 06:28 | |
*** arnewiebalck_ has joined #openstack-cinder | 06:29 | |
*** markvoelker has quit IRC | 06:29 | |
*** KeithMnemonic has joined #openstack-cinder | 06:30 | |
*** arnewiebalck_ has quit IRC | 06:30 | |
*** KeithMnemonic has quit IRC | 06:35 | |
Brin | How to deal with this error:http://logs.openstack.org/45/527345/3/gate/legacy-tempest-dsvm-py35/410ce85/job-output.txt.gz#_2017-12-28_23_54_16_386636 | 06:39 |
*** edmondsw has joined #openstack-cinder | 06:44 | |
*** KeithMnemonic has joined #openstack-cinder | 06:46 | |
*** kholkina has joined #openstack-cinder | 06:48 | |
*** edmondsw has quit IRC | 06:49 | |
*** KeithMnemonic has quit IRC | 06:51 | |
*** pgadiya has joined #openstack-cinder | 06:58 | |
tommylikehu | Hey Brin not sure about the root cause currently, how about another recheck command? | 07:02 |
*** zhurong has quit IRC | 07:03 | |
*** wanghao_ has joined #openstack-cinder | 07:05 | |
Brin | Thank you tommy, I will do recheck once. | 07:07 |
*** dengxiafubi has joined #openstack-cinder | 07:07 | |
*** wanghao has quit IRC | 07:08 | |
*** KeithMnemonic has joined #openstack-cinder | 07:12 | |
*** KeithMnemonic has quit IRC | 07:17 | |
*** e0ne has joined #openstack-cinder | 07:22 | |
*** markvoelker has joined #openstack-cinder | 07:25 | |
*** markvoelker has quit IRC | 07:29 | |
*** gkadam has quit IRC | 07:30 | |
openstackgerrit | wanghao proposed openstack/python-cinderclient master: Support for reporting backend state in service list https://review.openstack.org/530446 | 07:32 |
*** e0ne has quit IRC | 07:33 | |
*** KeithMnemonic has joined #openstack-cinder | 07:35 | |
*** gkadam has joined #openstack-cinder | 07:38 | |
*** KeithMnemonic has quit IRC | 07:40 | |
*** lhx__ has joined #openstack-cinder | 07:46 | |
*** lhx_ has quit IRC | 07:46 | |
*** KeithMnemonic has joined #openstack-cinder | 07:53 | |
*** gkadam has quit IRC | 07:56 | |
*** KeithMnemonic has quit IRC | 07:57 | |
*** gkadam has joined #openstack-cinder | 08:02 | |
*** gkadam has quit IRC | 08:05 | |
*** gkadam has joined #openstack-cinder | 08:05 | |
*** alexchadin has joined #openstack-cinder | 08:06 | |
*** wanghao_ has quit IRC | 08:12 | |
*** wanghao has joined #openstack-cinder | 08:12 | |
*** lhx_ has joined #openstack-cinder | 08:15 | |
*** lhx__ has quit IRC | 08:16 | |
*** zhurong has joined #openstack-cinder | 08:17 | |
*** KeithMnemonic has joined #openstack-cinder | 08:18 | |
*** arnewiebalck_ has joined #openstack-cinder | 08:19 | |
*** arnewiebalck_ has quit IRC | 08:20 | |
*** e0ne has joined #openstack-cinder | 08:20 | |
*** arnewiebalck_ has joined #openstack-cinder | 08:21 | |
*** KeithMnemonic has quit IRC | 08:23 | |
*** AlexeyAbashkin has joined #openstack-cinder | 08:23 | |
*** e0ne has quit IRC | 08:25 | |
*** jdurgin has joined #openstack-cinder | 08:25 | |
*** pgadiya has quit IRC | 08:28 | |
*** edmondsw has joined #openstack-cinder | 08:32 | |
*** edmondsw has quit IRC | 08:37 | |
*** vivsoni has quit IRC | 08:39 | |
*** pgadiya has joined #openstack-cinder | 08:41 | |
*** vivsoni has joined #openstack-cinder | 08:46 | |
*** KeithMnemonic has joined #openstack-cinder | 08:47 | |
*** dengxiafubi has quit IRC | 08:48 | |
*** KeithMnemonic has quit IRC | 08:51 | |
*** wanghao has quit IRC | 09:06 | |
*** KeithMnemonic has joined #openstack-cinder | 09:08 | |
*** pgadiya has quit IRC | 09:09 | |
*** KeithMnemonic has quit IRC | 09:12 | |
*** zhurong has quit IRC | 09:14 | |
*** wanghao has joined #openstack-cinder | 09:15 | |
*** pgadiya has joined #openstack-cinder | 09:22 | |
*** KeithMnemonic has joined #openstack-cinder | 09:24 | |
*** markvoelker has joined #openstack-cinder | 09:26 | |
*** wanghao has quit IRC | 09:29 | |
*** KeithMnemonic has quit IRC | 09:29 | |
*** Brin has quit IRC | 09:30 | |
*** markvoelker has quit IRC | 09:31 | |
*** gkadam has quit IRC | 09:33 | |
*** gcb has quit IRC | 09:45 | |
*** KeithMnemonic has joined #openstack-cinder | 09:52 | |
*** lhx_ has quit IRC | 09:56 | |
*** KeithMnemonic has quit IRC | 09:57 | |
*** alexchadin has quit IRC | 10:01 | |
*** alexchadin has joined #openstack-cinder | 10:03 | |
*** KeithMnemonic has joined #openstack-cinder | 10:07 | |
*** daidv has quit IRC | 10:09 | |
*** KeithMnemonic has quit IRC | 10:12 | |
*** namnh has quit IRC | 10:14 | |
*** edmondsw has joined #openstack-cinder | 10:20 | |
*** _ix has joined #openstack-cinder | 10:23 | |
*** edmondsw has quit IRC | 10:25 | |
*** alexchadin has quit IRC | 10:25 | |
*** alexchadin has joined #openstack-cinder | 10:26 | |
*** KeithMnemonic has joined #openstack-cinder | 10:29 | |
*** KeithMnemonic has quit IRC | 10:34 | |
*** _ix has quit IRC | 10:35 | |
*** KeithMnemonic has joined #openstack-cinder | 10:51 | |
*** e0ne has joined #openstack-cinder | 10:54 | |
*** KeithMnemonic has quit IRC | 10:56 | |
*** arnewiebalck_ has quit IRC | 10:58 | |
*** alexchadin has quit IRC | 10:58 | |
*** alexchadin has joined #openstack-cinder | 10:58 | |
openstackgerrit | Merged openstack/cinder master: Add instructions for the Error response codes https://review.openstack.org/527345 | 11:05 |
*** KeithMnemonic has joined #openstack-cinder | 11:09 | |
*** KeithMnemonic has quit IRC | 11:13 | |
*** wanghao has joined #openstack-cinder | 11:17 | |
*** alexchadin has quit IRC | 11:21 | |
*** markvoelker has joined #openstack-cinder | 11:27 | |
*** AlexeyAbashkin has quit IRC | 11:28 | |
*** AlexeyAbashkin has joined #openstack-cinder | 11:29 | |
*** KeithMnemonic has joined #openstack-cinder | 11:30 | |
*** markvoelker has quit IRC | 11:32 | |
*** KeithMnemonic has quit IRC | 11:35 | |
*** pgadiya has quit IRC | 11:45 | |
*** wanghao has quit IRC | 11:47 | |
*** wanghao has joined #openstack-cinder | 11:48 | |
*** gkadam has joined #openstack-cinder | 11:49 | |
*** wanghao_ has joined #openstack-cinder | 11:50 | |
*** wanghao_ has quit IRC | 11:50 | |
*** wanghao_ has joined #openstack-cinder | 11:50 | |
*** wanghao has quit IRC | 11:51 | |
*** arnewiebalck_ has joined #openstack-cinder | 11:55 | |
*** KeithMnemonic has joined #openstack-cinder | 11:59 | |
*** wanghao has joined #openstack-cinder | 11:59 | |
*** wanghao has quit IRC | 11:59 | |
*** arnewiebalck_ has quit IRC | 11:59 | |
*** wanghao_ has quit IRC | 12:02 | |
*** gkadam has quit IRC | 12:03 | |
*** KeithMnemonic has quit IRC | 12:04 | |
*** edmondsw has joined #openstack-cinder | 12:08 | |
*** edmondsw has quit IRC | 12:13 | |
*** KeithMnemonic has joined #openstack-cinder | 12:25 | |
*** KeithMnemonic has quit IRC | 12:29 | |
*** chrisyang_0660 has quit IRC | 12:32 | |
*** yangyapeng has quit IRC | 12:40 | |
*** yangyapeng has joined #openstack-cinder | 12:41 | |
*** KeithMnemonic has joined #openstack-cinder | 12:44 | |
*** yangyapeng has quit IRC | 12:48 | |
*** KeithMnemonic has quit IRC | 12:49 | |
*** kholkina has quit IRC | 12:59 | |
*** kholkina has joined #openstack-cinder | 12:59 | |
*** nicolasbock has joined #openstack-cinder | 13:02 | |
*** niraj_singh has quit IRC | 13:03 | |
*** KeithMnemonic has joined #openstack-cinder | 13:04 | |
*** KeithMnemonic has quit IRC | 13:09 | |
*** jistr has quit IRC | 13:14 | |
*** yangyapeng has joined #openstack-cinder | 13:15 | |
*** jistr has joined #openstack-cinder | 13:16 | |
*** markvoelker has joined #openstack-cinder | 13:28 | |
*** kholkina has quit IRC | 13:30 | |
*** kholkina has joined #openstack-cinder | 13:30 | |
*** KeithMnemonic has joined #openstack-cinder | 13:30 | |
*** markvoelker has quit IRC | 13:32 | |
*** KeithMnemonic has quit IRC | 13:35 | |
*** KeithMnemonic has joined #openstack-cinder | 13:56 | |
*** edmondsw has joined #openstack-cinder | 13:57 | |
*** edmondsw has quit IRC | 14:01 | |
*** KeithMnemonic has quit IRC | 14:01 | |
*** mmethot_ has joined #openstack-cinder | 14:11 | |
*** mmethot has quit IRC | 14:13 | |
*** KeithMnemonic has joined #openstack-cinder | 14:18 | |
*** KeithMnemonic has quit IRC | 14:23 | |
*** kholkina has quit IRC | 14:34 | |
*** KeithMnemonic has joined #openstack-cinder | 14:36 | |
*** KeithMnemonic has quit IRC | 14:41 | |
*** pooja-jadhav has joined #openstack-cinder | 14:55 | |
*** pooja_jadhav has quit IRC | 14:59 | |
*** danpawlik has quit IRC | 15:05 | |
*** markvoelker has joined #openstack-cinder | 15:29 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Add tempest plugin to LIO tests https://review.openstack.org/528511 | 15:32 |
*** markvoelker has quit IRC | 15:33 | |
*** KeithMnemonic has joined #openstack-cinder | 15:39 | |
*** abishop has joined #openstack-cinder | 15:43 | |
*** edmondsw has joined #openstack-cinder | 15:45 | |
*** edmondsw has quit IRC | 15:49 | |
*** AlexeyAbashkin has quit IRC | 15:58 | |
*** _ix has joined #openstack-cinder | 16:01 | |
*** Apoorva has joined #openstack-cinder | 16:17 | |
*** e0ne has quit IRC | 16:19 | |
*** _ix has quit IRC | 16:20 | |
*** abishop has quit IRC | 16:30 | |
*** abishop has joined #openstack-cinder | 16:32 | |
*** KeithMnemonic has quit IRC | 16:42 | |
*** itlinux_ has joined #openstack-cinder | 16:42 | |
*** jmlowe has quit IRC | 16:50 | |
*** _ix has joined #openstack-cinder | 16:53 | |
*** Apoorva_ has joined #openstack-cinder | 17:05 | |
*** Apoorva has quit IRC | 17:07 | |
*** Apoorva_ has quit IRC | 17:08 | |
openstackgerrit | Merged openstack/cinder master: Validate volume status again when snapshot created https://review.openstack.org/529993 | 17:14 |
*** AlexeyAbashkin has joined #openstack-cinder | 17:27 | |
*** lyarwood has quit IRC | 17:28 | |
*** markvoelker has joined #openstack-cinder | 17:30 | |
*** AlexeyAbashkin has quit IRC | 17:31 | |
*** edmondsw has joined #openstack-cinder | 17:33 | |
*** markvoelker has quit IRC | 17:34 | |
*** edmondsw has quit IRC | 17:38 | |
*** e0ne has joined #openstack-cinder | 17:43 | |
*** e0ne has quit IRC | 17:46 | |
*** _ix has quit IRC | 17:57 | |
*** _ix has joined #openstack-cinder | 18:00 | |
*** Apoorva has joined #openstack-cinder | 18:08 | |
*** Apoorva has quit IRC | 18:13 | |
*** itlinux_ has quit IRC | 18:36 | |
*** arnewiebalck_ has joined #openstack-cinder | 19:01 | |
*** Apoorva has joined #openstack-cinder | 19:10 | |
*** Apoorva has quit IRC | 19:14 | |
*** arnewiebalck_ has quit IRC | 19:18 | |
*** edmondsw has joined #openstack-cinder | 19:21 | |
*** edmondsw has quit IRC | 19:25 | |
*** markvoelker has joined #openstack-cinder | 19:31 | |
*** Apoorva has joined #openstack-cinder | 19:33 | |
*** markvoelker has quit IRC | 19:35 | |
*** itlinux has joined #openstack-cinder | 19:38 | |
*** Apoorva has quit IRC | 19:43 | |
*** abhi has quit IRC | 20:00 | |
*** rsjethani has joined #openstack-cinder | 20:21 | |
rsjethani | enabling nfs snapshot support is causing nfs service to appear down?? | 20:22 |
*** markvoelker has joined #openstack-cinder | 20:31 | |
*** markvoelker has quit IRC | 20:36 | |
mnaser | we're working on adding CI for our deployment (so we can test every change fully with tempest). i've ran into this failing test: tempest.scenario.test_volume_migrate_attached.TestVolumeMigrateRetypeAttached.test_volume_migrate_attached | 20:49 |
mnaser | it looks like https://review.openstack.org/#/c/474789/ reverted the change to make it run in admin context | 20:50 |
mnaser | and it seems when this https://github.com/openstack/cinder/blob/11.0.2/cinder/volume/manager.py#L2098-L2100 is called, it fails because nova says "you're not allowed!" because updating volume attachments is admin api only | 20:50 |
*** e0ne has joined #openstack-cinder | 20:53 | |
mnaser | looks like that code path is fixed except if it can't be retyped, it falls back to migrating | 20:55 |
mnaser | and the migration happens in user context, which cannot be used to contact nova | 20:55 |
*** jmlowe has joined #openstack-cinder | 21:00 | |
*** e0ne_ has joined #openstack-cinder | 21:02 | |
*** jmlowe has quit IRC | 21:04 | |
*** e0ne_ has quit IRC | 21:05 | |
mnaser | https://bugs.launchpad.net/cinder/+bug/1740544 -- if anyone's interested | 21:06 |
openstack | Launchpad bug 1740544 in Cinder "Volume retype fails when migration occurs" [Undecided,New] | 21:06 |
smcginnis | mnaser: I think that's operating as expected. | 21:07 |
smcginnis | mnaser: If only admin can migrate, and retyping would require a migration, then it enforces the policy. | 21:07 |
mnaser | smcginnis: but it looks/sounds like a user can retype? then perhaps this is a tempest bug? | 21:07 |
smcginnis | mnaser: A user can retype, yes. But retype does not necessary imply migration. | 21:08 |
mnaser | smcginnis: so the user can retype, but if it falls back to a migration, the retype would fail? | 21:08 |
*** edmondsw has joined #openstack-cinder | 21:09 | |
smcginnis | mnaser: Most user-initiated retypes should be to change some setting of the volume. If the deployment doesn't want to allow users to move the volumes around (migration) then yes, I would expect a retype that would need to perform migration to fail. | 21:09 |
*** jmlowe has joined #openstack-cinder | 21:11 | |
mnaser | smcginnis: so is this more of a tempest issue? i see https://review.openstack.org/#/c/474789/ to make it admin only and then https://review.openstack.org/#/c/496700/ to revert it to non-admin | 21:11 |
mnaser | (and I guess we can't retype ceph/rbd so that's why it fails in our environment) | 21:12 |
*** edmondsw has quit IRC | 21:13 | |
*** e0ne has quit IRC | 21:13 | |
smcginnis | mnaser: Yeah, I don't think that last one should have gone through. | 21:14 |
*** e0ne has joined #openstack-cinder | 21:15 | |
*** jmlowe has quit IRC | 21:16 | |
mnaser | smcginnis: i will propose a revert i guess and leave comments about it | 21:20 |
smcginnis | mnaser: Cool, I'll look for it and add my 2 cents as well. | 21:20 |
*** e0ne has quit IRC | 21:22 | |
mnaser | smcginnis: https://review.openstack.org/#/c/530508/ | 21:23 |
smcginnis | mnaser: Thanks! | 21:23 |
mnaser | no problem! thanks for your input | 21:23 |
mnaser | let me related-bug to cinder bug i guess | 21:23 |
*** jmlowe has joined #openstack-cinder | 21:37 | |
*** AlexeyAbashkin has joined #openstack-cinder | 22:21 | |
*** AlexeyAbashkin has quit IRC | 22:25 | |
*** nicolasbock has quit IRC | 22:26 | |
*** jose-phillips has quit IRC | 22:34 | |
*** _ix has quit IRC | 22:39 | |
*** nicolasbock has joined #openstack-cinder | 22:39 | |
*** rsjethani has quit IRC | 22:46 | |
*** edmondsw has joined #openstack-cinder | 22:57 | |
*** edmondsw has quit IRC | 23:02 | |
*** e0ne has joined #openstack-cinder | 23:07 | |
*** e0ne has quit IRC | 23:09 | |
*** markvoelker has joined #openstack-cinder | 23:34 | |
*** markvoelker has quit IRC | 23:39 | |
*** Faster-Fanboi has joined #openstack-cinder | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!