| *** erlon has quit IRC | 00:26 | |
| *** e0ne has joined #openstack-manila | 04:11 | |
| *** e0ne has quit IRC | 04:24 | |
| *** pcaruana has joined #openstack-manila | 06:38 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/manila-ui master: Imported Translations from Zanata https://review.openstack.org/590168 | 07:38 |
|---|---|---|
| *** assassin has quit IRC | 07:48 | |
| openstackgerrit | Maurice Schreiber proposed openstack/manila master: NetApp ONTAP: change cifs server valid dns hostname https://review.openstack.org/508439 | 08:12 |
| openstackgerrit | Merged openstack/manila-ui master: Imported Translations from Zanata https://review.openstack.org/590168 | 09:28 |
| *** erlon has joined #openstack-manila | 10:49 | |
| *** ganso has joined #openstack-manila | 11:01 | |
| *** luizbag has joined #openstack-manila | 11:22 | |
| *** eharney has quit IRC | 12:17 | |
| *** tpsilva has joined #openstack-manila | 12:44 | |
| *** vgreen has joined #openstack-manila | 12:47 | |
| *** ianychoi has quit IRC | 12:49 | |
| *** eharney has joined #openstack-manila | 12:58 | |
| *** dustins has joined #openstack-manila | 13:17 | |
| *** eharney has quit IRC | 13:45 | |
| *** eharney has joined #openstack-manila | 13:49 | |
| gouthamr | tpsilva: was looking at https://review.openstack.org/508439 - is the ntap-ss job broken? | 14:15 |
| tpsilva | gouthamr: it wasn't yesterday | 14:15 |
| tpsilva | let me check | 14:15 |
| gouthamr | tpsilva: thanks! http://logs.openstack.netapp.com/logs/39/508439/4/upstream-check/manila-cDOT-ss/ded53a1/logs/screen-m-shr.txt.gz#_Aug_09_10_05_09_869894 seems to suggest vlans are duplicated, missing cleanup perhaps | 14:16 |
| tpsilva | gouthamr: :( we have been facing issues with cleanup on SS jobs for a while now, but the storage used by this job looks fine | 14:23 |
| tpsilva | I'll re-trigger it and debug this error | 14:23 |
| gouthamr | tpsilva: np, thanks for the attention | 14:24 |
| tpsilva | thank you | 14:24 |
| *** markstur has joined #openstack-manila | 14:49 | |
| *** markstur has quit IRC | 14:50 | |
| *** markstur has joined #openstack-manila | 14:50 | |
| tbarron | weekly meeting will start in one minute | 14:59 |
| tbarron | no courtesy ping cause I don't want to get BANNED | 14:59 |
| markstur | hi | 15:00 |
| vgreen | !ban | 15:00 |
| openstack | vgreen: Error: "ban" is not a valid command. | 15:00 |
| gouthamr | vgreen it is /ban tbarron | 15:00 |
| vgreen | works on team fortress server | 15:00 |
| vgreen | don't want to get in hot water gouthamr, you are trying to stir the pot | 15:01 |
| markstur | gouthamr you just kicked Tom out it works anywhere in the line | 15:01 |
| markstur | is this a coup? | 15:01 |
| gouthamr | shh markstur, silent one | 15:02 |
| jun2222 | hi | 15:03 |
| markstur | :) hopefully I was just kidding about that ban working (trying to scare gouthamr) | 15:03 |
| gouthamr | believed that for a second, because i've hidden general event messages | 15:04 |
| openstackgerrit | OpenStack Release Bot proposed openstack/manila stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590391 | 15:27 |
| openstackgerrit | OpenStack Release Bot proposed openstack/manila stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590392 | 15:27 |
| openstackgerrit | OpenStack Release Bot proposed openstack/manila master: Update reno for stable/rocky https://review.openstack.org/590393 | 15:27 |
| gouthamr | fireworks ^ | 15:27 |
| openstackgerrit | OpenStack Release Bot proposed openstack/manila-ui stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590394 | 15:27 |
| openstackgerrit | OpenStack Release Bot proposed openstack/manila-ui stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590395 | 15:28 |
| openstackgerrit | OpenStack Release Bot proposed openstack/manila-ui master: Update reno for stable/rocky https://review.openstack.org/590396 | 15:28 |
| dustins | gouthamr: So it was a bug in the API that manifested through the UI? | 16:01 |
| gouthamr | dustins: yes | 16:02 |
| gouthamr | not through the UI, we just wanted to be sure to remove it from the UI as well as the client | 16:02 |
| bswartz | #link https://www.archlinux.org/packages/community/x86_64/pycharm-community-edition/ | 16:02 |
| dustins | gouthamr: Perhaps "was discovered through the UI" is the better choice of words | 16:02 |
| bswartz | ^ open source pycharm | 16:03 |
| vgreen | so we can just check a non-tenant and see if the info is there or not? | 16:03 |
| gouthamr | dustins: perhaps.. | 16:03 |
| * dustins isn't as good of a wordsmith when he's hungry | 16:04 | |
| gouthamr | vgreen: yes, we can remove that field in the project view | 16:04 |
| dustins | So the API won't present it, but the field is present in the UI | 16:04 |
| gouthamr | bswartz: +1 nice, but the professional edition has deployment tools | 16:04 |
| dustins | So we just have to remove that now empty field from the UI and we're good? | 16:05 |
| gouthamr | dustins: yep, imo | 16:05 |
| dustins | gouthamr: And we can get the pro edition by being open source contributors | 16:05 |
| dustins | At least you could a couple of years ago | 16:05 |
| gouthamr | dustins: yep, true! | 16:05 |
| gouthamr | still :) coolsvap's got a google form to request one still | 16:06 |
| gouthamr | https://docs.google.com/forms/d/e/1FAIpQLSe5JMbtZEKB95AMVnyOBh4-7Y55hDgQChjg5Ed3auO74Tt2fQ/viewform | 16:06 |
| bswartz | The community edition has always been more than enough for my needs | 16:06 |
| bswartz | And jetbrains has enough of my money, they don't need more | 16:07 |
| gouthamr | bswartz: can you help with a zfsonlinux issue? | 16:07 |
| dustins | Their tools are worth the cash, IMHO | 16:07 |
| bswartz | gouthamr: possibly | 16:07 |
| bswartz | I'm about to disappear to find lunch though | 16:08 |
| gouthamr | bswartz: okay, np, i'll leave messages here for when you're back | 16:08 |
| bswartz | Well go ahead and ask the question and I'll see if it's a quick answer or not | 16:08 |
| bswartz | PM or in channel? | 16:08 |
| gouthamr | bswartz: nirg22 was trying to add a scenario test around manage/unmanage and the zfsonlinux driver fails his test | 16:08 |
| gouthamr | bswartz: https://bugs.launchpad.net/manila/+bug/1785180 | 16:09 |
| openstack | Launchpad bug 1785180 in Manila "ZFSonLinux driver fails to manage share in scenario test" [Medium,In progress] - Assigned to Goutham Pacha Ravi (gouthamr) | 16:09 |
| gouthamr | bswartz: this is strange because we run manage/unmanage API tests in the gate, except in Nir's case, he's using the share (adding access rules, mounting it into vm, writing stuff, unmounting and unmanaging) | 16:09 |
| bswartz | There are timing issues with ZFS | 16:10 |
| bswartz | Adding sleeps in the manage/unmanage code paths might be enough to solve it | 16:10 |
| gouthamr | bswartz: oh | 16:10 |
| bswartz | When you overwhelm ZFS with a lot of fast operations, it behaves strangely | 16:10 |
| gouthamr | bswartz: well, i tried a retry loop with a sanity check to see if the mountpoint is really there - https://review.openstack.org/#/c/588667/ | 16:11 |
| bswartz | And it doesn't offer a good asynchronous interface | 16:11 |
| gouthamr | bswartz: ack, i noticed that in the rest of the driver.. | 16:11 |
| bswartz | Does the retry loop sleep at all? | 16:11 |
| gouthamr | bswartz: it does, exponential backoff | 16:12 |
| gouthamr | but in the logs, i didn't notice it had to try more than once.. | 16:12 |
| bswartz | That should be enough to sort it out | 16:12 |
| gouthamr | bswartz: coolio, now go enjoy Richard's kicking chicken | 16:12 |
| gouthamr | :P | 16:12 |
| bswartz | But some of those operation aren't idempotent either, so retrying them could get you into different trouble | 16:12 |
| bswartz | Today lunch is Catalan Chicken Saute, Rice Pilaf with Almonds and Saffron | 16:13 |
| bswartz | And I still haven't figured out what that means | 16:13 |
| gouthamr | too much going on in there | 16:13 |
| gouthamr | Spain, Persia, South Asia | 16:13 |
| bswartz | I guess I'll go take a look and see if its appetizing | 16:14 |
| gouthamr | bswartz: +1, thanks.. please take a look at https://review.openstack.org/#/c/588667/ when you get a chance | 16:14 |
| bswartz | Don't tell the Catalan people they're part of Spain, they'd rather be their own country | 16:14 |
| gouthamr | that's true.. | 16:15 |
| *** dustins has quit IRC | 16:15 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila master: [ZFSOnLinux] Retry unmounting old datasets during manage https://review.openstack.org/588667 | 16:31 |
| openstackgerrit | Merged openstack/manila master: NetApp cDOT driver switch volume efficiency https://review.openstack.org/539996 | 17:28 |
| *** vgreen has quit IRC | 17:35 | |
| *** gouthamr is now known as gouthamr_away | 17:37 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila-ui master: Use 'trimmed' for blocktrans in multi-line strings https://review.openstack.org/583444 | 18:47 |
| *** pcaruana has quit IRC | 19:33 | |
| *** eharney has quit IRC | 19:57 | |
| *** nirg22 has quit IRC | 20:10 | |
| openstackgerrit | Merged openstack/manila-ui master: Use 'trimmed' for blocktrans in multi-line strings https://review.openstack.org/583444 | 20:49 |
| *** gouthamr_away is now known as gouthamr | 20:53 | |
| *** gouthamr is now known as gouthamr|brb | 21:20 | |
| *** gouthamr|brb is now known as gouthamr | 21:20 | |
| *** gouthamr is now known as gouthamr|brb | 21:21 | |
| *** luizbag has quit IRC | 21:33 | |
| *** gouthamr|brb is now known as gouthamr | 21:40 | |
| *** tpsilva has quit IRC | 23:10 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!