05:00 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
05:15 |
|
jonadab joined #evergreen |
06:07 |
|
jonadab joined #evergreen |
06:27 |
|
abowling joined #evergreen |
13:30 |
NawJo |
Thank you all for your help, thank you Dyrcona and Berick :) |
13:33 |
Dyrcona |
You can login now? |
13:36 |
NawJo |
Yes, yes :) |
13:37 |
NawJo |
I wrote the rtl stylesheets for web client days ago, and now I want to test them :) |
13:38 |
Dyrcona |
OK. Have fun and thanks for the stylesheets. |
13:38 |
berick |
NawJo++ |
13:39 |
NawJo |
:) thank you all |
16:16 |
Dyrcona |
echo...echo.. echo... My brain can spell, it's just my fingers that can't. |
16:17 |
berick |
just tried for grins, no dice. I also tried exporting TZ before the srfsh line |
16:17 |
* berick |
confirms it's in the osrf message w/ the original command |
16:18 |
gmcharlt |
berick: yep, the TZ should be passsed in |
16:18 |
gmcharlt |
and I've verified in the srfsh log on my test system that the tz is being passed in the request |
16:18 |
gmcharlt |
now checking the cstore side of things |
16:18 |
berick |
gmcharlt++ |
16:19 |
Dyrcona |
OK. I've never mangled the env with pipes much. |
16:19 |
Dyrcona |
It looks...well, weird and wrong. |
16:27 |
* berick |
nods |
16:35 |
|
BAM_ joined #evergreen |
16:39 |
|
kmlussier joined #evergreen |
16:40 |
Dyrcona |
I tested with circulation, so transactions. |
16:40 |
Dyrcona |
I don't think I thought to test it without a transaction. |
16:41 |
Dyrcona |
"I don't think I thought..." heh.... |
16:41 |
gmcharlt |
friends don't let friends do circulation transactions outside of transactions |
16:41 |
Dyrcona |
:) |
16:53 |
bshum |
Leading to a bad video |
16:53 |
bshum |
But, hey, fun! |
16:53 |
bshum |
:D |
17:01 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
17:08 |
|
mmorgan joined #evergreen |
17:14 |
|
mmorgan left #evergreen |
17:29 |
|
jvwoolf left #evergreen |
21:41 |
bshum |
Oh special. |
21:41 |
bshum |
https://bugs.launchpad.net/ubuntu/+source/ejabberd/+bug/1659801 |
21:41 |
pinesol_green |
Launchpad bug 1659801 in ejabberd (Ubuntu) "apparmor rules block ejabberdctl" [Undecided,New] |
21:41 |
bshum |
Just came across this segmentation error on Ubuntu 16.04 dry run while trying to run through ejabberd user registration :\ |
21:41 |
bshum |
Ruined my asciinema recording, bah |
21:42 |
bshum |
Guess I'll have to try it all over again |
21:43 |
* bshum |
blows away his VM to test anew |
21:49 |
* bshum |
considers making the recording on Trusty instead |
21:50 |
bshum |
Or Debian... *shudders* |
23:05 |
Dyrcona |
I have not encountered that one, but I always start ejabberdctl with sudo. |
23:06 |
bshum |
I ran it as root |
23:06 |
bshum |
And got that error |
02:18 |
|
dbwells_ joined #evergreen |
04:46 |
|
gsams joined #evergreen |
04:55 |
|
gsams joined #evergreen |
05:02 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
06:40 |
|
rlefaive joined #evergreen |
06:53 |
|
agoben joined #evergreen |
07:12 |
|
rjackson_isl joined #evergreen |
10:36 |
kmlussier |
If we do keep it, though, I agree with Dyrcona that we might want to name it .next so that we don't need to rename it every time we make a big jump in release numbers. |
10:36 |
berick |
i guess the rule is when you add a pullrequest for a wishlist/feature bug, you have to set a milestone of 'next' or it will likley be ignored ? |
10:36 |
berick |
yes, definitely needs renaming |
10:38 |
kmlussier |
berick: I've always followed that rule when submitting my own code because I was told it would be more likely to get attention if it was targeted. In general, when looking for code to test, I usually just look for the pullrequest tag. |
10:39 |
bshum |
One of the old ideas was to rename 2.next into 2.actualnumber-alpha but that got tiresome to keep reshuffling bugs away from the actual milestones. So we ended up leaving 2.next as is as a general placeholder to target future dev |
10:40 |
bshum |
I'd prefer to keep a target around for next dev just cause I do find that untargeted bugs can easily get lost in the mire of open tickets depending on how you set your search scopes |
10:40 |
berick |
kmlussier: same here, i just look for the tag |
10:42 |
kmlussier |
Overall, I think it would be good practice to try to go through all those bugs with pullrequest tags, merge the ones that are ready, and actively remove the pullrequest tags, possibly adding a needsrepatch tag, for ones that aren't ready. |
10:42 |
kmlussier |
But now I'm living in the land of dreams. |
10:43 |
kmlussier |
I do find that many of the bugs that sit without a pullrequest tag are there because we're reluctant to remove the pullrequest if it's not ready or needs a test. |
10:43 |
kmlussier |
OK, now I'm just talking gibberish. /s/without/with |
10:45 |
berick |
that's consistent w/ my lp-world view |
10:46 |
berick |
pullrequest means merge me as soon as possible, if I don't have a milestone, put me in master. |
10:46 |
* berick |
is not pushing for anything, just like knowing how people work |
11:05 |
berick |
as to me they are the same thing |
11:05 |
bshum |
As long as bugs get worked on, I think that's the key goal :D |
11:06 |
berick |
very diplomatic ;) |
11:06 |
bshum |
gmcharlt: Speaking of, sorry I saw your question... but got distracted, lol. So uh, my answer is maybe |
11:06 |
bshum |
gmcharlt: I didn't get to play with it directly myself yet |
11:06 |
bshum |
But I'm always happy to test and merge stuff that people push my way |
11:07 |
kmlussier |
Yeah, if I were proposing anything, it's probably that bugs get worked on. :) I know core committers are busy, but one or two bugs a month seems reasonable to me. |
11:30 |
csharp |
gmcharlt: I'll take a look |
11:33 |
gmcharlt |
csharp: great! |
15:08 |
berick |
booting_up_the_thunderdome++ |
15:09 |
berick |
(it runs on windows 98) |
15:09 |
csharp |
https://www.youtube.com/watch?v=tajDxBaPBBM |
15:09 |
berick |
you've got mail! |
15:10 |
|
kmlussier joined #evergreen |
15:11 |
berick |
miker: another thought, assuming a non-hatch version of offline is viable, starting there lowers the barrier to entry for broader testing. |
15:12 |
miker |
it does, certainly. (sans printing stuff above) |
15:36 |
|
hbrennan joined #evergreen |
15:39 |
|
Jillianne joined #evergreen |
15:40 |
Dyrcona |
gmcharlt | csharp: https://bugs.launchpad.net/evergreen/+bug/1669868/comments/9 |
15:40 |
pinesol_green |
Launchpad bug 1669868 in Evergreen ""make check" failure" [High,Confirmed] |
15:40 |
Dyrcona |
I developed that on Ubuntu 16.04. I'll test on wheezy, jessie, and trusty next. |
15:41 |
gmcharlt |
Dyrcona: thanks |
15:48 |
Dyrcona |
Works for me on jessie. |
15:51 |
Dyrcona |
Works on wheezy. |
16:27 |
|
mmorgan joined #evergreen |
16:52 |
bshum |
Taco time :) See you guys later... |
17:00 |
Dyrcona |
I ..am.. outta here! |
17:00 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
17:09 |
|
mmorgan left #evergreen |
17:10 |
|
khuckins joined #evergreen |
17:11 |
* kmlussier |
heads out to get pizza. |
05:01 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
07:05 |
|
NawJo joined #evergreen |
07:22 |
|
rjackson_isl joined #evergreen |
07:54 |
|
collum joined #evergreen |
09:51 |
* Dyrcona |
thinks his initial assessment of a "bug" might be wrong. |
10:00 |
Dyrcona |
Just doing this: "select distinct * from biblio.record_entry where not deleted" and print the marc field from the result leads to memory starvation. |
10:01 |
Dyrcona |
That is more or less the query if you do marc_export -a with no other selection options. |
10:02 |
Dyrcona |
I don't believe the starvation happened on Wheezy or Trusty. I can build a vm on one of those later to test it. |
10:02 |
|
mmorgan1 joined #evergreen |
10:06 |
jeff |
Dyrcona: in your test case above, you're still using perl + dbi, or do you have that issue even executing the query via psql? |
10:06 |
Dyrcona |
jeff: That is Perl + DBI. |
10:06 |
jeff |
Dyrcona: and, which distros have you encountered the issue on? |
10:06 |
Dyrcona |
This is Debian 8 Jessie. I've seen similar on Ubuntu 16.04, but has not tested this specific script. |
10:07 |
Dyrcona |
I'm doing it without the distinct to see if that makes a difference. |
10:07 |
Dyrcona |
Also, it's important to note that we have approximately 2.7 million bibs that are not deleted. |
10:07 |
* jeff |
nods |
10:41 |
Dyrcona |
Yes, understood. And cursors are so 20th century.... :) |
10:43 |
Dyrcona |
I don't think it's a question of the amount of RAM changing. I believe the server that I used to run this on had 8GB, and I've run it on a vm with 8GB with similar results. |
10:43 |
Dyrcona |
Similar results meaning it crashes. |
10:43 |
jeff |
Yeah, I was about to suggest re-testing to see if you can reproduce on older distro with this same dataset. |
10:44 |
jeff |
But at this point, that's probably mostly to satisfy curiosity. |
10:44 |
jeff |
grabbing a configurable chunk of bibs at a time will probably be the fix. |
10:44 |
jeff |
default to 10k or 50k or whatever testing shows to use a semi-reasonable amount of ram. |
10:45 |
jeff |
which version of perl are you running on the problematic system? |
10:46 |
|
Christineb joined #evergreen |
10:47 |
jeff |
oh. jessie, therefore 5.20.2 |
10:59 |
Dyrcona |
Xenial is 5.22.something |
11:00 |
jeff |
https://rt.cpan.org/Public/Bug/Display.html?id=93266 is the DBD::Pg bug for fixing this long-term, which also isn't immediately useful. |
11:06 |
Dyrcona |
Right. |
11:06 |
Dyrcona |
I'm going to build some 8GB VMS with different distros: Wheezy, Jessie, Trusty, and Xenial to test this. |
11:07 |
Dyrcona |
Mabye not today, but soon. |
11:07 |
Dyrcona |
I may have a non-issue. :) |
11:20 |
|
fbeaudry joined #evergreen |
16:19 |
|
eady joined #evergreen |
16:20 |
|
rlefaive joined #evergreen |
16:29 |
|
kmlussier joined #evergreen |
17:01 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
17:05 |
|
mmorgan left #evergreen |
23:20 |
|
genpaku joined #evergreen |
05:01 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
06:53 |
|
agoben joined #evergreen |
07:10 |
|
rjackson_isl joined #evergreen |
07:30 |
graced |
Happy St. Patrick's Day , #evergreen! |
15:51 |
kmlussier |
Actually, it makes sense. The ones that don't have covers in the feed also don't have them in the regular catalog. |
15:52 |
Dyrcona |
Ah ha! |
15:53 |
* Dyrcona |
learned something: The action.hold_notification is created by the cleanup routine on the action trigger runner, but that still doesn't tell me why it failed on Monday. |
15:56 |
* kmlussier |
looks longingly at bug 1236979, which retrieved that feed so much more quickly, but was not quite right in testing. |
15:56 |
pinesol_green |
Launchpad bug 1236979 in Evergreen "Speed up bibs-by-item-age" [Medium,Confirmed] https://launchpad.net/bugs/1236979 - Assigned to Galen Charlton (gmc) |
15:57 |
kmlussier |
Oh! I hadn't noticed gmcharlt's name on that one. |
15:57 |
StomproJ |
Dyrcona, I just tried it out and ISBN lookup does still work for /ac/jacket/small/ISBN It just won't handle records with ISSN/UPC. |
16:49 |
mmorgan |
[% END %] |
16:49 |
mmorgan |
terran++ |
16:57 |
kmlussier |
terran++ |
17:01 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
17:07 |
|
mmorgan left #evergreen |
01:39 |
|
StomproJ joined #evergreen |
01:45 |
|
Guest45619 joined #evergreen |
05:02 |
pinesol_green |
News from qatests: Test Failure <http://testing.evergreen-ils.org/~live> |
06:40 |
|
rlefaive joined #evergreen |
07:11 |
|
rjackson_isl joined #evergreen |
07:16 |
|
agoben joined #evergreen |
07:39 |
* pinesol_green |
brews and pours a cup of Ethiopia Yirga Cheffe Koke Espresso, and sends it sliding down the bar to graced |
07:39 |
graced |
mmmmmm |
07:53 |
|
kmlussier joined #evergreen |
07:55 |
kmlussier |
PgTAP live test failure. Must be a release day. |
08:02 |
rhamby |
kmlussier: think of it as getting it out of the way early |
08:06 |
kmlussier |
Beware the Ides of March |
08:13 |
rhamby |
kmlussier: yes but in revenge Octavian took revenge on 300 senators and slew them on an altar dedicated to Ceasar... so slaying bugs is revenge? |
08:14 |
|
rlefaive joined #evergreen |
08:17 |
kmlussier |
rhamby: Do I have to slay 300 bugs? |
08:19 |
rhamby |
kmlussier: I think the actual number should be relative to the crime, so failing a pgtap test isn't quite up there with murder |
08:19 |
kmlussier |
OK, the Concerto users shifted in actor.usr. |
08:20 |
* kmlussier |
is experiencing Déjà vu |
08:22 |
kmlussier |
If I change the id in the test to point to new id 189, we should be good. |
08:23 |
kmlussier |
And add a comment to bug 1672434 that we also need to address the addition of usr records to the sample dataset. |
08:23 |
pinesol_green |
Launchpad bug 1672434 in Evergreen "Improved method for adding new bib records to test dataset" [Undecided,New] https://launchpad.net/bugs/1672434 |
08:40 |
kmlussier |
bug 1673059 for anyone who would like to test and signoff. |
08:40 |
pinesol_green |
Launchpad bug 1673059 in Evergreen "Update passwd storage test" [Undecided,New] https://launchpad.net/bugs/1673059 |
08:44 |
bshum |
kmlussier: I'll test and get that in, since I broke it. |
08:44 |
bshum |
Should only take a moment or two |
08:48 |
|
rlefaive joined #evergreen |
08:51 |
|
mmorgan joined #evergreen |
08:54 |
bshum |
kmlussier++ # test passed for me, pushed to master for you |
08:55 |
kmlussier |
bshum++ Thank you! |
08:58 |
pinesol_green |
[evergreen|Kathy Lussier] LP#1673059: Update passwd storage test - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=6cd11aa> |
08:59 |
|
Dyrcona joined #evergreen |
09:01 |
|
Callender joined #evergreen |
09:38 |
|
maryj joined #evergreen |
09:44 |
gmcharlt |
I've now branched rel_2_12 in preparation for the RC |
09:44 |
gmcharlt |
and now that that branch exists... I'm declaring a freeze on it :) |
09:45 |
gmcharlt |
specifically, please don't merge anything that is not directly related to the release until I say otherwise |
09:50 |
dbs |
bshum: what the heck? we both ran the live tests and didn't see any issues, right? *sigh* |
09:50 |
JBoyer |
gmcharlt++ |
09:51 |
kmlussier |
gmcharlt++ |
09:51 |
dbs |
gmcharlt++ |
16:55 |
remingtron |
frank_guel: you may need to view that record in your Evergreen staff client and use the "Clear AddedContent Cache" link on the right sidebar |
16:57 |
remingtron |
frank_guel: here is the documentation for this feature, in case it helps: http://docs.evergreen-ils.org/2.11/_including_external_content_in_your_public_interface.html#_clear_external_added_content_cache |
16:59 |
frank_guel |
remingtron: at IPICYT we are still on 2.8.4 EG version, so there is no the Clear addedContent cache option, |
17:00 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
17:05 |
remingtron |
frank_guel: Okay. It looks like this feature was added in 2.9: https://bugs.launchpad.net/evergreen/+bug/1435938 |
17:05 |
pinesol_green |
Launchpad bug 1435938 in Evergreen "New Feature: Allow Staff to clear Added Content cache" [Wishlist,Fix released] |
17:06 |
|
Jillianne2 joined #evergreen |
18:02 |
dbwells |
gmcharlt++ |
18:08 |
gmcharlt |
also, I have created LP milestones for _both_ 2.12.0 and 2.12.1 and divvied up the unfixed bugs that were assigned to 2.12-rc |
18:10 |
gmcharlt |
I was conservative in how I did that: if it wasn't something that visibly affected installation on one or more platforms or caused a lot of JS console spewage, I bumped it to 2.12.1 rather than 2.12.0 |
18:21 |
frank_guel |
remingtron: thanks for your help. It let me understand how it works. |
18:22 |
frank_guel |
As a part of the 2.11.3 EG update testing, I already install EG and it looks working well on opac, but I am getting this message when I want to access to staff client: XML Parsing Error: undefined entity Location: chrome://open_ils_staff_client/content/main/main.xul Line Number 21, Column 1:<window id="main_win" ^ |
18:34 |
dbs |
frank_guel: if you want to remove the cached images right away, you can do something like 'memcrm --servers=localhost ac.jacket.small.0471828726' |
18:34 |
dbs |
(repeating for medium, large in place of small) |
18:35 |
dbs |
memcdump --servers=localhost | grep ac.jacket.large # will show you all of the keys for cached large jackets |
20:04 |
* bshum |
should start writing stuff like this down |
20:04 |
bshum |
For said talk :) |
20:06 |
frank_guel |
thanks , it works, bshum |
20:07 |
bshum |
I'm not sure why the built clients don't have the right content in them. gmcharlt should check his install directory to see what's where. On my test system when I build all the locales, lang.dtd looks fine. And we only just branched rel_2_12 today, so I would have thought the new 2.12-rc client should be fine too. |
20:08 |
bshum |
frank_guel++ Thanks for checking that out for us! |
20:09 |
frank_guel |
:) |
20:09 |
* bshum |
feels like Bmagic encountered this error recently too, but we just weren't sure what we were looking at then; check above for details if you still want Spanish :) |
20:10 |
gmcharlt |
I'll look at it tomorrow |
04:04 |
|
Jillianne joined #evergreen |
05:01 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
06:41 |
|
rlefaive joined #evergreen |
07:29 |
|
rjackson_isl joined #evergreen |
07:34 |
|
JBoyer joined #evergreen |
10:25 |
jeff |
sunshine. |
10:25 |
Dyrcona |
Well, it's supposed to turn into a blizzard. |
10:25 |
Dyrcona |
So far, no blizzard, just steady snow. |
10:27 |
kmlussier |
remingtron: Regarding your question about perl tests on bug 1586567, my thought was that it's small enough that we could merge it with the two signoffs in place of the test. I don't know if anyone else has strong opinions that it should have a test. |
10:27 |
pinesol_green |
Launchpad bug 1586567 in Evergreen "Acq: Year Drop Down Doesn't Sort Chronologically" [Low,Confirmed] https://launchpad.net/bugs/1586567 |
10:28 |
kmlussier |
Or, I should say, two additional signoffs in addition to your own. |
10:32 |
kmlussier |
It's snowing a lot here now. I don't think the blizzard is supposed to start until the afternoon. |
10:42 |
pinesol_green |
[evergreen|Jason Etheridge] lp1261835 stop colliding bill UI refreshes - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=bb5d651> |
10:51 |
Dyrcona |
kmlussier: The consensus at one time was if there are no tests, an extra signoff was necessary, so if there are 3 total signoffs, including the original, I think it is OK. |
10:52 |
kmlussier |
Dyrcona: Yes, but the consensus was that the three signoffs only happened if the bug was accompanied by "a statement from the patch author explaining why a test is infeasible without significant refactoring." |
10:55 |
kmlussier |
IMO, I'm thinking that a test is probably more effort than its worth for what it is fixing. |
10:57 |
Dyrcona |
It could be. I haven't looked. |
10:59 |
Dyrcona |
I take it the yearstore.items.sort().reverse() wasn't working? |
10:59 |
Dyrcona |
Ah, yes. So says the commit message. |
11:02 |
* Dyrcona |
puts another maple log on the fire. |
11:11 |
Dyrcona |
I suppose that doesn't need a test, though a test would be pretty easy to write. |
11:22 |
pinesol_green |
[evergreen|Christine Morgan] lp1629016: Hide List Option In Staff Client - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=5437490> |
11:30 |
|
Christineb joined #evergreen |
11:36 |
|
maryj_ joined #evergreen |
16:18 |
pinesol_green |
Launchpad bug 1621178 in Evergreen "webclient: Copy status field missing from column pickers" [Medium,Confirmed] https://launchpad.net/bugs/1621178 |
16:20 |
kmlussier |
Bug 1671936 definitely needs to get in before the release is cut. |
16:20 |
pinesol_green |
Launchpad bug 1671936 in Evergreen "1006 upgrade script needs reingest instructions" [High,New] https://launchpad.net/bugs/1671936 |
16:20 |
kmlussier |
That's all I have for now. |
16:20 |
kmlussier |
gmcharlt++ for helping with release notes today. |
16:22 |
kmlussier |
Actually, I do have one more thing I can merge that I've already tested. Then I will really, really be done. |
16:23 |
kmlussier |
Also, berick++for taking on some of the bugs I posted in here yesterday. |
16:33 |
pinesol_green |
[evergreen|Remington Steed] LP#1586567 Always return Fund Years sorted descending - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=c17e66a> |
16:40 |
gmcharlt |
kmlussier: noting that miker might have a couple last-minute booking and other webstaff fixes to merge in this afternoon |
16:40 |
kmlussier |
gmcharlt: OK, noted. |
16:51 |
miker |
yeah, looks like just one, for booking from holdings view |
16:52 |
kmlussier |
heh...controversial booking code. |
16:53 |
miker |
pushed |
16:55 |
pinesol_green |
[evergreen|Jason Etheridge] webstaff: Make Items Bookable and Book Item Now for Holdings View - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=549d6a0> |
17:00 |
|
khuckins__ joined #evergreen |
17:01 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
17:17 |
Bmagic |
We are interested in getting our catalog in google results. We are generating the sitexml files nightly. Any other tricks anyone tried? Like geo data in the meta tags? |
17:19 |
jeff |
Bmagic: dbs has done a bit of work in this area. have you looked at any of his writings/presentations? |
17:20 |
Bmagic |
It's vaguely familiar, I know that schema.org vocaular is sneaking into the template toolkit |
19:58 |
dbs |
Start with the sitemap and see how many pages Google adds to its index. |
20:20 |
|
kmlussier joined #evergreen |
22:24 |
pinesol_green |
[evergreen|Dan Scott] LP#1671112 Sample users and addresses with Unicode characters - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=9793d47> |
22:39 |
bshum |
Neato |
22:39 |
bshum |
First time using nginx to proxy websocket and stuff |
22:39 |
bshum |
Cool stuff :) |
22:41 |
bshum |
berick: Just FYI, I tested out the apache config changes for https://bugs.launchpad.net/evergreen/+bug/1648234 and it worked for me (minus a tiny tweak to setup on 7443 and not 70443 like the test plan says) |
22:41 |
pinesol_green |
Launchpad bug 1648234 in Evergreen "Internal Apache port leaks when using proxy" [Undecided,New] |
22:41 |
bshum |
I'll make a note on the bug and push a signoff branch |
22:42 |
bshum |
I couldn't figure out how to get haproxy going yet, but the nginx instructions are pretty straightforward thankfully. Though I did have to disable the default nginx config which is not mentioned in the OpenSRF readme |
01:58 |
|
NawJo joined #evergreen |
02:22 |
|
NawJo joined #evergreen |
03:22 |
|
NawJo joined #evergreen |
05:00 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
07:26 |
|
rjackson_isl joined #evergreen |
07:33 |
|
agoben joined #evergreen |
08:41 |
|
mmorgan joined #evergreen |
10:30 |
pinesol_green |
Launchpad bug 1522644 in Evergreen "webclient: Transfer title holds issues" [Medium,New] https://launchpad.net/bugs/1522644 |
10:38 |
|
NawJo joined #evergreen |
10:41 |
kmlussier |
Also, let me know if there is anything you would like me to review to get into this release. At the moment, I'm feeling pretty good about where we are with the release. |
10:43 |
berick |
kmlussier: is this a new bug.. testing 1522644, I notice when viewing holds, if I click Next or Previous to change records, it does not refresh the holds list to match the new record. |
10:43 |
berick |
unrelated to 1522644, of course, just noticed it |
10:45 |
* kmlussier |
checks |
10:46 |
berick |
i can look, just thought it might sound familiar |
10:48 |
kmlussier |
berick: Looks like that's an existing bug. I don't think I've seen that one in LP yet. Good catch! |
13:42 |
Dyrcona |
:) |
13:43 |
jeff |
JBoyer: can't remember if you were around last week when we were talking about bug 1671150 -- i did eventually realize why you were seeing the behavior you were (unaccent dictionary also needs to be in the search path or explicitly specified) |
13:43 |
pinesol_green |
Launchpad bug 1671150 in Evergreen "Unqualified references in evergreen.unaccent_and_squash lead to index creation failures with pg_restore" [Undecided,In progress] https://launchpad.net/bugs/1671150 - Assigned to Jeff Godin (jgodin) |
13:44 |
JBoyer |
I was, and if no one has signed off on your changes yet I'm planning to test them soon and do so. (Last week we had a migration that had me out of the office for a while) |
13:44 |
jeff |
i'll let you know when i push a branch. ideally later today, we'll see. |
13:46 |
jeff |
JBoyer: as someone with a decent amount of rows in actor.usr who has likely manually created those indexes a few times now, do you have any opinion on dropping and re-creating them vs trying to create them outside of a transaction and warning to ignore the resulting warning/error output as "normal"? |
13:46 |
jeff |
if the indexes don't take that long to create, i'm almost leaning toward drop/create. |
14:08 |
Dyrcona |
And, you can mix and match. :) |
14:09 |
Dyrcona |
Not that you should...but in real life...with real data... |
14:09 |
Dyrcona |
I love the records that come up "short" because they have some Windows smart quote in a field. Part of the multibyte sequence is a record terminator. |
14:10 |
* jeff |
nods |
14:10 |
jeff |
there was a recent patch for MARC::File::XML to try and handle those better. |
14:10 |
jeff |
i haven't tested to see how yaz tools handle it |
14:11 |
jeff |
oh, nevermind -- outstanding pull request from tsbere, actually: https://github.com/perl4lib/marc-perl/pull/4 |
14:12 |
jeff |
though there's something else similar that i saw elsewhere... hrm. |
14:14 |
Dyrcona |
Writing your MARC record splitter in Perl is remarkably simple. |
14:14 |
Dyrcona |
I keep words... :) |
14:14 |
jeff |
and this: https://rt.cpan.org/Public/Bug/Display.html?id=70169 |
15:59 |
Dyrcona |
Also, maybe someone else added it. I haven't checked. |
15:59 |
Dyrcona |
My --descendants option seems to be working, though. |
15:59 |
Dyrcona |
I'll Lp it tomorrow. |
16:00 |
Dyrcona |
I'm going to try combining the two libraries that I'm testing with. It's supposed to work with more than 1. I want to make sure. |
16:02 |
Dyrcona |
These are the two libraries that export records for EDS. |
16:09 |
jeff |
and yes, the 852 has two $b subfields. i left those and added the 999 tags that had been used in the mapping from one library's previous EDS catalog. :-) |
16:09 |
jeff |
i think i also added logic to remove any existing 999 fields even if not doing an --items export, since their imported bibs still have the legacy 999 at the moment. |
16:21 |
Dyrcona |
:) |
16:23 |
Dyrcona |
Well, I'm satisfied --descendants works for me. I'll put in in production for tomorrow night's monthly export. |
16:27 |
Dyrcona |
tramp++ |
17:01 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
17:07 |
|
mmorgan left #evergreen |
18:44 |
|
mllewellyn left #evergreen |
19:08 |
|
Jillianne joined #evergreen |
08:56 |
jeff |
"marc_export creating MARC data that yaz-marcdump dislikes" |
08:56 |
Dyrcona |
I think it's Encode.pm or possibly MARC::Charset with perl >= 5.20. |
08:57 |
|
jvwoolf joined #evergreen |
09:04 |
Dyrcona |
I will test something before I send that email. |
09:09 |
jeff |
okay, yep. extract_holdings in this environment emits iso-8859-1 in the main holdings file, and i can override this by adding an encoding to the open() call, like this: |
09:09 |
jeff |
open HOLDINGS, '>:encoding(UTF-8)', ($c->{prefix} . "-HOLDINGS.pg"); |
09:11 |
jeff |
(which may or may not be the best solution) |
09:54 |
Dyrcona |
I wonder if 20G is enough room for 2.7 million MARCXML records? :) |
09:56 |
Dyrcona |
It's weird, though. It seems almost like whatever is happening messes with Perl's output buffering. |
09:57 |
Dyrcona |
I'd expect to get some output as Perl flushes it buffers every 4 to 64K. |
10:01 |
phasefx |
kmlussier: test.evergreen-ils.org is out of disk space; I'll see if I can delete some stuff |
10:01 |
kmlussier |
phasefx: Thanks for looking into it! phasefx++ |
10:01 |
|
rlefaive_ joined #evergreen |
10:07 |
jeff |
bug 1671845 |
11:14 |
|
brahmina joined #evergreen |
11:20 |
|
khuckins__ joined #evergreen |
11:21 |
|
rlefaive joined #evergreen |
11:26 |
* Dyrcona |
goes to make another iced tea while his test runs. |
11:34 |
csharp |
@tea Dyrcona |
11:34 |
* pinesol_green |
brews and pours a pot of Dragon Pearl Jasmine, and sends it sliding down the bar to Dyrcona (http://ratetea.com/tea/harney/dragon-pearl-jasmine/1320/) |
12:11 |
pinesol_green |
News from qatests: Test Failure <http://testing.evergreen-ils.org/~live> |
12:12 |
kmlussier |
:( |
12:19 |
kmlussier |
egDate? |
12:22 |
berick |
ah |
12:22 |
berick |
the classic "2 days" is not always 48 hours. |
12:22 |
berick |
kmlussier: if we ignore it until monday it will go away (/me ducks) |
12:23 |
kmlussier |
Sounds like a good plan to me! At least until they do away with all this clock-changing nonsense. |
12:24 |
berick |
i'd say in this case egDate is fine, the unit test is faulty |
12:24 |
berick |
(well, and a comment in egDate that mimics the unit test) |
12:55 |
* csharp |
totally forgot that the time changes on Sunday |
12:56 |
Dyrcona |
Yeah. |
12:56 |
Dyrcona |
Two days = 48 hours give or take an hour or two. |
14:08 |
pinesol_green |
Dyrcona: Dunno #13: "Go away, or I'll replace you with a very small shell script!" (added by Dyrcona at 04:39 PM, October 02, 2012) |
14:08 |
Dyrcona |
:) |
14:12 |
kmlussier |
berick++ bug 1671904 |
14:12 |
pinesol_green |
Launchpad bug 1671904 in Evergreen "egDate unit test fails on time change boundary" [Undecided,New] https://launchpad.net/bugs/1671904 |
14:31 |
Dyrcona |
I love it when git diff tells me two lines are different, but I don't see and it doesn't look like whitespace changes, either. |
14:31 |
Dyrcona |
Although git diff -w doesn't show this particular difference. |
14:32 |
berick |
no one expects the zero-width space |
15:39 |
|
Jillianne joined #evergreen |
15:41 |
kmlussier |
I've added a branch at working/collab/kmlussier/2_12RC_release_note_additions where release note changes can be made. |
16:01 |
|
dkyle joined #evergreen |
16:31 |
pinesol_green |
News from qatests: Test Failure <http://testing.evergreen-ils.org/~live> |
16:55 |
jeff |
i think that marc_export with --all and --items may exclude bibs with zero holdings, vs marc_export --all |
17:03 |
|
mmorgan left #evergreen |
17:18 |
jeff |
yup, confirmed. |
18:18 |
jeffdavis |
I asked the other day about using the new nginx proxy setup for different OPAC skins on separate subdomains. Turns out the default nginx config works just fine for this, as long as you remember to adjust the port numbers in *all* your Apache vhosts. *shakes head at self* |
18:58 |
|
Enjabain joined #evergreen |
21:25 |
|
jvwoolf joined #evergreen |
23:28 |
pinesol_green |
[evergreen|Bill Erickson] LP#1671904 egDate unit test handles time change - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=482c984> |
09:36 |
kmlussier |
If there are any other bug fixes you think need to make it into the release, let me know. I probably won't get to them today, but will make some time to review branches before gmcharlt cuts the release next week. |
09:47 |
|
mmorgan1 joined #evergreen |
09:51 |
Dyrcona |
Well, I'm gonna try on another fresh vm elsewhere. |
09:51 |
Dyrcona |
I should really test what I want to test with production data, anyway. |
09:55 |
csharp |
Dyrcona: check journalctl for apport messages |
09:56 |
Dyrcona |
csharp: Too late. I blew it all away. |
09:56 |
Dyrcona |
Here's a middle finger for systemd-- :) |
15:36 |
Dyrcona |
What I don't get is why something that worked fine on Trusty and Wheezy eats all the RAM on Jessie and Xenial and it's basically doing what marc_export does, only less... |
15:36 |
|
NawJo_ joined #evergreen |
15:36 |
csharp |
"Boopsie extract" sounds like 19th century snake oil for babies :-) |
15:37 |
Dyrcona |
That was my main reason for testing marc_export, to see if it used all the RAM. |
15:37 |
Dyrcona |
:) |
15:37 |
Dyrcona |
Well, I think they named Boopsie after Betty Boop. |
15:37 |
Dyrcona |
So, close....:) |
15:42 |
Dyrcona |
it's work not to fun? |
15:42 |
berick |
Dyrcona: that checkouts out. transitive property |
15:43 |
Dyrcona |
:) |
15:43 |
bshum |
Huh... is the live tests system unhappy? Can't access the page, and no test success message from expected 5 am run this morning |
15:43 |
Dyrcona |
heh heh...he said transitive.... |
15:49 |
|
rlefaive joined #evergreen |
15:51 |
jeffdavis |
kmlussier: ebook API release notes pushed to working/user/jeffdavis/lp1541559-ebook-api-phase1-release-notes |
05:00 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
07:11 |
|
rjackson_isl joined #evergreen |
07:22 |
|
Callender joined #evergreen |
07:53 |
|
collum joined #evergreen |
09:16 |
dbs |
jeff: i filed my branch and bug 1671112 |
09:16 |
pinesol_green |
Launchpad bug 1671112 in Evergreen "Sample users and addresses do not contain any Unicode (non-ASCII) characters" [Undecided,New] https://launchpad.net/bugs/1671112 |
09:17 |
jeff |
dbs++ |
09:17 |
dbs |
Might be useful for testing search & display of Unicode in the user/address forms in webby? |
09:18 |
Dyrcona |
And other things. |
09:19 |
jeff |
looks like i got as far as a summary for the bug i started last night: ``Unqualified references in evergreen.unaccent_and_squash lead to index creation failures with pg_restore'' |
09:22 |
csharp |
jeff: yeah, I have a self-signed cert, but I thought it worked before with self-signed certs |
09:40 |
Dyrcona |
That is, pending the inclusion of Lp 1373690 |
09:40 |
pinesol_green |
Launchpad bug 1373690 in Evergreen "Direct EDI generation for ACQ orders -- AKA kill ruby webrick" [Wishlist,New] https://launchpad.net/bugs/1373690 |
09:40 |
Dyrcona |
I'm tempted to try that in production. |
09:41 |
csharp |
Dyrcona: I thought of updating the script to be more distro-accommodating, but thought we'd already be moving to the "new" way before it became necessary |
09:41 |
csharp |
Dyrcona: also, everything but B&T was well-tested at the Hack-A-Way |
09:41 |
csharp |
so probably good to go |
09:42 |
Dyrcona |
Well, looks like the new way won't make it before 3.0. |
09:42 |
Dyrcona |
2.12-RC is tomorrow, yeah? |
09:42 |
csharp |
right |
09:50 |
Dyrcona |
Nope. /var/lib/gems and /usr/lib/ruby/gems are not links to each other. Maybe the gems under 2.3.0 are? |
09:51 |
Dyrcona |
Nope. Guess they're both in the gems path. |
09:51 |
* Dyrcona |
has seen things like that before, like with Perl and Python. |
09:52 |
csharp |
Dyrcona: the way berick and I were testing at the hackaway was to do "./edi_order_pusher.pl --test-mode --po-id 227 > brodart_new.edi" then compare to the original acq.edi_message in the DB, in case you want to go towards the non-Ruby EDI option |
09:53 |
Dyrcona |
csharp: Right. I recall. I was gonna help but had an emergency to deal with. |
09:53 |
Dyrcona |
I spoke briefly with tspindler about putting it in production and he wants to wait a bit. |
09:54 |
csharp |
Bmagic supplied an example or two of a B&T order, and it looked okay, but could definitely do with more tests with real-life data |
09:54 |
Dyrcona |
When I get the time...ha! ha! ha!...I'll give testing a whirl with our data. |
09:54 |
csharp |
heh - let me know where you find the time, cuz I need some too :-) |
09:54 |
Dyrcona |
And, bshum corrected me about 2.12-rc. It is next Wednesday. |
09:54 |
csharp |
ah |
10:01 |
berick |
FF requires both |
10:03 |
csharp |
berick: oh - cool - I'll do that |
10:03 |
|
mmorgan1 joined #evergreen |
10:03 |
berick |
Dyrcona: if you want to test the edi stuff, I'd be happy to rebase |
10:04 |
csharp |
berick++ |
10:04 |
berick |
well, i'll go ahead and rebase anyway. would love to continue progress there regardless |
10:15 |
|
dbwells joined #evergreen |
10:16 |
csharp |
berick: count me in for more testing/prototyping |
10:16 |
Dyrcona |
berick: That's OK. I'll rebase if I need to. I might try it with my custom 2.11 branch also. |
10:17 |
csharp |
we're also up for piloting this here as soon as it's up to feature-parity |
10:17 |
Dyrcona |
We'll probably backport it. |
10:18 |
berick |
now have working/user/berick/lp1373690-acq-edi-perl-attrs -- should be easy to backport. it's currently just 3 commits. |
10:18 |
berick |
i would def. recommend testing a variety of vendor PO's before switching over |
10:19 |
Dyrcona |
I could probably test all of ours with a bit of Perl. :) |
10:20 |
Dyrcona |
Maybe pick a date range and let it go. |
10:20 |
berick |
if you have any local JEDI changes, they will have to be replicated in the edi-attrs |
10:20 |
berick |
which is currently accessible only via direct db access |
13:09 |
|
dbwells joined #evergreen |
13:27 |
Bmagic |
jeff++ # I agree, Postgres management documentation specific to Evergreen would be nice. I have some things to contribute to that |
13:28 |
|
jvwoolf joined #evergreen |
13:32 |
jeff |
I was starting to draft a test plan for a bugfix on bug 1671150 when I thought to look. |
13:32 |
pinesol_green |
Launchpad bug 1671150 in Evergreen "Unqualified references in evergreen.unaccent_and_squash lead to index creation failures with pg_restore" [Undecided,In progress] https://launchpad.net/bugs/1671150 - Assigned to Jeff Godin (jgodin) |
13:34 |
jeff |
I think common practice is "use create_database_extensions.sql to create the db and then restore into it", and even less documented is "set the search_path on the new database", and of course "make sure any users exist if this is a new/different cluster than where the dump originated..." |
13:53 |
csharp |
@dunno add PHRASING!!! |
15:08 |
|
_bott_ joined #evergreen |
15:27 |
|
rlefaive joined #evergreen |
16:43 |
|
kmlussier joined #evergreen |
17:01 |
pinesol_green |
News from qatests: Test Failure <http://testing.evergreen-ils.org/~live> |
17:05 |
kmlussier |
Oh, good! We already have a fix for that! |
17:09 |
|
mmorgan left #evergreen |
17:18 |
csharp |
@blame pinesol_green |
04:22 |
|
tsbere_ joined #evergreen |
05:02 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
06:40 |
|
rlefaive joined #evergreen |
07:15 |
|
rjackson_isl joined #evergreen |
07:23 |
graced |
good morning #evergreen |
08:47 |
* dbs |
goes and looks |
08:47 |
|
bos20k joined #evergreen |
08:49 |
dbs |
yeah I don't think so, from a quick scan. that's probably not great |
08:51 |
Dyrcona |
I messed with a few of the names after loading that to test that branch. |
08:52 |
dbs |
right but we should have names and addresses as part of the stock set to avoid surprises down the road |
08:57 |
jeff |
and there are pg_tap tests for the unaccent function, but i agree -- names with accents would be useful to add to the sample data. |
08:57 |
* dbs |
will see about putting together a branch to add some baseball players to the stock set and figure out a way to tie it into web staff client so it gets attention |
08:57 |
* dbs |
is sick today so has some time to run repeated live test builds :) |
09:08 |
|
jvwoolf joined #evergreen |
09:08 |
jeff |
@weather --wind ktvc |
09:08 |
pinesol_green |
jeff: Cherry Capital, MI :: Mostly Cloudy :: 50F/10C | Wind: N20mph (31mph gusts) | Tuesday: Partly cloudy and windy. High 53F. Winds SW at 20 to 30 mph. Winds could occasionally gust over 40 mph. Tuesday Night: Partly cloudy. Slight chance of a shower throughout the evening. Low 38F. Winds SW at 15 to 25 mph. | Updated: 15m ago |
10:29 |
bshum |
Honestly, it doesn't surprise me if the bug still manifests since 2.2 with new XUL. Nothing substantial has changed with XUL client in years |
10:31 |
Dyrcona |
The user isn't using a 485 are they? ;) |
10:31 |
Dyrcona |
oops. |
10:31 |
kmlussier |
bshum: Yeah, my surprise was based on a faulty memory that there was a tested fix there. I should add a fixedinwebby tag to that LP bug. |
10:31 |
Dyrcona |
Spoiled my own joke. |
10:31 |
Dyrcona |
It was tested. It mostly sorta worked. :) |
10:32 |
bshum |
Hehe |
10:32 |
Dyrcona |
Apparently, it might have broken other things. |
10:32 |
bshum |
Like most things |
10:57 |
|
JBoyer joined #evergreen |
11:11 |
|
brahmina joined #evergreen |
11:28 |
|
khuckins joined #evergreen |
11:37 |
dbs |
berick: am I reading ubuntu-auto-installer correctly, in that the the web staff client tests don't get run? |
11:39 |
berick |
dbs: looking at the 16.04 installer, I see a 'grunt test' in there |
11:39 |
berick |
ditto 14.04 |
11:40 |
dbs |
ah, not reading correctly! I was looking at the live tests section. thanks! |
11:41 |
berick |
cool, no prob |
11:52 |
|
mmorgan1 joined #evergreen |
12:04 |
|
mmorgan joined #evergreen |
15:36 |
Dyrcona |
:) |
15:36 |
bshum |
No publicly resolvable hostnames for me anyways |
15:36 |
bshum |
Maybe someday again |
15:37 |
Dyrcona |
Right for test vms, it's a self-signed cert, though I've considered using my own CA. |
15:37 |
jeffdavis |
ah, looks like an issue with hostnames in apache2.conf, not ssl |
15:37 |
bshum |
I always wished I had spent more time with the CA options in our Windows environment |
15:37 |
Dyrcona |
You can make your own CA, just don't expect to be automatically added to nss-cacerts. |
15:44 |
bshum |
Been there plenty of times |
15:46 |
* Dyrcona |
double checks his passwords in opensrf_core.xml. |
15:46 |
Dyrcona |
That was a recent instance of that, I had a router password where I wanted the opensrf password. |
15:47 |
Dyrcona |
And, I should be ready to test OpenSRF. |
15:48 |
Dyrcona |
And, no surprises, it works. |
15:49 |
|
kmlussier joined #evergreen |
15:49 |
Dyrcona |
I installed prerequisites from master, but installed 2.4.2 and configured partly using the 2.5.0-rc instructions with the addition of setting max_stanza_size in ejabberd.yml. |
15:50 |
Dyrcona |
Fun stuff... |
15:50 |
Dyrcona |
Now for something similar with Evergreen 2.10.... |
15:51 |
|
Jillianne joined #evergreen |
17:00 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
17:01 |
|
jvwoolf joined #evergreen |
17:06 |
|
jvwoolf1 joined #evergreen |
17:07 |
|
mmorgan1 left #evergreen |