Log of the #fcrepo channel on chat.freenode.net

Using timezone: Eastern Standard Time
* ksclarke leaves04:23
* sassrobi joins04:57
* sassrobi leaves
* awead joins07:07
* dwilcox joins07:18
* rlefaive joins07:48
* rlefaive leaves08:05
* dwilcox leaves08:22
* osmandin joins08:28
* dhlamb joins08:36
* dhlamb leaves08:37
* dhlamb joins08:38
* dwilcox joins08:42
* whikloj joins09:13
* github-ff joins09:18
[fcrepo-camel] daniel-dgi pushed 2 new commits to master: http://git.io/vcOrs
fcrepo-camel/master 95e74c7 Aaron Coburn: added maintainer section
fcrepo-camel/master 5d2fb30 Daniel Lamb: Merge pull request #94 from acoburn/fcrepo-1745...
* github-ff leaves
* travis-ci joins09:21
fcrepo4-exts/fcrepo-camel#225 (master - 5d2fb30 : Daniel Lamb): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-camel/compare/e65105f8d9a7...5d2fb305814c
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-camel/builds/82735098
* travis-ci leaves
* acoburn joins09:22
dhlamb: can you add a note to https://jira.duraspace.org/browse/FCREPO-1745 indicating that you merged fcrepo-camel PR #9409:23
* ajs6f joins09:24
<f4jenkins>Yippee, build fixed!09:27
Project fcrepo-camel-toolbox build #216: FIXED in 6 min 56 sec: http://jenkins.fcrepo.org/job/fcrepo-camel-toolbox/216/
<acoburn>dhlamb++09:29
* ksclarke joins09:41
* umgrosscol joins09:45
<acoburn>ajs6f: not sure if it's relevant, but the f-w-p artifact has an empty MANIFEST file inside the .war09:53
<ajs6f>acoburn: Dunno. I don't know much about this java crap. I try to avoid it. I was just trying to erase your extra deps and make a case that is actually comparable to a working case.09:54
acoburn: I'd like to see this tried out on Ubuntu, which is the only situation in which we've seen a problem.
<acoburn>ajs6f: I am not particularly familiar with the maven-war plugin, but it seems there may be an issue there (in the f-w-p project)
ajs6f: of course, that may just be a red herring
<ajs6f>acoburn: Yes, that's what I thought at first too. This overlay stuff can be tricky.09:55
<acoburn>ajs6f: initially we had a problem with everything (not just ubuntu)
<ajs6f>acoburn: I want to get awoods' PR merged first. Then move on.
acoburn: And your extra deps fixed that?
<acoburn>ajs6f: adding those extra deps solved (read: obscured) the problem on a mac and centos
<ajs6f>acoburn: Yeah, I'm sorry, but I really don't like that "solution".09:56
<acoburn>ajs6f: it's not a solution
<ajs6f>acoburn: Unless you can actually explain to me what's happening and why it workd.
<acoburn>ajs6f: and I can't
<ajs6f>acoburn: Then we have to kill those deps and get to the root of the thing.09:57
<acoburn>ajs6f: exactly, shall I amend my PR to fcrepo-camel-toolbox, or shall I leave it as-is for now?
<ajs6f>acoburn: Meaning take out the deps and let it fail universally?09:58
<acoburn>ajs6f: right — they need to come out anyway
<ajs6f>acoburn: Yeah, let's do that. Then at least we are solving the right problem. Or at least a righter problem.09:59
<acoburn>ajs6f: the deps have been removed10:01
<ajs6f>awoods: Is anyone from Duraspace coming to the APTrust Partner meeting next week?
<awoods>ajs6f: let me check...10:02
ajs6f: I will let you know when I hear back. Are you just curious? or do you have a specific reason for asking?10:04
<ajs6f>awoods: Nothing important. I'm going and I was wondering who I'll see.
acoburn: So now the build sequence shoul fail everywhere?10:06
<acoburn>ajs6f: yes
<ajs6f>acoburn: Good. More failure is a step forward. If we could get all the code to fail all the tests on every platform, we would finally be done.10:07
<acoburn>ajs6f: I now get this in the integration tests: Unable to locate Spring NamespaceHandler for XML schema namespace [http://www.springframework.org/schema/context]10:09
<ajs6f>acoburn: Yeah, that's what I suspected was caused by the mismatch in Spring versions.
Caused by: org.springframework.beans.factory.parsing.BeanDefinitionParsingException: Configuration problem: Unable to locate Spring NamespaceHandler for XML schema namespace [http://www.springframework.org/schema/context]
Offending resource: class path resource [spring/auth-repo.xml]
Right?
<acoburn>ajs6f: basically yes — I currently have a f-w-p built without authZ, but the error is functionally equivalent10:10
<ajs6f>acoburn: And you are working on a Mac?10:11
<acoburn>ajs6f: yes
<ajs6f>acoburn: Good, because I am using the Ubuntu vagrant thing.
<acoburn>ajs6f: the error above is what I get when everything is built from the fcrepo-1670 branches
ajs6f: that is: Offending resource: class path resource [spring/auth-repo.xml]10:12
<ajs6f>acoburn: Right, so now I think we want to introduce awoods' PR, which fixes the namespace. But I don't know whether we can do that directly, because we also need whikloj's edits to set in fcrepo-transform.
We need both.
<acoburn>ajs6f: I already tried adding awoods' edits, but no dice10:13
<ajs6f>acoburn: Meaning you got the exact same thing?!
<acoburn>ajs6f: yes
ajs6f: also, I thought it's recommended practice *not* to use spring versions at all in XSD locations10:14
ajs6f: (I tried that too, though with no better luck)
<ajs6f>acoburn: I don't care. Whatever you think is better. I *&*&)&)* hate Spring, anyway.
acoburn: That error normally means that the schema mapping files aren't being found in the classpath.10:15
<acoburn>ajs6f: right, but if you open up the war file, all the spring jars are right there
<ajs6f>acoburn: Is that enough? The files live in META-INF, but is that any META-INF in any jar in WEB-INF/lib, or is it META-INF for the WAR?10:16
acoburn: I thought it was the former, but..
<acoburn>ajs6f: no you're right — the namespace mappings are supposed to be in META-INF, but those mappings aren't there (at least I didn't see them)10:17
ajs6f: I just meant the jar files in WEB-INF/lib
<ajs6f>acoburn: Yeah, I checked that early on. It was actually that which made me notice the versions.10:18
acoburn: So how do we put the mappings in META-INF. If it is happen for f-w, why isn't it happening for f-w-p. Some kind of problem with Maven WAR overlay?10:19
acoburn: Was it working before? Presumably. So whikloj's PR breaks it?
<acoburn>ajs6f: we were never using f-w-p before10:20
<ajs6f>acoburn: Didn't we deploy it to the demo site?
<acoburn>ajs6f: yes, but fcrepo-transform has always been part of f-w, and f-w has always worked10:21
<ajs6f>acoburn: So you are saying that f-w-p has never worked?10:22
<acoburn>ajs6f: I'm saying that I've never tried using f-w-p with jetty-maven before
ajs6f: afaik, f-w-p works fine in standalone mode
<ajs6f>acoburn: II'm starting to wonder if I shouldn't just start from scratch, discard whikloj's PR to f-c-t, and do it over with Cargo.10:24
acoburn: But you tried that, didn't you?
<acoburn>ajs6f: only sort of10:25
ajs6f: I have never used cargo and I think I was doing it wrong
<ajs6f>acoburn: Do you have a PR?
<acoburn>ajs6f: no
<ajs6f>acoburn: Or rather, a branch.
<acoburn>ajs6f: I don't think so, but I'll look10:26
<ajs6f>acoburn: Don't forget to look on the kitchen counters. I'm always putting stuff down there and forgetting it.10:27
<acoburn>ajs6f: nope, I must not have kept the branch10:28
<ajs6f>acoburn: So what you're saying is that you planned this out so that I would suffer dreary toil? Okay, if that's how you feel. I'll go make a Cargo branch.10:29
<acoburn>ajs6f: that has been my intention all along, I had just hoped you wouldn't catch on10:30
<ajs6f>acoburn: I assume that about all of my colleagues and plan for it for being so blatantly indifferent that it's not usually worth anyone's time to get work out of me.10:32
acoburn: I'm starting from master here for f-c-t and I don't see any of this profiles for different auth providers stuff. When did that come in? Do you want that?10:38
<acoburn>ajs6f: the way it's currently set up is it uses the "default" profile for f-w-p, which is rbacl10:39
ajs6f: (I agree that these different auth providers should be separate artifacts, but that's a separate issue)
<ajs6f>acoburn: Okay. Is master using J-M for ITs? I don't see it config'd to do that?10:40
acoburn: Oh, wait. I see it sometimes used and sometimes not used. Like it's not used for fcrepo-audit. Are you using EmbeddedFuskei there?10:41
<acoburn>ajs6f: we must be (I know we should move away from that)10:42
<ajs6f>acoburn: I'm going wholesale on this. I'm going to use Fuskei2 and Cargo. It's not hard, and we need to stop using Fuseki1.
<acoburn>ajs6f++
<ajs6f>acoburn: ITs are supposed to be models of behavior.
<acoburn>ajs6f: I know. At the time I was also cribbing from fcrepo-message-consumer10:43
<ajs6f>acoburn: Here:10:45
https://github.com/ocymum/trippi-sparql-update/blob/master/pom.xml
is an example I did for another reason.
<acoburn>ajs6f: nice, cargo is so much cleaner10:46
<ajs6f>acoburn: and more powerful.
<osmandin>Speaking of IT, it looks like ModeShape has tests even for clustering. https://github.com/ModeShape/modeshape/blob/master/modeshape-jcr/src/test/java/org/modeshape/jcr/clustering/ClusteringServiceTest.java10:53
<ajs6f>I thought awoods said no one cares about clustering any more?10:55
<osmandin>I think my sytem team cares, but they are not on F4 yet.10:56
<awoods>ajs6f: people care in the abstract about clustering.10:57
<ajs6f>awoods: I care in the abstract about America's inner cities.11:00
<awoods>ajs6f: exactly
<osmandin>It's a non-functional requirement, so it's probably going to be abstract.11:01
<ajs6f>acborun: These Camel recipes, they are bundles?11:02
<acoburn>ajs6f: yes, they are all bundles
ajs6f: actually, I've only ever used them with karaf
<ajs6f>acoburn: That is tricker. Cargo wants to deploy all the components for an IT into the same container. Now I have to find a container that accepts bundle and WARs like Fuskei and Solr.11:03
<acoburn>ajs6f: ah that sounds familiar — I think that's where I got stuck
<whikloj>ajs6f: have you considered PaxExam?11:04
<ajs6f>whikloj: Yes. and I may do that. but that will be more work, and I don't get paid to work. I get paid to avoid work.
<whikloj>ajs6f++11:05
<acoburn>ajs6f: for the ITs you only need to deploy fedora in cargo, not the camel recipes, correct?
ajs6f: running the camel routes is handled by BlueprintTestSupport
<ajs6f>acoburn: No, that's not how cargo works. Cargo wants to deploy the artifact you are actually building, because that is in fact how an IT normally works. I might be bale to tun that off, tho'.11:06
<acoburn>ajs6f: oh right, now I remember11:07
* acoburn leaves11:17
<ruebot>awoods: shall i add stroop? https://github.com/duraspace/pcdm/wiki/PCDM-Committers13:07
<awoods>ruebot: please
<ruebot>awoods: i guess i should make it alphabetical too :-)
<awoods>ruebot: your call13:08
<ruebot>awoods: done.
awoods: where is the committers policy/process going to live? github wiki or confluence wiki?13:13
<awoods>ruebot: github would be good13:14
<ajs6f>acoburn: ping
<ruebot>awoods: shall i just create a page then?13:15
awoods: or are we still waiting?
<awoods>ruebot: please, then send it out for committer review
<ruebot>awoods: done.13:27
<awoods>ruebot++
* rlefaive joins14:37
<ruebot>awoods: can i ask a fedora committers process question here? curious how y'all count -1 -- is it a veto, or just minus 1? we're seeking clarification in islandoraland.14:51
* acoburn joins15:02
<awoods>ruebot: veto
<ruebot>awoods: thanks
<acoburn>ruebot: veto votes usually require some further elaboration, though15:03
ruebot: it mostly follows asf: http://www.apache.org/foundation/voting.html
* acoburn leaves15:05
<ruebot>acoburn++ awoods++
<ajs6f>ruebot: I think I'm the only committer who has issued a -1, and it got followed up by a very long and full discussion. -1 is veto, but it really means "veto accompanied by full explanation and readiness to continue the discussion".15:08
<ruebot>ajs6f++15:11
thank y'all! this is all very helpful.15:13
ajs6f: do you mind if i quote you on a reply to a islandora-committers thread?15:14
<ajs6f>ruebot: You mean what I just wrote about -1?15:18
<ruebot>ajs6f: yes.15:19
<ajs6f>ruebot: Sure
<ruebot>ajs6f++ thank you!
* acoburn joins15:34
* github-ff joins15:48
[fcrepo-camel-toolbox] awoods pushed 2 new commits to master: http://git.io/vcsxy
fcrepo-camel-toolbox/master 0577257 Aaron Coburn: use xtokenize() function for xml streaming
fcrepo-camel-toolbox/master 274d2d3 Andrew Woods: Merge pull request #56 from acoburn/fcrepo-1748...
* github-ff leaves
<osmandin>I'm trying to build ModeShape, and getting a variety of test failures (on Cent OS and Windows). Has anyone had recent success with "mvn clean install" for ModeShape lately?15:53
<awoods>osmandin: I have never been able to successfully build ModeShape (ubuntu)
<osmandin>awoods: Thanks15:54
<f4jenkins>Project fcrepo-camel-toolbox build #217: UNSTABLE in 7 min 28 sec: http://jenkins.fcrepo.org/job/fcrepo-camel-toolbox/217/15:55
Aaron Coburn: use xtokenize() function for xml streaming
<ruebot>osmandin, awoods: i can give it a shot in ubuntu too if you want.15:58
<awoods>ruebot: it would be interesting to see
<ruebot>awoods: https://github.com/ModeShape/modeshape -- that? if so, which branch?16:00
<awoods>ruebot: building their master branch would be adequate to know if you can get it to work. See: https://github.com/ModeShape/modeshape#building-modeshape16:01
* dhlamb leaves
* rlefaive leaves16:02
<ruebot>awoods: here goes nothing!16:08
<awoods>ruebot: good luck
afk
<acoburn>ajs6f: actually, I just got Update::visit to work with UpdateEngineWorker16:12
ajs6f: it wasn't too difficult, I just needed to read the jena source code16:13
<ajs6f>acoburn: I believe you. But that seems wrong to me. (Using UpEW is wrong.) That's a misuse of the semantics of that type.16:14
<acoburn>ajs6f: so you're suggesting implementing our own UpdateVisitor
ajs6f: here's the relevant diff: https://gist.github.com/acoburn/a32f5955f59a5b78544016:16
<ajs6f>acoburn: More or less. I'm not the Jena King. You may want to check it on users@jena. UpEW is meant for actually executing, not validating.
acoburn: Hence all those protected exec*() methods.16:17
<acoburn>ajs6f: yes, I saw those
<ajs6f>acoburn: The expectation thereby implied is that after UpEW visits a request, that request will have been executed against the DatasetGraph that was used to construct that UEw.16:18
<acoburn>ajs6f: there were only two implementations of UpdateWorker in the jena source code, and UpEW was the more relevant — the other was for serializing to an output stream
<ajs6f>acoburn; Using a subtype of UEW to do only validation would violate that. Unless you are talking about actually _doing_ the execution with our potential subtype?16:19
<acoburn>s/UpdateWorker/UpdateVisitor/
<ajs6f>acoburn: Yes, there isn't a convenient impl available for you to subclass. That's why I suggested starting with UpdateVisitor tself.
<acoburn>ajs6f: that makes sense16:20
<ajs6f>acoburn: The design question is whether we want to use the Jena API to validate and execute in one move, or whether we want to validate and execute in two moves using (for each move) either our fresh code of Jena's API.
s/of/or
acoburn: I'm not a fan of total committment to a library, but we do have a special relationship with jena.16:21
* dwilcox leaves16:41
* osmandin leaves16:44
* ajs6f leaves16:46
<ruebot>awoods: https://gist.github.com/ruebot/5c15cca8c97948a20e96 -- failed for me too.16:53
* dwilcox joins17:02
* whikloj leaves17:03
<awoods>ruebot: we will have to check with osmandin tomorrow to see if he gets the same errors.17:13
<ruebot>awoods: sounds like a plan.17:14
* acoburn leaves17:25
* jgpawletko leaves17:26
* dwilcox leaves17:34
* github-ff joins17:46
[fcrepo4] awoods pushed 3 new commits to master: http://git.io/vcGrX
fcrepo4/master 5317d95 Aaron Coburn: set build-tools version
fcrepo4/master c43416d Aaron Coburn: remove build-tools dependency in bom
fcrepo4/master 2f5751b Andrew Woods: Merge pull request #909 from acoburn/fcrepo-1728...
* github-ff leaves
* github-ff joins
[fcrepo4-oaiprovider] awoods pushed 2 new commits to master: http://git.io/vcGr7
fcrepo4-oaiprovider/master 4f59cea Aaron Coburn: decouple build-tools version from project.version
fcrepo4-oaiprovider/master 9587c72 Andrew Woods: Merge pull request #23 from acoburn/fcrepo-1728...
* github-ff leaves
* github-ff joins17:47
[fcrepo-message-consumer] awoods closed pull request #96: decouple build tools version from project version (master...fcrepo-1728) http://git.io/vn2q6
* github-ff leaves
* github-ff joins
[fcrepo4-client] awoods pushed 2 new commits to master: http://git.io/vcGov
fcrepo4-client/master f665d98 Aaron Coburn: decouple fcrepo-build-tools version from project version
fcrepo4-client/master e96b1a4 Andrew Woods: Merge pull request #30 from acoburn/fcrepo-1728...
* github-ff leaves
<f4jenkins>Project fcrepo4-oaiprovider build #422: UNSTABLE in 1 min 29 sec: http://jenkins.fcrepo.org/job/fcrepo4-oaiprovider/422/17:48
Aaron Coburn: decouple build-tools version from project.version
* jgpawletko joins17:51
* jgpawletko leaves
* travis-ci joins
fcrepo4-exts/fcrepo-message-consumer#161 (master - 3c1169d : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-message-consumer/compare/cbacb084c0ca...3c1169db0fca
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-message-consumer/builds/82827021
* travis-ci leaves
* travis-ci joins17:52
fcrepo4-labs/fcrepo4-client#82 (master - e96b1a4 : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4-labs/fcrepo4-client/compare/c5e14b9e95bc...e96b1a4802db
Build details : https://travis-ci.org/fcrepo4-labs/fcrepo4-client/builds/82827049
* travis-ci leaves
* travis-ci joins
fcrepo4-labs/fcrepo4-oaiprovider#92 (master - 9587c72 : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4-labs/fcrepo4-oaiprovider/compare/8a44d386282d...9587c729b5de
Build details : https://travis-ci.org/fcrepo4-labs/fcrepo4-oaiprovider/builds/82826979
* travis-ci leaves
* travis-ci joins17:53
fcrepo4-exts/migration-utils#96 (master - 7a2f539 : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4-exts/migration-utils/compare/e81c6ae5dd13...7a2f5394c97b
Build details : https://travis-ci.org/fcrepo4-exts/migration-utils/builds/82827004
* travis-ci leaves
<f4jenkins>Yippee, build fixed!17:55
Project fcrepo4 build #3089: FIXED in 9 min 29 sec: http://jenkins.fcrepo.org/job/fcrepo4/3089/
* Aaron Coburn: set build-tools version
* Aaron Coburn: remove build-tools dependency in bom
Yippee, build fixed!17:57
Project fcrepo4-oaiprovider build #423: FIXED in 1 min 17 sec: http://jenkins.fcrepo.org/job/fcrepo4-oaiprovider/423/
* travis-ci joins18:03
fcrepo4/fcrepo4#4069 (master - 2f5751b : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/51d92027bc6a...2f5751b03371
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/82826908
* travis-ci leaves
<f4jenkins>Yippee, build fixed!18:08
Project fcrepo-camel-toolbox build #218: FIXED in 7 min 11 sec: http://jenkins.fcrepo.org/job/fcrepo-camel-toolbox/218/
* umgrosscol leaves18:18
* ksclarke leaves18:20
* dwilcox joins18:34
* ksclarke joins18:46
* dwilcox leaves19:05
* ksclarke leaves19:35
* the_mgt_ joins19:52
* the_mgt leaves19:55
* ksclarke joins19:59
* ksclarke leaves20:04
* dhlamb joins20:12
* ksclarke joins20:18
* ksclarke leaves20:24
* ksclarke joins20:43
* ksclarke leaves20:47
* ksclarke joins20:48
* github-ff joins21:03
[fcrepo4-vagrant] ruebot pushed 3 new commits to master: http://git.io/vcZsi
fcrepo4-vagrant/master 8e3278a Aaron Coburn: update README to reflect new github location
fcrepo4-vagrant/master d71bba6 Aaron Coburn: add a maintainers section
fcrepo4-vagrant/master 770e37b Nick Ruest: Merge pull request #24 from acoburn/fcrepo-1683...
* github-ff leaves
* awead_away leaves21:28
* awead joins21:29
* jgpawletko joins21:39
* awead leaves22:06
* awead joins22:10
* awead leaves22:48
* awead joins22:51
* awead leaves22:58
* dhlamb leaves23:30
* jgpawletko leaves00:23