| 13:40 |
jeff |
alas. |
| 13:43 |
jwoodard |
@librarian |
| 13:43 |
pinesol_green |
jwoodard: Management:14, Cataloging:14, Acquisitions:10, Reference:15, Circulation:5, Systems:16, Research:16, Custodial:14 |
| 13:44 |
berick |
kmlussier: do you still have access to your test server that's running the patron circ history code? bug 1527342 |
| 13:44 |
pinesol_green |
Launchpad bug 1527342 in Evergreen "Maintain patron reading history in a dedicated table." [Wishlist,New] https://launchpad.net/bugs/1527342 |
| 13:44 |
berick |
kmlussier: wondering if you could confirm something for me.. |
| 13:44 |
kmlussier |
berick: yup |
| 15:36 |
miker |
berick++ |
| 15:39 |
gmcharlt |
berick: and to expand on what I meant about a form-builder, basically what I have in mind is doing the equivalents of AutoFieldWidget and EditDialog... just more easily, because angular |
| 15:40 |
berick |
gmcharlt: that would be rad :) |
| 15:41 |
gmcharlt |
berick: working repo commit 535239952d to get a test of what I have in mind; just making it more generic |
| 15:41 |
gmcharlt |
*taste |
| 15:41 |
|
sal_ joined #evergreen |
| 15:41 |
berick |
hm, looks like bug 1206936 busted the baseline schema (premature reference to money.materialized_billable_xact_summary) |
| 15:41 |
pinesol_green |
Launchpad bug 1206936 in Evergreen "money.transaction_billing_summary view displays incorrect billing_type and billing_note for the actual last transaction" [High,Fix committed] https://launchpad.net/bugs/1206936 |
| 15:52 |
* berick |
pushed user/berick/lp1206936-base-schema-repair |
| 15:52 |
berick |
will comment int he ticket too |
| 15:53 |
|
jlitrell joined #evergreen |
| 15:53 |
kmlussier |
berick: I can test that right now |
| 15:53 |
berick |
kmlussier++ |
| 16:15 |
kmlussier |
Looks like it worked. I'll merge it in a few |
| 16:17 |
kmlussier |
Multitasking is not all that it's cracked up to be. |
| 10:03 |
kmlussier |
berick: Thanks! I'm going to look at that branch again in a few minutes once I finish up on some other tasks. |
| 10:03 |
berick |
awesome. i pushed a fix to for the seed data. |
| 10:03 |
kmlussier |
berick++ |
| 10:05 |
berick |
kmlussier: did I understand correctly that you all are testing bug 1333254 as well? |
| 10:05 |
pinesol_green |
Launchpad bug 1333254 in Evergreen "EDI invoices automatically expend debits" [Wishlist,Confirmed] https://launchpad.net/bugs/1333254 |
| 10:07 |
kmlussier |
berick: Yes, we have somebody looking at it now, and I also sent login information to some Sitka folks who were interested in looking at it. |
| 10:07 |
berick |
kmlussier++ |
| 12:40 |
bshum |
berick++ |
| 13:01 |
|
jlitrell joined #evergreen |
| 13:03 |
|
jihpringle joined #evergreen |
| 13:10 |
kmlussier |
Christineb / jihpringle: Are either of you testing acq on the MassLNC VM right now? I was planning to take it down for a few minutes to add an updated branch. |
| 13:10 |
Christineb |
kmlussier: not right now |
| 13:10 |
Christineb |
will be doing more testing this afternoon |
| 13:11 |
Christineb |
kmlussier thanks for heads up :D |
| 13:11 |
kmlussier |
It will be up again in 15 / 20 minutes. |
| 13:11 |
Christineb |
perfect |
| 13:43 |
jeff |
berick: but don't worry, an earlier attempt (before i opened dev tools to see what the heck was going on) loaded enough for me to see which image it was, and it was one i was familiar with. :-) |
| 13:44 |
jeff |
anyway, ads-- |
| 13:44 |
berick |
jeff: oh my |
| 13:45 |
kmlussier |
Christineb: The test server is back up again. |
| 13:45 |
berick |
the internet is a scary place |
| 13:45 |
berick |
(which is kind of what I like about it) |
| 13:45 |
Christineb |
kmlussier: awesome thank you, I plan to finish testing by EOD |
| 13:45 |
kmlussier |
Christineb: You'll need to download a new client too. Sorry. |
| 13:46 |
csharp |
@blame The Internet |
| 13:46 |
pinesol_green |
csharp: The Internet 's bugfix broke csharp's feature! |
| 13:49 |
berick |
@redact [blame The Internet] |
| 13:49 |
pinesol_green |
berick: It really IS The Internet's fault! |
| 14:24 |
dbwells |
grabbing 0959 |
| 14:31 |
pinesol_green |
[evergreen|Chris Sharp] LP#1206936 - Fix wrong billing info in money.transaction_billing_summary - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=6d24cb9> |
| 14:31 |
pinesol_green |
[evergreen|Dan Wells] LP#1206936 Stamping upgrade script; adding test - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=b4415fa> |
| 14:46 |
mmorgan |
I have a number of mark lost trigger events that are stuck in state "collected". If I add a repeatability delay and run the trigger again, will new events be created and these "collected" ones just be ignored? |
| 14:47 |
mmorgan |
Or will these collected ones cause me even more grief than triggers genrally do? |
| 14:47 |
mmorgan |
/genrally/generally |
| 13:52 |
Dyrcona |
Specifically, I'm always seeing more details regardless of the state of the show more/fewer details button. |
| 13:52 |
Dyrcona |
My suspicion is we botched something with a rebase/merge and our customizations, so I will try a plain master in a bit. |
| 13:52 |
Dyrcona |
Thought I'd ask first, though. |
| 13:53 |
Dyrcona |
kmlussier: Are you actively testing on my development vm? |
| 13:54 |
dbs |
Dyrcona: what's show_more_details.default set to? We're at 2.7-ish so our experiences don't matter anyway :/ |
| 13:54 |
kmlussier |
Dyrcona: Other than quickly doing a search to see the problem with fewer details? Nope. |
| 13:54 |
jeff |
Dyrcona: what value do you have for show_more_details.default? true, false, or hide? |
| 21:43 |
|
Topic for #evergreen is now Welcome to the #evergreen library system channel! | We are publicly logged: http://irc.evergreen-ils.org/evergreen | Large pastes at http://paste.evergreen-ils.org |
| 21:46 |
kmlussier |
@later tell Dyrcona are you still planning to look at bug 1013786? |
| 21:46 |
pinesol_green |
kmlussier: The operation succeeded. |
| 22:02 |
* kmlussier |
sees loads of LP bugs with promises from kmlussier to load the code on a VM for testing. |
| 22:02 |
kmlussier |
Empty, empty promises |
| 22:02 |
bshum |
I think priority goes to bugs sponsored by people who offer candy. |
| 22:02 |
bshum |
Or some other tasty treat. |
| 22:02 |
bshum |
But that's just my opinion. |
| 22:02 |
* bshum |
will also test patches for karma |
| 22:04 |
* kmlussier |
has already fed cookies to bshum |
| 22:05 |
kmlussier |
Of course, I do believe bshum was the person who merged my most-recent code. |
| 22:05 |
kmlussier |
But, now that you mention it, nobody ever makes cookies for me. |
| 22:08 |
kmlussier |
bshum: What do you call those? |
| 22:08 |
bshum |
I baked the cupcakes, with assistance from my mother and sister, who got tired of how long I was taking as I painstakingly measured out the batter to each one. |
| 22:09 |
bshum |
kmlussier: Eh, I'm not sure actually. I make them for afterl's christmas parties :P |
| 22:13 |
jlitrell |
bshum++ |
| 22:13 |
jlitrell |
That's for upcoming bug testing, not cupcakes. |
| 22:15 |
kmlussier |
Have a nice night late-night #evergreen folks! |
| 22:40 |
bshum |
kmlussier / jlitrell : re: https://bugs.launchpad.net/evergreen/+bug/1519055 , I can't recall, but wouldn't adding a new attribute definition require reingesting the bibs to find and tag them with the new attribute? |
| 22:40 |
pinesol_green |
Launchpad bug 1519055 in Evergreen "Simple checkbox to exclude electronic resources from search results" [Wishlist,New] - Assigned to Jake Litrell (jake-3) |
| 23:01 |
pinesol_green |
[evergreen|blake] LP#1466990: Detailed search results shows parts for items that dont have parts - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=4b40e68> |
| 23:03 |
pinesol_green |
[evergreen|Galen Charlton] LP#1545178: remove two unused methods - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=49590f0> |
| 23:48 |
bshum |
Calling 0958 |
| 23:53 |
pinesol_green |
[evergreen|Galen Charlton] LP#1505286: limit number of facets retrieved - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=01ffba4> |
| 23:53 |
pinesol_green |
[evergreen|Galen Charlton] LP#1505286: add pgTAP tests - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=24a7665> |
| 23:53 |
pinesol_green |
[evergreen|Galen Charlton] LP#1505286: add release notes - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=9cf7350> |
| 23:53 |
pinesol_green |
[evergreen|Ben Shum] LP#1505286: stamping upgrade script for limit facets retrieved - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=70c1da0> |
| 10:35 |
krvmga |
csharp: i think that may be exactly it. i'm seeing what i can do now. |
| 10:35 |
krvmga |
tsbere++ |
| 10:35 |
krvmga |
csharp++ |
| 10:43 |
kmlussier |
berick: If I wanted to test the code from bug 1333254, I also need to load the code from bug 1380709? |
| 10:43 |
pinesol_green |
Launchpad bug 1333254 in Evergreen "EDI invoices automatically expend debits" [Wishlist,Confirmed] https://launchpad.net/bugs/1333254 |
| 10:43 |
pinesol_green |
Launchpad bug 1380709 in Evergreen "invoice print amounts-per-fund uses wrong value when item price varies" [Medium,Confirmed] https://launchpad.net/bugs/1380709 |
| 10:44 |
kmlussier |
Actually, that code already has a signoff. I should just take a quick look at it and merge it. |
| 10:47 |
|
Christineb joined #evergreen |
| 10:49 |
|
rjackson_isl joined #evergreen |
| 11:03 |
|
mmorgan left #evergreen |
| 11:11 |
pinesol_green |
Showing latest 5 of 6 commits to Evergreen... |
| 11:11 |
pinesol_green |
[evergreen|Galen Charlton] LP#1367926: release notes - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=b3b433d> |
| 11:11 |
pinesol_green |
[evergreen|Galen Charlton] LP#1367926: add some unit tests - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=8e60e0a> |
| 11:11 |
pinesol_green |
[evergreen|Galen Charlton] LP#1367926: various improvements and bugfixes - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=f0ac3da> |
| 11:11 |
pinesol_green |
[evergreen|Mike Rylander] LP#1367926: Expose non-MARC bre data via U2 - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=b1892a6> |
| 11:11 |
pinesol_green |
[evergreen|Mike Rylander] LP#1367926: Add support for (nearly) direct access to the full unapi backend - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=4fbeac6> |
| 11:12 |
|
vlewis joined #evergreen |
| 11:15 |
dbwells |
As I think Dyrcona mentioned earlier, it seems like that commit plugin got reversed or something. It shows the first give commits backwards, when I feel like it used to show the last five commits, ordered oldest to newest. Doesn't matter much, just throwing it out in case anyone who can wanted to look into it. |
| 11:15 |
dbwells |
Possible my memory is off, but in any case, it isn't doing what it claims to do :) |
| 11:20 |
kmlussier |
Speaking of our IRC plugins, the one that used to show the results of our QA tests still doesn't seem to be hooked up. |
| 11:20 |
kmlussier |
Just in case anyone wants to look at that one while looking at the commit plugin. ;) |
| 11:23 |
jeff |
@rss announce |
| 11:23 |
pinesol_green |
jeff: Error: Couldn't get RSS feed. |
| 11:23 |
jeff |
ah. |
| 11:23 |
jeff |
well, that's a problem. |
| 11:24 |
jeff |
@rss qatests |
| 11:24 |
pinesol_green |
jeff: 11:24 AM, February 18, 2016: Test Success - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 11:24 |
jeff |
@rss announce list |
| 11:24 |
pinesol_green |
jeff: qatests |
| 11:24 |
|
sandbergja joined #evergreen |
| 11:25 |
jeff |
@rss info qatests |
| 11:25 |
pinesol_green |
jeff: Title: Test Output Summary; URL: <http://testing.evergreen-ils.org/~live/test.html>; Description: Live Test Suite; Last updated: time unavailable. |
| 11:25 |
jeff |
okay, todo list entry created. |
| 11:26 |
kmlussier |
jeff++ |
| 11:30 |
Bmagic |
anyone have a general idea of why an item (in item status) on the holds tab will say "not captured for a hold, however its status is incorrectly set to "on holds shelf" please check in this item to correct the status." |
| 11:34 |
tsbere |
Bmagic: I suspect that someone cleared the shelf but didn't check the book in |
| 11:35 |
Bmagic |
tsbere: yeah, they couldn't find the book |
| 11:35 |
* tsbere |
generally prefers the clear shelf expired holds checkin modifier these days |
| 11:38 |
kmlussier |
Heh, just noticed we have three live-t tests with the number 10 |
| 11:39 |
|
bmills joined #evergreen |
| 11:42 |
kmlussier |
Does it matter if we have duplicate test numbers in that directory? I ask because I was just about to merge a branch that will add another duplicate. |
| 11:42 |
Dyrcona |
I don't think it does matter, but it is not esthetically pleasing. :) |
| 11:45 |
kmlussier |
Well, maybe I'll change the number for the one I'm merging, and if others want to clean up the others for aesthetic reasons, they can. :) |
| 11:45 |
Dyrcona |
Sounds good to me. |
| 11:55 |
pinesol_green |
[evergreen|Kathy Lussier] LP#1380709 Renaming live-t test to avoid duplicate numbers - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=5c3d3e3> |
| 11:55 |
pinesol_green |
[evergreen|Bill Erickson] LP#1380709 Perl live test - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=136b6b1> |
| 11:55 |
pinesol_green |
[evergreen|Bill Erickson] LP#1380709 acq.lineitem_detail.fund_debit INDEX - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=9f8444f> |
| 11:55 |
pinesol_green |
[evergreen|Bill Erickson] LP#1380709 fund_debit.invoice_entry INDEX - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=81804c9> |
| 11:55 |
pinesol_green |
[evergreen|Bill Erickson] LP#1380709 Fund debit links to invoice entry - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=94ec1e1> |
| 11:58 |
berick |
kmlussier++ |
| 11:58 |
berick |
I'll rebase 1333254 |
| 12:00 |
kmlussier |
I think I'm going to put a few acq branches on one server and hand it over to our acq folks. |
| 13:12 |
Christineb |
dluch anything you can do is appreciated :D |
| 13:14 |
dluch_ |
Ok. :-) |
| 13:15 |
Christineb |
#action dluch christineb jihpringle to review current development list as time permists |
| 13:15 |
Christineb |
#topic Review existing acq bugs needing testing |
| 13:15 |
|
afterl joined #evergreen |
| 13:16 |
Christineb |
3 Acq bugs on Launchpad with pullrequest tag - https://docs.google.com/spreadsheets/d/1iEWNjR3yhluIA5ViLZAj_QANEmaKlJ5L8JMubtO_mZc/edit?usp=sharing |
| 13:18 |
jihpringle |
does anyone have the ability/time/resources to test any of these bugs? |
| 13:18 |
jihpringle |
I think if they were tested and signed off they would have a better chance of making it into the next release |
| 13:19 |
|
montgoc1 joined #evergreen |
| 13:19 |
jihpringle |
I would love to see 1333254 make it into 2.10 (it would make year end so much easier) but we don't currently have a test server we can test it on |
| 13:21 |
jihpringle |
if they don't make it into 2.10 we'll want to make sure they get included in bug squashing day so they can get included in 2.11 (or whatever comes next) |
| 13:21 |
Christineb |
#topic New Business |
| 13:21 |
Christineb |
anyone have anything new to discuss? |
| 13:22 |
jihpringle |
Leslie from PINES wasn't able to attend today but had a couple questions |
| 13:46 |
Danielle2 |
mllewellyn: Ok that makes a lot of sense-we get ours within 15 minutes of Activating the PO. |
| 13:47 |
Christineb |
There will be no formal Acquisitions meeting at the 2016 Evergreen Conference |
| 13:47 |
Christineb |
But we would like to schedule a time for an In-person meeting, similar to last year where we met during lunch or break period |
| 13:48 |
kmlussier |
Sorry to interrput, but back to the question above regarding testing of the acq bugs with pull request tags (http://bit.ly/1XAcriq), I just loaded the code for all three on a MassLNC Sandbox. If anyone wants to test, let me know and I'll send you login details. |
| 13:48 |
jihpringle |
kmlussier++ |
| 13:48 |
Christineb |
kmlussier++ |
| 13:48 |
Christineb |
you are the best!!!!! |
| 13:48 |
mllewellyn |
kmlussier ++ |
| 13:48 |
jihpringle |
I will definitely test if you send me login details |
| 13:48 |
kmlussier |
OK. great! jihpringle++ |
| 13:49 |
jihpringle |
thanks :) |
| 13:49 |
Christineb |
I can test as well, I won't have time until next week though |
| 13:49 |
Christineb |
#action kmlussier to send jihpringle christineb MassLNC sandbox login details |
| 13:50 |
kmlussier |
Christineb++ |
| 13:50 |
Christineb |
#action jihpringle Christineb to test 3 acq bugs with pull requests https://docs.google.com/spreadsheets/d/1iEWNjR3yhluIA5ViLZAj_QANEmaKlJ5L8JMubtO_mZc/edit#gid=1659472283 |
| 13:51 |
|
rfrasur joined #evergreen |
| 13:52 |
dluch_ |
informal gathering/meeting at conference would be nice |
| 13:52 |
jihpringle |
there are already a few sessions for lunch and breakfast - https://docs.google.com/spreadsheets/d/1cgXALUY3pDP_Bt2TeRRm0IKekZOet0jPFLHEkOV2CpQ/edit#gid=1589049103 |
| 16:37 |
Bmagic |
kmlussier - yes and no, I've had to put it on the back burner for awhile |
| 16:37 |
Bmagic |
kmlussier: if someone else is wanting to take it for a ride, go for it |
| 16:37 |
kmlussier |
Bmagic: Not yet, but maybe later. I'll let you know. |
| 16:38 |
Bmagic |
I had some code that you tested on the last bug sqaushing day |
| 16:38 |
Bmagic |
kmlussier: that addressed another bug, but they are related, in fact might be the same |
| 16:39 |
Bmagic |
or at least the same code would solve both |
| 16:39 |
kmlussier |
Bmagic: Really? I thought they were different issues. |
| 09:58 |
pinesol_green |
Launchpad bug 1544606 in Evergreen "tpac translations broken in 2.9" [High,Confirmed] https://launchpad.net/bugs/1544606 |
| 10:09 |
|
mrpeters joined #evergreen |
| 10:24 |
Dyrcona |
kmlussier && anyone else who uses it: my regular development vm is being shut down and a new concerto vm will take its place. |
| 10:24 |
Dyrcona |
I'll be testing leading up to a 2.9.2 release. |
| 10:28 |
Dyrcona |
Hrm.... Maybe I need to fix our training/development db server first.... |
| 10:32 |
|
abowling joined #evergreen |
| 10:36 |
Dyrcona |
Looks like it ran out of free disk space last night. |
| 11:41 |
|
bmills joined #evergreen |
| 11:44 |
kmlussier |
Dyrcona / berick: I'm just getting started on point release notes now. I should have those done shortly. |
| 11:44 |
Dyrcona |
kmlussier++ |
| 11:45 |
Dyrcona |
I'm still working on testing bshum's i18n fixes for tpac/webstaff. |
| 11:45 |
berick |
kmlussier++ |
| 11:45 |
Dyrcona |
I ran into an unexpected delay with our second database server. |
| 11:45 |
|
jwoodard joined #evergreen |
| 15:55 |
berick |
10, I think |
| 15:56 |
berick |
actualy, no, 14 |
| 15:56 |
gmcharlt |
berick: Koha uses a bcrypt variant with 8 iterations |
| 15:56 |
jeff |
In your and dbwells' testing, what is the wallclock difference in time required to log in? |
| 15:57 |
berick |
jeff: quick test shows .1 seconds to 1 second (roughly) |
| 15:57 |
berick |
for just the auth calls combined |
| 15:57 |
kmlussier |
berick: Is that an OPAC login? |
| 15:57 |
berick |
i.e. via srfsh |
| 15:57 |
berick |
opac will take a little longer w/ API overhead |
| 15:04 |
|
mmorgan left #evergreen |
| 15:06 |
|
jeff_ joined #evergreen |
| 15:14 |
|
lualaba joined #evergreen |
| 15:14 |
lualaba |
hello time to time on my server processes died |
| 15:14 |
lualaba |
* timed out waiting on open-ils.acq pid=37626 to die * cannot signal open-ils.acq: process 37626 is not running * sending TERM signal to pid=37604 router * sending TERM signal to pid=37605 router |
| 15:15 |
lualaba |
after restart osrf everyrhing is working |
| 15:19 |
lualaba |
this is in logs: evergreen CRON[50526]: (root) CMD (test -x /usr/sbin/anacron || ( cd / && run-parts --report /etc/cron.daily )) |
| 15:23 |
jeff |
lualaba: that log entry is expected -- it's cron running daily tasks. it is possible that you are running out of memory, in which case you'll probably have a recent note to that effect in the output of dmesg or in the file /var/log/dmesg (and likely some other places as well) |
| 15:23 |
jeff |
lualaba: how much memory does the machine have? |
| 15:27 |
lualaba |
2 GB ram |
| 15:41 |
lualaba |
Thank you, there is any suggestion to disable some of services or jobs? |
| 15:59 |
|
jvwoolf1 joined #evergreen |
| 16:02 |
Dyrcona |
lualaba: If you're not using acq, you could probably disable that. There are not that many you can actually disable without causing problems. |
| 16:02 |
Dyrcona |
lualaba: You could also configure fewer minimum drones and maximum drones. I've done that for test and development servers. |
| 16:04 |
* Dyrcona |
would like to make it so that circ does not depend on booking, for instance. |
| 16:04 |
* Dyrcona |
had an idea how to do it once, but ran out of tuits. |
| 16:04 |
lualaba |
how fewer minimum drones and maximum drones? |
| 16:08 |
Dyrcona |
lualaba: You don't want to set <min_children> below 1, though. |
| 16:08 |
Dyrcona |
miker: I think we tried once and circ blew up. |
| 16:08 |
Dyrcona |
miker: I know there are calls to open-ils.booking in there. give me a minute. |
| 16:09 |
miker |
related to Dyrcona's advice (for the room), I have a branch that needs testing that teaches C apps how to respect {min|max}_spare_servers, like perl does (or is supposed to) |
| 16:10 |
miker |
Dyrcona: yeah, there are def calls, but I thought we had fenced those off, and there was even a setting at one time to say "booking is not running, don't even try to use it" |
| 16:10 |
Dyrcona |
lualaba: Also, I would keep <max_children> at or above 2 also. |
| 16:11 |
lualaba |
i check and in most cases parameter is 15 |
| 16:11 |
lualaba |
<min_children>5</min_children> <max_children>15</max_children> |
| 16:17 |
Dyrcona |
I'll give it a try next time I build a new branch on my development vm. |
| 16:17 |
miker |
I had the advantage of helping senator with that particular feature. and dbwells was involved as well, I think... |
| 16:18 |
Dyrcona |
lualaba: Yeah, those are defaults. I've found in some production environments they need to be higher. |
| 16:19 |
Dyrcona |
lualaba: If you're testing/kicking the tires, you can probably drop most of the <max_children> settings. |
| 16:19 |
lualaba |
4 GB ram will be enough |
| 16:19 |
Dyrcona |
I have numbers for how many drones are spawned on our production servers, but not for training or development. |
| 16:19 |
lualaba |
tomorrow i will try increase nd after that will change settings |
| 16:20 |
Dyrcona |
lualaba: 4GB is cutting it close with the database running on the same hardware. |
| 16:20 |
lualaba |
this mean i should move DB on another server? |
| 16:21 |
Dyrcona |
It should work with the right tuning. If you don't want to test booking or acq, I'd out right disable those services. |
| 16:21 |
miker |
lualaba: for production, almost certainly. or add a lot of CPU cores to the VM (if it's a VM) |
| 16:21 |
lualaba |
yes it is VM |
| 16:21 |
Dyrcona |
Oh, right. I would not run production on 4GB. |
| 16:22 |
Dyrcona |
I use 6GB for development and I do run the database on is own server. |
| 16:22 |
Dyrcona |
Granted, that database server also runs our training and other test databases. |
| 16:23 |
Dyrcona |
What you need for production depends a lot on your size and how many transactions you do. |
| 16:23 |
|
mmorgan joined #evergreen |
| 16:23 |
lualaba |
Thank you for information i will try some of changes in opensrf.xml |
| 05:16 |
csharp |
@later tell bshum re: bug 1486592... I don't see any reason why it shouldn't be targeted for 2.10 as long as someone tests/signs off |
| 05:16 |
pinesol_green |
csharp: The operation succeeded. |
| 06:40 |
|
rlefaive joined #evergreen |
| 06:52 |
bshum |
csharp: Sure, I had a feeling that would be the case, but since you put needstest on it, I assumed it was still work in progress of some sort. I just don't like bugs without a milestone cause they're harder to filter for. |
| 07:46 |
csharp |
bshum: is there a better way than "needstest" to indicate "all this is waiting for is a test and signoff"? |
| 07:46 |
|
mrpeters joined #evergreen |
| 07:49 |
|
rjackson_isl joined #evergreen |
| 07:54 |
|
JBoyer joined #evergreen |
| 07:58 |
|
rlefaive joined #evergreen |
| 08:24 |
|
ericar joined #evergreen |
| 08:29 |
|
Dyrcona joined #evergreen |
| 08:35 |
bshum |
csharp: "needstest" is the tag that is meant for bugs that require additional tests to be added, like unit test, pgTap, etc. |
| 08:36 |
csharp |
bshum: ohhhhh |
| 08:36 |
csharp |
ok - got it |
| 08:36 |
bshum |
It doesn't mean test me, that's what "pullrequest" is for |
| 08:36 |
csharp |
aha - okay - my bad |
| 08:37 |
bshum |
No worries, at least I think that's what it means. Hopefully I haven't been using it wrong either.. :) |
| 08:38 |
csharp |
I'm sure you're right about that |
| 08:59 |
|
collum joined #evergreen |
| 09:01 |
|
RoganH joined #evergreen |
| 09:19 |
|
maryj joined #evergreen |
| 09:38 |
kmlussier |
Ah, that explains things. I was perplexed as to why the new Concerto data needed a test. ;) |
| 09:39 |
|
yboston joined #evergreen |
| 09:42 |
tsbere |
kmlussier: To ensure that it is in fact the data that is expected! ;) |
| 09:48 |
Dyrcona |
Thank you, Rhythmbox for sorting Vivaldi under A for Antonio. |
| 10:44 |
pinesol_green |
maryj missed something. |
| 10:47 |
Dyrcona |
So, it boils down to our production being affected by lp 1414112. |
| 10:47 |
pinesol_green |
Launchpad bug 1414112 in Evergreen 2.7 "Audience filter no longer searching blank spaces" [Medium,Fix released] https://launchpad.net/bugs/1414112 |
| 10:47 |
Dyrcona |
I will test on training. |
| 10:52 |
Dyrcona |
Yes, it is fixed on our training server which is running 2.9.1. |
| 10:53 |
Dyrcona |
So, I can say, "fixed at our next update." |
| 10:54 |
tsbere |
I get to be happy that I don't have to change the code I wrote that is involved. ;) |
| 12:21 |
|
mllewellyn joined #evergreen |
| 12:26 |
|
afterl joined #evergreen |
| 12:48 |
|
jvwoolf joined #evergreen |
| 12:56 |
jeffdavis |
I'm getting a perplexing error from a Perl live test I'm working on: Can't locate object method "create_actor_org_unit_setting" via package "OpenILS::Utils::CStoreEditor" at live_t/12-lp1533329-opt-in.t line 80. |
| 12:56 |
jeffdavis |
http://git.sitka.bclibraries.ca/gitweb/?p=sitka/evergreen.git;a=blob;f=Open-ILS/src/perlmods/live_t/12-lp1533329-opt-in.t;h=690167a;hb=4987b9d |
| 12:57 |
Dyrcona |
jeffdavis: You need to either init CStoreEditor or the Fieldmapper or both. |
| 12:58 |
jeffdavis |
Dyrcona: I think I am initializing CStoreEditor correctly: |
| 12:58 |
jeffdavis |
my $e = new_editor(xact => 1); |
| 16:09 |
tsbere |
current_copy is for "captured" and "pull list" lookup, basically. The hold copy maps cover opportunistic capture. I don't think prev_check_time should stay null, though... |
| 16:10 |
dbs |
ahcm should be populated when hold_targeter is run, and ahr.prev_check_time should be set when hold_targeter is run too |
| 16:10 |
JBoyer |
dbs opportunistic holds kind of run in reverse, because what you're checking is for holds at the location the item is returned to, not the location that owns it, etc. like the pull lists care about. (Still doesn't explain why that wasn't targeted though. :( ) |
| 16:10 |
dbs |
okay. I'll check on that in a future test when I have a few minutes |
| 16:10 |
mmorgan |
dbs: one more shot in the dark - Did the hold specify mint_condition = true, and the copy mint_condition = false? |
| 16:11 |
dbs |
But our circ staff have filed tons of tickets about books getting checked in, and then a few hours later they show up on the holds shelf pull list, rather than flagged at checkin time |
| 16:11 |
dbs |
mmorgan: lemme see |
| 11:43 |
Dyrcona |
I mostly poke around with PgAdmin to see what's what. |
| 11:43 |
lualaba_ |
same here :) |
| 11:43 |
lualaba_ |
but i spent a lot of time |
| 11:44 |
Dyrcona |
Also, I highly recommend having a test/development database set up so you don't break production. |
| 11:44 |
Dyrcona |
Yes, it is time consuming. |
| 11:44 |
lualaba_ |
what i need now is to know relations between tables |
| 11:45 |
Dyrcona |
I think someone made a relational chart of the database once, but I could have hallucinated that. |
| 12:43 |
Dyrcona |
The ones with the $9 by itself the value of $9 is the consortium shortname. |
| 12:44 |
Dyrcona |
Some how, I don't think that is enough to make it show up, but I'll do a search before changing the flag. |
| 12:45 |
Dyrcona |
And, I'm naturally logged in at the wrong library. :) |
| 12:45 |
Dyrcona |
But, it's a good test anyway. :) |
| 12:46 |
Dyrcona |
So, yep, the $9 MVC by itself doesn't do anything. The $u has to be there. |
| 12:50 |
jeff |
were you hoping to use/abuse the feature to have some records that didn't have URIs show up somewhere? |
| 12:50 |
jeff |
or just testing an edge case? |
| 12:53 |
Dyrcona |
jeff: Just testing what it does in general. |
| 12:53 |
* jeff |
nods |
| 12:53 |
Dyrcona |
We have a couple buckets of electronic records that catalogers/reference staff want things done to, so I'm testing some of those things. |
| 12:54 |
Dyrcona |
I'm not sure where the $9 MVC by itself comes, probably cataloging staff. |
| 12:54 |
Dyrcona |
I've not mangled the records for real, yet. :) |
| 12:54 |
Dyrcona |
Someone else was probably trying to get them to show up. |
| 15:16 |
pinesol_green |
[evergreen|Dan Wells] LP#1526547 Prevent bogus notes when adjusting lost/lod overdues - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=98477d3> |
| 15:18 |
kmlussier2 |
snow-- |
| 15:31 |
|
jvwoolf1 left #evergreen |
| 16:08 |
berick |
kmlussier: re: bug 1474051 did you confirm your org unit table had more values before applying the patch? that almost sounds like an install without the seed/test data installed. |
| 16:08 |
pinesol_green |
Launchpad bug 1474051 in Evergreen "Avoid storing partial credit card payment info" [Wishlist,Triaged] https://launchpad.net/bugs/1474051 |
| 16:09 |
kmlussier |
berick: I'm using tsbere's script that merges the patch before the install. But I've done other installs today on the same machine without any problem. There's also other seed data in the tables. |
| 16:12 |
berick |
kmlussier: grr, I see the problem. base schema is missing a "," |
| 06:35 |
jeff |
commit a5d2ba32 |
| 06:36 |
jeff |
maybe a working repo commit? i don't recall if pinesol_green reacts to those. |
| 06:40 |
|
rlefaive joined #evergreen |
| 06:44 |
kmlussier |
hmmmm |
| 06:46 |
kmlussier |
chreeus: On bug 1541801, we noticed the same issue in the web client when we were testing Z39.50. |
| 06:46 |
pinesol_green |
Launchpad bug 1541801 in Evergreen "search fields in z39.50 sort in random order" [Undecided,New] https://launchpad.net/bugs/1541801 |
| 06:47 |
kmlussier |
chreeus: I wonder if work on the web client caused the change in the xul client interface |
| 07:02 |
chreeus |
kmlussier: that's our theory |
| 07:03 |
bshum |
chreeus, jeff: right pinesol doesn't track working |
| 07:04 |
bshum |
I think it was mostly a matter of avoiding double reporting since it would likely spit back master changes in both repos |
| 07:05 |
bshum |
Never quite tested beyond that. |
| 07:06 |
kmlussier |
Using working to look up commits would be useful. Getting an announcment every time somebody pushes a branch to working? Not so much. |
| 07:06 |
bshum |
Hmm |
| 07:10 |
bshum |
kmlussier: Well what would happen is that it would only announce master changes |
| 07:18 |
pinesol_green |
bshum: evg-working (Evergreen-Working, branch: master) |
| 07:18 |
pinesol_green |
bshum: opensrf (OpenSRF, branch: master) |
| 07:18 |
pinesol_green |
bshum: sipserver (SIPServer, branch: master) |
| 07:18 |
bshum |
I'm going to test an idea |
| 07:18 |
bshum |
and try asking it to track another branch for announce |
| 07:18 |
bshum |
if it works anyways |
| 07:19 |
bshum |
@git repolist |
| 07:19 |
pinesol_green |
bshum: evergreen (Evergreen, branch: master) |
| 07:19 |
pinesol_green |
bshum: evg-working (Evergreen-Working, branch: master) |
| 07:19 |
pinesol_green |
bshum: opensrf (OpenSRF, branch: master) |
| 09:33 |
|
yboston joined #evergreen |
| 09:35 |
|
maryj joined #evergreen |
| 09:44 |
JBoyer |
Dyrcona++ |
| 09:45 |
JBoyer |
Thanks for the help yesterday re: deadlocks, we were able to load 100K+ records in an hour or so in our test, migrations here are going to look at a lot different in the future. |
| 09:45 |
JBoyer |
look a lot. I try not to look directly into the operational end of a migration in progress.. |
| 09:46 |
jeff |
JBoyer: ``Do not look into laser with remaining good eye.'' |
| 09:47 |
JBoyer |
I feel like I should know that, but it's not coming to me. |
| 10:25 |
JBoyer |
Completely unrelated news: This morning I tracked down the application guide for a temp/humidity sensor I've been looking for and I haven't been this excited to play around with code since I was playing with OpenGL in my college dorm room. |
| 10:32 |
|
collum joined #evergreen |
| 11:28 |
|
sandbergja joined #evergreen |
| 11:30 |
pinesol_green |
[opensrf|Mike Rylander] LP#1494486: Limit damage caused by dropped drone XMPP sockets - <http://git.evergreen-ils.org/?p=OpenSRF.git;a=commit;h=5580724> |
| 11:36 |
pinesol_green |
[opensrf|Jason Etheridge] LP#1474507: fix interval_to_seconds for weeks and seconds - <http://git.evergreen-ils.org/?p=OpenSRF.git;a=commit;h=69cbe80> |
| 11:36 |
pinesol_green |
[opensrf|Jason Etheridge] LP#1474507: tests for interval_to_seconds - <http://git.evergreen-ils.org/?p=OpenSRF.git;a=commit;h=7a714ae> |
| 11:40 |
|
mllewellyn joined #evergreen |
| 11:59 |
|
bmills joined #evergreen |
| 12:01 |
kmlussier |
@coffee |
| 12:40 |
|
mllewellyn joined #evergreen |
| 12:40 |
|
Bmagic joined #evergreen |
| 12:51 |
JBoyer |
Ah, makes sense. |
| 13:12 |
pinesol_green |
[opensrf|Galen Charlton] LP#1350457: add test case for perl2JSONObject change - <http://git.evergreen-ils.org/?p=OpenSRF.git;a=commit;h=b6cf3eb> |
| 13:12 |
pinesol_green |
[opensrf|Mike Rylander] LP#1350457: Pass caller's session to subrequests called via method_lookup - <http://git.evergreen-ils.org/?p=OpenSRF.git;a=commit;h=e1581d4> |
| 13:36 |
* chreeus |
wonders if the addition of UPC to the fields in commit 2038e93a caused weird sorting in bug 1541801 |
| 13:36 |
pinesol_green |
Launchpad bug 1541801 in Evergreen "search fields in z39.50 sort in random order" [Medium,Confirmed] https://launchpad.net/bugs/1541801 |
| 13:36 |
pinesol_green |
[evergreen|Josh Stompro] LP#1519925 - Allow MARC Federated Search to search UPC index of local catalog. - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=2038e93> |
| 14:35 |
yboston |
shoudl we make it an action item to have a DIG member check with the devs about what is productionr eady for 2.10? |
| 14:36 |
remingtron |
I can't remember what gmcharlt (the release manager) said his goals were |
| 14:36 |
remingtron |
but I think it was only circ desk use. |
| 14:36 |
jihpringle |
will there be a web client test server for 2.10? We're not planning to release the web client to our libraries until we make the final switch so it won't be part of our test server |
| 14:36 |
jihpringle |
^^ community web client test server for 2.10 |
| 14:38 |
remingtron |
Here's gmcharlt's "plan for 2.10": |
| 14:38 |
remingtron |
#link http://georgialibraries.markmail.org/thread/wlnvqe53umjjevae |
| 14:38 |
yboston |
we can hope that the "webby" server would get updated for us to use |
| 14:39 |
remingtron |
I think yes, someone should probably ask gmcharlt or whomever has power over webby |
| 14:39 |
remingtron |
yboston++ |
| 14:40 |
jihpringle |
yboston++ |
| 14:40 |
gmcharlt |
well, it won't be webby, which is primarily for testing by the funding partners for the web staff client sprints, but we can update the ESI-hosted demo server to 2.10 closer in |
| 14:40 |
yboston |
#action yboston will check in with 2.10 release manager (gmcharlt ) about havign Webby ready for us to use and confirmation of what is produciton ready in the web client |
| 14:41 |
gmcharlt |
strike that action item, please, per my statement above |
| 14:41 |
yboston |
damm I typed too slow |
| 14:55 |
yboston |
or somethign along those lines |
| 14:55 |
yboston |
thoughts? |
| 14:55 |
sandbergja |
Sounds good to me |
| 14:55 |
remingtron |
yboston: do we already have a test DIG server? |
| 14:55 |
krvmga |
good idea. |
| 14:55 |
yboston |
remingtron: it is not set up |
| 14:55 |
gmcharlt |
indeed, that's available |
| 14:59 |
dluch |
yboston++ |
| 14:59 |
jihpringle |
yboston++ |
| 15:00 |
Christineb |
yboston++ |
| 15:00 |
chreeus |
dbs: I'm creating a PG 9.3 test VM to test your implicit sorting theory - it sounds like it would explain everything |
| 15:05 |
kmlussier |
yboston++ |
| 15:06 |
Stompro |
Stompro |
| 15:06 |
* Stompro |
oops |
| 15:16 |
chreeus |
kmlussier: is it 9.4? |
| 15:16 |
jeff |
ah, see... i went with differing perl versions, didn't think about differing postgres versions. :-) |
| 15:17 |
kmlussier |
chreeus; Nope 9.3.10 |
| 15:17 |
chreeus |
hmm |
| 15:17 |
chreeus |
my non-random-sorting server is PG 9.3 |
| 15:18 |
chreeus |
9.3.10 |
| 15:18 |
chreeus |
argh |
| 15:20 |
|
bshum joined #evergreen |
| 15:22 |
|
bshum joined #evergreen |
| 15:23 |
chreeus |
ok - gonna build a precise VM with PG 9.4 to test the perl version issu |
| 15:23 |
chreeus |
e |
| 15:25 |
|
bshum joined #evergreen |
| 15:27 |
jeff |
chreeus: what are the perl versions in question? |
| 15:27 |
jeff |
oh, i see you provided that elsewhere. |
| 15:28 |
kmlussier |
chreeus: I'm updating the bug, but I was on 5.18.2 as well |
| 15:28 |
|
jvwoolf joined #evergreen |
| 15:29 |
jeff |
Perl 5.18 introduced major changes in hashes, including changes that affect order. |
| 15:29 |
jeff |
and randomization. |
| 15:29 |
jeff |
see http://perldoc.perl.org/perl5180delta.html |
| 15:30 |
jeff |
under "Hash overhaul" |
| 15:30 |
|
jlitrell joined #evergreen |
| 15:30 |
jeff |
iirc, we hit those changes with some tests, which needed to be adjusted slightly. |
| 15:32 |
kmlussier |
I've updated the bug and now try remembering exactly what it was I was working on today. |
| 15:32 |
|
dkyle1 left #evergreen |
| 15:33 |
jlitrell |
An excellent time to interrupt you with exclusion checkboxes. :-D |
| 16:38 |
jeff |
my gut reaction to the idea is rather curmudgeonly. |
| 16:39 |
jeff |
it might be possible, moreso now with the ability to fetch jackets by record id. i don't know if the receipt templates have been prevented from pulling remote images now or not. |
| 16:46 |
jeff |
jihpringle: are you doing this in the self checkout interface with success? |
| 16:46 |
jihpringle |
I just had a library tell me that they print out on the self check items out receipt |
| 16:46 |
jihpringle |
I need to test myself to double check |
| 16:48 |
jeff |
i'm curious if they're printing in greyscale on a receipt printer, or printing to a letter sized sheet on a laser/inkjet, black and white or color, etc. |
| 16:52 |
jihpringle |
I'll do some testing and find out more details from the library |
| 17:00 |
|
jvwoolf left #evergreen |
| 17:05 |
|
mmorgan left #evergreen |
| 17:12 |
kmlussier |
Good night #evergreen! |
| 10:58 |
kmlussier |
Dyrcona: Nope |
| 10:58 |
|
ericar_ joined #evergreen |
| 11:00 |
Dyrcona |
OK. I plan to start up my precise vm to update the software, then maybe make a new vm for concerto. |
| 11:00 |
Dyrcona |
I use the latter to test tarballs and run the perl and db tests. |
| 11:03 |
Dyrcona |
BTW, I plan to finally release 2.9.2 on the 17th. |
| 11:16 |
JBoyer |
I wonder how many users actually make use of wildcard searching, vs those that think that you really do have to search for M*A*S*H to find the DVD you want... |
| 11:16 |
Dyrcona |
JBoyer: I'm going to build just master on my concerto vm to see if the grunt all command still fails. |
| 11:31 |
kmlussier |
JBoyer: Does it work if you surround it in quotes? |
| 11:32 |
Dyrcona |
I don't think I would enter MASH that way, and I usually search in lowercase, 'cause the shift key is hard. :) |
| 11:32 |
kmlussier |
Actually, it works with and without quotes |
| 11:33 |
JBoyer |
It's quicker with the quotes, but neither of my tests took 90+ seconds like the search that was in my log analyzer... |
| 11:34 |
JBoyer |
I'm with Dyrcona, it just seems like such a tedious thing to type. What do these people say out loud? "What are you looking for?" "Emm, star, aye, star, ess, star, aich." |
| 11:34 |
kmlussier |
JBoyer: Yeah, I tried it on C/W MARS. The unquoted one was longer, but it wasn't terrible. |
| 11:35 |
JBoyer |
My test was fine too, there must have been something else about when they searched, or the phase of the moon, etc. |
| 11:36 |
JBoyer |
The person that searched for the entire copy/pasted contents of a Google Play book search page must have had some interesting ideas. |
| 11:36 |
* Dyrcona |
should automate the prerequisite installation for OpensRF and Evergreen on his VMs. It will save time without typos. |
| 11:37 |
* JBoyer |
knows a guy... |
| 11:54 |
tsbere |
Stompro: Fair note, that is (obviously) a little dated, and doesn't take into account details like parts being flagged as deleted. |
| 11:54 |
tsbere |
So if you have that code loaded there are adjustments that need to be made |
| 11:55 |
Stompro |
I don |
| 11:55 |
Dyrcona |
I think grunt test failed, but I'll try it manually in a bit. |
| 11:56 |
Dyrcona |
It scrolls by so fast. |
| 11:56 |
Stompro |
tsbere, I don't think that 2.8.4 has the parts deletion bits, no deleted field in biblio.monograph_parts here. |
| 11:57 |
tsbere |
Stompro: Specifically, the "SELECT INTO existing_part id FROM... line would need to have "AND NOT deleted" added to the where clause when you do have the flag. |
| 12:10 |
Dyrcona |
It stats with couldn't load egCoreWeb because of the above. |
| 12:11 |
Dyrcona |
gmcharlt: Fresh checkout of master. |
| 12:11 |
Dyrcona |
Fresh vm. |
| 12:11 |
Dyrcona |
That happens with grunt test. |
| 12:12 |
Dyrcona |
The test output is "this should parse the IDL" |
| 12:14 |
gmcharlt |
Dyrcona: OK, I think I see what the problem is, one moment |
| 12:14 |
berick |
if it's a new dependency, it may need to be added to the test manifest |
| 12:15 |
gmcharlt |
ayup |
| 12:17 |
gmcharlt |
Dyrcona: check the tip of collab/gmcharlt/webstaff-sprint2-sprint3-round2 |
| 12:19 |
|
bmills joined #evergreen |
| 12:21 |
Dyrcona |
gmcharlt++ that resolves it. |
| 12:21 |
Dyrcona |
Should I just cherry-pick that into master since it is a fix for a test? |
| 12:21 |
|
_bott_ joined #evergreen |
| 12:28 |
berick |
Dyrcona: +1 to cherry-pick |
| 12:28 |
gmcharlt |
+1 |
| 12:29 |
gmcharlt |
Dyrcona++ |
| 12:29 |
Dyrcona |
breick++ |
| 12:29 |
Dyrcona |
berick++ |
| 12:31 |
pinesol_green |
[evergreen|Galen Charlton] webstaff: add angular-file-saver to test manifest - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=e32a1d4> |
| 12:35 |
csharp |
@karma breick |
| 12:35 |
pinesol_green |
csharp: Karma for "breick" has been increased 1 time and decreased 0 times for a total karma of 1. |
| 12:35 |
csharp |
@karma chreeus |
| 13:28 |
pinesol_green |
Launchpad bug 1499086 in Evergreen 2.9 "Slowness/timeout on loading bookbags in OPAC" [Medium,Triaged] https://launchpad.net/bugs/1499086 |
| 13:36 |
|
jihpringle joined #evergreen |
| 13:46 |
csharp |
new_glasses++ |
| 13:48 |
bshum |
jeffdavis: If as you're working through and see stuff that will be signed off or tested as part of 2.10 I would go ahead and update the bug |
| 13:49 |
bshum |
Otherwise, it's just extra churn to update the milestone for work that may go unfinished and end up back on 2.next or later |
| 13:51 |
* Dyrcona |
agrees with bshum. |
| 13:51 |
jeffdavis |
Makes sense, thanks. |
| 14:12 |
JBoyer |
Is anyone using the migration tools to load bibs in parallel? We're working on a local method because a single process isn't getting it done, but we're hitting deadlocks on occasion. I know there's src/extras/import/parallel_pg_loader.pl but I don't think it's ready to use as-is for the output of the migration tools. |
| 17:14 |
gsams |
mmorgan: I appreciate it, I'd still be interested because it might be relevant for another library if nothing else |
| 17:16 |
mmorgan |
ok will dig up the info tomorrow. Good night all! |
| 17:16 |
|
mmorgan left #evergreen |
| 17:58 |
jeffdavis |
So I'm in the middle of writing a Perl live test for the opt-in feature. |
| 17:58 |
jeffdavis |
One of the test steps is to opt in a patron with open-ils.actor.user.org_unit_opt_in.create. |
| 17:59 |
jeffdavis |
I want to return to a "clean" state when the testing is done, so I need to delete the opt-in during my test. |
| 18:00 |
jeffdavis |
But there's no method for deleting opt-ins, and I can't delete arbitrary records with something like json_query (it only does SELECT statments, IIRC). |
| 18:00 |
jeffdavis |
So I'm thinking the easiest solution is just to add a new open-ils.actor.user.org_unit_opt_in.method to EG. |
| 18:00 |
jeffdavis |
Which we ought to have anyway. |
| 18:00 |
jeffdavis |
Here ends a case study in why tests are a Good Thing. |
| 18:01 |
jeffdavis |
(a new open-ils.actor.user.org_unit_opt_in.delete method, that is) |
| 18:08 |
dbwells |
jeffdavis: or pcrud? (not familiar with the feature, just throwing it out there) |
| 18:09 |
* dbwells |
is not familiar with opt-in. He knows about pcrud :) |
| 18:10 |
jeffdavis |
dbwells: pcrud ought to work, unless there's some reason live tests can't use it. |
| 18:10 |
jeffdavis |
I just like the idea of tests giving me an excuse to add a new feature. :) |
| 18:16 |
miker |
jeffdavis: pcrud or cstore should work fine in live tests |
| 18:16 |
miker |
fwiw |
| 18:16 |
dbwells |
Well, live tests can do about whatever they want, so even cstore's an option. |
| 18:16 |
dbwells |
right :) |
| 18:17 |
jeffdavis |
good to know |
| 18:20 |
dbwells |
We've got plenty of CRUD methods hanging around in EG, so not a big deal, but if simple client access is something desirable, think of pcrud as a good way to _not_ add a new feature :) |
| 18:21 |
* dbwells |
disappears |
| 08:22 |
Dyrcona |
pinesol_green: Got any laters for me? |
| 08:22 |
pinesol_green |
Dyrcona: Have you confirmed your ISBN SPIDs with your service provider? |
| 08:22 |
pinesol_green |
Dyrcona: I am only a bot, please don't think I'm intelligent :) |
| 08:25 |
kmlussier |
Dyrcona: No laters, but gmcharlt mentioned last night that collab/gmcharlt/webstaff-sprint2-sprint3-round2 is ready for testing. |
| 08:36 |
Dyrcona |
OK. Looks like i use the round2 branch. |
| 08:37 |
|
mrpeters joined #evergreen |
| 08:38 |
jeff |
and we have reached the point in the morning where i wish i had a rolling packet capture on all SIP2 traffic. |
| 14:37 |
tsbere |
May need some adjusting for you as I hardcoded a bib source or two in the bibs sans call numbers query |
| 14:38 |
* tsbere |
has that (and some other things) running nightly for MVLC, though I think we have upped the period variable to 90 days instead of 30 |
| 14:39 |
kmlussier |
gsams: I just checked. Transferring items to volumes will delete the volume too if it's left empty. |
| 14:40 |
gsams |
kmlussier++ #I appreciate the quick test! |
| 14:40 |
|
jlitrell joined #evergreen |
| 14:40 |
kmlussier |
gsams: I just happened to be in holdings maintenance when you asked the question. :) |
| 14:41 |
tsbere |
I know empty volumes still crop up for us. Probably from failed creates and other things. |
| 14:43 |
* jeff |
tries to sort out his feelings for call numbers like "DVD AVE RATED PG-13" |
| 14:43 |
Bmagic_ |
yeah, lol |
| 14:44 |
|
RoganH_ joined #evergreen |
| 14:46 |
kmlussier |
OK, everything looks okay on Dyrcona's test system. I think I'm ready to merge the sprint2/sprint 3 branch |
| 14:47 |
kmlussier |
Thank you, cat, for hitting the Enter key before I was done. |
| 14:47 |
tsbere |
kmlussier: The cat apparently thought it was a good place to stop. |
| 14:47 |
kmlussier |
Does the branch need an LP bug before I merge it? |
| 14:48 |
kmlussier |
tsbere: Yeah, he then hit All Caps afterwards, which is why it took me a while to pose my next question. |
| 15:30 |
csharp |
and that's obviously not what the feature was built to accommodate |
| 15:31 |
csharp |
yeah, I'll play with that in the short term |
| 15:31 |
tsbere |
csharp: If you want standing penalties to generally be at the top level then set the depths to 0. I think that will generally work across the board. |
| 15:31 |
csharp |
tsbere: cool - I'll do some testing |
| 15:31 |
tsbere |
We set many of them that way, if you want to poke me on some of that |
| 15:31 |
csharp |
thanks! |
| 15:32 |
tsbere |
We also have some "System-level" versions that staff can set for more local issues (blocking another library's patron from their library, for example) |
| 09:26 |
|
maryj joined #evergreen |
| 09:42 |
|
bwicksall joined #evergreen |
| 09:44 |
kmlussier |
Dyrcona++ gmcharlt++ |
| 09:47 |
Dyrcona |
kmlusser: I might have to build a new branch after the rebase if you want to continue testing on my development system. |
| 09:48 |
Dyrcona |
If that's the case, do you want to just load the sprint2-sprint3 branch, and maybe the patron editor branch? |
| 09:53 |
kmlussier |
Dyrcona: Yes, that's fine. That's all I'm focusing on ATM |
| 09:55 |
Dyrcona |
I could reload the database in the meantime, and probably should. |
| 10:02 |
|
mmorgan left #evergreen |
| 10:16 |
Dyrcona |
kmlussier: Would want the hotkeys branch? |
| 10:16 |
Dyrcona |
Should be a "you" in there somewhere. |
| 10:17 |
kmlussier |
Dyrcona: I'll probably be testing that soon after the sprint2 branch goes in, so yes. |
| 10:17 |
* kmlussier |
checks to see if there is even a pullrequest tag on that one. |
| 10:18 |
kmlussier |
gmcharlt: Do you want a pullrequest on bug 1508477? |
| 10:18 |
pinesol_green |
Launchpad bug 1508477 in Evergreen "browser client: hotkeys don't work if an input element has focus" [Wishlist,New] https://launchpad.net/bugs/1508477 |
| 10:18 |
kmlussier |
It's worked very well in my testing on webby. Even if there is still work left to be done, it would be nice to have it in before 2.10 |
| 10:20 |
Dyrcona |
I can always add it again later. |
| 10:21 |
gmcharlt |
kmlussier: sure |
| 10:23 |
Dyrcona |
I just figured that I put the branch together with what's ready so I could run upgrade scripts before hand. |
