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

Using timezone: Eastern Standard Time
<bljenkins>Project fcrepo-kitchen-sink build #60: FAILURE in 2 hr 47 min: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/60/02:16
Yippie, build fixed!02:21
Project fcrepo-kitchen-sink build #61: FIXED in 4 min 40 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/61/
* kaarefc joins02:40
* kaarefc leaves03:10
* kaarefc joins03:54
* nbanks joins04:47
* nbanks leaves05:04
* nbanks joins05:32
* nbanks leaves05:46
* nbanks joins06:15
* nbanks leaves06:30
* nbanks joins06:58
* nbanks leaves07:12
* fasseg joins07:15
morning!07:16
* kaarefc leaves07:24
* nbanks joins07:40
<bljenkins>Yippie, build fixed!07:43
Project fcrepo-fixity build #17: FIXED in 1 min 8 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/17/
<fasseg>redeploy seems to work now too...
eddies: I could fly over there and kiss you! I had a heard time sleeping because of this and now i have hint what the problem might be....07:44
*hard
too early in the morning :/ I just posted jenkins credentials on the ff-tech list :( I was able to delete the message afterwards, but i guess it's already gone through the internet ...07:51
sorry about that...
cbeer: I'll change the manager credentials on futures6's tomcat installation and the corresponding jenkins projects if it's alright with you...07:54
* nbanks leaves07:56
<bljenkins>Project fcrepo-fixity build #18: SUCCESS in 1 min 1 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/18/08:00
<fasseg>cbeer: I updated the tomcat manager credetnials used for deploying to tomcat...08:04
I'll checkout how often we will have to restart tomcat, because of redeployment/permgen outofmemory errors, so it might get a bit unstable...08:11
<bljenkins>Project fcrepo-fixity build #19: SUCCESS in 1 min 5 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/19/08:13
Project fcrepo-kitchen-sink build #62: SUCCESS in 2 min 12 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/62/08:14
Project fcrepo-kitchen-sink build #63: SUCCESS in 2 min 24 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/63/08:17
Project fcrepo-fixity build #20: SUCCESS in 1 min 7 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/20/08:19
* VincentNG joins
<bljenkins>Project fcrepo-fixity build #21: SUCCESS in 1 min 5 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/21/08:20
Project fcrepo-fixity build #22: SUCCESS in 1 min 26 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/22/08:22
Project fcrepo-fixity build #23: SUCCESS in 1 min 14 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/23/08:24
* nbanks joins
* jcoyne joins08:27
<bljenkins>Project fcrepo-fixity build #24: SUCCESS in 1 min 22 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/24/08:28
Project fcrepo-fixity build #25: SUCCESS in 1 min 2 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/25/08:29
Project fcrepo-fixity build #26: SUCCESS in 1 min 24 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/26/08:31
* nbanks leaves08:39
* kaarefc joins08:41
<bljenkins>Project fcrepo-fixity build #27: SUCCESS in 1 min 11 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/27/09:00
<pivotal-bot>Frank Asseg started "Expose fcrepo 4.x url as a property in fcrepo-fixity-webapp" https://www.pivotaltracker.com/story/show/4619501709:01
Frank Asseg finished "Expose fcrepo 4.x url as a property in fcrepo-fixity-webapp" https://www.pivotaltracker.com/story/show/46195017
Frank Asseg edited "Expose fcrepo 4.x url as a property in fcrepo-fixity-webapp" https://www.pivotaltracker.com/story/show/46195017
Frank Asseg edited "Use RestEasy instead of CXF in fcrepo4" https://www.pivotaltracker.com/story/show/4618366709:02
* nbanks joins09:06
* kaarefc leaves09:13
<bljenkins>Project fcrepo-fixity build #28: SUCCESS in 1 min 20 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/28/
* nbanks leaves09:21
<pivotal-bot>Vincent Nguyen edited "Pass user authentication information from JAX-RS endpoints to the JCR repository" https://www.pivotaltracker.com/story/show/4545805509:27
Vincent Nguyen started "Add datastream delete to the batch-modify-object endpoint" https://www.pivotaltracker.com/story/show/4550656309:28
* nbanks joins09:39
* nbanks leaves09:43
* nbanks joins
* kaarefc joins09:47
* kaarefc leaves09:57
* kaarefc joins09:59
* kaarefc leaves10:18
* barmintor joins10:19
fasseg: is it desirable that the fixity web ui stay in a seperate webapp, or should it be an optionally included rest endpoint for the kitchen-sink?10:28
<cbeer>eddies: yes, i thought we knew fcrepo 4.x and the fixity app don't play nice together10:29
unless i missed something over night and fasseg fixed that?
<fasseg>cbeer: you missed eddie's fix
<cbeer>ok
barmintor: that was exactly my question too, and i feel like eddies waved his hands and i didn't feel like pressing the matter.10:30
<fasseg>on ff-tech, he posted that he'd put the webapps in /opt and created a deployment descriptor for the fixity webapp
now everything runs as smooth as on my local tomcat
<barmintor>cbeer: ok, I'm going to bring it up again at the standup :P
<fasseg>additionally i tried downloading a distribution of tomcatot 7 and installed that in /opt and there everything worked fine also
<cbeer>ok, well... those deployment descriptors will mess with the autodeploy, right?10:31
<fasseg>so it seems like a permission problem
it works fine right now
barmintor: as i undertood it the fixity-webapp is a standalone thing which you can also deploy on a different host, so i dont think merging those two would make sense...
<barmintor>fasseg: okay10:32
<cbeer>so, you're telling me if i...
right, so i restarted tomcat and it seems pretty hung to me.10:33
<fasseg>even my ssh connection is blocking ...10:35
let's talk to eddies and what he's done differently..
hmm tomcat is not running atm? are you restarting again?10:36
<cbeer>fasseg: yes.10:37
there's no point having tomcat hung.
") I've created fcrepo.xml and fcrepo-fixity.xml which point to the respective generated war files for those projects in /opt."10:38
i'm not seeing those files
<jcoyne>http://lh4.ggpht.com/-fM9glU4Zw8s/Tu-7SIiUEhI/AAAAAAAAGe4/lxfKnybHWls/Hung%252520Cat%25255B2%25255D.jpg?imgmax=80010:39
hanged tomcat ^
<fasseg> <exclusions>
<exclusion>
<groupId>commons-logging</groupId>
<artifactId>commons-logging</artifactId>
</exclusion>
</exclusions>
oops wrong one sry
/opt/fcrepo-fixity/fcrepo-fixity-webapp/target/fcrepo-fixity-webapp-4.0-SNAPSHOT.war
<cbeer>fasseg: no, the .xml files.10:40
<fasseg>but the deployment descriptors are gone
<cbeer>i don't see them, and we're not using those wars afaict.
* kaarefc joins10:42
<fasseg>hmm now did jenkins remove those, or the restart, afaik, the restart should not remove them....10:43
<cbeer>i
<fasseg>or rather thee tomcat redeploy
<cbeer>i'm going to disable the deploy step for the jenkins build for the fixity app10:46
it's clear they don't play well together. maybe stepping on each others toes or something.
<fasseg>did you recreate those files now or how did you restart it?
* kaarefc leaves
<cbeer>maybe barmintor can take a look
i recreated the fcrepo-fixity descriptor
<fasseg>cbeer: try running them in a local tomcat or with the tomcat in /opt on futures6....they play well together.10:47
<cbeer>"so, sorry fcrepo community.. you can't use the tomcat that comes installed on your system. use ours instead."10:48
<barmintor>:S
<fasseg>let me check if the rededploy step removes the xmls, or the restart...10:53
so maybe the serv will be hung for a bit
<eddies>hey gang…was out for dinner/drinks. back now10:54
let's plan to stay on after standup and talk through deployment-y issues10:55
at least fasseg, cbeer and whoever else is interested/has some thoughts on the matter10:56
<cbeer>+1. i'm curious what you did (and how it could possibily have fixed anything)
servlet containers are too much magic for me
ok, i have a stupid question.. how is "-Dorg.fcrepo.fixity.fcrepo.url" working? i thought i'd see something in the spring xml, but maybe there's a different place?11:00
<fasseg>it's in fedoraClient.java11:02
just a simple setter hack
<cbeer>ooh
<eddies>yeah, that threw me for a loop too11:03
<cbeer>i forgot about that project
<eddies>i kept grepping fcrepo-fixity for that
<fasseg>https://github.com/futures/fcrepo-client/blob/master/src/main/java/org/fcrepo/client/FedoraClient.java#L39
<eddies>and coming up emtpy
cbeer: joining us?
<cbeer>VincentNG: you have the misfortune of always sorting to the far side, sorry.11:04
<pivotal-bot>Edwin Shin accepted "Build HTML response for repository describe" https://www.pivotaltracker.com/story/show/4413451111:06
<barmintor>Out of nowhere, eddies has an avatar on G+11:07
<eddies>;-)
<VincentNG>cbeer: im actually on both ends since im the phone icon as well :-)
<pivotal-bot>Chris Beer added "Get code coverage statistics for fcrepo projects" https://www.pivotaltracker.com/story/show/4627457111:13
Chris Beer edited "Get code coverage statistics for fcrepo projects" https://www.pivotaltracker.com/story/show/46274571
<cbeer>fortunately, that ticket is assigned to barmintor11:14
<barmintor>Agenda: 1) Deployment; 2) Fixity division of labor11:17
<bljenkins>Project fcrepo-kitchen-sink build #64: ABORTED in 27 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/64/11:23
Project fcrepo-kitchen-sink build #65: SUCCESS in 3 min 17 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/65/11:27
<cbeer>building fixity.11:30
<barmintor>I wonder if we could use the message queue to avoid using that env variable for the fcrepo url
<bljenkins>Project fcrepo-fixity build #29: SUCCESS in 1 min 30 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/29/11:31
Project fcrepo-fixity build #30: SUCCESS in 1 min 17 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/30/11:44
<cbeer>fasseg: you've asked me to check this now?11:47
<fasseg>yes pls
<cbeer>k
<fasseg>changed the brokerurl to just use tcp and i could restart
<cbeer>i've seen this in the logs so far.. not sure it's an issue:11:48
WARN 15:48:29.673 (DefaultMessageListenerContainer) Could not refresh JMS Connection for destination 'org.fcrepo.services.fixity.queue' - retrying in 5000 ms. Cause: Could not connect to broker URL: tcp://localhost:61616. Reason: java.net.ConnectException: Connection refused
INFO 15:48:49.842 (DefaultMessageListenerContainer) Successfully refreshed JMS Connection11:49
guess not
fasseg++
<fasseg>yeah thats because the borker of fcrepo was not up yet
but as soon as its up it'll connect
<barmintor>. /fcrepo/obj/foo/ds/bar/fixity11:50
<cbeer>and without fcrepo up, there's no reason we'd try to send messages anyway
so i think this is good enough
<fasseg>or the user's concern with properties...
<barmintor>. /fcrepo/obj/foo/ds/bar?checksum=XXX&size=YYY
<eddies>&algorithm=xxx
<cbeer>this vaguely reminds me of the ARK? EZID? whatever service that lets you do ds/bar/fixity?? and get some metadata back11:51
<barmintor>http://otherhost/fixity
<fasseg>cbeer: the broker is also used as a queue, so the fixity webapp needs a broker toactually queue objects for checking..
so there could be a use case with a distinct broker for the fixity webapp
<cbeer>https://wiki.ucop.edu/download/attachments/16744455/arkspec.txt?version=1&modificationDate=126103680000012:03
. /fcrepo/obj/foo/ds/bar/fixity12:04
<barmintor>The third degenerate special case of an ARK request (and no other
cases will be described in this document) is the string "??",12:05
corresponding to a minimal permanence policy request.
<cbeer>. /fcrepo/obj/foo/ds/bar/fixity???
. /fcrepo/obj/foo/ds/bar/fixity
-- binary data --
<eddies>http://localhost:8080/rest/objects/foo/datastreams/bar
http://localhost:8080/rest/objects/foo/datastreams/bar/content
http://localhost:8080/rest/objects/foo/datastreams/bar??12:06
<cbeer>http://localhost:8080/rest/objects/foo/datastreams/bar/content
http://localhost:8080/rest/objects/foo/datastreams/bar/content???
. /fcrepo/obj/foo/ds/bar/fixity12:08
http://localhost:8080/rest/objects/foo/datastreams/bar/content12:09
X-FIXITY-CHECK-URL: . /fcrepo/obj/foo/ds/bar/fixity
http://localhost:8080/rest/objects/foo/datastreams/bar/content???
<barmintor>. /bar/content?fixity
<cbeer>http://localhost:8080/rest/objects/foo/datastreams/bar/content12:10
Accept: text/x-fixity-result
(death to the last case.)12:14
429 Too Many Requests (RFC 6585)12:22
The user has sent too many requests in a given amount of time. Intended for use with rate limiting schemes
ha:12:23
451 Unavailable For Legal Reasons (Internet draft)
Defined in the internet draft "A New HTTP Status Code for Legally-restricted Resources".[22] Intended to be used when resource access is denied for legal reasons, e.g. censorship or government-mandated blocked access. A reference to the 1953 dystopian novel Fahrenheit 451, where books are outlawed.[23]
http://tools.ietf.org/html/draft-nottingham-http-link-header-10#section-312:27
eddies: want to accept some tickets?12:29
<pivotal-bot>Chris Beer delivered "Expose a fcrepo java api that allows us to replace a data blob from a single infinispan cache store" https://www.pivotaltracker.com/story/show/46197281
* VincentNG leaves
<bljenkins>Project fcrepo-fixity build #31: SUCCESS in 1 min 0 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/31/
<pivotal-bot>Chris Beer added comment: "http://fcrepo4.fcrepo.org/fcrepo/rest/objects/sufia:td96k250z/datastreams/descMetadata/fixity" https://www.pivotaltracker.com/story/show/4611773512:30
<eddies>cbeer: can you make some events happen on fcrepo so i can see them at http://fcrepo4.fcrepo.org/fcrepo/rest/rss?
<cbeer>and, barmintor, deliver that one ^?
<eddies>then i can accept that one
<cbeer>done12:31
<fasseg>I added the following property using the PropertyPlaceholderConfigurer "org.fcrepo.fixity.brokerurl" with a default value of tcp://localhost:61616 but you can use a different one by e.g. runing tomcat with CATALINA_OPTS="-Dorg.fcrepo.fixity.brokerurl=vm://fixityBroker" bin/catalina.sh run
<cbeer>(ooh, we need a jms -> irc component..)
+1
thanks fasseg
<eddies>they all happened at the same time :P
<cbeer>1 second apart12:32
<fasseg>im off for a while now...
<barmintor>cbeer: good, now we can use that to cause the fixity errors for tests
<eddies>cbeer, fasseg: that should be JAVA_OPTS added to /etc/default/tomcat712:33
<pivotal-bot>Edwin Shin accepted "Object search-like API built on top of JCR query API" https://www.pivotaltracker.com/story/show/45464711
<cbeer>yes, because i think the default should be a vm:// broker
maybe
<barmintor>cbeer++ // delivered
<cbeer>except that it's clear that doesn't work in a deployment scenario, i guess
<pivotal-bot>Edwin Shin added comment: "Frank, can you add a pointer to the commit as a comment here?" https://www.pivotaltracker.com/story/show/4619501712:34
Edwin Shin accepted "Expose fcrepo 4.x url as a property in fcrepo-fixity-webapp" https://www.pivotaltracker.com/story/show/46195017
<cbeer>hm. i appear to have misplaced a test
<pivotal-bot>Edwin Shin accepted "fcrepo 4 events feed should be sorted reverse-chronological." https://www.pivotaltracker.com/story/show/4551205712:35
<cbeer>huh.12:36
<pivotal-bot>Edwin Shin added comment: "Is there some trivial service we can hook into this to demonstrate?" https://www.pivotaltracker.com/story/show/4546462312:37
<cbeer>ides--12:38
<eddies>there's a joke about the ides of march to be made here12:40
or at least, in your time zones if not mine anymore
<cbeer>anyway, this is good enough.. barmintor, i assume you saw https://github.com/futures/fcrepo4/blob/master/fcrepo-kernel/src/test/java/org/fcrepo/utils/LowLevelCacheStoreTest.java#L9112:41
for messing up your fixity
eddies: https://www.pivotaltracker.com/story/show/46122883
<pivotal-bot>feature: (Placeholder ticket to do something useful with webhooks) (unscheduled) / owner:
* github-ff joins12:42
[fcrepo4] cbeer pushed 1 new commit to master: http://git.io/fH-2WQ
fcrepo4/master 4f9dc8f Chris Beer: Merge pull request #37 from futures/webhooks...
* github-ff leaves
<cbeer>or if you want really trivial:
https://github.com/futures/fcrepo4/commit/24157f1f0ee642479b74365d1836ef29043189b4#L2R58
there's always that :P
i think the problem is, we'd just write some jms listener or eventbus listener whereever that'd be useful12:43
maybe there's a mash-up with https://github.com/captn3m0/ifttt-webhook12:44
<eddies>oh wait. that wasn't merged till just now?12:45
so it's not available on futures6 at the moment?
<cbeer>yeah, guess not, sorry.12:46
<eddies>i was just going to run it against http://requestb.in/ys17ehys?inspect for now
<cbeer>oh, hey, that's pretty nifty.
<barmintor>out to lunch, back soon12:47
<bljenkins>Project fcrepo-kitchen-sink build #66: SUCCESS in 2 min 31 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/66/12:51
Project fcrepo-fixity build #32: SUCCESS in 2 min 25 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/32/12:52
<eddies>cbeer: am i supposed to be able hit http://fcrepo4.fcrepo.org/fcrepo/rest/webhooks?12:53
<cbeer>oh, it's probably not wired into the kitchen sink rest.xml12:55
* nbanks leaves12:56
<eddies>when fasseg gets cxf switched out for resteasy, hopefully these issues will become a distant memory
* github-ff joins12:58
[fcrepo-kitchen-sink] cbeer pushed 1 new commit to master: http://git.io/asrLuw
fcrepo-kitchen-sink/master 6895af0 Chris Beer: wire in webhooks
* github-ff leaves
* travis-ci joins13:02
[travis-ci] futures/fcrepo4#242 (master - 4f9dc8f : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/f0e3103e49bc...4f9dc8f72707
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/5531546
* travis-ci leaves
<bljenkins>Project fcrepo-kitchen-sink build #67: SUCCESS in 1 min 56 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/67/13:09
Project fcrepo-fixity build #33: SUCCESS in 1 min 36 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/33/
<eddies>there's a whole lotta WARN 17:10:02.946 (DefaultMessageListenerContainer) Could not refresh JMS Connection for destination 'org.fcrepo.services.fixity.queue' - retrying in 5000 ms. Cause: Could not connect to broker URL: tcp://localhost:61616. Reason: java.net.ConnectException: Connection refused13:20
looks like kitchen sink didn't redeploy correctly...13:21
nested exception is org.springframework.beans.factory.CannotLoadBeanClassException: Cannot find class [org.fcrepo.webhooks.FedoraWebhooks] for bean with name 'org.fcrepo.webhooks.FedoraWebhooks#50524b34' defined in ServletContext resource [/WEB-INF/classes/spring/rest.xml]; nested exception is java.lang.ClassNotFoundException: org.fcrepo.webhooks.FedoraWebhooks13:22
* nbanks joins
<eddies>I don't see fcrepo-webhooks in /var/lib/tomcat7/webapps/fcrepo/WEB-INF/lib13:24
* github-ff joins13:29
[fcrepo-kitchen-sink] eddies pushed 1 new commit to master: http://git.io/fPBr4g
fcrepo-kitchen-sink/master 85c54e8 Edwin Shin: 45464605: REST API support for registering webhook listeners...
* github-ff leaves
* github-ff joins13:31
[fcrepo4] eddies pushed 1 new commit to master: http://git.io/j5xWOw
fcrepo4/master 891f181 Edwin Shin: minor: clean up imports
* github-ff leaves
* github-ff joins13:35
[fcrepo4] eddies pushed 2 new commits to master: http://git.io/jxozhA
fcrepo4/master 1773bd7 Edwin Shin: minor: clean up imports
fcrepo4/master 84fc2d1 Edwin Shin: minor: removes unused get-checksum-as-a-string call
* github-ff leaves
<bljenkins>Project fcrepo-kitchen-sink build #68: SUCCESS in 2 min 13 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/68/13:36
* travis-ci joins13:37
[travis-ci] futures/fcrepo4#243 (master - 891f181 : Edwin Shin): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/4f9dc8f72707...891f18145d79
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/5532834
* travis-ci leaves
<bljenkins>Project fcrepo-kitchen-sink build #69: FAILURE in 2 min 54 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/69/13:43
<cbeer>permgen space.
<bljenkins>Yippie, build fixed!13:46
Project fcrepo-kitchen-sink build #70: FIXED in 2 min 11 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/70/
Project fcrepo-fixity build #34: SUCCESS in 1 min 56 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/34/13:47
* github-ff joins13:49
[fcrepo4] eddies pushed 1 new commit to master: http://git.io/0MkuPQ
fcrepo4/master 06c1a27 Edwin Shin: minor: cleans up dependency declarations. I'm pretty sure we're still pulling in too much in the way of transitive dependencies, however.
* github-ff leaves
<eddies>i'm not sure where the following is coming from in the tomcat logs:13:50
gJAXRSServerFactoryBean--370414220': Initialization of bean failed; nested exception is java.lang.NoClassDefFoundError: org/apache/http/conn/ClientConnectionManager
at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:532) ~[spring-beans-3.2.0.RELEASE.jar:3.2.0.RELEASE]
<cbeer>hm, i thought we saw that before13:52
<bljenkins>Project fcrepo4 build #197: FAILURE in 3 min 6 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo4/197/
Edwin Shin: minor: cleans up dependency declarations. I'm pretty sure we're still pulling in too much in the way of transitive dependencies, however.
<cbeer>and one of you told me what to do about it
<eddies>eddies— for breaking the build13:54
weird. tests ran fine locally. clean install and it's broken here too though13:55
sigh
* travis-ci joins
[travis-ci] futures/fcrepo4#244 (master - 84fc2d1 : Edwin Shin): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/891f18145d79...84fc2d152c78
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/5532956
* travis-ci leaves
<cbeer>do we need https://github.com/futures/fcrepo4/commit/06c1a27f69d75e50098d7a717628d418fac49958#L0L63 ?
i think it's declared as a test dependency somewhere else
<eddies>it's still there13:56
https://github.com/futures/fcrepo4/commit/06c1a27f69d75e50098d7a717628d418fac49958#L0R27
<cbeer>k
the diff is just a little hard to read
would have been nice if it was that easy13:57
oh, re: https://github.com/futures/fcrepo4/commit/84fc2d152c7814846536be53673705254a0c9c8f
we should also kill the method too13:58
that's the only place it was being used
but i didn't take it another step to see if we were using the result somewhere :P
<eddies>oh i think i know what the error is...sec14:00
cbeer: you mean get rid of: https://github.com/futures/fcrepo4/blob/master/fcrepo-kernel/src/main/java/org/fcrepo/utils/ContentDigest.java#L2614:06
<cbeer>yes
<eddies>you're right, the only thing i see using it now is the test for it
<cbeer>yep
<eddies>cbeer: is https://github.com/futures/fcrepo4/blob/master/fcrepo-kernel/src/test/java/org/fcrepo/utils/infinispan/StoreChunkInputStreamTest.java#L2414:10
having some side-effect?
<cbeer>like?
<eddies>i dunno. we're not using store anywhere in that test
same with https://github.com/futures/fcrepo4/blob/master/fcrepo-kernel/src/test/java/org/fcrepo/utils/infinispan/StoreChunkInputStreamTest.java#L4114:11
<cbeer>oh, no, we're not.
nope, that was debugging
<eddies>so go ahead and delete those lines?
<cbeer>please
hm. who creates the property "size" on /objects?14:14
<eddies>me? dunno. haven't really looked at that code14:19
<cbeer>just musing. i'm pretty confused at that code, and it was breaking coverage on the http-api14:20
so i'm going to assume it's well tested and just refactor it14:21
must be tested well enough, in that it's broke now.14:23
* nbanks leaves
<cbeer>ah, we have multiple copies of the cnd floating around.14:26
* github-ff joins14:28
[fcrepo4] eddies pushed 6 new commits to master: http://git.io/IZfSWw
fcrepo4/master f38142a Edwin Shin: minor: more import cleanups
fcrepo4/master 607b066 Edwin Shin: Corrects my overzealous dependency cleanup
fcrepo4/master 102482a Edwin Shin: Removes unused asChecksumString method
* github-ff leaves
<eddies>ok, stepping out to take the dog out. hopefully when i return, jenkins will report success14:30
* VincentNG joins14:31
<eddies>that'll teach me to do refactoring after drinks
<bljenkins>Yippie, build fixed!14:33
Project fcrepo4 build #198: FIXED in 5 min 18 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo4/198/
* Edwin Shin: minor: more import cleanups
* Edwin Shin: Corrects my overzealous dependency cleanup
* Edwin Shin: Removes unused asChecksumString method
* Edwin Shin: should be using org.junit.Assert.assertEquals, not junit.framework.Assert.assertEquals (latter is deprecated)
* Edwin Shin: Removes unused code. Cleans up imports.
* Edwin Shin: FixityInputStream wasn't being closed
<cbeer>eddies++ so far, so good14:34
<VincentNG>anyone know how to write a test case for this call? curl "http://localhost:8080/rest/objects/test1/datastreams?dsids={ds1,ds2,ds3}" -X DELETE14:36
<bljenkins>Project fcrepo-kitchen-sink build #71: SUCCESS in 2 min 27 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/71/
<VincentNG>tried using BasicHttpParams, but no luck
<bljenkins>Project fcrepo-fixity build #35: SUCCESS in 1 min 33 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/35/14:37
<cbeer>hm. apparently fcrepo-http-commons is where we create an object critical to all fcrepo. interesting.
<barmintor>VincentNG: could the query params be restructured like ?dsid=ds1&dsid=ds2&dsid=ds314:38
<VincentNG>would that come in as a list still?14:40
* nbanks joins
<barmintor>VincentNG: you should get an array of dsid vals at the http layer, so I imagine that restEasy would map it to a list14:41
might be worth checking into
<VincentNG>ok, let me try that. thanks
<eddies>vincentng: that's basically what's happening here: http://mediashelf.github.com/fedora-batch/14:45
and there are some test cases for it in fedora-client (BatchIT or Batch-something-or-other)14:46
i'm going to restart tomcat14:47
(on futures6)
<barmintor>dsID or dsId?14:55
<cbeer>Caused by: java.lang.ClassFormatError: Absent Code attribute in method that is not native or abstract in class file javax/transaction/SystemException
does that make sense to anyone?
<eddies>barmintor: although i reserve the right to needlessly bitch and moan later, my only immediate request is to pick one style and stick with it consistently ;-)14:56
cbeer: i don't think i've run into that before
<barmintor>cbeer: no, but it sounds like two different components were built against non-binary compat version of the jtx library
<cbeer>k. i'm trying to get tests in fcrepo-http-commons and flailing bad.
eddies: what are all the exclusions in the http-commons pom about (for fcrepo-kernel)?14:57
<barmintor>eddies: agreed on all counts. but which one? whcih?
<eddies>fcrepo-webapp is fine, but kitchen-sink not so much…so my spectacular deductive skills say it must have something to do w/ the addition of fcrepo-webhooks
cbeer: i was attempting to exclude transitive dependencies that weren't actually needed14:58
so far as i could tell, i didn't break anything at the time
<cbeer>ok. so, if i think i need some of them for tests.. should i just re-add them as scope:test dependencies?
<eddies>yah14:59
barmintor: dsId?
<barmintor>THE PRODUCT MANAGER HAS SPOKEN
<eddies>isn't that the camelCase way?
pfft
<barmintor>I think it's silly to have two different jaxb bindings for a fixity check, so I'm trying to put the missing info in the http-commons binding15:00
cbeer: I'm going to get rid of the FixityStatus jaxb binding in http-comons, and elaborate kernel/org/fcrepo/util/FixityResult to have it satisfy both cases.15:07
<cbeer>sure.
<barmintor>that is also a silly duplication15:08
<cbeer>it's a durability strategy.
<eddies>ok. it must be late. but i lol'ed
<barmintor>you want to insulate the kernel from the needs of the http components?15:09
* barmintor considers this
<cbeer>when you put it like that, it sounds like i made a serious suggestion.
<eddies>i think ajs6f expressed that intent, actually
or something along those lines...15:10
<barmintor>whatever, I love deleting code.
<cbeer>actually, we were keeping jax-b stuff out of kernel
<barmintor>NOOOOOOOOOO
technically it's just annotations15:11
do you have to pull in a bg framework for that?
I can just find out by trying to mark the class up, hold on15:12
<pivotal-bot>Edwin Shin added comment: "https://github.com/ModeShape/modeshape/commit/557c141a71469823f37bb033f3841ce64b293c35" https://www.pivotaltracker.com/story/show/4566637915:13
<barmintor>I don;t think it adds any dependencies beyond the existing javaee-api
<eddies>cbeer: i'm fairly certain fcrepo-webhooks has some maven dependency scope issue15:14
<cbeer>eddies: i'm fairly certain i don't know what i'm doing, so you're probably right.
<eddies>did you actually test it in a running fcrepo4 webapp
(not just the integration tests)15:15
<cbeer>hm. maybe just the tests
(REJECTED!)
hm. i got a lot of webhooks failures now.15:16
<eddies>i think because some of the various httpclient/httpcomponents dependencies are marked as test scope in the fcrepo4 parent pom, they're not getting included in the war
which is why we get the classnotfound errors for the org.apache.http* classes
<cbeer>k. i'm going to have lunch, then i'll take a look15:17
<eddies>i'm gonna pull the webhooks dependency out of kitchen-sink for now, though, so the futures6 deploy gets back to running
* github-ff joins15:31
[fcrepo-kitchen-sink] eddies pushed 1 new commit to master: http://git.io/FJaBlA
fcrepo-kitchen-sink/master 247519e Edwin Shin: Explicitly adds httpclient as a compile scope dependency....
* github-ff leaves
<eddies>cbeer: (for when you get back from lunch since i should be in bed by then) nm, i think i have a workaround for it in kitchen-sink
<cbeer>i'm here now15:32
<eddies>works locally anyway
<cbeer>that's.. unfortunate. someday you'll have to explain pom dependencies to me
<eddies>seems to be sorted now. but i think the real issue is that we have httpclient in the dependencyManagement section of the fcrepo parent pom that explicitly declare it as scope test15:33
* github-ff joins
[fcrepo4] cbeer pushed 1 new commit to master: http://git.io/h4xE1Q
fcrepo4/master 5549e1f Chris Beer: remove duplicate dependency
* github-ff leaves
<eddies>probably because whoever added it was only using httpclient for tests at the time
<bljenkins>Project fcrepo4 build #199: UNSTABLE in 2 min 57 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo4/199/15:34
* Chris Beer: move /objects initialization into the kernel
* Chris Beer: we need -http-commons as a compile dependency
<eddies>but i think we should probably pull out the test scope from the dependencyManagement section. And then, for modules that actually need httpclient and only need it for testing, those modules can explicitly add test scope
<cbeer>although i suspect most of them use it for testing15:35
<eddies>hmm15:36
well, maybe there's a cleverer way to keep it as test in dependencyManagement but not get jammed up like we did in kitchen-sink…but i'm definitely too tired to see it now. mebbe barmintor or ajs6f would have a better idea15:38
<bljenkins>Yippie, build fixed!
Project fcrepo4 build #200: FIXED in 4 min 21 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo4/200/
Chris Beer: remove duplicate dependency
<barmintor>eddies: sorry, I've gone deep down a rabbit hole15:39
<eddies>it's really not that important now15:40
<bljenkins>Project fcrepo-kitchen-sink build #72: SUCCESS in 1 min 58 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/72/
<eddies>thank you jenkins
<bljenkins>Project fcrepo-fixity build #36: SUCCESS in 1 min 41 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/36/
<eddies>http://fcrepo4.fcrepo.org/fcrepo-fixity/static/results.html seems to be working at long last15:42
ok. i'm off to bed
<cbeer>oh, right, i was doing all that so i could try to run coverage on http-api15:43
* travis-ci joins15:48
[travis-ci] futures/fcrepo4#247 (master - 09a4e71 : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/6e11d7a67757...09a4e714cae7
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/5535581
* travis-ci leaves
* github-ff joins15:49
[fcrepo4] vnguyen745 opened pull request #39: Added delete to batch modify endpoint (master...batch-delete) http://git.io/GPI-YA
* github-ff leaves
* travis-ci joins
[travis-ci] futures/fcrepo4#248 (master - 5549e1f : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/09a4e714cae7...5549e1f68b3d
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/5535642
* travis-ci leaves
<pivotal-bot>Vincent Nguyen added comment: "https://github.com/futures/fcrepo4/pull/39" https://www.pivotaltracker.com/story/show/45506563
Vincent Nguyen edited "Add datastream delete to the batch-modify-object endpoint" https://www.pivotaltracker.com/story/show/45506563
Vincent Nguyen started "Pass user authentication information from JAX-RS endpoints to the JCR repository" https://www.pivotaltracker.com/story/show/45458055
* github-ff joins15:52
[fcrepo4] cbeer pushed 2 new commits to master: http://git.io/FMD25A
fcrepo4/master e632bcb Vincent Nguyen: Added delete to batch modify endpoint
fcrepo4/master 7eb89d2 Chris Beer: Merge pull request #39 from vnguyen745/batch-delete...
* github-ff leaves
* github-ff joins15:54
[fcrepo4] cbeer pushed 1 new commit to master: http://git.io/T1csQQ
fcrepo4/master 0fc8137 Chris Beer: fix weird etag match failure under some environments
* github-ff leaves
* travis-ci joins15:57
[travis-ci] futures/fcrepo4#250 (master - 7eb89d2 : Chris Beer): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/5549e1f68b3d...7eb89d214997
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/5536008
* travis-ci leaves
<bljenkins>Project fcrepo-kitchen-sink build #73: SUCCESS in 2 min 12 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/73/15:59
Project fcrepo-kitchen-sink build #74: SUCCESS in 2 min 14 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/74/16:03
Project fcrepo-fixity build #37: SUCCESS in 1 min 48 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/37/
* github-ff joins16:04
[fcrepo4] cbeer pushed 1 new commit to master: http://git.io/thTKXQ
fcrepo4/master 6e0b387 Chris Beer: also add delete multiple datastreams to the POST .../datastreams/ method.
* github-ff leaves
<barmintor>we're not auto-deploying the fixity app? I'm about to break some things temporarily16:06
<cbeer>i think we are
but do it anyawy.
<barmintor>crap
<cbeer>http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/16:08
* travis-ci joins16:09
[travis-ci] futures/fcrepo4#252 (master - 6e0b387 : Chris Beer): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/0fc81375b020...6e0b387c961f
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/5536312
* travis-ci leaves
<barmintor>wasn't me
(yet)
* travis-ci joins16:10
[travis-ci] futures/fcrepo4#251 (master - 0fc8137 : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/7eb89d214997...0fc81375b020
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/5536030
* travis-ci leaves
* github-ff joins16:11
[fcrepo4] barmintor pushed 1 new commit to master: http://git.io/HAZDBw
fcrepo4/master c0497ca Benjamin Armintor: refactoring out redundant classes
* github-ff leaves
<bljenkins>Project fcrepo-kitchen-sink build #75: FAILURE in 6 min 15 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/75/16:14
Project fcrepo-fixity build #38: FAILURE in 5 min 37 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/38/
Project fcrepo-kitchen-sink build #76: STILL FAILING in 1 min 3 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/76/16:16
* travis-ci joins
[travis-ci] futures/fcrepo4#253 (master - c0497ca : Benjamin Armintor): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/6e0b387c961f...c0497ca11308
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/5536453
* travis-ci leaves
<cbeer>auto-deploys would be a lot slicker if we didn't run out of permgen space
<bljenkins>Yippie, build fixed!16:17
Project fcrepo-fixity build #39: FIXED in 1 min 54 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/39/
Yippie, build fixed!16:19
Project fcrepo-kitchen-sink build #77: FIXED in 2 min 19 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/77/
<cbeer>eddies:16:20
curl --form "callbackUrl=http://requestb.in/ys17ehys" "http://fcrepo4.fcrepo.org/fcrepo/rest/webhooks/requestbin" -X POST
curl "http://fcrepo4.fcrepo.org/fcrepo/rest/objects/new" -X POST
>> http://requestb.in/ys17ehys?inspect
success.
<barmintor>dammit16:25
damn you, hibernate markup16:26
whatever, I can still make this class thinner16:27
* VincentNG leaves16:29
<cbeer>why are we storing stuff in hibernate anyway?16:31
why not persist it back to the repo?
<barmintor>one question at a time16:32
<cbeer>barmintor: i'm going to start building up some of the self-healing stuff, i guess16:42
<barmintor>ok
do I have to have fcrepo4 running to run the tests on the fixity-service?17:09
ah, bo17:10
I need to stub the service
<cbeer>grr. now i really have to figure out how to use different modeshape configurations across tests17:16
i guess there's probably nothing stopping me from spinning up a completely different repository instance17:22
* nbanks leaves17:35
<cbeer>singletons--17:45
<barmintor>they do have passive-evil tendencies17:46
<cbeer>i'm spending all my time writing this test fighting with the service singletons and spring17:48
* nbanks joins18:05
<cbeer>huh.18:10
* nbanks leaves18:15
<cbeer>oh, i bet i know what's happening.
modeshape is being too smart
(and i should purge the ISPN stores when we're testing)
* jcoyne leaves18:22
<cbeer>it's easier to get tests to pass if the code they are testing isn't broken18:32
<barmintor>they're closing the library, I am being required to skidaddle18:37
bye!
* barmintor leaves
<bljenkins>Project fcrepo-fixity build #40: SUCCESS in 1 min 7 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/40/18:38
* nbanks joins18:41
* nbanks leaves18:46
<cbeer>oh how i wish i got my commit in before barmintor.19:05
* github-ff joins19:29
[fcrepo4] cbeer pushed 6 new commits to master: http://git.io/d0zmmw
fcrepo4/master c756d46 Chris Beer: add setRepository method to make it easier to mess with the services' repository outside of spring (e.g. in poorly thought out tests.)
fcrepo4/master 22412ed Chris Beer: rename LowLevelCacheStore => LowLevelCacheEntry, and let it handle all the ISPN operations internally
fcrepo4/master e686677 Chris Beer: make sure to log-out our repository initialization session
* github-ff leaves
<bljenkins>Project fcrepo-kitchen-sink build #78: SUCCESS in 2 min 43 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-kitchen-sink/78/19:36
Project fcrepo-fixity build #41: SUCCESS in 2 min 38 sec: http://ci.projectblacklight.org/jenkins/job/fcrepo-fixity/41/
* nbanks joins19:43
* travis-ci joins19:48
[travis-ci] futures/fcrepo4#254 (master - 4971b28 : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/c0497ca11308...4971b28ce9de
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/5540392
* travis-ci leaves
* nbanks leaves19:54
* jcoyne joins20:51
* jcoyne leaves20:55
<pivotal-bot>Chris Beer started "Self-healing repository" https://www.pivotaltracker.com/story/show/4556566721:06
Chris Beer added comment: "https://github.com/futures/fcrepo4/blob/master/fcrepo-kernel/src/main/java/org/fcrepo/Datastream.java#L172" https://www.pivotaltracker.com/story/show/4556566721:07
Chris Beer added "Log fixity failures to the audit log" https://www.pivotaltracker.com/story/show/46316149
Chris Beer edited "Log fixity failures to the audit log" https://www.pivotaltracker.com/story/show/46316149
Chris Beer added "Call self-healing Datastream#runFixityAndFixProblems when fixity problems are discovered" https://www.pivotaltracker.com/story/show/4631616121:08
Chris Beer edited "Call self-healing Datastream#runFixityAndFixProblems when fixity problems are discovered" https://www.pivotaltracker.com/story/show/46316161
Chris Beer edited "Self-healing repository plumbing" https://www.pivotaltracker.com/story/show/4556566721:09
Chris Beer added comment: "To be invoked by an external service of some nature TBD, see https://www.pivotaltracker.com/story/show/46316161" https://www.pivotaltracker.com/story/show/4556566721:10
Chris Beer edited "Self-healing repository plumbing" https://www.pivotaltracker.com/story/show/45565667
<pivotal-bot>Edwin Shin accepted "REST API support for registering webhook listeners" https://www.pivotaltracker.com/story/show/4546460523:46
Edwin Shin accepted "Firing webhook callbacks" https://www.pivotaltracker.com/story/show/45464623
Edwin Shin delivered "CMIS endpoint NPE on kitchen sink" https://www.pivotaltracker.com/story/show/45666379
Edwin Shin accepted "CMIS endpoint NPE on kitchen sink" https://www.pivotaltracker.com/story/show/4566637923:58
Edwin Shin edited "Report on aggregate object access" https://www.pivotaltracker.com/story/show/4608856900:00
Edwin Shin added "Report on object access" https://www.pivotaltracker.com/story/show/4631825100:04
Edwin Shin edited "Report on object access" https://www.pivotaltracker.com/story/show/46318251
* nbanks joins00:45
* nbanks leaves00:52
* nbanks joins00:58

Generated by Sualtam