Evergreen ILS Website

IRC log for #evergreen, 2019-10-15

| Channels | #evergreen index | Today | | Search | Google Search | Plain-Text | summary | Join Webchat

All times shown according to the server's local time.

Time Nick Message
00:08 sandbergja joined #evergreen
07:02 agoben joined #evergreen
07:08 rjackson_isl joined #evergreen
08:01 Dyrcona joined #evergreen
08:05 bos20k joined #evergreen
08:06 rfrasur joined #evergreen
08:47 Dyrcona Um... I don't think the following is supposed to happen:
08:50 Dyrcona Using Chromium 77 with a normal tab, I get a blank window when I go to Local Administration with a recent checkout of master.  If I reload, I get "Welcome to Webby."  The link to go to the splash page seems to do nothing. I tried emptying the cache and hard reload.
08:51 Dyrcona Using Firefox 69.0.3 and a Private Window to login to the same evergreen instance, I get prompted to register the workstation again when I go to Local Administration.  After registering the workstation, clicking Use Now, and logging in again, it seems to work.
08:56 Dyrcona That's on a test system set up at CW MARS last week with Pg 10 for the database, Ubuntu 16.04 for the brick vms, and haproxy in front of the bricks.
08:57 Dyrcona I have some VMs that I built on my laptop Sunday that I can also test with, and I will do so in a bit.
09:00 Dyrcona Trying again with Chromium 77, an Incognito Window, and the default admin, rather than a Global Administrator that I added, seems to just work.
09:00 yboston joined #evergreen
09:01 Dyrcona Yeah, I only get the blank screen with my added global administrator account....
09:02 Dyrcona Oh, nifty! main.de76656881bc89a98b3d.js:1 ERROR Error: Uncaught (in promise): User does not have staff permissions
09:02 Dyrcona The Global Administrator profile doesn't give you that?
09:03 Dyrcona Guess it's a Concerto data bug?
09:05 Dyrcona Well, maybe not. Global Administrator has perm -1 at depth 0, i.e. "Everything at Everywhere." It also has 1 other permission added, so this looks like a case of a perm check not looking for perm -1.
09:08 Dyrcona Looks like the bug may be here: open-ils.actor.user.has_work_perm_at.batch
09:19 csharp are you behind nginx?  could it be the IP address forwarding issue we've seen before?
09:26 Dyrcona csharp: No, it's haproxy on the load balancer, and I have the IP forward headers configured properly. It's only failing with a user who's not a super user and has the Global Administrator profile. I now suspect it's a problem with the depth parameter on the permission look up request, but I'm still looking.
09:28 csharp gotcha
09:31 miker Dyrcona: .has_[work_]perm_at is the bane of my existence...
09:32 agoben We've got a library that's having trouble with offline mode not caching the workstation info.  If they log in and *then* go offline, it works fine, but if they haven't logged in on a given day, it's wiped out.  Any suggestions?
09:32 miker we should add a short circuit test for -1 in those
09:34 agoben Note: the workstation registration isn't also disappearing, just the offline info.
09:36 Dyrcona miker: It checks for -1 in joins, but it doesn't short circuit the way that super user does.
09:37 Dyrcona agoben: Do they do something locally to clear the cache? Windows policies, maybe?
09:39 Dyrcona miker: I'm a bit stumped as to why this isn't work. Looks like it should, but I'll have to find that place where the permissions check is made.
09:40 agoben Dyrcona: Claiming no.  And I would expect a cache-clear to take the whole registration and not just the offline info.
09:41 JBoyer Dyrcona, does your GA user have a working location set? I wouldn't expect that to matter with a -1 perm, but (not having looked at the queries involved) if there's a query that expects *something* to be returned for working location and there's nothing, that might be an issue.
09:42 Dyrcona JBoyer: No, but I'll add one and see what happens.
09:42 csharp agoben: cache clear doesn't affect workstation, just cookies - possibly a misunderstanding between cache/cookies on staff's part?
09:44 agoben Ok, so offline isn't part of site data?
09:44 remingtron joined #evergreen
09:45 csharp offline is stored in IndexedDB (which is visible if you browse storage in the dev tools)
09:45 agoben This has coincided with an update from their ISP, so that *could* be part of it, but no one else on that ISP (and there are a lot) is reporting the same issue.
09:46 csharp any caching proxies in the mix?  (just throwing things out there)
09:47 agoben I'll have to ask about their proxy setups
09:47 Christineb joined #evergreen
09:47 agoben Thanks for the help!
09:48 Dyrcona JBoyer++ That fixed it!
09:50 miker Dyrcona: ah, nm, I was thinking of the angular js wrapper
09:50 miker JBoyer++
09:50 Dyrcona I always forget about setting working locations. I almost never do this sort of thing.
09:51 JBoyer That's so easy to miss probably because you can't set it when registering the account. One common thing I've seen is running a cron job that just assigns your home lib as a working location if your profile is a child of Staff and none are set.
09:51 JBoyer There's probably a better way to deal with those.
09:52 Dyrcona Well, it was quick enough to insert for the two accounts that I made.
09:55 mdriscoll joined #evergreen
10:04 Dyrcona We're also using Pg 10 on this test cluster, and so far, no problems.
10:09 sandbergja joined #evergreen
10:13 cmalm joined #evergreen
10:58 sandbergja joined #evergreen
11:03 pinesol News from qatests: Testing Success <http://testing.evergreen-ils.org/~live>
12:01 jihpringle joined #evergreen
12:08 sandbergja joined #evergreen
12:11 aabbee joined #evergreen
12:29 yboston joined #evergreen
12:49 khuckins joined #evergreen
13:16 jwoodard joined #evergreen
13:22 jwoodard left #evergreen
13:24 jwoodard joined #evergreen
13:32 jeffdavis Here's a strange bug. On 3.3, after adding a title to your basket, a subsequent search is giving us an internal server error. Removing the cartcache cookie makes the error go away. We're only seeing this when load-balancing between multiple servers.
13:33 jeffdavis Call to [open-ils.storage.biblio.multic​lass.staged.search_fts.atomic] failed for session [...], thread trace [1]: Can't call method "opac_visible" on an undefined value at /usr/local/share/perl/5.22.1/OpenILS/Appl​ication/Storage/Driver/Pg/QueryParser.pm line 1177.
13:35 jeffdavis hm, wonder if it is a locale thing...
13:49 jeffdavis looks like default_locale in opensrf.xml was set to en-CA in our multi-server environment but en-US in single-server test environments; changing to en-US seems to be avoiding the issue so far; never had a problem with en-CA there until now though
14:57 nfBurton joined #evergreen
15:07 yboston joined #evergreen
15:31 jeffdavis Definitely seeing much higher open-ils.actor drone counts on 3.3. I increased max_children from 50 to 75 but we're still hitting that new limit (currently hovering just below it on 2 out of 3 production servers).
15:34 Dyrcona jeffdavis: It's the web staff client.
15:36 Dyrcona jeffdavis: Ours is 60 per drone, so 120 per brick.
15:37 dbs jeffdavis / Dyrcona: do you know how many simultaneous users you have, to help figure out a rough drone-per-user rule of thumb?
15:38 Dyrcona dbs: Nope. I've not bothered trying to figure it out.
15:40 jeffdavis Hmm, we could try counting authtokens in memcached I guess?
15:41 csharp jeffdavis: which release did you upgrade from?
15:41 jeffdavis I wonder if we can reduce the number of open-ils.actor calls though, a la bug 1745499
15:41 pinesol Launchpad bug 1745499 in Evergreen 3.0 "Web client: Barcode file upload inflates pcrud drone count" [Medium,Fix released] https://launchpad.net/bugs/1745499
15:41 jeffdavis csharp: 3.1.7 -> 3.3.4
15:41 csharp yeah, I suspect lots of service calls could be batched somehow
15:42 jeffdavis we just went live this morning (well, over the weekend but this is most libraries' first day using it), so haven't had a chance to track down specific causes yet
15:42 csharp jeffdavis: I'm pretty sure we saw significant increases going from 3.0 -> 3.2 and made similar adjustments
15:42 csharp jeffdavis: congratulations!
15:43 jeffdavis Thanks! It's early yet but it's been quite smooth so far.
15:43 csharp yeah - our headaches were mostly invisible to end users - stuff like high drone counts/system load
15:45 Dyrcona We still run out of some drones. I need to look into that again and up a few counts.
15:47 Dyrcona Like csharp said, we really noticed it going from 3.0 to 3.2 and staff using the web staff client more.
16:00 jihpringle joined #evergreen
16:00 khuckins joined #evergreen
16:17 miker re actor being more heavily used in web client, it's all because each "page" is stateless.  discarding cstore calls, open-ils.actor.settings.retrieve.atomic is consistently the #2 most called method (~6% of all non-cstore calls), behind open-ils.auth.session.retrieve at a full 50%+ of all calls. for those still using xul, it can't even be found in the top 20 (less than 1% of calls)
16:18 miker in the xul client, you may recall, there was a login-time dance where we fetched, well, TONS of stuff. user and library settings were two sets of data cached for the whole xul session. not so in webclient, we get them on demand.
16:27 jeffdavis We did see an increase in actor calls when we moved to web client-only with 3.1, but this upgrade to 3.3 is another big step up in actor usage.
16:29 jeffdavis I wonder if there is a difference between AngJS and Ang too.
16:31 jeffdavis Should we consider caching settings?
16:31 jwoodard joined #evergreen
16:39 miker jeffdavis: we did, at the session level (effectively) in xul, so ... sure?
16:40 jihpringle joined #evergreen
16:41 miker of course, one of the remaining xul "bugs" is "make the login process faster" ... only way to do that would be to disable some of those caching calls ;) (can't just delay them in the background, because they'll be needed for the first thing you want to do, very likely)
16:44 miker one thing we could do is make heavier use of memcache.  but that means figuring out proper cache segmentation, and cache invalidation logic, which is ... tricky. or, "push all individual settings into k/v pairs in memcache, and teach $something_in_the_middle_of​_everything_maybe_a_trigger to invalidate on value change for a key"
16:51 miker we did offer something like that, for a time, for org tree and perm lookups. there are vestiges of that in our pgmemcache-*.sql files. csharp may recall those -- I wrote that for PINES many thousands of years ago.  I am not personally excited about reviving that :)
17:11 * jeffdavis contemplates adding a "cacheable" column on config.org_unit_setting_type, then generating a JSON blob for cacheable setting values with autogen
17:19 jihpringle joined #evergreen
18:08 sandbergja_ joined #evergreen
19:53 b_bonner left #evergreen
20:13 JBoyer_ joined #evergreen
23:03 pinesol News from qatests: Testing Success <http://testing.evergreen-ils.org/~live>
23:52 sandbergja joined #evergreen

| Channels | #evergreen index | Today | | Search | Google Search | Plain-Text | summary | Join Webchat