01:06 |
|
stozza joined #evergreen |
01:37 |
|
stozza joined #evergreen |
04:51 |
|
stozza joined #evergreen |
05:01 |
pinesol_green |
News from qatests: Test Failure <http://testing.evergreen-ils.org/~live> |
05:26 |
|
stozza joined #evergreen |
07:29 |
|
rjackson_isl joined #evergreen |
07:31 |
|
agoben joined #evergreen |
08:51 |
|
kmlussier joined #evergreen |
09:06 |
|
JBoyer joined #evergreen |
09:19 |
gmcharlt |
by the way, as a reminder and/or heads-up - I will be cutting the 2.5 beta release tomorrow in conjuction with Evergreen 2.12-beta |
09:28 |
kmlussier |
There are a few things I'm still working on with the release, so we still have a small window today where people can test/merge code if there is something they want to get into 2.12beta. |
09:30 |
kmlussier |
I'm planning to look again at bug 1541559, hopefully with some help from jeffdavis, but I'm at a stopping point right now because I'm getting a 500 response from the Overdrive server, which *may* be an issue on the OD side of things. |
09:30 |
pinesol_green |
Launchpad bug 1541559 in Evergreen "OneClickdigital API integration" [Wishlist,New] https://launchpad.net/bugs/1541559 - Assigned to Jeff Davis (jdavis-sitka) |
09:31 |
kmlussier |
I would love to get more eyes on that code. I know it's something a lot of libraries would love to see in Evergreen. |
09:32 |
kmlussier |
I also don't know how many people are working today. :) |
09:32 |
Dyrcona |
Unfortunately, I don't think I can test it. |
09:32 |
kmlussier |
Dyrcona: Are you doing your database upgrade today? |
09:32 |
Dyrcona |
Not exactly. |
09:32 |
Dyrcona |
We're upgrading the O/S packages, etc. |
09:33 |
Dyrcona |
The database upgrade is postponed for lack of free disk space. |
09:33 |
kmlussier |
:( |
09:33 |
Dyrcona |
I'm planning to take Friday as the holiday in compensation, which dovetails nicely with vacation next week. :) |
09:34 |
kmlussier |
I also want to make sure all of our tests are succeeding before gmcharlt cuts the release. I'm pretty sure I told phasefx at the hack-a-way that QA test failures would be showstoppers. :) |
09:35 |
kmlussier |
rhamby: Did you have a chance to pull together some bib records from a metarecord group for the Concerto data? |
09:35 |
rhamby |
rhamby: I got the bibs pulled together for three that will join the same metarecord and don't have $n $p to avoid that issue but I haven't had time to bundle it into a patch yet |
09:36 |
rhamby |
kmlussier: ^^ |
09:36 |
* rhamby |
needs a gallon of coffee still this am |
09:36 |
kmlussier |
@coffee rhamby |
09:36 |
rhamby |
apparently |
09:36 |
* pinesol_green |
brews and pours a cup of Guatemala La Conception, and sends it sliding down the bar to rhamby |
09:37 |
kmlussier |
The other feature I had hoped to get into 2.12, but won't have a chance to test today is bug 1612752. |
09:37 |
pinesol_green |
Launchpad bug 1612752 in Evergreen "Feature Request: Cancel Transits, Don't Delete Them" [Wishlist,Confirmed] https://launchpad.net/bugs/1612752 |
09:37 |
rhamby |
all Sunday dissolved into chaose as new foster dogs (puppies) arrived at the house |
09:37 |
kmlussier |
We can put that one off until the next release, but I put it out there in case anyone is looking for something to test. |
09:37 |
rhamby |
chaos even /sigh ... it's going to be one of those typing days |
09:37 |
kmlussier |
Ooh, puppies! |
09:39 |
gmcharlt |
Stay on target! </GoldFive> |
10:48 |
|
Christineb joined #evergreen |
10:52 |
|
stozza left #evergreen |
11:18 |
|
jeff_ joined #evergreen |
11:37 |
kenstir |
JBoyer: open-ils.storage.direct.config.sms_carrier.retrieve.all does not work, it does not fail but returns an empty payload. |
11:37 |
kenstir |
Looking at config.pm I do not see how it would be registered, so I also tried open-ils.storage.direct.config.copy_status.retrieve.all, which also returns an empty payload. Onto the next test. |
11:39 |
Dyrcona |
kenstir: You tried this where? |
11:40 |
kenstir |
gapines |
11:40 |
JBoyer |
Sorry about that, when Dyrcona said storage wouldn't work that's what he meant. Requesting it through pcrud is likely what you have to do. |
11:40 |
kenstir |
No problem, took me a while to get my testing infrastructure bootstrapped. Trying pcrud now |
11:40 |
Dyrcona |
Yeah, it won't work remotely, only directly on the server talking to the private osrf router. |
11:53 |
|
kmlussier joined #evergreen |
11:55 |
kenstir |
pcrud.search.csc.atomic is not working for me yet. I feel like I'm close but getting empty array as a payload. My query is http://gapines.org/osrf-gateway-v1?service=open-ils.pcrud&method=open-ils.pcrud.search.csc.atomic&param=%22auth_token%22&param=%7B%22active%22:true%7D |
14:42 |
JBoyer |
Thanks for the tips everyone! |
14:42 |
JBoyer |
kmlussier++ |
14:42 |
JBoyer |
dbwells++ |
14:51 |
JBoyer |
preliminary testing (i.e. a hand insert of a single row in metabib.author_field_entry) does show that that's enough to get this up and going. Hmm. |
14:53 |
JBoyer |
How to find the "best" auth record to pull from when the bib is indexed may be interesting. OR, that could just be done at the time that authority_control_linker.pl is run, as then there'll be a $0 and it's smooth sailing... |
14:54 |
kmlussier |
JBoyer: Yes, I was thinking it would use whatever is in subfield 0. |
14:55 |
JBoyer |
Which means that everything needed to turn this on happens in metabib.reindex_something_something. |
15:32 |
kmlussier |
OK, I want to look at this a little more closely, but since I have Overdrive connectivity working, I would like to merge the code from bug 1541559 today for inclusion in 2.12. |
15:32 |
pinesol_green |
Launchpad bug 1541559 in Evergreen "OneClickdigital API integration" [Wishlist,New] https://launchpad.net/bugs/1541559 - Assigned to Jeff Davis (jdavis-sitka) |
15:34 |
jeffdavis |
kmlussier: yay! |
15:34 |
kmlussier |
However, I also want to go on record as saying I have not tested OneClickDigital because our libraries do not have an subscription that I can use for testing. I just want to put that out there in case anyone has concerns about me merging the code without testing the code with the other vendor that is supported. |
15:35 |
kmlussier |
IIRC, I also signed off on EDI code way back in the day after only testing it with one vendor, so I think there might be precedent. |
15:38 |
jeffdavis |
I should note there is a minor issue with the OneClickdigital API: it does not currently work with non-numeric barcodes. This is an issue on their end, not Evergreen's. They assure me that this will be fixed with an update in early March. |
15:39 |
jeffdavis |
In the meantime, numeric barcodes worked fine with Oneclick for me, but it would be nice if someone else were able to test. |
15:58 |
jeffdavis |
One change in that branch that could definitely use some developer eyeballs is the new OpenILS::Utils::HTTPClient Perl module: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commitdiff;h=98eff96 |
15:58 |
jeffdavis |
It's a utility for sending HTTP requests to external servers and handling responses, basically a wrapper around LWP::UserAgent. |
16:25 |
|
Jillianne joined #evergreen |
16:32 |
bshum |
@dessert |
16:32 |
* pinesol_green |
grabs some Mint Chocolate Chip Ice Cream for bshum |
16:39 |
stephengwills |
i'll re-make the extras and try again. thanks Ben |
16:39 |
stephengwills |
hi Kathy :) |
16:39 |
* stephengwills |
picks a few keys out of his forehead and wades back in ... |
16:52 |
kmlussier |
OK - I'm wrapping up testing for bug 1541559 and will be working on the fix to the metarecord test. Is there anything else anyone is reviewing or wants to have reviewed for inclusion in 2.12? |
16:52 |
pinesol_green |
Launchpad bug 1541559 in Evergreen "OneClickdigital API integration" [Wishlist,New] https://launchpad.net/bugs/1541559 |
16:52 |
|
teletype01 joined #evergreen |
16:54 |
kmlussier |
Looks like gmcharlt is working on bug 1665933 for inclusion in tomorrow's release. |
16:54 |
pinesol_green |
Launchpad bug 1665933 in Evergreen "Ability to skip building staff client in make_release" [Wishlist,New] https://launchpad.net/bugs/1665933 |
16:54 |
kmlussier |
Also, if you want something to be reviewed, it should be something small. :) |
16:56 |
jeffdavis |
kmlussier++ |
16:58 |
kmlussier |
jeffdavis / gmcharlt: Regarding https://bugs.launchpad.net/evergreen/+bug/1541559/comments/13, in one of my earlier tests, I updated the code to https in two places. That code was in place during my first successful test. |
16:58 |
pinesol_green |
Launchpad bug 1541559 in Evergreen "OneClickdigital API integration" [Wishlist,New] |
16:59 |
kmlussier |
Is that all that needs to be done to get https by default? jeffdavis, would you have concerns about setting the default there? |
16:59 |
|
NawJo joined #evergreen |
16:59 |
kmlussier |
If it doesn't work in someone's environment, they could then make the choice to change it to http using the LSE? |
17:01 |
* kmlussier |
tries out the change to Overdrive.pm just to make sure she's remembering that it did indeed work. |
17:01 |
pinesol_green |
News from qatests: Test Failure <http://testing.evergreen-ils.org/~live> |
17:08 |
jeffdavis |
kmlussier: yes, changing OverDrive.pm so that the default values for circulation/discovery base URIs use HTTPS should be all that's necessary. Strictly speaking, it would be good to update the SQL changes as well since the description for those settings says "http" not "https". |
17:08 |
jeffdavis |
Want me to push a commit for that? |
17:08 |
kmlussier |
jeffdavis: Yes, please. |
17:56 |
jeffdavis |
HTTPS fix and CSS signoff pushed |
17:57 |
jeffdavis |
also updated the bug with details on the issues I had with HTTPS: https://bugs.launchpad.net/evergreen/+bug/1541559/comments/17 |
17:57 |
pinesol_green |
Launchpad bug 1541559 in Evergreen "OneClickdigital API integration" [Wishlist,New] |
18:22 |
kmlussier |
If anyone is still around, I could use a signoff to the fix for our recent test failure. bug 1665626 |
18:22 |
pinesol_green |
Launchpad bug 1665626 in Evergreen "Need more metarecord groups in sample dataset" [Undecided,New] https://launchpad.net/bugs/1665626 |
18:23 |
* kmlussier |
turns her attention to jeffdavis' branch and then gets ready to cut off merging for 2.12beta. |
18:25 |
Dyrcona |
Well, I'm setting a temporary replacement NFS server, so no time. |
18:26 |
Dyrcona |
I'll probably be back asking about filesystem tree structure, but I imagine I won't get answers until the morning. |
18:26 |
Dyrcona |
More than full... |
18:27 |
bshum |
kmlussier: Let me know if I can help. |
18:27 |
kmlussier |
bshum: Do you want to verify that the test succeeds and sign off on/merge it? |
18:28 |
bshum |
kmlussier: It'll take a few minutes for me to spin up a new system. But yeah I can check it over. |
18:28 |
bshum |
Point me at the bug / branch |
18:28 |
kmlussier |
bug 1665626 |
18:28 |
pinesol_green |
Launchpad bug 1665626 in Evergreen "Need more metarecord groups in sample dataset" [Undecided,New] https://launchpad.net/bugs/1665626 |
18:28 |
kmlussier |
You'll need the new sample data for the test to work. |
18:36 |
kmlussier |
ebook integration will have to wait until kmlussier eats dinner. |
18:58 |
kmlussier |
Calling 1027 and 1028 |
18:59 |
bshum |
I always have to look up how to run live tests :) |
18:59 |
bshum |
Rebuilding a fresh DB now and then running the tests shortly kmlussier |
19:00 |
kmlussier |
bshum++ |
19:05 |
* kmlussier |
is going to put off writing release notes for now since they need to be more extensive than 'we haz e-book statuses.' |
19:05 |
kmlussier |
That's all about all I could handle writing right now. |
19:07 |
bshum |
Hmm |
19:07 |
bshum |
It blew up on my first runthrough |
19:07 |
bshum |
Some of the other tests seemed unhappy |
19:08 |
bshum |
Going to rebuild the DB again, start up services and retest |
19:08 |
pinesol_green |
Showing latest 5 of 12 commits to Evergreen... |
19:08 |
pinesol_green |
[evergreen|Jeff Davis] LP#1541559: eliminate duplicate entries in ebook API transaction details - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=2729bbb> |
19:08 |
pinesol_green |
[evergreen|Jeff Davis] LP#1541559: improve display of ebook API transaction details in My Account - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=45b0f2a> |
19:08 |
pinesol_green |
[evergreen|Jeff Davis] LP#1541559: remove non-functional sort on ebook API transaction details in My Account - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=1d599f4> |
19:08 |
pinesol_green |
[evergreen|Kathy Lussier] LP#1541559: Minor tweaks to e-books circ in My Account - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=d3ada6d> |
19:08 |
pinesol_green |
[evergreen|Jeff Davis] LP#1541559: Use HTTPS for OverDrive requests - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=6d721a8> |
19:09 |
bshum |
Oh I see, we're testing the pgtap test. Not the perl live tests. |
19:09 |
* bshum |
can try that too |
19:10 |
bshum |
The perl live tests seem unhappier |
19:11 |
kmlussier |
Yes, sorry, it's the pgTap test that has been failing. |
19:11 |
bshum |
But maybe it's something I did wrong in my testing run |
19:11 |
bshum |
Or maybe it's the new bibs doing something |
19:11 |
kmlussier |
The perl live tests have been doing okay in the twice-daily test runs. |
01:19 |
pinesol_green |
Showing latest 5 of 11 commits to Evergreen... |
01:19 |
pinesol_green |
[evergreen|Bill Erickson] LP#1596595 Targeter accepts a list of hold ID's - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=dbcd6ec> |
01:19 |
pinesol_green |
[evergreen|Bill Erickson] LP#1596595 Targeter use child editor for settings - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=129a38b> |
01:19 |
pinesol_green |
[evergreen|Bill Erickson] LP#1596595 AOUS lookup batch by org id - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=85e73bc> |
01:19 |
pinesol_green |
[evergreen|Bill Erickson] LP#1596595 Targeter leverages batch AOUS lookups - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=895f8bd> |
01:19 |
pinesol_green |
[evergreen|Kathy Lussier] LP#1596595: Stamping upgrade scripts for hold targeter refactoring - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=eee584e> |
01:43 |
pinesol_green |
[evergreen|Jason Boyer] LP1517137: Add Permissions Missing From Stock Data - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=06e1f29> |
01:43 |
pinesol_green |
[evergreen|Kathy Lussier] LP#1517137: Release note entry for addition of missing permissions - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=7e590bd> |
01:43 |
pinesol_green |
[evergreen|Kathy Lussier] LP#1517137: Stamping upgrade script for adding overlooked permissions - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=b3a6cba> |
01:59 |
pinesol_green |
[evergreen|Josh Stompro] LP#1494748 - Change pay fines link to a button & increase checkbox sizes - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=c222f68> |
01:59 |
pinesol_green |
[evergreen|Kathy Lussier] LP#1494748: Decrease the input size on Firefox by a smidge - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=ced1e6c> |
01:59 |
pinesol_green |
[evergreen|Kathy Lussier] LP#1494748: Release notes entry for self check interface improvements - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=f9f830e> |
05:00 |
pinesol_green |
News from qatests: Test Failure <http://testing.evergreen-ils.org/~live> |
07:21 |
|
kmlussier joined #evergreen |
07:24 |
kmlussier |
Good morning #evergreen! Happy DIG hackaway day! |
07:25 |
|
agoben joined #evergreen |
07:25 |
kmlussier |
And happy 2.12 feature freeze day! |
07:25 |
* kmlussier |
sees more test failures and wonders if she can break Evergreen even more today. |
07:33 |
kmlussier |
rhamby: Following up on our conversation last night, I have filed bug 1665626 |
07:33 |
pinesol_green |
Launchpad bug 1665626 in Evergreen "Need more metarecord groups in sample dataset" [Undecided,New] https://launchpad.net/bugs/1665626 |
07:38 |
phasefx |
kmlussier: one option may be to create the test data needed as part of the test. Some pros and cons to that, but I've tried it before with some things |
07:40 |
rhamby |
kmlussier: sounds good. It'll be something I can do tomorrow morning when the house is asleep :) |
07:41 |
kmlussier |
phasefx: Yeah, that would work too, but, either way, we should have so grouped records in the test dataset. It makes it easier to test metarecord things. |
07:41 |
phasefx |
kmlussier: yeah, you're right |
07:42 |
kmlussier |
Of course, the nice thing about making it part of the test is that you don't have to worry about the test data moving on you. On the other hand, the person who wrote the test is on vacation right now. :) |
08:02 |
kmlussier |
I just ran the hold targeter test that failed. It failed for me too, but I got different errors, all of which related to the metarecord hold. |
08:10 |
kmlussier |
I think I know what the problem is there. It's targeting metarecord 42, but with the change in parting, metarecord 42 probably is the master of a totally new record now. |
08:10 |
* kmlussier |
sees if she can trace it down. |
08:11 |
|
Stompro joined #evergreen |
08:12 |
|
Dyrcona joined #evergreen |
08:18 |
Dyrcona |
What was it Helmuth von Moltke said? "No plan survives first contact with the enemy." |
08:32 |
pinesol_green |
Dyrcona: Your current monologue is at least 9 lines long. |
08:34 |
kmlussier |
My suspicion was wrong. The metarecord is pointing to the same bib in concerto now. |
08:38 |
Dyrcona |
Evergreen 2.9 is past its security release date. Should it be removed from the downloads page? |
08:38 |
kmlussier |
berick: Do you think you might have time today to look at 20-hold-targeter.t? This morning's test failed for hold 1 - http://testing.evergreen-ils.org/~live/test.24. |
08:38 |
kmlussier |
When I ran it on my VM, I didn't have trouble with that particular test, but I had errors with the test for hold 263. |
08:38 |
* kmlussier |
will past the output in a second. |
08:38 |
Dyrcona |
Also, I think the release date for 2.11 is wrong on the downloads page. |
08:39 |
kmlussier |
Dyrcona: yes. I noticed that yesterday but forgot to say something. |
08:39 |
kmlussier |
That is, I noticed that 2.9 was past the security date. |
08:41 |
Dyrcona |
Oh, yeah. Duh. |
08:42 |
Dyrcona |
I just noticed it was the same as 2.10 and thought that was wrong. |
08:42 |
Dyrcona |
Silly brain.... :) |
08:42 |
kmlussier |
berick: The output when I run that test is http://pastebin.com/DJr1L6z2 |
08:44 |
kmlussier |
That just leaves the web client build test. I'm not sure what's going on there. |
08:45 |
|
mmorgan joined #evergreen |
08:49 |
|
bos20k joined #evergreen |
09:15 |
|
afterl joined #evergreen |
09:15 |
kmlussier |
berick: Actually, ignore my output. Just realized I have the old hold targeter running in a cron job, which may have affected the test results? When I manually retargeted hold 263, I was left with 1 error. |
09:16 |
kmlussier |
It expected 22 mapped copies, but only got 21. 21 seems right to me since I only see 21 visible copies in the catalog. But I may be missing something. |
09:22 |
|
jvwoolf joined #evergreen |
09:23 |
gmcharlt |
oh, in case it wasn't obvious, I lift my hold on merging to rel_2_11 and rel_2_10 |
09:25 |
kmlussier |
:) |
09:28 |
kmlussier |
Starting my morning off right. Testing code that hasn't actually been loaded on the server I'm using. |
09:28 |
kmlussier |
@coffee |
09:28 |
* pinesol_green |
brews and pours a cup of La Esperanza Colombia Huila, and sends it sliding down the bar to kmlussier |
09:29 |
dbs |
kmlussier: that's your control group, right? :) |
09:29 |
gmcharlt |
@coffee kmlussier |
09:33 |
gmcharlt |
(I'm dealing with updating milestones in LP, hence my question) |
09:34 |
kmlussier |
Yes, looks like it. |
09:40 |
|
yboston joined #evergreen |
09:57 |
berick |
kmlussier: hey just to confirm, you are testing on a new concerto data set? |
09:58 |
berick |
kmlussier++ merging the midnight oil |
10:00 |
kmlussier |
berick: The only new thing I know of that has changed in the concerto data set is that some grouped records are no longer living in the same metarecord group. But it doesn't appear to affect the records in your tests. |
10:00 |
berick |
kmlussier: there's a concerto change in one of the hold targeter commits |
10:00 |
berick |
it adds a metarecord hold |
10:01 |
kmlussier |
berick: Yes, that's there. |
10:01 |
berick |
ok |
10:01 |
berick |
possible some parallel changes occurred that affect the test. kmlussier, I can revisit in a bit, after I deploy some security patches |
10:02 |
kmlussier |
berick: Great, thanks! Yes, secure your system first. :) |
10:02 |
kmlussier |
berick++ |
10:21 |
kmlussier |
Calling 1022 and 1023 |
10:38 |
kmlussier |
If somebody else doesn't beat me to it, that is. |
10:39 |
kmlussier |
Would anyone be willing to look at bug 1661747 for JBoyer today? |
10:39 |
pinesol_green |
Launchpad bug 1661747 in Evergreen "Add get_org_unit_ancestor_at_depth to action trigger reactor helpers" [Wishlist,New] https://launchpad.net/bugs/1661747 |
10:40 |
kmlussier |
Dyrcona: Did you ever get any sample ISBN's to test the czech added content module? |
10:41 |
Dyrcona |
kmlussier: Better than that. They sent me some MARC records, that I admittedly have not looked at, yet. |
10:41 |
Dyrcona |
I've been doing C/W MARS stuff this morning. |
10:41 |
kmlussier |
Dyrcona: Do you think you'll have a chance to look at it today? If not, maybe you can send them my way and I can give it a try. |
10:58 |
Dyrcona |
Your branch and 'origin/user/jkotrla/lp1624366-added_content_obalkyknih_2' have diverged, and have 102 and 2 different commits each, respectively |
10:58 |
Dyrcona |
I'm the king of typos. |
10:59 |
Dyrcona |
Maybe I meant an em dash? |
10:59 |
Dyrcona |
Working is my origin on my test vms for obvious reasons. |
10:59 |
Dyrcona |
So guess I'll install again. |
11:02 |
Dyrcona |
I just heard from Eva and it's OK to add the records that she sent to Concerto, so I'll add them with this branch when I push it later. |
11:03 |
* Dyrcona |
is confident the testing will be a success. |
11:03 |
Dyrcona |
Should I add the documentation as a release note, or will someone else take care of that? |
11:03 |
|
Christineb joined #evergreen |
11:05 |
kmlussier |
Dyrcona: Jane was going to take care of the documentation. |
11:34 |
kmlussier |
Ooh! Something isn't looking good in the web client patron interface. I don't know if it's in master or is one of the branches I loaded. |
11:35 |
kmlussier |
Dyrcona: Did you say you just built the web client? |
11:36 |
Dyrcona |
Yeah, after rebasing the obalky branch and installing. I build the web client and xul client. |
11:36 |
Dyrcona |
I've been using the web client in a lot of my testing on my vm lately. |
11:36 |
gmcharlt |
Dyrcona: the branch is a little dusty (*cough* *hack*), but is there anything you still have out there would would benefit from mergin 939535? |
11:37 |
kmlussier |
I think the problems I'm seeing are related to a branch I haven't merged yet. |
11:37 |
Dyrcona |
gmcharlt: Not at the moment, and I forgot that I even did that. |
11:41 |
kmlussier |
Webby looks fine too, so I'll just focus on making sure I don't merge whatever broke my checkout screen. |
11:41 |
Dyrcona |
Maybe I'll check it out and take it for a spin. I'm probably more amenable to berick's suggestion today. |
11:42 |
gmcharlt |
speaking of OpenSRF 2.5-beta - I'll realize that simultaneously with the EG 2.12-beta release |
11:43 |
berick |
kmlussier: fyi, looking at hold tests. data has changed in a variety of ways. unclear how, since it was OK a few days ago. metarecord 42 picked up a record. bib 45 only has 21 in holdable locations, before it has 22. also, if I run the fill test suite, patrons pick up penatles that aren't there when running in isolation. (also seeing neg. balance test failures, not sure if that has any effect). |
11:43 |
berick |
more to follow.. |
11:43 |
Dyrcona |
I'll have to see if I can find my client code that used it. I probably do not still have it hanging around. |
11:44 |
kmlussier |
OK, I haven't seen any of the negative balance test failures. I ran the full suite of pgtap tests yesterday. |
11:44 |
Dyrcona |
Unless it's a branch at mvlc. |
11:44 |
berick |
kmlussier: perl live tests |
11:44 |
Dyrcona |
pgtap tests should not be a problem, IIRC. |
11:44 |
Dyrcona |
perl live tests mostly can only run once, then you have to reload the db. |
11:44 |
|
mmorgan joined #evergreen |
11:45 |
Dyrcona |
been a while since I ran all of the pgtap tests, though. |
11:45 |
Dyrcona |
that may have changed. |
11:45 |
kmlussier |
berick: Ah, ok. For some reason, I was thinking they were pgtap. But I didn't see any hiccups with negative balance tests in this morning's test results. |
11:45 |
berick |
it occurs to me if we don't enforce a leave-it-like-you-found-it policy on perl live tests, writing new tests is going to be painstakingly slow process |
11:46 |
berick |
rebuild db, run all tests, new test fails, start over, ... |
11:46 |
gmcharlt |
agreed |
11:46 |
Dyrcona |
But, I had a test that worked, then started failing after some other data/code was changed. IIRC, someone else ended up fixing it. |
11:46 |
Dyrcona |
Anyway, back to what I was doing. |
11:49 |
berick |
kmlussier: it's possible (nay, likely) my fines test failure was a result of running the live tests multiple times |
11:50 |
berick |
kmlussier: will confirm shortly |
11:52 |
kmlussier |
berick: As far as the changes in the metarecords, I don't have a good handle on how the sample data is built, but I wonder if my code that broke up the existing metarecord groups somehow threw off how the holdings were applied to the records. |
11:53 |
dbwells |
berick: when we wrote the negative balance tests, we also included a neg_bal_testing_reset.sql which might do what you need. |
11:57 |
dbwells |
berick: I think the main reason that doesn't just happen is that its awfully helpful to actually see the resulting test-generated data when it fails. I agree we should think about a better system or convention for that. |
11:58 |
berick |
dbwells: thanks |
11:59 |
Dyrcona |
Well, I think it would be hard to leave it like you found it with Perl tests that are doing backend calls. There's no undo mechanism for that, and a lot of things can't be deleted or undone, really. |
11:59 |
Dyrcona |
Bills could be paid, etc., but don't think they can be removed without gymnastics. :) |
12:00 |
dbwells |
yeah, the "reset" just deletes data above a certain id in certain tables. It isn't exactly future- or fool-proof by any stretch. |
12:08 |
kmlussier |
bug 1661754 seems more like a bug fix than new feature to me. Does anyone object to my backporting it? |
12:08 |
pinesol_green |
Launchpad bug 1661754 in Evergreen "Staff users should be prevented from marking a Long Overdue item Lost" [Undecided,Confirmed] https://launchpad.net/bugs/1661754 - Assigned to Kathy Lussier (klussier) |
12:11 |
berick |
dbwells: Dyrcona: yeah, i don't have any big plans. just some light griping. |
12:11 |
berick |
i'm going to avoi the issue for now by testing data that other scripts don't touch |
12:13 |
|
NawJo joined #evergreen |
12:18 |
miker |
kmlussier: I tend to agree. and I think there are policy decisions being made by the code that might not work for existing use cases (I have no hard examples, though) |
12:19 |
berick |
yeehaw, all tests pass now |
12:20 |
kmlussier |
berick++ |
12:22 |
berick |
kmlussier: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1596595-hold-target-tests-update -- i took the liberty of tagging it w/ the same hold targeter LP # |
12:22 |
kmlussier |
miker: Thanks. I think there is general agreement that we would like to eventually see a full-featured resolution with bug 1562061 |
12:22 |
pinesol_green |
Launchpad bug 1562061 in Evergreen "Marking a Long Overdue transaction Lost adds a second bill to the patron record" [Undecided,New] https://launchpad.net/bugs/1562061 |
12:22 |
berick |
if you want a new LP, i can add one |
12:23 |
kmlussier |
berick: No, I think that's fine. I'll look at that one before I move on to NawJo's and bshum's rtl-support branch. |
12:23 |
berick |
cool, thanks |
12:27 |
|
jihpringle joined #evergreen |
12:28 |
kmlussier |
Actually, I think I need to test this on a clean database. |
12:31 |
pinesol_green |
[evergreen|Galen Charlton] LP#1662902: do not re-download EDI files that failed processing - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=3573009> |
12:31 |
pinesol_green |
[evergreen|Bill Erickson] LP#1662902: do not re-download EDI files that failed parsing - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=82732c5> |
12:33 |
jeff |
"I think that's what he says, but I need to hear it on a Maxell." |
12:55 |
pinesol_green |
[evergreen|Jason Boyer] Add Release Note for new helper - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=cb37d04> |
12:58 |
bshum |
NawJo++ |
13:00 |
NawJo |
:) |
13:01 |
kmlussier |
LOL - https://twitter.com/gmcharlt/status/832621006191669248 |
13:04 |
kmlussier |
berick: All tests successful. Huzzah! |
13:10 |
berick |
@bartender #evergreen |
13:10 |
* pinesol_green |
fills a pint glass with Hitachino Nest Japanese Classic Ale, and sends it sliding down the bar to #evergreen (http://beeradvocate.com/beer/profile/697/16429) |
13:11 |
pinesol_green |
[evergreen|Bill Erickson] LP#1596595 Hold targeter Perl live test repairs - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=86cce7f> |
13:11 |
Dyrcona |
So, stock concerto, should I have a record match set option in vandelay? |
13:11 |
Dyrcona |
Do I need to make one? |
13:12 |
kmlussier |
Dyrcona: I usually don't set one when I'm doing imports in testing. |
13:12 |
kmlussier |
Especially if I know they aren't in the stock data. I just select the option to import new records, and, if a matchpoint isn't set, it sees all the records as new. |
13:13 |
Dyrcona |
Well it says the value enterd is invalid, but I haven't tried importing. |
13:13 |
kmlussier |
It should work. I do it all the time. |
13:14 |
Dyrcona |
OK. thanks. i'll give it a whirl. |
13:18 |
Dyrcona |
jabok_library++ |
13:19 |
Dyrcona |
Even has a URL: Digitalizovaný dokument |
13:19 |
kmlussier |
Huzzah again! |
13:20 |
Dyrcona |
So, I'll figure out how to add these to concerto after I do some more testing. |
13:21 |
Dyrcona |
Think I need to add copies and/or target the urls for opac testing, though. |
13:21 |
|
frank__ joined #evergreen |
13:22 |
kmlussier |
Calling 1024 and 1025 |
13:22 |
* Dyrcona |
thinks Prague in the Spring would make a nice setting for a Evergreen conference. |
15:22 |
kmlussier |
jeffdavis: Should that bug have a pullrequest tag? |
15:25 |
jeffdavis |
I'm not sure. There are a couple of minor issues I'm aware of and working on. |
15:25 |
jeffdavis |
1. Better CSS on detailed checkout/hold view in My Account, (2) occasional duplicate entries in the same place. |
15:28 |
kmlussier |
jeffdavis: OK, if I get it working, I'll see how polished it looks from my perspective. :) |
15:28 |
* kmlussier |
needs to steal some test records from NOBLE first. |
15:42 |
* Dyrcona |
counts backwards with Throwing Muses. |
15:46 |
Dyrcona |
I should have squashed those two commits into one... |
15:46 |
Dyrcona |
oh well. |
16:35 |
jeffdavis |
one sec |
16:35 |
jeffdavis |
overdrive has a ton of different values that you need, it's very overcomplicated |
16:36 |
kmlussier |
jeffdavis: Circulation API, Discovery API, Granted Authorization Redirect - are those things I need to enter now with the current functionality we have? |
16:38 |
jeffdavis |
Circulation API and Discovery API base URIs will default to OverDrive's production API. If you're using that API you don't need to touch those settings. If you want to use their test ("integration") API instead, you would need to add values for those settings. |
16:39 |
jeffdavis |
(There's probably a better way to handle that but I was trying to avoid hardcoding URIs that might change with little notice.) |
16:39 |
jeffdavis |
the Granted Auth one is not required for right now, that piece is not functional yet |
16:39 |
kmlussier |
sure |
16:39 |
jeffdavis |
as for the other settings... |
16:40 |
jeffdavis |
OverDrive will provide you with a client key and secret. You combine those and base-64 encode the result (following the instructions they provided) to get the value for the "basic token" setting. |
16:56 |
jeffdavis |
which would imply the need for authorization names for individual libraries |
16:58 |
|
Christineb joined #evergreen |
16:58 |
kmlussier |
Yeah, I don't think we're doing Advantage titles. |
17:00 |
pinesol_green |
News from qatests: Test Failure <http://testing.evergreen-ils.org/~live> |
17:01 |
* kmlussier |
has to finish up for the day. I'll take another look at this over the weekend. |
17:01 |
kmlussier |
Oh good! The only test failure is the one rhamby and I have a plan for. |
17:02 |
kmlussier |
Thanks to everyone for their help with the release today! Have a nice weekend! |
17:02 |
jeffdavis |
kmlussier++ |
17:02 |
bshum |
kmlussier++ |
17:07 |
|
jvwoolf left #evergreen |
01:37 |
* dbs |
wonders what madness is suddenly causing a request for / to redirect to /eg/opac/home and then to /eg/opac/login?redirect_to=%2Feg%2Fopac%2Flogin%3Fredirect_to%3D%252Feg%252Fopac%252Flogin%253Fredirect_to... |
01:49 |
dbs |
ahh. if you happen to create an /openils/var/web/eg directory, you get... much pain |
05:01 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
07:14 |
|
rjackson_isl joined #evergreen |
07:23 |
|
agoben joined #evergreen |
08:12 |
jeff |
dbs: yep, i think jeffdavis and i ran into that a while back. |
10:29 |
|
jvwoolf joined #evergreen |
10:29 |
kmlussier |
JBoyer: Yes, I'm about to head into a meeting too. I'll check back later. |
10:31 |
|
mmorgan1 joined #evergreen |
10:45 |
JBoyer |
Meeting over. I'll fix it. My testing method was just lacking (add perm at >1000, run script, woo.) This time I'll assign it to a few groups to make sure the right thing is done. |
11:03 |
|
Christineb joined #evergreen |
11:15 |
JBoyer |
kmlussier, Given the way the script is written I'm rather confused how it would keep the >1000 permissions around. Did it not insert the <1000 versions? (they can't both exist in the table simultaneously) |
11:15 |
JBoyer |
I will say if you've inserted anything in the id = 564 to 589 range those will be "skipped" under the assumption you needed it before running the script and have dealt with everything yourself. |
14:54 |
kmlussier |
That's it. |
14:54 |
graced |
#topic Release Manager's Report |
14:54 |
graced |
kmlussier? |
14:54 |
kmlussier |
The 2.12 feature freeze deadline is tomorrow. We've been busy this week testing new features to be included in the release. |
14:55 |
kmlussier |
On the agenda, I highlighted some of the new features that have already made it into 2.12. |
14:55 |
kmlussier |
gmcharlt will be cutting the beta release next week. It should be available on Wednesday. |
14:56 |
kmlussier |
Also, the web client seems to be making some nice progress for 2.12. |
14:56 |
kmlussier |
That's all I have. |
14:56 |
graced |
kmlussier++ |
14:57 |
tspindler |
kmlussier++ |
14:57 |
collum |
kmlussier++ |
15:14 |
berick |
kmlussier: i could go either way on squashing. The bulk was already squashed. some commits makes sense as-is. other new commits each address bugs in the LP, so keeping them separate is not a bad thing, imo. if squashing is desired, though, i'm happy to. othwerwise, I say go for it. |
15:15 |
kmlussier |
berick: I had no strong opinions, so I'll go for it. Thanks! |
15:15 |
kmlussier |
berick++ |
15:16 |
berick |
miker: my sincere hope is future hatches can work with older EG's. so far through recent testing, the API has been stable. i expect we'll need some hatch-specific versioning, though. |
15:16 |
kmlussier |
I'm thinking at release time, we probably should put the hatch installer on the downloads page? |
15:16 |
berick |
kmlussier: makes sense. |
15:16 |
berick |
kmlussier: are you planning to also merge the Hatch branch into Hatch master? I can do that if you're not |
16:35 |
gmcharlt |
similarly, https://evergreen-ils.org/egdownloads/ is now updated |
16:37 |
kmlussier |
gmcharlt++ dbwells++ |
16:53 |
gmcharlt |
https://evergreen-ils.org/security-releases-opensrf-2-4-2-and-2-5-0-alpha2-evergreen-2-10-10-and-evergreen-2-11-3/ |
16:57 |
pinesol_green |
[opensrf|Mike Rylander] LP#1652382: normalization of memcache keys in C code - <http://git.evergreen-ils.org/?p=OpenSRF.git;a=commit;h=dbc54c6> |
16:57 |
pinesol_green |
[opensrf|Mike Rylander] LP#1652382: Make use of the clean key just created - <http://git.evergreen-ils.org/?p=OpenSRF.git;a=commit;h=3277648> |
16:57 |
pinesol_green |
[opensrf|Galen Charlton] LP#1652382: handle cases where supplied key is longer than 250 bytes - <http://git.evergreen-ils.org/?p=OpenSRF.git;a=commit;h=7524221> |
16:57 |
pinesol_green |
[opensrf|Galen Charlton] LP#1652382: more improvements to cache key munging - <http://git.evergreen-ils.org/?p=OpenSRF.git;a=commit;h=cbd252b> |
16:58 |
|
khuckins__ joined #evergreen |
17:01 |
pinesol_green |
[evergreen|Chris Sharp] LP#16663435 - Stripe org settings lack view permissions. - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=e664df4> |
17:01 |
pinesol_green |
[evergreen|Jason Stephenson] LP#16663435 - Release Note for Missing Stripe Settings Permissions - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=a02bcf5> |
17:01 |
pinesol_green |
[evergreen|Galen Charlton] LP#16663435: stamp database update - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=c9af31b> |
17:02 |
pinesol_green |
News from qatests: Test Failure <http://testing.evergreen-ils.org/~live> |
17:03 |
kmlussier |
uh oh |
17:03 |
kmlussier |
That's must be from the code I merged last night. |
17:03 |
pinesol_green |
[evergreen|Galen Charlton] release notes for 2.10.10 - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=9af4ae7> |
17:04 |
kmlussier |
But the test passed this morning. hmmmm |
17:07 |
|
mmorgan left #evergreen |
17:15 |
|
khuckins_ joined #evergreen |
17:37 |
jeffdavis |
updated bug 1541559 with a branch for review and some basic setup instructions |
18:54 |
|
Jillianne joined #evergreen |
19:09 |
|
kmlussier joined #evergreen |
19:36 |
Dyrcona |
Oh. I'm still signed in... I forgot. |
20:39 |
* kmlussier |
thinks she knows why the test failed today. |
20:44 |
kmlussier |
When I was working on bug 1553287, I noticed that the addition of parts would break up all of our metarecord groups in the test data. |
20:44 |
pinesol_green |
Launchpad bug 1553287 in Evergreen "Incorporating part information into biblio fingerprint" [Wishlist,Fix committed] https://launchpad.net/bugs/1553287 |
20:44 |
kmlussier |
And the test relies on 3 records being part of the same metarecord group. |
20:46 |
kmlussier |
So we need new sample records that can be grouped together and then point the test to those records. |
20:50 |
kmlussier |
Maybe we can get a few Harry Potter and the Sorcerer's Stone records to add to the test data. They have the added benefit of being easy to remember. |
21:04 |
jeff |
sounds reasonable. :-) |
21:16 |
rhamby |
kmlussier: I vote for at least one lovecraft collection to be added |
21:17 |
kmlussier |
rhamby: Sure! You got the MARC records? |
05:01 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
06:22 |
|
JBoyer_alt joined #evergreen |
06:56 |
csharp |
95ce9261 |
06:56 |
pinesol_green |
csharp: [evergreen|miker] add view and change permission slots to config.org_unit_setting_type, for controlling access to specific ou settings - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=95ce926> |
10:22 |
Thad |
Please suck in the full view of that issue, especially WHY it was initally created with the 1st comment. And then provide your feedback on it. |
10:29 |
gmcharlt |
I'd like to request review of the PR for bug 1639236, as I'd like to get that fix in for 2.11.3 today |
10:29 |
pinesol_green |
Launchpad bug 1639236 in Evergreen 2.11 "Temporary List Display Broken in 2.11/Master" [Medium,Confirmed] https://launchpad.net/bugs/1639236 |
10:29 |
csharp |
gmcharlt: I'll test it |
10:29 |
gmcharlt |
csharp: thanks! |
10:30 |
csharp |
that issue has been really bugging us (no pun intended) since the upgrade - I hate it when we lose features we had before :-/ |
10:31 |
* bshum |
thinks about https://bugs.launchpad.net/evergreen/+bug/1647819 too, which he tested before finding out about the other bug |
10:31 |
pinesol_green |
Launchpad bug 1647819 in Evergreen "Typo in anon_list.tt2" [Low,New] |
10:32 |
csharp |
yeah - there's a copy-paste-o at work there |
10:33 |
pinesol_green |
[evergreen|Chris Sharp] LP#1647819 - Missing closing angle bracket in anon_list.tt2 - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=5160e5f> |
10:59 |
|
Christineb joined #evergreen |
11:13 |
kmlussier |
@coin |
11:13 |
pinesol_green |
kmlussier: heads |
11:13 |
kmlussier |
Looks like I'll be testing hatch this morning. :) |
11:13 |
kmlussier |
Errrr...more likely to be afternoon. |
11:16 |
|
bmills joined #evergreen |
11:18 |
berick |
heh, watched the futurama episode recently where the key difference between 2 parallel universes was that a coin toss in one was always the opposite of the other. |
11:18 |
berick |
so, rest assured kmlussier, in a parallel universe, you are testing something else. |
11:19 |
berick |
who needs clones in the multiverse? |
11:20 |
Dyrcona |
Every time you flip a coin, a new universe is created. |
11:21 |
* mmorgan |
's head is starting to hurt. |
11:21 |
berick |
mmorgan: re: bug 1661754, i wonder if there are legit cases for marking a long-overdue item as lost.. (maybe that's in the IRC chatter -- I haven't read it) |
12:09 |
|
brahmina joined #evergreen |
12:20 |
|
bmills joined #evergreen |
12:30 |
|
kmlussier2 joined #evergreen |
12:32 |
kmlussier2 |
berick: I ran the new Hatch installer & updated print.js with the changes from your last commit. Still having trouble printing if I change any of the printer options. If I use default settings, I can print the Test page. |
12:33 |
kmlussier2 |
I'll paste output in a second. Having trouble connecting to pastebin |
12:35 |
kmlussier2 |
berick: http://pastebin.com/eCA0RLaV |
12:36 |
* kmlussier2 |
heads off to lunch before testing more |
12:38 |
berick |
kmlussier2: *sigh* same bug, different line of code. pushing a fix. |
12:42 |
berick |
kmlussier2: fix and updated .exe posted to LP |
12:42 |
berick |
(and enjoy your lunch!) |
12:55 |
Dyrcona |
ha |
12:56 |
JBoyer |
And a hell of a thing to debug if it doesn't already exist, heh. |
12:56 |
kmlussier2 |
lol |
12:56 |
Dyrcona |
I plan on deleting the image when my test is done. |
12:56 |
JBoyer |
That also saves a lot of space :D |
12:56 |
Dyrcona |
:) |
12:57 |
Dyrcona |
I made it so it could go up to 600 GB. I think that's enough to load our production data. |
12:58 |
Dyrcona |
On a Pg-related note, does "no action" on delete on a fkey contraint prevent the delete from happening, or does it leave the junk id in the table? |
12:59 |
* Dyrcona |
should just RTFM. :) |
12:59 |
JBoyer |
I would guess it prevents a real delete anywhere else. Unless you loosen the FK. |
13:00 |
JBoyer |
Also, +1 to testing on "real" databases. |
13:00 |
Dyrcona |
Yeah, that's what I think, but I'll look it up. Someone wants to delete a patron group used by some hold matrix entries. |
13:00 |
Dyrcona |
+elebenty! |
13:02 |
Dyrcona |
JBoyer++ |
17:00 |
Dyrcona |
Oh.... you have say apache2.service... because Lennaert is what Linus called him. :) |
17:01 |
* Dyrcona |
can't wait to get home to the sanctity of his BSD castle. :) |
17:01 |
Dyrcona |
And, it's time to go! |
17:01 |
pinesol_green |
News from qatests: Test Success <http://testing.evergreen-ils.org/~live> |
17:05 |
|
frank_guel joined #evergreen |
17:06 |
frank_guel |
Hi all, how are you today, I am from Mexico, I am triying to upgrade from the 2.8.4v to the 2.11.2 v, but I can´t find the 2.11.1-2.11.2-upgrade-db.sql file, Does it exist? |
17:06 |
frank_guel |
sorry. the 2.11.0-2.11.1-upgrade-db.sql |