*** edmondsw has quit IRC | 00:01 | |
*** garthb has quit IRC | 00:02 | |
*** diogogmt has quit IRC | 00:03 | |
*** tsekiyam_ has joined #openstack-cinder | 00:09 | |
*** mtanino has quit IRC | 00:12 | |
*** tsekiyama has quit IRC | 00:13 | |
*** tsekiyam_ has quit IRC | 00:14 | |
*** dsariel_ has quit IRC | 00:21 | |
*** chlong has joined #openstack-cinder | 00:22 | |
*** _cjones_ has quit IRC | 00:23 | |
*** bill_az has quit IRC | 00:25 | |
*** markvoelker has joined #openstack-cinder | 00:30 | |
*** jwcroppe has joined #openstack-cinder | 00:32 | |
*** markvoelker has quit IRC | 00:35 | |
*** daneyon has joined #openstack-cinder | 00:40 | |
*** daneyon_ has joined #openstack-cinder | 00:42 | |
*** daneyon has quit IRC | 00:46 | |
*** IlyaG has quit IRC | 00:59 | |
*** daneyon has joined #openstack-cinder | 01:02 | |
*** EinstCrazy has joined #openstack-cinder | 01:02 | |
*** daneyon__ has joined #openstack-cinder | 01:03 | |
*** daneyon_ has quit IRC | 01:04 | |
*** daneyon has quit IRC | 01:07 | |
*** leeantho has quit IRC | 01:08 | |
*** EinstCrazy has quit IRC | 01:12 | |
*** amit213 has quit IRC | 01:24 | |
*** apoorvad has quit IRC | 01:24 | |
*** amit213 has joined #openstack-cinder | 01:24 | |
*** Lee1092 has joined #openstack-cinder | 01:31 | |
*** apoorvad has joined #openstack-cinder | 01:31 | |
*** dims has quit IRC | 01:34 | |
*** mudassirlatif has quit IRC | 01:34 | |
*** daneyon__ has quit IRC | 01:38 | |
*** gouthamr has joined #openstack-cinder | 01:46 | |
*** EinstCrazy has joined #openstack-cinder | 01:46 | |
*** gouthamr_ has quit IRC | 01:49 | |
*** buster_604 has quit IRC | 02:02 | |
*** jwcroppe has quit IRC | 02:13 | |
openstackgerrit | Apoorva Deshpande proposed openstack/cinder: Snapshot GET API to return provider_id https://review.openstack.org/230240 | 02:14 |
---|---|---|
*** dave-mccowan has quit IRC | 02:14 | |
*** EinstCrazy has quit IRC | 02:14 | |
*** apoorvad_ has joined #openstack-cinder | 02:15 | |
*** apoorvad has quit IRC | 02:18 | |
*** apoorvad_ has quit IRC | 02:19 | |
*** chlong has quit IRC | 02:27 | |
*** markvoelker has joined #openstack-cinder | 02:32 | |
*** martyturner has joined #openstack-cinder | 02:33 | |
*** markvoelker has quit IRC | 02:36 | |
*** buster_604 has joined #openstack-cinder | 02:37 | |
*** martyturner has quit IRC | 02:37 | |
*** bill_az_ has quit IRC | 02:40 | |
*** chlong has joined #openstack-cinder | 02:42 | |
*** r-daneel has quit IRC | 02:44 | |
*** wN has quit IRC | 02:48 | |
*** buster_604 has quit IRC | 02:49 | |
*** stevemar has joined #openstack-cinder | 02:54 | |
*** wN has joined #openstack-cinder | 02:54 | |
*** wN has joined #openstack-cinder | 02:54 | |
*** edtubill has joined #openstack-cinder | 02:58 | |
*** IlyaG has joined #openstack-cinder | 02:58 | |
*** stevemar has quit IRC | 02:59 | |
*** IlyaG has quit IRC | 03:02 | |
*** changbl has joined #openstack-cinder | 03:03 | |
*** edtubill has quit IRC | 03:04 | |
*** changbl has quit IRC | 03:08 | |
*** IlyaG has joined #openstack-cinder | 03:08 | |
*** mriedem has quit IRC | 03:15 | |
*** IlyaG has quit IRC | 03:21 | |
*** stevemar_ has joined #openstack-cinder | 03:28 | |
*** gouthamr has quit IRC | 03:30 | |
*** links has joined #openstack-cinder | 03:31 | |
*** julim has quit IRC | 03:33 | |
*** julim has joined #openstack-cinder | 03:34 | |
*** setmason has joined #openstack-cinder | 03:35 | |
*** stevemar_ has quit IRC | 03:41 | |
*** stevemar has joined #openstack-cinder | 03:42 | |
*** salv-orlando has joined #openstack-cinder | 03:43 | |
*** salv-orl_ has quit IRC | 03:45 | |
*** bnemec has quit IRC | 04:02 | |
*** stevemar_ has joined #openstack-cinder | 04:07 | |
*** setmason_ has joined #openstack-cinder | 04:08 | |
*** stevemar has quit IRC | 04:09 | |
*** setmason has quit IRC | 04:11 | |
*** setmason has joined #openstack-cinder | 04:12 | |
*** setmason_ has quit IRC | 04:15 | |
*** markvoelker has joined #openstack-cinder | 04:33 | |
*** stevemar has joined #openstack-cinder | 04:34 | |
*** stevemar_ has quit IRC | 04:34 | |
*** stevemar has quit IRC | 04:39 | |
*** apoorvad has joined #openstack-cinder | 04:48 | |
*** stevemar has joined #openstack-cinder | 04:50 | |
openstackgerrit | Apoorva Deshpande proposed openstack/cinder: Snapshot GET API to return provider_id https://review.openstack.org/230240 | 04:54 |
*** setmason has quit IRC | 04:54 | |
*** jwcroppe has joined #openstack-cinder | 04:58 | |
*** jwcroppe has quit IRC | 05:02 | |
*** stevemar has quit IRC | 05:05 | |
*** stevemar has joined #openstack-cinder | 05:06 | |
*** mragupat has joined #openstack-cinder | 05:06 | |
*** markvoelker has quit IRC | 05:07 | |
*** setmason has joined #openstack-cinder | 05:09 | |
*** stevemar has quit IRC | 05:10 | |
*** mudassirlatif has joined #openstack-cinder | 05:11 | |
*** apoorvad has quit IRC | 05:24 | |
*** nkrinner has joined #openstack-cinder | 05:27 | |
*** e0ne has joined #openstack-cinder | 05:45 | |
*** setmason has quit IRC | 05:48 | |
*** stevemar has joined #openstack-cinder | 05:50 | |
*** stevemar_ has joined #openstack-cinder | 05:51 | |
*** stevemar has quit IRC | 05:55 | |
*** e0ne has quit IRC | 05:56 | |
*** setmason has joined #openstack-cinder | 06:18 | |
*** nkrinner has quit IRC | 06:22 | |
*** nkrinner has joined #openstack-cinder | 06:22 | |
*** setmason has quit IRC | 06:24 | |
*** mragupat has quit IRC | 06:30 | |
*** wilson1 has quit IRC | 06:31 | |
*** wilson1 has joined #openstack-cinder | 06:33 | |
*** medberry has quit IRC | 06:39 | |
openstackgerrit | yogeshprasad proposed openstack/cinder: Retype support for CloudByte iSCSI cinder driver https://review.openstack.org/218645 | 06:41 |
*** med_ has joined #openstack-cinder | 06:45 | |
*** med_ is now known as Guest43458 | 06:46 | |
*** setmason has joined #openstack-cinder | 06:56 | |
openstackgerrit | Victor Stinner proposed openstack/cinder: Fix volume throttling to Python 3 https://review.openstack.org/229991 | 07:01 |
*** lpetrut has joined #openstack-cinder | 07:03 | |
*** ronis has joined #openstack-cinder | 07:21 | |
*** mudassirlatif has quit IRC | 07:22 | |
*** chlong has quit IRC | 07:31 | |
*** haomaiwang has joined #openstack-cinder | 07:44 | |
*** mudassirlatif has joined #openstack-cinder | 07:45 | |
*** openstackgerrit has quit IRC | 07:46 | |
*** openstackgerrit has joined #openstack-cinder | 07:47 | |
*** jordanP has joined #openstack-cinder | 07:47 | |
openstackgerrit | Victor Stinner proposed openstack/cinder: Fix various Python 3 issues https://review.openstack.org/230302 | 07:48 |
*** bauzas is now known as bauwser | 07:56 | |
*** haypo has joined #openstack-cinder | 07:57 | |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwa_ has joined #openstack-cinder | 08:01 | |
*** stevemar_ has quit IRC | 08:01 | |
*** stevemar has joined #openstack-cinder | 08:02 | |
*** lpetrut has quit IRC | 08:03 | |
*** stevemar has quit IRC | 08:06 | |
*** bluex-pl has joined #openstack-cinder | 08:11 | |
*** bluex-pl has quit IRC | 08:11 | |
*** bluex-pl has joined #openstack-cinder | 08:12 | |
*** lpetrut has joined #openstack-cinder | 08:13 | |
*** setmason has quit IRC | 08:16 | |
openstackgerrit | Szymon Wróblewski proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 08:16 |
openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Cleanup for cinder tests with CGSnapshot https://review.openstack.org/207866 | 08:17 |
*** harlowja has quit IRC | 08:19 | |
*** jseiler has quit IRC | 08:19 | |
*** jseiler has joined #openstack-cinder | 08:20 | |
*** jwcroppe has joined #openstack-cinder | 08:23 | |
*** jistr has joined #openstack-cinder | 08:24 | |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port scheduler host manager to Python 3 https://review.openstack.org/230313 | 08:29 |
*** vgridnev has quit IRC | 08:31 | |
*** zhonghua-lee has quit IRC | 08:39 | |
*** jwcroppe has quit IRC | 08:40 | |
*** alexschm has joined #openstack-cinder | 08:44 | |
*** links has quit IRC | 08:50 | |
*** aix has joined #openstack-cinder | 08:54 | |
*** mudassirlatif has quit IRC | 08:56 | |
*** e0ne has joined #openstack-cinder | 08:57 | |
*** haomaiwa_ has quit IRC | 09:01 | |
*** haomaiwang has joined #openstack-cinder | 09:01 | |
*** markvoelker has joined #openstack-cinder | 09:04 | |
*** markvoelker has quit IRC | 09:09 | |
*** lpetrut has quit IRC | 09:10 | |
*** openstack has joined #openstack-cinder | 09:22 | |
*** openstackstatus has joined #openstack-cinder | 09:22 | |
*** ChanServ sets mode: +v openstackstatus | 09:22 | |
*** alexpilotti has quit IRC | 09:22 | |
*** e0ne has quit IRC | 09:23 | |
*** vgridnev has joined #openstack-cinder | 09:23 | |
*** ndipanov has joined #openstack-cinder | 09:23 | |
*** e0ne has joined #openstack-cinder | 09:31 | |
*** vgridnev has quit IRC | 09:31 | |
*** vgridnev_ has joined #openstack-cinder | 09:31 | |
*** salv-orlando has quit IRC | 09:33 | |
e0ne | zigo: hi! do you still need help with config generation? | 09:37 |
zigo | e0ne: I did a hack which fixes my CI, but it's not satisfying at all, so yes. | 09:37 |
*** lprice has quit IRC | 09:37 | |
e0ne | zigo: I hope, you resolved all issues today, just want to confirnm | 09:37 |
zigo | e0ne: enabled_backends is still not present by default, so I'm using wc -l / tail / cat / echo to add it in my debian/rules file. | 09:37 |
*** lprice has joined #openstack-cinder | 09:38 | |
e0ne | zigo: enabled_backends is not required if you have only one backend | 09:38 |
zigo | e0ne: Then, I've been told in my bug report that the [lvm] section isn't present by default in the generated cinder.conf.sample. I don't think that's normal !!! | 09:38 |
zigo | e0ne: Well, the generator should be showing absolutely *all* config options. | 09:38 |
zigo | It's like that for every project in OpenStack. | 09:39 |
zigo | There's no point in generating the config file if it's not exhaustive. | 09:39 |
*** strony-zhang has quit IRC | 09:42 | |
e0ne | zigo: we've got mutually options in configs. config generator should know about them | 09:44 |
*** dims__ has joined #openstack-cinder | 09:44 | |
*** haomaiwang has quit IRC | 10:01 | |
*** haomaiwang has joined #openstack-cinder | 10:01 | |
openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Setup error check & minor bug fix in CloudByte https://review.openstack.org/230068 | 10:01 |
*** daemontool__ has joined #openstack-cinder | 10:07 | |
*** daemontool_ has quit IRC | 10:07 | |
*** ollie1 has left #openstack-cinder | 10:08 | |
*** pcaruana has joined #openstack-cinder | 10:13 | |
*** IanGovett has joined #openstack-cinder | 10:17 | |
*** e0ne has quit IRC | 10:21 | |
*** e0ne has joined #openstack-cinder | 10:34 | |
*** jaypipes has joined #openstack-cinder | 10:36 | |
*** jaypipes is now known as leakypipes | 10:38 | |
*** ndipanov is now known as ndipanoff | 10:47 | |
*** ig0r_ has joined #openstack-cinder | 10:55 | |
*** ig0r_ has quit IRC | 10:56 | |
*** e0ne has quit IRC | 10:57 | |
*** takedakn has joined #openstack-cinder | 10:59 | |
*** haomaiwang has quit IRC | 11:01 | |
*** haomaiwang has joined #openstack-cinder | 11:01 | |
*** markvoelker has joined #openstack-cinder | 11:05 | |
*** markvoelker has quit IRC | 11:09 | |
*** kevincarr1991 has joined #openstack-cinder | 11:13 | |
*** salv-orlando has joined #openstack-cinder | 11:15 | |
*** delatte has joined #openstack-cinder | 11:16 | |
*** cdelatte has joined #openstack-cinder | 11:16 | |
*** salv-orlando has quit IRC | 11:20 | |
*** kevincarr1991 has quit IRC | 11:24 | |
*** kevincarr1991 has joined #openstack-cinder | 11:27 | |
kevincarr1991 | I have installed Cinder on my controller node and used the nfs driver to attach the service to a nfs share. Cinder attaches to the nfs and I am able to create volumes, but I can't attach the volumes to instances using the dashboard | 11:29 |
*** salv-orlando has joined #openstack-cinder | 11:32 | |
*** dzamboni has joined #openstack-cinder | 11:36 | |
*** julim has quit IRC | 11:39 | |
*** julim has joined #openstack-cinder | 11:44 | |
*** lpetrut has quit IRC | 11:46 | |
*** pcaruana has quit IRC | 11:47 | |
*** haomaiwang has quit IRC | 12:01 | |
*** e0ne has joined #openstack-cinder | 12:01 | |
*** haomaiwang has joined #openstack-cinder | 12:01 | |
*** zul has quit IRC | 12:01 | |
*** zul has joined #openstack-cinder | 12:04 | |
*** lpetrut has joined #openstack-cinder | 12:05 | |
*** markvoelker has joined #openstack-cinder | 12:05 | |
openstackgerrit | Keiichi KII proposed openstack/os-brick: iSCSI Stop calling multipath command line https://review.openstack.org/230391 | 12:05 |
*** markvoelker has quit IRC | 12:09 | |
*** markvoelker has joined #openstack-cinder | 12:09 | |
*** haomaiwang has quit IRC | 12:10 | |
*** 20WAB697A has joined #openstack-cinder | 12:11 | |
*** martyturner has joined #openstack-cinder | 12:11 | |
*** ociuhandu has joined #openstack-cinder | 12:13 | |
*** 20WAB697A has quit IRC | 12:25 | |
*** ociuhandu has quit IRC | 12:27 | |
*** delatte has quit IRC | 12:36 | |
*** cdelatte has quit IRC | 12:36 | |
*** edmondsw has joined #openstack-cinder | 12:37 | |
*** delatte has joined #openstack-cinder | 12:37 | |
*** cdelatte has joined #openstack-cinder | 12:37 | |
*** Yogi1 has joined #openstack-cinder | 12:38 | |
kevincarr1991 | I have installed Cinder on my controller node. I have Cinder attached to a nfs share. I am able to create volumes but I cannot attach the volumes to any instances. I have tried to attach volumes using the dashboard and cli and both state that they work successfully, but the instances wont have a new volume attached | 12:40 |
*** porrua has joined #openstack-cinder | 12:41 | |
*** CBADER has quit IRC | 12:41 | |
*** kevincarr1991 has quit IRC | 12:42 | |
*** vgridnev_ has quit IRC | 12:43 | |
*** akerr has joined #openstack-cinder | 12:44 | |
*** vgridnev has joined #openstack-cinder | 12:44 | |
haypo | hum, one more time someone asks me to change how to link my Python 3 patches to the Python 3 blueprint with a -1 vote | 12:44 |
haypo | I'm using the syntax "Blueprint cinder-python3" as I did for my previous +30 changes already merged in Cinder | 12:44 |
haypo | but Kendall Nelson asked me to write "Partial-Implements: blueprint cinder-python3" | 12:44 |
haypo | can we please agree on one syntax and stop nitpicking on this stupid link? | 12:45 |
dims__ | haypo: we have to start charging for -1's :) $1 per -1 | 12:45 |
haypo | ("Partial-Implements: blueprint cinder-python3" and "Blueprint cinder-python3" are technically the same, both add the change to the blueprint when a new change is submitted) | 12:45 |
*** leitan has joined #openstack-cinder | 12:45 | |
haypo | dims__: +1 for $1 per -1 | 12:45 |
*** diablo_rojo has quit IRC | 12:46 | |
*** Yogi1 has quit IRC | 12:46 | |
*** CBADER has joined #openstack-cinder | 12:46 | |
haypo | it's so annoying that i even wrote an email about this line of the commit message: | 12:46 |
haypo | http://lists.openstack.org/pipermail/openstack-dev/2015-September/073476.html | 12:46 |
haypo | [openstack-dev] [cinder] How to link a change to the completed cinder-python3 blueprint? | 12:46 |
haypo | ... and i didn't get any reply | 12:46 |
haypo | so what? | 12:46 |
*** bluex-pl has quit IRC | 12:49 | |
*** eharney has joined #openstack-cinder | 12:50 | |
*** kevincarr1991 has joined #openstack-cinder | 12:51 | |
*** leitan has quit IRC | 12:55 | |
*** leitan has joined #openstack-cinder | 12:55 | |
*** dustins has joined #openstack-cinder | 12:57 | |
*** diablo_rojo has joined #openstack-cinder | 12:58 | |
*** martyturner has quit IRC | 12:59 | |
*** salv-orlando has quit IRC | 12:59 | |
*** dave-mccowan has joined #openstack-cinder | 13:00 | |
*** timcl has joined #openstack-cinder | 13:02 | |
*** leitan has quit IRC | 13:02 | |
*** martyturner has joined #openstack-cinder | 13:02 | |
*** jseiler has quit IRC | 13:10 | |
*** xyang1 has joined #openstack-cinder | 13:10 | |
e0ne | haypo: partial implements looks better, but I won't -1 if it's only "Blueprint bp-name" | 13:11 |
*** leitan has joined #openstack-cinder | 13:12 | |
*** Yogi1 has joined #openstack-cinder | 13:13 | |
haypo | e0ne: would you be brave enough to write it in a comment at https://review.openstack.org/#/c/229991/2//COMMIT_MSG ? thanks :) | 13:14 |
e0ne | haypo: sure, I'll do | 13:14 |
e0ne | haypo: probably, I have to post to ML too | 13:15 |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS iSCSI manage does not work with spaces https://review.openstack.org/228575 | 13:15 |
haypo | e0ne: cool :) | 13:15 |
*** raildo-afk is now known as raildo | 13:16 | |
*** merooney has joined #openstack-cinder | 13:18 | |
*** gouthamr has joined #openstack-cinder | 13:19 | |
smcginnis | zigo, e0ne: I think there is a misunderstanding on the config sample output. There should not be an [lvm] section in that file. | 13:22 |
*** martyturner has quit IRC | 13:22 | |
zigo | smcginnis: I think *you* are not understanding what other projects are doing. Take for example Nova: they do have a [vmware] section, even though not everyone uses vmware. They do have absolutely *all* of the options shown in the .sample file. | 13:23 |
zigo | Why should Cinder be different? | 13:23 |
zigo | Is it that adding the [lvm] section breaks something if we don't use lvm? | 13:23 |
smcginnis | zigo: It's not. [lvm] would be whatever you name your backend. | 13:23 |
zigo | smcginnis: How do our users guess the options for lvm then? | 13:24 |
smcginnis | And all the options are in there, including enabled_backends from what I've seen from several tests. | 13:24 |
smcginnis | All the lvm options are in there. | 13:24 |
zigo | Ah? Where? | 13:24 |
diablo_rojo | smcginnis: zigo The reason there is no lvm section being created is becuase when those options get registered, they aren't being registed with a group. | 13:24 |
smcginnis | Whether you name that backend lvm or not is up to you. | 13:24 |
*** jgregor has joined #openstack-cinder | 13:24 | |
zigo | Ok, got it. | 13:25 |
*** haomaiwa_ has joined #openstack-cinder | 13:25 | |
smcginnis | Right, they are not a group. The examples shown in the docs are for a backend named lvm. | 13:25 |
zigo | smcginnis: Now, how would users know all of the options of [backend-foo] ? | 13:25 |
smcginnis | Maybe the doc should be updated to be named [my-lvm] or something to make it obvious. | 13:25 |
zigo | Yup, sounds reasonable. | 13:25 |
zigo | [lvm-foo] :P | 13:26 |
*** mriedem has joined #openstack-cinder | 13:26 | |
zigo | smcginnis: I still insist that I don't have the enabled_backends option anywhere. Sorry to be a pain here, but I think it's important to fix it for every downstream distro, especially that maybe, it hides the fact that some other options may be also not there. | 13:27 |
smcginnis | zigo: No problem, that part would be legitimate. | 13:27 |
smcginnis | zigo: I've not been able to reproduce that. I always get enabled_backends in every place I've tried it. | 13:28 |
zigo | smcginnis: You only tried with devstack, right? | 13:28 |
zigo | Using tox... | 13:28 |
smcginnis | Only using tox, but that should really matter if you are passing in from_tox and running in the correct root directory. | 13:29 |
smcginnis | diablo_rojo: I'm not lying, am I? ^ | 13:29 |
diablo_rojo | smcginnis: I believe that is correct. | 13:29 |
smcginnis | Line 544 is my latest test. | 13:30 |
smcginnis | zigo: What's the line count you get? wc -l etc/cinder/cinder.conf.sample | 13:31 |
diablo_rojo | smcginnis: zigo The reason tox was made to be the only way to run it was because people were running it in other ways from other directories and it was missing options. Tox was decided as being the only way it should be run to guarantee that all options would be picked up. | 13:31 |
zigo | smcginnis: 805 lines | 13:32 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Detach volume on device validation failure https://review.openstack.org/230429 | 13:32 |
diablo_rojo | zigo: It should have over 3000 | 13:32 |
smcginnis | zigo: OK, something definitely wrong there. I get over 3000. | 13:32 |
zigo | diablo_rojo: I understand, but that's not downstream distribution friendly... | 13:32 |
zigo | diablo_rojo: I can't run tox when building the Debian package, as there's no virtual env involved. | 13:33 |
*** jungleboyj has joined #openstack-cinder | 13:33 | |
zigo | smcginnis: Could you, as a temporary hack fix, send me the file? | 13:33 |
zigo | It's problematic too, as it wont be generated with the build-depends that I am using in Debian though ... | 13:33 |
smcginnis | zigo: Sure, I can send you my output. | 13:34 |
zigo | So it can only be considered as a *very* temporary hack. | 13:34 |
smcginnis | Let me see how paste.os.o likes 3000 lines. ;) | 13:34 |
zigo | smcginnis: Is it even possible to past 3000 lines there? | 13:34 |
smcginnis | I wasn't aware distributors were packaging that any more. I seem to remember a long discussion of getting rid of it. | 13:34 |
zigo | otherwise, zigo@debian.org | 13:34 |
smcginnis | zigo: OK, that's probably better. On its way. | 13:35 |
zigo | smcginnis: Ubuntu people don't care. *I* do care having a correct conf file in Debian that works by default. | 13:35 |
eharney | smcginnis: we package sample conf too | 13:35 |
jungleboyj | Howdy all, sorry I am just jumping in now zigo and smcginnis | 13:35 |
zigo | jungleboyj: No prob. | 13:35 |
smcginnis | eharney: Oh good, I was just wondering about RH. | 13:35 |
smcginnis | eharney: How do you guys handle it? | 13:35 |
zigo | eharney: Are you working for Red Hat and doing Cinder packaging? | 13:36 |
jungleboyj | So, did we land on the fact that we were running on an old opts.py file? | 13:36 |
eharney | smcginnis: manually at this point, like zigo referred to, running things like tox and having it install deps in our build environments is completely disallowed, so tox-only is not really a whole solution | 13:37 |
*** setmason has joined #openstack-cinder | 13:37 | |
*** haomaiwa_ has quit IRC | 13:37 | |
zigo | eharney: Thanks for also explaining it! :) | 13:38 |
eharney | smcginnis: IMO we should just be checking in a sample conf at the end of the release upstream and shipping it w/ Cinder | 13:38 |
*** haomaiwang has joined #openstack-cinder | 13:38 | |
zigo | eharney: That's the wrong answer to the problem. | 13:38 |
smcginnis | eharney: That does sound more reasonable. | 13:38 |
zigo | eharney: The .sample file *must* be generated from within the dependencies that you have in the OS. | 13:38 |
zigo | Otherwise, you may risk to have a wrong file. | 13:38 |
eharney | we already have code that ensures the sample is generated successfully, we just don't ship it | 13:38 |
smcginnis | zigo: We only generate from within cinder. | 13:38 |
eharney | (as part of the gate) | 13:39 |
*** nkrinner has quit IRC | 13:39 | |
zigo | For example, what if one module (let's say, oslo.messaging) adds an option, and the version in pypi is different from the one packaged in the distro? Well, I'll tell you what: the sample config file will simply be wrong. | 13:39 |
zigo | Which is why it isn't recommended to store the config file in upstream git. | 13:39 |
zigo | Downstream distros *must* be allowed to build them. | 13:40 |
jungleboyj | So, if we need to have the opts.py file out there for the deployers do we need to add that to the distribution and add a pep8 check for when it needs to be updated as the result of a change. That shouldn't happen very often. | 13:40 |
jungleboyj | zigo: You are right. | 13:40 |
eharney | i don't want to build a sample file downstream, because i'm not going to package random dependencies for drivers we don't support, and without doing that, i can't build one w/o tox | 13:40 |
eharney | at least, can't build a full sample file | 13:40 |
zigo | eharney: I have a different view. | 13:41 |
* smcginnis wonders if we need a packager working session | 13:41 | |
zigo | I do want to generate it, because I want the sample file to include only drivers that I support. | 13:41 |
zigo | I don't care whatever driver that isn't maintained within Debian. | 13:41 |
eharney | zigo: well, that's more straightforward to solve | 13:41 |
jungleboyj | smcginnis: We probably do. I am glad we are having this discussion. | 13:41 |
smcginnis | Me too | 13:41 |
smcginnis | I wasn't aware of any of this. | 13:42 |
eharney | zigo: in that case, you just need to get a generate_sample fixed up that runs w/o tox, which shouldn't be that bad... though i don't know the details of all the new dynamic conf stuff | 13:42 |
zigo | smcginnis: It's been a long time I'm thinking about doing a session about what's going on downstream, and how we expect upstream stuff to behave... | 13:42 |
jungleboyj | smcginnis: I wondered if we might hit something like this but had hoped not. | 13:42 |
smcginnis | AFAIK, as long as you pass in | 13:42 |
eharney | smcginnis: distribution build environments are highly controlled, have tracked dependencies, don't allow downloads from the internet during build time, etc | 13:42 |
smcginnis | "from_tox" and run from the right dir, it should be fine. | 13:42 |
zigo | eharney: The current issue is that it's broken for me right now. | 13:42 |
eharney | smcginnis: it's a whole different world from all of this "just pip install everything" theory upstream | 13:42 |
jungleboyj | eharney: So, that is what I just proposed above. | 13:43 |
zigo | smcginnis: Well, it "should" be fine, but it isn't ... :( | 13:43 |
*** lprice1 has joined #openstack-cinder | 13:43 | |
jungleboyj | If we need generate_sample to work without tox, we need to ship an opts.py file. | 13:43 |
eharney | jungleboyj: i think your proposal makes sense, just don't know the ins and outs of it | 13:43 |
zigo | jungleboyj: Why do you think so? | 13:43 |
jungleboyj | The part that zigo is worried about (other libraries) is handled outside of the opts.py file. | 13:43 |
*** haomaiwang has quit IRC | 13:43 | |
zigo | jungleboyj: Ah, good then! | 13:44 |
zigo | (and, yes of course, stupid me...) | 13:44 |
jungleboyj | zigo: the oslo-config-generator looks at a separate library for the dependencies. Right diablo_rojo | 13:44 |
*** haomaiwang has joined #openstack-cinder | 13:44 | |
diablo_rojo | jungleboyj: correct | 13:44 |
*** lprice has quit IRC | 13:45 | |
eharney | all of this was not an issue when we did "you must update the sample conf at review time when changing options" | 13:45 |
eharney | fwiw | 13:45 |
*** martyturner has joined #openstack-cinder | 13:46 | |
jungleboyj | So, what I think diablo_rojo and I could do is add the opts.py file to the tree. Then add a pep8 check like we used to have that will generate the opts.py file and verify there are no changes. If there are changes it will flag that so the person committing the code knows they need to also update that file. | 13:46 |
smcginnis | jungleboyj: Ah, I like. | 13:46 |
jungleboyj | That should be a rare occasion as that file only changes if a whole new group of options is added. | 13:46 |
jungleboyj | smcginnis: What is eating at me is the fact that I feel like there is a reason we didn't do that. | 13:46 |
smcginnis | As long as we have some way to enforce it's being done and it's not up to someone to review and realize it's missed. | 13:46 |
smcginnis | jungleboyj: Yeah... | 13:46 |
jungleboyj | smcginnis: Right. | 13:46 |
smcginnis | But I think the intent of the generation was to take out user error in missing that. | 13:47 |
smcginnis | As long as we can protect against that, I guess I'm fine. | 13:47 |
jungleboyj | It isn't going to break the gate like it used to because we are only checking the opts file which is only based on Cinder code. | 13:47 |
jungleboyj | If the developer doesn't run pep8 before pushing up, that isn't our fault. | 13:48 |
smcginnis | It's still one more manual step (update the opts.py file) but it's guarded. | 13:48 |
smcginnis | We have enough missed pepp8 before pushing. ;) | 13:48 |
diablo_rojo | smcginnis: That we do ;) | 13:48 |
jungleboyj | Yeah, not ideal, but it appears that we can't go fully automated. | 13:49 |
*** baumann has joined #openstack-cinder | 13:49 | |
jungleboyj | Want our deployers happy. :-) | 13:49 |
* eharney needs to go study this opts.py thing again | 13:49 | |
jungleboyj | diablo_rojo: Now has a 'pep8!?!' post it note on her computer from jungleboyj | 13:49 |
smcginnis | Hah | 13:49 |
jungleboyj | eharney: Look away, look away! | 13:49 |
jungleboyj | eharney: Just trust us. :-) | 13:49 |
smcginnis | At least thanks to eharney's quick8 patch you can install a hook! | 13:50 |
eharney | jungleboyj: upgrade the note to "fast8" :) | 13:50 |
*** stevemar has joined #openstack-cinder | 13:50 | |
smcginnis | s/quick8/fast8/ | 13:50 |
jungleboyj | Huh? What is fast8? | 13:50 |
*** stevemar has quit IRC | 13:50 | |
smcginnis | The best thing sinced sliced bread. :) | 13:50 |
* smcginnis can't spell today | 13:50 | |
eharney | jungleboyj: "tox -e fast8" | 13:50 |
jungleboyj | Mmmm, raisin bread? I am hungry. | 13:50 |
jungleboyj | Ok, I will have to check that out. | 13:51 |
smcginnis | jungleboyj: https://review.openstack.org/#/c/226487/3/tools/enable-pre-commit-hook.sh | 13:51 |
smcginnis | jungleboyj: Automation!! ^^ | 13:51 |
diablo_rojo | smcginnis: yay for automation! | 13:51 |
jungleboyj | diablo_rojo: Oh the irony of that statement at the moment. :-) | 13:52 |
smcginnis | So none of this actually helps deployers now for liberty though. :\ | 13:52 |
smcginnis | jungleboyj: LOL | 13:52 |
eharney | smcginnis: i think i'll turn that on and see how life goes trying to use that hook | 13:52 |
*** jwcroppe has joined #openstack-cinder | 13:52 | |
smcginnis | eharney: Yeah, it would be great if it works well and is commonly used. | 13:52 |
smcginnis | Could save a lot of time. | 13:52 |
kaisers | Hey Cinders! Scheduling Q: Can the instance_locality_filter be used during nova boot? i.e. can i use create volume from image and boot instance in conjunction with instance_locality_filter in the cinder scheduler in order to achieve booting from an image stored on a local volume? | 13:53 |
eharney | smcginnis: i kind of suspect i'm going to end up wanting it as pre-push instead of pre-commit | 13:54 |
jungleboyj | smcginnis: We could backport the fix? | 13:54 |
* jungleboyj says uneasily | 13:54 | |
smcginnis | eharney: True. At least we will have a sample showing how. Should be easy enough to modify. Unless you think I should change that to be the default? | 13:54 |
smcginnis | jungleboyj: Yeah, not sure. | 13:54 |
jungleboyj | The stable maintainer in me says it adds a file and I should say now, but it is for the conf file which seems ok. We aren't going to break things worse. | 13:54 |
eharney | smcginnis: dunno, i'll play with it | 13:54 |
* jungleboyj needs to resist the urge to Ninja merge. | 13:55 | |
*** mriedem1 has joined #openstack-cinder | 13:55 | |
*** alejandrito has joined #openstack-cinder | 13:55 | |
smcginnis | jungleboyj: Well, I get an out here because I'm not a stable maintainer, so I'll leave that to you. :P | 13:55 |
*** mriedem has quit IRC | 13:55 | |
jungleboyj | smcginnis: Chicken. | 13:56 |
smcginnis | Yep. | 13:56 |
eharney | smcginnis: as a side note on stable things... i'm still wondering what's up w/ liberty-rc2 | 13:56 |
jungleboyj | smcginnis: eharney zigo So, are we in agreement that we should add the opts.py file to our tree? That we should then run a check to verify that the code is up to date as part of pep8? | 13:56 |
*** merooney has quit IRC | 13:56 | |
jungleboyj | eharney: Me too! | 13:57 |
smcginnis | jungleboyj: Seriously though, if it is just a matter of adding back in an opt file that's fine. But wouldn't it require a bigger change? | 13:57 |
zigo | jungleboyj: I'm fine with this solution, yes. | 13:57 |
eharney | jungleboyj: probably in agreement, i'll make sure once i look at a patch and see how it works | 13:57 |
* jungleboyj figured zigo would be. :-) | 13:57 | |
zigo | Sorry, I couldn't follow some of the discussion, reading backlog. | 13:57 |
jungleboyj | eharney: Fair enough. | 13:57 |
smcginnis | eharney: I haven't heard anything lately, but I believe RC2 is imminent. | 13:57 |
eharney | smcginnis: just wondering since we have some must-land patches | 13:58 |
smcginnis | I definitely will also make it a point to have some downstream discussions in Tokyo. | 13:58 |
jungleboyj | smcginnis: It is going to be checking in a opt.py file and adding a diff between the existing and a new one into pep8. | 13:58 |
smcginnis | eharney: I think we should probably make sure thingee is aware of those. | 13:58 |
smcginnis | "adding a diff between the existing and a new one" << bigger than it sounds? That's my concern. | 13:59 |
jungleboyj | smcginnis: eharney zigo I need to drive into the office. diablo_rojo and I will get something up hopefully this morning to look at. | 13:59 |
jungleboyj | smcginnis: Yee have little faith. ;-) | 13:59 |
smcginnis | Everything sounds easy until you dig into it. ;) | 14:00 |
jungleboyj | :-p Anyway, gotta drive in. Back online in a bit. | 14:00 |
zigo | Thanks a lot everyone. | 14:00 |
smcginnis | zigo: Thanks for bringing more of the context to our attention. | 14:00 |
*** mriedem1 is now known as mriedem_meeting | 14:00 | |
smcginnis | zigo: We obvious need to get a little more educated here. | 14:00 |
*** haomaiwang has quit IRC | 14:01 | |
eharney | smcginnis: there are probably some packaging-related discussions happening in Tokyo already, just not sure what/where | 14:01 |
smcginnis | eharney: I'll check the schedule for them once it's up. | 14:02 |
smcginnis | eharney: Hopefully we don't have conflicts. | 14:02 |
* smcginnis feels like the summit could be a few days longer to fit everything in | 14:02 | |
*** stevemar has joined #openstack-cinder | 14:02 | |
*** lcurtis has joined #openstack-cinder | 14:03 | |
e0ne | smcginnis: did you mean few weeks? | 14:03 |
smcginnis | e0ne: No doubt. :) | 14:04 |
*** haomaiwa_ has joined #openstack-cinder | 14:04 | |
*** lcurtis has quit IRC | 14:04 | |
*** jungleboyj has quit IRC | 14:04 | |
*** lcurtis has joined #openstack-cinder | 14:04 | |
*** merooney has joined #openstack-cinder | 14:05 | |
*** stevemar_ has joined #openstack-cinder | 14:05 | |
*** mdenny has joined #openstack-cinder | 14:06 | |
*** stevemar has quit IRC | 14:06 | |
dulek | geguileo: I've seen you've answered "done" to my comments on 205834, but you probably forgot to execute git-review. ;) | 14:07 |
*** takedakn1 has joined #openstack-cinder | 14:08 | |
geguileo | dulek: I didn't really forgot to do it... I wanted to see if I can improve something else | 14:09 |
*** takedakn has quit IRC | 14:09 | |
*** takedakn1 is now known as takedakn | 14:09 | |
dulek | geguileo: :) | 14:09 |
geguileo | And I don't want you guys having to review every little change XD | 14:09 |
*** haomaiwa_ has quit IRC | 14:12 | |
*** ociuhandu has joined #openstack-cinder | 14:12 | |
*** haomaiwa_ has joined #openstack-cinder | 14:13 | |
*** dims__ has quit IRC | 14:14 | |
*** dims__ has joined #openstack-cinder | 14:15 | |
*** bswartz has quit IRC | 14:18 | |
*** erhudy has joined #openstack-cinder | 14:19 | |
*** ronis has quit IRC | 14:23 | |
*** takedakn has quit IRC | 14:26 | |
*** thangp has joined #openstack-cinder | 14:28 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/230460 | 14:28 |
*** kevincarr1991 has quit IRC | 14:30 | |
*** haomaiwa_ has quit IRC | 14:33 | |
smcginnis | e0ne: Thanks for the ML list post re: XML API. I'll wait to see if we get any complaints, then approve the spec. | 14:33 |
*** timcl has quit IRC | 14:34 | |
*** kevincarr1991 has joined #openstack-cinder | 14:35 | |
*** setmason has quit IRC | 14:35 | |
*** haomaiwang has joined #openstack-cinder | 14:35 | |
smcginnis | tbarron: Sorry, I know that's a weird one. I don't really like it myself. | 14:36 |
*** martyturner has quit IRC | 14:36 | |
*** bswartz has joined #openstack-cinder | 14:36 | |
e0ne | smcginnis: np. I'm totally agree that we need get feedback first | 14:37 |
tbarron | smcginnis: well, you can make it not puke with "with", you just have to use the right parens for continuation | 14:37 |
tbarron | smcginnis: that said, I think I agree that the rule shouldn't be enforced there | 14:37 |
*** timcl has joined #openstack-cinder | 14:37 | |
smcginnis | tbarron: I could have sworn I tried that when getting rid of contextlib.nested... | 14:37 |
tbarron | smcginnis: cause it's kinda ugly | 14:38 |
tbarron | smcginnis: well, I typed that form into a python 2.7 interpreter and it liked it | 14:38 |
tbarron | just a moment ago | 14:38 |
smcginnis | tbarron: Oh, awesome. Good, maybe if we clean up we can fix it. | 14:38 |
smcginnis | tbarron: We still have a ton of places using \. | 14:39 |
smcginnis | At least it's mostly just in tests. | 14:39 |
tbarron | all that said, I was pushing the point a bit because I personally am often in yuriy's spot and if there are corner-cases where we | 14:39 |
tbarron | don't apply rules, then I wonder whether the rules actually make sense to enforce in the first place | 14:39 |
*** haomaiwang has quit IRC | 14:40 | |
tbarron | seems like a lot of energy goes into that stuff that might be better channeled | 14:40 |
smcginnis | That's certainly true. :\ | 14:40 |
tbarron | I've been doing -1s for commit message line lengths, as I received those myself and thought it was to be enforced. | 14:40 |
nikeshm | hi,one question,i know answer, just for confirmation, currently most of the cinder backup drivers support incremental feature from kilo onwards, can we add this feature in juno? | 14:40 |
tbarron | but in the larger openstack community, there was just a big discussion of that, leaning the other way. | 14:41 |
smcginnis | tbarron: Yeah, I had done that too. Don't know if you saw the LONG ML thread about that. :) | 14:41 |
smcginnis | tbarron: I'm mixed on that. | 14:41 |
tbarron | smcginnis: yes, that is the thread I'm referring to./ | 14:41 |
smcginnis | If it's a guideline then I think it should be enforced. | 14:41 |
tbarron | smcginnis: if we had crisper review standards it would save everybody time. | 14:41 |
smcginnis | But if it's just a recommendation if you feel like following it, maybe we shouldn't have it. | 14:41 |
smcginnis | It sends mixed messages if some -1 on it and some don't. | 14:42 |
tbarron | agreed. | 14:42 |
smcginnis | tbarron: Agree | 14:42 |
*** tsekiyama has joined #openstack-cinder | 14:43 | |
*** stevemar_ is now known as stevemar | 14:43 | |
*** dansmith is now known as superdan | 14:44 | |
nikeshm | ? | 14:45 |
smcginnis | nikeshm: Juno is security fixes only at this point. | 14:47 |
smcginnis | nikeshm: You can certainly backport it yourself and offer it as a custom option for your customers. | 14:47 |
smcginnis | nikeshm: But not officially in tree. | 14:47 |
*** timcl1 has joined #openstack-cinder | 14:53 | |
nikeshm | smcginnis: thanks | 14:53 |
*** ociuhandu has quit IRC | 14:53 | |
*** timcl has quit IRC | 14:54 | |
*** mtanino has joined #openstack-cinder | 14:57 | |
*** mriedem_meeting is now known as mriedem | 14:59 | |
*** edtubill has joined #openstack-cinder | 14:59 | |
*** ociuhandu has joined #openstack-cinder | 15:01 | |
*** setmason has joined #openstack-cinder | 15:02 | |
*** stlbigdog has joined #openstack-cinder | 15:04 | |
*** takedakn has joined #openstack-cinder | 15:04 | |
*** lpetrut has quit IRC | 15:06 | |
openstackgerrit | xing-yang proposed openstack/cinder: VMAX Truncate Storage Group Name https://review.openstack.org/230485 | 15:08 |
*** chlong has joined #openstack-cinder | 15:10 | |
*** lpetrut has joined #openstack-cinder | 15:13 | |
*** alexschm has quit IRC | 15:24 | |
*** takedakn has quit IRC | 15:24 | |
bswartz | eharney: ping | 15:25 |
eharney | bswartz: hi | 15:25 |
*** hemnafk is now known as hemna | 15:27 | |
scottda | I've a change to /etc/cinder/api-paste.ini for my WIP cinder-api-microversions and it needs to be manually copied over from the cinder repo on Devstack in order to get c-api to start.... | 15:28 |
scottda | This is causing Grenade to fail in starting Cinder API. | 15:29 |
scottda | Does anyone know how to fix this? | 15:29 |
*** martyturner has joined #openstack-cinder | 15:29 | |
scottda | i.e. What needs to get done to make the Grenade tests copy over the change to /etc/cinder/api-paste.ini ? | 15:29 |
*** jdurgin1 has joined #openstack-cinder | 15:30 | |
*** vgridnev has quit IRC | 15:31 | |
bswartz | smcginnis: ping | 15:33 |
*** Yogi1 has quit IRC | 15:35 | |
*** lpetrut has quit IRC | 15:36 | |
*** ronis has joined #openstack-cinder | 15:37 | |
scottda | Is there a way to make upgrade perform the copy of api-paste.ini? | 15:37 |
*** setmason has quit IRC | 15:38 | |
dulek | scottda: No. You need to write from-liberty script./ | 15:40 |
dulek | scottda: https://review.openstack.org/#/c/217625/ | 15:40 |
dulek | scottda: Something similar to that. | 15:40 |
dulek | scottda: To get that accepted you need both a PTL approval and a release note entry linked. | 15:41 |
scottda | dulek: Thanks. That helps. | 15:42 |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port netapp SSC Cmode to Python 3 https://review.openstack.org/230503 | 15:42 |
scottda | so, does that mean Grenade would pass with this script? i.e. does that script get run by Grenade? | 15:42 |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port test_netapp to Python 3 https://review.openstack.org/230504 | 15:43 |
*** setmason has joined #openstack-cinder | 15:44 | |
dulek | scottda: Yes, that's the way it works. | 15:44 |
*** harlowja_at_home has joined #openstack-cinder | 15:44 | |
scottda | OK, cool. | 15:45 |
dulek | scottda: Grenade runs from-sth while upgrading sth->foo. | 15:45 |
dulek | scottda: It's annoying you need all that stuff to make a simple change into config files… | 15:45 |
scottda | Yeah, that's OK. At this point, I'm trying to get Jenkins to pass, so your pointer is a big help. | 15:47 |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port test_quobyte to Python 3 https://review.openstack.org/230510 | 15:47 |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port test_netapp to Python 3 https://review.openstack.org/230504 | 15:48 |
e0ne | are anyone looking on failed pytho34 jobs? | 15:48 |
e0ne | I didn't found filed bug | 15:48 |
e0ne | haypo: ^^ | 15:48 |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port netapp SSC Cmode to Python 3 https://review.openstack.org/230503 | 15:48 |
e0ne | I would like to take a look if needed | 15:48 |
haypo | e0ne: what is the problem? | 15:48 |
e0ne | https://jenkins03.openstack.org/job/gate-cinder-python34/1191/ | 15:49 |
haypo | e0ne: oh, i always saw it once | 15:50 |
haypo | e0ne: it may be related to hash randomization | 15:50 |
*** tsekiyama has quit IRC | 15:50 | |
haypo | e0ne: the problem is that i was unable to reproduce the bug locally in .tox/py34/ venv | 15:50 |
e0ne | haypo: I can reproduce it:) | 15:51 |
*** lpetrut has joined #openstack-cinder | 15:51 | |
jgriffith | mgagne: ping | 15:51 |
haypo | e0ne: how? do you run "tox -e py34"? or only test_volume? | 15:52 |
haypo | e0ne: i'm running python -bb -m testtools.run cinder.tests.unit.test_volume in the py34 venv | 15:53 |
e0ne | haypo: "tox -e py34", ubuntu, python 3.4.0 | 15:53 |
haypo | e0ne: can you please try my testtools commmand? | 15:54 |
haypo | e0ne: tox sets the PYTHONHASHSEED to 0 | 15:54 |
e0ne | haypo: sure, one moment | 15:54 |
xyang1 | e0ne: are you talking about this one https://bugs.launchpad.net/cinder/+bug/1501745 | 15:55 |
openstack | Launchpad bug 1501745 in Cinder "gate-cinder-python34 failed" [High,Confirmed] - Assigned to Eric Harney (eharney) | 15:55 |
e0ne | xyang1: yes | 15:56 |
xyang1 | e0ne: eharney made one fix yesterday | 15:56 |
eharney | it didn't work | 15:56 |
xyang1 | eharney: oh | 15:56 |
e0ne | haypo: your command works | 15:58 |
haypo | e0ne: oh, it is supposed to fail :-p | 16:00 |
e0ne | haypo: :( | 16:00 |
smcginnis | Looks like we will have a Liberty RC2 cut early next week. | 16:01 |
e0ne | haypo: http://9gag.com/gag/anYZ9Eo/my-code-works-but-i-don-t-know-why | 16:01 |
smcginnis | Please tag anything that should go back with liberty-backport-potential. | 16:01 |
haypo | e0ne: lol | 16:01 |
xyang1 | smcginnis: so we can backport now? | 16:01 |
smcginnis | xyang1: Only if it makes sense. | 16:01 |
smcginnis | Isolated and safe changes only I think. | 16:01 |
haypo | e0ne: i'm now "fuzzing" the test with various random seeds... | 16:01 |
xyang1 | smcginnis: sure | 16:02 |
eharney | smcginnis: not liberty-rc-potential? | 16:02 |
smcginnis | xyang1: I don't think we want to dump too much in rc2, but if it's already in master and we're going to backport anyway, nows the time. | 16:02 |
smcginnis | eharney: Oh, or that I guess. :) | 16:02 |
e0ne | haypo: I removed everything except cinder.tests.unit.windows.test_smbfs in tox.ini. and now it works:( | 16:03 |
haypo | e0ne: hum, maybe it's an eventlet trap | 16:05 |
eharney | e0ne: haypo: so this current bug looks basically identical to the one fixed here: https://review.openstack.org/#/c/193343/ | 16:07 |
eharney | which was just badly-structured code sharing things it shouldn't have been | 16:07 |
eharney | but i'm not sure why it's happening again | 16:07 |
*** harlowja_at_home has quit IRC | 16:07 | |
haypo | e0ne: hum, maybe it's an eventlet trap | 16:08 |
haypo | oops | 16:08 |
e0ne | eharney: yes, it looks similar | 16:08 |
*** lpetrut has quit IRC | 16:08 | |
*** lpetrut has joined #openstack-cinder | 16:08 | |
haypo | e0ne: https://review.openstack.org/#/c/193343/2/cinder/tests/unit/fake_notifier.py looks good to me | 16:09 |
haypo | e0ne: it cannot be exactly the same bug, since fake_notifier was modified | 16:09 |
*** jdurgin1 has quit IRC | 16:09 | |
eharney | it may be that the CG test has a race where sometimes it finishes before "volume.create.end" notifications are fired and sometimes it doesn't | 16:09 |
*** jordanP has quit IRC | 16:09 | |
eharney | in the case of the previous bug, those notifications were coming from unrelated tests | 16:10 |
haypo | eharney: that makes sense, i don't see why we get a backup here | 16:10 |
haypo | it also explains why the test doesn't fail when it's run alone | 16:11 |
*** jordanP has joined #openstack-cinder | 16:11 | |
*** salv-orlando has joined #openstack-cinder | 16:11 | |
eharney | haypo: the previous bug also never showed up until we turned on concurrency in the gate | 16:11 |
xyang1 | eharney: the CG test along should not have race, but I can take a look | 16:12 |
eharney | xyang1: thanks | 16:14 |
*** apoorvad has joined #openstack-cinder | 16:14 | |
*** kevincarr1991 has quit IRC | 16:14 | |
eharney | haypo: do we have any way to make unit tests run in a more randomized order? | 16:16 |
haypo | eharney: right now, tox.ini uses a sequential list, but i don't know if they are run in this exact order | 16:16 |
eharney | haypo: they aren't, i was just hoping i could purposefully shuffle them around more to try to shake out problems | 16:17 |
haypo | eharney: testtools.run is the low-level stuff under testr, it doesn't have a wide choice of options | 16:17 |
e0ne | eharney: https://review.openstack.org/#/c/193343/2/cinder/test.py, line #144 - does it mock per test case all one mock for all test cases? | 16:18 |
haypo | e0ne: if you are able to reproduce the failure, try to remove test_backup_* in tox.ini | 16:19 |
*** leeantho has joined #openstack-cinder | 16:19 | |
haypo | e0ne: i expect that the backup notifications come from these tests :) | 16:19 |
haypo | i'm now running tox -e py34 instead of testtools | 16:19 |
e0ne | haypo: sounds reasonable:) | 16:19 |
eharney | haypo: are those failures logged somewhere? the ones i saw in the gate were different notifications i think | 16:20 |
eharney | e0ne: well... setUp is run per-test | 16:20 |
*** vgridnev has joined #openstack-cinder | 16:21 | |
haypo | eharney: you can play with http://logstash.openstack.org/, search for project:"openstack/cinder" for example | 16:21 |
haypo | i don't know how to use this tool :-p | 16:21 |
haypo | ask on #openstack-infra | 16:21 |
eharney | haypo: yeah, i've used it a bit | 16:22 |
openstackgerrit | LisaLi proposed openstack/cinder: Copy encryptors from Nova to Cinder https://review.openstack.org/217557 | 16:23 |
*** mudassirlatif has joined #openstack-cinder | 16:23 | |
hemna | eharney, ping | 16:23 |
eharney | hemna: hey | 16:24 |
hemna | this might be unrelated, but I'm seeing strange failures in my os-brick patch related to lio | 16:24 |
haypo | here http://logs.openstack.org/98/229998/1/check/gate-cinder-python34/92f5db2/console.html we got MismatchError: 'consistencygroup.create.start' != 'volume.create.end' | 16:24 |
hemna | eharney, http://logs.openstack.org/99/230199/3/check/gate-cinder-python34/45fb6b1/console.html | 16:24 |
hemna | 2 rechecks = same error | 16:24 |
*** salv-orlando has quit IRC | 16:24 | |
krotscheck | wilson1: There's a comment on https://review.openstack.org/#/c/219489/ for you. | 16:24 |
eharney | hemna: looks like the same thing | 16:24 |
krotscheck | Also, in case anyone wants to help look at a CORS patch :D | 16:25 |
krotscheck | https://review.openstack.org/#/c/219489/ | 16:25 |
eharney | it also always seems to be that test and the same notifications... let me go write a query for this | 16:25 |
hemna | I've not seen this before | 16:25 |
hemna | glanceclient.exc.ServiceUnavailable: ServiceUnavailable (HTTP 503) | 16:25 |
hemna | also that | 16:25 |
*** julim has quit IRC | 16:25 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/230460 | 16:25 |
e0ne | eharney: yes, but does mock returns different objects for tests? | 16:26 |
*** julim has joined #openstack-cinder | 16:26 | |
e0ne | eharney: in othe words: is notifiere mocked only once or each time for every test? | 16:27 |
*** _cjones_ has joined #openstack-cinder | 16:27 | |
eharney | e0ne: it's supposed to be different objects | 16:27 |
e0ne | hemna: hi. I've got few questions about brickclient and attch volume. will you be able here in ~2 hours? | 16:27 |
e0ne | eharney: need to check it | 16:27 |
hemna | e0ne, yah | 16:28 |
e0ne | hemna: cool | 16:28 |
eharney | e0ne: well it did eliminate a rampant gate failure at the time... | 16:28 |
hemna | e0ne, I haven't had time to play with it yet | 16:28 |
hemna | found some replication issues yesterday that I patched and got those in | 16:28 |
hemna | well in gerrit at least | 16:28 |
hemna | but am seeing odd cinder py34 failures now | 16:28 |
e0ne | I need to head home. I'll be online in ~2 hours | 16:28 |
e0ne | hemna: I've got questions about user expirience of this tool. what is expectes and what is not | 16:29 |
hemna | e0ne, ok | 16:29 |
*** timcl1 has quit IRC | 16:30 | |
haypo | i started "tox -e py34" on a fresh cinder clone: it pass... | 16:30 |
haypo | i'm still unable to reproduce the issue on fedora 22 with python 3.4.2 | 16:30 |
mgagne | jgriffith: pong | 16:31 |
*** e0ne has quit IRC | 16:31 | |
haypo | the workaround is to disable test_volume on python 3.4 (remove it from tox.ini) | 16:31 |
*** diogogmt has joined #openstack-cinder | 16:32 | |
haypo | i opened https://bugs.launchpad.net/cinder/+bug/1502229 test_volume randomly fails on Python 3.4 | 16:35 |
openstack | Launchpad bug 1502229 in Cinder "test_volume randomly fails on Python 3.4" [Undecided,New] | 16:35 |
*** yuriy_n17 has quit IRC | 16:35 | |
*** timcl has joined #openstack-cinder | 16:36 | |
*** jordanP has quit IRC | 16:36 | |
*** ociuhandu has quit IRC | 16:39 | |
xyang1 | eharney: are the notifications not isolated per test? | 16:40 |
eharney | xyang1: the fix we made previously was to isolate them per test | 16:40 |
xyang1 | eharney: ok, i don't know how volume can be created before cg | 16:41 |
eharney | xyang1: there's enough complexity in here that maybe they aren't isolated properly anymore | 16:42 |
eharney | i don't really know how to explain this yet :/ | 16:42 |
xyang1 | eharney: I wonder if we should just check if the event happened, but not check the exact order | 16:43 |
apoorvad | jgriffith: Thanks for the insights on 1501976 | 16:43 |
eharney | xyang1: i dunno, i think the idea of how these tests work is the right thing... something just isn't implemented correctly | 16:43 |
*** kevincarr1991 has joined #openstack-cinder | 16:44 | |
*** setmason has quit IRC | 16:47 | |
*** diogogmt_ has joined #openstack-cinder | 16:48 | |
*** dims__ is now known as dimsum__ | 16:49 | |
haypo | ok, i found a search for Kibana: build_name:"gate-cinder-python34" AND build_status:"FAILURE" AND test_create_consistencygroup_from_src | 16:49 |
haypo | type it in http://logstash.openstack.org/ | 16:50 |
*** diogogmt has quit IRC | 16:50 | |
xyang1 | eharney: if you send notification #1 first and immediately send #2, is it possible that #2 will be received before #1 | 16:51 |
eharney | xyang1: some of the failures indicate that the wrong total number of notifications is present | 16:51 |
xyang1 | eharney: so missing notifications | 16:52 |
eharney | xyang1: extra, actually | 16:52 |
eharney | this was one: http://logs.openstack.org/75/228575/3/check/gate-cinder-python34/55881e1/console.html | 16:52 |
*** baumann has quit IRC | 16:52 | |
*** diogogmt_ has quit IRC | 16:53 | |
*** julim has quit IRC | 16:53 | |
xyang1 | I see | 16:54 |
haypo | sorry guys, i have to go. ping me next week if you still hit the issue | 16:54 |
haypo | you can take notes in my bug report: https://bugs.launchpad.net/cinder/+bug/1502229 | 16:54 |
openstack | Launchpad bug 1502229 in Cinder "test_volume randomly fails on Python 3.4" [Undecided,New] | 16:54 |
*** julim has joined #openstack-cinder | 16:54 | |
*** kevincarr1991 has quit IRC | 16:56 | |
*** diablo_rojo has quit IRC | 16:56 | |
*** jgregor has quit IRC | 16:56 | |
*** jgregor has joined #openstack-cinder | 16:58 | |
eharney | https://review.openstack.org/#/c/230128/ is to help debug these failures if anyone would like to review it | 16:59 |
xyang1 | eharney: in this case, backup failed, so it should not send notifications. failed to obtain backup success notification from server. | 17:00 |
*** diogogmt has joined #openstack-cinder | 17:00 | |
xyang1 | eharney: in that log, backup failed for tsm driver, so no notification should be sent out | 17:01 |
eharney | xyang1: which run was that? | 17:01 |
*** harlowja has joined #openstack-cinder | 17:01 | |
xyang1 | eharney: the one you just posted, copy and paste does not work for me | 17:02 |
*** jgregor has quit IRC | 17:02 | |
xyang1 | http://logs.openstack.org/75/228575/3/check/gate-cinder-python34/55881e1/console.html | 17:02 |
eharney | xyang1: i see that message | 17:02 |
xyang1 | eharney: ok | 17:02 |
eharney | xyang1: not really sure if that's related or not, though? | 17:03 |
xyang1 | eharney: i don't know why backup failed there, I just noticed that notification should not be expected | 17:04 |
eharney | xyang1: well there are tsm tests that purposefully fail backups i think | 17:05 |
xyang1 | eharney: Ok, I haven't looked at their tests yet | 17:05 |
*** daneyon has joined #openstack-cinder | 17:06 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/230460 | 17:07 |
*** lpetrut has quit IRC | 17:08 | |
*** daneyon_ has joined #openstack-cinder | 17:10 | |
*** stlbigdog has quit IRC | 17:10 | |
*** jistr has quit IRC | 17:10 | |
*** daneyon has quit IRC | 17:12 | |
*** garthb has joined #openstack-cinder | 17:13 | |
*** ociuhandu has joined #openstack-cinder | 17:14 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/230460 | 17:15 |
*** daneyon_ has quit IRC | 17:17 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/230574 | 17:20 |
*** ociuhandu has quit IRC | 17:22 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/230460 | 17:22 |
bswartz | xyang1: ping | 17:26 |
xyang1 | bswartz: hi | 17:26 |
*** mriedem has quit IRC | 17:28 | |
*** garthb has quit IRC | 17:29 | |
*** daemontool__ has quit IRC | 17:29 | |
*** daemontool__ has joined #openstack-cinder | 17:30 | |
*** mriedem has joined #openstack-cinder | 17:32 | |
*** pv_ has joined #openstack-cinder | 17:32 | |
pv_ | is there any way we can store storage backend specific data in the sql database cinder uses? it would eliminate the need for multiple api calls and stuff for us | 17:33 |
hemna | hehe | 17:33 |
jgriffith | pv_: you can do some things like provider_id | 17:34 |
jgriffith | but depends on what specifically you're looking at | 17:34 |
hemna | jgriffith, didn't we add a generic table to store credentials ? | 17:34 |
hemna | I guess it was specific to storing chap credentials for pure's driver | 17:35 |
apoorvad | pv_: snapshots and volume tables can store provider_id | 17:35 |
hemna | https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/migrate_repo/versions/038_add_driver_initiator_data_table.py | 17:35 |
hemna | I think that was it no ? | 17:35 |
jgriffith | hemna: yes | 17:36 |
hemna | https://github.com/openstack/cinder/commit/f3575eab34a6cd532be7f8862838e9b85dc47fb8 | 17:36 |
jgriffith | hemna: but again I don't know specifically what pv_ is asking :) | 17:36 |
hemna | yah true | 17:36 |
pv_ | ahh i see so it's specific data that is supported | 17:36 |
hemna | it's not uber 'generic' | 17:36 |
pv_ | yeah | 17:36 |
jgriffith | pv_: no | 17:37 |
jgriffith | hemna: no | 17:37 |
jgriffith | pv_: what kind of data do you want to store? | 17:37 |
*** ChanServ has quit IRC | 17:37 | |
jgriffith | Is it initiator/connection data? | 17:37 |
jgriffith | is it mapping Cinder resource ID's to your internal ID's? | 17:38 |
hemna | pv_, that's for data that you want to store and have passed into initialize_connection time though | 17:38 |
jgriffith | or is it something completely different? | 17:38 |
hemna | it's not a general purpose store for driver 'data' | 17:38 |
pv_ | the issue im running into with the way our backend API is set up is to find a specific snapshot, volume or anything without having its ID i have to iterate through a list object of everything before finding the thing i need | 17:38 |
tbarron | we talked about the pure chap case at midcycle Austin and said it would be great to have general purpose driver storage | 17:38 |
tbarron | but it didn't happen | 17:38 |
hemna | tbarron, yah, the fear I thought was it would be a dumping ground | 17:39 |
pv_ | i was talking to my boss and he told me to check with you guys to see if there was a way to upload that info into the sql data base that cinder uses instead of iteratively going through our API every time | 17:39 |
jgriffith | pv_: hemna and that's what I thought, and why I suggested provider_id fields before out little detour through the db :) | 17:39 |
hemna | and drivers would just started talking to the db directly :( | 17:39 |
pv_ | they don't right, its the manager that talks to the db | 17:39 |
pv_ | ? | 17:39 |
pv_ | volume manager | 17:39 |
hemna | pv_, yah supposed to not talk to the db. | 17:39 |
tbarron | hemna: but it should be a way to *keep* drivers from talking to the db, as they could return something to the manager | 17:39 |
hemna | it's 'bad form' | 17:39 |
hemna | but a few drivers in certain cases have to currently. | 17:40 |
pv_ | yeah i remember seeing that somewhere in boldish text, the driver isn't supposed to make db calls | 17:40 |
*** ChanServ has joined #openstack-cinder | 17:40 | |
*** cameron.freenode.net sets mode: +o ChanServ | 17:40 | |
hemna | but we'd like to prevent it if at all possible | 17:40 |
jgriffith | pv_: https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L160 | 17:40 |
tbarron | hemna: pv_: and if they "have" to today, if they had a way to return some private stuff to the manager, manager could stash it for them. | 17:40 |
jgriffith | pv_: https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L470 | 17:40 |
jgriffith | pv_: when you create a volume or snapshot you have the ability to return provider_id in the model | 17:41 |
tbarron | note that I avoided use of the word that must not be said (starts with 'm') and said 'private stuff' instead | 17:41 |
jgriffith | pv_: that is intended to be for example your devices internal ID of the resource | 17:41 |
pv_ | OHHH | 17:41 |
pv_ | yes | 17:41 |
hemna | jgriffith, +1 | 17:41 |
jgriffith | pv_: so that then when say a delete or whatever comes through you can just access that | 17:41 |
pv_ | thats awesome, thats what i need because our id is different than cinder's id scheme | 17:41 |
bswartz | jgriffith: you mean provider_location? | 17:41 |
pv_ | i see thanks | 17:41 |
pv_ | ill definitely look into that | 17:41 |
bswartz | or is provider_id different than provider_location? | 17:41 |
jgriffith | pv_: it's not required, but it's a handy resource for some that can't do look ups very easily | 17:41 |
jgriffith | bswartz: yes! | 17:42 |
pv_ | yeah definitely would help i think | 17:42 |
jgriffith | it's VERY different | 17:42 |
pv_ | thanks guys | 17:42 |
pv_ | :D | 17:42 |
jgriffith | bswartz: which is why it has it's own column :) | 17:42 |
hemna | pv_, and you need to be careful with that during volume migration | 17:42 |
*** rhagarty_ has quit IRC | 17:42 | |
jgriffith | bswartz: provider_location is the actual target info... like iscsi IP, iqn, chap etc | 17:43 |
bswartz | ok | 17:43 |
jgriffith | bswartz: first thought was to put provider id in there, but there's a risk of overflowing the 255 char limit :) | 17:44 |
bswartz | jgriffith: do you recall when provider_id was added? is it new in liberty or kilo? | 17:45 |
apoorvad | bswartz: Kilo | 17:45 |
jgriffith | kilo | 17:45 |
jgriffith | :) | 17:45 |
jgriffith | Yeah... what apoorvad said | 17:45 |
bswartz | ty | 17:46 |
bswartz | that's a good feature | 17:46 |
jgriffith | bswartz: if you consider using it keep in mind there are some pretty big problems with upgrades | 17:46 |
bswartz | what kind of problems? | 17:46 |
jgriffith | bswartz: well, if you change your driver to just use that and dump your old lookups, you might be surprised | 17:46 |
jgriffith | bswartz: after an upgrade for example (to the new driver) you won't have that info | 17:46 |
bswartz | oh right | 17:46 |
jgriffith | bswartz: unless you implement my little update mapping on init | 17:47 |
jgriffith | bswartz: as of L there's a manager call to do that so that's good | 17:47 |
jgriffith | bswartz: honestly I found that it's actually easier/more robust in my case to just keep doing the lookups | 17:47 |
bswartz | yeah I would hope that would go without saying that volumes which existed at upgrade time wouldn | 17:47 |
jgriffith | anyway... | 17:47 |
bswartz | wouldn't have a value in that column | 17:47 |
*** anish__ is now known as anish | 17:47 | |
jgriffith | bswartz: yeah | 17:48 |
jgriffith | bswartz: there's a number of little corner cases | 17:48 |
jgriffith | bswartz: the other is things like a failure on create | 17:48 |
jgriffith | the provider_id field never gets set | 17:48 |
jgriffith | but depending on where/when the failure occured you may have a resource on the backend | 17:48 |
bswartz | oh we have a strategy to address that kind of problem in manila | 17:49 |
bswartz | in manila the driver can attach a model update to the exception it throws | 17:49 |
jgriffith | In my case after looking at a number of corner cases like that it was honestly easier/safer to just keep doing the lookups the way I'd always done | 17:49 |
jgriffith | Until I rewrite my driver to V3 :) | 17:49 |
jgriffith | which means by by backports... so I'm avoiding it | 17:49 |
bswartz | but yeah with no model updates the error state volume will be hard to fix | 17:50 |
jgriffith | yeah, it's easy to just add a "hard lookup on delete if provider-id == None" or something | 17:50 |
jgriffith | but anyway... just little things that in looking at other peoples code they didn't think of | 17:51 |
bswartz | yeah backports suck | 17:51 |
jgriffith | bswartz: indeed... I'd love to rewrite and make a much cleaner/optimized version of my driver | 17:51 |
jgriffith | but then I loose the ability to do "things" | 17:51 |
jgriffith | If my driver were out of tree I could write something super elegant/cool :) | 17:52 |
bswartz | if only there weren't any damn customers.... | 17:52 |
jgriffith | Make an "uber" driver that works for all OpenStack versions | 17:52 |
jgriffith | LOL... yeah, the price of success right :) | 17:52 |
jgriffith | bswartz: my latest experiment is a type of micro-versions for drivers | 17:53 |
jgriffith | bswartz: but the only way I can conceivably see that ever working is by introducing a manager for my own driver | 17:53 |
jgriffith | bswartz: because I know better than to think I'll get everybody to agree/implement it :) | 17:54 |
bswartz | I don't think that out-of-tree drivers is the panacea some people think it is | 17:55 |
bswartz | you missed the lively discussion on that topic in ft. collins | 17:55 |
bswartz | you're right that it would make backports less painful, but it would make compatibility with future cinders much harder | 17:56 |
jgriffith | nahh... makes EVERYTHING easier :) | 17:58 |
bswartz | also nothing prevents you from maintaining and out-of-tree driver in addition to your upstream driver -- netapp does this https://github.com/NetApp/cinder | 17:58 |
jgriffith | bswartz: indeed | 17:58 |
smcginnis | bswartz: Isn't that a maintenance headache? | 17:59 |
bswartz | smcginnis: you have no idea | 17:59 |
tbarron | smcginnis: not for bswartz. he's doing manila :-) | 17:59 |
tbarron | smcginnis: ++ | 18:00 |
bswartz | but it's the only choice if you want to backport to releases that no longer exist upstream | 18:00 |
smcginnis | tbarron: Hah! | 18:00 |
tbarron | yeah, we have to do it b/c we have to support customers more than 6-12 months back | 18:00 |
* akerr still thinks OpenStack should start implementing LTS releases | 18:01 | |
*** e0ne has joined #openstack-cinder | 18:01 | |
bswartz | akerr: I started a ML thread on that and got shot down | 18:01 |
hemna | heh | 18:01 |
tbarron | it would be a big investment by the community, but we at least would benefit. probably helion and rhel osp too, right? | 18:02 |
tbarron | now we all have to maintain backports and fixes downstream. | 18:02 |
e0ne | haypo, eharney: did I miss anything with py34? | 18:03 |
tbarron | smcginnis: does dell or dell/compellent have to do this? | 18:03 |
*** Lee1092 has quit IRC | 18:03 | |
smcginnis | tbarron: Not yet at least. I won't be surprised if we end up in the same situation though. | 18:03 |
eharney | e0ne: i don't think so | 18:04 |
akerr | bswartz: :( | 18:04 |
e0ne | eharney: :(. | 18:04 |
e0ne | eharney: I will take a look on it this weekend or Monday it issue will be still reproducible | 18:05 |
*** delatte has quit IRC | 18:06 | |
*** cdelatte has quit IRC | 18:06 | |
*** gouthamr has quit IRC | 18:06 | |
tbarron | On a different subject, I've recently had occasion to work with backup to swift with ssl/tls enabled. The ssl part wasn't what we were | 18:06 |
*** garthb has joined #openstack-cinder | 18:06 | |
tbarron | testing. We couldn't turn it off. And there were certificate issues that were painful. | 18:07 |
xyang1 | e0ne: do you want to merge this to help debug the problem https://review.openstack.org/#/c/230128 | 18:07 |
e0ne | xyang1: looking on it | 18:07 |
*** ociuhandu has joined #openstack-cinder | 18:07 | |
tbarron | I noticed that glance has an option (like curl -k) to run with https and ignore ssl cert issues. | 18:07 |
tbarron | Seems to me that this might be useful in cinder backup, if only for test purposes. Certainly off by default. | 18:08 |
tbarron | Anyone see problems with implementing this? | 18:08 |
smcginnis | tbarron: Seems reasonable if it is off by default. | 18:08 |
tbarron | smcginnis: thanks for the feedback | 18:09 |
e0ne | eharney: done | 18:09 |
e0ne | hemna: are you still around? | 18:09 |
e0ne | xyang1: done | 18:09 |
xyang1 | e0ne: thanks | 18:10 |
hemna | e0ne, yuh | 18:11 |
e0ne | hemna: so, I've got iscsi PoC and tried to implement rbd attach | 18:11 |
e0ne | hemna: as you said in the comments "user should not may know what protocol do we use" | 18:12 |
hemna | https://github.com/e0ne/python-brickclient | 18:12 |
hemna | is that the latest ? | 18:12 |
e0ne | hemna: yes, rbd part is not ready | 18:12 |
hemna | ok | 18:12 |
hemna | so I think os-brick needs something as well to make sure this works best | 18:12 |
e0ne | hemna: TBH, nothing new, I only moved all code out of cinderclient | 18:12 |
hemna | the client needs a way to test to see which connectors will even work on the host. | 18:13 |
hemna | we kinda know | 18:13 |
hemna | re: FC HBA's need to exist and at least 1 listed as active | 18:13 |
hemna | for a PoC, we can live w/o it for the time being | 18:13 |
e0ne | hemna: good point, noted | 18:14 |
hemna | I had plans on looking at that for M | 18:14 |
e0ne | hemna: this client it mu highest priority at the moment | 18:15 |
hemna | and adding a new connector API that can be called to test to see if it think it's "enabled" on the host | 18:15 |
e0ne | hemna: my quation is: for different protocols, we need to install some tools: open-iscsi, ceph-common, etc | 18:15 |
e0ne | hemna: how should we handle it? | 18:16 |
hemna | and then add get_enabled_connectors() or something and return a dict of enabled/disabled connectors | 18:16 |
hemna | e0ne, yup | 18:16 |
hemna | some connectors need tools installed, drivers enabled, etc | 18:16 |
hemna | FC needs at least 1 active HBA | 18:16 |
*** Yogi1 has joined #openstack-cinder | 18:16 | |
hemna | the vendor specific connectors need their apps installed, etc | 18:16 |
hemna | bleh | 18:16 |
e0ne | hemna: yea, this is a question. we need to document it and let user to install needed software | 18:17 |
hemna | so my plan was to add a new API to the connector to ask if it's viable/enabled something like that | 18:17 |
hemna | then each connector could do it's own test for it's required things | 18:17 |
e0ne | hemna: we can't get it works out of the box w/o additional tools:( | 18:17 |
hemna | that's another good point | 18:18 |
hemna | should update the documentation to the lib to say what's needed for each connector to work | 18:18 |
*** Yogi11 has joined #openstack-cinder | 18:18 | |
e0ne | hemna: ageed | 18:18 |
e0ne | hemna: I'm glad that we are on the same page about this usability issue | 18:19 |
hemna | so for now, I'd ignore the testing of the connector for the PoC | 18:19 |
e0ne | hemna: will you have some time later to implement attachement using FC? | 18:19 |
e0ne | hemna: I didn't have env to test it | 18:19 |
hemna | e0ne, yah I can try it | 18:20 |
hemna | https://github.com/e0ne/python-brickclient/blob/master/brickclient/client.py#L78 | 18:20 |
hemna | probably should add a call to reserve_volume | 18:20 |
e0ne | hemna: cool | 18:20 |
*** Yogi1 has quit IRC | 18:21 | |
e0ne | hemna: noted | 18:21 |
*** daemontool__ has quit IRC | 18:22 | |
hemna | begin_detaching | 18:22 |
e0ne | ok, will do, thanks | 18:22 |
hemna | and try blocks around that | 18:22 |
hemna | expecting exceptions | 18:22 |
hemna | etc etc | 18:22 |
e0ne | hemna: sure | 18:23 |
e0ne | hemna: it's all in my TODO list. I'll add it to get github | 18:23 |
hemna | ok | 18:24 |
e0ne | hemna: thank you for the feedback. I'll update this poc on Monday | 18:24 |
hemna | I'll see if I can do a test | 18:24 |
*** julim has quit IRC | 18:25 | |
e0ne | hemna: I've tested it only throw iscsi. rbd stuff is almost ready. will be ready early next week, Mon or Tue | 18:25 |
e0ne | hemna: please, ping me if you'll get any feedback | 18:26 |
hemna | ok | 18:26 |
e0ne | I'm going to take a break until for the weekend | 18:26 |
e0ne | :) | 18:26 |
e0ne | have a good Friday! | 18:27 |
smcginnis | e0ne: Have a good one! | 18:27 |
*** e0ne has quit IRC | 18:28 | |
*** thangp has quit IRC | 18:29 | |
*** julim has joined #openstack-cinder | 18:30 | |
*** salv-orlando has joined #openstack-cinder | 18:31 | |
*** timcl has quit IRC | 18:33 | |
*** gouthamr has joined #openstack-cinder | 18:37 | |
*** salv-orlando has quit IRC | 18:47 | |
*** timcl has joined #openstack-cinder | 18:47 | |
*** e0ne has joined #openstack-cinder | 18:48 | |
*** dzamboni has quit IRC | 18:52 | |
*** cfriesen has joined #openstack-cinder | 18:56 | |
cfriesen | when using ceph/rbd for cinder volumes, is it safe to set volume_clear=none ? (i.e. will ceph return zeros if the user tries to access parts of their volume that they haven't written to, or will they get the previous contents of the disk at that location?) | 18:58 |
cfriesen | from what I've read RBD uses thin provisioning, which would seem to imply that the above is true. | 19:00 |
*** bswartz has quit IRC | 19:00 | |
eharney | cfriesen: the rbd driver doesn't use that option | 19:01 |
eharney | cfriesen: deletions are handled within ceph, new volumes won't get old data | 19:02 |
cfriesen | eharney: you mean the "volume_clear" option? so when using ceph we don't actually zero out the volumes first even if it's set to volume_clear=zero? | 19:03 |
eharney | cfriesen: correct, only the LVM and block_device drivers use that option to actually do any wiping | 19:03 |
eharney | cfriesen: and that's just because those really need it, in general, storage backends are assumed to handle this sufficiently for us | 19:04 |
*** marcusvrn_ has quit IRC | 19:05 | |
*** julim has quit IRC | 19:05 | |
*** julim has joined #openstack-cinder | 19:05 | |
cfriesen | eharney: okay, thanks for the clarification. Just to confirm, if using LVM with thin provisioning then does it automatically ignore the volume_clear setting or do I have to explicitly set it to none? | 19:06 |
eharney | cfriesen: it ignores it | 19:06 |
cfriesen | eharney: how refreshingly logical. :) | 19:06 |
*** cfriesen has left #openstack-cinder | 19:07 | |
*** e0ne has quit IRC | 19:08 | |
*** erlon has quit IRC | 19:08 | |
*** wolsen has quit IRC | 19:10 | |
*** julim has quit IRC | 19:10 | |
*** julim has joined #openstack-cinder | 19:11 | |
*** merooney has quit IRC | 19:12 | |
*** daemontool_ has joined #openstack-cinder | 19:14 | |
*** julim_ has joined #openstack-cinder | 19:16 | |
*** wolsen has joined #openstack-cinder | 19:16 | |
*** tsekiyama has joined #openstack-cinder | 19:16 | |
openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 19:17 |
*** julim has quit IRC | 19:17 | |
*** e0ne has joined #openstack-cinder | 19:22 | |
*** subscope has joined #openstack-cinder | 19:24 | |
*** subscope_ has joined #openstack-cinder | 19:25 | |
*** marcusvrn_ has joined #openstack-cinder | 19:26 | |
*** martyturner has quit IRC | 19:33 | |
*** stevemar has quit IRC | 19:35 | |
*** david-lyle has quit IRC | 19:35 | |
*** akerr has quit IRC | 19:35 | |
*** merooney has joined #openstack-cinder | 19:35 | |
*** david-lyle has joined #openstack-cinder | 19:35 | |
*** merooney has quit IRC | 19:37 | |
*** porrua has quit IRC | 19:40 | |
*** e0ne has quit IRC | 19:40 | |
openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Add options when uploading images to Glance https://review.openstack.org/230622 | 19:42 |
*** Yogi11 has quit IRC | 19:45 | |
*** david-ly_ has joined #openstack-cinder | 19:45 | |
openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Add options when uploading images to Glance https://review.openstack.org/230622 | 19:47 |
*** david-lyle has quit IRC | 19:49 | |
*** david-ly_ has quit IRC | 19:49 | |
*** apoorvad has quit IRC | 19:59 | |
*** apoorvad has joined #openstack-cinder | 20:12 | |
*** tsekiyama has quit IRC | 20:12 | |
*** akerr has joined #openstack-cinder | 20:16 | |
*** tsekiyama has joined #openstack-cinder | 20:17 | |
nikeshm | hi | 20:21 |
nikeshm | is there any good link for how to use ceph cluster in devstack | 20:22 |
nikeshm | i mean external ceph cluster | 20:23 |
nikeshm | not devstack created ceph cluster | 20:23 |
*** david-lyle has joined #openstack-cinder | 20:24 | |
*** akerr has quit IRC | 20:26 | |
*** bauwser is now known as bauzas | 20:28 | |
openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Add options when uploading images to Glance https://review.openstack.org/230622 | 20:28 |
*** leakypipes has quit IRC | 20:28 | |
*** gouthamr has quit IRC | 20:30 | |
*** ntpttr has joined #openstack-cinder | 20:32 | |
*** timcl has quit IRC | 20:34 | |
*** Yogi1 has joined #openstack-cinder | 20:38 | |
openstackgerrit | Scott DAngelo proposed openstack/cinder: cinder-api-microversions code https://review.openstack.org/224910 | 20:47 |
*** julim_ has quit IRC | 20:47 | |
*** tsekiyama has quit IRC | 20:47 | |
*** julim has joined #openstack-cinder | 20:48 | |
*** setmason has joined #openstack-cinder | 20:52 | |
*** julim has quit IRC | 20:53 | |
*** julim has joined #openstack-cinder | 20:53 | |
*** gouthamr has joined #openstack-cinder | 20:53 | |
*** ronis has quit IRC | 20:56 | |
*** raildo is now known as raildo-afk | 20:56 | |
*** julim has quit IRC | 20:58 | |
*** tsekiyama has joined #openstack-cinder | 20:58 | |
*** tsekiyam_ has joined #openstack-cinder | 21:03 | |
*** dustins has quit IRC | 21:03 | |
*** mtanino has quit IRC | 21:04 | |
*** bswartz has joined #openstack-cinder | 21:05 | |
*** Yogi1 has quit IRC | 21:06 | |
*** ntpttr has quit IRC | 21:06 | |
*** tsekiyama has quit IRC | 21:07 | |
*** tsekiyam_ has quit IRC | 21:07 | |
*** stevemar has joined #openstack-cinder | 21:12 | |
openstackgerrit | xing-yang proposed openstack/cinder: VMAX Target iSCSI IP Address https://review.openstack.org/229712 | 21:13 |
*** ociuhandu has quit IRC | 21:18 | |
*** salv-orlando has joined #openstack-cinder | 21:19 | |
*** gman-tx has joined #openstack-cinder | 21:19 | |
*** kevincarr1991 has joined #openstack-cinder | 21:25 | |
*** kevincarr1991 has quit IRC | 21:26 | |
*** diogogmt_ has joined #openstack-cinder | 21:26 | |
*** salv-orlando has quit IRC | 21:27 | |
*** diogogmt has quit IRC | 21:29 | |
*** diogogmt_ is now known as diogogmt | 21:29 | |
*** netapp has joined #openstack-cinder | 21:32 | |
*** CBADER has quit IRC | 21:41 | |
*** bnemec has joined #openstack-cinder | 21:44 | |
*** jungleboyj has joined #openstack-cinder | 21:44 | |
*** marcusvrn_ has quit IRC | 21:45 | |
*** edtubill has quit IRC | 21:47 | |
*** mriedem has quit IRC | 21:51 | |
*** subscope_ has quit IRC | 21:56 | |
*** xyang1 has quit IRC | 22:03 | |
*** bnemec has quit IRC | 22:06 | |
*** ntpttr has joined #openstack-cinder | 22:07 | |
*** dave-mccowan has quit IRC | 22:08 | |
*** dimsum__ has quit IRC | 22:09 | |
*** gouthamr has quit IRC | 22:10 | |
*** gman-tx has left #openstack-cinder | 22:11 | |
*** stevemar has quit IRC | 22:14 | |
*** stevemar has joined #openstack-cinder | 22:14 | |
*** stevemar has quit IRC | 22:18 | |
*** gman-tx has joined #openstack-cinder | 22:19 | |
*** gman-tx has left #openstack-cinder | 22:25 | |
*** lcurtis has quit IRC | 22:26 | |
*** jwcroppe has quit IRC | 22:26 | |
*** ntpttr has quit IRC | 22:28 | |
*** subscope has quit IRC | 22:30 | |
*** julim has joined #openstack-cinder | 22:34 | |
openstackgerrit | Jay Bryant proposed openstack/cinder: Remove unused 'deprecated' option from auth_strategy opt https://review.openstack.org/230687 | 22:37 |
*** david-lyle has quit IRC | 22:45 | |
*** edmondsw has quit IRC | 22:46 | |
*** david-lyle has joined #openstack-cinder | 22:47 | |
*** jasondotstar is now known as jasondotstar|afk | 22:48 | |
*** gouthamr has joined #openstack-cinder | 22:52 | |
*** setmason has quit IRC | 22:53 | |
*** IanGovett has quit IRC | 22:55 | |
*** jwcroppe has joined #openstack-cinder | 22:58 | |
*** mdenny has quit IRC | 22:59 | |
*** jwcroppe has quit IRC | 22:59 | |
*** jwcroppe has joined #openstack-cinder | 23:00 | |
*** dimsum__ has joined #openstack-cinder | 23:02 | |
*** jwcroppe has quit IRC | 23:05 | |
*** eharney has quit IRC | 23:09 | |
*** julim has quit IRC | 23:10 | |
*** alejandrito has quit IRC | 23:10 | |
*** julim has joined #openstack-cinder | 23:11 | |
*** leitan has quit IRC | 23:15 | |
*** julim has quit IRC | 23:17 | |
*** julim has joined #openstack-cinder | 23:18 | |
*** setmason has joined #openstack-cinder | 23:22 | |
*** julim has quit IRC | 23:23 | |
*** gouthamr has quit IRC | 23:24 | |
*** julim has joined #openstack-cinder | 23:24 | |
*** hemna is now known as hemnafk | 23:24 | |
*** julim has quit IRC | 23:28 | |
*** wN has quit IRC | 23:32 | |
*** garthb has quit IRC | 23:34 | |
*** wN has joined #openstack-cinder | 23:34 | |
*** wN has joined #openstack-cinder | 23:34 | |
*** smcginnis has quit IRC | 23:41 | |
*** Swanson has quit IRC | 23:42 | |
*** dimsum__ has quit IRC | 23:46 | |
*** erhudy has quit IRC | 23:49 | |
*** diogogmt_ has joined #openstack-cinder | 23:51 | |
*** pv_ has quit IRC | 23:51 | |
*** diogogmt has quit IRC | 23:52 | |
*** diogogmt_ is now known as diogogmt | 23:52 | |
*** julim has joined #openstack-cinder | 23:54 | |
*** diogogmt has quit IRC | 23:54 | |
*** diogogmt has joined #openstack-cinder | 23:56 | |
*** setmason has quit IRC | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!