| 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 |
