| *** tommylikehu_ has joined #openstack-manila | 00:03 | |
| *** tommylikehu_ has quit IRC | 00:08 | |
| *** xyang_ has quit IRC | 00:09 | |
| *** xyang_ has joined #openstack-manila | 00:16 | |
| *** xyang_ has quit IRC | 00:17 | |
| *** zengyingzhe_ has quit IRC | 00:48 | |
| *** mtanino has quit IRC | 01:55 | |
| openstackgerrit | liusheng proposed openstack/python-manilaclient master: Use more specific asserts in tests https://review.openstack.org/431249 | 02:02 |
|---|---|---|
| *** tommylikehu_ has joined #openstack-manila | 02:04 | |
| *** tommylikehu_ has quit IRC | 02:09 | |
| *** ianychoi has quit IRC | 02:13 | |
| *** ianychoi has joined #openstack-manila | 02:13 | |
| *** gouthamr has quit IRC | 02:16 | |
| *** kberger has quit IRC | 02:33 | |
| *** kberger has joined #openstack-manila | 02:33 | |
| *** tongli has quit IRC | 02:50 | |
| openstackgerrit | liusheng proposed openstack/python-manilaclient master: Use more specific asserts in tests https://review.openstack.org/431249 | 02:54 |
| *** openstackstatus has quit IRC | 02:58 | |
| *** openstackstatus has joined #openstack-manila | 03:01 | |
| *** ChanServ sets mode: +v openstackstatus | 03:01 | |
| *** xyang_ has joined #openstack-manila | 03:38 | |
| *** xyang_ has quit IRC | 03:40 | |
| *** crushil has quit IRC | 04:04 | |
| *** tommylikehu_ has joined #openstack-manila | 04:05 | |
| *** tommylikehu_ has quit IRC | 04:09 | |
| *** crushil has joined #openstack-manila | 04:17 | |
| *** faizy has joined #openstack-manila | 04:35 | |
| openstackgerrit | TommyLike proposed openstack/manila master: Fix: [LVM] Can't extend share with snapshots exist https://review.openstack.org/431315 | 04:53 |
| *** tommylikehu_ has joined #openstack-manila | 05:06 | |
| *** wiebalck has joined #openstack-manila | 05:07 | |
| *** tommylikehu_ has quit IRC | 05:10 | |
| *** mtanino has joined #openstack-manila | 05:13 | |
| *** lpetrut has joined #openstack-manila | 05:20 | |
| *** crushil has quit IRC | 05:21 | |
| *** shausy has joined #openstack-manila | 05:21 | |
| *** sandanar has joined #openstack-manila | 05:44 | |
| openstackgerrit | Béla Vancsics proposed openstack/manila master: Use more specific asserts in tests https://review.openstack.org/430698 | 05:47 |
| *** lpetrut has quit IRC | 06:01 | |
| *** wiebalck has quit IRC | 06:08 | |
| tommylikehu | tbarron, bswartz, vponomaryov, could you take a look at this bug fix patch? https://review.openstack.org/#/c/431315/ I think this one is related to mountable snapshot and revert-to-snapshot features. | 06:11 |
| *** zhonghua has joined #openstack-manila | 06:14 | |
| *** lpetrut has joined #openstack-manila | 06:16 | |
| *** zhonghua2 has quit IRC | 06:16 | |
| *** mtanino has quit IRC | 06:18 | |
| *** markstur has quit IRC | 06:24 | |
| *** markstur has joined #openstack-manila | 06:25 | |
| *** zhonghua2 has joined #openstack-manila | 06:28 | |
| *** kberger has quit IRC | 06:32 | |
| *** ociuhandu has quit IRC | 06:33 | |
| *** zhonghua has quit IRC | 06:33 | |
| *** nkrinner_afk is now known as nkrinner | 06:38 | |
| *** dsariel has joined #openstack-manila | 06:41 | |
| *** tuanluong has joined #openstack-manila | 07:01 | |
| *** ociuhandu has joined #openstack-manila | 07:06 | |
| *** tommylikehu_ has joined #openstack-manila | 07:06 | |
| *** tommylikehu_ has quit IRC | 07:11 | |
| *** pcaruana has joined #openstack-manila | 07:36 | |
| *** makowals has joined #openstack-manila | 07:46 | |
| *** makowals_ has quit IRC | 07:48 | |
| *** gcb has quit IRC | 07:59 | |
| *** jprovazn has joined #openstack-manila | 08:00 | |
| *** faizy has quit IRC | 08:11 | |
| *** faizy has joined #openstack-manila | 08:12 | |
| *** rraja has joined #openstack-manila | 08:35 | |
| *** openstackgerrit has quit IRC | 09:03 | |
| *** tommylikehu_ has joined #openstack-manila | 09:07 | |
| *** tommylikehu_ has quit IRC | 09:12 | |
| *** lpetrut has quit IRC | 09:13 | |
| *** lpetrut has joined #openstack-manila | 09:39 | |
| *** lpetrut has quit IRC | 09:55 | |
| *** yumiriam has joined #openstack-manila | 10:08 | |
| *** gcb has joined #openstack-manila | 10:13 | |
| *** ganso has joined #openstack-manila | 10:13 | |
| *** faizy has quit IRC | 10:13 | |
| *** faizy has joined #openstack-manila | 10:14 | |
| *** alyson_ has joined #openstack-manila | 10:20 | |
| *** jprovazn has quit IRC | 10:31 | |
| *** jprovazn has joined #openstack-manila | 10:31 | |
| *** ociuhandu has quit IRC | 10:32 | |
| *** pcaruana has quit IRC | 10:32 | |
| *** pcaruana has joined #openstack-manila | 10:33 | |
| vponomaryov | tommylikehu: https://review.openstack.org/#/c/431315/ is not finished | 10:47 |
| *** tuanluong has quit IRC | 10:48 | |
| *** jcsp has quit IRC | 10:48 | |
| vponomaryov | tommylikehu: all drivers that support "extend share" feature are broken now except 2 changed in commit | 10:48 |
| *** lpetrut has joined #openstack-manila | 10:49 | |
| vponomaryov | tommylikehu: not saying that unit tests are not updated | 10:49 |
| tommylikehu | vponomaryov: yes, you are right. I add comments there , I uploaded this earlier to collect your response. | 10:50 |
| tommylikehu | vponomaryov: I am not sure whether we should treat it as a bug。。。。 | 10:52 |
| vponomaryov | tommylikehu: everything that may break resources that you use is a bug | 10:52 |
| vponomaryov | everything that has unexpected behavior is a bug | 10:54 |
| tommylikehu | vponomaryov: thanks, let me finish that. | 10:54 |
| vponomaryov | tommylikehu: but I didn't check whether you fix it in proper way or not | 10:54 |
| tommylikehu | vponomaryov: maybe :) | 10:56 |
| ganso | tommylikehu: please see my comments | 11:07 |
| *** tommylikehu_ has joined #openstack-manila | 11:08 | |
| tommylikehu | ganso, vponomaryov: thanks, before the next patch, I have a question about that. | 11:08 |
| tommylikehu | can we just prevent extending share when have snapshot? | 11:08 |
| tommylikehu | do you have any figure that can point out how many driver would support extend a share when snapshots exist? I heard that our product doesn't support that. | 11:11 |
| *** tommylikehu_ has quit IRC | 11:13 | |
| vponomaryov | tommylikehu: yes, if some backends do not support extending having snapshots, we should have possibility to restrict such action | 11:14 |
| vponomaryov | tommylikehu: but it will require one more share "capability" stored in "share DB model" so API could use it | 11:15 |
| vponomaryov | tommylikehu: BUT, "extend" operation is required feature | 11:16 |
| *** shausy has quit IRC | 11:17 | |
| tommylikehu | vponomaryov: yes, I believe this depends on how many driver support this. if most of the driver does not support this. could we just tell the end user that. | 11:17 |
| vponomaryov | http://docs.openstack.org/developer/manila/devref/driver_requirements.html | 11:17 |
| vponomaryov | tommylikehu: so, it is good topic to have discussion about | 11:17 |
| vponomaryov | tommylikehu: also, looks like it has been a problem for long time | 11:18 |
| tommylikehu | vponomaryov: yes, and maybe extend share and extend share with snapshot could be different. | 11:18 |
| tommylikehu | if that is true, we don't need to calculate the quota in snapshot revert either. | 11:19 |
| vponomaryov | tommylikehu: to be able to have better decision on it, we should have concrete knowledge of drivers possibilities in this case | 11:19 |
| vponomaryov | not drivers, backends in general | 11:19 |
| tommylikehu | vponomaryov: I thought we already have webpage for that. | 11:20 |
| vponomaryov | tommylikehu: I am talking about possibilities of real storages to handle extending in both cases | 11:21 |
| vponomaryov | tommylikehu: we cannot have such webpage | 11:21 |
| vponomaryov | tommylikehu: we allowed extending of shares that have snapshots all the time since implementation of "extend" feature | 11:22 |
| tommylikehu | vponomaryov: ok, that would be great if we can discuss this during this week's meeting. | 11:24 |
| *** ociuhandu has joined #openstack-manila | 11:26 | |
| *** dgonzalez has quit IRC | 11:30 | |
| *** xinyanzhang has quit IRC | 11:49 | |
| *** xinyanzhang has joined #openstack-manila | 11:50 | |
| *** openstackgerrit has joined #openstack-manila | 11:50 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila master: [Grenade] Switch base to stable/ocata https://review.openstack.org/431462 | 11:50 |
| *** dgonzalez has joined #openstack-manila | 11:51 | |
| *** faizy has quit IRC | 11:59 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: HNAS: Fix concurrency creating/deleting snapshots https://review.openstack.org/431100 | 12:04 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Add Share Migration devref docs https://review.openstack.org/375771 | 12:19 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila master: [Grenade] Add test with creation of share snapshot https://review.openstack.org/430957 | 12:22 |
| *** mtanino has joined #openstack-manila | 12:32 | |
| *** yankee has quit IRC | 12:35 | |
| *** gcb has quit IRC | 13:00 | |
| *** tommylikehu_ has joined #openstack-manila | 13:09 | |
| *** faizy has joined #openstack-manila | 13:13 | |
| *** tommylikehu_ has quit IRC | 13:14 | |
| *** eharney has quit IRC | 13:20 | |
| ganso | vponomaryov: Hi Valeriy. Could you please take a look at https://review.openstack.org/#/c/431090 ? Thanks in advance | 13:23 |
| *** jprovazn has quit IRC | 13:25 | |
| *** jprovazn has joined #openstack-manila | 13:26 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila master: [Grenade] Switch base to stable/ocata https://review.openstack.org/431462 | 13:27 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila master: [Grenade] Add test with creation of share snapshot https://review.openstack.org/430957 | 13:28 |
| *** xyang_ has joined #openstack-manila | 13:34 | |
| *** porrua has joined #openstack-manila | 13:41 | |
| *** eharney has joined #openstack-manila | 13:44 | |
| *** kberger has joined #openstack-manila | 13:45 | |
| *** porrua_ has joined #openstack-manila | 13:46 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila master: [Grenade] Add test with creation of share snapshot https://review.openstack.org/430957 | 13:49 |
| *** porrua has quit IRC | 13:49 | |
| *** gouthamr has joined #openstack-manila | 13:56 | |
| *** tommylikehu1 has joined #openstack-manila | 13:59 | |
| *** tommylikehu has quit IRC | 13:59 | |
| *** tommylikehu1 is now known as tommylikehu | 13:59 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila master: [Grenade] Add test with creation of share snapshot https://review.openstack.org/430957 | 14:02 |
| *** tommylikehu_ has joined #openstack-manila | 14:05 | |
| openstackgerrit | Lucian Petrut proposed openstack/manila master: Fix Windows SMB helper https://review.openstack.org/424112 | 14:07 |
| *** jprovazn has quit IRC | 14:07 | |
| *** jprovazn has joined #openstack-manila | 14:09 | |
| *** wiebalck has joined #openstack-manila | 14:12 | |
| *** wiebalck has quit IRC | 14:16 | |
| *** dustins has joined #openstack-manila | 14:20 | |
| *** dustins_ has joined #openstack-manila | 14:25 | |
| *** dustins has quit IRC | 14:28 | |
| *** catintheroof has joined #openstack-manila | 14:31 | |
| *** gcb has joined #openstack-manila | 14:40 | |
| *** crushil has joined #openstack-manila | 14:47 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila master: [Grenade] Switch base to stable/ocata https://review.openstack.org/431462 | 14:51 |
| vponomaryov | lpetrut: are you aware that windows driver third-party CI does not report results to openstack CI? | 14:57 |
| *** cknight has joined #openstack-manila | 14:59 | |
| gouthamr | bswartz: meeting time | 15:00 |
| *** rraja has quit IRC | 15:01 | |
| tommylikehu_ | canceled? | 15:01 |
| *** dustins_ is now known as dustins | 15:01 | |
| bswartz | --> #openstack-meeting-alt <-- | 15:01 |
| *** wiebalck has joined #openstack-manila | 15:02 | |
| lpetrut | vponomaryov: Hi, yep, thanks for the heads up. We had some issues lately after some hardware updates but as far as I know it should be back up now. | 15:05 |
| smcginnis | bswartz: Good point on that schedule timing. | 15:05 |
| lpetrut | vponomaryov: while the CI was down, the driver got broken by this change: I6e7eca54725d1aef994d617f253e73909a9d8303 I've sent this fix on this topic: https://review.openstack.org/#/c/424112/ | 15:06 |
| *** jprovazn has quit IRC | 15:14 | |
| *** sandanar has quit IRC | 15:26 | |
| *** rraja has joined #openstack-manila | 15:36 | |
| *** rraja_ has joined #openstack-manila | 15:38 | |
| *** rraja has quit IRC | 15:38 | |
| *** rraja_ has quit IRC | 15:42 | |
| *** rraja_ has joined #openstack-manila | 15:42 | |
| ianychoi | tbarron, bswartz hi :) | 15:44 |
| tbarron | ianychoi: hi | 15:44 |
| ianychoi | I have been accidently seeing your irc meeting. Everything is fine for I18n? | 15:45 |
| bswartz | ianychoi: yes I think so | 15:45 |
| bswartz | guys have we seen any zanata patches lately? | 15:45 |
| * bswartz goes to look | 15:45 | |
| tbarron | ianychoi: have you noticed anything wrong? | 15:46 |
| ianychoi | One thing I would like to tell is that.. could you assign one member for i18n - manila cross project liaison? https://wiki.openstack.org/wiki/CrossProjectLiaisons#I18n | 15:46 |
| bswartz | ianychoi: I will ask for volunteers | 15:46 |
| bswartz | I know vponomaryov has done a little translation work | 15:46 |
| bswartz | not sure if the i18n meetings happen at a good time for him | 15:47 |
| ianychoi | bswartz, thanks a lot. Yep I have seen that stable/ocata branch sync with manila-ui works pretty well: https://review.openstack.org/#/c/430963/ | 15:47 |
| bswartz | oh yes, the UI project has seen a ton of patches | 15:47 |
| ianychoi | bswartz, attending to i18n meeting would not be a requirement. :) | 15:47 |
| bswartz | thank you guys for that | 15:47 |
| ianychoi | It's my great pleasure. Previously vponomaryov tried to accept more qualitifed translation on Russian. I think he will do i18n - manila cross project liaison well :) | 15:48 |
| *** wiebalck has quit IRC | 15:53 | |
| *** tommylikehu_ has quit IRC | 15:56 | |
| vponomaryov | wait wait wait, contribute to project and be liaison is two big differences | 16:01 |
| vponomaryov | ianychoi: where list of responsibilities can be found? | 16:01 |
| ianychoi | vponomaryov, hello: https://wiki.openstack.org/wiki/CrossProjectLiaisons#I18n | 16:02 |
| *** porrua_ has quit IRC | 16:02 | |
| ianychoi | vponomaryov, just sharing manila (especially manila-ui) release plan with i18n team. | 16:03 |
| ianychoi | I think actual patching with translation would be optional | 16:03 |
| ianychoi | I think you have more knowledge on syncing with i18n :) Doesn't it, vponomaryov? :) | 16:04 |
| *** erlon has joined #openstack-manila | 16:06 | |
| vponomaryov | ianychoi: me? syncing with i18n? more knowledge? it is some mistake ) | 16:06 |
| *** porrua has joined #openstack-manila | 16:08 | |
| ianychoi | vponomaryov, I meant experience like https://review.openstack.org/#/c/379994/ | 16:08 |
| ganso | gouthamr: ping | 16:11 |
| vponomaryov | ianychoi: just coincidence , I was just passing by )) | 16:11 |
| *** nkrinner is now known as nkrinner_afk | 16:12 | |
| ianychoi | vponomaryov, yep becoming you as i18n-manila liaison is just from my proposal. But.. I would like kindly to ask your team to nominate one cross project liaison with i18n team. | 16:13 |
| vponomaryov | ianychoi: it is good question for our weekly meeting. Next one will be in 6 days, 22 hours and 47 minutes | 16:14 |
| ianychoi | There are so many translators but they live in so different area, and I believe that the liaison would help better conversation with i18n and more actual projects whose teams are lots of aware with i18n :) | 16:15 |
| vponomaryov | ianychoi: just for clarification, liaison will be assigned to group of projects? not for each of them? For example "manila" and "manila-ui" | 16:16 |
| ianychoi | Now I have been updating the information on https://wiki.openstack.org/wiki/CrossProjectLiaisons#I18n and I accidently see manila team meeting. :) | 16:16 |
| vponomaryov | ianychoi: someone would say: "It is the Sign!" | 16:16 |
| ianychoi | vponomaryov, it would be for per official project so "manila" would be a good idea, but I am fine with just considering with manila-ui :) | 16:17 |
| ianychoi | Since.. I18n now tends to more focus on user facing projects such as horizon and plugin projects | 16:17 |
| ianychoi | I do not want more pressure to i18n cross project liaison members, but I would like to know more on stable branch creation progress from so many different projects. | 16:18 |
| ianychoi | For example, some horizon plugin projects follow cycle-with-intermidiary release model. I did not know such thing.. sorrily. | 16:19 |
| gouthamr | ganso: pong | 16:19 |
| ianychoi | Then.. I18n team needs to more take care of when each project which is interested in translation will have stable branch | 16:19 |
| ianychoi | This is my main purpose :) | 16:20 |
| openstackgerrit | Lucian Petrut proposed openstack/manila master: Fix Windows SMB helper https://review.openstack.org/424112 | 16:20 |
| ganso | gouthamr: Hi Goutham. I noticed there is a adminref folder within the docs folder in manila repo, I am assuming I am not supposed to use that one instead of the admin-guide folder from openstack-manuals | 16:20 |
| bswartz | ianychoi: I put it on next week's meeting agenda: https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting | 16:21 |
| vponomaryov | lpetrut, I guess you want this bugfix be backported to Ocata? | 16:21 |
| gouthamr | ganso: for now put it in openstack-manuals | 16:21 |
| ganso | gouthamr: yes... but I see migration in this page http://docs.openstack.org/admin-guide/shared-file-systems.html but I do not see migration here https://github.com/openstack/openstack-manuals/tree/master/doc/admin-guide/source | 16:21 |
| ianychoi | bswartz, vponomaryov: thanks a lot for considering with I18n! :) | 16:21 |
| vponomaryov | bswartz: is broken windows driver good reason to have RC2? | 16:21 |
| ganso | gouthamr: so I am a bit confused | 16:21 |
| gouthamr | ganso: i'm following the discussion in the docs team to try to move admin guide into project-repos.. | 16:21 |
| ganso | gouthamr: oh so they are going to be moved to project repos? | 16:22 |
| ganso | gouthamr: why not create doc in project repo already then? | 16:22 |
| vponomaryov | bswartz: https://bugs.launchpad.net/manila/+bug/1663300 and https://review.openstack.org/#/c/424112/ | 16:23 |
| openstack | Launchpad bug 1663300 in Manila "Broken Windows share helper" [Undecided,In progress] - Assigned to Lucian Petrut (petrutlucian94) | 16:23 |
| gouthamr | ganso: they're still working out the details. admin guides are only published from openstack-manuals | 16:23 |
| gouthamr | ganso: the one in our repo is linked from within the devref | 16:23 |
| ganso | gouthamr: hmmm | 16:23 |
| gouthamr | ganso: your file is here: https://github.com/openstack/openstack-manuals/blob/11f711ce0f9f65fc0d77860b8c27d3b8702e399a/doc/admin-guide/source/shared-file-systems-manage-shares-cli.rst | 16:23 |
| vponomaryov | bswartz: if yes, then we may want to include LVM driver extend share feature fix too | 16:24 |
| ganso | gouthamr: why the hell is it named manage-shares-cli ? o_O | 16:24 |
| ganso | gouthamr: that would be the name for the user-guide for "manage_share" | 16:24 |
| lpetrut | vponomaryov: yep, I'll send a backport as soon as it merges on master | 16:31 |
| vponomaryov | lpetrut: then it should be marked with "ocata-backport-potential" | 16:32 |
| gouthamr | ganso: not sure.. you can delete that file and create a new one.. | 16:34 |
| lpetrut | vponomaryov: good point, I've just set the tag. thanks | 16:34 |
| ganso | gouthamr: yea I'm renaming it | 16:34 |
| ganso | gouthamr: thanks | 16:34 |
| gouthamr | ganso: np.. will take a look again at your devref change tonight.. i was seeking more clarification around the process and the options (--writable/nondisruptive) | 16:35 |
| vponomaryov | lpetrut: emmm, where? I was talking about https://bugs.launchpad.net/manila/+bug/1663300 )) | 16:36 |
| openstack | Launchpad bug 1663300 in Manila "Broken Windows share helper" [Undecided,In progress] - Assigned to Lucian Petrut (petrutlucian94) | 16:36 |
| ganso | gouthamr: humm please add comments about those if you feel the latest PS should be improved | 16:36 |
| ganso | gouthamr: I think I addressed your previous comments | 16:36 |
| ganso | gouthamr: or at least I tried to xD | 16:36 |
| gouthamr | ganso: sure thing | 16:36 |
| lpetrut | vponomaryov: hmm, for some reason launchpad did not set it, it should be ok now | 16:37 |
| ganso | gouthamr: I may not have succeded xD | 16:37 |
| vponomaryov | lpetrut: yes, now it is ok ) | 16:37 |
| vponomaryov | lpetrut: welcome to Manila World! ^_^ | 16:37 |
| gouthamr | ganso: :P | 16:37 |
| vponomaryov | welcome "back" ) | 16:37 |
| lpetrut | vponomaryov: :D | 16:38 |
| gouthamr | vponomaryov: if something doesn't have a reno, you will see a -1 from me. Aren't you getting used to it by now? | 16:38 |
| vponomaryov | gouthamr: ? | 16:39 |
| vponomaryov | gouthamr: your question would be understandable only in case you consider lpetrut's question as mine | 16:41 |
| gouthamr | vponomaryov: grrr.. my bad: "Would not say this worth -1" was on a styling nitpick | 16:42 |
| gouthamr | vponomaryov: but the -1 was for a missing bug/reno | 16:42 |
| lpetrut | gouthamr: vponomaryov: I'm a bit confused about the release notes usage, trivial bug fixes usually don't require release notes but if manila requests so, I can add this | 16:42 |
| gouthamr | lpetrut: i don't believe this is trivial.. | 16:42 |
| vponomaryov | lpetrut: fix of broken driver is not "trivial" )) | 16:42 |
| *** xyang_ has quit IRC | 16:43 | |
| lpetrut | vponomaryov: gouthamr: well, if you put it that way :D | 16:43 |
| gouthamr | +1 lpetrut: we'd like for your users/deployers to know you fixed the issue with the last milestone/release :) | 16:43 |
| gouthamr | s/your/our | 16:43 |
| lpetrut | got it, adding a release note right now. thanks for the clarifications | 16:44 |
| vponomaryov | caught train that has already left the station? )) | 16:44 |
| gouthamr | in software, there's another train in no time. | 16:44 |
| gouthamr | in some softwares :P | 16:45 |
| lpetrut | that train some usually comes once every 6 months :) | 16:45 |
| ganso | vponomaryov: anything holding back your +W ? https://review.openstack.org/#/c/431090 | 16:48 |
| vponomaryov | ganso: does it block you? | 16:48 |
| ganso | vponomaryov: my tester cherry picked the change to complete his tests... so I am just wondering what's holding you back | 16:49 |
| vponomaryov | ganso: workflowed, just was giving some time for others | 16:49 |
| ganso | vponomaryov: well it is trivial, we agreed trivial changes do not need to wait for 24h | 16:50 |
| openstackgerrit | Lucian Petrut proposed openstack/manila master: Fix Windows SMB helper https://review.openstack.org/424112 | 16:50 |
| vponomaryov | ganso: did I say anything about 24h? ) | 16:50 |
| vponomaryov | ganso: also, tag "trivial" does nto mean it is "really" trivial | 16:51 |
| vponomaryov | so, may require some time | 16:51 |
| ganso | vponomaryov: if it doesn't then it is incorrectly tagged | 16:51 |
| *** xyang_ has joined #openstack-manila | 16:52 | |
| vponomaryov | ganso: I mean "trivial" or "not trivial" is dependent on human judgement | 16:52 |
| vponomaryov | lpetrut: "Write in past tense, unless you are writing an imperative statement" | 16:54 |
| vponomaryov | lpetrut: it is from http://docs.openstack.org/developer/manila/devref/adding_release_notes.html | 16:54 |
| lpetrut | vponomaryov: are you talking about the release note? | 16:55 |
| vponomaryov | lpetrut: yes | 16:55 |
| vponomaryov | lpetrut: also, it is too ambigious | 16:56 |
| lpetrut | vponomaryov: 'has been fixed' sounds like a past tense to me :) I was not sure how much details should be specified there | 16:56 |
| vponomaryov | lpetrut: better to say that creation of access rules was fixed | 16:56 |
| vponomaryov | lpetrut: it is not clear for user what does "helper" do | 16:57 |
| vponomaryov | lpetrut: and should not be required to be clear ) | 16:57 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Add Share Migration devref docs https://review.openstack.org/375771 | 16:58 |
| lpetrut | vponomaryov: ok, I'll go with that | 16:59 |
| lpetrut | although, in this case, the share creation itself was failing due to the incorrect signature | 17:00 |
| vponomaryov | lpetrut: moreover, it sounds like "windows driver was resurrected" | 17:01 |
| vponomaryov | lpetrut: so, creation of shares and allowing/denying access was fixed from user-point-of-view | 17:01 |
| lpetrut | well, that wouldn't be far from truth unfortunately. I'm not really sure what's the story about share access rules, we did have some issues on this some time ago but those were fixed in Newton | 17:04 |
| *** catintheroof has quit IRC | 17:06 | |
| *** catintheroof has joined #openstack-manila | 17:07 | |
| openstackgerrit | Lucian Petrut proposed openstack/manila master: Fix Windows SMB helper https://review.openstack.org/424112 | 17:08 |
| *** catintheroof has quit IRC | 17:11 | |
| gouthamr | bswartz: for some more "formatted" etherpads: https://ethercalc.openstack.org | 17:24 |
| bswartz | ??? | 17:24 |
| bswartz | just a spreadsheet version of etherpad? | 17:24 |
| gouthamr | bswartz: yeah... probably if you ever felt like our etherpad need better formatting | 17:25 |
| bswartz | what etherpad needs is to be indexed by google | 17:26 |
| bswartz | anything you write on an etherpad can't be searched by search engines | 17:26 |
| *** crushil has quit IRC | 17:26 | |
| ganso | gouthamr: it does not show different colors for other users | 17:27 |
| gouthamr | bswartz: that'd be bad.. we love etherpads too much, we sometimes let designs just sit there instead of in specs and real documentation | 17:27 |
| ganso | gouthamr: nor their names | 17:27 |
| gouthamr | ganso: ah.. you're right | 17:28 |
| *** pcaruana has quit IRC | 17:30 | |
| bswartz | I don't like how easy it is to vandalize etherpads | 17:31 |
| bswartz | not that it happens much, but the threat of it makes it a poor choice for some kinds of information | 17:31 |
| *** Yogi1 has joined #openstack-manila | 17:33 | |
| ganso | gouthamr: also graph functionality is not working very well for me | 17:36 |
| gouthamr | ganso: i didn't explore it completely. we need to file bugs :P | 17:37 |
| *** crushil has joined #openstack-manila | 17:41 | |
| ganso | gouthamr: file bugs in ethercalc? :P | 17:42 |
| *** Administrator__ has joined #openstack-manila | 17:43 | |
| ganso | bswartz, gouthamr: btw my team mate found a rather nasty bug in python-manilaclient | 17:45 |
| ganso | bswartz, gouthamr: how do RCs work for those? | 17:45 |
| bswartz | ganso: there are no RCs for libs, just fix it in master and we'll backport it and issue a stable release | 17:46 |
| ganso | bswartz: cool, thanks | 17:46 |
| bswartz | ganso: the part I don't know is if we can get a requirements update in stable/ocata | 17:46 |
| *** zhugaoxiao has quit IRC | 17:46 | |
| bswartz | but as long as u-c.txt is high enough the bugfix will actually get picked up and used | 17:46 |
| *** xyang_ has quit IRC | 17:46 | |
| gouthamr | ganso: how nasty? | 17:47 |
| ganso | gouthamr: #link https://bugs.launchpad.net/python-manilaclient/+bug/1662988 | 17:47 |
| openstack | Launchpad bug 1662988 in python-manilaclient "Extra-specs are shown with incorrect values for non-admin tenants" [Undecided,New] | 17:47 |
| gouthamr | ganso: not nasty | 17:48 |
| gouthamr | ganso: that's by design, lol | 17:48 |
| ganso | gouthamr: whaaa? | 17:48 |
| gouthamr | ganso: "optional public extra-specs" are special in manila | 17:49 |
| gouthamr | ganso: quoting from cknight's spec from a long time ago | 17:49 |
| ganso | gouthamr: my team mate was so confused that he did not define an extra-spec so it has the don't care behavior... but the user does not see the don't care behavior | 17:49 |
| gouthamr | ganso: if you don't request them, shares created can be put on backends that may or may not support the feature you requested | 17:49 |
| ganso | gouthamr: the user sees it as "False" | 17:49 |
| bswartz | ganso: that's how it's designed | 17:50 |
| gouthamr | ganso: but the user will always see it as "False", i.e, not capable of that extra-spec | 17:50 |
| *** wiebalck has joined #openstack-manila | 17:50 | |
| bswartz | if you don't ask for it, you don't get it | 17:50 |
| ganso | gouthamr: but the scheduler does not behave according to what's shown there | 17:50 |
| gouthamr | ganso: the bug is that we didn't document it probably | 17:50 |
| *** ociuhandu has quit IRC | 17:51 | |
| bswartz | "don't care" means the share can get scheduled anywhere and the user can't use the feature no matter where the share lands -- if that's not what the admin intends then the admin needs to select True or False | 17:51 |
| gouthamr | ganso: doesn't behave accordingly? | 17:51 |
| gouthamr | ganso: ^ bswartz said it better | 17:51 |
| openstackgerrit | Merged openstack/manila master: Optimize opposite driver modes migration test https://review.openstack.org/431090 | 17:52 |
| ganso | bswartz, gouthamr: thing is, if the admin says a share_type has revert_to_snapshot_support=False, means it can only land in a backend that has that extra-spec False | 17:52 |
| bswartz | ganso: correct | 17:52 |
| ganso | bswartz, gouthamr: that is not a don't care | 17:52 |
| bswartz | ganso: correct | 17:52 |
| bswartz | where is the confusion then? | 17:53 |
| ganso | bswartz, gouthamr: so when the tenant user is using command "type-list", he is seeing like that, instead of not seeing the extra-spec defined | 17:53 |
| ganso | bswartz, gouthamr: if the admin uses "type-list", the admin sees it correctly, the extra-spec is not shown, thus "don't care behavior", omitted | 17:53 |
| bswartz | ganso: yes that's also what we designed | 17:54 |
| bswartz | basically end users can't tell the difference between False and don't care because from their perspective they're exactly the same | 17:54 |
| ganso | bswartz: showing different output like this is very confusing | 17:55 |
| *** xyang_ has joined #openstack-manila | 17:55 | |
| ganso | bswartz: it is like overloading the meaning of type-list | 17:55 |
| bswartz | ganso: it's only confusing if you expect end users and admins to have the same view | 17:55 |
| bswartz | ganso: we decided that they should have different views | 17:56 |
| bswartz | perhaps it would have been more clear to split the API entirely so there were different APIs for admins | 17:56 |
| *** xyang_ has quit IRC | 17:57 | |
| ganso | bswartz: yes, perhaps so | 17:57 |
| *** lseki has joined #openstack-manila | 17:57 | |
| bswartz | we inherited some API/CLI design from cinder here, as well as some of cinder's traditions around extra specs, but we also decided to make some manila-specific changes | 17:58 |
| bswartz | we never took a step back and asked how we would have designed these APIs from scratch though | 17:59 |
| gouthamr | ganso: responded to the bug report with the info | 17:59 |
| *** xyang_ has joined #openstack-manila | 17:59 | |
| ganso | gouthamr: thanks | 18:00 |
| markstur | should have a #blamecinder tag ^ | 18:00 |
| ganso | markstur: +1 | 18:01 |
| bswartz | markstur: +1 | 18:01 |
| *** xyang_ has quit IRC | 18:04 | |
| *** xyang_ has joined #openstack-manila | 18:05 | |
| *** lpetrut has quit IRC | 18:11 | |
| *** zhugaoxiao has joined #openstack-manila | 18:15 | |
| *** Administrator__ has quit IRC | 18:18 | |
| *** catintheroof has joined #openstack-manila | 18:26 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila master: [Grenade] Add test with creation of share snapshot https://review.openstack.org/430957 | 18:27 |
| *** lpetrut has joined #openstack-manila | 18:54 | |
| *** yumiriam has quit IRC | 19:01 | |
| *** lpetrut has quit IRC | 19:01 | |
| *** gouthamr has quit IRC | 19:23 | |
| *** lpetrut has joined #openstack-manila | 19:34 | |
| *** faizy has quit IRC | 19:39 | |
| *** porrua has quit IRC | 19:42 | |
| *** gouthamr has joined #openstack-manila | 19:52 | |
| -openstackstatus- NOTICE: Restarting gerrit due to performance problems | 20:20 | |
| *** crushil has quit IRC | 20:28 | |
| *** xyang_ has quit IRC | 20:29 | |
| *** harlowja_ has joined #openstack-manila | 20:35 | |
| *** crushil has joined #openstack-manila | 20:35 | |
| *** xyang_ has joined #openstack-manila | 20:36 | |
| *** harlowja has quit IRC | 20:37 | |
| *** xyang_ has quit IRC | 20:47 | |
| *** gregsfortytwo has quit IRC | 20:54 | |
| *** xyang_ has joined #openstack-manila | 20:54 | |
| *** gregsfortytwo has joined #openstack-manila | 20:55 | |
| *** xyang_ has quit IRC | 21:04 | |
| *** alyson_ has quit IRC | 21:07 | |
| *** rraja_ has quit IRC | 21:09 | |
| *** porrua has joined #openstack-manila | 21:17 | |
| *** wiebalck has quit IRC | 21:19 | |
| *** ociuhandu has joined #openstack-manila | 21:20 | |
| *** ganso has quit IRC | 21:25 | |
| *** ociuhandu has quit IRC | 21:32 | |
| *** gouthamr has quit IRC | 21:38 | |
| *** Yogi1 has quit IRC | 21:38 | |
| openstackgerrit | Merged openstack/manila master: [Grenade] Switch base to stable/ocata https://review.openstack.org/431462 | 21:45 |
| openstackgerrit | Merged openstack/manila master: [Grenade] Add test with creation of share snapshot https://review.openstack.org/430957 | 21:46 |
| *** cknight has quit IRC | 21:50 | |
| *** harlowja_ has quit IRC | 21:58 | |
| *** xyang_ has joined #openstack-manila | 21:58 | |
| *** xyang_ has quit IRC | 22:05 | |
| *** harlowja has joined #openstack-manila | 22:05 | |
| *** lseki has quit IRC | 22:10 | |
| *** dustins has quit IRC | 22:13 | |
| *** gouthamr has joined #openstack-manila | 22:17 | |
| *** xyang_ has joined #openstack-manila | 22:24 | |
| *** crushil has quit IRC | 22:33 | |
| *** xyang_ has quit IRC | 22:39 | |
| *** xyang_ has joined #openstack-manila | 22:40 | |
| *** xyang_ has quit IRC | 22:45 | |
| *** lpetrut has quit IRC | 22:48 | |
| *** draynium has joined #openstack-manila | 23:10 | |
| *** xyang_ has joined #openstack-manila | 23:22 | |
| *** porrua has quit IRC | 23:27 | |
| *** xyang_ has quit IRC | 23:28 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!