Open Source Integrated Library System

Evergreen on IRC

#evergreen Logs for Monday, April 9th, 2012

< Sunday, April 8th, 2012Raw Log FileTuesday, April 10th, 2012 >
#TimeNickMessage
#00:03:43artunit has quit IRC
#00:04:28artunit has joined #evergreen
#01:33:26artunit has quit IRC
#01:34:44artunit has joined #evergreen
#02:04:10artunit has quit IRC
#02:04:51artunit has joined #evergreen
#03:03:40artunit has quit IRC
#03:05:01artunit has joined #evergreen
#04:04:23artunit has quit IRC
#04:05:11artunit has joined #evergreen
#04:16:42Gentlecat has joined #evergreen
#04:32:53artunit has quit IRC
#04:35:06artunit has joined #evergreen
#05:03:54artunit has quit IRC
#05:05:07artunit has joined #evergreen
#05:31:54Gentlecat has quit IRC
#05:33:29artunit has quit IRC
#05:35:10artunit has joined #evergreen
#06:34:40artunit has quit IRC
#06:35:43artunit has joined #evergreen
#06:44:36dmagick_ has joined #evergreen
#06:49:24fortin has joined #evergreen
#07:14:26fortin has quit IRC
#07:34:23artunit has quit IRC
#07:35:26artunit has joined #evergreen
#07:39:17Neil__ has joined #evergreen
#07:48:12collum has joined #evergreen
#07:56:21fortin has joined #evergreen
#08:03:00artunit has quit IRC
#08:03:33akilsdonk has joined #evergreen
#08:05:36artunit has joined #evergreen
#08:23:58hopkinsju has quit IRC
#08:31:06mrpeters-islshould I be converting GSoC submissions to branches, or asking the students to do that themselves?
#08:31:28mrpeters-islor are patches being accepted?
#08:32:59tsbereI think patches are being accepted. If you want to gitify them (and sign off on them yourself) don't forget the author option ;)
#08:33:07mrpeters-islhaha yes!
#08:33:16artunit has quit IRC
#08:33:20gmcharltmrpeters-isl: yeah, you've missed the GSoC application deadline ;)
#08:33:25tsbereUnless they are git compatible patches. Then just run them through git as-is to get the author information.
#08:34:03gmcharltbut seriously, if you do gitify patches for them, that would be great, but please also point them to the documentation of the relevent procedures
#08:34:10Neil__ has left #evergreen
#08:34:35mrpeters-islyeah, thats what i had started doing, but i figured i could gitify them and signoff at the same time, at least on some of the easy one
#08:35:08tsbere has minimal problem with gitify as a means of signing off on on a patch, but agrees that pointing people at git instructions is a good thing
#08:35:41artunit has joined #evergreen
#08:35:45denialsIf we're looking at cutting 2.2beta2 soon, I think I'll fix up that problem in 950.sql, update the README to specifiy --create-database-9.1 (or make 9.1 the default for --create-database and swap to --create-database-9.0 given that we're recommending 9.1...)
#08:36:25tsberedenials: Uhhh, --create-database with version numbers? The raw SQL file reference maybe, but not command line options.
#08:37:06tsbereThe command line option to eg_db_config auto-detects 9.0 and earlier compared to 9.1 and later already
#08:37:35timhome has joined #evergreen
#08:37:48tsberedenials: Otherwise, +1 to fixing that stuff
#08:38:32tsberedenials: If you want to play basic test and sign-off dances let me know.
#08:38:56denialstsbere: psql -d postgres -qtc 'show server_version; only works for limited scenarios, right?
#08:39:24denialslike, situations in which you don't need a username, password, or host to connect to the database?
#08:39:53denialsoh - hmm, ENV
#08:39:56tsberedenials: Ummm.....If you are pulling that from the perl file, the host, username, and password are exported
#08:40:39tsbereyay for "lets not put passwords in places that show in running process list information" ;)
#08:40:42denials was following up on a comment bshum had made about create_database_9_1...
#08:41:04denialsall hail tsbere
#08:41:07tsberedenials: I think there is a point in there for "standalone DB server setup" that likely talks about the SQL files
#08:41:36tsberedenials: Don't hail me for that. Someone else was doing that already. If I put code like that in it was because it was already being done elsewhere. <_<'
#08:43:09gdunbar has joined #evergreen
#08:43:13denialsyeah, we say " For PostgreSQL 9.1 and later you should be
#08:43:13denialsable to continue to use the --create-database flag on eg_db_config.pl, without
#08:43:17denialsneeding to install any server modules on your application machine."
#08:43:45denialsBut apparently bshum ran into something, or else he wouldn't have mentioned that. Oh well, wait until bshum shows up I guess
#08:46:27tsberemaybe bshum ran into "didn't read the README" ;)
#08:47:21denialsIf it were anyone but bshum, maybe :)
#08:47:38tsberedenials: Any more thoughts on https://bugs.launchpad.net/evergreen/+bug/974638 ?
#08:47:38pinesol_greenLaunchpad bug 974638 in Evergreen "EG 2.2 beta1's client MARC edit view gives syntax error" (affected: 1, heat: 6) [Undecided,New]
#08:47:44mrpeters-islbshum is on his way to the office
#08:48:05tsbere goes to mark https://bugs.launchpad.net/evergreen/+bug/832836 as a duplicate of that one....going with "the one with more useful information should stick around"
#08:48:05pinesol_greenLaunchpad bug 832836 in Evergreen "Marc Editor - FIXME error" (affected: 2, heat: 10) [Undecided,Confirmed]
#08:58:35bshumdenials: Still home actually, troubleshooting a holds issue, upgrade madness, whee!
#08:58:50bshumdenials: In the README for 2.2 in the section "Creating the database on a remote server"
#08:59:09bshumIt still reads: " Copy the Open-ILS/src/sql/Pg/create_database.sql script "
#08:59:25bshumWhich is wrong, you really want the create_database_9_1.sql if you're using PG 9.1
#08:59:45bshumThough yes, I did read the eg_db_config.pl piece and would have used that except that I didn't have our application servers ready at the time.
#09:01:02bshumThough I guess, I read it wrong anyways, it says "For PostgreSQL 9.0, you can either..."
#09:01:16fortin has quit IRC
#09:01:19bshumSo I guess that is right for PG 9.0 and I shouldn't be tinkering with stuff at 11 pm :)
#09:03:38artunit has quit IRC
#09:04:33Meliss has joined #evergreen
#09:05:47artunit has joined #evergreen
#09:06:42denialsupdated a fix for bug 957453
#09:06:42pinesol_greenLaunchpad bug 957453 in Evergreen "Mix of explicit event definition IDs and CURRVAL results in unhappiness" (affected: 1, heat: 6) [Critical,New] https://launchpad.net/bugs/957453
#09:08:13denialsbshum: I could separate out the "For PostgreSQL 9.0..." line into its own paragraph; that would probably scan better
#09:14:30tsbere grabs that launchpad bug for testing/merging
#09:16:51denialsbshum: pushed a tiny fix for the README that will hopefully clarify things
#09:17:21tsberedenials: rel_22?
#09:17:33denials headdesks
#09:17:49tsbere is already en route to branch removal
#09:17:55denialsrel_2_2
#09:17:56denials:)
#09:18:10tsbererel_22 removed
#09:18:22tspindler has joined #evergreen
#09:23:35denials would really like to see bug 965907 integrated
#09:23:35pinesol_greenLaunchpad bug 965907 in Evergreen "Integrate TPAC i18n support into regular infrastructure" (affected: 1, heat: 6) [High,New] https://launchpad.net/bugs/965907
#09:29:23tsbere rebuilds his dev machine DB a couple of times to make sure that errors do, in fact, go away
#09:32:17denialstsbere: cool. I just pushed an updated user/dbs/tpac_i18n to use the latest strings for tpac.pot (there were a couple of changes due to serials and such)
#09:33:13artunit has quit IRC
#09:33:15denials needs to deal with a tiny sql-extraction issue where db.seed-i18n.py doesn't grok "(1 , 'foo', 'bar', 'frak')" due to the space after the comma
#09:35:08mrpeters-isl has quit IRC
#09:35:22mrpeters-isl has joined #evergreen
#09:35:46artunit has joined #evergreen
#09:37:30kmlussier has joined #evergreen
#09:39:44denialsuser/dbs/tpac_dbseed_tolerance
#09:39:51denials(will open a bug shortly)
#09:44:23denialsbug 977223
#09:44:23pinesol_greenLaunchpad bug 977223 in Evergreen "i18n: db-seed-i18n.py doesn't deal with spaces in key values for oils_18n_gettext()" (affected: 1, heat: 6) [Undecided,New] https://launchpad.net/bugs/977223
#09:45:30jenny has joined #evergreen
#09:49:17tsbere fights with update issues
#09:49:20tsbere<_<
#09:58:19hopkinsju has joined #evergreen
#10:03:39artunit has quit IRC
#10:04:31AaronZ-PLS has joined #evergreen
#10:04:56berkleeintern has joined #evergreen
#10:05:50Gentlecat has joined #evergreen
#10:06:44artunit has joined #evergreen
#10:11:41tsberequestion: Anyone with an updated master system having issues with the report writer?
#10:11:51tsbereor whatever you want to call that part of the system
#10:13:47mrpeters-islclark kent you mean?
#10:13:59tsbereStaff client interface, specifically
#10:14:07mrpeters-islwhat kind of problem...ill test
#10:14:23tsberemrpeters-isl: Unable to make and/or load folders seems to be my primary right now
#10:14:37tsbereNo, wait, a make just worked
#10:14:53tsbereLoading still busted for me, though <_<
#10:15:10mrpeters-islclean system, or production?
#10:15:14tsbereBoth.
#10:15:30tsbereGetting "Error: folder.owner().id is not a function" in the JS Console
#10:15:32mrpeters-isli've had this issue before. it was a problem with folder ownership. Callender helped me
#10:15:45mrpeters-islyep, i know exactly what it is...let me find his little SQL spell
#10:16:44tsberemrpeters-isl: Looks 100% fine in the DB. owner is 1, I am logged in as user ID 1, no parent because it shouldn't have one, etc.
#10:17:00mrpeters-islselect * from reporter.template_folder where shared is true and parent in (select id from reporter.template_folder where shared is false);
#10:17:27mrpeters-islsometimes a folder somewhere in the tree is shared wrong and it will hit that, then stop loading
#10:17:43tsberemrpeters-isl: Also, the clean system has no shared folders, nor any parent folders. It has ONE FOLDER OWNED BY THE SUPERUSER, and that folder isn't loading. For the superuser.
#10:17:50mrpeters-islok
#10:18:02mrpeters-isllogging into my master now to see
#10:20:33mrpeters-isl discovers i don't have any templates/folders
#10:20:48tsbereSo make a folder?
#10:21:09mrpeters-islyeah, i know :) i thought i had some stuff in here is all.
#10:21:37tsberemrpeters-isl: If you did perhaps you are seeing the issue
#10:21:46tsbereIt looks like I have no folders when I know I do
#10:24:01mrpeters-islnah, i dont have anything in there -- i checked the db
#10:24:09bshumOh nice.
#10:24:10mrpeters-islsorry...pulled away for a minute...brb
#10:24:17bshumNothing in reports at all.
#10:24:22bshumWell that's awesome....
#10:26:56mrpeters-isluh oh
#10:27:01mrpeters-islthats not good
#10:27:22mrpeters-islmeanwhile, my test system isnt listening on 443 for some damn reason
#10:29:34akilsdonk_ has joined #evergreen
#10:31:08akilsdonk has quit IRC
#10:31:16akilsdonk_ is now known as akilsdonk
#10:33:55denialsSo, I guess a question would be if this is a regression since beta1 or whether it hasn't been tested for some unknown period of time
#10:33:56artunit has quit IRC
#10:34:06fortin has joined #evergreen
#10:35:06mrpeters-islmy beta1 server isi out there, if anyone wants to try there
#10:36:14artunit has joined #evergreen
#10:36:24kmlussier_ has joined #evergreen
#10:36:37hopkinsju has quit IRC
#10:36:41kmlussier has quit IRC
#10:37:55tsbereI bet fleshing is busted somewhere. :/
#10:38:41tsbere"open-ils.reporter-store is not listed as a controller for au, moving on"
#10:41:14denialsheh, it's true, open-ils.reporter-store isn't listed as a controller...
#10:41:32denialsbut that seems to be a fairly significant change in behaviour :/
#10:43:54tsbereYea, Pcrud fleshing busted stuff. :(
#10:44:15tsbere runs more checks just to be sure
#10:45:48swills_ has joined #evergreen
#10:47:33tsbereAdding reporter-store to au does, in fact, fix things. But the fact that non-pcrud interfaces can't flesh things that they aren't listed on controllers on is a significant break from the way things worked before pcrud fleshing was added.
#10:48:39tsbere pokes berick, senator, and eeevil about this issue
#10:48:54denialsYep. Would be nice to have a set of tests (manual or automated) that should be run before merging big core-infrastructure-changing branches (to catch stuff like this)
#10:51:04tsbere supposes he should go make a LP entry for this now
#10:57:11kmlussier__ has joined #evergreen
#10:57:12kmlussier_ has quit IRC
#11:02:08senatori'm looking now for whatever code is requiring the controller to be listed on fleshed classes for services other than pcrud, since that was not the intent
#11:02:47tsbere opened https://bugs.launchpad.net/evergreen/+bug/977271 for reference
#11:02:47pinesol_greenLaunchpad bug 977271 in Evergreen "Reporter-store (and likely cstore) no longer fleshes when it isn't a controller" (affected: 1, heat: 6) [Critical,New]
#11:05:31Gentlecat has quit IRC
#11:07:34eeevilfwiw, and IIRC, there was a hack that allowed reporter-store to do whatever cstore could. that was wrong, of course, being a hack, but whatever "fixed" that obv broke reporter. still, simple fix in the IDL: 1) %s/open-ils.reporter-store//g 2) %s/open-ils.cstore/open-ils.cstore open-ils.reporter-store/g
#11:10:10tsbereeeevil: At that point I would be tempted to instead do a search/replace on the entire *rest* of the codebase to say reporter-store becomes cstore. I don't understand why we have two anyway. <_<
#11:11:28mrpeters-islis there any way to tie " Rolling back a transaction with Open-ILS XACT-ID [1333986967.6866767.2559718456756]" back to an actual error? Trying to register a patron and get this every time. Can't figure ot what's causing the rollback.
#11:11:43eeeviltsbere: if you add stuff to the idl that is only useful in the reporter (local additions, say) that you'd rather not expose through cstore (security, privacy, etc), then you've just removed that ability
#11:13:46tsbereeeevil: If someone can access things I have added to cstore can't they do the same for reporter-store, though? Same level of access to the system is required either way, right? If my private router isn't secure then I have bigger problems.
#11:17:19eeeviltsbere: ok, another way of putting the request: please don't remove features when a simple regex can fix the problem :)
#11:19:55mrpeters-islnevermind on Rolling back....got it sorted
#11:20:07tsbereeeevil: I suppose I am mainly confused as to why it was done in the first place, given that both services act pretty much identically, both are private services only, etc. If you can call one you can call both, so I see no security or privacy benefit to having two. Am I missing some obvious benefit?
#11:33:41artunit has quit IRC
#11:35:11eeeviltsbere: reporter-store can be pointing at a different db, say a replica with extra stuff in it (perhaps sensitive info from outside the evergreen dataset). not many sites use it, but at least one does (or, has ... haven't looked recently)
#11:36:15artunit has joined #evergreen
#11:40:08tsbereeeevil: See, "can point at a different DB" was the semi-obvious benefit I was missing.
#11:40:51eeeviltsbere: ahh, k
#11:41:38denials does find it weird that reporter-store can issue anything other than SELECT stmts but that's not pertinent to this morning's discussions
#11:42:10denialstsbere / gmcharlt: can you please add my ssh key to the (unfortunately named) Evergreen-DocBook.git?
#11:42:40tsberedenials: I can do that
#11:42:48denialstsbere++
#11:44:14tsberedenials: I believe you should be set
#11:44:45denialsyep! thanks sir
#11:57:30timhome has quit IRC
#11:57:53timhome has joined #evergreen
#12:02:30mrpeters-isltspindler: you here?
#12:03:04bshumsenator: The commit has on our selection list fix is a30dcca536acc833d6323763485e3d63b1c37574
#12:03:08tspindlermpeters-isl yes
#12:03:10bshumLooks the same as the one in master.
#12:03:15mrpeters-islwhat was the result on my server?
#12:03:51tspindleri went to create a test fund under br1, i was able to get to the new fund screen and clicked on save and nothing would happen
#12:04:00senatorbshum: we'd want that bit of log to troubleshoot further then, since i can't reproduce
#12:04:02tspindleri can't remember what time if it was before you did the update
#12:04:05artunit has quit IRC
#12:04:13mrpeters-isloh, i didnt do any updates
#12:04:18bshumsenator: Yep, I'll try and capture it
#12:04:41tspindlerok, i did it today, ithought you pulled something off git but misunderstood you message then
#12:05:03mrpeters-islwhat user were you logged in as?
#12:05:08tspindleradmin
#12:05:13mrpeters-isli see a "FIONA" fund, and my "Test" fund
#12:05:25mrpeters-islmine was created this morning. so it is working.
#12:05:28tspindleri tried to create a third fund
#12:05:39mrpeters-islcan you give me the values?
#12:05:46mrpeters-islill try over here
#12:06:17artunit has joined #evergreen
#12:07:02tspindlerok, i set waring to 0 and stop percent was 0, currency type USD, active checked, and rollover checked
#12:07:16tspindleryear was 2013
#12:07:21swills_can I point an e-resource at a proxy URI from an 852 subfield as an EG feature or am I going to have to customize the rdetail to make that happen? follow up am I even on the right track for supporting ebooks?
#12:07:58mrpeters-islaha
#12:08:03mrpeters-islyou shouldn't use 0 and 0
#12:08:06mrpeters-isljust leave em blank :)
#12:08:11mrpeters-islI think thats the problem
#12:08:47mrpeters-islhmm, no that doesn't seem to be it either
#12:09:38Elene_ has joined #evergreen
#12:10:24Elene_hi all, i have a strange problem. whenever I find a book in OPAC and click on Copy Details, the barcode is not displayed
#12:10:26denialsswills_: maybe paste your example MARC 852 / 856 somewhere with your expectations of what the user would see?
#12:10:46tspindlermrpeters-isl: i used 0 and 0 in previous versions without problems
#12:10:48mrpeters-islosrfsys.log:open-ils.pcrud 2012-04-09 13:41:08 [ERR :5757:oils_sql.c:2265:13338807101106125] open-ils.pcrud ERROR inserting acq::fund object using query [INSERT INTO acq.fund (id,org,name,year,currency_type,code,rollover,propagate,active,balance_warning_percent,balance_stop_percent) VALUES (DEFAULT,4,'Test2',2013,'USD','Test2','t','t','t',2,1);]: 0 ERROR: duplicate key value violates unique
#12:10:48mrpeters-islconstraint "name_once_per_org_year"
#12:11:08mrpeters-islosrfsys.log:open-ils.pcrud 2012-04-09 12:45:48 [ERR :5775:oils_sql.c:2265:1333880709110556] open-ils.pcrud ERROR inserting acq::fund object using query [INSERT INTO acq.fund (id,org,name,year,currency_type,code,rollover,propagate,active,balance_warning_percent,balance_stop_percent) VALUES (DEFAULT,4,'test fund',2013,'USD','tfund','t','f','t',0,0);]: 0 ERROR: new row for relation "fund"
#12:11:08mrpeters-islviolates check constraint "acq_fund_rollover_implies_propagate"
#12:11:12tspindleri don't think i reused a name
#12:11:20mrpeters-islits one of those, i think
#12:11:21Elene_here is the example link: http://isi.evergreencatalog.com/opac/en-US/skin/default/xml/rdetail.xml?r=99399
#12:11:41mrpeters-isli see it
#12:11:45mrpeters-islElene_
#12:11:50mrpeters-isl300000008523
#12:12:09denials sees it too (in Firefox 11)
#12:12:13mrpeters-islsame
#12:12:17denialsElene_: what browser are you using?
#12:12:17mrpeters-islFF1
#12:12:20mrpeters-islerr, 11
#12:12:59denials sees it in Chromium too
#12:13:55Elene_denials: I am using Google Chrome on OS X
#12:14:04mrpeters-isltspindler: is it possible you can't use 2013 as the year when using propigate/rollover?
#12:14:22tspindlermrpeters-isl: maybe
#12:14:34mrpeters-islyeah, thats gotta be it
#12:14:39mrpeters-islif i switch to 2012 no issues at all
#12:15:07bshumsenator: http://pastie.org/3756383
#12:15:21mrpeters-isllet me know if that works, tspindler and then ill mark the bug invalid
#12:15:29bshumsenator: It seems to have made two checks? One for each drone of a given brick
#12:15:31mrpeters-islthough, this really should supply the user with an error message....so maybe its valid
#12:15:37denialstspindler: check constraint "acq_fund_rollover_implies_propagate" suggests that if rollover = true, that propagate must also be true
#12:15:50bshumOr I probably read it wrong
#12:15:58denials checks the database definition
#12:16:13tspindlermrpeters-isl: i'll check
#12:16:33mrpeters-isldenials is right
#12:16:51mrpeters-islyou were using false, for propagate
#12:17:00mrpeters-isli'll update the bug accordingly
#12:17:20Elene_denials: I think I have the wrong link :( here is the correct link that has a non-working barcode
#12:17:20Elene_http://isi.evergreencatalog.com/opac/en-US/skin/default/xml/rdetail.xml?r=53539
#12:17:48Elene_mpreters-isl: yes, I a afraid I gave you the wrong link
#12:18:14mrpeters-islis that copy, copy location opac_visible?
#12:18:28senatorbshum: the query being made on two bricks seems bizarre and alarming to me. i would not expect that; no ideas.
#12:19:04bshumsenator: Well it's two drones of the same brick
#12:19:49mrpeters-islgood catch, tspindler. Thank you!
#12:20:02tspindlermrpters-isl: it is working for me
#12:20:28mrpeters-islyeah, it's not the year. it's the fact that you were choosing rollover, but not propagate
#12:20:40mrpeters-islsee my comment on the bug for a full report
#12:21:11mrpeters-isl wishes you could edit launchpad bug descriptions. i'm afraid the ACTUAL bug is going to get lost in the comments
#12:21:14denialsElene_: hmm, copy looks okay in http://isi.evergreencatalog.com/opac/extras/unapi?id=tag:opac:biblio-record_entry/53539/CONS&format=marcxml-full
#12:21:33senatorbshum: oh ok, i think i see what's happening there. first log line is logged by CStoreEditor.pm showing what it's /going/ to ask open-ils.cstore. the second line is open-ils.cstore itself actually receiving the query
#12:21:38senatorso that's normal actually
#12:22:05bshumSounds reasonable.
#12:22:07senatoranyway, that query is as it should be. you have acq selection lists owned by this bishop user and they don't show up?
#12:22:22bshumThat's generally the problem, yep.
#12:22:34bshumI'll double check though to make sure I'm not being led astray there...
#12:22:35senatorif you paste this query into srfsh, do you get any results?
#12:24:46bshumI've never tried that before
#12:24:50bshumThe whole thing after CALL?
#12:25:25senatoryes. and prepend the word 'request' to that
#12:27:31hopkinsju has joined #evergreen
#12:28:32bshumTesting
#12:29:30bshumOh nice
#12:29:36bshum"received no data from server"
#12:29:41bshum120.12 seconds
#12:30:44dbwells_ is now known as dbwells
#12:31:36kmlussier__ has quit IRC
#12:34:10senatorbshum: hrm. assuming what you ran looked like this: http://pastie.org/3756487 then your result suggests a deeper problem
#12:34:21senatorany errors or warnings in the log around that time?
#12:37:06bshumNo errors in the log
#12:37:13bshumAt least nothing I saw
#12:37:25bshumI'm watching a running output of osrferror.log on our production systems
#12:37:35bshumWe're still troubleshooting whether we've got major performance issues
#12:37:36senatorfor the moment at least i'm at a loss as to why that query should time otu
#12:37:41senators/otu/out/
#12:38:21senatormy test system is master rather than rel_2_2, but i can't think of any important difference for this
#12:45:44bshumProbably not, rel_2_2 hadn't diverged too far from master before we implemented the system.
#12:46:00kmlussier__ has joined #evergreen
#12:46:07kmlussier__ is now known as kmlussier
#12:49:37bshumsenator: I tried the srfsh query on two other bricks and got timeouts too
#12:50:58bshumI wonder if it's taking too long because we have too much stuff.
#12:52:04bshumAnd it's dying pretty regularly at 120 seconds
#13:01:41Elene_ has quit IRC
#13:02:56gmcharltbshum: which it would, as 120 is one of the defined timeouts for OpenSRF requests IIRC
#13:03:14gmcharltbshum: apologies if this is in scrollback, but what request are you trying?
#13:03:36bshumgmcharlt: Oh I was trying to see selection lists stuff
#13:03:48bshumgmcharlt: senator had me try: http://pastie.org/3756487
#13:04:53Neil__ has joined #evergreen
#13:05:27tsbere finally opens a bug about his hold targeter tweaking: https://bugs.launchpad.net/evergreen/+bug/977358
#13:05:27pinesol_greenLaunchpad bug 977358 in Evergreen "Hold Targeter leaves lockfiles after failure" (affected: 1, heat: 6) [Undecided,New]
#13:09:13gmcharltbshum: next place to check are the postgres logs
#13:10:15gmcharltand I also recommend turning on statement duration logging - say, to capture all statements that take more than a second or two to cmplete
#13:13:56bshumgmcharlt: I'll give that a whirl when I get a moment. We're watching the logs like hawks right now to see if we can spot where things are dying for circ :(
#13:14:20bshumThanks.
#13:18:13Neil__ has left #evergreen
#13:19:00luisb has joined #evergreen
#13:20:23luisb has joined #evergreen
#13:34:22artunit has quit IRC
#13:36:09denialsmrpeters-isl: fwiw, at least some people are able to edit bug descriptions - maybe limited to the opener of the bug? There's a little italicized "!" to the right of "Bug Description"
#13:36:39tsberedescriptions are nice. Comments would be nicer, but we don't get a button for that. :(
#13:36:46artunit has joined #evergreen
#13:37:25denials was responding to "* mrpeters-isl wishes you could edit launchpad bug descriptions. i'm afraid the ACTUAL bug is going to get lost in the comments"
#13:37:49denialscomments, I don't care
#14:04:28artunit has quit IRC
#14:05:12enhancin has joined #evergreen
#14:05:20enhancinSilly Question, anyone know where the fees are stored in the database?
#14:05:51enhancinright as i type this i find the money schema.
#14:06:56artunit has joined #evergreen
#14:08:23mrpeters-islmoney.billing or money.grocery :)
#14:08:47mrpeters-isldenials: oh, interesting. maybe i can talk tspindler into doing that.
#14:09:14tspindlermrpeters-isl: huh
#14:09:28mrpeters-isladding the actual bug, to your bug report :)
#14:09:47mrpeters-islput my last comment in your original post so it doesn't get lost
#14:09:57fortin has quit IRC
#14:09:59mrpeters-isloh, wait...looks like i can do it as a bug wrangler
#14:10:00tspindlerok i see
#14:10:35mrpeters-islnevermind, tspindler. thanks
#14:11:16mrpeters-islmind if i make the title more appropriate too?
#14:12:10mrpeters-islperhaps "Acq: Attempting to save certain fund configurations fails without notice to user"
#14:15:35denialsOr "Acq: attempting to set a fund's propagate to TRUE without rollover = TRUE fails without notice to user" if you want it to be really precise?
#14:15:50mrpeters-islthat works
#14:16:07mrpeters-islthanks for helping us catch that, denials
#14:16:12tsbereHow about a general "database failures rarely make it out to the user in config interfaces" >_>
#14:16:48tsbere ran into problems with other interfaces and not seeing the errors go by in the past
#14:16:52mrpeters-isltrue
#14:17:05mrpeters-islran into another one today too
#14:18:12enhancinMy favorite error messages are "Something appears to be wrong. You should probably fix it."
#14:19:01denialsSure. Specifics are easier to deal with than generalities, though. Who wants to work on a bug that has many tentacles and will take a long time to fix, even if it means doing it right, rather than something that can have an immediate impact and not require extensive testing, etc
#14:19:19denials(and then try and get sign-off on that)
#14:19:51tsbere looks at his new_xulrunner branch
#14:20:00tsberedenials: I suspect I get to answer "me, apparently"
#14:20:33mrpeters-islThrowing at least a "Something went wrong" is good. There's always the sys admin who can help at that point. No error at all makes you think everything went correctly though. I agree with denials, it would be really tough to touch everything at one.
#14:21:24tsbere re-reads this bug description and laughs
#14:21:38tsbereThe IP is x'd out in the first url, but isn't in the second.
#14:21:59denialstsbere: Yes, you are awesome
#14:22:59tsbere wants to poke at https://bugs.launchpad.net/evergreen/+bug/974638 some more, but isn't sure if anyone else has opinions on what he said he thought was wrong there
#14:22:59pinesol_greenLaunchpad bug 974638 in Evergreen "EG 2.2 beta1's client MARC edit view gives syntax error" (affected: 3, heat: 14) [Undecided,New]
#14:24:01tsbereI also think I am supposed to be building a beta2 today <_< I should probably get to that.
#14:24:51enhancinIs there a way to set all future billing to $0.00, I've looked at the rule_max_fine table and set those to 0 but we're getting things coming through still...is there another place to change it or is tehre a place to disable automatic fee generating?
#14:25:21tsbereenhancin: What are you aiming to do?
#14:25:28enhancinremove late fees.
#14:25:41tsbereIn general?
#14:25:47tsbereFor a specific library?
#14:25:57enhancinthere is only one library, so both.
#14:26:04tsbere points at the "Amnesty Mode" checkin modifier
#14:27:26enhancintsbere: can I make that a default instead of having to select it every time?
#14:27:33denials reads the description of the updated bug - hey, I thought I was the one who pointed out propagate vs. rollover
#14:27:51tsbereenhancin: Once you select it the workstation should remember it.
#14:28:02enhancinawesome.
#14:34:33artunit has quit IRC
#14:36:56artunit has joined #evergreen
#14:46:48fortin has joined #evergreen
#14:52:50tsbere notes 5 beta2 targeted but not committed bugs.....isn't sure if he should shoot for "review some of those" or just build another beta without them
#14:53:34tspindler has quit IRC
#14:56:25Gentlecat has joined #evergreen
#14:58:05swills__ has joined #evergreen
#14:59:45swills_ has quit IRC
#15:03:21denials3 of the 5 being mine, swell
#15:03:31artunit has quit IRC
#15:04:07tsberedenials: I get the feeling I can relate ;)
#15:05:06tsbere loads one up for testing
#15:05:53denials retargets bug 965907 from beta1 to beta2 to make it 4/6 :)
#15:05:53pinesol_greenLaunchpad bug 965907 in Evergreen "Integrate TPAC i18n support into regular infrastructure" (affected: 1, heat: 6) [High,New] https://launchpad.net/bugs/965907
#15:06:59artunit has joined #evergreen
#15:07:54tsbere looks at a "MARC RECORD" of "HASH(0x7f891614f7c0)
#15:07:54tsbere" when testing the "don't crash when the record doesn't exist" TPac stuff
#15:08:03tsberestupid newline <_<
#15:09:00tsbereSo, do we consider bad data in places like that a problem when the record doesn't exist?
#15:09:19denialstsbere: huh. worked here when I pushed it; I'll try on a clean rebase of master.
#15:09:50tsberedenials: Seems to work fine.....until I tell it to load up the MARC Record. Then the "MARC Record" area has a hash value (rest of the page is fine)
#15:09:52denialsIt's a problem if the server throws a complete unhandled error
#15:09:58denialsah, okay
#15:10:13denialsthat's better than what I thought you were saying :)
#15:11:27denialsI'd say that's better behaviour than the alternative, which is a straight-up 5xx in the user's face. Definitely some follow-on work to be done (like a message like "this record does not exist")
#15:11:38tsbereyea
#15:13:05denialsjspac in that situation throws a bunch of JS errors (but what else is new for jspac)
#15:13:06tsbere is tempted to make a follow-up that makes the page say "This record doesn't exist at this time, and we can't find the TARDIS to search other times either right now." or similar
#15:14:42tsberedenials: For the TPAC i18n stuff......I feel like I should complain about tpac.pot's header info
#15:15:30tsbere"# SOME DESCRIPTIVE TITLE." etc <_<
#15:15:31denialsComplain away, but please commit it.
#15:15:50tsbere hasn't tested it, he just used git show while thinking about the hash issue ;)
#15:18:22tsbere runs a quick test on rel_2_2 for the "loading a record that doesn't exist" checks
#15:28:21tsberedenials: https://bugs.launchpad.net/evergreen/+bug/975577 isn't getting any testing from me.....due to my general cluelessness with located uris. Or even non-located uris. I just don't know a lot about how that works in the system.
#15:28:21pinesol_greenLaunchpad bug 975577 in Evergreen "Preferred library should affect search scope for located URIs" (affected: 1, heat: 6) [Undecided,New]
#15:28:36tsbere is poking at i18n related items quick, though
#15:28:38denialstsbere: no problemo
#15:32:05tsberedenials: You are down to 1 code branch, I think, targeted at beta2. Which is the only one left with a pullrequest, I think.
#15:32:14tsbere suspects it is time for him to build beta2 <_<
#15:33:29denialstsbere++ # sweet, thanks
#15:34:41tsbereAnything else I should look at before I fire up the release builder script?
#15:34:53artunit has quit IRC
#15:37:18artunit has joined #evergreen
#15:37:43moodaepotsbere++
#15:39:12denials peruses the remaining 41 pullrequest tagged bugs quickly
#15:42:11tsbere goes to grab something to drink while denials peruses
#15:43:05denialseh - maybe no low-enough hanging fruit there
#15:48:00tsbere kicks off the process then
#16:01:46Meliss has quit IRC
#16:03:54artunit has quit IRC
#16:05:14collum has quit IRC
#16:07:23artunit has joined #evergreen
#16:08:06akilsdonk has quit IRC
#16:20:19tsbere successfully built a beta that doesn't extract <_<
#16:22:14denialsperrrfect
#16:22:29denials starts ripping RHEL/CentOS mentions out of Makefile.install
#16:23:02fortin has quit IRC
#16:23:42tsberedoh, I left a TAR_OPTIONS variable kicking around.
#16:25:04tsbere confirms that his tar file isn't happy because of the extra options, wipes said ENV var, and re-runs his script
#16:25:52moodaepo waits itching to do a quick beta2 upgrade test from 2.1
#16:26:05denialstsbere: that script should be in the build/ directory somewhere, eh?
#16:26:32tsberedenials: I should update and rebase the branch. Again. <_<
#16:28:30tsbere will see about doing that after the script finishes
#16:28:34denialssure
#16:28:48tsbere should probably also make a launchpad entry for the branch, instead of poking it into IRC every so often
#16:31:00berkleeintern has quit IRC
#16:35:57Gentlecat has quit IRC
#16:38:07denialswhilst throwing branch names into IRC, user/dbs/update_makefile_prereqs in working (for ripped out CentOS/RHEL/Hardy stuff, and updated deps to their latest versions)
#16:39:05denialslibjs / spidermonkey both work on fedora, yaz doesn't count because a high enough version is packaged for fedora
#16:43:28denials laughs looking at http://git.evergreen-ils.org/gitstats/Evergreen/authors.html again - last commit for anyone was 2012-03-19. Doesn't seem to be able to run properly without tsbere's loving touch.
#16:45:25denialstsbere++ # upload announcement
#16:45:49tsberedenials: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/tsbere/make_release
#16:45:56tsbere will go throw something on launchpad about that now
#16:48:04tsberehttps://bugs.launchpad.net/evergreen/+bug/977535
#16:48:04pinesol_greenLaunchpad bug 977535 in Evergreen "The Release Building script should be in master" (affected: 1, heat: 6) [Undecided,New]
#16:48:48fortin has joined #evergreen
#17:00:58denialsahh, we're redistributing the xulrunner binaries in the tarball? Now I understand why the tarball's so big :)
#17:01:09moodaepotsbere: Just started a test upgrade for beta2 and noticed that when I run say "make STAFF_CLIENT_BUILD_ID=rel_2_2_beta2 install" as root I get /openils/var/web/xul/2_2-beta2 not rel_2_2_beta2 as expected.
#17:01:29kmlussier has quit IRC
#17:01:45tsberemoodaepo: That is because you want STAFF_CLIENT_STAMP_ID, not STAFF_CLIENT_BUILD_ID
#17:02:04moodaepoDo'h
#17:02:18bshumYou're still living in a 1.6 past moodaepo! :D
#17:02:42denials updates the DIG server_upgrade.txt document
#17:02:50denials(it still said BUILD_ID)
#17:02:55moodaepobshum: Yea yea
#17:02:56artunit has quit IRC
#17:03:41moodaepoIf I remember correctly it was the same in 2.0 so not as archaic as 1.6!
#17:03:58bshumI know, I was probably just exaggerating for effect :(
#17:04:11moodaepobshum++
#17:07:30artunit has joined #evergreen
#17:09:01denials updates a lot of stuff in the server_upgrade.txt document. having commit access to the docs is fun!
#17:11:18berickwith commit access comes great respons^H^H^H^H^fun!
#17:11:43rangispeaking of fun
#17:11:55rangihttp://ranginui.github.com/git-achievements/
#17:12:04rangifrom https://github.com/icefox/git-achievements
#17:12:10rangipointless .. but fun
#17:12:21denialsrangi++
#17:13:32berickomg, we need evergreen achievements
#17:16:08tsbereberick: For us, or for our users? Because the latter would need things like "Complain about not having permissions while obviously logged in as the wrong user" ;)
#17:19:35bericktsbere: hah. i'd like to avoid getting run out of town w/ pitchforks if possible
#17:19:58berickthough, 'run out of town with pitchforks' would be a developer achievment
#17:21:06tsberegroup achievement! "All developers attending the conference get run out of town together" ;)
#17:24:08fortin has quit IRC
#17:27:12phasefx wants the Pesky Developer achievement
#17:35:04artunit has quit IRC
#17:37:58artunit has joined #evergreen
#17:55:06hopkinsju has quit IRC
#17:58:01swills__ has left #evergreen
#18:03:02artunit has quit IRC
#18:06:54jenny has quit IRC
#18:07:55artunit has joined #evergreen
#18:33:08artunit has quit IRC
#18:38:22artunit has joined #evergreen
#19:07:22artunit has quit IRC
#19:08:31artunit has joined #evergreen
#19:34:10artunit has quit IRC
#19:38:13artunit has joined #evergreen
#19:50:48finnx has joined #evergreen
#20:03:01artunit has quit IRC
#20:08:14artunit has joined #evergreen
#20:32:47artunit has quit IRC
#20:38:41artunit has joined #evergreen
#20:43:02hopkinsju has joined #evergreen
#21:03:25artunit has quit IRC
#21:08:54artunit has joined #evergreen
#21:34:28artunit has quit IRC
#21:39:01artunit has joined #evergreen
#22:04:40artunit has quit IRC
#22:07:31enhancin has quit IRC
#22:09:17artunit has joined #evergreen
#22:34:58artunit has quit IRC
#22:39:40artunit has joined #evergreen
#22:58:41finnx has quit IRC
#23:04:26artunit has quit IRC
#23:09:17artunit has joined #evergreen
#23:32:44luisb has quit IRC
#23:33:47artunit has quit IRC
#23:40:03artunit has joined #evergreen
< Sunday, April 8th, 2012Raw Log FileTuesday, April 10th, 2012 >