| *** david-lyle has joined #openstack-manila | 00:08 | |
| *** david-lyle has quit IRC | 00:15 | |
| *** cknight has joined #openstack-manila | 00:18 | |
| *** catintheroof has quit IRC | 00:46 | |
| *** david-lyle has joined #openstack-manila | 00:54 | |
| *** david-lyle has quit IRC | 00:54 | |
| *** cknight has quit IRC | 00:59 | |
| *** mtanino has quit IRC | 01:35 | |
| *** tuanluong has joined #openstack-manila | 01:40 | |
| *** david-lyle has joined #openstack-manila | 01:52 | |
| *** tommylikehu has quit IRC | 02:12 | |
| *** erlon has quit IRC | 02:25 | |
| *** tommylikehu has joined #openstack-manila | 02:29 | |
| *** tinwood has quit IRC | 04:05 | |
| *** tinwood has joined #openstack-manila | 04:06 | |
| *** cknight has joined #openstack-manila | 04:46 | |
| *** gouthamr has quit IRC | 04:46 | |
| *** cknight has quit IRC | 05:22 | |
| *** rraja has joined #openstack-manila | 05:59 | |
| *** nkrinner_afk is now known as nkrinner | 05:59 | |
| *** markstur has quit IRC | 06:35 | |
| *** markstur has joined #openstack-manila | 06:36 | |
| *** markstur has quit IRC | 06:37 | |
| *** lpetrut has joined #openstack-manila | 07:02 | |
| *** sandanar has joined #openstack-manila | 07:12 | |
| openstackgerrit | Thomas Bechtold proposed openstack/manila master: [api-ref]: Add missing share statuses https://review.openstack.org/437149 | 07:15 |
|---|---|---|
| *** a-pugachev has joined #openstack-manila | 07:19 | |
| *** vkmc has quit IRC | 07:23 | |
| *** jprovazn has joined #openstack-manila | 07:24 | |
| *** dsariel has joined #openstack-manila | 07:28 | |
| *** vkmc has joined #openstack-manila | 07:29 | |
| *** databus23_ has joined #openstack-manila | 07:34 | |
| *** mkoderer_ has joined #openstack-manila | 07:34 | |
| *** david_1 has joined #openstack-manila | 07:34 | |
| *** databus23_ has quit IRC | 07:36 | |
| *** mkoderer_ has quit IRC | 07:36 | |
| *** david_1 has quit IRC | 07:36 | |
| *** swap-nilangekar has joined #openstack-manila | 07:41 | |
| *** kaisers_ has joined #openstack-manila | 07:54 | |
| *** a-pugachev has quit IRC | 07:56 | |
| *** lpetrut has quit IRC | 08:02 | |
| *** makowals has quit IRC | 08:06 | |
| *** makowals has joined #openstack-manila | 08:07 | |
| *** arnewiebalck has quit IRC | 08:10 | |
| *** arnewiebalck has joined #openstack-manila | 08:10 | |
| *** markstur has joined #openstack-manila | 08:15 | |
| *** swap-nilangekar has quit IRC | 08:16 | |
| *** lpetrut has joined #openstack-manila | 08:17 | |
| *** markstur has quit IRC | 08:19 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/manila master: Indicating the location tests directory in oslo_debug_helper https://review.openstack.org/443495 | 08:47 |
| *** lpetrut has quit IRC | 08:47 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/manila-ui master: Indicating the location tests directory in oslo_debug_helper https://review.openstack.org/443497 | 08:48 |
| *** swap-nilangekar has joined #openstack-manila | 08:49 | |
| *** huyang has joined #openstack-manila | 09:09 | |
| *** a-pugachev has joined #openstack-manila | 10:01 | |
| *** tuanluong has quit IRC | 10:18 | |
| *** lpetrut has joined #openstack-manila | 10:34 | |
| *** ganso has joined #openstack-manila | 10:40 | |
| openstackgerrit | Jan Provaznik proposed openstack/manila master: User Messages https://review.openstack.org/313549 | 10:43 |
| openstackgerrit | Jan Provaznik proposed openstack/manila master: Provide filter name in user messages https://review.openstack.org/443102 | 10:43 |
| openstackgerrit | Jan Provaznik proposed openstack/manila master: Extend usage of user messages https://review.openstack.org/443101 | 10:43 |
| *** furlongm has quit IRC | 11:13 | |
| *** JoseMello has joined #openstack-manila | 11:21 | |
| *** swap-nilangekar has quit IRC | 11:25 | |
| *** markstur has joined #openstack-manila | 11:26 | |
| *** markstur has quit IRC | 11:31 | |
| *** alyson_ has joined #openstack-manila | 11:35 | |
| *** kaisers_ has quit IRC | 11:57 | |
| openstackgerrit | Merged openstack/manila master: Send resize parameters in rpc as list in the Quobyte driver https://review.openstack.org/437501 | 11:58 |
| *** kaisers_ has joined #openstack-manila | 11:59 | |
| *** kaisers_ has quit IRC | 11:59 | |
| *** kaisers_ has joined #openstack-manila | 12:00 | |
| *** tuanluong has joined #openstack-manila | 12:06 | |
| *** mkoderer_ has joined #openstack-manila | 12:06 | |
| *** databus23_ has joined #openstack-manila | 12:06 | |
| *** seife_ has joined #openstack-manila | 12:06 | |
| *** david_1 has joined #openstack-manila | 12:06 | |
| *** seife_ has quit IRC | 12:08 | |
| *** databus23_ has quit IRC | 12:08 | |
| *** mkoderer_ has quit IRC | 12:08 | |
| *** david_1 has quit IRC | 12:08 | |
| openstackgerrit | Merged openstack/manila master: Remove unused "share_id" parameter https://review.openstack.org/441678 | 12:09 |
| openstackgerrit | Merged openstack/manila master: Update share server provisioning for share groups https://review.openstack.org/443164 | 12:09 |
| *** tuanluong has quit IRC | 12:21 | |
| *** furlongm has joined #openstack-manila | 12:31 | |
| *** kaisers_ has quit IRC | 12:48 | |
| openstackgerrit | Tom Barron proposed openstack/manila master: CephFS driver: change CG variables to SG variables https://review.openstack.org/443325 | 12:51 |
| *** markstur has joined #openstack-manila | 13:16 | |
| *** markstur has quit IRC | 13:20 | |
| *** markstur has joined #openstack-manila | 13:29 | |
| *** kaisers_ has joined #openstack-manila | 13:29 | |
| *** chlong_ has joined #openstack-manila | 13:33 | |
| *** markstur has quit IRC | 13:33 | |
| *** kaisers_ has quit IRC | 13:34 | |
| *** dustins has joined #openstack-manila | 13:51 | |
| *** eharney has joined #openstack-manila | 14:03 | |
| *** gouthamr has joined #openstack-manila | 14:08 | |
| *** dustins has quit IRC | 14:10 | |
| openstackgerrit | Tom Barron proposed openstack/manila master: Rename wrapped methods in share manager https://review.openstack.org/443656 | 14:19 |
| tbarron | vponomaryov: that may be more agreeable to you ^^^ | 14:19 |
| vponomaryov | tbarron: yeah, this kind of renaming is more obvious | 14:22 |
| tbarron | vponomaryov: thanks, the minds meet :D | 14:23 |
| *** cknight has joined #openstack-manila | 14:35 | |
| openstackgerrit | Merged openstack/manila master: CephFS driver: change CG variables to SG variables https://review.openstack.org/443325 | 14:42 |
| vponomaryov | tbarron: about https://review.openstack.org/#/c/438892/ -> the change is order in which DB records are updated. Approach for real resources provisioning/processing has not been changed at all. | 14:44 |
| tbarron | ganso: gouthamr do you agree with that premise? ^^^^ | 14:45 |
| tbarron | ganso: gouthamr if so, then no reno is needed, right? | 14:45 |
| *** kaisers_ has joined #openstack-manila | 14:45 | |
| gouthamr | tbarron: i think any script can uncover that race condition... | 14:46 |
| vponomaryov | tbarron: above statement is fact, not opinion. opinion is whether it satisfies reno rules or not | 14:46 |
| tbarron | vponomaryov: gouthamr is disagreeing with your "fact". | 14:46 |
| ganso | tbarron: seems to me like a race condition that can lead to user observable error | 14:46 |
| ganso | tbarron: so in that case, it deserves a reno IMO | 14:46 |
| tbarron | the disagreement is whether it is a test-only fix | 14:47 |
| vponomaryov | tbarron: what do YOU think? ) | 14:47 |
| tbarron | ganso and gouthamr are saying the test reveals a behavior that users can experience. | 14:47 |
| tbarron | vponomaryov: do you disagree with that? | 14:47 |
| vponomaryov | tbarron: real resource processing was not changed | 14:47 |
| vponomaryov | tbarron: so, I am saying that users are not affected | 14:48 |
| tbarron | *I* don't have a belief without an understanding (at least I try not to). | 14:48 |
| * tbarron has fewer and fewer beliefs as time goes by | 14:48 | |
| gouthamr | haha | 14:49 |
| vponomaryov | tbarron: make pressure with your life experience )) | 14:50 |
| tbarron | vponomaryov: old man tries to make the younguns feel bad ... | 14:50 |
| vponomaryov | tbarron: everyone can be young in soul )) | 14:51 |
| *** nkrinner is now known as nkrinner_afk | 14:54 | |
| tbarron | vponomaryov: so if the order of DB updates is changed, what ensures that there is no race w.r.t. "real" resource provisioning? | 14:55 |
| tbarron | ^^^ this is an honest question, I don't know. | 14:55 |
| *** porrua has joined #openstack-manila | 14:55 | |
| vponomaryov | tbarron: because these DB updates is the least thing is done there, all resource processing is done before it | 14:56 |
| tbarron | last thing | 14:56 |
| vponomaryov | tbarron: yes, "last" )) | 14:57 |
| vponomaryov | oh those foreigners... | 14:57 |
| vponomaryov | ^_^ | 14:57 |
| tbarron | they make typos too | 14:57 |
| vponomaryov | tbarron: like "teh" ? )) | 14:58 |
| tbarron | Yup! | 14:58 |
| bswartz | it's only a matter of time before "teh" becomes a valid spelling | 14:58 |
| *** Yogi1 has joined #openstack-manila | 14:59 | |
| tbarron | I'm kinda meh about the | 14:59 |
| tbarron | the | 14:59 |
| *** eharney has quit IRC | 14:59 | |
| tbarron | the | 14:59 |
| * bswartz notices tbarron has autocorrect | 14:59 | |
| tbarron | wow, is there a bot? | 14:59 |
| bswartz | tbarron: it's your client | 14:59 |
| tbarron | learned something today, I can go home now | 15:00 |
| vponomaryov | tbarron: only after meeting! ) | 15:00 |
| bswartz | tbarron: don't you work from home? | 15:00 |
| tbarron | bswartz: sometimes | 15:00 |
| bswartz | --> #openstack-meeting-alt <-- | 15:00 |
| tbarron | well, for all x, if x is a time I work, it's at home | 15:00 |
| *** markstur has joined #openstack-manila | 15:01 | |
| *** dustins has joined #openstack-manila | 15:02 | |
| *** dustins_ has joined #openstack-manila | 15:07 | |
| *** dustins has quit IRC | 15:09 | |
| *** eharney has joined #openstack-manila | 15:14 | |
| *** chlong_ has quit IRC | 15:17 | |
| *** dustins_ is now known as dustins | 15:38 | |
| *** kaisers_ has quit IRC | 15:47 | |
| *** jprovazn is now known as jprovazn_bbl | 15:53 | |
| *** cknight has left #openstack-manila | 15:53 | |
| vponomaryov | tbarron: I guess you missed the window here -> https://review.openstack.org/#/c/442633/ =) | 15:54 |
| vponomaryov | tbarron: it should have been this -> https://review.openstack.org/#/c/438892/ | 15:54 |
| tbarron | vponomaryov: thanks, will fix | 15:55 |
| vponomaryov | ganso: what are you upcoming plans in scope of manila? | 15:55 |
| vponomaryov | s/you/your/ | 15:55 |
| vponomaryov | ganso: and yes, thank you for all your work, it was pleasure to work with you, even having lots of disagreements ))) | 15:56 |
| ganso | vponomaryov: reviews and meetings... whenever I can | 15:56 |
| ganso | vponomaryov: hold your horses I am not 100% away yet :P | 15:57 |
| *** esker has joined #openstack-manila | 15:57 | |
| vponomaryov | ganso: you just turned on other way on crossroad, just haven't ride away far enough yet )) | 15:57 |
| ganso | vponomaryov: yes, save your goodbye for another day :P | 15:58 |
| ganso | vponomaryov: and let's hope that day never comes | 15:58 |
| ganso | vponomaryov: :) | 15:58 |
| vponomaryov | ganso: everything ends sometime | 15:59 |
| vponomaryov | ganso: so, we should appreciate any moment | 15:59 |
| vponomaryov | any happy moment we have/had | 15:59 |
| ganso | vponomaryov: yes, I agree 100% :) | 16:00 |
| *** rraja has quit IRC | 16:00 | |
| gouthamr | vponomaryov just wanted to say it was a pleasure having disagreements :P | 16:10 |
| vponomaryov | gouthamr: now I know why you have your special opinion always )) | 16:29 |
| vponomaryov | gouthamr: just too comunicative person that wants more attention )) | 16:29 |
| gouthamr | lol vponomaryov | 16:43 |
| *** sandanar has quit IRC | 16:47 | |
| *** kaisers_ has joined #openstack-manila | 16:50 | |
| *** kaisers_ has quit IRC | 16:55 | |
| *** lpetrut has quit IRC | 17:01 | |
| *** Yogi1 has quit IRC | 17:05 | |
| openstackgerrit | Merged openstack/manila master: Rename wrapped methods in share manager https://review.openstack.org/443656 | 17:10 |
| openstackgerrit | Merged openstack/manila master: [api-ref]: Add missing share statuses https://review.openstack.org/437149 | 17:10 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila master: Local copy of scenario test base class https://review.openstack.org/442719 | 17:19 |
| *** lpetrut has joined #openstack-manila | 17:21 | |
| *** dsariel has quit IRC | 17:24 | |
| *** sapcc-bot2 has joined #openstack-manila | 17:25 | |
| *** lpetrut has quit IRC | 17:25 | |
| *** sapcc-bot has quit IRC | 17:25 | |
| *** mtanino has joined #openstack-manila | 17:32 | |
| *** kaisers_ has joined #openstack-manila | 17:56 | |
| *** lpetrut has joined #openstack-manila | 17:56 | |
| openstackgerrit | Dustin Schoenbrun proposed openstack/manila master: Local copy of scenario test base class https://review.openstack.org/442719 | 17:58 |
| *** kaisers_ has quit IRC | 17:59 | |
| *** kaisers_ has joined #openstack-manila | 17:59 | |
| dustins | vponomaryov: sorry, I didn't see that you'd already done the changes needed to that patch | 18:00 |
| * dustins just returned from lunch and was finishing what I started | 18:00 | |
| vkmc | vponomaryov, not entirely sure if this is what you had in mind https://review.openstack.org/#/c/443737/4 | 18:02 |
| *** kaisers_ has quit IRC | 18:03 | |
| vponomaryov | vkmc: also, you need to update https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/projects.yaml | 18:06 |
| vkmc | vponomaryov, yes | 18:06 |
| vponomaryov | vkmc: now you have lots of defined and not used jobs, and use undefined ones | 18:06 |
| vponomaryov | vkmc: strange that you have LVM specific env var there | 18:07 |
| * dustins works to undo my patch | 18:07 | |
| vponomaryov | vkmc: I would define some common | 18:07 |
| vponomaryov | dutins: also, there can be unused parts of that module | 18:08 |
| vponomaryov | dustins: I just didn't investigate it | 18:08 |
| dustins | vponomaryov: It's possible, yeah | 18:08 |
| dustins | I think for now we can add the file as is, and perhaps remove the unneeded parts later | 18:08 |
| vponomaryov | dustins: also, looks like tempest is not py3 completely compatible | 18:09 |
| dustins | Or, since this seems to be a temporary thing, just not worry about it and swap it out for the stabilized manager when it becomes available | 18:09 |
| dustins | vponomaryov: Oh? | 18:09 |
| vponomaryov | dustins: "str" stuff and other | 18:09 |
| dustins | Yeah, I saw that flake8 got hung up on that | 18:10 |
| dustins | Which is why we had to use six.text_type() there | 18:10 |
| vkmc | vponomaryov, done, thanks for the quick review | 18:11 |
| dustins | vponomaryov: Is there an easier way to "unsubmit" my patch other than cherry-picking your commit and submitting it as patchset 5? | 18:11 |
| vponomaryov | dustins: it is the easiest | 18:12 |
| dustins | Okay, I'll do that, then, sorry about that | 18:12 |
| vponomaryov | vkmc: what about env var? | 18:13 |
| vponomaryov | vkmc: now it is LVM-specific | 18:13 |
| vponomaryov | vkmc: but I do not see a real reason to have it LVM-specific | 18:13 |
| vkmc | vponomaryov, just if the driver is lvm... it was lvm specific already | 18:13 |
| openstackgerrit | Dustin Schoenbrun proposed openstack/manila master: Local copy of scenario test base class https://review.openstack.org/442719 | 18:13 |
| vponomaryov | vkmc: I mean CI job config could be more flexible, for example, this key could be provided to Pre_test_hook | 18:15 |
| vkmc | vponomaryov, this key... as... specify if you want to use IPv6 then we set those? | 18:16 |
| *** alyson_ has quit IRC | 18:17 | |
| vponomaryov | vkmc: I mean manila-specific logic will be in manila repo | 18:17 |
| vponomaryov | vkmc: and define all such data as args to Pre_test_hook | 18:17 |
| vkmc | vponomaryov, yeah, that's another way to do it :) | 18:17 |
| vponomaryov | vkmc: so, you play and fix everything using one single commit in manila | 18:17 |
| vponomaryov | s/you/you could/ | 18:18 |
| vkmc | vponomaryov, we would need to add a fourth argument | 18:19 |
| vkmc | right? | 18:19 |
| vponomaryov | yes | 18:19 |
| vponomaryov | forth fifth, whatever - one more | 18:19 |
| vkmc | in that case we would need to change calls within jenkins as well | 18:20 |
| vponomaryov | within jenkins? | 18:20 |
| vkmc | yeah, this calls over here https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/manila.yaml#L83-L86 | 18:21 |
| vkmc | s/this/these/g | 18:21 |
| vponomaryov | yes | 18:22 |
| vponomaryov | only when you start using ipv6 there | 18:23 |
| vponomaryov | assume ipv4 is default value | 18:23 |
| vkmc | all right | 18:23 |
| vkmc | let me see how can I change that | 18:23 |
| vkmc | but wait... I still need a way to define when we are passing that param or not | 18:24 |
| vkmc | so all the changes that are now in for review are still necessary | 18:24 |
| *** a-pugachev has quit IRC | 18:35 | |
| vkmc | 704392 | 18:38 |
| vkmc | ops | 18:38 |
| *** dustins has quit IRC | 18:48 | |
| *** dustins has joined #openstack-manila | 18:49 | |
| *** david-lyle has quit IRC | 19:14 | |
| *** kaisers_ has joined #openstack-manila | 20:02 | |
| *** mtanino has quit IRC | 20:02 | |
| *** dsariel has joined #openstack-manila | 20:04 | |
| *** esker has quit IRC | 20:14 | |
| *** cknight has joined #openstack-manila | 20:29 | |
| openstackgerrit | Merged openstack/manila master: [Tempest] Fix concurrency in test with listing share servers https://review.openstack.org/442633 | 20:29 |
| *** xyang1 has joined #openstack-manila | 20:43 | |
| *** chlong_ has joined #openstack-manila | 20:47 | |
| *** jprovazn_bbl has quit IRC | 20:53 | |
| *** JoseMello has quit IRC | 20:55 | |
| *** eharney has quit IRC | 21:05 | |
| *** porrua has quit IRC | 21:13 | |
| *** kaisers_ has quit IRC | 21:46 | |
| vkmc | bswartz, https://review.openstack.org/#/c/443737 could you give me more context on your review? | 21:55 |
| *** cknight has quit IRC | 21:55 | |
| vkmc | are you expecting to run tests with a dual stack config? | 21:55 |
| *** lpetrut has quit IRC | 22:39 | |
| *** catintheroof has joined #openstack-manila | 22:39 | |
| *** gouthamr has quit IRC | 22:48 | |
| *** gouthamr has joined #openstack-manila | 23:10 | |
| *** chlong_ has quit IRC | 23:21 | |
| *** ganso has quit IRC | 23:25 | |
| *** xyang1 has quit IRC | 23:33 | |
| *** kaisers_ has joined #openstack-manila | 23:46 | |
| *** kaisers_ has quit IRC | 23:51 | |
| *** dustins has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!