*** dpawlik has joined #openstack-cinder | 00:00 | |
*** dpawlik has quit IRC | 00:04 | |
*** zigo has quit IRC | 00:05 | |
*** zigo has joined #openstack-cinder | 00:09 | |
*** mriedem_away has quit IRC | 00:10 | |
*** mvkr has quit IRC | 00:27 | |
*** lixiaoy1 has joined #openstack-cinder | 00:29 | |
*** brinzhang has joined #openstack-cinder | 00:31 | |
*** icey has quit IRC | 00:36 | |
*** mvkr has joined #openstack-cinder | 00:38 | |
*** LiangFang has joined #openstack-cinder | 00:42 | |
*** icey has joined #openstack-cinder | 01:02 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder master: VMware: Implement retype https://review.openstack.org/604948 | 01:05 |
---|---|---|
*** erlon has quit IRC | 01:21 | |
*** zhaochao has joined #openstack-cinder | 01:22 | |
*** swamireddy has joined #openstack-cinder | 01:31 | |
*** swamireddy has joined #openstack-cinder | 01:31 | |
openstackgerrit | Sam Morrison proposed openstack/cinder master: Include availability-zone of a pool in get-pools API request https://review.openstack.org/599866 | 01:35 |
*** mmethot has quit IRC | 01:41 | |
*** Dinesh_Bhor has joined #openstack-cinder | 01:41 | |
*** Bhujay has joined #openstack-cinder | 02:29 | |
*** dave-mccowan has quit IRC | 02:29 | |
*** Bhujay has quit IRC | 02:30 | |
*** Bhujay has joined #openstack-cinder | 02:30 | |
*** sapd1_ has quit IRC | 02:37 | |
*** sapd1 has joined #openstack-cinder | 02:39 | |
*** psachin has joined #openstack-cinder | 02:43 | |
*** hoangcx has quit IRC | 02:45 | |
*** hoangcx has joined #openstack-cinder | 02:45 | |
*** imacdonn has quit IRC | 02:50 | |
*** dave-mccowan has joined #openstack-cinder | 02:50 | |
*** imacdonn has joined #openstack-cinder | 02:50 | |
openstackgerrit | Merged openstack/cinder master: api-ref: mark name as optional in volume create API https://review.openstack.org/604803 | 03:32 |
openstackgerrit | Merged openstack/cinder stable/pike: VMware: Improve scalability of querying volumes https://review.openstack.org/604408 | 03:33 |
*** Bhujay has quit IRC | 03:41 | |
openstackgerrit | iain MacDonnell proposed openstack/os-brick master: Don't give up on iscsiadm -m session https://review.openstack.org/604961 | 03:42 |
*** Dinesh_Bhor has joined #openstack-cinder | 03:49 | |
LiangFang | Does any export know how to enable python3 for cinder? I mean if the environment only have python3 installed, how to run cinder? thanks. | 03:50 |
*** dpawlik has joined #openstack-cinder | 04:00 | |
*** hoangcx has quit IRC | 04:01 | |
*** Dinesh_Bhor has quit IRC | 04:01 | |
*** hoangcx has joined #openstack-cinder | 04:01 | |
*** dpawlik has quit IRC | 04:05 | |
*** vivsoni has quit IRC | 04:08 | |
*** vivsoni has joined #openstack-cinder | 04:18 | |
*** rcernin has quit IRC | 04:24 | |
*** e0ne has joined #openstack-cinder | 04:28 | |
*** Dinesh_Bhor has joined #openstack-cinder | 04:34 | |
*** Bhujay has joined #openstack-cinder | 04:37 | |
*** rcernin has joined #openstack-cinder | 04:37 | |
*** vivsoni has quit IRC | 04:48 | |
*** e0ne has quit IRC | 04:51 | |
*** vivsoni has joined #openstack-cinder | 04:58 | |
*** Dinesh_Bhor has quit IRC | 05:00 | |
openstackgerrit | Vivek Soni proposed openstack/cinder stable/rocky: 3PAR: Update Storage Driver docs https://review.openstack.org/603448 | 05:03 |
*** Dinesh_Bhor has joined #openstack-cinder | 05:08 | |
*** e0ne has joined #openstack-cinder | 05:09 | |
*** e0ne has quit IRC | 05:10 | |
*** rcernin_ has joined #openstack-cinder | 05:17 | |
*** lbragstad has quit IRC | 05:18 | |
*** rcernin has quit IRC | 05:19 | |
*** lbragstad has joined #openstack-cinder | 05:22 | |
*** lbragstad has left #openstack-cinder | 05:23 | |
*** vivsoni has quit IRC | 05:33 | |
*** pcaruana has joined #openstack-cinder | 05:41 | |
*** vivsoni has joined #openstack-cinder | 05:44 | |
*** vivsoni has quit IRC | 05:49 | |
*** vivsoni has joined #openstack-cinder | 05:49 | |
*** belmoreira has joined #openstack-cinder | 05:59 | |
*** Bhujay has quit IRC | 06:00 | |
*** Bhujay has joined #openstack-cinder | 06:14 | |
*** Bhujay has quit IRC | 06:15 | |
*** Bhujay has joined #openstack-cinder | 06:16 | |
*** dpawlik has joined #openstack-cinder | 06:21 | |
openstackgerrit | LisaLi proposed openstack/cinder-specs master: Driver reinitialization after failure https://review.openstack.org/599505 | 06:21 |
*** brinzh has joined #openstack-cinder | 06:29 | |
*** Bhujay has quit IRC | 06:32 | |
*** brinzhang has quit IRC | 06:32 | |
*** Bhujay has joined #openstack-cinder | 06:33 | |
*** Luzi has joined #openstack-cinder | 06:53 | |
*** lpetrut has joined #openstack-cinder | 06:54 | |
*** lpetrut has quit IRC | 06:56 | |
*** lpetrut has joined #openstack-cinder | 06:56 | |
*** strigazi has joined #openstack-cinder | 06:56 | |
*** rcernin_ has quit IRC | 07:05 | |
*** icey has quit IRC | 07:05 | |
*** Dinesh_Bhor has quit IRC | 07:07 | |
*** Dinesh_Bhor has joined #openstack-cinder | 07:36 | |
*** psachin has quit IRC | 07:41 | |
*** alexchadin has joined #openstack-cinder | 07:42 | |
*** psachin has joined #openstack-cinder | 07:49 | |
*** e0ne has joined #openstack-cinder | 07:52 | |
*** e0ne has quit IRC | 07:53 | |
*** Dinesh_Bhor has quit IRC | 08:28 | |
*** jiaopengju has quit IRC | 08:42 | |
*** jiaopengju has joined #openstack-cinder | 08:42 | |
*** Dinesh_Bhor has joined #openstack-cinder | 08:53 | |
*** lixiaoy1 has quit IRC | 08:53 | |
*** lixiaoy1 has joined #openstack-cinder | 08:54 | |
openstackgerrit | Merged openstack/cinder master: Add policy granularity to the encryption API https://review.openstack.org/571563 | 08:55 |
*** e0ne has joined #openstack-cinder | 08:59 | |
*** lixiaoy1 has quit IRC | 09:02 | |
*** dpawlik has quit IRC | 09:05 | |
*** dpawlik has joined #openstack-cinder | 09:06 | |
*** zhubx007 has joined #openstack-cinder | 09:06 | |
*** zhubx007 has quit IRC | 09:07 | |
*** alexchadin has quit IRC | 09:12 | |
*** alexchadin has joined #openstack-cinder | 09:16 | |
*** Emine has joined #openstack-cinder | 09:18 | |
*** vivsoni has quit IRC | 09:22 | |
*** jackyzhu has joined #openstack-cinder | 09:25 | |
*** jackyzhu has quit IRC | 09:29 | |
*** vivsoni has joined #openstack-cinder | 09:32 | |
openstackgerrit | suguangfeng proposed openstack/cinder master: Fix create volume error https://review.openstack.org/605020 | 09:35 |
*** viveksoni has joined #openstack-cinder | 09:57 | |
*** moshele has joined #openstack-cinder | 10:02 | |
*** viveksoni has quit IRC | 10:09 | |
*** e0ne has quit IRC | 10:12 | |
*** luizbag has joined #openstack-cinder | 10:14 | |
*** LiangFang has quit IRC | 10:22 | |
*** Bhujay has quit IRC | 10:31 | |
*** Bhujay has joined #openstack-cinder | 10:32 | |
*** rcernin_ has joined #openstack-cinder | 10:43 | |
*** rcernin_ has quit IRC | 10:52 | |
*** alexchadin has quit IRC | 10:52 | |
*** Dinesh_Bhor has quit IRC | 10:56 | |
*** e0ne has joined #openstack-cinder | 10:58 | |
*** erlon has joined #openstack-cinder | 11:00 | |
*** ganso has joined #openstack-cinder | 11:03 | |
*** pcaruana has quit IRC | 11:15 | |
*** e0ne_ has joined #openstack-cinder | 11:22 | |
*** e0ne has quit IRC | 11:25 | |
*** raghavendrat has joined #openstack-cinder | 11:25 | |
*** moshele has quit IRC | 11:27 | |
*** brinzh has quit IRC | 11:29 | |
*** moshele has joined #openstack-cinder | 11:33 | |
tobias-urdin | smcginnis: sorry for ping, are you here? | 11:55 |
*** alexchadin has joined #openstack-cinder | 11:56 | |
tobias-urdin | having a issue with rocky, horizon uses cinderclient that calls the cinder v3 api with these requests | 11:57 |
tobias-urdin | http://paste.openstack.org/show/730695/ | 11:57 |
tobias-urdin | but cinder responds with 400 bad request HTTP exception thrown: Invalid filters status are found in query options. | 11:57 |
tobias-urdin | is this a bug or is the status field query option removed? or maybe cinder api has started enforcing query options and cinderclient was not updated? | 11:58 |
tobias-urdin | http://localhost:8776/v3/3fad4eac76ae4f3fb8df25ce4e911a3c/snapshots/detail?status=available | 11:59 |
tobias-urdin | same for this call /v3/<project>/volumes/detail?bootable=1&status=available | 12:00 |
tobias-urdin | HTTP exception thrown: Invalid filters bootable,status are found in query options. | 12:00 |
*** psachin has quit IRC | 12:07 | |
tobias-urdin | jungleboyj: sorry for ping, want to verify if this is intentional or a regression, a kind of big one if it is | 12:12 |
openstackgerrit | Shay Halsband proposed openstack/cinder stable/queens: XtremIO: support multiattach https://review.openstack.org/605051 | 12:17 |
tobias-urdin | found it, probably a packaging issue /etc/cinder/resource_filters.json is not there | 12:18 |
tobias-urdin | sorry for the noice | 12:18 |
*** raghavendrat has quit IRC | 12:19 | |
*** tpsilva has joined #openstack-cinder | 12:24 | |
*** vivsoni has quit IRC | 12:26 | |
*** Bhujay has quit IRC | 12:31 | |
*** Bhujay has joined #openstack-cinder | 12:32 | |
*** Bhujay has quit IRC | 12:33 | |
*** Bhujay has joined #openstack-cinder | 12:33 | |
*** lpetrut has quit IRC | 12:33 | |
*** alexchadin has quit IRC | 12:36 | |
*** lpetrut has joined #openstack-cinder | 12:36 | |
*** alexchadin has joined #openstack-cinder | 12:37 | |
*** alexchadin has quit IRC | 12:37 | |
*** alexchadin has joined #openstack-cinder | 12:37 | |
*** alexchadin has quit IRC | 12:38 | |
*** alexchadin has joined #openstack-cinder | 12:38 | |
*** alexchadin has quit IRC | 12:38 | |
*** alexchadin has joined #openstack-cinder | 12:39 | |
*** alexchadin has quit IRC | 12:39 | |
*** amoralej has joined #openstack-cinder | 12:47 | |
*** alexchadin has joined #openstack-cinder | 12:49 | |
*** alexchadin has quit IRC | 12:54 | |
*** lbragstad has joined #openstack-cinder | 12:59 | |
*** belmoreira has quit IRC | 13:05 | |
*** lbragstad has quit IRC | 13:09 | |
*** alexchadin has joined #openstack-cinder | 13:16 | |
*** icey has joined #openstack-cinder | 13:28 | |
*** dustins has joined #openstack-cinder | 13:28 | |
*** belmoreira has joined #openstack-cinder | 13:29 | |
luizbag | smcginnis, jungleboyj could we get some reviews on this: https://review.openstack.org/#/c/599720, please? | 13:30 |
*** jdillaman has quit IRC | 13:34 | |
*** jdillaman has joined #openstack-cinder | 13:38 | |
sayalilunkad | hi! can anyone help me understand what the provider_location in the cinder volume db is supposed to be? | 13:48 |
openstackgerrit | Tiago Pasqualini da Silva proposed openstack/cinder stable/queens: Fix IPv6 for Cinder NetApp ONTAP drivers https://review.openstack.org/605082 | 14:04 |
*** alexchadin has quit IRC | 14:07 | |
openstackgerrit | Maciej Szwed proposed openstack/cinder master: Adding SPDK NVMe-oF target driver https://review.openstack.org/572759 | 14:08 |
openstackgerrit | Maciej Szwed proposed openstack/cinder master: Adding SPDK volume driver https://review.openstack.org/564229 | 14:08 |
*** e0ne_ has quit IRC | 14:10 | |
*** e0ne has joined #openstack-cinder | 14:10 | |
*** alexchadin has joined #openstack-cinder | 14:12 | |
*** LiangFang has joined #openstack-cinder | 14:13 | |
*** alexchadin has quit IRC | 14:17 | |
*** alexchadin has joined #openstack-cinder | 14:19 | |
*** mriedem has joined #openstack-cinder | 14:26 | |
*** Luzi has quit IRC | 14:33 | |
*** lixiaoy1 has joined #openstack-cinder | 14:33 | |
*** moshele has quit IRC | 14:35 | |
*** vivsoni has joined #openstack-cinder | 14:42 | |
*** yikun has quit IRC | 14:47 | |
*** dpawlik has quit IRC | 14:52 | |
*** zhaochao has quit IRC | 14:55 | |
*** dpawlik has joined #openstack-cinder | 14:57 | |
*** alexchadin has quit IRC | 14:57 | |
LiangFang | @smcginnis regarding https://review.openstack.org/#/c/602571/, I added some comments, would you mind to give a review again?:) | 15:00 |
*** Bhujay has quit IRC | 15:03 | |
*** david-lyle has quit IRC | 15:04 | |
*** lpetrut has quit IRC | 15:04 | |
*** dklyle has joined #openstack-cinder | 15:05 | |
*** dave-mccowan has quit IRC | 15:14 | |
*** sapd1_ has joined #openstack-cinder | 15:19 | |
*** dpawlik has quit IRC | 15:25 | |
*** LiangFang has quit IRC | 15:26 | |
*** e0ne has quit IRC | 15:37 | |
smcginnis | sayalilunkad: provider_location is up to each driver what they want to use it for. | 15:38 |
smcginnis | sayalilunkad: Some use it to store an internal storage device identifier, some stuff other useful information in there, some don't use it at all. | 15:38 |
smcginnis | Really up to the driver maintainer if they want to use it and what they want to use it for, | 15:38 |
*** sapd1_ has quit IRC | 15:42 | |
sayalilunkad | smcginnis: oh ok, thanks! | 15:44 |
imacdonn | geguileo: you around? | 15:45 |
geguileo | barely XD | 15:45 |
imacdonn | geguileo: :) Wonder if you saw my updates on https://review.openstack.org/#/c/604961/ ? | 15:46 |
imacdonn | er I mean https://bugs.launchpad.net/nova/+bug/1732199 | 15:46 |
openstack | Launchpad bug 1732199 in OpenStack Compute (nova) "test_extend_attached_volume fails with Unexpected compute_extend_volume result 'Error'" [Medium,Confirmed] | 15:46 |
geguileo | no, I didn't look at the bug today | 15:47 |
imacdonn | The short version is that os-brick is looking at the stderr of 'iscsiadm -m session' and deciding that it failed, when it actually only produced some warning-type text ... the exit status was 0 ... and this happens when another session is being established at the same time | 15:48 |
imacdonn | my assertion is that it should rely on the exit status, log the warning as a warning, and move on | 15:49 |
*** dustins has quit IRC | 15:49 | |
geguileo | imacdonn: were is the other connection being stablished? on the same service? | 15:49 |
imacdonn | but I don't know if there's some case where stderr, with an otherwise acceptable exit status, is bad | 15:49 |
imacdonn | geguileo: no, it was from cinder-backup, at least in the case of the failure case in the bug | 15:50 |
mriedem | imacdonn: ack, read your updates on the bug report, thanks for digging into that | 15:50 |
geguileo | the problem is that os-brick is not using file locks | 15:51 |
mriedem | so cinder-backup is running some periodic task or something? and that's why we hit this intermittently in the gate? | 15:51 |
imacdonn | I assume it's just another test that tempest is running in parallel | 15:51 |
geguileo | cinder-backup should not have any periodic task afaik | 15:51 |
geguileo | yup, that could be the case, os-brick should be using file locks | 15:51 |
geguileo | and iirc it did in the past | 15:51 |
geguileo | I don't know who removed them, or if I am imagining us using file locks in os-brick | 15:52 |
*** belmoreira has quit IRC | 15:52 | |
mriedem | 2018-09-24 10:56:00.195771 | controller | {2} tempest.api.volume.test_volumes_backup.VolumesBackupsV39Test.test_update_backup [40.188674s] ... ok | 15:53 |
mriedem | 2018-09-24 10:55:33.251780 | controller | {1} tempest.api.volume.test_volumes_extend.VolumesExtendAttachedTest.test_extend_attached_volume [25.490844s] ... FAILED | 15:53 |
mriedem | there was a backup test running around the same time as the failed in-use extend test | 15:53 |
imacdonn | it was probably the next backup test after that | 15:53 |
geguileo | I'm looking at why we don't have file locks | 15:54 |
imacdonn | oh no, that's right | 15:54 |
mriedem | ah here | 15:54 |
mriedem | 2018-09-24 10:54:56.689173 | controller | {2} tempest.api.volume.admin.test_volumes_backup.VolumesBackupsAdminTest.test_volume_backup_reset_status [42.044665s] ... ok | 15:54 |
mriedem | 2018-09-24 10:55:33.251780 | controller | {1} tempest.api.volume.test_volumes_extend.VolumesExtendAttachedTest.test_extend_attached_volume [25.490844s] ... FAILED | 15:54 |
imacdonn | failure was at 10:55:26 | 15:54 |
mriedem | sure, anyway, there was a backup test before and after the failure in the extend attached volume test | 15:54 |
imacdonn | geguileo: didn't you (or someone) rework a lot of that code to allow it to work concurrently? the stuff around the manual rescan, etc. | 15:57 |
geguileo | imacdonn: I reworked it so that it would be more robust and didn't fail all the time | 15:57 |
geguileo | imacdonn: I was going to work on the concurrency on this release | 15:58 |
imacdonn | ah ok | 15:58 |
geguileo | and the problem is that we cannot have Nova and Cinder running on the same nova | 15:58 |
geguileo | and we need to use file locks in os-brick | 15:58 |
geguileo | "on the same node" | 15:58 |
geguileo | not on the same nova | 15:59 |
imacdonn | wouldn't this be an issue if two nova threads tried to do iscsiadm things in parallel too? | 15:59 |
geguileo | nop | 15:59 |
geguileo | we have locks | 15:59 |
geguileo | the problem is that they are just locks within the process | 16:00 |
*** lixiaoy1 has quit IRC | 16:00 | |
imacdonn | @synchronized('extend_volume') .... ok | 16:01 |
geguileo | mmmmmm, that could still be a problem | 16:02 |
geguileo | because the locks we have are for connect/disconnect | 16:02 |
geguileo | but extend uses a different one | 16:02 |
imacdonn | oh, yeah, I went to check for that, and misread it ... I need more coffee | 16:03 |
imacdonn | so yeah, that is a problem .. seems the lock should be higher-level .. like "this thing involves fiddling with the iscsi initiator" | 16:04 |
imacdonn | although ... the extend case only needs to "read" an existing session | 16:05 |
imacdonn | so if it can be made that the other tasks don't break that........ | 16:05 |
geguileo | but I'm still not sure why that command is puking... I only knew that iscsi concurrent connect/disconnect "may" run into errors | 16:05 |
imacdonn | it's not actually failing .. it's just noting, on stderr, that there's an incomplete session that it can't get the info for | 16:06 |
imacdonn | (is my interpretation) | 16:06 |
*** pcaruana has joined #openstack-cinder | 16:07 | |
imacdonn | do you know why os-brick is not just using the exit status? | 16:07 |
*** Emine has quit IRC | 16:12 | |
*** dklyle has quit IRC | 16:40 | |
*** dklyle has joined #openstack-cinder | 16:44 | |
geguileo | imacdonn: well, if it's not an error and the warning is going to stdout, then the parsing of that output will fail | 17:04 |
geguileo | imacdonn: and we need that information for that code to work iirc | 17:04 |
imacdonn | geguileo: parsing of what output ? | 17:04 |
geguileo | imacdonn: the iscsiadm -m session command | 17:04 |
imacdonn | geguileo: the output you want to parse goes to stdout, not stderr | 17:04 |
geguileo | imacdonn: and is that warning going to stderr? | 17:05 |
imacdonn | geguileo: yes | 17:05 |
jungleboyj | tobias-urdin: Thanks for the update. Sorry I didn't respond earlier. | 17:05 |
geguileo | imacdonn: then I don't know what it was checking to fail... But in other cases were that same error has happened we were not getting status 0 | 17:06 |
geguileo | though this could be a different case | 17:06 |
imacdonn | geguileo: it does the usual check_exit_code thing .. 0, 21, 255 and something else are considered OK | 17:06 |
imacdonn | check_exit_code=[0, 1, 21, 255]) | 17:07 |
geguileo | Oh, I think that was a 21 status code iirc | 17:07 |
imacdonn | 21 | 17:07 |
imacdonn | ISCSI_ERR_NO_OBJS_FOUND - no records/targets/sessions/portals found to execute operation on. | 17:07 |
imacdonn | this is more troubling, though: 1 = ISCSI_ERR - generic error code. | 17:08 |
imacdonn | geguileo: not sure why exit code 1 is considered acceptable here https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connectors/iscsi.py#L1093-L1094 | 17:10 |
geguileo | imacdonn: that, I don't know :-( | 17:24 |
geguileo | probably it's before my time on OpenStack | 17:24 |
imacdonn | geguileo: yeah, I suspect that a lot of this is due to the way iscsi behaved years ago | 17:25 |
imacdonn | geguileo: think I'm going to revise my suggested fix to remove [1] from that check_exit_code list | 17:29 |
openstackgerrit | iain MacDonnell proposed openstack/os-brick master: 'iscsiadm -m session' failure handling https://review.openstack.org/604961 | 17:32 |
*** luizbag has quit IRC | 17:35 | |
*** luizbag has joined #openstack-cinder | 17:39 | |
*** amoralej is now known as amoralej|off | 17:52 | |
openstackgerrit | Raunak Kumar proposed openstack/cinder master: nimble storage: retype support https://review.openstack.org/605166 | 18:19 |
openstackgerrit | Raunak Kumar proposed openstack/cinder master: nimble storage: retype support https://review.openstack.org/601492 | 18:23 |
openstackgerrit | Raunak Kumar proposed openstack/cinder master: nimble storage: retype support https://review.openstack.org/601492 | 18:26 |
openstackgerrit | Raunak Kumar proposed openstack/cinder master: nimble storage: retype support https://review.openstack.org/601492 | 18:28 |
openstackgerrit | Merged openstack/cinder stable/rocky: 3PAR: Update Storage Driver docs https://review.openstack.org/603448 | 18:30 |
openstackgerrit | Raunak Kumar proposed openstack/cinder master: nimble storage: retype support https://review.openstack.org/601492 | 18:37 |
*** dustins has joined #openstack-cinder | 18:48 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Fix multiattach set to false after retype https://review.openstack.org/604040 | 18:52 |
*** gouthamr has quit IRC | 18:52 | |
*** gouthamr has joined #openstack-cinder | 19:12 | |
openstackgerrit | Chuck Short proposed openstack/cinder master: Remove unecessary pass https://review.openstack.org/605173 | 19:12 |
*** e0ne has joined #openstack-cinder | 19:41 | |
*** erlon has quit IRC | 19:43 | |
*** _alastor_ has joined #openstack-cinder | 19:50 | |
*** dustins has quit IRC | 19:56 | |
*** dustins has joined #openstack-cinder | 19:57 | |
*** gouthamr_ has joined #openstack-cinder | 20:05 | |
*** e0ne has quit IRC | 20:05 | |
*** e0ne has joined #openstack-cinder | 20:19 | |
*** luizbag has quit IRC | 20:24 | |
*** gouthamr has quit IRC | 20:38 | |
*** gouthamr_ is now known as gouthamr | 20:39 | |
*** pcaruana has quit IRC | 20:43 | |
*** e0ne has quit IRC | 20:43 | |
*** dustins has quit IRC | 21:14 | |
openstackgerrit | Merged openstack/cinder stable/rocky: VMAX Driver - Initiator retrieval short hostname fix https://review.openstack.org/601595 | 21:23 |
openstackgerrit | Jon Bernard proposed openstack/cinder master: RBD: add support for multiattach https://review.openstack.org/595827 | 21:25 |
openstackgerrit | Merged openstack/cinder stable/rocky: LVM: Disable multiattach for LIO iSCSI target https://review.openstack.org/596493 | 21:27 |
mriedem | is it possible to delete a volume type if there are volumes / snapshots that are using that type? | 21:34 |
smcginnis | mriedem: No, it shouldn't be. | 21:37 |
mriedem | ok; | 21:37 |
mriedem | question came up on the nova change to support passing a volume type during bfv, | 21:37 |
mriedem | if nova should store the volume type name or id, | 21:37 |
mriedem | because if the volume type is deleted and recreated with the same name, the id would change and we could potentially fail to look it up later, | 21:38 |
mriedem | having said that, i don't know that we have any need to look it up later | 21:38 |
*** gnufied has quit IRC | 21:39 | |
*** gnufied has joined #openstack-cinder | 21:39 | |
smcginnis | Yeah, as long as that boot volume is there, ID should be safe. | 21:40 |
mriedem | this is the case that nova creates the volume using the type | 21:41 |
smcginnis | But not sure you would need it later unless doing a full rebuild. | 21:41 |
mriedem | if we get that rebuild api in cinder, rebuilding a volume-backed server shouldn't require creating a new root volume | 21:41 |
mriedem | b/c that's one of the complications with doing it that way, plus quota, etc etc | 21:41 |
mriedem | looking at DB API volume_type_destroy it looks like if there are any volumes, group_volume_type_mapping or consistencygroups that reference the volume type you get a 400 | 21:42 |
mriedem | curiously snapshots aren't in that check | 21:43 |
mriedem | but the snapshots table has a volume_type_id column | 21:43 |
mriedem | is that a bug? | 21:43 |
smcginnis | Hmm. | 21:43 |
smcginnis | There would still need to be one volume if there are snapshots, so I think that is safe. | 21:43 |
mriedem | how about encryption? | 21:44 |
mriedem | my guess is you might get a referential constraint if you try to delete a volume type that has related encryption records | 21:45 |
mriedem | oh, well, | 21:45 |
smcginnis | That might be the same kind of situation as a snapshot, but not sure. | 21:45 |
mriedem | maybe not until archive | 21:45 |
smcginnis | Hmm, but maybe with backups... | 21:45 |
mriedem | deleting the volume type just flips the deleted flag | 21:45 |
mriedem | so the encryption records would still reference the deleted type | 21:45 |
mriedem | archive would fail though | 21:46 |
mriedem | i don't have a devstack handy to test that out though | 21:46 |
mriedem | melwitt: re your earlier question about nova-consoleauth in the install guide, i don't know what the best way to call that out in the install docs as it being deprecated besides just putting a note in the section of the install guide(s) that says nova-consoleauth is deprecated; i don't know what all upgrade stuff needs to happen for that so people can stop installing it. on fresh installs i'd think it wouldn't be needed any | 21:51 |
mriedem | , but that's why i routed the bug to you. | 21:51 |
mriedem | if you actually have to run the service in rocky then it shouldn't really be deprecated | 21:51 |
mriedem | melwitt: crap wrong channel | 21:55 |
smcginnis | :D | 21:55 |
imacdonn | smcginnis: if you're back, this could use a review - not urgent, though: https://review.openstack.org/604230 | 22:05 |
imacdonn | smcginnis: also would like to get this one finished, but I assume you don't want to +2 your own code :) https://review.openstack.org/593955 | 22:06 |
*** mriedem has quit IRC | 22:10 | |
*** sorrison has joined #openstack-cinder | 22:20 | |
*** mvkr has quit IRC | 22:22 | |
*** ganso has quit IRC | 22:29 | |
*** mvkr has joined #openstack-cinder | 22:35 | |
smcginnis | imacdonn: I'm at a conference this week, so I'm kind of in and out whenever I get some down time. | 22:36 |
imacdonn | smcginnis: ack | 22:37 |
*** dklyle has quit IRC | 22:48 | |
*** tpsilva has quit IRC | 22:52 | |
openstackgerrit | Raunak Kumar proposed openstack/cinder master: nimble storage: retype support https://review.openstack.org/601492 | 22:53 |
*** rcernin has joined #openstack-cinder | 23:07 | |
*** dave-mccowan has joined #openstack-cinder | 23:20 | |
*** hoangcx has quit IRC | 23:22 | |
*** hoangcx has joined #openstack-cinder | 23:23 | |
*** erlon has joined #openstack-cinder | 23:27 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!