*** zhanglong has joined #openstack-cinder | 00:04 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder stable/rocky: RBD: Cleanup temporary file during exception https://review.opendev.org/746619 | 00:12 |
---|---|---|
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Modify default/delete volume type logic https://review.opendev.org/741498 | 00:16 |
*** zhanglong has quit IRC | 00:18 | |
*** zhanglong has joined #openstack-cinder | 00:20 | |
*** brinzhang_ has joined #openstack-cinder | 00:24 | |
*** brinzhang0 has quit IRC | 00:27 | |
openstackgerrit | Alan Bishop proposed openstack/os-brick stable/train: ScaleIO: Connection info backward compatibility https://review.opendev.org/746621 | 00:45 |
openstackgerrit | Alan Bishop proposed openstack/os-brick stable/train: Add release note for scaleio connector upgrade https://review.opendev.org/746622 | 00:45 |
*** spatel has joined #openstack-cinder | 00:50 | |
*** n-saito has joined #openstack-cinder | 01:20 | |
*** sapd1 has joined #openstack-cinder | 01:51 | |
*** zhanglong has quit IRC | 01:59 | |
*** zhanglong has joined #openstack-cinder | 02:00 | |
*** brinzhang0 has joined #openstack-cinder | 02:14 | |
*** brinzhang_ has quit IRC | 02:17 | |
openstackgerrit | Merged openstack/cinder master: Fix dfs-sdk package name in extra reqs https://review.opendev.org/726336 | 02:36 |
*** zhanglong has quit IRC | 02:46 | |
*** whoami-rajat__ has quit IRC | 02:48 | |
*** zhanglong has joined #openstack-cinder | 02:51 | |
*** sapd1 has quit IRC | 02:52 | |
*** psachin has joined #openstack-cinder | 03:37 | |
*** zhanglong has quit IRC | 03:37 | |
openstackgerrit | Alan Bishop proposed openstack/os-brick stable/train: ScaleIO: Connection info backward compatibility https://review.opendev.org/746621 | 03:38 |
openstackgerrit | Alan Bishop proposed openstack/os-brick stable/train: Add release note for scaleio connector upgrade https://review.opendev.org/746622 | 03:38 |
*** dave-mccowan has quit IRC | 03:47 | |
openstackgerrit | Merged openstack/cinder stable/ussuri: Creating snapshot on NFS backend fails https://review.opendev.org/742581 | 04:03 |
*** laurent\ has quit IRC | 04:05 | |
*** sapd1 has joined #openstack-cinder | 04:17 | |
*** sfernand has quit IRC | 04:22 | |
*** rishabhhpe has joined #openstack-cinder | 04:34 | |
*** vishalmanchanda has joined #openstack-cinder | 04:47 | |
*** zhanglong has joined #openstack-cinder | 04:50 | |
*** ajitha has joined #openstack-cinder | 04:51 | |
*** strigazi has quit IRC | 04:52 | |
*** strigazi has joined #openstack-cinder | 04:54 | |
*** ajitha has quit IRC | 05:02 | |
*** spatel has quit IRC | 05:08 | |
*** abdysn has joined #openstack-cinder | 05:11 | |
*** strigazi has quit IRC | 05:12 | |
*** m75abrams has joined #openstack-cinder | 05:21 | |
*** sapd1 has quit IRC | 05:28 | |
*** ociuhandu has joined #openstack-cinder | 05:43 | |
*** ociuhandu has quit IRC | 05:47 | |
*** zhanglong has quit IRC | 06:43 | |
*** zhanglong has joined #openstack-cinder | 06:45 | |
*** laurent\ has joined #openstack-cinder | 06:58 | |
*** rcernin has quit IRC | 07:02 | |
*** sapd1 has joined #openstack-cinder | 07:03 | |
*** zhanglong has quit IRC | 07:03 | |
*** rcernin has joined #openstack-cinder | 07:03 | |
*** zhanglong has joined #openstack-cinder | 07:07 | |
*** zhanglong has quit IRC | 07:15 | |
*** zhanglong has joined #openstack-cinder | 07:18 | |
*** lpetrut has joined #openstack-cinder | 07:23 | |
*** e0ne has joined #openstack-cinder | 07:26 | |
*** whoami-rajat__ has joined #openstack-cinder | 07:30 | |
*** sapd1 has quit IRC | 07:38 | |
*** tosky has joined #openstack-cinder | 07:40 | |
openstackgerrit | Luigi Toscano proposed openstack/python-cinderclient master: zuul functional job: drop the custom playbooks https://review.opendev.org/746237 | 07:44 |
*** rosmaita has quit IRC | 07:49 | |
*** ociuhandu has joined #openstack-cinder | 08:06 | |
*** ociuhandu has quit IRC | 08:11 | |
*** Lucas_Gray has joined #openstack-cinder | 08:13 | |
*** sapd1 has joined #openstack-cinder | 08:23 | |
*** benfelin has joined #openstack-cinder | 08:27 | |
openstackgerrit | Hui Jiang proposed openstack/cinder master: [DNM] just for test, trigger CI. https://review.opendev.org/741349 | 08:28 |
*** rcernin has quit IRC | 08:34 | |
*** zhanglong has quit IRC | 08:34 | |
*** zhanglong has joined #openstack-cinder | 08:34 | |
*** martinkennelly has joined #openstack-cinder | 08:35 | |
*** priteau has joined #openstack-cinder | 08:35 | |
*** ociuhandu has joined #openstack-cinder | 08:37 | |
*** zhanglong has quit IRC | 08:38 | |
*** zhanglong has joined #openstack-cinder | 08:40 | |
*** zhanglong has quit IRC | 08:51 | |
*** aarora06 has joined #openstack-cinder | 08:54 | |
*** zhanglong has joined #openstack-cinder | 08:55 | |
*** rishabhhpe has quit IRC | 08:58 | |
*** rishabhhpe has joined #openstack-cinder | 08:58 | |
*** rishabhhpe has quit IRC | 09:08 | |
*** sapd1 has quit IRC | 09:09 | |
*** rishabhhpe has joined #openstack-cinder | 09:10 | |
*** benfelin has quit IRC | 09:33 | |
*** brinzhang_ has joined #openstack-cinder | 09:34 | |
*** rishabhhpe has quit IRC | 09:36 | |
*** brinzhang0 has quit IRC | 09:37 | |
*** rishabhhpe has joined #openstack-cinder | 09:37 | |
*** martinkennelly has quit IRC | 09:38 | |
openstackgerrit | Luigi Toscano proposed openstack/cinder stable/ussuri: Brocade: Fix AttributeError when raising exception https://review.opendev.org/746535 | 09:44 |
openstackgerrit | Luigi Toscano proposed openstack/cinder stable/ussuri: Brocade: Fix lookup UnboundLocalError https://review.opendev.org/746536 | 09:44 |
*** benfelin has joined #openstack-cinder | 09:47 | |
*** Luzi has joined #openstack-cinder | 09:53 | |
*** tkajinam has quit IRC | 09:58 | |
*** zhanglong has quit IRC | 10:05 | |
*** chhagarw has joined #openstack-cinder | 10:14 | |
*** sapd1 has joined #openstack-cinder | 10:14 | |
*** martinkennelly has joined #openstack-cinder | 10:15 | |
*** ociuhandu has quit IRC | 10:17 | |
*** ociuhandu has joined #openstack-cinder | 10:18 | |
*** aarora06 has quit IRC | 10:25 | |
*** whoami-rajat__ has quit IRC | 10:38 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Replacing generations with snap_ids https://review.opendev.org/740433 | 10:53 |
*** manoj_kumar_kata has joined #openstack-cinder | 11:26 | |
manoj_kumar_kata | https://review.opendev.org/#/c/745576/ | 11:31 |
*** Lucas_Gray has quit IRC | 11:32 | |
manoj_kumar_kata | @sean mcginnis: could you please review this change for ibm storwize driver | 11:32 |
*** brinzhang0 has joined #openstack-cinder | 11:34 | |
*** brinzhang_ has quit IRC | 11:36 | |
*** dave-mccowan has joined #openstack-cinder | 12:03 | |
*** rosmaita has joined #openstack-cinder | 12:04 | |
*** rosmaita has quit IRC | 12:04 | |
smcginnis | manoj_kumar_kata: There doesn't appear to be third party CI results on that patch yet. Best to wait to ask for reviews until that happens. | 12:07 |
*** enriquetaso has joined #openstack-cinder | 12:08 | |
*** rosmaita has joined #openstack-cinder | 12:13 | |
*** zzzeek has quit IRC | 12:19 | |
*** amar7ibm has joined #openstack-cinder | 12:22 | |
*** zzzeek has joined #openstack-cinder | 12:23 | |
*** amar7ibm49 has joined #openstack-cinder | 12:25 | |
*** amar7ibm has quit IRC | 12:25 | |
*** amar7ibm49 has quit IRC | 12:25 | |
*** amar7ibm has joined #openstack-cinder | 12:26 | |
*** rishabhhpe has quit IRC | 12:26 | |
*** rishabhhpe has joined #openstack-cinder | 12:27 | |
*** manoj_kumar_kata has quit IRC | 12:31 | |
*** Lucas_Gray has joined #openstack-cinder | 12:42 | |
*** enriquetaso has quit IRC | 12:50 | |
*** rishabhhpe has quit IRC | 12:52 | |
*** rishabhhpe has joined #openstack-cinder | 12:52 | |
openstackgerrit | Luigi Toscano proposed openstack/python-cinderclient master: Bump pylint to 2.3.0 (same version used by cinder) https://review.opendev.org/746705 | 12:53 |
*** psachin has quit IRC | 12:57 | |
*** whoami-rajat__ has joined #openstack-cinder | 12:59 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder stable/rocky: RBD: Cleanup temporary file during exception https://review.opendev.org/746619 | 13:00 |
*** sfernand has joined #openstack-cinder | 13:04 | |
*** priteau has quit IRC | 13:06 | |
*** manoj_kumar_kata has joined #openstack-cinder | 13:11 | |
*** enriquetaso has joined #openstack-cinder | 13:13 | |
*** brinzhang_ has joined #openstack-cinder | 13:13 | |
*** brinzhang0 has quit IRC | 13:17 | |
*** manoj_kumar_kata has quit IRC | 13:17 | |
*** manoj_kumar_kata has joined #openstack-cinder | 13:17 | |
*** sapd1 has quit IRC | 13:20 | |
*** amar7ibm has quit IRC | 13:30 | |
*** Luzi has quit IRC | 13:34 | |
*** vishakha has joined #openstack-cinder | 13:43 | |
*** rishabhhpe has quit IRC | 13:46 | |
*** tkajinam has joined #openstack-cinder | 13:50 | |
*** jv_ has quit IRC | 13:54 | |
*** abdysn has quit IRC | 14:04 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Replacing generations with snap_ids https://review.opendev.org/740433 | 14:09 |
*** rosmaita has quit IRC | 14:18 | |
*** brinzhang0 has joined #openstack-cinder | 14:19 | |
*** davee_ has joined #openstack-cinder | 14:20 | |
*** brinzhang_ has quit IRC | 14:21 | |
devfaz | hi, is there a way to "flatten" volumes created from a snapshot (ceph backend)? | 14:21 |
*** rosmaita has joined #openstack-cinder | 14:22 | |
*** davee_ has quit IRC | 14:22 | |
devfaz | if im reading the code correctly, then openstack is not saving any information about the "im based on a snapshot"-state, so doing the flatten with ceph-tools should be ok? Is someone able to confirm this? | 14:30 |
eharney | devfaz: there is an "rbd_flatten_volume_from_snapshot" option for the rbd driver which does this | 14:33 |
*** mvorwerk has joined #openstack-cinder | 14:34 | |
devfaz | eharney: yes, but i have an existing volume, this setting is used for newly created ones :( | 14:36 |
*** rosmaita has quit IRC | 14:47 | |
*** lpetrut has quit IRC | 14:48 | |
*** rosmaita has joined #openstack-cinder | 14:51 | |
openstackgerrit | Luigi Toscano proposed openstack/python-cinderclient master: zuul functional job: drop the custom playbooks https://review.opendev.org/746237 | 15:04 |
*** priteau has joined #openstack-cinder | 15:04 | |
*** vishalmanchanda has quit IRC | 15:06 | |
*** sapd1 has joined #openstack-cinder | 15:12 | |
*** lpetrut has joined #openstack-cinder | 15:23 | |
*** mvorwerk has quit IRC | 15:24 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Replacing generations with snap_ids https://review.opendev.org/740433 | 15:40 |
*** brinzhang_ has joined #openstack-cinder | 15:48 | |
*** brinzhang0 has quit IRC | 15:51 | |
*** m75abrams has quit IRC | 16:01 | |
openstackgerrit | Simon O'Donovan proposed openstack/cinder master: PowerMax Driver - Force add rep group volume https://review.opendev.org/746758 | 16:20 |
*** brinzhang0 has joined #openstack-cinder | 16:39 | |
*** brinzhang_ has quit IRC | 16:42 | |
*** Lucas_Gray has quit IRC | 16:43 | |
*** ajitha has joined #openstack-cinder | 16:44 | |
*** whoami-rajat__ has quit IRC | 16:48 | |
*** sapd1 has quit IRC | 16:54 | |
*** ociuhandu has quit IRC | 17:03 | |
openstackgerrit | Andrei Perepiolkin proposed openstack/cinder master: [WIP] Add Cinder driver for Open-E JovianDSS data storage https://review.opendev.org/726906 | 17:12 |
*** ociuhandu has joined #openstack-cinder | 17:17 | |
*** ociuhandu has quit IRC | 17:22 | |
*** lpetrut has quit IRC | 17:29 | |
*** manoj_kumar_kata has quit IRC | 18:12 | |
*** _erlon_ has joined #openstack-cinder | 18:17 | |
*** jopdorp has quit IRC | 19:01 | |
*** arne_wiebalck has quit IRC | 19:01 | |
*** rajinir has quit IRC | 19:01 | |
*** ajitha has quit IRC | 19:01 | |
*** jopdorp has joined #openstack-cinder | 19:04 | |
*** rajinir has joined #openstack-cinder | 19:08 | |
*** arne_wiebalck has joined #openstack-cinder | 19:09 | |
hemna | anyone know what the volume_type_projects table is used for ? | 19:20 |
hemna | I'm seeing some strange data in it for the deleted column | 19:20 |
hemna | http://paste.openstack.org/show/796923/ | 19:20 |
hemna | would seem to break this: https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L454 | 19:21 |
smcginnis | hemna: What does that break? | 19:33 |
hemna | I dunno, but looking at the code, the values should be 0 or 1 afaik | 19:33 |
smcginnis | Well, 0 or non-0. | 19:33 |
smcginnis | Not sure why the deleted value would be incremental, but... | 19:34 |
hemna | matches the id ? | 19:34 |
smcginnis | Could be what it's doing. | 19:34 |
hemna | hrmm, production db looks like it matches the id column | 19:35 |
hemna | volume_types has a deleted column as well, but it's a 0 or a 1 only | 19:36 |
smcginnis | Probably just coincidence that they both increment sequentially. | 19:36 |
smcginnis | Original patch where it was added: https://review.opendev.org/#/c/114395/ | 19:36 |
hemna | looks like other tables just have a 0 or 1 for deleted column | 19:37 |
hemna | looks like that's the only place that uses soft_delete() | 19:40 |
smcginnis | It came from nova, and I know they were handling deletes of things differently than we do. | 19:41 |
*** mvorwerk has joined #openstack-cinder | 19:41 | |
openstackgerrit | Ajitha Robert proposed openstack/cinder master: Add multiattach in Nimble driver https://review.opendev.org/740125 | 19:43 |
*** priteau has quit IRC | 19:44 | |
openstackgerrit | Andrei Perepiolkin proposed openstack/cinder master: [WIP] Add Cinder driver for Open-E JovianDSS data storage https://review.opendev.org/726906 | 19:44 |
*** brinzhang_ has joined #openstack-cinder | 19:44 | |
hemna | I think I might have found the problem | 19:46 |
hemna | Notice: | 19:47 |
hemna | https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L47 | 19:47 |
hemna | vs | 19:47 |
*** brinzhang0 has quit IRC | 19:47 | |
hemna | https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L446 | 19:47 |
hemna | oopsie | 19:47 |
hemna | :( | 19:48 |
hemna | heh | 19:51 |
hemna | https://bugs.launchpad.net/cinder/+bug/1496747 | 19:51 |
openstack | Launchpad bug 1496747 in Cinder "Removing project access to volume type will not work after 128+ access rule" [Medium,Fix released] - Assigned to Michal Dulko (michal-dulko-f) | 19:51 |
*** vishakha has quit IRC | 19:52 | |
hemna | that 'fix' just extended the range to int sizes, instead of tinyint | 19:52 |
hemna | so, that fix was wrong to do | 19:54 |
hemna | all other tables that have a deleted column are tinyint | 19:54 |
hemna | but they act as a boolean, because the model declares it as a boolean | 19:54 |
hemna | instead of an int. | 19:54 |
*** ociuhandu has joined #openstack-cinder | 19:58 | |
smcginnis | We should remove the boolean constraint and have a migration that changes anything > 1 to be 1. | 20:01 |
hemna | yah it needs to be changed back to tinyint, then data migrated, then redeclared as Boolean in code | 20:01 |
smcginnis | I wonder if the soft_delete needs that to match the ID to be able to do some sort of "delete anything older than X" type operation. | 20:02 |
*** ociuhandu has quit IRC | 20:02 | |
smcginnis | But since we never actually implemented soft delete management in cinder, not sure if it really matters. | 20:02 |
hemna | I can't find where that soft_delete is defined | 20:02 |
hemna | https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L44-L45 | 20:03 |
*** martinkennelly has quit IRC | 20:04 | |
*** martinkennelly has joined #openstack-cinder | 20:04 | |
smcginnis | Looks like it is some sort of "move to trash" / "restore from trash" type mechanism. | 20:07 |
smcginnis | Like "oops, I didn't mean to delete all my instances, I better get those back quick" | 20:07 |
*** e0ne has quit IRC | 20:21 | |
*** brinzhang0 has joined #openstack-cinder | 20:25 | |
*** e0ne has joined #openstack-cinder | 20:26 | |
*** brinzhang_ has quit IRC | 20:28 | |
*** e0ne has quit IRC | 20:33 | |
*** martinkennelly has quit IRC | 20:57 | |
*** mvorwerk has quit IRC | 20:59 | |
*** Lucas_Gray has joined #openstack-cinder | 21:03 | |
*** _erlon_ has quit IRC | 21:12 | |
*** JaySee has quit IRC | 21:17 | |
*** rcernin has joined #openstack-cinder | 21:18 | |
*** JaySee has joined #openstack-cinder | 21:18 | |
*** rcernin has quit IRC | 21:35 | |
*** chhagarw has quit IRC | 21:39 | |
*** enriquetaso has quit IRC | 21:39 | |
*** rcernin has joined #openstack-cinder | 21:52 | |
*** brinzhang_ has joined #openstack-cinder | 21:54 | |
*** brinzhang0 has quit IRC | 21:56 | |
*** rcernin has quit IRC | 21:57 | |
*** rcernin has joined #openstack-cinder | 22:08 | |
*** rcernin has quit IRC | 22:09 | |
*** brinzhang0 has joined #openstack-cinder | 22:14 | |
*** brinzhang_ has quit IRC | 22:17 | |
*** tosky has quit IRC | 22:18 | |
*** rcernin has joined #openstack-cinder | 22:33 | |
*** lxkong has quit IRC | 22:42 | |
*** patrickeast has quit IRC | 22:42 | |
*** sri_ has quit IRC | 22:42 | |
*** dviroel has quit IRC | 22:43 | |
*** knikolla has quit IRC | 22:43 | |
*** sri_ has joined #openstack-cinder | 22:51 | |
*** patrickeast has joined #openstack-cinder | 22:54 | |
*** dviroel has joined #openstack-cinder | 22:55 | |
*** lxkong has joined #openstack-cinder | 22:55 | |
*** knikolla has joined #openstack-cinder | 22:55 | |
openstackgerrit | Merged openstack/python-cinderclient stable/train: Fix typo: dow -> down https://review.opendev.org/744858 | 22:56 |
*** Lucas_Gray has quit IRC | 23:05 | |
openstackgerrit | Lucio Seki proposed openstack/cinder master: NetApp SolidFire: Enable driver IPv6 api request https://review.opendev.org/746813 | 23:08 |
*** benfelin has quit IRC | 23:11 | |
*** brinzhang_ has joined #openstack-cinder | 23:44 | |
*** brinzhang0 has quit IRC | 23:47 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!