| *** zzzeek has quit IRC | 00:20 | |
| *** zzzeek has joined #openstack-cinder | 00:23 | |
| *** rcernin has quit IRC | 00:26 | |
| *** zenkuro has quit IRC | 00:50 | |
| *** rcernin has joined #openstack-cinder | 00:54 | |
| *** n-saito has quit IRC | 00:54 | |
| *** Liang__ has joined #openstack-cinder | 01:21 | |
| *** mpsairam has quit IRC | 01:27 | |
| *** pmannidi has joined #openstack-cinder | 01:27 | |
| *** ociuhandu has joined #openstack-cinder | 02:30 | |
| *** ociuhandu has quit IRC | 02:34 | |
| *** pmannidi has quit IRC | 03:03 | |
| *** udesale has joined #openstack-cinder | 03:05 | |
| *** Liang__ has quit IRC | 03:27 | |
| *** Liang__ has joined #openstack-cinder | 03:27 | |
| *** ircuser-1 has quit IRC | 03:49 | |
| *** psachin has joined #openstack-cinder | 03:51 | |
| *** pmannidi has joined #openstack-cinder | 04:19 | |
| *** manoj_kumar_kata has joined #openstack-cinder | 04:21 | |
| *** vishalmanchanda has joined #openstack-cinder | 04:39 | |
| *** psachin has quit IRC | 04:39 | |
| *** rcernin has quit IRC | 04:47 | |
| *** m75abrams has joined #openstack-cinder | 05:01 | |
| *** dsariel has joined #openstack-cinder | 05:02 | |
| *** abdysn has joined #openstack-cinder | 05:11 | |
| *** rcernin has joined #openstack-cinder | 05:18 | |
| *** Abdallahyas has joined #openstack-cinder | 05:32 | |
| *** abdysn has quit IRC | 05:36 | |
| *** Abdallahyas has quit IRC | 05:42 | |
| *** abdysn has joined #openstack-cinder | 05:42 | |
| *** Liang__ has quit IRC | 06:02 | |
| *** Liang__ has joined #openstack-cinder | 06:03 | |
| *** e0ne has joined #openstack-cinder | 07:01 | |
| *** tosky has joined #openstack-cinder | 07:03 | |
| *** zenkuro has joined #openstack-cinder | 07:09 | |
| *** zenkuro has quit IRC | 07:15 | |
| *** zenkuro has joined #openstack-cinder | 07:15 | |
| *** JaySee has quit IRC | 07:15 | |
| *** rcernin has quit IRC | 07:26 | |
| *** zenkuro has quit IRC | 07:30 | |
| *** zenkuro has joined #openstack-cinder | 07:30 | |
| *** sharathkacham has joined #openstack-cinder | 07:31 | |
| *** brinzhang has joined #openstack-cinder | 07:32 | |
| *** rpittau|afk is now known as rpittau | 07:35 | |
| *** benfelin has joined #openstack-cinder | 07:43 | |
| *** sharathkacham has quit IRC | 07:46 | |
| *** priteau has joined #openstack-cinder | 07:51 | |
| *** sharathkacham has joined #openstack-cinder | 07:53 | |
| *** ociuhandu has joined #openstack-cinder | 08:07 | |
| *** rcernin has joined #openstack-cinder | 08:10 | |
| *** e0ne has quit IRC | 08:22 | |
| *** e0ne_ has joined #openstack-cinder | 08:22 | |
| *** e0ne has joined #openstack-cinder | 08:23 | |
| *** e0ne_ has quit IRC | 08:23 | |
| *** rcernin has quit IRC | 08:33 | |
| *** brinzhang_ has joined #openstack-cinder | 08:33 | |
| *** brinzhang has quit IRC | 08:35 | |
| *** elod is now known as elod_pto | 08:41 | |
| zenkuro | hello guys, can you please review https://review.opendev.org/#/c/726906/, it has passed many code review and been approved by smcginnis and lseki. With last fix I removed redundant code and improved CI stability | 08:43 |
|---|---|---|
| *** manoj_kumar_kata has quit IRC | 08:51 | |
| *** manoj_kumar_kata has joined #openstack-cinder | 08:51 | |
| *** dsariel has quit IRC | 09:09 | |
| *** zenkuro has quit IRC | 09:17 | |
| *** zenkuro has joined #openstack-cinder | 09:17 | |
| *** manoj_kumar_kata has quit IRC | 09:40 | |
| *** manoj_kumar_kata has joined #openstack-cinder | 09:40 | |
| *** dsariel has joined #openstack-cinder | 09:41 | |
| *** Abdallahyas has joined #openstack-cinder | 09:46 | |
| *** abdysn has quit IRC | 09:49 | |
| *** openstackgerrit has joined #openstack-cinder | 09:50 | |
| openstackgerrit | Wenping Song proposed openstack/cinder master: Use TOX_CONSTRAINTS_FILE instead of obsolete UPPER_CONSTRAINTS_FILE https://review.opendev.org/757522 | 09:50 |
| *** Liang__ has quit IRC | 10:09 | |
| openstackgerrit | fengjiankui proposed openstack/cinder master: create encryption volume with existing secret https://review.opendev.org/757528 | 10:21 |
| openstackgerrit | fengjiankui proposed openstack/cinder master: create encryption volume with existing secret https://review.opendev.org/757528 | 10:25 |
| *** pcaruana has quit IRC | 10:31 | |
| *** lpetrut has joined #openstack-cinder | 10:36 | |
| *** Luzi has joined #openstack-cinder | 10:52 | |
| *** pcaruana has joined #openstack-cinder | 11:03 | |
| *** udesale_ has joined #openstack-cinder | 11:04 | |
| *** udesale has quit IRC | 11:07 | |
| *** brinzhang_ has quit IRC | 11:37 | |
| *** brinzhang_ has joined #openstack-cinder | 11:37 | |
| zenkuro | lseki: hi, can you please give a blessing to a patch-set https://review.opendev.org/#/c/726906/ once more? | 11:58 |
| *** priteau has quit IRC | 11:59 | |
| zenkuro | smcginnis: thanks a lot! | 12:08 |
| smcginnis | zenkuro: No problem, thanks for getting all of the comments addressed. We should be in great shape now that wallaby development is open. | 12:10 |
| zenkuro | smcginnis: now when I have +2 there should be one more person to give +2? or something else should happen?(checking if I missing something) | 12:35 |
| openstackgerrit | Ajitha Robert proposed openstack/cinder master: Add support for reverting to snapshot in the Nimble Storage iSCSI and FC drivers https://review.opendev.org/757554 | 12:38 |
| kmarc | eharney: I'm happy to see that cinder is going to be type checked now! :-) kudos! | 12:50 |
| eharney | kmarc: thanks, i'm looking forward to it :) | 12:51 |
| *** GirishChilukuri has joined #openstack-cinder | 12:53 | |
| kmarc | one tiny question, why the `import typing as ty` and ten use the "built-in" types as ty.Dict etc? I understand that import * is not favored and also that `import Dict,Whatever from typing` is a bit cumbersome, but I always do it like that; def fun() -> ty.Dict[...] looks a bit messy to me | 12:54 |
| kmarc | although, super polite! (ty == thank you) | 12:54 |
| kmarc | :-) | 12:54 |
| eharney | kmarc: i'm still a bit undecided on what style would be best, i've considered going that route too | 12:57 |
| eharney | so, no strong reason really | 12:58 |
| *** rosmaita has joined #openstack-cinder | 12:58 | |
| kmarc | I'd guess IDEs like pycharm can do the auto-import, but TBH even using plain vim (without any fancy plugins to do this kinda stuff) was sufficient for me, although small project | 13:00 |
| eharney | not sure, i stil mostly live in the vim world | 13:02 |
| eharney | i'll try posting a patch with that style too for review and see what folks think | 13:04 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: mypy: annotate volume manager (alternate style) https://review.opendev.org/757560 | 13:22 |
| hemna | man that syntax is odd/ugly | 13:26 |
| hemna | re: type checking | 13:26 |
| *** Luzi has quit IRC | 13:26 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder stable/rocky: PowerMax Driver - Legacy volumes fail to live migrate https://review.opendev.org/757563 | 13:26 |
| eharney | hemna: well, this patch may make it nicer ^ | 13:26 |
| eharney | hemna: but, any specific issues with it? | 13:26 |
| hemna | just haven't seen it before | 13:28 |
| hemna | kinda clutters things up | 13:28 |
| hemna | this might make backporting bug fixes to py2 code messy | 13:30 |
| hemna | why is there # type: ignore in some placess for self, and not others? | 13:32 |
| hemna | seems inconsistent | 13:32 |
| eharney | args aren't currently matched between the Manager and VolumeManager classes, i didn't want to get into fixing all that right out of the gate | 13:33 |
| eharney | it's as inconsistent as our code is currently :) | 13:33 |
| *** pcaruana has quit IRC | 13:35 | |
| zenkuro | rosmaita: hi, if you have some free time can you please take a look at https://review.opendev.org/#/c/726906/, Ive addressed most of an issues and improved CI stability | 13:36 |
| rosmaita | zenkuro: i was just looking, but would rather wait for lseki to make sure all his concerns were addressed | 13:37 |
| rosmaita | but if lseki tells me he is too busy today, i will review | 13:38 |
| rosmaita | zenkuro: you are talking about the new jovian dss driver, right? or am i confusing you with someone else | 13:38 |
| zenkuro | rosmaita: yes, JovianDriver | 13:39 |
| rosmaita | ok | 13:39 |
| zenkuro | rosmaita: thanks for your attention, I will wait for lseki then | 13:39 |
| rosmaita | zenkuro: feel free to ping me later if there is no action ... now that i think about it, it may be a holiday where lseki is today | 13:40 |
| zenkuro | rosmaita: ok, will ping you later if nothing changes | 13:45 |
| *** Abdallahyas has quit IRC | 14:05 | |
| *** TusharTgite has joined #openstack-cinder | 14:14 | |
| hemna | I would think type checking would be more useful between the volume manager and the drivers | 14:14 |
| hemna | the volume manager hasn't changed much in a long time | 14:14 |
| hemna | what are we trying to gain by adding type checking now in the life of cinder? | 14:15 |
| eharney | yes, it will be handy there for sure | 14:15 |
| TusharTgite | hi team | 14:16 |
| hemna | the interface between the volume manager and the driver is where I think it might be useful | 14:16 |
| TusharTgite | Please review the code changes https://review.opendev.org/#/c/749658/ need +2 workflow approvals to merge code changes. | 14:16 |
| *** m75abrams has quit IRC | 14:20 | |
| noonedeadpunk | hey everyone! I'm trying to understand why I have volume creation error when barbican is used | 14:20 |
| noonedeadpunk | http://paste.openstack.org/show/798953/ | 14:20 |
| noonedeadpunk | log from barbican http://paste.openstack.org/show/798944/ | 14:20 |
| noonedeadpunk | I'd say that it's barbican issue, but have one nuance... | 14:21 |
| noonedeadpunk | I've tried to raise here to see if store is even called here https://opendev.org/openstack/barbican/src/branch/master/barbican/plugin/resources.py#L71 but it's not | 14:21 |
| redrobot | noonedeadpunk, looks like barbican is trying to decode some data that doesn't need decoding. | 14:22 |
| noonedeadpunk | so I'm under impression that we ask for a secret without trying to store it? | 14:22 |
| noonedeadpunk | redrobot: so, um, `b'\xe2\xca7\xc2i$\x99\x8a\xb5\xf6r\x9d&@\xe6\xf1\xfbW\xfeL\x1d\xae\xa1\x8e\x1b\xe6\xea\xaa\xc0\xf5\xd3\xa1'` doesn't need decoding? | 14:25 |
| redrobot | right, that is likely the actual secret | 14:25 |
| redrobot | noonedeadpunk, len(b'\xe2\xca7\xc2i$\x99\x8a\xb5\xf6r\x9d&@\xe6\xf1\xfbW\xfeL\x1d\xae\xa1\x8e\x1b\xe6\xea\xaa\xc0\xf5\xd3\xa1') == 32, so that' | 14:26 |
| redrobot | that's a 256 byte key | 14:26 |
| noonedeadpunk | hm, let me check that | 14:26 |
| noonedeadpunk | redrobot: you're right.... | 14:30 |
| noonedeadpunk | adding here https://github.com/openstack/oslo.serialization/blob/master/oslo_serialization/base64.py#L68 try/except pass and volume is created | 14:31 |
| redrobot | noonedeadpunk, base64 strings alwas end in "==", so it's easy to tell when the data is/isn't b64-encoded. | 14:34 |
| redrobot | Hmm... it's strange to try to force ascii right there | 14:35 |
| redrobot | noonedeadpunk, base64.b64decode() expects bytes and returns bytes, so I'm not sure I understand why Oslo is trying to force ascii encoding there | 14:41 |
| redrobot | given that ascii has very particular rules about what bytes are allowed. | 14:42 |
| noonedeadpunk | redrobot: maybe kind of left from py2? | 14:45 |
| *** TusharTgite has quit IRC | 14:47 | |
| noonedeadpunk | redrobot: are you going to patch it? | 14:48 |
| lseki | hi zenkuro, I'll take a look on your patch right now | 14:51 |
| zenkuro | lseki: cool! thanks a lot! | 14:53 |
| redrobot | noonedeadpunk, yeah, I can make a patch to oslo to remove that | 14:59 |
| noonedeadpunk | redrobot: well, don't really hurry. as once I dropped it another thing raised .... | 15:00 |
| noonedeadpunk | http://paste.openstack.org/show/798959/ | 15:01 |
| redrobot | noonedeadpunk, yeah, so that's gonna happen when you have something that is not encoded and you try to decode it | 15:01 |
| noonedeadpunk | but um, `encoded = encoded.decode('ascii')` makes things work in such case | 15:02 |
| redrobot | So, key == base64.decode(base64.encode(key)), but it seems that workflow is trying to do key == base64.decode(key) which is | 15:02 |
| redrobot | obviously not right | 15:03 |
| noonedeadpunk | it's totally not... | 15:06 |
| redrobot | the decode('ascii') function takes a string that is an ascii string and turns it into the bytes | 15:07 |
| redrobot | but the input HAS to be an ascii string | 15:07 |
| redrobot | but in that workflow the input is just random bytes (ie, a key) | 15:07 |
| redrobot | so there's no way that .decode('ascii') is going to work 100% of the time | 15:07 |
| GirishChilukuri | smcginnis lseki I have addressed review comments for code changes https://review.opendev.org/#/c/751994/, Please review the code changes for +2 approvals. | 15:07 |
| redrobot | it _might_ work some times, when every byte in key is in the ascii space | 15:07 |
| redrobot | but as soo as the key as a non-ascii byte anywhere in the byte string you'll get that encoding error | 15:08 |
| redrobot | *as soon as | 15:08 |
| noonedeadpunk | redrobot: well yes, I titally agree with that, but don't see plain fix... as seems that just removal of decode to ascii is not really a solution. and adding except: pass sounds weird a bit as well:) | 15:09 |
| noonedeadpunk | and I don't see issue in barbican logic itself | 15:09 |
| redrobot | noonedeadpunk, I'd have to look into the code doing the initial storage of the secret to be sure, but my guess is that: | 15:10 |
| redrobot | When doing the secret store, the code is sending payload=base64(key) and payload_content_type=base64. | 15:10 |
| redrobot | this tells Barbican API that the thing you want to store is "key", but that it has been base64 encoded to be put into a json string. (JSON does not support literals with arbitrary bytes, so base64 is necessary) | 15:11 |
| redrobot | so Barbican API does secret_payload=base64.decode(incoming_payload) | 15:12 |
| redrobot | and then later when you retrieve it from barbican the code is expecting base64(key) but barbican is only sending back key | 15:12 |
| redrobot | the whole ascii thing in oslo seems bad to me. especially because every string in Python 3 is UTF-8, not ascii. | 15:13 |
| redrobot | *payload_content_encoding=base64 | 15:14 |
| redrobot | sorry, I typed the wrong param first | 15:14 |
| *** manoj_kumar_kata has quit IRC | 15:15 | |
| noonedeadpunk | well yes, makes sense then | 15:16 |
| noonedeadpunk | but yeah, ascii thing left from py2 I'm pretty sure about that... it's smth that sould be removed/replaced for sure | 15:16 |
| rosmaita | redrobot: noonedeadpunk: i suggest writing some tests to prove this is an error before making any changes to oslo | 15:16 |
| lseki | GirishChilukuri: sure, I' take a look | 15:16 |
| rosmaita | redrobot: noonedeadpunk: we're dealing with a byte array in py3, so it has to be decoded to something before it can be interpreted as characters | 15:17 |
| zenkuro | lseki: thanks for a feedback. I will start working on recommendations and improvements right after this patch pass | 15:19 |
| redrobot | rosmaita, I don't think that's accurate. In the base64 module for the stdlib both encode and decode require byte strings as input and both output bytes as well. | 15:19 |
| redrobot | rosmaita, IIUC, the only time you need to decode something to pass it as a param to base64 is when you try to pass a string, in that case it has to be decoded from UTF-8 to get a byte array | 15:20 |
| rosmaita | redrobot: i don't claim to be an expert on this, but since you are using the cinder channel, i figure i can butt in and request that if you patch this, you include a bunch of tests | 15:22 |
| lseki | zenkuro: awesome, thanks! | 15:22 |
| redrobot | rosmaita, understood, will definitely add tests around that | 15:22 |
| rosmaita | :D | 15:23 |
| noonedeadpunk | I guess it's my fault about helding dicsussion here:) | 15:23 |
| redrobot | :D | 15:23 |
| rosmaita | this character encoding stuff is really tricky | 15:23 |
| redrobot | Heh, yes... we had to spend a lot of time looking into it to get the 2/3 compat stuff just right in barbican | 15:25 |
| *** GirishChilukuri has quit IRC | 15:37 | |
| rosmaita | smcginnis: i am sorry i missed this earlier, but should we revert https://review.opendev.org/#/c/756456/ to remove the change to the sqlalchemy migration file? i don't know that we want those different across releases unless absolutely necessary | 15:48 |
| *** GirishChilukuri has joined #openstack-cinder | 15:49 | |
| *** rpittau is now known as rpittau|afk | 16:01 | |
| *** udesale_ has quit IRC | 16:16 | |
| *** _pewp_ has quit IRC | 16:16 | |
| *** _pewp_ has joined #openstack-cinder | 16:19 | |
| *** priteau has joined #openstack-cinder | 16:20 | |
| *** zenkuro has quit IRC | 16:20 | |
| *** tosky has quit IRC | 16:21 | |
| *** zenkuro has joined #openstack-cinder | 16:22 | |
| *** ociuhandu_ has joined #openstack-cinder | 16:35 | |
| *** ociuhandu has quit IRC | 16:39 | |
| *** ociuhandu_ has quit IRC | 16:40 | |
| *** zenkuro has quit IRC | 16:44 | |
| *** zenkuro has joined #openstack-cinder | 16:45 | |
| *** zenkuro has quit IRC | 16:46 | |
| *** zenkuro has joined #openstack-cinder | 16:46 | |
| *** zenkuro has quit IRC | 16:48 | |
| *** hamalq has joined #openstack-cinder | 16:49 | |
| *** sharathkacham has quit IRC | 16:55 | |
| openstackgerrit | Ajitha Robert proposed openstack/cinder master: Nimble: Add support for revert to snapshot https://review.opendev.org/757554 | 17:05 |
| *** pcaruana has joined #openstack-cinder | 17:13 | |
| *** zenkuro has joined #openstack-cinder | 17:19 | |
| zenkuro | hi, if I have a GATE failure which is not related to my code, what should I do https://zuul.opendev.org/t/openstack/build/c1f526b8cc924ca2af1990fd361e15d7 ? | 17:20 |
| zenkuro | in integrated storage | 17:21 |
| *** zenkuro has quit IRC | 17:25 | |
| *** zenkuro has joined #openstack-cinder | 17:25 | |
| rosmaita | zenkuro: recheck | 17:32 |
| rosmaita | which i see you already did | 17:32 |
| zenkuro | rosmaita: yep, preventive recheck, Im asking since test will finish in 1.1 hour and you guys will go afk | 17:34 |
| zenkuro | rosmaita: and Im not sure if recheck will help with gate job as well | 17:35 |
| rosmaita | zenkuro: yes, it should go to gate after check has passed | 17:36 |
| rosmaita | i will be around and will keep an eye on things | 17:36 |
| zenkuro | rosmaita: \>_</ | 17:36 |
| *** ajitha has joined #openstack-cinder | 17:36 | |
| smcginnis | ajitha: No, release notes do not need to have a bug associated (better to talk in channel). | 17:50 |
| ajitha | ok | 17:51 |
| ajitha | i thought this would be the syntax, tox -e venv -- reno new <bug-id> | 17:54 |
| *** lpetrut has quit IRC | 17:59 | |
| GirishChilukuri | @core reviewers Please review this code change https://review.opendev.org/#/c/744664/ for +2 approvals. | 18:10 |
| *** GirishChilukuri has quit IRC | 18:15 | |
| *** zenkuro has quit IRC | 18:24 | |
| *** zenkuro has joined #openstack-cinder | 18:24 | |
| e0ne | hi. can anybody please help me with tools/config/generate_cinder_opts.py? it generates incorrect code for my change | 18:30 |
| e0ne | here is it's output: https://zuul.opendev.org/t/openstack/build/fb331539855048b8a48cb8e3c13bff84 | 18:31 |
| rosmaita | e0ne: i am looking, but no progress to report yet | 18:59 |
| e0ne | rosmaita: thanks | 18:59 |
| ajitha | Hi all, i have submitted the revert to snapshot patch for review. in our CI we are still using devstack-gate, and included DEVSTACK_GATE_TEMPEST_REGEX parameter to include tempest tests.. how to include cinder_tempest_plugin tests with this parameter?? | 19:01 |
| smcginnis | e0ne: Did you run "tox -e genopts" for your patch? | 19:01 |
| smcginnis | Hmm, does look messed up though. | 19:02 |
| e0ne | smcginnis: I didn't, because it generates invalid code:( | 19:02 |
| *** priteau has quit IRC | 19:04 | |
| e0ne | smcginnis: I don't understand what is going wrong with my patch | 19:04 |
| lseki | you may add an OR operator to the regex | 19:09 |
| lseki | something like `export DEVSTACK_GATE_TEMPEST_REGEX='tempest.api.volume|cinder_tempest_plugin'` | 19:10 |
| lseki | ajitha: ^^ | 19:11 |
| * smcginnis wants to rewrite generate_cinder_opts.py | 19:12 | |
| rosmaita | yeah, a bit too much manual string matching | 19:13 |
| smcginnis | My guess is that it doesn't anticipate the , at the end of line 102 here: https://review.opendev.org/#/c/722655/11/cinder/backup/drivers/ceph.py | 19:13 |
| rosmaita | yes, if you just have CONF.register_opts(CEPH_BACKUP_OPTS) that first error goes away | 19:14 |
| rosmaita | the [, problem still happens, though | 19:14 |
| *** masterpe has quit IRC | 19:16 | |
| e0ne | rosmaita: thanks, -1 error is a good start :) | 19:19 |
| smcginnis | I think ideally this should just be redone to use something like I did with the sphinx extension: https://opendev.org/openstack/cinder/src/branch/master/doc/ext/driver_opts.py | 19:20 |
| smcginnis | And get rid of all this weird string matching and parsing and formatting. | 19:20 |
| *** masterpe has joined #openstack-cinder | 19:23 | |
| rosmaita | smcginnis: e0ne: take a look at this: https://review.opendev.org/#/c/722655/11/cinder/volume/configuration.py@137 | 19:24 |
| e0ne | rosmaita: looking on it... | 19:26 |
| rosmaita | feel free to tell me it's a stupid suggestion, on closer look i'm not so sure | 19:26 |
| ajitha | @lseki | 19:30 |
| ajitha | thank you | 19:31 |
| e0ne | rosmaita, smcginnis: https://review.opendev.org/#/c/722655/11/cinder/volume/configuration.py@137 isn't related to genopts issue, but now I know where it is. thanks, for the help! | 19:34 |
| e0ne | I'll try to fix it tomorrow | 19:35 |
| rosmaita | \o/ | 19:35 |
| e0ne | here is it: https://review.opendev.org/#/c/722655/11/cinder/backup/manager.py@101 | 19:36 |
| *** benfelin has quit IRC | 19:49 | |
| *** e0ne has quit IRC | 19:50 | |
| *** e0ne has joined #openstack-cinder | 19:54 | |
| *** zenkuro has quit IRC | 19:56 | |
| *** zenkuro has joined #openstack-cinder | 19:57 | |
| *** e0ne has quit IRC | 19:59 | |
| *** zenkuro has quit IRC | 20:03 | |
| *** zenkuro has joined #openstack-cinder | 20:03 | |
| *** tosky has joined #openstack-cinder | 20:26 | |
| *** whoami-rajat__ has joined #openstack-cinder | 20:36 | |
| whoami-rajat__ | lseki: hi, around? | 20:36 |
| lseki | whoami-rajat__: o/ | 20:38 |
| whoami-rajat__ | lseki: does netapp nfs support qcow2 format volumes as well or only raw ? | 20:43 |
| lseki | I think it supports them both, but let me check | 20:50 |
| whoami-rajat__ | sure, if it supports qcow2 we might have a problem ... | 20:51 |
| lseki | whoami-rajat__: hmm qcow2 is converted to raw when I create a volume from a glance image, I need to try other ways to test it | 21:04 |
| lseki | I'll try it tomorrow and let you know | 21:04 |
| lseki | what's the problem? | 21:05 |
| *** e0ne has joined #openstack-cinder | 21:09 | |
| whoami-rajat__ | lseki: the problem is related to glance cinder store, it's a long issue, i will detail you out tomorrow | 21:15 |
| whoami-rajat__ | let me know if it supports qcow2 at any place | 21:16 |
| *** vishalmanchanda has quit IRC | 21:27 | |
| *** e0ne has quit IRC | 21:27 | |
| *** rcernin has joined #openstack-cinder | 21:47 | |
| *** rcernin has quit IRC | 21:47 | |
| *** rcernin has joined #openstack-cinder | 21:47 | |
| openstackgerrit | Brian Rosmaita proposed openstack/cinder master: Preparing for removal of six.reraise() https://review.opendev.org/757655 | 21:54 |
| *** rcernin has quit IRC | 22:05 | |
| *** ajitha has quit IRC | 22:14 | |
| *** rcernin has joined #openstack-cinder | 22:28 | |
| *** rcernin has quit IRC | 22:29 | |
| *** rcernin has joined #openstack-cinder | 22:30 | |
| *** watersj has quit IRC | 23:01 | |
| *** CeeMac has quit IRC | 23:01 | |
| *** tosky has quit IRC | 23:15 | |
| *** zenkuro has quit IRC | 23:37 | |
| *** hamalq has quit IRC | 23:52 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!