Log of the #duraspace-ff channel on chat.freenode.net

Using timezone: Eastern Standard Time
<barmintor>awoods: if you’re there, I finally got the trippi release to work00:21
* longshou leaves01:06
* barmintor leaves01:48
* dwilcox joins07:45
* mohamed joins08:13
* escowles joins08:42
* jcoyne joins09:01
* jcoyne leaves09:05
* bljenkins leaves09:07
* bljenkins joins09:09
<awoods>thanks, barmintor09:10
* mikeAtUVa joins09:11
* bljenkins leaves09:14
* bljenkins joins09:17
* bljenkins leaves09:20
* gregjansen joins
* bljenkins joins09:22
* bljenkins leaves09:34
* bljenkins joins09:37
* escowles leaves09:38
* escowles joins09:40
* tecoripa joins09:41
* jcoyne joins09:42
* scossu joins10:00
* ksclarke joins10:05
* barmintor joins10:14
good morning
awoods: trippi is promoted
<awoods>barmintor: nice work
<barmintor>tecoripa: those 3 NPE risks are mitigated, and the poolable factories don’t swallow exceptions on borrow anymore10:15
<tecoripa>barmintor: saw that, looks good10:16
<barmintor>awoods: Only had to rollback 4 times!
:P
<tecoripa>barmintor: I'll pull and rebuild
barmintor: is the new trippi now in the repo?10:17
<barmintor>https://search.maven.org/#search%7Cga%7C1%7Ctrippi10:18
tecoripa: ^^
<tecoripa>barmintor: great. I can update the fcrepo3 pom to point to that release now.
<barmintor>yeah, I was going to do that as part of the bump to 3.8.0-SNAPSHOT, but feel free10:19
<tecoripa>barmintor: I'll do it now
* longshou joins10:30
* jcoyne1 joins10:35
* jcoyne leaves
* bljenkins leaves10:38
* gregjansen leaves
* gregjansen joins10:39
* dwilcox leaves10:40
* bljenkins joins10:41
* gregjansen leaves
* gregjansen joins
* dwilcox joins10:42
<jcoyne1>awoods: what does a failed fixity check look like? Re: https://wiki.duraspace.org/display/FF/RESTful+HTTP+API+-+Fixity
<awoods>jcoyne1: https://wiki.duraspace.org/display/FF/2014-08-06+Acceptance+Test+-+Detection+of+corruption?focusedCommentId=61572335#comment-6157233510:43
<jcoyne1>awoods: Can that be moved into the documentation?10:44
<awoods>jcoyne1: sure. that is a good suggestion
<bljenkins>Yippee, build fixed!10:53
Project fcrepo4 build #2013: FIXED in 11 min: http://ci.fcrepo.org/jenkins/job/fcrepo4/2013/
<awoods>jcoyne1: https://wiki.duraspace.org/pages/diffpagesbyversion.action?pageId=41353400&selectedPageVersions=7&selectedPageVersions=810:54
<bljenkins>Project fcrepo-module-auth-rbacl build #22: UNSTABLE in 2 min 3 sec: http://ci.fcrepo.org/jenkins/job/fcrepo-module-auth-rbacl/22/10:55
* jcoyne joins10:57
* jcoyne1 leaves
<escowles>awoods: we can't hear you, are you there/speaking?11:15
awoods / mikeAtUVa / longshou / tecoripa / dwilcox : can anybody hear me? i don't hear anything11:16
<awoods>escowles: you voice dropped about 8min ago11:17
<mikeAtUVa>escowles: you were dead to us.
escowles: I guess we were to you as well.
<escowles>awoods: sorry about that -- everything looked normal and i was talking so i didn't notice... :-(11:18
did any of my report come through?
<awoods>escowles: you dropped shortly after mentioning your data-mapping work11:19
<escowles>awoods: ok, so i think the only other thing after that was: i looked at the many-child issue and it's being actively talked about by Randall and Hoirean, but not fixed yet, so it should be in beta111:20
<awoods>escowles/longshou: should we circle-up on a hangout to continue the java-client discussion?
<longshou>awoods sure
<escowles>awoods: yes, sounds good
<awoods>escowles/longshou: I have a call in 8 min, so will not be on the hangout.11:21
<escowles>awoods: ok, longshou and i can work everything out amongst ourselves
longshou: we can just do irc or email, if you'd rather...11:22
<longshou>I think hangout is good.11:23
<escowles>ok, i'm on the hangout now
* longshou leaves11:50
<tecoripa>awoods: just pushed my commit to the pom to 3.7.2-SNAPSHOT12:34
awoods: barmintor and I are still in the process of tweaking code... it's not ready for announcing to the world yet12:35
<awoods>tecoripa: thanks. keep me posted
<tecoripa>awoods: will do
* longshou joins12:42
<escowles>longshou: i just opened a PR for adding a base FedoraException class to the API, plus a sample NotFoundException that extends it: https://github.com/fcrepo4-labs/fcrepo4-client/pull/413:23
<longshou>escowles: Thanks. I am working on the jcr/xml persisted with hiearchy path now. I’ll check it out once it merged to the master.13:26
* github-ff joins13:28
[fcrepo-jms-indexer-pluggable] escowles force-pushed jms-baseurl from 25bd357 to 530816e: http://git.io/JILScA
fcrepo-jms-indexer-pluggable/jms-baseurl 209c9e1 Esmé Cowles: Removing hard-coded repository URL -- all repository URLs now come from JMS messages (or the reindex servlet's baseURI parameter)
fcrepo-jms-indexer-pluggable/jms-baseurl 530816e Esmé Cowles: Adding VisibleForTesting annotation, replacing magic numbers with variables for likely webapp/REST API path components
* github-ff leaves
<tecoripa>afk-bbs13:29
* tecoripa leaves
<escowles>awoods: this is the PR i mentioned earlier: https://github.com/fcrepo4/fcrepo-jms-indexer-pluggable/pull/4814:10
<awoods>escowles: yes, as a result of the PR on the main codebase, I recently had to address the impacts on the jms-indexer side with this ticket: https://www.pivotaltracker.com/story/show/7674008214:12
<pivotal-bot>feature: Propagate updates to jms-indexer from fcrepo4 (delivered) / owner: Andrew Woods
<awoods>escowles: In any case, the repositoryURL is still in indexer-core.xml: https://github.com/fcrepo4/fcrepo-jms-indexer-pluggable/blob/master/fcrepo-jms-indexer-webapp/src/main/resources/spring/indexer-core.xml#L4814:14
escowles: hence the new (unworked) ticket: https://www.pivotaltracker.com/story/show/76745482
<pivotal-bot>feature: Remove "repositoryURL" from IndexerGroup.constructor() (unscheduled) / owner:
<escowles>awoods: right, and PR 48 addresses that ticket and some of what longshou was talking about at standup (parsing the URIs from JMS events and trying to figure out what the baseURL is)14:18
<awoods>escowles: wait a moment, there are two PRs in your referenced ticket (#72351918), and only the main codebase PR was merged.
<pivotal-bot>Mike Durbin started "Create authz-configurable webapp" https://www.pivotaltracker.com/story/show/74346396
<escowles>awoods: and that PR now has IT failures because of the same ClassCastException we saw in fcrepo4 recently14:19
but the same fix (explicitly depending on the javaee-api) doesn't work because Solr requires the javaee api impl classes14:20
<awoods>escowles: apparently PR-48 also has checkstyle errors
escowles: are you working on PR-48 now?14:21
<escowles>awoods: i just rebased it and ran the tests, and tried adding javaee-api to pom.xml to see if that would fix the tests14:23
<awoods>escowles: are you still working on it?14:24
<escowles>awoods: no
<awoods>escowles: so the PR-48 now has checkstyle errors and failing ITs?
<escowles>awoods: i don't see any checkstyle errors on the jms-baseurl branch, but I do see failing ITs14:27
<awoods>escowles: 530816e8bbc4bbf06d4db9105cd983e8442cc585, when I checkout jms-baseurl I get immediate checkstyle errors.14:28
escowles: I will look into it and keep you posted.14:29
* bljenkins leaves14:30
* bljenkins joins14:33
* bljenkins leaves14:37
* bljenkins joins14:40
* tecoripa joins14:41
<cbeer>mikeAtUVa: you handled that fedora-community post gracefully.. i would have said "hate to break it to you, but you don't actually have XML, just a bunch of text with angle brackets"14:44
<bljenkins>Project fcrepo-jms-indexer-pluggable build #610: STILL UNSTABLE in 4 min 36 sec: http://ci.fcrepo.org/jenkins/job/fcrepo-jms-indexer-pluggable/610/14:46
<mikeAtUVa>cbeer: heh.. I remember the day (maybe it's even true now) that you could put valid XML in and it would move your namespace declarations to earlier in the FOXML and you'd get different (maybe invalid) XML back.14:47
<tecoripa>mikeAtUVa: yeah, I remember that, too.14:48
<mikeAtUVa>Probably still true, but we've all learned not to use "X" datastreams.
<barmintor>what a horrible idea those were14:57
in retrospect, of course
<tecoripa>of course.
we're, ummm, still using inline XML ingests for some stuff.14:58
<jcoyne>cbeer++15:01
* gregjansen leaves15:02
* bljenkins leaves15:10
<ksclarke>Ha! "hate to break it to you, but you don't actually have XML, just a bunch of text with angle brackets" ... great line15:12
* longshou leaves15:17
* bljenkins joins15:20
* bljenkins leaves15:21
* bljenkins joins15:22
* scossu leaves15:24
<bljenkins>Project fcrepo-jms-indexer-pluggable build #612: STILL UNSTABLE in 5 min 35 sec: http://ci.fcrepo.org/jenkins/job/fcrepo-jms-indexer-pluggable/612/15:27
* longshou joins
* dwilcox leaves15:32
* awoods leaves15:39
* github-ff joins15:48
[fcrepo4] escowles pushed 1 new commit to lastmod: http://git.io/t8HGlQ
fcrepo4/lastmod c1e7b0c Esmé Cowles: Adding check for frozen nodes to avoid trying to update read-only version/etc. nodes
* github-ff leaves
* awoods joins15:51
* bljenkins leaves15:54
* bljenkins joins15:55
* scossu joins
* travis-ci joins16:02
[travis-ci] fcrepo4/fcrepo4#2216 (lastmod - c1e7b0c : Esmé Cowles): The build passed.
[travis-ci] Change view : https://github.com/fcrepo4/fcrepo4/compare/e2ed14413de1...c1e7b0c42125
[travis-ci] Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/32369483
* travis-ci leaves
<pivotal-bot>Longshou Situ added comment: "PR https://github.com/fcrepo4/fcrepo-jms-indexer-pluggable/pull/49 again, commit https://github.com/lsitu/fc�" https://www.pivotaltracker.com/story/show/7564418416:12
Longshou Situ finished "jms-indexer: Store jcr/xml in id hierarchy" https://www.pivotaltracker.com/story/show/75644184
* github-ff joins16:14
[fcrepo4] escowles pushed 1 new commit to lastmod: http://git.io/To8CnQ
fcrepo4/lastmod ea73ec6 Esmé Cowles: Checking for frozen/locked nodes to avoid trying to update read-only nodes
* github-ff leaves
* github-ff joins16:15
[fcrepo-jms-indexer-pluggable] awoods pushed 1 new commit to master: http://git.io/cmFNsA
fcrepo-jms-indexer-pluggable/master 1ae653d Esmé Cowles: Remove hard-coded repository URL...
* github-ff leaves
<pivotal-bot>Esme Cowles added comment: "https://github.com/fcrepo4/fcrepo4/pull/436" https://www.pivotaltracker.com/story/show/76616282
Esme Cowles finished "Creation date should never be newer than last modified date" https://www.pivotaltracker.com/story/show/76616282
* github-ff joins16:16
[fcrepo-jms-indexer-pluggable] awoods closed pull request #48: Removing hard-coded repository URLs (master...jms-baseurl) http://git.io/vB2Qzg
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: ""
https://github.com/fcrepo4/fcrepo-jms-indexer-pluggable/commit/1ae653de2d8ab7f8fc97a21f1e7ede8�" https://www.pivotaltracker.com/story/show/72351918
Andrew Woods deleted "Remove "repositoryURL" from IndexerGroup.constructor()" https://www.pivotaltracker.com/story/show/76745482
<bljenkins>Yippee, build fixed!16:24
Project fcrepo-jms-indexer-pluggable build #613: FIXED in 8 min 30 sec: http://ci.fcrepo.org/jenkins/job/fcrepo-jms-indexer-pluggable/613/
awoods: Remove hard-coded repository URL
<escowles>longshou: my commit with the new client API exceptions has been merged: https://github.com/fcrepo4-labs/fcrepo4-client/commit/15dad406cdb3d5a8d508ac13e0975826a6e25865
<longshou>escowles: Thanks. I’ll check it out shortly.16:26
<pivotal-bot>Andrew Woods accepted "Fix kernel-impl TestHelpers package" https://www.pivotaltracker.com/story/show/76704378
Andrew Woods accepted "Propagate updates to jms-indexer from fcrepo4" https://www.pivotaltracker.com/story/show/76740082
* dwilcox joins16:27
* jcoyne leaves16:30
* dwilcox leaves16:37
* terrellt joins16:48
* github-ff joins16:50
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/CcM-3Q
fcrepo4/master 61f839e Esmé Cowles: Ensure last-modified header is not earlier than created date, weird breakage...
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/fcrepo4/fcrepo4/commit/61f839ecab98a5780e753497d8f83221b10c47e4" https://www.pivotaltracker.com/story/show/76616282
Andrew Woods delivered "Creation date should never be newer than last modified date" https://www.pivotaltracker.com/story/show/7661628216:51
* github-ff joins
[fcrepo4] awoods deleted lastmod at ea73ec6: http://git.io/XxtaWQ
* github-ff leaves
* gregjansen joins16:53
* jcoyne joins
* gregjansen leaves16:56
<pivotal-bot>Andrew Woods added comment: "@longshous, can you rebase this PR on master. I am seeing conflicts/compile errors." https://www.pivotaltracker.com/story/show/75644184
Andrew Woods rejected "jms-indexer: Store jcr/xml in id hierarchy" https://www.pivotaltracker.com/story/show/75644184
Andrew Woods added comment: "@longshous, I would suggest adding a repositoryURL parameter to the reindex REST call. Thoughts, @escowles?" https://www.pivotaltracker.com/story/show/7564418416:59
Esme Cowles added comment: "@awoods Yes, adding a parameter for the repository URL seems like a reasonable way to go." https://www.pivotaltracker.com/story/show/7564418417:00
* mikeAtUVa leaves17:02
* travis-ci joins17:04
[travis-ci] fcrepo4/fcrepo4#2219 (master - 61f839e : Esmé Cowles): The build passed.
[travis-ci] Change view : https://github.com/fcrepo4/fcrepo4/compare/0b0e628a035d...61f839ecab98
[travis-ci] Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/32374997
* travis-ci leaves
* dwilcox joins17:05
* escowles leaves17:12
<pivotal-bot>Longshou Situ started "jms-indexer: Store jcr/xml in id hierarchy" https://www.pivotaltracker.com/story/show/7564418417:29
* mohamed leaves
* dwilcox leaves17:49
* ksclarke leaves18:20
<barmintor>awoods: two questions18:25
1) Can we delete the last classes referring to the Exist XmlDB in FCR 3.8.0? It should allow us to drop the last references to Sun internal XML parsers18:26
2) Ought we try to have the installer use the Tomcat 7 zip from apache on mvn central?
3) OpenJDK?18:27
tecoripa has finished reviewing the SNAPSHOT with me, so those are our remaining questions
<tecoripa>awoods: regarding 1): apparently FeSL provided a EXist as a backend option for the policy index18:28
<awoods>barmintor: 1) Does anyone use the XmlDB? 2) A nice to have 3) We have not supported OpenJDK before, is there a reason to change now?
<tecoripa>https://wiki.duraspace.org/display/FEDORA37/FeSL+Installation
<awoods>barmintor: I would like to minimize the changes so that we in turn minimize the chance that we need a 3.8.1 bugfix release.18:29
<barmintor>awoods: fair enough
I think a Tomcat upgrade would be the last thing for the RC, then18:30
<awoods>barmintor: sounds reasonalbe
ble
<tecoripa>what bramintor said.
<barmintor>and it would be one less sparsely documented dependency in the duraspace mvn repo
<tecoripa>and what barmintor said.
<barmintor>and barminator, he usually says some stuff18:31
<tecoripa>that's your new nickname. The Barminator.18:32
<pivotal-bot>Longshou Situ added comment: "@awoods Yes, the compiling error is caused by method getRepositoryURL() that were removed. Now change it to �" https://www.pivotaltracker.com/story/show/75644184
Longshou Situ finished "jms-indexer: Store jcr/xml in id hierarchy" https://www.pivotaltracker.com/story/show/75644184
Scott Prater added comment: "@awoods, @barmintor : initial code review completed. I'll wait to close this story, pending any last-minute �" https://www.pivotaltracker.com/story/show/7624721618:33
* dwilcox joins18:35
* scossu leaves18:36
* dwilcox leaves18:45
* tecoripa leaves19:01
* jcoyne leaves19:06
<barmintor>awoods: I think we are 1 commit away from RC, hope to get it in between meetings tomorrow19:27
awoods: but I went ahead and bumped the SNAPSHOT version into a new branch
* barmintor leaves19:30
<awoods>barmintor: good news19:39
* escowles joins19:52
<pivotal-bot>Andrew Woods added "Document reindexing" https://www.pivotaltracker.com/story/show/7683967819:55
Andrew Woods edited "Document reindexing" https://www.pivotaltracker.com/story/show/7683967819:58
Andrew Woods added "Remove redundant jms-indexer:reindex() method" https://www.pivotaltracker.com/story/show/7683992220:01
* escowles leaves20:02
* escowles joins20:15
<pivotal-bot>Andrew Woods added "Error-handle reindexing REST request" https://www.pivotaltracker.com/story/show/7684055820:18
Andrew Woods added comment: "When trying to reindex with a hierarchy (/rest/collection/book/page) the logic in IndexerGroup.reindexURI() i�" https://www.pivotaltracker.com/story/show/7564418420:23
Andrew Woods rejected "jms-indexer: Store jcr/xml in id hierarchy" https://www.pivotaltracker.com/story/show/75644184
* escowles leaves20:26
* escowles joins20:28
* escowles leaves20:47
* escowles joins20:48
<pivotal-bot>Longshou Situ started "jms-indexer: Store jcr/xml in id hierarchy" https://www.pivotaltracker.com/story/show/7564418421:02
* escowles leaves21:12
<pivotal-bot>Longshou Situ added comment: "@awoods Is it a new issue or it is introduced with my changes? What's the parameter baseURI mean for reindex�" https://www.pivotaltracker.com/story/show/7564418421:14
Andrew Woods added comment: "@longshous, the parameter: baseURI is supposed to be the baseURI from which reindexing will begin (if the req�" https://www.pivotaltracker.com/story/show/7564418421:20
* escowles joins22:21
* terrellt leaves
* escowles leaves22:26
* ksclarke joins
* ksclarke leaves22:31
* jcoyne joins22:33
<pivotal-bot>Longshou Situ added comment: "@awoods I've updated it to use reindexURI( child, "", true ) instead, which is cleaner. But this should prod�" https://www.pivotaltracker.com/story/show/7564418422:43
Longshou Situ finished "jms-indexer: Store jcr/xml in id hierarchy" https://www.pivotaltracker.com/story/show/75644184
* longshou leaves22:44
* ksclarke joins22:47
* ksclarke leaves23:05
* ksclarke joins23:25

Generated by Sualtam