Time |
Nick |
Message |
00:43 |
|
Mark__T joined #evergreen |
04:21 |
|
mtcarlson_away joined #evergreen |
04:21 |
|
b_bonner joined #evergreen |
04:22 |
|
mnsri_away joined #evergreen |
04:22 |
|
rashma_away joined #evergreen |
06:40 |
|
rlefaive joined #evergreen |
07:33 |
|
rjackson_isl joined #evergreen |
08:09 |
|
ericar joined #evergreen |
08:35 |
|
mrpeters joined #evergreen |
08:35 |
|
Dyrcona joined #evergreen |
08:40 |
|
gsams joined #evergreen |
08:43 |
|
kmlussier joined #evergreen |
08:57 |
kmlussier |
Good morning #evergreen. Happy Friday! |
08:59 |
jeff |
happy friday to you as well! |
09:12 |
|
ericar joined #evergreen |
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 |
09:46 |
bshum |
jeff: Probably not horrible :) |
09:46 |
tsbere |
bshum: Copy the org unit version of the function for a base for replacing it? |
09:47 |
bshum |
tsbere: I wondered about that, I'll have to check it out |
09:47 |
bshum |
I just found that it uses it to check user permissions too |
09:48 |
bshum |
In permission.usr_has_perm_at_all_nd , maybe |
09:48 |
bshum |
I'm still re-reading the file |
10:03 |
|
rlefaive joined #evergreen |
10:53 |
|
Christineb joined #evergreen |
10:57 |
JBoyer |
"How many SNMP OIDs can we cram into this switch?" "I don't know, maybe 4-500?" "Nah, I bet we can stuff 49000 in there, easy." |
11:10 |
jeff |
The snmpwalking dead. |
11:12 |
|
sandbergja joined #evergreen |
11:20 |
|
bmills joined #evergreen |
11:37 |
|
ericar joined #evergreen |
11:38 |
|
bmills joined #evergreen |
11:53 |
JBoyer |
jeff++ |
11:59 |
csharp |
@band add The SNMPwalking Dead |
11:59 |
pinesol_green |
csharp: What we have here is a failure to communicate. |
12:03 |
JBoyer |
pinesol_green let go of it's dreams of ever being in a cool indie band, it seems only zoia still aspires to rock out. |
12:03 |
pinesol_green |
JBoyer: http://images.cryhavok.org/d/1291-1/Computer+Rage.gif |
12:03 |
pinesol_green |
JBoyer: I am only a bot, please don't think I'm intelligent :) |
12:04 |
|
jihpringle joined #evergreen |
12:09 |
|
kmlussier joined #evergreen |
12:12 |
jeff |
Okay, web staff client question.. and nevermind. |
12:12 |
jeff |
found the bug i was looking for: bug 1467663 :-) |
12:12 |
pinesol_green |
Launchpad bug 1467663 in Evergreen "Cannot login to web staff client if work station does not exist in database" [Medium,New] https://launchpad.net/bugs/1467663 |
12:15 |
|
brahmina joined #evergreen |
12:23 |
|
bmills joined #evergreen |
12:23 |
|
bmills1 joined #evergreen |
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 |
13:59 |
Dyrcona |
gmcharlt: Thanks. I'll try it on 9.3 in a bit. |
13:59 |
gmcharlt |
yeah, I agree with miker - the change to usage of connectby() in my suggestion makes it more correct, but isn't necessary the best appraoch |
13:59 |
Dyrcona |
miker: OK. |
13:59 |
* miker |
runs away to a meeting |
14:00 |
gmcharlt |
miker: I note that if we ditch connectby() in 006.schema.permissions.sql, it looks like we could drop the tablefunc dependency entirely |
14:01 |
Dyrcona |
gmcharlt: I like that idea. ;) |
14:06 |
csharp |
@band add Tablefunc Psychedelic |
14:06 |
pinesol_green |
csharp: I see nothing, I know nothing! |
14:07 |
pastebot |
"Dyrcona" at 64.57.241.14 pasted "Recursive CTE" (9 lines) at http://paste.evergreen-ils.org/19 |
14:07 |
csharp |
pinesol_green: but I want you to have a @band plugin! |
14:07 |
pinesol_green |
csharp: did you finish your beer? |
14:07 |
pinesol_green |
csharp: I am only a bot, please don't think I'm intelligent :) |
14:07 |
Dyrcona |
That's a paste from a script I wrote back in September to adjust permissions of certain patrons. |
14:07 |
Dyrcona |
I could use that as a start, yeah? |
14:21 |
miker |
Dyrcona: you could. might be simpler to just taking the relevant org unit versions and s/actor.org_unit/permission.grp_tree/ |
14:22 |
Dyrcona |
miker: OK. |
14:23 |
|
ericar joined #evergreen |
14:29 |
|
ericar joined #evergreen |
15:53 |
|
akilsdonk joined #evergreen |
15:53 |
|
miker joined #evergreen |
15:53 |
|
phasefx_ joined #evergreen |
15:56 |
|
jlitrell joined #evergreen |
16:04 |
Dyrcona |
Looks like the npm registry problems are resolved. |
16:04 |
* Dyrcona |
wonders if it is still worth opening a LP bug. |
16:05 |
|
ldw joined #evergreen |
16:05 |
jeff_ |
A "we might need to install npm on some distros from not-the-distro-package" bug or something else? |
16:06 |
Dyrcona |
jeff_: I'm not sure any more. :) |
16:07 |
|
jeff joined #evergreen |
16:07 |
|
jeff joined #evergreen |
16:24 |
|
rhamby joined #evergreen |
16:28 |
|
bmills joined #evergreen |
16:39 |
jeffdavis |
Dyrcona: I got the impression that we should not be relying on nodejs packages anyway, so I think still worth opening a bug for that |
16:39 |
Dyrcona |
jeffdavis: yeah, I get that impression from the comments on the github bug. |
16:39 |
Dyrcona |
Maybe this weekend..... |
16:44 |
jeffdavis |
I wonder if we want to contemplate version pinning for non-packaged dependencies as well (I think dbs mentioned this in passing the other day?) |
16:47 |
jeffdavis |
at least for official releases |
16:49 |
Dyrcona |
Probably, but I don't know enough about npm registry right now to do that. |
16:57 |
|
jeff_ joined #evergreen |
19:31 |
|
lualaba joined #evergreen |
19:31 |
lualaba |
Hello in which config file i can change opac theme name? |
19:47 |
jeffdavis |
bug 1568195 |
19:47 |
pinesol_green |
Launchpad bug 1568195 in Evergreen ""cannot pass more than 100 arguments" error with actor.org_unit_ancestor_setting_batch" [Undecided,New] https://launchpad.net/bugs/1568195 |
19:47 |
jeffdavis |
(I feel like I often report these things on Friday afternoons after everyone's gone home :S ) |
19:55 |
bshum |
Happens |
19:58 |
jeffdavis |
lualaba: Is this what you are looking for? http://docs.evergreen-ils.org/2.9/_how_to_override_templates.html |
19:59 |
lualaba |
thank you |
20:00 |
jeffdavis |
You're welcome, hope it helps! |
20:24 |
gmcharlt |
jeffdavis: so clearly, you should pick up BC and move it to the east coast! ;) |
20:32 |
jeffdavis |
ha! |
22:22 |
|
geoffsams joined #evergreen |
22:24 |
|
Librator joined #evergreen |