06:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
08:11 |
|
tlittle joined #evergreen |
08:15 |
|
mantis joined #evergreen |
08:32 |
|
Dyrcona joined #evergreen |
10:16 |
|
jvwoolf left #evergreen |
10:40 |
|
alynn26 joined #evergreen |
10:40 |
|
jvwoolf joined #evergreen |
10:47 |
terranm |
JBoyer - for https://bugs.launchpad.net/evergreen/+bug/1813191 I can confirm that the reports look good, but I cannot test the error that Galen found with the odd call number in comment 8 since the holdings view patch on the test server is broken. So, not sure if I should sign off or not. |
10:47 |
pinesol |
Launchpad bug 1813191 in Evergreen "Move calculated dewey ranges/blocks to their own reporting view" [Wishlist,New] - Assigned to Terran McCanna (tmccanna) |
10:49 |
JBoyer |
terranm, Wasn't aware holdings view was busted. I'll take a quick look. |
10:51 |
terranm |
Yes, it's hanging upon save - there are testing comments on https://bugs.launchpad.net/evergreen/+bug/1888723 |
10:51 |
pinesol |
Launchpad bug 1888723 in Evergreen "Holdings and Item Attributes Editors (VolCopy) Angular Port" [Wishlist,Confirmed] |
10:52 |
JBoyer |
Ah, "Cannot read property 'applyPendingChanges' of undefined" Bummer. |
10:52 |
JBoyer |
That'll be gone shortly because that kind of wrecks a lot of things. |
13:44 |
Dyrcona |
On an unrelated note, the split command is pretty much useless for splitting a file of binary MARC records. At least, I can't figure out how to use \x1E as the separator character. |
13:45 |
Dyrcona |
csplit also looks like it is not very useful for this task. |
13:56 |
|
jvwoolf joined #evergreen |
14:30 |
terranm |
JBoyer - I'm trying to test this patch on festivus but I'm getting browser console errors and an "Editing users in this group is disallowed " message when I try to edit a patron (logged in as admin) - https://bugs.launchpad.net/evergreen/+bug/1937299 |
14:30 |
pinesol |
Launchpad bug 1937299 in Evergreen "Multiple issues with the Collections: Exempt user setting" [Medium,Confirmed] |
14:30 |
terranm |
(Not sure if it's caused by the patch or by something else) |
14:32 |
JBoyer |
It is extremely not that, no. I'll see what it could be. |
14:44 |
terranm |
I mean, JBoyer++ |
14:44 |
JBoyer |
\o/ |
14:46 |
|
Stompro joined #evergreen |
15:06 |
Stompro |
Is there a procedure for requesting that a bug be included on a test system? |
15:07 |
mmorgan |
terranm^^ |
15:09 |
mmorgan |
Stompro: I think sending terranm an email is a good approach. |
15:11 |
terranm |
Stompro: yes |
15:12 |
terranm |
Did you have a particular patch in mind, or just asking generally? |
15:13 |
Stompro |
terranm, I would like to test the new pull list, #1919465 |
15:13 |
Stompro |
terranm, and just to check, we are not having the newdev meeting, right? |
15:14 |
mmorgan |
bug 1919465 |
15:14 |
terranm |
The new dev meeting is next week |
15:14 |
pinesol |
Launchpad bug 1919465 in Evergreen "Port holds pull list to Angular / Wide Holds API" [Undecided,New] https://launchpad.net/bugs/1919465 |
15:15 |
Stompro |
oops, thought it was right now. Thanks. |
15:15 |
terranm |
:) |
15:16 |
terranm |
I'll get that loaded (or ask Tiffany to load it on hers) as soon as one of our test servers has everything currently on it signed off and is free to refresh with a fresh set of patches. |
15:17 |
Stompro |
terranm, thank you. |
15:17 |
terranm |
(Probably tomorrow) |
15:31 |
Stompro |
terranm, bug 1930308 loaded on "Terran's Test Server 1" requires the TPAC opac. |
15:31 |
pinesol |
Launchpad bug 1930308 in Evergreen "Circ modifier column appears twice in OPAC view" [Undecided,New] https://launchpad.net/bugs/1930308 - Assigned to Josh Stompro (u-launchpad-stompro-org) |
15:31 |
terranm |
Oops! |
15:32 |
Stompro |
terranm, I can test that if it gets loaded on a test server running tpac later on. |
15:32 |
mmorgan |
Too many catalogs! |
15:32 |
terranm |
Thanks, I'll do that next |
15:32 |
terranm |
That's a good band name, too |
15:44 |
terranm |
Stompro - okay, tpac is on terran-master now |
15:51 |
Stompro |
terranm, thanks, will test in just a sec. |
16:00 |
|
jvwoolf left #evergreen |
16:07 |
Stompro |
terranm, I'm done testing 1930308 if you need to turn bootstrap back on. |
16:08 |
terranm |
Thanks! I don't think anything else on there needs bootstrap right now |
16:13 |
jeff |
Dyrcona: I use yaz-marcdump for splitting marc records. |
16:14 |
Dyrcona |
jeff: I wasn't sure what options to use for that, though I suspected it would be possible. In the end, I wrote a little program in Perl. |
16:17 |
jeff |
not sure what happens if you feed it a file with more than 9999999 records, but now I have something else to break. |
16:17 |
terranm |
Stompro: I went ahead and added the signedoff tag to 1903357 even though you haven't submitted your sign off branch yet |
16:17 |
Dyrcona |
My program has a few of the options similar to split, at least the few that I thought made sense. |
16:19 |
Stompro |
terranm, thanks, I'll try and get my signoff branches done later tonight for the 3 bugs that I just tested. |
16:19 |
terranm |
Stompro++ |
17:16 |
|
mmorgan left #evergreen |
17:37 |
* gmcharlt |
claims 1271 |
17:41 |
pinesol |
[evergreen|Jason Etheridge] lp1894005 Stripe payment intents - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=e7418bd> |
17:41 |
pinesol |
[evergreen|Jason Etheridge] lp1894005 Stripe payment intents, release notes - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=67778ab> |
17:41 |
pinesol |
[evergreen|Jason Etheridge] lp1894005 Stripe payment intents, live test - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=f2e9c5b> |
17:41 |
pinesol |
[evergreen|Galen Charlton] LP#1894005: stamp DB update - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=2aa315c> |
17:49 |
pinesol |
[evergreen|Bill Erickson] LP1904788 Staff catalog browse results paging - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=cd4c4c9> |
17:49 |
pinesol |
[evergreen|Bill Erickson] LP1904788 Browse navigation release notes - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=d4b0190> |
17:55 |
pinesol |
[evergreen|Jeff Godin] LP#1264746 Add "email password reset" to user editor - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=52a8141> |
17:55 |
pinesol |
[evergreen|Jeff Godin] LP#1264746 return boolean, not promise - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=b675d13> |
17:55 |
pinesol |
[evergreen|Galen Charlton] LP#1264746: add a release notes line - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=44578f1> |
18:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
18:06 |
|
rjackson_isl_hom joined #evergreen |
19:50 |
|
rjackson_isl_hom joined #evergreen |
20:22 |
|
jihpringle joined #evergreen |
01:05 |
|
gmcharlt joined #evergreen |
01:05 |
|
jweston joined #evergreen |
01:10 |
|
berick joined #evergreen |
06:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
07:32 |
|
rjackson_isl_hom joined #evergreen |
07:56 |
|
collum joined #evergreen |
08:33 |
|
Dyrcona joined #evergreen |
11:46 |
tlittle |
rfrasur Same! I think those are awesome |
11:46 |
Dyrcona |
Right, well, looks like I have to start over..... |
11:51 |
mmorgan |
:-( |
11:57 |
JBoyer |
I suppose this may be as good a time as any to drop the record count to around 1000 so it doesn’t take ~4 days to test? That should still give plpgprofiler plenty to work with. |
12:03 |
|
mmorgan1 joined #evergreen |
12:10 |
|
jihpringle joined #evergreen |
12:18 |
Dyrcona |
JBoyer: At this point, I might as well wait until the bad drive is replaced and start over with a new data set and a new batch of records. |
13:06 |
|
collum joined #evergreen |
13:24 |
|
jvwoolf joined #evergreen |
14:08 |
|
javier_guel joined #evergreen |
14:10 |
javier_guel |
Hi all, I am trying to upgrade from 3.4.2 to 3.7.1 and I am testing upgrade scripts in a dev db, but running 3.5.1-3.6.0-upgrade-db.sql script I am getting the Error "3.5.1-3.6.0-upgrade-db.sql:995: ERROR: syntax error at or near "ON" |
14:10 |
javier_guel |
LINE 2: ...S ('au.created', 'au', 'A user was created', 't') ON CONFLIC...", could someone help me? |
14:12 |
Dyrcona |
javier_guel: What PostgreSQL version are you running? |
14:12 |
javier_guel |
9.6.15 |
14:13 |
javier_guel |
Do I need to upgrade to 10v? |
16:33 |
|
jvwoolf left #evergreen |
16:49 |
|
mmorgan1 joined #evergreen |
17:03 |
|
mmorgan1 left #evergreen |
18:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
18:41 |
|
jihpringle joined #evergreen |
21:22 |
|
Christineb_ joined #evergreen |
21:22 |
|
phasefx_ joined #evergreen |
06:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
07:30 |
|
rjackson_isl_hom joined #evergreen |
07:56 |
|
mantis joined #evergreen |
08:42 |
|
mmorgan joined #evergreen |
11:12 |
Dyrcona |
I have started a load of 12,567 records with located URIs with pl_profiler enabled on Pg 10. I'll repeat this load on Pg 11 through 13 in the coming days. |
11:18 |
mmorgan |
Dyrcona: I'm interested in your results. Do you know how many URIs are in each record or does it vary? |
11:19 |
Dyrcona |
mmorgan: It's usually 1 in the incoming file, but some have 2. The majority of records, over 10,000, match existing records so the number of URIs varies. |
11:20 |
Dyrcona |
I've loaded this set of records into production already, which is why I'm using a dump from July 11 for this test. |
11:23 |
* mmorgan |
nods. We've been looking at how long it takes to load each record and have found a record with 10 links take on average 12 seconds. |
11:23 |
Dyrcona |
I suspect that Pg 10 will give the best results, and I'm doing it as a benchmark. Also, Pg 10 is currently optimized on this server. |
11:23 |
Dyrcona |
mmorgan: It gets slower on newer Pg versions, and I want to find out where/why so we can fix it. |
11:24 |
mmorgan |
Dyrcona++ |
11:25 |
Dyrcona |
When testing a timed version of this script, I've seen it take well over 20 seconds to update some records |
11:25 |
|
jvwoolf joined #evergreen |
11:26 |
Dyrcona |
I usually kill it after a few days on Pg 12. :( |
11:26 |
mmorgan |
:-( |
17:38 |
|
nfBurton joined #evergreen |
17:45 |
Bmagic |
oh, it looks like the reason comes out in the subsequent email trigger event. Unless you have the followup email to the patron, it doesn't look like we can tell (from the database) why something doesn't get auto-renewed? |
17:49 |
|
jihpringle joined #evergreen |
18:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
00:30 |
|
Keith_isl joined #evergreen |
06:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
07:09 |
JBoyer |
I guess that's finally the end of #evergreen on Freenode. As of ~ 1am it now requires TLS and SASL (good) with their relatively new nickserv database (bad). |
07:09 |
JBoyer |
csharp_++ # less + v is a good tip! |
07:12 |
|
rjackson_isl_hom joined #evergreen |
15:27 |
miker |
if anyone's interested in picking it up to coauthor, I'm happy to push it out for more brains to attack |
15:27 |
miker |
I don't have tuits for it ATM, but really want it to happen |
15:28 |
Dyrcona |
Yeah. There has been a long term shortage of tuits. |
15:30 |
Dyrcona |
I'll take a look at the rebased branch next week. I can't promise that I can do anything with the JavaScript. I'm going to be busy testing Quipu integration. |
15:48 |
|
miker joined #evergreen |
16:06 |
|
nfBurton joined #evergreen |
16:26 |
|
mmorgan1 joined #evergreen |
16:44 |
|
brettgilio left #evergreen |
17:08 |
|
mmorgan left #evergreen |
17:26 |
|
rlefaive joined #evergreen |
18:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
20:45 |
|
rlefaive joined #evergreen |
01:05 |
|
Keith__isl joined #evergreen |
06:02 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
06:39 |
|
brettgilio joined #evergreen |
07:23 |
|
mantis joined #evergreen |
08:35 |
|
mmorgan joined #evergreen |
09:27 |
Dyrcona |
So, for those following along (or not), I have run my db upgrade script on Pg 9.6, Pg 10, and Pg 11 so far. This includes a partial ingest of record attributes for MARC item type g. |
09:31 |
Dyrcona |
It took 28 hours 48 minutes on Pg 9.6. On Pg 10, it ran for 25 hours 3minutes. Pg 11 finished after 12 hours 7 minutes. This is with the db being "optimized" and the server rebooted in between each configuration change. |
09:31 |
berick |
pg11 FTW |
09:32 |
Dyrcona |
I probably won't get around to "testing" Pg 12 and Pg 13 until next week. I want to restore the configuration for Pg 10 for an automated production restore over the weekend. |
09:32 |
Dyrcona |
Yeah, my experience is newer Pg releases are faster over all, but some specific things have degraded performance. |
09:33 |
|
jvwoolf joined #evergreen |
09:35 |
Dyrcona |
I'm doing the full did you mean setup at the moment. I have a script that I run with time to get the timing of it. I thought about just using the files generated from the first process since they should be all the same, but that wouldn't quite be fair. |
09:37 |
Dyrcona |
That process took just about 4 hours (minus 2 or 3 minutes) on both Pg 9.6 and Pg 10, so it will be interesting to see the time for Pg 11. |
09:38 |
Dyrcona |
I may do all of this again if we get the bad drive replaced. |
09:43 |
Dyrcona |
Hmm. I may get to testing Pg 12 today after all. Pg 11 seems to be ripping right through the DYM stuff. |
09:45 |
Dyrcona |
66GB in cache.... |
09:45 |
|
mantis joined #evergreen |
09:49 |
berick |
too bad you can't do them all at once, we could take bets. |
12:08 |
|
jihpringle joined #evergreen |
13:49 |
Dyrcona |
Well, Pg 11 didn't make much difference for the did you mean setup. It finished only about 2 minutes faster than Pg 9.6. I think that's because the process spends most of its time reading and writing data files with Perl. |
14:33 |
|
stephengwills joined #evergreen |
15:24 |
Dyrcona |
berick: Do I have to md5 hash the password before using AppUtils::verify_user_password? I'm getting 0 when I try to verify either of the passwords on my test account. |
15:29 |
Dyrcona |
Apparently, I have to do more than that because the database function fails with the correct information. |
15:30 |
* Dyrcona |
looks at how open-ils.auth does it, again. |
15:33 |
berick |
Dyrcona: verify_user_password assumes non-md5'ed passwords. verify_migrated_user_password assumes md5-hashed passwords. |
15:55 |
berick |
mine |
15:55 |
Dyrcona |
berick: switching password and type looks wrong given the actor.verify_passwd signature and the code in AppUtils.pm. |
15:56 |
berick |
Dyrcona: right, your code is fine, it's just the sql that's different -- but the apputils code accounts for that |
15:56 |
Dyrcona |
berick: I accounted for that with my tests. |
15:57 |
Dyrcona |
Hey! patebot! You here? |
15:57 |
Dyrcona |
pastebot, even... |
15:58 |
Dyrcona |
http://paste.evergreen-ils.org/14413 |
16:27 |
Dyrcona |
typos-- :) |
16:28 |
Dyrcona |
Too many os, 0s, and uppercase vs. lowercase typos. :) |
16:28 |
Dyrcona |
IOW, it helps to use the correct password when trying to verify it. :) |
16:31 |
mmorgan |
Never hurts to test the wrong password, too, to make sure it's not verifying inapproriately :) |
16:32 |
Dyrcona |
Well, that's true. |
16:32 |
jeff |
yes. |
16:33 |
jeff |
Dropbox learned that lesson in a very public fashion in 2011. :-) |
16:55 |
Dyrcona |
Very few say "drone." |
16:58 |
* Dyrcona |
calls it a day. Thanks, everyone! |
17:22 |
|
mmorgan left #evergreen |
18:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
20:59 |
|
stephengwills left #evergreen |
00:12 |
|
Keith__isl joined #evergreen |
00:12 |
|
alynn26_away joined #evergreen |
06:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
07:11 |
|
rjackson_isl joined #evergreen |
07:14 |
|
mantis joined #evergreen |
07:34 |
|
tlittle joined #evergreen |
11:05 |
|
jvwoolf joined #evergreen |
11:11 |
|
nfBurton28 joined #evergreen |
11:11 |
|
nfBurton88 joined #evergreen |
11:30 |
Dyrcona |
I've added settings for the backend admin user and org unit in the Quipu Ecard code and modified it to use them. I've tested what happens with some similar code if the settings are not set. It fails, of course, and the message is: "Can't call method "id" on an undefined value." |
11:30 |
Dyrcona |
I see 2 ways to deal with this, just return if either or both settings are not found, or fallback to using the already retrieved vendor user and org unit 1. |
11:31 |
Dyrcona |
csharp_ || berick ^^ |
11:33 |
Dyrcona |
I'm leaning toward the fallback mode. Does anyone have an opinions either way? I know it's hard to say without context. |
16:17 |
Dyrcona |
I'm not convinced that we can add a print event as a standard part of Evergreen at this point. |
17:05 |
|
nfBurton joined #evergreen |
17:08 |
|
mmorgan left #evergreen |
18:02 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
18:39 |
|
jihpringle joined #evergreen |
02:28 |
|
eady joined #evergreen |
06:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
08:10 |
|
collum joined #evergreen |
08:32 |
|
mantis joined #evergreen |
08:32 |
|
Dyrcona joined #evergreen |
09:33 |
|
mmorgan left #evergreen |
09:35 |
|
mmorgan joined #evergreen |
09:42 |
mmorgan |
For a workstation using Hatch, is there a way to force the printer dialog? Staff want to have the option to choose a different printer for the pull list. |
09:45 |
JBoyer |
Dyrcona, so long as the array is messed up for all of them it should still a valid test. You're comparing the versions relative to each other, not to a similar box with a working array. |
10:01 |
|
jvwoolf joined #evergreen |
10:05 |
Dyrcona |
JBoyer: True, but I'd like to fix it. We're going to see if we have a drive we can swap in, soon. Failing that I have a couple of old database servers that I might be able to use, but they don't have enough space to hold multiple copies of our database. |
10:06 |
JBoyer |
Oh, sure; I don't mean you shouldn't look into it, just that the current work isn't wasted. Comparing things across the line where it's fixed is no good, but that's different. |
10:25 |
Dyrcona |
I haven't exercised 13 as much as 12, but if you're on a relatively recent schema, then there aren't blockers other than performance going to 12 that I am aware of. Everything seems to work. |
10:25 |
berick |
cool, thanks |
10:25 |
berick |
may just start pushing my dev VM's to 13 just cuz |
10:26 |
Dyrcona |
I've been using 12 with my test/development VMs at CW MARS. I just decided to switch port 5432 to Pg 10 last week in preparation for going to Pg 10 in production. |
10:29 |
Dyrcona |
On my generic VMs, where I install concerto data and run the DB locally, I've been installing Pg10. |
10:29 |
JBoyer |
I do have a few demo / concerto machines on 12/13 so there shouldn't be anything major, but there are a few things that never really get tested on those. (perf among them) |
10:31 |
Dyrcona |
We load 10,000+ batches of electronic resource records from time to time, and I have a Perl script for that which can output timing data with a command line switch. |
10:32 |
Dyrcona |
I'll run that on the test server with a batch occasionally. It will take a couple of seconds to update most records on Pg 9.6. On Pg 12 it's typically 5 to 10 seconds, with some records going over 20 to 30 seconds. |
10:32 |
Dyrcona |
A batch of 10,000 can take hours on Pg 9.6 and days on Pg 12. |
10:33 |
berick |
Dyrcona: and that's still located URI-related? |
10:33 |
Dyrcona |
I want to profile that with plprofiler. It will hopefully show where the slowness comes from. |
13:27 |
berick |
well, except in the printer settings UI where you can 'print with dialog' |
13:29 |
mmorgan |
berick: That's what I was looking for! I don't see it in printer settings, though. |
13:30 |
|
jvwoolf joined #evergreen |
13:33 |
berick |
mmorgan: it's under the Test Printing tab |
13:33 |
berick |
so, it's only there for testing |
13:34 |
berick |
if you test it, beware the dialog does not always (ever?) steal focus, so it sits open behind the browser window. |
13:35 |
berick |
i could imagine having a way to specify "print via browser" for certain print contexts / templates |
13:41 |
mmorgan |
Oh, ok. At least I know I didn't imagine it ;-). They are looking for a way to redirect to another printer when their main one has an issue. Likely not something that will happen that often. I will offer them the option of disabling hatch momentarily when that's necessary. |
13:41 |
mmorgan |
berick++ |
13:43 |
berick |
gotach. it's also a pretty quick change to tell Hatch to use a different printer |
16:34 |
|
jvwoolf1 joined #evergreen |
16:58 |
|
jvwoolf1 left #evergreen |
17:14 |
|
mmorgan left #evergreen |
18:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
01:10 |
|
akilsdonk joined #evergreen |
06:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
07:33 |
|
rjackson_isl joined #evergreen |
07:47 |
|
collum joined #evergreen |
08:36 |
|
mmorgan joined #evergreen |
14:51 |
|
jvwoolf2 joined #evergreen |
15:07 |
berick |
you know what they say, cleverness killed the c... cantaloupe! |
15:23 |
Dyrcona |
:) |
15:23 |
Dyrcona |
I think that I sometimes forget to do git pull before make install, also, so my fix doesn't always end up on the test system. |
15:51 |
Dyrcona |
Hrm... I got o eg/opac/home on my test VM and Chrome 92 complains about too many redirects. I add a trailing slash, and it loads then ends up at a 404. |
15:52 |
Dyrcona |
Hmm. Maybe that's not quire right, either. It's loading without /, now. |
16:01 |
|
jvwoolf2 left #evergreen |
16:06 |
mmorgan |
@blame the clever cantaloupes |
16:06 |
pinesol |
mmorgan: everything was going great until the clever cantaloupes came along |
16:06 |
Dyrcona |
@band add The Clever Cantaloupes |
16:06 |
pinesol |
Dyrcona: Band 'The Clever Cantaloupes' added to list |
16:26 |
|
mmorgan1 joined #evergreen |
16:42 |
|
mmorgan joined #evergreen |
17:04 |
|
mmorgan left #evergreen |
18:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
06:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
07:21 |
|
collum joined #evergreen |
08:31 |
|
mantis joined #evergreen |
08:31 |
|
stephengwills joined #evergreen |
12:16 |
Dyrcona |
berick++ |
12:21 |
Dyrcona |
A better patch might be to move the functional code for undelete_biblio_record_entry to BibCommon.pm and call that from Cat.pm and AssetCommon.pm. Just throwin' that out there. |
12:25 |
Dyrcona |
So, my upgrade to master-d9bb7c9205 from 3.5.3 took 25 hours, and that's not counting the "did you mean" setup. |
12:25 |
pinesol |
Dyrcona: [evergreen|Jane Sandberg] LP1718782: follow up to fix failing test - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=d9bb7c9> |
12:34 |
|
collum joined #evergreen |
12:54 |
|
collum joined #evergreen |
13:20 |
|
abowling joined #evergreen |
15:21 |
abowling |
berick: yeah. darndest thing. i've never run into this on other boxes |
15:21 |
abowling |
this is an AWS VM. i don't think that should matter, but... |
15:22 |
JBoyer |
abowling, anyhing interesting in your ~/.ssh/config? Seems odd that it's trying to use ssh to clone from github |
15:23 |
berick |
i think what Dyrcona said is the main issue, the package should not be using ssh: |
15:23 |
berick |
just tested on a VM with github key disabled and got the same error |
15:23 |
abowling |
JBoyer: berick: agree |
15:25 |
abowling |
JBoyer: not at all; straightforward |
15:26 |
JBoyer |
seeing berick's note about *removing* his makes my question kind of unnecessary. Agreed that Dyrcona nailed it above. Someone screwed up upstream. :/ |
16:10 |
Dyrcona |
Ah, so it was us... |
16:11 |
Dyrcona |
abowling++ |
17:19 |
|
denials joined #evergreen |
18:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
19:00 |
|
stephengwills left #evergreen |
20:43 |
|
stephengwills_ joined #evergreen |
20:44 |
|
stephengwills_ left #evergreen |
05:18 |
|
degraafk_ joined #evergreen |
06:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
08:18 |
|
mantis joined #evergreen |
08:19 |
|
Dyrcona joined #evergreen |
08:58 |
|
miker joined #evergreen |
09:30 |
Dyrcona |
It doesn't lose its mind, but changes don't always show up. |
09:31 |
Dyrcona |
I could be cargo culting.... |
09:37 |
Dyrcona |
I don't recall if the OPAC picks up changes right away, either. |
09:39 |
Dyrcona |
Bmagic | gmcharlt: FYI I'm messing with Pg settings on my test db server. I'm changing the ports that the different instances listen on and reviewing the optimization settings. I'm going to come up with a set of optimizations to use more of the hardware and possibly modify the "40%" settings. |
09:40 |
Dyrcona |
I'm also going to open a Lp bug and a Google doc for notes, etc. I think I'll make a folder for the project, so I can share multiple docs if necessary. |
09:50 |
|
jvwoolf joined #evergreen |
09:51 |
miker |
Bmagic: autogen's main purpose, outside of upgrades (where new classes are routinely added, and field positions change), is to flush the copy of the org tree (and related structures) from memcached. that's used by the web client just as much as it was the xul client. so, yes, you must run it after changes to the org tree that get cached there. also, there is some in-process caching in both mod_perl and certain service backends, so an apache and |
12:06 |
|
nfBurton joined #evergreen |
12:13 |
Dyrcona |
bshum++ |
12:53 |
|
sandbergja joined #evergreen |
13:08 |
jeff |
if you have a degraded array, you might want to correct that before putting effort into running tests. |
13:09 |
jeff |
though it's possible the results would be suitable for comparison with each other... maybe. |
13:12 |
Dyrcona |
jeff: I know. We replaced some drives in this machine a few years ago. I'll ask if we have any spares left over. If not, I may not have a choice at this point. |
13:13 |
Dyrcona |
If that's the case, then I'll see what other hardware we have available. We don't have anything else with as much disk space, so it would make the testing rather time consuming. |
13:14 |
jeff |
you might also be able to reconfigure the storage in a way that requires fewer physical drives. |
13:14 |
Dyrcona |
"If that's the case," meaning if we don't have spares. |
13:14 |
* jeff |
nods |
16:26 |
Keith_isl |
Dyrcona ++ |
16:27 |
Dyrcona |
I just ran totals for the day: 98 not connected in total, and only 8 for drones. So that's more like 1 in 12 is a drone. |
16:35 |
|
jvwoolf1 joined #evergreen |
18:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
18:05 |
|
jvwoolf1 left #evergreen |
06:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
07:54 |
|
brettgilio joined #evergreen |
08:29 |
|
Dyrcona joined #evergreen |
08:37 |
|
mantis joined #evergreen |
09:49 |
mantis |
Seems that the OPAC has the same results |
09:50 |
csharp_ |
we've talked about a "staff_visible" column on copies/locations but that's not been implemented yet as far as I know\ |
09:51 |
alynn26 |
OPAC Visible is only used in the public facing OPAC. Try it in incognito mode. And have you set the Actual Org unit as OPAC Visible. |
09:51 |
Bmagic |
mantis: be sure and log out before testing the OPAC. Even though you change the URL to the Patron side, you are still logged in (unless you actively click log off) - will will still show you staff results |
09:51 |
mantis |
select * from asset.copy where holdable = 'f' and opac_visible ='f' and not deleted and circ_lib = 95; comes up with the number of results I needed |
09:52 |
Dyrcona |
manits: What Bmagic said. I was typing up almost the same thing, or use an incognito/private window for the patron opac. |
09:52 |
mantis |
Bmagic: awesome thank you |
10:29 |
JBoyer |
nfBurton, I recall working on something like that; and possibly instance holds also. I'll see if I can find it. |
10:30 |
Dyrcona |
nfBurton: It doesn't look like the SIP code handles part holds at all. |
10:30 |
nfBurton |
I pushed a patch on my system from an LP bug where part holds would respond blank. |
10:31 |
nfBurton |
Still testing it though |
10:31 |
Dyrcona |
nfBurton: If you send your SIP logs through syslog, you'll get a timestamp added by syslog. |
10:31 |
JBoyer |
yeah, bug 1525394 |
10:31 |
pinesol |
Launchpad bug 1525394 in Evergreen 3.6 "SIP patron part level holds respond blank" [Undecided,New] https://launchpad.net/bugs/1525394 |
10:35 |
JBoyer |
1525394 has a signoff for Bmagic's commit, but not mine. Looks like it hasn't been touched since. |
10:35 |
JBoyer |
If you have some part holds that work and some that don't that might warrant investigation, otherwise it sounds like there's more to do on it. :/ |
10:35 |
Dyrcona |
nfButon: If it's working for you, you could always add a signoff. |
10:36 |
nfBurton |
I'm still testing but can once I get more results |
10:36 |
nfBurton |
i need to learn signoffs anyways |
10:36 |
nfBurton |
JBoyer Ah, I didn't see the second patch |
10:37 |
nfBurton |
JBoyer++ |
10:41 |
mantis |
Dyrcona: This one particular circ mod has a 14d_0r rule without any autorenewals |
10:42 |
mantis |
Yet I still got a notice saying "no autorenewals remaining" |
10:42 |
Dyrcona |
Yeah, that would seem to make sense. It does run for every circulation. |
10:42 |
mantis |
Ok thanks |
10:42 |
mantis |
Dyrcona++ |
10:43 |
mantis |
I guess I never had this come across via testing |
10:44 |
Dyrcona |
If you think of the notice as a reminder/courtesy to the patron, it makes sense to tell them that the item couldn't be renewed. After all, patrons can't be expected to know all of the circ rules. |
10:45 |
mantis |
Right. I agree. I can see why it also causes confusion with some people, too. |
10:45 |
mantis |
We're still on 3.5.4 but upgrading to 3.6.4 this year. I saw in the patch notes that the reasonings have been updated. |
15:53 |
Dyrcona |
So, it can be controlled without iptables, I hope. |
15:53 |
Dyrcona |
https://metacpan.org/pod/Net::Server#CONFIGURATION-FILE |
16:06 |
|
abneiman joined #evergreen |
18:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
06:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
07:09 |
|
rjackson_isl joined #evergreen |
08:05 |
|
Dyrcona joined #evergreen |
08:22 |
|
rfrasur joined #evergreen |
09:23 |
stephengwills |
no, at docs.evergreen-ils.org. and I thought I did make -f Open-ILS/src/extras/Makefile.install postgres-server-ubuntu-bionic-10 on my postgresql server. isn’t that enough to install the prerequisites? |
09:24 |
Dyrcona |
stephengwills: Maybe not. It could be a bug in the prerequisite install target. |
09:25 |
stephengwills |
i’ll try and cut some time to poke around more before I get complacent with my installed server and move on to other stuff. again, so helpful y’all. thanks. |
09:28 |
Dyrcona |
To be honest, I almost never refer to the documentation and go by release notes and trial and error. I've usually done the upgrade several times on test VMs before I do the production upgrade. |
09:30 |
stephengwills |
in the past I’ve done that too but I am an applications layer guy who spends most of my time in PHP so when I crawling onto the cloud the more NASA-like my steps are the smoother it goes for me. LOL! |
09:31 |
stephengwills |
Note to self: check to make sure the doc.eg-ils.org clearly states i must plug in my computer before I flip the ON switch. |
09:32 |
Dyrcona |
So, you could argue that it is a bug in the Pg server targets. They don't include any CPAN modules, and the Levenshtein, et al., are installed via CPAN. |
10:36 |
stephengwills |
I implied it when quoting the make install for ubuntu bionic and Pg10 |
10:36 |
Dyrcona |
Yeahp. |
10:37 |
Dyrcona |
I have confirmed that the CPAN modules don't get installed on Debian Buster. |
10:37 |
Dyrcona |
I should test again with a Bionic VM. |
10:37 |
Dyrcona |
I may have to amend my bug report. |
10:39 |
Dyrcona |
Maybe focal and stretch, too. Guess I'm spending the weekend rebuilding my local test VMs..... |
10:40 |
stephengwills |
my slice is an AWS EB2 Ubuntu 18.04 LTS with Pg v10 and I did make the extras from Eg3.7.1 on it and, ultimately had to install the two modules I mentioned above to make 3.6.2-3.7.0-upgrade-db.sql happy. |
10:45 |
Dyrcona |
stephengwills: Is the database on a separate host/instance from the Evergreen servers? If it was all on the same machine and the general ubunbut-bionic prerequisites were installed, then the modules should have been there. |
10:50 |
jeff |
I do really wish that Chrome had an option to allowlist an extension (Hatch) without force installing it. |
15:57 |
mmorgan |
Oooh. Dyrcona lives dangerously :) |
15:59 |
Dyrcona |
And, the patch works! |
16:00 |
Dyrcona |
I should update the commit message with verification instructions and then force push it. |
16:00 |
Dyrcona |
Still have to test on stretch and bionic, but I'm 99.9% certain that it will work there as well, since it's all the same code. |
16:15 |
|
mmorgan1 joined #evergreen |
17:21 |
|
mmorgan left #evergreen |
18:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
06:02 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
07:12 |
|
rjackson_isl joined #evergreen |
08:38 |
|
mmorgan joined #evergreen |
08:43 |
|
Dyrcona joined #evergreen |
11:50 |
|
jihpringle joined #evergreen |
11:56 |
stephengwills |
so, on line 30 fo Account.pm where it makes a backup of the user object there may not be one yet? |
11:56 |
stephengwills |
s/fo/of/ |
12:03 |
jeff |
seems likely. |
12:03 |
jeff |
$self->ctx->{user} may be undefined for unknown reasons at the point where that line defiens $usr: my $usr = $self->ctx->{user}; |
12:05 |
jeff |
$ctx->{user} should be getting set in OpenILS::WWW::EGCatLoader::load_common |
12:05 |
jeff |
I think that attempting a fresh login and consulting logs would be a good next step. Is this happening to all users on a test system? |
12:11 |
stephengwills |
i’ll tst with multiple users |
12:13 |
jeff |
how did the problem come to your attention? |
12:13 |
jeff |
ah, I see the "I just upgraded from 3.2.2 to 3.7.1" earlier in channel. |
15:51 |
stephengwills |
my apologies for useing so much bandwidth, my 3.6.2-3.7.0-upgrade-db.sql changes got rolled back to a missing String::KeyboardDistance module |
16:03 |
Dyrcona |
stephengwills: Yeah, there are some new prerequisites for 3.7. |
17:19 |
|
mmorgan left #evergreen |
18:00 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
01:48 |
|
dluch joined #evergreen |
01:48 |
|
devted joined #evergreen |
01:48 |
|
Bmagic joined #evergreen |
06:01 |
pinesol |
News from qatests: Failed Running pgTAP tests <http://testing.evergreen-ils.org/~live//archive/2021-07/2021-07-14_04:00:03/test.42.html> |
07:15 |
|
rjackson_isl joined #evergreen |
08:23 |
|
collum joined #evergreen |
08:26 |
|
rfrasur joined #evergreen |
10:39 |
rhamby |
"copier beware" |
10:43 |
mmorgan |
berick: Ok thanks. But I was wondering if not nullifying the update_process might cause issues. |
10:43 |
berick |
mmorgan: nah, it's purely informational and gets immediately overwritten. |
10:44 |
mmorgan |
ok, thanks! |
10:44 |
mmorgan |
berick++ |
10:44 |
mmorgan |
stephengwills++ |
10:52 |
mmorgan |
stephengwills: Are you using TPAC or Bootstrap OPAC? I have a recent master test system on Bootstrap and am not seeing any such errors. |
10:53 |
stephengwills |
TPAC, I don’t want to complicate applying my customizations with bootstrap under i get it stable |
10:53 |
stephengwills |
until (not under) |
10:54 |
stephengwills |
now i’ll confess that I have not commented out my custom_templates to see if I still getthe probelm |
11:13 |
stephengwills |
yup…thanks. I’m stepping away. getting frustrated and making mistakes :). bbilb |
11:36 |
|
Dyrcona joined #evergreen |
11:38 |
Dyrcona |
Just throwing this out there: Hidden access points are more trouble than they are worth. |
11:45 |
sandbergja |
that failing test sure looks like it's from my patch for bug 1718782; I can go ahead and push a fix (unless somebody already started working on it) |
11:45 |
pinesol |
Launchpad bug 1718782 in Evergreen 3.6 "webclient: MARC bib editor doesn't show SRCE in the fixed fields" [Low,Fix committed] https://launchpad.net/bugs/1718782 |
11:49 |
Dyrcona |
sandbergja: You pushing a fix is OK by me. I had a quick look last night, but didn't go so far as to think about a fix. |
12:01 |
|
Dyrcona joined #evergreen |
12:07 |
|
stephengwills left #evergreen |
12:10 |
sandbergja |
Dyrcona: that's too bad. I like how repair-able the pinebook pro sounds from its web site |
12:15 |
Dyrcona |
I figured that I'd try it in the office today. I don't use it much. I seem to be having issues with the KDE session manager mostly. |
12:28 |
pinesol |
[evergreen|Jane Sandberg] LP1718782: follow up to fix failing test - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=d9bb7c9> |
12:31 |
|
collum joined #evergreen |
12:41 |
eby |
besides running the cron at midnight are there any tricks to getting an action trigger delay/max_delay so it selects everything on a single day (midnight to midnight)? |
12:41 |
eby |
nothing is jumping out in the interval seconds calculations |
14:58 |
gmcharlt |
Acquisitions interest group meeting starting in two minutes: https://wiki.evergreen-ils.org/doku.php?id=acq:minutes:2021-07-14 |
16:39 |
|
mmorgan1 joined #evergreen |
17:11 |
|
mmorgan1 left #evergreen |
18:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
18:12 |
|
dluch joined #evergreen |