| 11:30 |
gmcharlt |
er, thanks |
| 11:30 |
gmcharlt |
and howdy folks - re bug 1579225, I think it warrants an early 2.10.3; I would appreciate eyes on it |
| 11:30 |
pinesol_green |
Launchpad bug 1579225 in Evergreen "Patron registration- unable to log in with password entered during registration" [High,Confirmed] https://launchpad.net/bugs/1579225 |
| 11:36 |
dbwells |
gmcharlt: I've given it a once (or twice) over and it seems sensible. Rolling it out now to try to test more angles in our live environment. Do you know the simplest means to reproduce the double-hashing problem? I think we experienced that last week, but I haven't tracked it down yet. |
| 11:38 |
gmcharlt |
dbwells: from the user side, it's just a matter of registering a completely new patron in either the XUL or web staff interfaces |
| 11:39 |
dbwells |
gmcharlt: sorry, I was unclear. I mean the later updates which corrupt the password (e.g. address updates). Does that happen for every address update? |
| 11:40 |
gmcharlt |
dbwells: ah, gotcha. no, I don't believe it would occur with every address updates, just ones that add or subtract an address and thereby causes an update of the base patron record |
| 12:16 |
|
bos20k joined #evergreen |
| 12:23 |
|
brahmina joined #evergreen |
| 12:24 |
|
jvwoolf joined #evergreen |
| 12:27 |
dbwells |
gmcharlt: I am off to lunch, but everything looks good in my testing. I posted a sign-off branch to the bug. Hopefully others will still weigh in. |
| 12:47 |
jeffdavis |
I'll be testing that branch today fwiw. |
| 12:48 |
|
bmills joined #evergreen |
| 12:52 |
dbs |
gmcharlt: +1 to your response on bib record / auth record $0 stuff |
| 12:53 |
dbs |
I guess I should just gmcharlt++ so it counts :) |
| 14:00 |
jeff |
And ideally don't require touching every SIP2 client config. |
| 14:01 |
JBoyer |
configs only need to be changed on selfcheks if they currently supply the CP field. (I suspect that's very unlikely since it's currently unused.) Release notes are coming in case it's ready to go as-is. |
| 14:01 |
JBoyer |
Blank CPs are fine, they're ignored. |
| 14:01 |
JBoyer |
Also testing notes will go in LP. |
| 14:08 |
jeff |
Yeah, most of our SIP2 clients set CP. I don't want to speak for anyone else's clients, though. |
| 14:11 |
|
bmills joined #evergreen |
| 14:15 |
jeff |
Oh... lovely. |
| 16:44 |
Dyrcona |
Use a RAID setup. |
| 16:44 |
jeff |
one point twenty-one gigaWALs! |
| 16:45 |
Dyrcona |
heh |
| 16:45 |
jihpringle |
kmlussier: if you're around and have time could you check your 2.9 community test server? it's giving me "There was an error testing this hostname" |
| 16:46 |
kmlussier |
jihpringle: You made need to add an SSL exception. |
| 16:48 |
|
jihpringle_ joined #evergreen |
| 16:48 |
* jeff |
cringes |
| 17:23 |
jihpringle |
you're welcome and thanks for fixing. I've now been able to confirm a bug is definitely new in 2.10 |
| 17:33 |
kmlussier |
I wonder if work on the Angular patron editor may have leaked into the xul client and caused the issue with bug 1579225 |
| 17:33 |
pinesol_green |
Launchpad bug 1579225 in Evergreen "Patron registration- unable to log in with password entered during registration" [Undecided,New] https://launchpad.net/bugs/1579225 |
| 17:34 |
gmcharlt |
jihpringle: roughly when did you test it on the ESI community demo? (I want to check logs) |
| 17:35 |
jihpringle |
gmcharlt: around 1:37pm PDT |
| 17:36 |
gmcharlt |
thanks |
| 17:36 |
dbwells |
Being related to the new password hashing infrastructure would also be a good bet. |
| 17:36 |
jihpringle |
and around 1:51pm PDT for the web client |
| 17:36 |
kmlussier |
dbwells: Yes, I was just remembering that's new too. |
| 17:41 |
* gmcharlt |
takes a look |
| 17:42 |
gmcharlt |
test record: a cat with poor password hygiene |
| 17:58 |
|
jihpringle joined #evergreen |
| 18:18 |
|
jlitrell joined #evergreen |
| 18:26 |
gmcharlt |
yep, it's the password hashing |
| 18:26 |
gmcharlt |
during registration, the hashed password is getting set to CRYPT( MD5( pw_salt || MD5(MD5(real_password)) ), pw_salt ) |
| 18:26 |
gmcharlt |
when comparisons are expecting CRYPT( MD5( pw_salt || MD5(real_password) ), pw_salt ) |
| 18:27 |
gmcharlt |
I'll poke more after dinner |
| 18:30 |
gmcharlt |
ok, seeing more: during the open-ils.actor.patron.update dance, a new actor.usr is created |
| 18:30 |
gmcharlt |
the password trigger on actor.usr sets au.passwd to md5(real_password) |
| 18:31 |
gmcharlt |
*then*, the new row gets retrieved and passed on to the rest of the process, including a call to actor.set_passwd |
| 18:31 |
gmcharlt |
and that's where the extraneous md5 is coming from |
| 18:31 |
gmcharlt |
more anon |
| 18:43 |
Christineb |
gmcharlt++ thank you for investigating |
| 19:41 |
|
brahmina joined #evergreen |
| 20:58 |
|
sandbergja joined #evergreen |
| 10:49 |
Dyrcona |
sitalk: It sounds like you have not completely installed OpenSRF and Evergreen. |
| 10:49 |
kmlussier |
mmorgan: Is there anything else that might block the hold beyond age protection? An issue with the patron's account? |
| 10:50 |
Dyrcona |
mmorgan: Do the staff have the permission to override at the consortium depth? It could be a depth issue. |
| 10:51 |
kmlussier |
I know this is hard to test in production, but I wonder if removing the two non-holdable copies (or making them holdable, but age-protected) would make the override appear. |
| 10:52 |
Dyrcona |
mmorgan: Our staff have the ITEM_AGE_PROTECTED.override at depth 0. |
| 10:52 |
Dyrcona |
Well, actually, all users have it at that depth. |
| 10:54 |
Dyrcona |
sitalk: Did you a2ensite eg.conf? |
| 11:40 |
terran |
Bmagic: there are definitely still some OPAC screens that still need more responsive design work |
| 11:41 |
terran |
Bmagic: I tweaked a few things when doing our customizations for our last upgrade, but I was hoping to get back to it at some point (hasn't happened yet) |
| 11:42 |
Bmagic |
It seems like the issue is the use of tables, but I might be wrong |
| 11:46 |
terran |
Hrrm - I'm looking at my test server with 2.9 on it on the opac items out page and don't see anything egregiously broken |
| 11:47 |
terran |
never mind, yes I do |
| 11:52 |
bshum |
Patches welcome :) |
| 11:52 |
|
rjackson_isl joined #evergreen |
| 11:56 |
kmlussier |
Bmagic / terran: I could be misremembering, but I think NOBLE may have done some nice customizations to that screen to address that issue. I think that's one of the screens they were working on at the hack-a-way. I can follow up with them. |
| 14:24 |
yboston |
seems like the 2.10 HTML is not building properly |
| 14:24 |
yboston |
the cahnges sandbergja made to the attributions page are not showing up |
| 14:24 |
jihpringle |
do we have access to the build log for it? |
| 14:25 |
yboston |
only Robert |
| 14:25 |
yboston |
it is building fine on dev amd on the “docs-testing” server |
| 14:25 |
sandbergja |
I could ask Robert to see if we can troubleshoot the issue together |
| 14:25 |
yboston |
http://docs.evergreen-ils.org/dev/attributions.html - here is the main dev version |
| 14:26 |
sandbergja |
I was very curious about that myself! |
| 14:26 |
yboston |
http://docs.evergreen-ils.org/2.10/attributions.html - this is the main 2.10 version |
| 14:26 |
jihpringle |
that explains my issue too, my additions are showing in dev |
| 14:26 |
yboston |
http://docs-testing.evergreen-ils.org/docs/2.10/attributions.html - here is the testing 2.10 |
| 14:26 |
remingtron |
I would expect the page to be blank if there was a build error |
| 14:27 |
remingtron |
are we sure it was backported to 2.10? |
| 14:27 |
yboston |
I can email Robert and icnlude you all to be on the same page |
| 14:27 |
jihpringle |
I don't know how it works for these docs, but with ours if it runs into a problem it just keeps the last version that worked |
| 14:27 |
yboston |
yes, the commits are there, unless a newer commit somhow undud the changes. then againt he testing server grabbed it correctly |
| 14:28 |
remingtron |
yboston: you are right, git.evergreen-ils.org shows the updated file |
| 14:28 |
remingtron |
here's the 2.10 version: http://git.evergreen-ils.org/?p=Evergreen.git;a=blob;f=docs/attributions.txt;h=3e6c629f3d64a43b53be4ba1ac0aa110bd9d67c2;hb=rel_2_10 |
| 14:29 |
yboston |
BTW, normally I just email Robert directly and usually CC remingtron |
| 14:34 |
yboston |
gmcharlt++ gmcharlt++ |
| 14:34 |
remingtron |
yboston: I have a topic |
| 14:35 |
remingtron |
ideas that came up at the conference |
| 14:35 |
yboston |
for setting up the http://docs-testing.evergreen-ils.org |
| 14:35 |
yboston |
go ahead |
| 14:35 |
yboston |
#topic confernece ideas |
| 14:35 |
yboston |
darn |
| 14:36 |
yboston |
(go ahead) |
| 14:36 |
remingtron |
#idea Add links in docs to Sitka doc videos on youtube |
| 14:37 |
remingtron |
#idea Investigate fixing the Help menu link on the XUL client (just needs HTML help/doc files in the right place) |
| 14:37 |
jihpringle |
I'll have a list of the ones that are still accurate for 2.10 by next week |
| 14:50 |
yboston |
I wonder how the links would behave in the PDF and epub version. I hoep they work, but I would still want to proceed if they don't |
| 14:50 |
jihpringle |
they work in our pdf |
| 14:50 |
yboston |
one mroe thing, does Sitka have plans to delete any older videos? concerned abotu “link rot” inthe future |
| 14:51 |
jihpringle |
(I just tested) |
| 14:51 |
yboston |
excellent on the testing |
| 14:51 |
jihpringle |
no, we have no plans to delete the videos |
| 14:51 |
yboston |
cool |
| 14:51 |
jihpringle |
at this point we figure even it if it's not current for us it may be for someone else |
| 14:52 |
jihpringle |
Christineb++ |
| 14:52 |
yboston |
Christineb++ |
| 14:53 |
Christineb |
:) |
| 14:53 |
yboston |
any ideas of how to proceed with this idea? |
| 14:54 |
yboston |
should we ask on the DIG list to see if anyone objects or just has any warnings? |
| 14:54 |
yboston |
shoudl we start with a few links as a test? |
| 14:54 |
Christineb |
We have asked our members for video suggestions - we could also ask DIG if they have any suggestions for short videos |
| 14:54 |
jihpringle |
we should ask PINES about including their videos too |
| 14:54 |
yboston |
shoudl we just discuss it further at the next meeting? |
| 00:36 |
bshum |
dcook: Hey man! |
| 00:36 |
dcook |
howdy :) |
| 00:37 |
* dcook |
knows nothing about Evergreen so steps back to the sidelines |
| 00:38 |
* bshum |
doesn't know what the error means but gets wary with Jessie dependency differences for apache 2.4 |
| 00:39 |
bshum |
I think while I was testing various builds I noticed that Jessie references were missing from the README somewhere |
| 00:39 |
* bshum |
doesn't remember offhand though |
| 00:41 |
* bshum |
is more of a Ubuntu guy |
| 00:45 |
dcook |
I don't know anything about Evergreen, but that's intriguing that there's a OSRFGatewayConfig directive. Does Evergreen have an Apache module dependency? |
| 00:45 |
bshum |
dcook: I'm sure it does somewhere |
| 00:46 |
dcook |
That's wild |
| 00:47 |
* dcook |
will stop troubleshooting software that he doesn't even work on :P |
| 00:48 |
bshum |
dcook++ # drive-by-talking-aloud-troubleshooting |
| 00:48 |
dcook |
hehe |
| 00:49 |
bshum |
A cursory google search doesn't reveal anything for that error message sitalk linked |
| 00:49 |
bshum |
But I wonder if this is either a missed dependency issue (which I know that they added something recently for Jessie) |
| 00:50 |
bshum |
Or if maybe the system was installed with the wrong apache config files |
| 00:50 |
bshum |
There's stuff in examples/apache/* and examples/apache_24/* |
| 00:50 |
bshum |
And for Jessie, you'd want the apache_24/* stuff |
| 00:50 |
bshum |
Since that's the version of Apache that ships with the system |
| 00:50 |
bshum |
I'm unsure what happens if you copy the wrong files into place, I assume it wouldn't work out due to syntax differences between the apache versions |
| 00:55 |
* bshum |
fires up his incomplete Jessie VM to find out |
| 00:56 |
bshum |
sitalk: Not sure if you're still around, but I know you did email the lists with your issue recently. This IRC channel isn't often so talkative at this time, most folks are active in the 9am-5pm Eastern americas timezone |
| 01:06 |
|
mceraso joined #evergreen |
| 01:06 |
|
bshum joined #evergreen |
| 01:27 |
bshum |
sitalk: So I tried running something like "apache2ctl -M" to get a list of loaded apache modules on my Debian Jessie test machine |
| 01:27 |
bshum |
Output looked like http://pastie.org/10823337 |
| 01:27 |
bshum |
Ignoring the 2nd and 3rd line warnings, the rest lists out the Loaded Modules entries |
| 01:28 |
bshum |
And maybe should compare and see if that indicates what might be different between the systems |
| 01:28 |
bshum |
I'll put that into an email reply to your question on the list, just in case you aren't looking at IRC further. |
| 01:30 |
* bshum |
wanders back off to sleep |
| 04:36 |
|
gsams joined #evergreen |
| 06:27 |
|
mrpeters joined #evergreen |
| 06:40 |
|
rlefaive joined #evergreen |
| 12:48 |
bshum |
I'm not sure if that ought to hold up the Xenial merge though. Since the angular 1.5 issue breaks across all the supported distros at the moment. |
| 12:48 |
Dyrcona |
Right. I don't think it should, either. |
| 12:49 |
Dyrcona |
Concerto works on xenial with Pg 9.5 with the above branch and the other xenial branches. |
| 12:49 |
Dyrcona |
All of the tests passed. |
| 12:49 |
bshum |
Dyrcona: Oh now I remember. Since the angular 1.5 bug points at 2.next (no milestone assignment yet for 2.11) I figured that maybe we should re-open the bug and mark it back to confirmed with notes on the breakage. If it had been on a fix released bug, then I'd be more wary. |
| 12:50 |
Dyrcona |
OK, I can go along with that. |
| 12:50 |
bshum |
Or fix committed onto an actual real milestone |
| 13:03 |
Dyrcona |
I love how the same conflict comes up multiple times when cherry-picking. :) |
| 13:07 |
Dyrcona |
There are 7 commits: 5 from csharp and 2 from me. |
| 13:07 |
Dyrcona |
csharp's lack the lp number, so I'll rebase and edit to add it. |
| 13:08 |
gmcharlt |
I'm fine leaving it out of rel_2_10 for now, but that's driven mostly by a desire to see more Xenial/Stretch testing occur before we make the decision whether to formally extend distro support to the maintenance releases |
| 13:14 |
Dyrcona |
gmcharlt: +1 |
| 13:14 |
Dyrcona |
bshum: http://git.evergreen-ils.org/?p=working/OpenSRF.git;a=shortlog;h=refs/heads/collab/dyrcona/ubuntu-1604-final |
| 13:15 |
bshum |
Dyrcona: Looks good to me |
| 14:32 |
rndrson2 |
Yes, it is! Thank you, hello everyone! |
| 14:32 |
yboston |
welcome |
| 14:33 |
rndrson2 |
I figured I would lurk a little and see what everyone was talking about |
| 14:33 |
yboston |
we have been mostly going down the contents of the Evergreen community IRC “quick start guide” |
| 14:33 |
yboston |
http://wiki.evergreen-ils.org/doku.php?id=community:irc-quick-start |
| 14:33 |
yboston |
we started with the concept of “replying” |
| 14:34 |
yboston |
rndrson2: which is done as you see here by starting my line with your IRC nickname , a colon then a spcae. then I add my message to you |
| 14:34 |
yboston |
rndrson2: we also sent each other “private messages” which are also explained in the “quick start guide” |
| 14:34 |
yboston |
rndrson2: please feel free to try to send me a private message as a test |
| 14:35 |
rndrson2 |
yboston: makes sense! I've been reading through that, as well as the slideshow from the conference |
| 14:35 |
yboston |
rndrson2: excellent |
| 14:35 |
yboston |
rndrson2: we also talked about “karma” and voting” |
| 14:40 |
yboston |
rndrson2: I beleive they are kept for years, since we are an open source community |
| 14:41 |
jeff |
the irc archives are not subject to any automated time-based purging. |
| 14:41 |
rndrson2 |
It looks like the calendars go back to 6/2013 |
| 14:41 |
yboston |
which reminds me to tell you all not to post your passwords or little test server IP addresses on IRC |
| 14:42 |
jeff |
this irc channel is public, in a way very similar to the community mailing lists. |
| 14:42 |
Dyrcona |
It's a public forum. You should treat as such. |
| 14:42 |
yboston |
one good thing about the log that it is great if you miss an IRC meeting |
| 10:58 |
Dyrcona |
berick: Thanks. That's what I thought. |
| 11:02 |
|
geoffsams joined #evergreen |
| 11:05 |
|
terran joined #evergreen |
| 11:09 |
kmlussier |
My goal for this year is to become clearer on which code contributions require a QA test and which do not. bug 1447746 alters a database function. Does this rise to the level of requiring a pgTap test? |
| 11:09 |
pinesol_green |
Launchpad bug 1447746 in Evergreen "Do not update bib source on match-only merge" [Wishlist,Confirmed] https://launchpad.net/bugs/1447746 |
| 11:19 |
Dyrcona |
At the very least, a pgtap test for the existence of the new column could be done. |
| 11:19 |
|
rlefaive joined #evergreen |
| 11:19 |
Dyrcona |
It can be difficult to write tests for database functions. |
| 11:20 |
|
ddale joined #evergreen |
| 11:21 |
kmlussier |
OK, thanks for the guidance Dyrcona. |
| 11:21 |
kmlussier |
Dyrcona++ |
| 11:22 |
Dyrcona |
You could test that by adding a queue and a record, running the overlay function, and then retrieving the results. |
| 11:22 |
Dyrcona |
It would be a rather involved test, like some of the ones I've written lately. ;) |
| 11:23 |
mceraso |
At the Hack-A-Way this past November, gmcharlt gave an excellent overview of Evergreen and all of its pieces. I thought a map of this was made...was this ever shared with the community? I thought that it was, but I'm having difficulty finding it. Anyone know, offhand? |
| 11:23 |
kmlussier |
mceraso: I think there is a link to it from the Google doc. |
| 11:23 |
Dyrcona |
Could be done as Perl tests or as pgtap. |
| 11:24 |
mceraso |
kmlussier: Aha! That sounds familiar! |
| 11:24 |
kmlussier |
mceraso: Is this it? https://docs.google.com/drawings/d/17aNEr8vLen5wBjCAP4NPnjL7fYT3VxK6_9wVArR9VII/edit?usp=sharing |
| 11:24 |
mceraso |
kmlussier: Yes! Excactly what I needed :) |
| 22:48 |
jeff |
Stompro: keep in mind that the --disable-triggers option is only relevant for data only dumps or restores. |
| 22:49 |
jeff |
Stompro: otherwise, pg_restore's going to copy the data into the tables before it adds things like constraints and triggers. |
| 23:39 |
|
sitalk joined #evergreen |
| 23:45 |
sitalk |
Hello everyone. |
| 23:45 |
sitalk |
I am having the following problem: |
| 23:48 |
sitalk |
The installation of Evergeen went smooth and there were no any errors after starting Evergreen. However I got the following error after restarting apache: |
| 23:49 |
sitalk |
[FAIL] Starting web server: apache2 failed! |
| 23:49 |
sitalk |
[warn] The apache2 configtest failed. ... (warning). |
| 23:49 |
sitalk |
Output of config test was: |
| 23:49 |
sitalk |
AH00526: Syntax error on line 43 of /etc/apache2/eg_vhost.conf: |
| 23:49 |
sitalk |
Invalid command 'OSRFGatewayConfig', perhaps misspelled or defined by a |
| 23:49 |
sitalk |
module not included in the server configuration |
| 23:49 |
sitalk |
Action 'configtest' failed. |
| 23:49 |
sitalk |
Has anyone seen such error before? |
| 23:49 |
sitalk |
Do you have any idea how this issue could be resolved? |
| 23:49 |
sitalk |
Thanks in advance. |
| 23:50 |
sitalk |
I am using Evergreen 2.10 on Debian 8.4 |
| 15:04 |
jeffdavis |
miker: that link above shows how it is working for us now: https://bwlcr.bc.catalogue.libraries.coop/eg/opac/record/108220410 |
| 15:05 |
jeffdavis |
but Liam's changes to make that happen are not a good fit for inclusion in the main codebase, so I'm looking at other ways to implement the same effect |
| 15:05 |
miker |
jeffdavis: ah, ok, so that's sitka-local. I wonder if there's any stock examples, because your page looks good to me |
| 15:06 |
jeffdavis |
https://bwlcr.testing-210-2.catalogue.libraries.coop/eg/opac/record/108220410 shows the same record in 2.10, with Liam's fix reverted |
| 15:07 |
jeffdavis |
(it doesn't work as-is in 2.10, so rather than just tweaking it slightly, this seemed like a good time to see if I could rework it before we upgrade next month) |
| 15:12 |
jeffdavis |
dbwells: fwiw I'm asking locally to see if there is any use case for ever NOT using chronological sorting for serials; if not, that should simplify things |
| 15:12 |
dbwells |
jeffdavis: cool, sounds good |
| 15:14 |
jeffdavis |
dbwells: also, thanks for the discussion! I'll think about this some more and update LP when I have things sorted (so to speak) in my own mind |
| 15:16 |
dbwells |
jeffdavis: I am going to dig up my branch and rework it. I had similar conversations with a few others at the conference, so I think there's enough interest in seeing this in action to better understand and react to it. I think it fits this bug enought that I'll post it there early next week. Does that sound alright? |
| 16:11 |
|
terranmc joined #evergreen |
| 16:12 |
dbs |
oh https://bugs.launchpad.net/bugs/1031335 some day you will be loved again |
| 16:12 |
pinesol_green |
Launchpad bug 1031335 in Evergreen "Email templates should always escape headers" [Medium,Confirmed] |
| 16:23 |
jeffdavis |
What is preventing that bugfix from getting into point releases? Signoff, tests, change in status...? |
| 16:27 |
pinesol_green |
[evergreen|Pasi Kallinen] Fix LP#1031335 by escaping some email headers automatically - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=d88e362> |
| 16:27 |
pinesol_green |
[evergreen|Dan Scott] LP#1031335 No-op the escape_email_header helper - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=6a8a4f0> |
| 16:29 |
jeffdavis |
heh |
| 11:49 |
|
Dyrcona joined #evergreen |
| 11:52 |
* miker |
claims 0978 for the YAOUS multitude |
| 11:53 |
|
bmills joined #evergreen |
| 11:57 |
pinesol_green |
[evergreen|Galen Charlton] LP#1568195: fix retrieving big OUS batches - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=18cb25a> |
| 11:57 |
pinesol_green |
[evergreen|Mike Rylander] LP#1568195: Stamping upgrade script - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=491c2cd> |
| 12:01 |
pinesol_green |
[evergreen|Galen Charlton] LP#1563531: add libpcre3/libpcre3-dev deps for Jessie - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=b07ef85> |
| 12:05 |
pinesol_green |
[evergreen|Galen Charlton] LP#1562153: add regression test - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=977f85b> |
| 12:05 |
pinesol_green |
[evergreen|Galen Charlton] LP#1562153: fix case where changing sort order to relevance can fail - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=e45c91d> |
| 12:07 |
gmcharlt |
Dyrcona: berick: absent something coming up unexpectedly, from my POV rel_2_10 is now ready for the 2.10.2 release-cutting mill |
| 12:08 |
gmcharlt |
i.e., all bugs targetted for 2.10.2 now have status fixed committed, and I've bump others to 2.10.3 |
| 12:16 |
JBoyer |
miker's "YAOS multitude" nearly gave me a heartastroke. I initially took it to mean rather something else. (Yo dawg, I heard you like settings...) |
| 12:46 |
pinesol_green |
[evergreen|Galen Charlton] updates to the 2.10.2 release notes - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=48c38f5> |
| 12:47 |
Dyrcona |
Awesome! |
| 12:54 |
|
mmorgan joined #evergreen |
| 12:54 |
Dyrcona |
I might not be able to build/test a release tarball until tonight. |
| 12:55 |
Dyrcona |
I have to go to a library for 3:00 pm EDT today. |
| 12:55 |
gmcharlt |
OK |
| 13:01 |
|
hbrennan joined #evergreen |
| 14:22 |
phasefx |
:D :D |
| 14:32 |
jeff |
nobody else has any SIP2 annoyances they'd like to share? :-) |
| 14:33 |
|
sandbergja joined #evergreen |
| 14:37 |
gmcharlt |
2.10.2 is available from https://evergreen-ils.org/egdownloads/ |
| 14:37 |
gmcharlt |
testing welcome, of course |
| 14:37 |
gmcharlt |
I'll await for 2.9.4 and 2.8.8 before making any sort of announcement |
| 14:40 |
gmcharlt |
demo.evergreencatalog.com is now running 2.10.2 as well |
| 14:40 |
pinesol_green |
[evergreen|Galen Charlton] 2.10.1 to 2.10.2 schema upgrade script - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=69b22e2> |
| 14:40 |
jeff |
gmcharlt++ |
| 14:41 |
kmlussier |
gmcharlt++ |
| 14:47 |
JBoyer |
At this point I don't know if it's worth pulling off of my laptop or just starting fresh on my work machine. |
| 14:48 |
jeff |
cool. if you're going to do it i won't try, but if you'd like to collab let me know. |
| 14:48 |
jeff |
the idea was to allow for default/override in the user def, but also permit the client to specify, right? |
| 14:51 |
JBoyer |
As for other annoyances, the fact that we over-sanitize the end of the message means that stronger passwords can't be used because they may be truncated before being tested. (I filed a bug on this one, if the password ends in ! for example, you can't use a selfcheck or SIP-based vendor service. |
| 14:53 |
JBoyer |
My idea re:workstations is that if you don't supply the CP field you get the same login type as current (opac or temp, can't remember) and if you do supply a CP then you get that workstation only. My changes should be very small and may not work for every vendor (nothing does.) |
| 14:56 |
|
rlefaive joined #evergreen |
| 15:07 |
|
rlefaive left #evergreen |
| 15:40 |
jeff |
Has anyone determined how different SIP2 clients handle a password containing a pipe character? Do they omit it, substitute it, try to pass it unmodified, and/or crash? |
| 15:41 |
|
ethomsen left #evergreen |
| 15:41 |
berick |
jeff: that's like staring into the void |
| 15:42 |
jeff |
I'm surprised that SIP3 didn't include an escape... no, I'm not surprised. |
| 15:43 |
jeff |
I can test a few implementations. |
| 15:43 |
jeff |
But not as many as I'd like. |
| 16:02 |
tsbere |
jeff: Most of the ones I have seen either bail on it (bad input), expect that field to be a *pin* and not a *password*, or pass it on as-is breaking the message. |
| 16:03 |
tsbere |
jeff: Which brings me, interestingly, to a thought of "that would be nice": SIP2 and maybe telephony only "PIN" number to avoid this kind of issue. <_< |
| 16:05 |
jeff |
yeah. |
| 17:38 |
kmlussier |
Ewwww |
| 17:42 |
Dyrcona |
Dah Beast! :) |
| 17:42 |
Dyrcona |
That's what my acquaintances used to call it. |
| 17:42 |
* Dyrcona |
is testing tarball installation. |
| 17:45 |
Dyrcona |
@brew |
| 17:45 |
pinesol_green |
Dyrcona: well, that's what you get for not being a shell script |
| 17:45 |
Dyrcona |
@bartender |
| 17:46 |
* pinesol_green |
fills a pint glass with Cambridge Brewing Company's Late Harvest IPA, and sends it sliding down the bar to kmlussier (http://beeradvocate.com/beer/profile/14/26930/) |
| 17:47 |
kmlussier |
Dyrcona: That's better, but I would have been much happier with your Octoberfest. |
| 17:47 |
Dyrcona |
We'll trade. :) |
| 17:49 |
* Dyrcona |
is preparing to test the upgrade script. Just have to build 2.9.3 enough and rebuild the database, first. |
| 17:52 |
Dyrcona |
I recall being in a roleplaying channel years ago where the bartender bot would distribute a wider variety of beverages than just beer. |
| 17:53 |
Dyrcona |
And, the upgrade script works. |
| 17:59 |
kmlussier |
Yay for working upgrade scripts! |
| 18:44 |
|
gsams joined #evergreen |
| 18:52 |
Dyrcona |
Assuming 2.8.8 is ready to go (the files are on the server), I will update the downloads page for 2.9.4 and 2.8.8 at the same time. |
| 18:52 |
* jeffdavis |
notices bug 1573372, feels bad about introducing it |
| 18:52 |
pinesol_green |
Launchpad bug 1573372 in Evergreen 2.10 "Fix 15-opt-in live test, use SKIP instead of BAILOUT" [Undecided,Fix committed] https://launchpad.net/bugs/1573372 |
| 19:00 |
Dyrcona |
Umm.... WordPress obviously doesn't like me. |
| 19:01 |
Dyrcona |
@blame touchpads |
| 19:01 |
pinesol_green |
Dyrcona: touchpads musta been an Apple employee. |
| 09:40 |
|
mllewellyn joined #evergreen |
| 09:45 |
kmlussier |
rhamby_: I'm not sure about the @brains plugin. Since coffee is what gets my brain working, I was thinking it would suffice. |
| 09:46 |
rhamby_ |
kmlussier: I had a sleepless night of a child coughing and feeling miserable. I think zombie like enough that I might need the actual brains. |
| 09:47 |
kmlussier |
rhamby_: Ugh, sorry to hear that. |
| 09:48 |
|
awitter joined #evergreen |
| 10:09 |
|
mmorgan1 joined #evergreen |
| 10:16 |
kmlussier |
I've tested the fix for bug 1564079 and it looks good to me. I want to make sure I handle the database bits correctly when I merge it. |
| 10:16 |
pinesol_green |
Launchpad bug 1564079 in Evergreen "Checkout history fails on serials checkouts / deleted copies" [High,Confirmed] https://launchpad.net/bugs/1564079 - Assigned to Kathy Lussier (klussier) |
| 10:16 |
berick |
kmlussier++ |
| 10:16 |
kmlussier |
It looks like the upgrade script was modified directly for most of the changes, and the only upgrade script I need to stamp is schema.fix_circ_history_copy_ref.sql. Is that correct? |
| 11:10 |
awitter |
rlefaive nice!! |
| 11:11 |
csharp |
rlefaive: we'll look into making it work with vagrant/vbox |
| 11:12 |
rlefaive |
csharp++ awitter++ |
| 11:14 |
awitter |
rlefaive thanks for testing this |
| 11:16 |
Dyrcona |
rlefaive: Did you install to a virtual machine or to actual hardware? |
| 11:17 |
rlefaive |
it was a virtual machine (virtualbox on my mac), but I suffered through the purple-screen install steps instead of using vagrant up. |
| 11:17 |
jeff |
what basebox were you using with vagrant? |
| 11:29 |
Dyrcona |
We already have staff complaining that account adjustment shows up as a payment, so I'd argue "No" for amnesty using an adjustment. |
| 11:29 |
Dyrcona |
So, the patron has a negative balance of $5.00? |
| 11:29 |
Bmagic |
yep! |
| 11:29 |
* csharp |
is now testing ubuntu/trusty64 on vagrant so we can compare with base ubuntu installs from ISO |
| 11:30 |
awitter |
csharp++ apt-get install evergreen-ils should work in all cases |
| 11:30 |
Dyrcona |
So, lost didn't just void the overdue billings.... |
| 11:31 |
Dyrcona |
dbwells took this branch over from me and added the account adjustments. |
| 15:55 |
Dyrcona |
BTW, I'm signing off on lp 1569884, it could be included in this round of updates. |
| 15:55 |
pinesol_green |
Launchpad bug 1569884 in Evergreen 2.9 "Reusing deleted monograph parts" [Undecided,Confirmed] https://launchpad.net/bugs/1569884 |
| 15:55 |
gmcharlt |
+1 |
| 15:55 |
Dyrcona |
There are no notes or tests, but we're using the upgrade script version in production. |
| 15:57 |
kmlussier |
jeffdavis: I don't know how far you got in your explorations, but in answer to one of the questions Liam asked, there is a serials compressed holdings view that sorts the serials holdings display by date. |
| 15:58 |
kmlussier |
jeffdavis: We don't use that display because of bug 963341. I actually don't think the patch that came with the bug fixes the problem originally reported. I'll have to look into it at some point and file a new one if needed. |
| 15:58 |
pinesol_green |
Launchpad bug 963341 in Evergreen "Allow the JSPac and TPac to display both MFHD records and Serial Control/Alternate records," [Medium,Fix released] https://launchpad.net/bugs/963341 |
| 16:36 |
dbs |
kmlussier++ # thanks, "Replace barcode" is a bit unintuitive :) I had 20 copies with 20 separate callnums to assign parts to and unify callnums, so the bulk "Edit items/volumes per bib" did the trick in one shot |
| 16:37 |
dbs |
People are still getting used to parts here, so even when I create the catalogue record & the parts in advance and say "please use the Computer & Charger parts for a single callnum" they sometimes fall back to old habits |
| 16:38 |
dbs |
(not to mention "LC" callnum type for accession callnums of "1001", "1002", etc... heh) |
| 16:44 |
kmlussier |
dbs: Yeah, we tested parts at the same time we were testing the unified volume/copy editor, so I didn't notice the "unintuitive for non-unified-editor users" piece until others started using the feature. |
| 16:46 |
|
gmcharlt_ joined #evergreen |
| 16:46 |
|
dbs joined #evergreen |
| 16:48 |
|
awitter joined #evergreen |
| 13:45 |
|
krvmga joined #evergreen |
| 13:51 |
|
mrpeters joined #evergreen |
| 13:53 |
bshum |
@rss announce list |
| 13:53 |
pinesol_green |
bshum: qatests |
| 13:55 |
|
ericar_ joined #evergreen |
| 13:57 |
pinesol_green |
News from qatests: Test Failure - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 14:03 |
|
rlefaive joined #evergreen |
| 14:04 |
|
jvwoolf joined #evergreen |
| 14:26 |
|
mrpeters left #evergreen |
| 14:30 |
* tsbere |
looks at the test failure message for kicks, finds out that it was a karma failure and considers that mildly amusing |
| 15:05 |
|
mmorgan1 joined #evergreen |
| 15:29 |
kmlussier |
phasefx++ #Nicely stated! |
| 15:30 |
* phasefx |
still remembers the early days when folks thought we were nuts |
| 17:27 |
jeffdavis |
@seen collum |
| 17:27 |
pinesol_green |
jeffdavis: collum was last seen in #evergreen 3 days, 2 hours, 47 minutes, and 14 seconds ago: <collum> The Zucker jokes won't stop forming in my brain. |
| 17:27 |
awitter |
gotit |
| 17:28 |
csharp |
rlefaive: we're testing to see if the hostname ending with a number is the problem |
| 17:28 |
rlefaive |
csharp: ridonculous! but plausible. |
| 17:37 |
csharp |
rlefaive: nope - not the issue |
| 17:37 |
rlefaive |
csharp: are you getting the issue? |
| 17:50 |
rlefaive |
awitter, csharp: should i run apt-get install as opensrf user? |
| 17:51 |
rlefaive |
well poop, i wasn’t root. |
| 17:51 |
rlefaive |
i wasn’t root, but i did sudo. so that’s probably not it. |
| 17:52 |
awitter |
rlefaive when you get a chance can you test it on a full ubuntu server 14.04 proper rather than a canned image? |
| 17:54 |
|
awitter joined #evergreen |
| 17:58 |
rlefaive |
The server iso is still half-downloaded and I’m 2h late for going home. Sorry folks, i’ll have to continue tomorrow. |
| 18:40 |
awitter |
ok no worries |
| 02:15 |
bshum |
Karma has been reset. Made copy output of final karma count here: https://evergreen-ils.org/~bshum/karma.final.2016.txt |
| 02:22 |
bshum |
chocolate++ # testing karma reset and also thinking about tasty things |
| 02:22 |
bshum |
@karma |
| 02:22 |
pinesol_green |
bshum: Highest karma: "chocolate" (1). Lowest karma: "chocolate" (1). |
| 02:22 |
* bshum |
wanders off to sleep. |
| 03:55 |
|
gsams joined #evergreen |
| 11:52 |
Dyrcona |
Well, humbug. I *thought* I changed all of my openils-* list subscriptions to my new email address..... |
| 11:54 |
Bmagic |
Has anyone seen a circulation case where the max_fine_rule is set to a rule but the resulting max_fine column received "0.00" - Is there another factor that overrides the rule? |
| 11:55 |
Bmagic |
The rule in this case dictates "10.00" - and I have other circulations that use that rule and the max_fine gets 10.00. But in this one case, the max_fine column was set to 0.00 |
| 11:56 |
Dyrcona |
Bmagic: Multiple rows can affect that depending on the circulation. You can test that with some functions in the database. |
| 11:57 |
|
HoloIRCUser5 joined #evergreen |
| 11:57 |
mmorgan |
Bmagic: I've seen similar when the circ predates a rule change, or when more than one rule can apply, as Dyrcona says |
| 11:57 |
Dyrcona |
I *think* also if the item charges no fines, then max fine comes out to 0 regardless, but that would take more digging. |
| 11:15 |
jeff |
@rss announce add qatests |
| 11:15 |
pinesol_green |
jeff: The operation succeeded. |
| 11:15 |
jeff |
@rss qatests |
| 11:15 |
pinesol_green |
jeff: 11:15 AM, April 23, 2016: Test Failure - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 11:15 |
bshum |
jeff: It won't announce I think cause the date doesn't change on the RSS feed |
| 11:15 |
bshum |
Or rather, there is no date timestamp |
| 11:16 |
bshum |
So it views it as no change, whenever it stays failed |
| 11:16 |
bshum |
The RSS feed in #openils-evergreen works fine for reading off bugs, commits, etc. so I'm pretty sure that's the culprit area, not the plugin anyways. |
| 11:16 |
bshum |
I've been meaning to ask phasefx too, but didn't get around to it yet. |
| 11:17 |
jeff |
seems logical. |
| 11:17 |
bshum |
Oh good, the build test has the same error I see :) |
| 11:17 |
jeff |
also, it's the kind of rss feed where the url doesn't change. that could be it also. |
| 11:18 |
bshum |
Is it normal that it grabs two versions of phantomjs? I see a part where it grabs 1.9.x and also 2.1.x |
| 11:18 |
bshum |
jeff: Yeah, enhancing the way the RSS generates for live test would be cool :) |
| 11:18 |
jeff |
bshum: i don't know (re: two phantomjs versions) |
| 11:19 |
bshum |
It shows up here, along with where the build test fails and stops - http://testing.evergreen-ils.org/~live/test.16.html |
| 11:19 |
bshum |
Just an oddity I guess |
| 11:22 |
berickm |
bshum: looking at my 14.04 server now.. |
| 11:24 |
|
hbrennan joined #evergreen |
| 11:25 |
berickm |
bshum: when done, do you have 2 versions or only last one installed? |
| 11:25 |
berickm |
i don't have the install log anymore for my test server, but can confirm there is only one version available |
| 11:26 |
bshum |
berickm: I'm not sure yet, let me figure out how to test what I have |
| 11:26 |
berickm |
actually, i could re-run install. not as many people on the network now |
| 11:26 |
* berickm |
does so |
| 11:30 |
berickm |
bshum: i can confirm it's installed twice on 14.04. looks like Jasmine installs 1.9 as a pre-req, then we (intentionally) fetch version 2 |
| 16:14 |
jeff |
drat. i was hoping it was just a failure to do the steps in https://bugs.launchpad.net/evergreen/+bug/1554714/comments/15 |
| 16:14 |
pinesol_green |
Launchpad bug 1554714 in Evergreen "Migrate Browser Client to Angular 1.5 + Dependencies" [Undecided,Fix committed] |
| 16:14 |
jeff |
but now that i re-read that, it seems even less likely (plus, you already ruled that out) |
| 16:16 |
bshum |
jeff: The test build broke, but I was hoping that when it said it compiled without errors, it wasn't lying |
| 16:16 |
bshum |
Guess something was still awry beyond the test |
| 16:17 |
bshum |
I have to build a new trusty VM. Going to see if I get any similar issues on a fresh install there. |
| 16:38 |
|
hbrennan joined #evergreen |
| 16:41 |
|
jihpringle joined #evergreen |
| 16:48 |
|
jlitrell joined #evergreen |
| 20:53 |
jeff |
potential to set threshold of what percentage of translated strings mean we make that language in a build |
| 20:54 |
jeff |
straw poll: +1/0/-1 for experiment of release manger + 1 or more buildmasters for next release? |
| 20:55 |
berick |
+1 to experiment |
| 21:01 |
jeff |
bit of discussion about taking advantage of buildmasters taking load off the RM to enable us to commit to an alpha, beta, rc, etc. |
| 21:01 |
jeff |
encourage more testing. |
| 21:01 |
jeff |
acknowledgement of the previous pattern of "RM announces timeline, dates arrive and people are 'surprised'" |
| 21:03 |
jeff |
some talk of sandboxes for testing alpha/beta/rc/etc |
| 21:04 |
jeff |
general round-the-room consensus is +1 to the experiment |
| 21:04 |
jeff |
miker has volunteered to be RM, and with reduced reluctance due to there being buildmasters |
| 21:05 |
berick |
miker++ |
| 21:08 |
|
bottotr joined #evergreen |
| 21:09 |
jeff |
Jim Keenan has also voiced interest in cat-herding, possibly to co-RM with miker |
| 21:23 |
jeff |
bug squashing days |
| 21:25 |
jeff |
bshum notes that the "webstaff update" and "dev update" are two distinct updates on the agenda tomorrow |
| 21:25 |
berick |
yeah, they gave us a /lot/ of time |
| 21:26 |
* jeff |
nods |
| 21:27 |
jeff |
dev update may be a brief mention of webstaff quasi-beta for circ, desire for more testing, then the "web client update" 30 mins may be 15 mins show-and-tell, 15 mins q&a |
| 21:32 |
jeff |
kmlussier points out importance of covering new features in the two releases since last conference |
| 21:33 |
jeff |
Dyrcona and gmcharlt to cover those |
| 21:34 |
jeff |
berick: are you back for tomorrow at all? i forget. |
| 21:35 |
berick |
jeff: yes, I'll be there |
| 21:35 |
jeff |
got it! |
| 21:35 |
jeff |
berick: we'll hand you some bullet points when you walk in -- you've got 60 minutes to cover. |
| 09:13 |
|
ericar joined #evergreen |
| 09:19 |
|
yboston joined #evergreen |
| 09:32 |
|
maryj joined #evergreen |
| 09:41 |
bshum |
So, I think we may have an issue with tablefunc in PG 9.5, use of the connectby() stuff in permission.grp_ancestors function is broken there. In previous versions, it would return a list back of all the group ancestors, in PG 9.5 it just bombs with some type mismatch error |
| 09:42 |
bshum |
Testing with concerto on PG 9.1 (haha), something like SELECT permission.grp_ancestors (5); throws me back the 1 users, 5 circulators, 3 staff |
| 09:43 |
bshum |
I have to spin up another DB in PG 9.3 to test again there, but I presume it's "fine" |
| 09:44 |
bshum |
For PG 9.5, it goes "ERROR: invalid return type |
| 09:44 |
bshum |
DETAIL: SQL key field type text does not match return key field type integer" |
| 09:44 |
bshum |
So that's what bombs out in PG 9.5 when we try to run transactions.sql for loading concerto data. |
| 09:45 |
bshum |
So far as I can see, permissions.grp_ancestors is one of the only places we use the connectby() function from tablefunc contrib |
| 09:45 |
jeff |
sounds like you've enough for a bug report. :-) |
| 09:45 |
bshum |
Well, so far. |
| 09:45 |
bshum |
jeff: Yep, it's something |
| 13:12 |
|
jihpringle_ joined #evergreen |
| 13:26 |
|
ericar joined #evergreen |
| 13:55 |
pastebot |
"gmcharlt" at 64.57.241.14 pasted "for bshum and miker - a patch for the connectby() usage issue in Pg 9.5" (13 lines) at http://paste.evergreen-ils.org/18 |
| 13:57 |
Dyrcona |
I might have time to test that this weekend. I just got an Ubuntu 16.04 vm created. |
| 13:57 |
Dyrcona |
I am concerned that any fix not break on earlier versions of postgres, so I will try that with a trusty vm also. |
| 13:58 |
gmcharlt |
Dyrcona: it didn't break on 9.4 for me, as a data opint |
| 13:58 |
miker |
Dyrcona: I'd really recommend just using recursive CTEs if you're going to touch them ... those are sure to work on older versions |
| 09:31 |
kmlussier |
pinesol_green: Very good choice! |
| 09:31 |
pinesol_green |
kmlussier: Mr. Spock: Something fascinating just happened. |
| 09:31 |
pinesol_green |
kmlussier: I am only a bot, please don't think I'm intelligent :) |
| 09:33 |
* bshum |
wonders if it makes sense to spin up all new buildbot slaves for http://testing.evergreen-ils.org/buildbot/buildslaves |
| 09:34 |
bshum |
Squeeze is not connected, fedora's been broken for awhile. And Ubuntu 12? Hmm. |
| 09:36 |
Stompro |
I had a dream about the conference last night. It was strange, my brain filled in details about what the conference hotel looks like, and I don't remember any of the sessions. Since I'm not attending this year my mind is trying to compensate. |
| 09:36 |
gmcharlt |
Stompro: dang, I was about to ask if you took notes |
| 09:36 |
kmlussier |
Stompro: You're not going? Sorry to hear it. :( |
| 09:37 |
kmlussier |
Also, that sounds like one of those dreams where it's the end of the school year, and you don't remember taking any of the classes. |
| 09:37 |
Stompro |
I do remember the food being excelent in the dream... there was a lot of standing around in the halls and eating. |
| 09:37 |
kmlussier |
Sounds like a good conference! |
| 09:37 |
bshum |
Also, the live tests have been broken for awhile - http://testing.evergreen-ils.org/~live/test.html -- one of the perl live tests dies cause opt-in isn't enabled in opensrf.xml (sounds like a new test) |
| 09:37 |
Stompro |
Two of my coworkers are going this year, say hello to Sharon and Pauline if you see them. |
| 09:38 |
bshum |
I wonder too if maybe the reason the RSS feed doesn't announce for the live test anymore is cause the new RSS module for pinesol is trying to look for some timestamp for the polling and it's not part of our RSS feed off live test. |
| 09:40 |
|
maryj joined #evergreen |
| 09:40 |
* csharp |
needs to update the fedora host - we should also build an Ubuntu 14.04 one |
| 09:41 |
kmlussier |
bshum: I could file a bug for the broken test, if it hasn't been filed already. |
| 09:41 |
kmlussier |
I know it's been mentioned in channel before. |
| 09:49 |
|
yboston joined #evergreen |
| 09:51 |
bshum |
kmlussier: I don't think it's been filed yet. Test came from https://bugs.launchpad.net/evergreen/+bug/1533329 I think. |
| 09:51 |
pinesol_green |
Launchpad bug 1533329 in Evergreen "Selectively disallow opt-in based on patron's home library" [Wishlist,Fix released] |
| 09:51 |
* bshum |
wanders off to training for the rest of today |
| 09:51 |
* kmlussier |
nods |
| 14:45 |
jihpringle |
does next Thursday work as a deadline for the polls to close or do we need it to be sooner? |
| 14:45 |
yboston |
#info the DIG community server is coming along |
| 14:45 |
remingtron |
gmcharlt++ |
| 14:45 |
yboston |
#link http://docs-testing.evergreen-ils.org |
| 14:45 |
yboston |
I may try to recuit others at DIG to help out |
| 14:46 |
yboston |
should we wrap up for now? |
| 14:46 |
remingtron |
sounds fine |
| 14:47 |
jihpringle |
works for me |
| 14:47 |
yboston |
OK folks, thanks for your attendance |
| 15:03 |
|
mmorgan1 joined #evergreen |
| 15:07 |
berick |
should have asked this during the dev PG discussion the other day.. does EG work w/ PG 9.5? Anyone using it? |
| 15:07 |
|
bmills joined #evergreen |
| 15:08 |
miker |
berick: no reason it shouldn't. 9.3 and 9.4 forced our hand on a few lies we were telling PG |
| 15:09 |
|
mmorgan joined #evergreen |
| 15:10 |
miker |
berick: not that that actually answers your question .... but, IMO, I'd say try on test and if nothing EXPLODES, then you're good |
| 15:10 |
csharp |
especially if nothing explodes in production after go live! :-) |
| 15:10 |
berick |
miker: no, that helps. knowing it should work in theory gets us past the first hurdle. |
| 15:47 |
jeffdavis |
Do the fixes to accommodate PG 9.3+ exist in EG 2.7 though? |
| 09:32 |
csharp |
pinesol_green: nobody thinks you're intelligent - jeez! |
| 09:32 |
pinesol_green |
csharp: I am only a bot, please don't think I'm intelligent :) |
| 09:32 |
pinesol_green |
csharp: Thank you csharp! But our princess is in another castle! |
| 09:32 |
Dyrcona |
VM is ready, I'll be back in about half an hour to say yay or nay if the test works. |
| 09:36 |
|
maryj joined #evergreen |
| 09:36 |
|
yboston joined #evergreen |
| 09:47 |
|
kmlussier joined #evergreen |
| 11:03 |
jeff |
berick++ -- we're using 1.7 still, but I've been chewing through the "breaking changes" docs for 2.3 |
| 11:03 |
rlefaive |
beric++ jeff++ |
| 11:04 |
berick |
jeff: not sure if this is a 2.x thing, but I also had to tweak some of the queries for nested data. E.g. https://github.com/berick/marc-indexing-for-es/commit/665ff895a064ce64e81be5f3809e3f3734e94b4b |
| 11:08 |
jeff |
berick: ah! yep, I likely didn't update the rest of the examples when I added https://github.com/tadl/marc-indexing-for-es/commit/5ae1b4380fecef1fb40f9eb2f79753019ba77053 |
| 11:08 |
jeff |
(heh. that test is pretty... specific to us.) |
| 11:09 |
berick |
jeff: oh, cool, glad I'm on the right track. |
| 11:10 |
berick |
jeff: whoa, you're moving to catalog.apps.tadl.org as your main catalog? |
| 11:11 |
jeff |
not until tomorrow. :-) |
| 15:45 |
* Dyrcona |
pretty much agrees with gmcharlt and miker. |
| 15:46 |
bshum |
PG 9.3 buys us a year to decide what happens next time 'round |
| 15:46 |
gmcharlt |
any disagreement with that (i.e., somebody who wants to aim for 9.4 or 9.5)? |
| 15:46 |
bshum |
Well more than a year anyways |
| 15:46 |
bshum |
Plus I feel like there are already lots of us on PG 9.3 or better |
| 15:46 |
bshum |
So it should be "well tested" |
| 15:46 |
miker |
9.3 FTW |
| 15:47 |
gmcharlt |
going once... |
| 15:47 |
jeffdavis |
FWIW, looks like CentOS 6 (EOL 2020) comes with PG8.4, and CentOS 7 (EOL 2024) comes with PG9.2; not sure if major PG version advances happen in minor CentOS releases |
| 15:53 |
berick |
thanks, dbwells |
| 15:53 |
Dyrcona |
Did we talk about sqitch? Do we need to? |
| 15:53 |
kmlussier |
It was an outstanding action item from the last meeting. |
| 15:54 |
berick |
the sqitch branch is ready for wider testing. |
| 15:54 |
gmcharlt |
I can make some time between now and the EG conference to test |
| 15:54 |
berick |
if nothing else, it would be great if someone else got a feel for how it works |
| 15:54 |
Dyrcona |
I should see what it's about, but I have enough action items already. :) |
| 15:54 |
berick |
so I'm not stabbing too long in the dark. i only have my own experiences |
| 15:56 |
gmcharlt |
berick: shall we set the EG conference as a date for the merge? |
| 15:56 |
miker |
gmcharlt: have we decided it's a done deal? |
| 15:56 |
berick |
miker: that's kind of my question too |
| 15:56 |
gmcharlt |
well, that would depend on testing, ultimately |
| 15:57 |
berick |
gmcharlt: right |
| 15:57 |
gmcharlt |
I think to rephrase: assuming that we end up /agreeing/ to run with sqitch, the conference would be a good time to make the switch |
| 15:57 |
jeff |
we usually have a dev meeting at the conf -- if enough people have tested and gotten a feel for squitch in general we could make a merge/no-merge call then? |
| 15:58 |
miker |
I do a good bit of db mangling ... and I won't have time to really dig in before the conf. I'm certainly not The Decider, but I'd like to hear the pitch (I know I missed one at the hack-a-way ... no way I could have made it there, though) |
| 15:58 |
berick |
i'm happy to review, discuss, etc at the conf. |
| 15:58 |
bshum |
It's a cool pitch. I like the concept, just have to see it in practice more. |
| 16:03 |
bshum |
(before the end!) |
| 16:04 |
Dyrcona |
heh |
| 16:04 |
Bmagic |
#info Bmagic = Blake GH, MOBIUS |
| 16:05 |
gmcharlt |
OK, I got distracted there for a moment |
| 16:05 |
gmcharlt |
but I think we can summarize with |
| 16:06 |
gmcharlt |
#action Various folks will do more detailed evaluation/testing of Sqitch prior to the EG conference, with an eye towards having more informed discussions |
| 16:07 |
berick |
cool, thanks everyone |
| 16:08 |
DPearl |
#info DPearl = Dan Pearl, C/W MARS |
| 16:09 |
dbs |
#info dbs = Dan Scott, Laurentian University |
| 17:12 |
|
mrpeters left #evergreen |
| 17:16 |
Bmagic |
alright, so I get the patron object $r = OpenSRF::AppSession->create('open-ils.actor')->request('open-ils.actor.user.fleshed.retrieve', $authtoken, $r, 0)->gather(1); |
| 17:16 |
Bmagic |
then try to update it $r = OpenSRF::AppSession->create('open-ils.actor')->request('open-ils.actor.patron.update', $authtoken, $r); |
| 17:17 |
Bmagic |
no errors, but the row in the DB doesn't get changed (I skipped a step here where I changed the day_phone $r->day_phone('test'); |
| 17:20 |
Bmagic |
hmmm, maybe it's because there are no addresses in the system for the patron. Looking at the osrfsys.log |
| 17:21 |
|
mmorgan left #evergreen |
| 17:24 |
StomproJosh |
Ahh, I wish I could delete my launchpad comments within a few seconds. sigh. |
| 10:04 |
Dyrcona |
live_t is mainly meant to test that functionality remains as expected, not connectivity. |
| 10:05 |
Bmagic |
editing a patron, stuff like that |
| 10:05 |
Dyrcona |
Bmagic: There are tests in live_t that do some of those things, but as I said they are aimed at concerto and meant to be run from the server. |
| 10:06 |
Dyrcona |
For remote testing, you'll need to connect via the gateway or websockets. |
| 10:07 |
Bmagic |
yeah, I wasn't going to use those tests exactly, but perhaps as a guide. But if using the OpenSRF perl modules do not test the servers as the staff client would, then I can't really use them |
| 10:07 |
kmlussier |
miker / berick: Should bug 1347774 still be open? It looks like code was merged at the end. |
| 10:07 |
pinesol_green |
Launchpad bug 1347774 in Evergreen "Backend logic has leaked into the TPAC (and friends)" [Wishlist,Confirmed] https://launchpad.net/bugs/1347774 |
| 10:07 |
Dyrcona |
Is there a particular issue you are trying to resolve? |
| 10:07 |
Dyrcona |
kmlussier: I really doubt that bug is finished. |
| 10:08 |
Bmagic |
I want to be able to test the servers the same way our libraries do when using the staff client. I need to be able to do this easily and regularly |
| 10:11 |
jeff |
you may find some examples in constrictor. |
| 10:11 |
jeff |
are you attempting to add some monitoring, or do you have a different goal? |
| 10:12 |
Dyrcona |
Bmagic: This might be helpful: http://evergreen-ils.org/~denials/workshop.html#_javascript |
| 10:22 |
Bmagic |
Dyrcona: that web page looks helpful. It does seem like the perl bits rely on OpenSRF perl modules, which in turn rely on direct access to the DB? |
| 10:22 |
Dyrcona |
I usually decide everything is OK when all services are running and I can log in with the staff client. |
| 10:22 |
Dyrcona |
Bmagic: The perl does, but the JavaScript uses the translator or the gateway. That's why I linked to where the JavaScript starts. |
| 10:23 |
Bmagic |
Dyrcona: yes, that is how we test it right now as well, but there are things that we miss, especially when autogen is out of whack. And when there are 8 app servers in rotation, the issue may not be obvious |
| 10:23 |
Dyrcona |
Bmagic: One could write Perl modules to use the gateway, or add it as a transport in the OpenSRF Perl modules, if it isn't there already. |
| 10:24 |
Dyrcona |
Bmagic: Always run autogen.sh after an upgrade. |
| 10:24 |
Bmagic |
of course! |
| 10:24 |
Dyrcona |
The IDL usually changes and if it hasn't, there's no harm in it. |
| 10:25 |
Bmagic |
after running autogen - restart services - check each server and verify. It would be nice if a script could check much more like, editing a patron, circing an item and renewing, etc. Just to be able to closely replicate the expierence of our libraries |
| 10:26 |
Bmagic |
the test could time everything and give real data |
| 10:27 |
Dyrcona |
You can write that with the JavaScript. You could also write it in Perl and run it on an app server. |
| 10:27 |
Dyrcona |
The difference is minimal, though doing it from JavaScript would test if the Apache is working, too. |
| 10:28 |
Bmagic |
sounds promising |
| 10:40 |
Bmagic |
executing JS without a browser - from the command line looks challenging. node.js might be required... |
| 10:43 |
Dyrcona |
You could put it in a html file and open it from a browser. |
| 10:43 |
Dyrcona |
I believe that is what dbs expects one to do with the examples from the workshop. |
| 10:45 |
Bmagic |
yeah, I am trying to think of a way to make this more of an automated thing |
| 10:47 |
dbs |
http://jasmine.github.io/2.4/introduction.html says "It does not depend on any other JavaScript frameworks" |
| 10:48 |
dbs |
Pretty sure we already have some javacript tests in the codebase |
| 10:53 |
|
ericar_ joined #evergreen |
| 10:53 |
dbs |
Oh, heh. Good old Dojo Objective Harness in OpenSRF: http://git.evergreen-ils.org/?p=OpenSRF.git;a=blob;f=src/javascript/tests/README;h=d46224aef890afdfbb0b2a03b6a57477d96ecf43;hb=HEAD |
| 10:55 |
|
ericar__ joined #evergreen |
| 11:03 |
|
Christineb joined #evergreen |
| 11:18 |
|
bshum joined #evergreen |
| 11:23 |
Bmagic |
dbs: thanks, I will take a close look |
| 11:34 |
|
JBoyer joined #evergreen |
| 11:36 |
dbs |
I *think* webby has tests but I haven't touched it for months |
| 11:37 |
Dyrcona |
Yeah, I've been meaning to learn how to use karma to write staff client tests. |
| 11:37 |
Dyrcona |
If that is how its done. |
| 11:40 |
|
brahmina joined #evergreen |
| 11:41 |
|
evergreener joined #evergreen |
| 17:09 |
Bmagic |
I should be |
| 17:14 |
|
mmorgan left #evergreen |
| 17:28 |
sandbergja |
Quick question: does the editor column in asset.copy get modified when a staff usr circulates an item to somebody, or is it only stuff like changing barcode/circmod/etc. that tracks the editor? |
| 17:33 |
kmlussier |
sandbergja: I'm pretty sure the editor column only updates if you make actual edits to the copy, not circulations. But I haven't tested it. |
| 17:37 |
sandbergja |
thanks! I'm trying to track down whodunnit, and that's at least a possible lead |
| 17:47 |
bshum |
sandbergja: Whodunnit like who edited the item last or who checked it out? |
| 17:48 |
bshum |
The auditor table could tell you more either way. auditor.asset_copy_history |
