| Time |
Nick |
Message |
| 07:10 |
|
collum joined #evergreen |
| 08:46 |
|
mmorgan joined #evergreen |
| 09:48 |
|
Dyrcona joined #evergreen |
| 10:01 |
|
mmorgan1 joined #evergreen |
| 10:06 |
|
collum joined #evergreen |
| 10:27 |
Dyrcona |
I just got his from eg_db_config: psql:load_all.sql:120: ERROR: insert or update on table "org_unit_setting_type_log" violates foreign key constraint "org_unit_setting_type_log_field_name_fkey" |
| 10:27 |
Dyrcona |
DETAIL: Key (field_name)=(lib.info_url) is not present in table "org_unit_setting_type". |
| 10:30 |
|
Christineb joined #evergreen |
| 10:30 |
Dyrcona |
It might be something in the branch I'm working on. I switch to main and do eg_db_config with --load-all-sample and it doesn't happen. |
| 10:32 |
csharp_ |
re: the pcrud perms stuff in bug 1847805 |
| 10:32 |
pinesol |
Launchpad bug 1847805 in Evergreen "pcrud search can fail to retrieve rows that the user has access to" [High,Fix released] https://launchpad.net/bugs/1847805 |
| 10:33 |
csharp_ |
seeing that having that enabled (per opensrf.xml) is keeping fine generation from happening |
| 10:33 |
csharp_ |
in 3.16 |
| 10:34 |
csharp_ |
for now, we've turned it off - I'm trying to get my head around what's happening but the fine generator script doesn't have any references to a "user" that we could adjust perms on |
| 10:34 |
Dyrcona |
Yeahp. My issues is a syntax error in 950.data.seed-values.sql |
| 10:35 |
csharp_ |
it fails with "request error open-ils.cstore.direct.actor.user_standing_penalty.delete" with "<400> Invalid object or insufficient permissions" |
| 10:36 |
csharp_ |
disabling the new feature allows it all to work |
| 10:36 |
csharp_ |
so... is this me being dense about what's required by the new feature or is this a bug? |
| 10:38 |
Dyrcona |
csharp_: Is the patch for Lp 2132031 also applied? |
| 10:38 |
pinesol |
Launchpad bug 2132031 in Evergreen "SQL Problems Applying Fix From 1847805" [Critical,Fix released] https://launchpad.net/bugs/2132031 |
| 10:38 |
csharp_ |
lemme look |
| 10:39 |
csharp_ |
should be in there, we're on 3.16.3 |
| 10:39 |
Dyrcona |
I think there was another related bug about IDL fields that are sQL keywords, like "asc." |
| 10:40 |
csharp_ |
yeah, I did work on that one |
| 10:41 |
Dyrcona |
Yeah, and I misremembered. That's about moving the PCRUD checks into the database wasn't it? |
| 10:43 |
csharp_ |
that was bug 2138747 |
| 10:43 |
pinesol |
Launchpad bug 2138747 in Evergreen "Generated SQL with unquoted class names" [High,Fix released] https://launchpad.net/bugs/2138747 |
| 10:45 |
Dyrcona |
My first thought is it was something like that. The fine generator doesn't run as a particular user does it? This sounds like maybe a permission was added to an IDL entry. |
| 10:45 |
csharp_ |
your question tells me that it's Not Just Me™, so that's helpful |
| 10:45 |
csharp_ |
I'll keep digging in |
| 10:46 |
Dyrcona |
Well, it might just be you. I haven't tried the fine generator on 3.16, yet, that I recall. |
| 10:47 |
csharp_ |
Dyrcona: also affecting users applying bill payments, I think |
| 10:48 |
Dyrcona |
OK. That seems about right. |
| 10:50 |
Dyrcona |
My problem turned out to be an extra comma. |
| 10:51 |
Dyrcona |
Too bad they all can't be that obvious. :) |
| 11:01 |
Dyrcona |
ugh. Looks like I may need a perl test so I can check things in and clear holds to test these setting values. |
| 11:02 |
Dyrcona |
Thought that I could get away with pgtap tests. |
| 11:06 |
|
collum joined #evergreen |
| 11:36 |
jeff |
csharp_: do you have any local ON DELETE rules in place on actor.usr_standing_penalty? |
| 11:43 |
jeff |
csharp_: this sounds like something I ran into the other day, which (in my case, at least) was due to that. There is a check in the new code to ensure that a row was deleted, and it special cases soft delete tables that have a "deleted" column, but in our case we were force-archiving some standing penalties with an ON DELETE rule on the table, and that was unexpected enough that it broke. |
| 11:44 |
jeff |
csharp_: I mentioned it in IRC last week, but there's not much additional context there other than the specific commit hash: http://irc.evergreen-ils.org/evergreen/2026-01-26#i_589075 |
| 11:45 |
jeff |
csharp_: so I'd be curious to know if you have any rules in place on actor.usr_standing_penalty. It's also possible that you're running into a slightly different issue, I think there were a few places where that error was possible. |
| 11:50 |
pinesol |
News from commits: LP2115525 OPAC Prefs Page Accessibility Updates <http://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=569b982c6d88899c5b76161e29560b88abe439e1> |
| 11:50 |
jeff |
I suppose another possibility with the fine generator could be a parallelization thing, if there's a situation where trying to delete an ausp object that no longer exists was a common thing that safely failed before, and now raises OSRF_STATUS_BADREQUEST. |
| 11:59 |
|
jihpringle joined #evergreen |
| 13:11 |
|
Lorne joined #evergreen |
| 14:43 |
|
Lorne joined #evergreen |
| 15:50 |
pinesol |
News from commits: LP1868173 Don't "upload" when re-importing queue <http://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=ff0e7ba2bdfb716ef45acd05263e0708da293f54> |
| 16:53 |
Bmagic |
I'm wondering if I should just merge bug 2140411 ? |
| 16:53 |
pinesol |
Launchpad bug 2140411 in Evergreen "950 seed data Single Sign on for Evergreen Staff Client" [Undecided,New] https://launchpad.net/bugs/2140411 |
| 17:07 |
|
mmorgan left #evergreen |
| 17:21 |
pinesol |
News from commits: LP#2120735: Improve the monograph parts grid in the staff catalog <http://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=669d3e6de726129da9ac976f8888d2f6168e0999> |
| 17:46 |
|
jihpringle joined #evergreen |
| 18:08 |
|
jihpringle18 joined #evergreen |
| 22:19 |
|
Lorne joined #evergreen |
