| 01:56 |
|
tsadok joined #evergreen |
| 02:52 |
|
tsadok joined #evergreen |
| 03:02 |
|
tsadok joined #evergreen |
| 06:01 |
pinesol |
News from qatests: Failed Running autogen.sh <http://testing.evergreen-ils.org/~live//archive/2022-04/2022-04-08_04:00:03/test.44.html> |
| 06:56 |
|
rjackson_isl_hom joined #evergreen |
| 07:47 |
|
collum joined #evergreen |
| 07:51 |
|
_collum joined #evergreen |
| 09:19 |
|
jvwoolf joined #evergreen |
| 09:58 |
|
Keith-isl joined #evergreen |
| 10:49 |
|
terranm joined #evergreen |
| 11:08 |
miker |
re the autogen failure, I'm going to do as (someone? Dyrcona?) suggested and not worry about testing $COVERDIR for write up front, rather just have the mkdir's run and then check. unless someone beats me to it with a different implementation. |
| 11:13 |
miker |
someone did not beat me to it. :) |
| 11:15 |
pinesol |
News from commits: LP#1967770: Followup fix for cover image directories <https://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=b5eb3fe1d3a23b8670bb13ba523845bb04f20cb6> |
| 11:22 |
terranm |
Bmagic: I'm getting lots of errors when I try to log on to https://bugsquash.mobiusconsortium.org/eg/staff |
| 11:53 |
Bmagic |
terranm: I'm not seeing them. Like in the console? |
| 11:59 |
Bmagic |
the game of "which one" is fun to play |
| 11:59 |
terranm |
+1 |
| 12:00 |
Bmagic |
services and autogen are done... still getting the partial splash screen |
| 12:00 |
Bmagic |
other than the splash screen - can the patches be tested? |
| 12:01 |
terranm |
AFAIK, the ones marked developer review can't be tested from the interface side |
| 12:02 |
mmorgan |
cusppe is from bug 1938835 |
| 12:02 |
pinesol |
Launchpad bug 1938835 in Evergreen "wishlist: customizable staff homepage" [Wishlist,Fix committed] https://launchpad.net/bugs/1938835 |
| 12:04 |
terranm |
mmorgan++ |
| 12:05 |
terranm |
I'm trying to test bug 1891369 but the circulation policies page doesn't want to populate |
| 12:05 |
pinesol |
Launchpad bug 1891369 in Evergreen "Circulation renewals near the due date should be extended" [Wishlist,New] https://launchpad.net/bugs/1891369 |
| 12:05 |
Bmagic |
I'm happy to give acces to whomeever wants to test the developer patches |
| 12:07 |
|
jihpringle joined #evergreen |
| 12:09 |
terranm |
When I try to load the circ policies page, the browser console has a bunch of 404 errors for .js files |
| 12:13 |
Bmagic |
hmm, looking |
| 12:36 |
Dyrcona |
Oh yeah, definitely revert the code for bug 1847805. The error message that Bmagic pasted at 11:57:10 is exactly the error caused by that branch. |
| 12:36 |
pinesol |
Launchpad bug 1847805 in Evergreen 3.8 "pcrud search can fail to retrieve rows that the user has access to" [High,Confirmed] https://launchpad.net/bugs/1847805 |
| 13:10 |
|
collum joined #evergreen |
| 13:14 |
Dyrcona |
Hmm. Lp doesn't refresh, and I missed miker's comment on lp 1931737 today. Now that I think about it, I wonder how backporting that bug to 3.7 and then upgrading to 3.8 is going to work. I suppose I can always test that after the ingest finishes. |
| 13:14 |
pinesol |
Launchpad bug 1931737 in Evergreen 3.8 "Did you mean breaks parallel reingest and causes deadlocks when loading/overlaying bib records in the client" [High,Confirmed] https://launchpad.net/bugs/1931737 |
| 13:16 |
Dyrcona |
I don't have to test it. It's going to remove the fix if 1298 gets applied after whatever the fix for the above bug is. This means the patch for 3.8 needs a different upgrade number for the patch in 3.7, so it will apply over it later. |
| 13:28 |
Bmagic |
Dyrcona: yes to the question about db load |
| 13:46 |
Dyrcona |
Bmagic: Did you see my comments about reverting 1847805? |
| 13:47 |
Bmagic |
Dyrcona: yep, just finished that task, It does seem to have fixed the splash screen |
| 16:48 |
|
rfrasur joined #evergreen |
| 17:10 |
|
mmorgan left #evergreen |
| 17:32 |
* miker |
reads up .. uh, rebase, not rebate. (fun-times with autocorrect) |
| 18:01 |
pinesol |
News from qatests: Failed Running perl live tests <http://testing.evergreen-ils.org/~live//archive/2022-04/2022-04-08_16:00:04/test.49.html> |
| 18:09 |
miker |
Dyrcona++ # in absentia, for 3.7-izing the pingest vs dym patch |
| 06:01 |
pinesol |
News from qatests: Failed Running autogen.sh <http://testing.evergreen-ils.org/~live//archive/2022-04/2022-04-06_04:00:02/test.44.html> |
| 07:16 |
|
rjackson_isl_hom joined #evergreen |
| 07:43 |
|
collum joined #evergreen |
| 08:07 |
|
rfrasur joined #evergreen |
| 10:09 |
|
jvwoolf joined #evergreen |
| 10:29 |
|
terranm joined #evergreen |
| 10:31 |
|
Keith-isl joined #evergreen |
| 10:46 |
Dyrcona |
The test server failures look like something peculiar to the test server environment. When I run autogen.sh and the tests on a local VM, they work. |
| 11:10 |
bshum |
Hmm, what I've found is that on the first attempt to run, it fails. But subsequent attempts work. |
| 11:10 |
bshum |
First try -- Unable to write to /openils/var/web/opac/extras/ac, please check |
| 11:10 |
bshum |
Second try, autogen goes through |
| 12:00 |
|
jihpringle joined #evergreen |
| 12:23 |
JBoyer |
It may be better to just create those empty dirs and drop a .placeholder in them so they can just be committed and not special-cased. |
| 12:24 |
JBoyer |
Though I guess depending on how explicit the release notes were re: creating those dirs yourself I can see some appeal to auto-creating them somehow. Not a big fan though. |
| 12:40 |
* Dyrcona |
didn't read the release notes to be honest. I was only looking into it because bshum mentioned that the test was failing to me the other day. |
| 12:42 |
Dyrcona |
bshum: I have installed Evergreen before on the VM where I checked, so that may have something to do with the differences in what you and I see. I didn't try on a clean virtual machine. |
| 12:55 |
Dyrcona |
bshum: You are correct. The $COVERDIR is not created before the check for its existence in autogen.sh, then it is created when the small, medim, and large subdirectories are created. |
| 12:56 |
Dyrcona |
My inclination is to remove the check for the COVERDIR being writable and just let it be created if necessary when the small, medium, and/or large subdirectories are created. |
| 16:39 |
pinesol |
News from commits: LP#1954923: Fix current date issue in Boostrap OPAC Circ History CSV export <https://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=e08d76965aaa5c30d1344e6d778577803b2d0723> |
| 17:05 |
|
mmorgan left #evergreen |
| 17:30 |
|
jvwoolf left #evergreen |
| 18:01 |
pinesol |
News from qatests: Failed Installing Angular web client <http://testing.evergreen-ils.org/~live//archive/2022-04/2022-04-06_16:00:03/test.29.html> |
| 18:13 |
|
jmurray-isl joined #evergreen |
| 18:13 |
|
rjackson_isl_hom joined #evergreen |
| 18:41 |
|
rjackson_isl_hom joined #evergreen |
| 01:32 |
|
JBoyer joined #evergreen |
| 06:01 |
pinesol |
News from qatests: Failed Running perl live tests <http://testing.evergreen-ils.org/~live//archive/2022-04/2022-04-05_04:00:02/test.49.html> |
| 07:14 |
|
rjackson_isl_hom joined #evergreen |
| 07:56 |
|
collum joined #evergreen |
| 08:32 |
|
rfrasur joined #evergreen |
| 09:21 |
rfrasur |
Jboyer, I'm getting a bad gateway error on pattypan eg and eg2. Not sure if that's intentional or not. |
| 09:22 |
rfrasur |
festivus is up |
| 09:27 |
rfrasur |
JBoyer (is IRC case sensitive? It's not right? Am I awake? Is this a lucid dream?) |
| 09:28 |
JBoyer |
It isn't really ready to test, I was loading something to see what state its in. |
| 09:29 |
Dyrcona |
rfrasur: The notification that a person gets when mentioned in channel depends on the client, not the server, so it could be case sensitive, though hopefully it isn't. |
| 09:29 |
rfrasur |
Dyrcona, good info. Thank you :-). |
| 09:29 |
JBoyer |
Also, alerts aren't case sensitive (for my client) and I also added some additional words to make more noise. ;) |
| 09:29 |
Dyrcona |
I was going to say that bad gateway usually means Apache isn't running or is otherwise broken. |
| 09:30 |
rfrasur |
JBoyer++ Dyrcona++ |
| 09:44 |
|
jvwoolf joined #evergreen |
| 09:58 |
miker |
rfrasur: I'm testing a fix for that failed test. the code works, it just assumes more than it should about directory layout in the test env |
| 09:58 |
|
terranm joined #evergreen |
| 10:07 |
Dyrcona |
Going back half an hour, I don't get notifications when I'm mentioned in channel because the Ubuntu packagers decided not to include that module for my IRC client in Ubuntu 20.04. Hopefully, it's there in Ubuntu 22.04. |
| 10:07 |
* Dyrcona |
could always offer to maintain it, I suppose. |
| 10:26 |
|
rjackson_isl_hom joined #evergreen |
| 10:29 |
|
rjackson_isl_hom joined #evergreen |
| 11:01 |
|
mmorgan joined #evergreen |
| 11:04 |
pinesol |
News from commits: LP#1967770: Fix test for cover image upload <https://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=4badce58a0e99d5f427724471ec27f87d013cf71> |
| 11:04 |
pinesol |
News from commits: LP#1967770: Make sure cover image dirs exist <https://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=a17a6aa05b36c9b78526a354aa92d10217f93b1b> |
| 12:05 |
|
jihpringle joined #evergreen |
| 12:33 |
|
collum joined #evergreen |
| 12:48 |
Dyrcona |
@band search overflow |
| 13:01 |
Dyrcona |
Yeah, just for giggles. This one looks the leader was "copied" to a tag "lea" with indicators e & r, followed by the leader. |
| 13:02 |
JBoyer |
"The best part is that you can take a plain text marc record from one system and paste it into another!" |
| 13:03 |
Dyrcona |
JBoyer++ |
| 13:27 |
Dyrcona |
terranm: I was thinking yesterday that we could use a "test" tag on Lp for bugs in tests. |
| 13:27 |
Dyrcona |
I always assumed "needstest" means that someone thinks the code should a test added. |
| 13:27 |
Dyrcona |
...should have a test added.... |
| 13:29 |
terranm |
If it were just "test" then I think people might interpret it a different way. What about sys-test or auto-test or something along those lines? |
| 13:30 |
Dyrcona |
Would "tests" plural be better? I'd like to keep it as simple as possible but no simpler. :) |
| 13:33 |
Dyrcona |
We could go nuts and add pg-test, pg-live-test, perl-test, and perl-live-test, since we do have the 4 kinds of tests. Then I guess we could add tags for angular-test and angularjs-test. Do we have any more? |
| 14:11 |
JBoyer |
If there's a need for an LP tag for bugs in tests how about 'testbugs'? covers the bases without being too easily confused with needstest or similar. |
| 14:28 |
Dyrcona |
testbugs sounds good to me. |
| 14:31 |
terranm |
okay, I'll add it |
| 15:52 |
|
jvwoolf left #evergreen |
| 16:06 |
Dyrcona |
I wanted: git rev-list -n4 --reverse lp1847805 |
| 16:07 |
Dyrcona |
Guess I'll have to continue this tomorrow.... |
| 16:08 |
Dyrcona |
mmorgan++ |
| 16:13 |
Dyrcona |
Hmm... Now that I've gotten the code to apply to one test machine it isn't working for me there. I'll test it again tomorrow before I update bug 1847805. |
| 16:13 |
pinesol |
Launchpad bug 1847805 in Evergreen 3.8 "pcrud search can fail to retrieve rows that the user has access to" [High,Confirmed] https://launchpad.net/bugs/1847805 |
| 16:13 |
Dyrcona |
I want to test on a different Evergreen release. |
| 17:04 |
|
jihpringle joined #evergreen |
| 17:11 |
|
mmorgan left #evergreen |
| 18:01 |
pinesol |
News from qatests: Failed Running autogen.sh <http://testing.evergreen-ils.org/~live//archive/2022-04/2022-04-05_16:00:02/test.44.html> |
| 21:52 |
|
jvwoolf joined #evergreen |
| 21:57 |
|
jvwoolf left #evergreen |
