| Time |
Nick |
Message |
| 00:12 |
|
stozza joined #evergreen |
| 00:12 |
|
stozza joined #evergreen |
| 01:06 |
|
stozza joined #evergreen |
| 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:38 |
|
Dyrcona joined #evergreen |
| 08:40 |
|
jlundgren 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 |
rhamby: Are you talking to yourself again? ;) |
| 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> |
| 09:40 |
rhamby |
PUPPIES! |
| 09:40 |
rhamby |
Anyway, I'll try to steal some time today to shove them into a patch and update the launchpad ticket. |
| 09:41 |
Dyrcona |
Oooh, Puppies in Evergreen. :) |
| 09:41 |
kmlussier |
rhamby: Great! If you find you're unable to steal the time, feel free to send the MARC records my way. |
| 09:41 |
kmlussier |
It's about time! There have been way too many cats in Evergreen up until now. We need balance. |
| 09:42 |
Dyrcona |
Use the Force, Luke. |
| 09:42 |
Dyrcona |
Now, I have to play some old (really? 1996 is old?) Jamiroquai. |
| 09:42 |
rhamby |
kmlussier: I'm trying to get a bunch of small projects done before noon so hopefully after lunch I'll have time. |
| 09:43 |
kmlussier |
OK. I'm going to wander away briefly, but will return soon when jeffdavis is more likely to be around. |
| 09:43 |
* Dyrcona |
waits for the all clear to commence with the updates. |
| 09:44 |
Dyrcona |
And possibly rebuilding some VMs. I have new versions of OpenSRF and Evergreen (customization updates), I might as well build new VMS. |
| 09:58 |
JBoyer |
Dyrcona, re: disk space, does that mean that you are able to do a vacuum full today to prepare for the upgrade, or do you physically need more space on the partition? |
| 09:59 |
JBoyer |
I need to find a good 8+ hour stretch to do a v full on our system sometime. Still around 100G of fat to trim. :/ |
| 09:59 |
Dyrcona |
Yes, and I'll know after the vacuum full. |
| 10:00 |
JBoyer |
Good luck! |
| 10:00 |
Dyrcona |
The replication server may be unpossible to upgrade in the normal way. I may have to delete the data and let the replication fill it from scratch. |
| 10:01 |
Dyrcona |
It has even less disk space on the db partion. |
| 10:01 |
miker |
just tossing this our here, apropos of nothing: http://reorg.github.io/pg_repack/ |
| 10:01 |
* miker |
runs off to a meeting |
| 10:01 |
Dyrcona |
Oh, and the db server gets a reboot. It's been up for 547 days! :) |
| 10:01 |
Dyrcona |
miker: Thanks, I'll look at that. |
| 10:02 |
|
kenstir joined #evergreen |
| 10:05 |
JBoyer |
"pg_repack cannot reorganize tables using GiST indexes." Bummer. |
| 10:05 |
Dyrcona |
GIN is better, right? |
| 10:06 |
JBoyer |
Yes, but bigger (at least until some version of Pg that neither of us are on. 9.4 or 9.5?) |
| 10:06 |
Dyrcona |
That's another thing, I noticed one table with the same indexed defined both as gin and gist. I should probably drop the gist index, and search for other dupes. |
| 10:06 |
Dyrcona |
I was gonna upgrade to 9.5. :) |
| 10:06 |
JBoyer |
That's a decent way to get some space back |
| 10:06 |
JBoyer |
Ah, I thought I saw you say you were going to be sticking to 9.2 for a while longer. That should help. |
| 10:07 |
Dyrcona |
Well, we will stay on 9.2 until we figure out when/how to upgrade the database. |
| 10:09 |
kenstir |
Happy presidents day, ever-greeners! |
| 10:12 |
|
stozza joined #evergreen |
| 10:12 |
kenstir |
Any clues how I should go about fetching the list of SMS carriers (via osrf)? I think I can get the library setting and account prefs. I'm trolling the source code... |
| 10:15 |
|
stephengwills joined #evergreen |
| 10:17 |
Dyrcona |
kenstir: I don't think you can get the list of SMS carriers via opensrf. |
| 10:17 |
Dyrcona |
I'm not aware of any calls for that. |
| 10:17 |
JBoyer |
Not with one of the storage calls? |
| 10:18 |
JBoyer |
open-ils.storage.direct.config.sms_carrier.retrieve *looks* like it would work, though I've not tried it. |
| 10:18 |
JBoyer |
actually ....retrieve.all would be better. |
| 10:19 |
Dyrcona |
Well, you can't do storage calls from the public router. |
| 10:19 |
Dyrcona |
At least, not if Evergreen is configured properly. |
| 10:20 |
JBoyer |
Ah. I didn't know if that was "not at all" or "not unless you're logged in" |
| 10:20 |
Dyrcona |
I assume kenstir wants to retrieve them for Hemlock. |
| 10:20 |
kenstir |
Dyrcona: yes that's right |
| 10:20 |
Dyrcona |
I say, open a Lp to request the interfaces and it will make it in by 3.0 or so-ish. |
| 10:20 |
JBoyer |
I don't suppose pcrud can reach it? |
| 10:20 |
Dyrcona |
Maybe pcrud can. I never tried. |
| 10:21 |
Dyrcona |
I'm a bit busy to try right now. :) |
| 10:21 |
JBoyer |
Oh, sure, I'm throwing ideas out for kenstir as much as asking what sounds reasonable. |
| 10:22 |
kenstir |
JBoyer: much appreciated. |
| 10:22 |
Dyrcona |
I know, but I like to try things. |
| 10:23 |
JBoyer |
kenstir, you can use pcrud to access the config.sms_carrier table, it's the csc class. (HOW to do that I'll leave up to you, but there are pcrud calls all over the place to look at) |
| 10:23 |
kenstir |
Dyrcona: you must be busy with the upgrade? |
| 10:23 |
Dyrcona |
pcrud can only reach it if there is a pcrud controller in the IDL, so that's one way to check. |
| 10:23 |
kenstir |
I will try it thanks |
| 10:23 |
Dyrcona |
kenstir: Yes. |
| 10:23 |
kenstir |
Dyrcona: good luck |
| 10:24 |
Dyrcona |
Thanks. rebooted the db server and started the vaccum full analyze in a screen. |
| 10:24 |
Dyrcona |
I'm moving on to the bricks. |
| 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 |
| 12:00 |
dbwells |
kenstir: I think the bareword 'true' is going to trip it up. Try either '1' or '"t"' (or probably any other truthy value). |
| 12:01 |
dbwells |
1 seems to work alright for me. |
| 12:01 |
kenstir |
I'm playing blind man's bingo here but I hit on something that works: param="token"¶m={"id":{"!=":null}} |
| 12:02 |
kenstir |
found that in sms_carrier.js |
| 12:04 |
dbwells |
kenstir: Well, I suppose it depends on if you actually want to remove in the inactive entries or now :) |
| 12:05 |
|
jeff__ joined #evergreen |
| 12:05 |
dbwells |
Does this not work for you? http://gapines.org/osrf-gateway-v1?service=open-ils.pcrud&method=open-ils.pcrud.search.csc.atomic&param="AUTH_TOKEN"¶m={"active":1} |
| 12:06 |
dbwells |
s/or now/or not/ # sorry |
| 12:08 |
|
jlundgren joined #evergreen |
| 12:09 |
|
dcook joined #evergreen |
| 12:12 |
dbwells |
oh man, more typos. For the record, let me try again. "I suppose it depends on if you actually want to remove the inactive entries or not :)" Ok, I feel better now. |
| 12:24 |
|
jihpringle joined #evergreen |
| 12:27 |
|
brahmina joined #evergreen |
| 12:29 |
|
rfrasur joined #evergreen |
| 12:29 |
rfrasur |
graced: you around? |
| 12:29 |
graced |
rfrasur: Where else would I be? |
| 12:30 |
rfrasur |
asquare? I dunno. |
| 12:30 |
rfrasur |
Is there a preference for survey monkey over Google forms? |
| 12:31 |
graced |
rfrasur: I don't think so. Google forms was still a bit wonky until recently but I think it'd be fine now. |
| 12:32 |
rfrasur |
Okay, I'm more confident with it, but didn't want to go ahead if there was some reason I didn't know about that'd make it unusable. |
| 12:33 |
graced |
rfrasur: No, no reason. I think it should be fine for our needs. |
| 12:33 |
graced |
Thanks! |
| 12:33 |
rfrasur |
Awesome. I'll get that set up next and get it sent out this afternoon. |
| 12:35 |
graced |
rfrasur++ |
| 12:36 |
rfrasur |
lol, the one thing I get done on time. thank you though. |
| 12:59 |
kenstir |
dbwells: that *did* work for me, thanks! |
| 12:59 |
kenstir |
dbwells+ |
| 12:59 |
kenstir |
dbwells++ even |
| 13:09 |
|
kmlussier joined #evergreen |
| 13:09 |
|
kmlussier joined #evergreen |
| 13:10 |
kmlussier |
I hate when I've been typing long conversations into IRC only to find out that I was disconnected the whole time. :( |
| 13:11 |
kmlussier |
gmcharlt: OK, I see your point on the subject name headings now. Imported a Napolean record on stock Evergreen, and the punctuation looks much nicer. Thanks for clarifying! |
| 13:21 |
kmlussier |
For the web client, I was thinking of creating a wiki page with known issues that end users should be aware of before piloting in production. The release notes entry could then link to the wiki page. |
| 13:21 |
kmlussier |
Does that make sense? |
| 13:24 |
bshum |
Sounds like a safe and nice thing to do. |
| 13:24 |
gmcharlt |
and certain more changeable than release notes, which are in effect set in stone |
| 13:24 |
gmcharlt |
or updatable, I should ay |
| 13:32 |
kmlussier |
Yes, the ability to update easily was my primary concern. |
| 13:44 |
Dyrcona |
cssh++ # It's neat watching the text scroll in 8 windows as updates are installed. It's like something from the movies. |
| 13:46 |
bshum |
So much fun! I miss it :( |
| 13:48 |
Dyrcona |
The new drones on VMs seem faster. They finish building OpenSRF before the hardware-based drones do. |
| 13:51 |
|
rfrasur joined #evergreen |
| 13:52 |
rfrasur |
graced, do you mind to check this link? |
| 13:52 |
rfrasur |
https://goo.gl/forms/JyQYRxtACHhDUfGT2 |
| 13:53 |
rfrasur |
(or anyone else that's particularly interested in proofreading) |
| 13:53 |
bshum |
rfrasur: Well, I would give it a try, but it says I need permission outside the organization |
| 13:53 |
graced |
what he said ^^ |
| 13:53 |
rfrasur |
that's what I wanted to know. hold on |
| 13:54 |
rfrasur |
k, try again, if you would. |
| 13:54 |
graced |
access granted |
| 13:55 |
graced |
response submitted |
| 13:55 |
rfrasur |
And the questions sound alright? |
| 13:56 |
graced |
yup |
| 13:56 |
rfrasur |
Got the response and spreadsheet is set up. Okay, rock on. You want me to share that with you and Sharon? |
| 13:58 |
rfrasur |
Done, no matter. Okay. Next up. |
| 13:58 |
graced |
rfrasur++ |
| 13:59 |
bshum |
rfrasur++ |
| 14:01 |
kmlussier |
rfrasur++ |
| 14:02 |
kmlussier |
gmcharlt: Do you know if this is still an issue? https://bugs.launchpad.net/evergreen/+bug/1511433 |
| 14:02 |
pinesol_green |
Launchpad bug 1511433 in Evergreen "teach webstaff about logical deletion of parts" [Medium,New] - Assigned to Galen Charlton (gmc) |
| 14:03 |
kmlussier |
When I use the web client copy/volume editor, I don't see the deleted parts. But I don't know if that's what you were referring to in the report. |
| 14:09 |
JBoyer |
Anyone with any deep-down authorities knowledge in here? Is this situation possible: bib record only contains 100 Twain, Mark; user searches for author: Clemens, Samuel; TPAC returns bib record anyway because of Auth record See Also entry? |
| 14:11 |
kmlussier |
JBoyer: A keyword search? |
| 14:12 |
kmlussier |
If it's keyword searching, I'm putting my money on not possible. |
| 14:12 |
JBoyer |
kmlussier, keyword, author, anything. The point is that clemens is no where in the bib. |
| 14:12 |
JBoyer |
That's what I'm thinking too, though it would seem one of the major points of auth records. |
| 14:13 |
kmlussier |
Yeah, the only place I know of where cross-reference are recognized is browse searching. We've talked about making cross-references available for "Did You Mean" or some kind of "Similar Searches" functionality, but that's future and not automatic. |
| 14:13 |
kmlussier |
And, even in browse search, you have to actively click that See Also reference. |
| 14:13 |
gmcharlt |
kmlussier: I'll take a look re 1511433 |
| 14:14 |
JBoyer |
That's what I thought too, I just wanted to gather more opinions. |
| 14:14 |
JBoyer |
Well, less opinions and more knowledge, but you know what I mean. :) |
| 14:14 |
* kmlussier |
is apparently more knowledgeable about authorities than she wants to be. |
| 14:20 |
dbwells |
JBoyer: We did quite a bit of prep about a year ago, only to eventually opt to not load authority records. We outsource our actually authority control to a vendor annually, and there wasn't yet enough other "secret sauce" in Evergreen authorities otherwise to seem worth it for us at that time. |
| 14:23 |
dbwells |
I think the feature you are advocating is something folks would generally appreciate if the details could be worked out, and I also feel that doing so is more a question of effort than feasibility. |
| 14:24 |
kmlussier |
Oh, yes, by not possible, I meant not possible with current functionality. In case that wasn't clear. :) |
| 14:26 |
kmlussier |
In our case, I think we would have people who would be concerned if it automatically incorporated See Also references. Unless, of course, the interface were clear about what had been added to the search and gave users the option to opt out of incorporating the See Also references into the search. |
| 14:28 |
Dyrcona |
What was it Helmuth von Moltke said? "No plan survives first contact with the enemy." |
| 14:28 |
dbwells |
It was clear, just hoping to encourage some exploration :) |
| 14:29 |
JBoyer |
AS I understand it (shaky, at best) all that needs to happen is that the appropriate 4XX entries from the Twain auth record would need to make their way into metabib.author_field_entry, is that about it? |
| 14:30 |
dbwells |
Dyrcona: heck, most of my plans don't even survive the trip from my brain to my fingers :) |
| 14:30 |
JBoyer |
Then they start to appear for the appropriate searches (though perhaps without it being immediately obvious why) |
| 14:30 |
Dyrcona |
dbwells++ |
| 14:30 |
Dyrcona |
Most of my words don't survive that trip, literally. |
| 14:40 |
dbwells |
JBoyer: I think that's a reasonable way to start poking at it, though maybe starting with 5XX fields would have fewer unintended consequences. Some authority records have a *lot* of unauthorized headings in those 4XXs. |
| 14:41 |
JBoyer |
I'll admit I was flying blind when I said 4XX, I need to look up what the various fields are about in an auth record; I'd likely be sticking to the 5XX fields it sounds like. |
| 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. |
| 14:55 |
JBoyer |
Very nice. |
| 15:23 |
kmlussier |
jeffdavis: It works! |
| 15:24 |
kmlussier |
Well, I should say, e-book availability works. Now need to log in with a patron account that has checkouts. |
| 15:25 |
kmlussier |
Huzzah! I see my e-book checkout too! |
| 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:32 |
bshum |
YES, finally! |
| 16:32 |
* bshum |
is happy. Well, happier. |
| 16:33 |
Dyrcona |
:) |
| 16:34 |
Dyrcona |
@bartender |
| 16:34 |
* pinesol_green |
fills a pint glass with Live Oak HefeWeizen, and sends it sliding down the bar to Dyrcona (http://beeradvocate.com/beer/profile/383/1062/) |
| 16:36 |
stephengwills |
trying to configure OperSRF 2.4.2 on ubuntu 14.04 step 4, the configure is crashing with libmemcached >= 0.8.0 req not met. that happen to anyone else? |
| 16:37 |
kmlussier |
stephengwills! How are you? |
| 16:37 |
Dyrcona |
stephengwills: No, but I've not tried it on 14.04. |
| 16:37 |
Dyrcona |
Just wheezy, jessie, and xenial. |
| 16:37 |
bshum |
stephengwills: Sounds like a missed make prereqs to me |
| 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:08 |
kmlussier |
jeffdavis++ |
| 17:09 |
kmlussier |
jeffdavis: If you want to sign off on the small commit I made at the same time, it will make it easier for me to merge when your changes are done. |
| 17:09 |
gmcharlt |
and if folks are in the area, tweaking the setting's description to discourage changing it to HTTP would be a niceness |
| 17:09 |
Dyrcona |
To quote Throwing Muses: "Things look bad. Things look tragic." |
| 17:09 |
kmlussier |
working/user/kmlussier/lp1541559-ebook-api-master |
| 17:10 |
kmlussier |
gmcharlt++ |
| 17:10 |
jeffdavis |
kmlussier: will do. |
| 17:10 |
jeffdavis |
gmcharlt: will also do. |
| 17:10 |
gmcharlt |
jeffdavis++ |
| 17:14 |
jeffdavis |
gmcharlt: "Using HTTPS here is strongly encouraged" <- do you think that covers it? |
| 17:19 |
|
teletype02 joined #evergreen |
| 17:22 |
gmcharlt |
jeffdavis: +1 |
| 17:22 |
gmcharlt |
(since language about making kittens cry if you use HTTP is perhaps a bit much ;) ) |
| 17:23 |
jeffdavis |
hey, as long as someone is willing to sign off on it... :) |
| 17:23 |
gmcharlt |
:) |
| 17:23 |
bshum |
WTS signoffs for chocolate cookies (and/or scotch) |
| 17:24 |
kmlussier |
gmcharlt: I'm quite sure my kitty wouldn't cry over use of http. In fact, he's a bit evil and would probably be happy about me using an unsecure protocol. |
| 17:25 |
|
kenstir joined #evergreen |
| 17:26 |
gmcharlt |
https://cdn.meme.am/cache/instances/folder645/500x/75753645.jpg |
| 17:28 |
kmlussier |
heh |
| 17:46 |
rhamby |
gmcharlt: kmlussier: at least it's not the elf on the shelf |
| 17:46 |
kmlussier |
rhamby: I am happy to say that the elf on the shelf was never invited into our household. |
| 17:46 |
rhamby |
kmlussier: that is wise and right |
| 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 |
kmlussier |
Dyrcona: You've got your hands full at the moment. |
| 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. |
| 19:17 |
bshum |
Fwiw, it passes pg tests successfully now |
| 19:17 |
bshum |
Let me try the live tests both ways to be sure and then I'll go ahead and commit it |
| 19:17 |
bshum |
But probably after I eat dinner |
| 19:21 |
bshum |
Or you know, right now |
| 19:21 |
bshum |
Cause I'm super curious... |
| 19:22 |
bshum |
Okay, so reverting the new bib change from rhamby, and the perl live test succeeds, except for something with the hold targeter. Probably a bad install on my end |
| 19:22 |
bshum |
I'll double check all my install configs to be sure the hold targeter stuff is setup right |
| 19:22 |
bshum |
This is a reinstalled system so maybe I need to update something |
| 19:23 |
bshum |
When i ran the live tests against the new test bibs, things blew up for stuff all the way down to some of the early tests with bills and circs |
| 19:24 |
bshum |
So maybe we moved a bib assignment or something for an early test |
| 19:24 |
* bshum |
goes to get dinner first now |
| 19:24 |
Dyrcona |
dinner would be nice, but the family is not home, yet. |
| 19:24 |
Dyrcona |
the wife wants to make pasta or hambugers or something. |
| 19:24 |
Dyrcona |
I hinted that she should get pizza and chicken wings on the way home. |
| 19:25 |
kmlussier |
bshum: Which tests are failing? |
| 19:26 |
Dyrcona |
What's not failing would be a shorter list for me.... |
| 19:26 |
kmlussier |
I had been thinking Rogan's new bibs got later ids, but I see they jumped ahead of a few records in the dataset. |
| 19:26 |
|
dcook left #evergreen |
| 19:28 |
kmlussier |
So, generally, when people add new records to the test dataset, I think they add a new sql file, rather than appending it to an existing sql file. Maybe we need to do that with these records to make sure it doesn't bump anything being used by the tests. |
| 19:29 |
Dyrcona |
And, they get home while I'm trying to concentrate on writing an email and the dog goes nuts barking. |
| 19:32 |
* kmlussier |
can try to make that happen while she waits for her VM to fire up. |
| 19:43 |
jeffdavis |
kmlussier: I will write release notes for the ebook API stuff, hopefully tomorrow |
| 19:43 |
kmlussier |
jeffdavis: Great, thank you! |
| 19:43 |
kmlussier |
I was going to try writing some up when I merged the code, but, at this point, I can't guarantee anything I write will be intelligible. |
| 19:44 |
kmlussier |
Also, jeffdavis++ |
| 19:44 |
kmlussier |
I know you've been working on that code for a very long time. :) |
| 19:56 |
Dyrcona |
On the plus side, I'm learning lots of libivirt/qemu/kvm tricks. |
| 20:31 |
Dyrcona |
And just like that, we have a 293GB file for extra storage on the VM. |
| 20:58 |
kmlussier |
Yay! |
| 21:00 |
bshum |
kmlussier: I think I distracted you but did you push the change for the upgrade stamps for the overdrive branch? |
| 21:00 |
bshum |
I think it's still unstamped |
| 21:01 |
kmlussier |
I didn't push those? |
| 21:01 |
kmlussier |
Huh, I know I did those. |
| 21:02 |
bshum |
I'm circling back to finishing testing the test stuff |
| 21:02 |
kmlussier |
I'm still working on that. I just fixed an issue with a missing comma. |
| 21:03 |
bshum |
Oh okay |
| 21:03 |
bshum |
:) |
| 21:18 |
pinesol_green |
[evergreen|Kathy Lussier] LP#1541559: Stamping upgrade scripts for e-book integration - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=8c620be> |
| 21:26 |
Dyrcona |
So, the trouble is, can I disable the staff client and still allow acces to the opac? |
| 21:29 |
|
Stompro joined #evergreen |
| 21:29 |
bshum |
Dyrcona: Well you could rename the server folder in /openils/var/web/xul/ |
| 21:30 |
bshum |
And then no one could use their existing clients |
| 21:30 |
bshum |
Even if you have an upgrade XUL client around |
| 21:30 |
Dyrcona |
that's a thought. |
| 21:30 |
Dyrcona |
The upgrade won't be available. That was on the nfs server that won't boot. |
| 21:30 |
Dyrcona |
I did build new clients on the replacement. |
| 21:31 |
Dyrcona |
but of course, the upgrade files are missing. |
| 21:33 |
Dyrcona |
bshum: That's what I'll do, I'll rename the xul version directory temporarily and I'll delete the old ones still hanging around on some of the servers. |
| 22:08 |
kmlussier |
To follow up on fixing our tests, bshum and I have found that the addition of any new bib records to the test dataset is busting our perl live tests. See my comment at https://bugs.launchpad.net/evergreen/+bug/1665626/comments/4 |
| 22:08 |
pinesol_green |
Launchpad bug 1665626 in Evergreen "Need more metarecord groups in sample dataset" [Undecided,New] |
| 22:09 |
Dyrcona |
Yay! more f'd up legacy server configuration that only manifests when you try to restart stuff. |
| 22:09 |
bshum |
kmlussier++ # trying to fix tests with me |
| 22:09 |
kmlussier |
I would like to hold off on merging that code until we can see if there is a way we can add bibs to the dataset without shifting the existing data. Or, maybe, we just need to accept that we have to fix up a bunch of perl live tests every time we add bibs. |
| 22:10 |
Dyrcona |
This and hardware failure is why a "simple" upgrade has taken 12 hours so far. |
| 22:10 |
kmlussier |
bshum++ |
| 22:12 |
kmlussier |
@swill Dyrcona |
| 22:12 |
* pinesol_green |
grabs a forty of Jeremiah Weed and sends it sliding down the bar to Dyrcona |
| 22:39 |
bshum |
kmlussier: This is a really terrible hack |
| 22:39 |
bshum |
But hey, it works: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/bshum/move-test-ebook-bibs |
| 22:40 |
bshum |
And all live tests succeed for me after I move the ebook bibs out of the way of being loaded the first time around |
| 22:40 |
Dyrcona |
yay. at least something works for somebody. |
| 22:40 |
Dyrcona |
Simple day of installing o/s and security updates, turns into a 3-hour tour on the SS Minnow. |
| 22:41 |
bshum |
"Those poor people..." |
| 22:41 |
bshum |
https://www.youtube.com/watch?v=dQppUxTpdzU |
| 22:41 |
kmlussier |
bshum: Do you want to post that somewhere to LP to get more eyes on it? |
| 22:41 |
kmlussier |
My eyes aren't going to do anyone any good at this point. |
| 22:44 |
kmlussier |
bshum++ #For helping out tonight |
| 22:44 |
bshum |
kmlussier: I put it back on the original LP for the ebook change set |
| 22:44 |
bshum |
https://bugs.launchpad.net/evergreen/+bug/1541559 |
| 22:44 |
pinesol_green |
Launchpad bug 1541559 in Evergreen "OneClickdigital API integration" [Wishlist,Fix committed] |
| 22:44 |
kmlussier |
Dyrcona++ #For working hard to get C/W MARS back up and running. |
| 22:44 |
bshum |
Since I had already used that label for the LP# on the commit message |
| 22:45 |
* kmlussier |
is calling it a night. |
| 22:45 |
kmlussier |
bshum: Thanks! |
| 22:45 |
bshum |
I consider it an extension of the work there |
| 22:45 |
Dyrcona |
But, they're not up and running... |
| 22:45 |
bshum |
But I'll defer since I consider it a bad hack |
| 22:45 |
bshum |
It doesn't fix the next step of bibs we need to get added |
| 22:45 |
Dyrcona |
hardware failures and load balancer mess.... |
| 22:45 |
bshum |
And I'm not sure it's the "best solution". But hey, working code wins? |
| 22:45 |
bshum |
:) |
| 22:45 |
Dyrcona |
For certain definitions of "working." |
