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

Using timezone: Eastern Standard Time
* mcritchlow leaves00:29
* dwilcox joins07:50
* acoburn joins08:59
* kefo joins09:00
* ajs6f joins09:09
* kefo leaves09:15
* dwilcox leaves09:22
* dwilcox joins09:33
* mcritchlow joins09:46
* peichman joins09:47
* bseeger joins09:53
* peichman leaves09:55
* peichman joins09:57
* dwilcox leaves10:00
* peichman leaves10:01
* peichman joins10:03
* mikeAtUVa joins10:10
* osmandin joins10:11
* mcritchlow leaves10:21
* whikloj joins10:36
* mcritchlow joins
* kefo joins10:39
* dwilcox joins10:40
* apb18 joins10:49
* aelkiss joins10:59
* dshalvi joins11:05
<ruebot>apb18: got a link to that ticket?11:10
* bseeger leaves
I _think_ all the JMeter tickets can be found here: https://jira.duraspace.org/issues/?jql=project%20%3D%20FCREPO%20AND%20labels%20%3D%20performance11:29
Acutally, this one is better: https://jira.duraspace.org/issues/?jql=project%20%3D%20FCREPO%20AND%20component%20%3D%20f4-jmeter11:30
* peichman leaves11:47
* peichman joins11:50
<ruebot>sorry, dropped again.11:53
<ajs6f>acoburn:whikloj: I kind of feel like we should rewrite the fcrepo-http-api integration tests to use: https://hc.apache.org/httpcomponents-client-ga/tutorial/html/fluent.html12:01
<whikloj>ajs6f/acoburn: what is the benefit of that?12:03
<ajs6f>whikloj: Shorter, much more readabe, gets rid of a bunch of protected methods in AbstractIT that no one remembers to use.
whikloj: Not a priority.12:04
<whikloj>ajs6f: gotcha
<ajs6f>whikloj: There are no priorities in Fedora. There are only happy accidents.12:05
<acoburn>ajs6f: that API looks A LOT nicer
* bseeger joins
<ajs6f>acoburn: Yeah, that API definitely hits the gym.12:06
<acoburn>ajs6f: so are you volunteering to rewrite those 3,000 lines of code?
<ajs6f>acoburn: No, I'm volunteering to write the ticket, then rm the test and just go forward. You see, it's trivial that a class of zero length uses all possible APIs, so that would close the ticket.12:07
* aelkiss leaves12:23
* bseeger leaves12:25
* bseeger joins12:32
* bseeger leaves12:34
* github-ff joins12:38
[ontology] mjgiarlo opened pull request #37: Add `inactive` state to Object Status Property list (master...patch-1) https://git.io/vrnUl
* github-ff leaves
* apb18 leaves12:39
* cmills joins12:41
* bseeger joins12:44
* cmmills leaves12:45
* bseeger leaves
* bseeger joins13:10
* bseeger leaves13:22
* github-ff joins13:23
[fcrepo4-vagrant] awoods deleted rc-4.5.0 at 7c67fe6: https://git.io/vrnYG
* github-ff leaves
* github-ff joins13:24
[fcrepo4-vagrant] awoods deleted fcrepo4-rbacl at d946411: https://git.io/vrnY8
* github-ff leaves
* mikeAtUVa leaves13:52
* kefo leaves14:10
* bseeger joins14:29
* arebenji` joins14:42
* mikeAtUVa joins14:45
* arebenji` leaves14:46
* arebenji` joins14:47
* kefo joins14:56
<acoburn>whikloj/awoods/ajs6f: I've updated https://github.com/fcrepo4/fcrepo4/pull/1033 if y'all could take a look15:06
<awoods>acoburn: thanks, I plan on reviewing tickets today and tomorrow.15:08
* dshalvi leaves
<awoods>acoburn/ajs6f: Have you looked into the Jenkins failures: http://jenkins.fcrepo.org/15:09
acoburn/ajs6f: I get the same error locally for fcrepo-transform15:11
<acoburn>awoods: the camel-tests error looks like a port conflict
<awoods>acoburn: fcrepo-transform seems to have started breaking with: https://github.com/fcrepo4/fcrepo4/commit/05e489e07e4807106c3a79e5bd5fdcb6f343109015:12
<acoburn>awoods: that means it's clearly my doing15:13
* kefo leaves15:15
* jcoyne joins15:17
* cgalarza joins15:19
* peichman leaves
* peichman joins15:20
<acoburn>awoods: ok, I've got it. expect a PR in just a sec15:23
<awoods>acoburn: thanks
<acoburn>awoods: I assume it'll be an addon to the ticket above?
<awoods>acoburn: sounds right
* github-ff joins15:26
[fcrepo-transform] acoburn opened pull request #21: Update expectations about FedoraResource::getPath (master...fcrepo-1694) https://git.io/vrnaF
* github-ff leaves
<acoburn>awoods: question about https://jira.duraspace.org/browse/FCREPO-200615:29
awoods: if I give you a fuseki configuration that implements this, would you be able to apply it to the vagrant build?15:30
awoods: vagrant and I are having a stand-off at the moment
<awoods>acoburn: Sure, add your updates to the ticket or a GitHub branch and I can move it into fcrepo4-vagrant.15:31
<acoburn>awoods: ok, I'll see if I can get to that this afternoon
<awoods>acoburn: that is great... anytime this week.15:32
<cgalarza>Hi all, The latest release of fedora commons (4.5.1) doesn't seem to have an md5sum. I ran into the problem using fcrepo_wrapper, which validates the download using the md5sum on github. Could it be added to the release?
<awoods>cgalarza: yes... stay tuned.15:33
<cgalarza>awoods: thanks!15:34
<acoburn>awoods: I assume that'd mean just taking the md5/sha1 from here: http://repo1.maven.org/maven2/org/fcrepo/fcrepo-webapp/4.5.1/
<awoods>acoburn: yes, I am updating the "Release Process" docs as well.15:35
<bseeger>awoods: Hi! When you have some time, could you review and merge: https://github.com/fcrepo4-exts/fcrepo-camel-toolbox/pull/8415:39
* github-ff joins15:42
[fcrepo-camel-toolbox] awoods pushed 2 new commits to master: https://git.io/vrnro
fcrepo-camel-toolbox/master 9f4c30a Aaron Coburn: Support authenticated connections to the JMS broker...
fcrepo-camel-toolbox/master 9082a6a Andrew Woods: Merge pull request #84 from acoburn/fcrepo-2013...
* github-ff leaves
<bseeger>awoods: thanks so much!!15:43
<awoods>bseeger: sorry for the delay
<bseeger>awoods: delay? that was pretty quick. ;) Finally got the point where I could really use this code.15:44
<awoods>acoburn: the checksums of the Maven artifacts are different from the artifacts published to GitHub... because of timestamps, presumably.
<acoburn>awoods: that doesn't seem right15:45
* dwilcox leaves
<awoods>acoburn: why not? The GitHub artifacts are build by the release manager, and the maven artifacts are built by Sonatype.
<acoburn>awoods: oh, in that case, that makes more sense15:46
<awoods>acoburn: alternatively, we can change the release process to pull the maven artifacts into the GitHub release.
acoburn: I kind of like that idea.15:48
<ruebot>awoods: https://wiki.duraspace.org/display/FF/2016-05-16+Performance+-+Scale+Meeting -- review when you get a chance, and then i'll send a note out to the list
<acoburn>awoods: I do, too
<awoods>acoburn: I will change the Release Process to pull artifacts from maven central15:49
<awoods>ruebot: ship it
<ruebot>awoods: yessir15:51
* kefo joins15:58
* travis-ci joins16:01
fcrepo4-exts/fcrepo-camel-toolbox#232 (master - 9082a6a : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-camel-toolbox/compare/3da141b7e38d...9082a6a07498
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-camel-toolbox/builds/130654304
* travis-ci leaves
<awoods>cgalarza: done: https://github.com/fcrepo4/fcrepo4/releases/tag/fcrepo-4.5.116:02
<acoburn>awoods: I'd like to put out a new release of fcrepo-camel-toolbox, but I was thinking of upgrading the parent-pom version first (meaning the license headers will require a slight formatting change)16:03
<cgalarza>awoods: thanks!!
<acoburn>awoods: I could either make that change before the release or after — thoughts?
<awoods>cgalarza: I used the artifacts from maven central and their associated checksum files... which are formatted differently than the checksum files previously published. I hope that does not cause an issue.16:04
acoburn: It depends on the importance of updating the parent-pom... is there additional value in getting that update in before the release?16:06
<acoburn>awoods: not really
<awoods>acoburn: it is your call
<acoburn>awoods: ok, I'll move forward with the release without that update16:07
<awoods>acoburn: I was thinking a documentation page like the following could be helpful: https://wiki.duraspace.org/display/FEDORA35/Messaging
<acoburn>awoods: you mean like this: https://wiki.duraspace.org/display/FEDORA4x/Setup+Camel+Message+Integrations16:08
<awoods>acoburn: Yes, thanks. Although there may be folks who want to know about configuring the messaging who do not care about Camel. That page has a mix of information... but is probably sufficient.16:10
<cgalarza>awoods: It didn't cause any problems, fcrepo_wrapper was ignoring anything after the space in the previous files.16:11
<awoods>cgalarza: glad to hear it.16:12
<acoburn>awoods: we could certainly split that page into two (or three)16:13
* github-ff joins
[fcrepo-transform] awoods pushed 1 new commit to master: https://git.io/vrnXE
fcrepo-transform/master 103cc63 Aaron Coburn: Update expectations about FedoraResource::getPath to not include a trailing "/jcr:content" string (#21)...
* github-ff leaves
* kefo leaves16:14
<awoods>acoburn: I was looking at the email you sent to the list: https://groups.google.com/d/msg/fedora-tech/nY7-8Hjghq0/6GcKjpqUBwAJ
acoburn: I thinking that a simple reference to the wiki in the future would be nice.16:15
acoburn: ..and thinking that a simple reference to the wiki in the future would be nice.
<acoburn>awoods: that makes sense
<awoods>acoburn: the question is, "would your current wiki page serve that need"?16:16
<acoburn>awoods: probably
<awoods>acoburn: then we should leave it.16:17
* travis-ci joins16:19
fcrepo4-exts/fcrepo-transform#74 (master - 103cc63 : Aaron Coburn): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-transform/compare/cbd9885adb36...103cc631e28e
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-transform/builds/130660800
* travis-ci leaves
* apb18 joins16:20
* arebenji` leaves16:30
* kefo joins
* github-ff joins16:42
[fcrepo-camel-toolbox] acoburn tagged fcrepo-camel-toolbox-4.5.2 at 422a6de: https://git.io/vrnHr
fcrepo-camel-toolbox/fcrepo-camel-toolbox-4.5.2 4f1d53f Aaron Coburn: [maven-release-plugin] prepare release fcrepo-camel-toolbox-4.5.2
* github-ff leaves
* cgalarza leaves16:48
* osmandin_afk leaves
* mikeAtUVa leaves16:50
* travis-ci joins17:01
fcrepo4-exts/fcrepo-camel-toolbox#233 (fcrepo-camel-toolbox-4.5.2 - 4f1d53f : Aaron Coburn): The build has errored.
Change view : https://github.com/fcrepo4-exts/fcrepo-camel-toolbox/commit/4f1d53f48018
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-camel-toolbox/builds/130670450
* travis-ci leaves
* bseeger leaves17:02
<apb18>awoods: I'm typing up a comment for FCREPO-2025. The gist is that with Jersey 2.17+, just about every IT fails due to "java.text.ParseException: Error parsing Quality value: a decimal place is expected rather than '0'"17:03
I can make a PR that specifies Jersey 2.8, but also contains a few small updates that will allow it to at least compile if you substitute a more contemporary Jersey version17:05
* github-ff joins17:07
[fcrepo-camel-toolbox] acoburn pushed 1 new commit to gh-pages: https://git.io/vrnF4
fcrepo-camel-toolbox/gh-pages 12ac71d Aaron Coburn: Creating site for fcrepo-camel-toolbox, 4.5.2
* github-ff leaves
<awoods>apb18: currently fcrepo uses Jersey-2.13. Are you suggesting the option that has the least impact is to revert to Jersey-2.8?17:09
apb18: I need to step away for a moment17:10
* github-ff joins17:13
[fcrepo-camel-toolbox] acoburn pushed 1 new commit to gh-pages: https://git.io/vrnbX
fcrepo-camel-toolbox/gh-pages 9141e2b Aaron Coburn: Creating site for fcrepo-audit-triplestore, 4.5.2
* github-ff leaves
<apb18>awoods: I know reverting to 2.8 builds & tests OK, and doesn't have the memory leak. So one possibility is to revert to 2.8, then create another ticket for investigating what it would take to fix the q parsing anomaly with contemporary Jersey versions.17:14
I haven't been able to confirm that the newest Jersey versions truly do fix the leak for us, all request to the repo end in a 500 it seems17:15
* github-ff joins17:19
[fcrepo-camel-toolbox] acoburn pushed 1 new commit to gh-pages: https://git.io/vrnN9
fcrepo-camel-toolbox/gh-pages 664fc42 Aaron Coburn: Creating site for fcrepo-fixity, 4.5.2
* github-ff leaves
* github-ff joins17:25
[fcrepo-camel-toolbox] acoburn pushed 1 new commit to gh-pages: https://git.io/vrnAA
fcrepo-camel-toolbox/gh-pages ba51f4f Aaron Coburn: Creating site for fcrepo-indexing-solr, 4.5.2
* github-ff leaves
* travis-ci joins17:28
fcrepo4-exts/fcrepo-camel-toolbox#233 (fcrepo-camel-toolbox-4.5.2 - 4f1d53f : Aaron Coburn): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-camel-toolbox/commit/4f1d53f48018
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-camel-toolbox/builds/130670450
* travis-ci leaves
* github-ff joins17:30
[fcrepo-camel-toolbox] acoburn pushed 1 new commit to gh-pages: https://git.io/vrnxA
fcrepo-camel-toolbox/gh-pages 158b9ac Aaron Coburn: Creating site for fcrepo-indexing-triplestore, 4.5.2
* github-ff leaves
<awoods>apb18: if there is no loss of functionality, reverting to Jersey-2.8 seems like an easy win.17:37
* github-ff joins
[fcrepo-camel-toolbox] acoburn pushed 1 new commit to gh-pages: https://git.io/vrnpD
fcrepo-camel-toolbox/gh-pages 3936467 Aaron Coburn: Creating site for fcrepo-reindexing, 4.5.2
* github-ff leaves
* github-ff joins17:43
[fcrepo-camel-toolbox] acoburn pushed 1 new commit to gh-pages: https://git.io/vrnh9
fcrepo-camel-toolbox/gh-pages c32b103 Aaron Coburn: Creating site for fcrepo-serialization, 4.5.2
* github-ff leaves
<apb18>awoods: There was no loss of functionality I could see, though my eyes were entirely made of integration tests and microbenchmarks
<awoods>apb18: the integration tests are designed to be comprehensive.17:44
* peichman leaves17:45
* whikloj leaves
* github-ff joins17:46
[fcrepo-camel-toolbox] acoburn pushed 1 new commit to gh-pages: https://git.io/vrnj8
fcrepo-camel-toolbox/gh-pages a3a8168 Aaron Coburn: Creating site for toolbox-features, 4.5.2
* github-ff leaves
* github-ff joins17:47
[fcrepo-camel-toolbox] acoburn pushed 2 new commits to master: https://git.io/vrnj2
fcrepo-camel-toolbox/master 4f1d53f Aaron Coburn: [maven-release-plugin] prepare release fcrepo-camel-toolbox-4.5.2
fcrepo-camel-toolbox/master 0ccef1f Aaron Coburn: [maven-release-plugin] prepare for next development iteration
* github-ff leaves
<apb18>awoods: great. It works trivially if you simply change 2.13->2.8 in the POM. I also have a few local modifications that allow the codebase to additionally compile with 2.17+ (but works OK with 2.8 too). Shall I include those modifications in a PR?17:48
<awoods>apb18: let's keep it simple for now (only 2.13->2.8) and have a follow-on ticket for upgrading Jersey with all of the required modifications.17:49
<apb18>Sounds good. Do you need me to do a PR for that kind of really small change?17:51
* github-ff joins
[fcrepo-camel-toolbox] acoburn opened pull request #87: update docs to include new release API site (gh-pages...update_docs) https://git.io/vrcew
* github-ff leaves
<awoods>apb18: Yes, a PR please... for future tracking and reporting.17:52
<apb18>OK, will do
<acoburn>awoods: do you need a jira ticket for this PR: https://github.com/fcrepo4-exts/fcrepo-camel-toolbox/pull/8717:53
awoods: otherwise, the release is finished; I'll announce it tomorrow
<awoods>acoburn: no need, thanks.17:54
<acoburn>awoods: there are big things in store for the next dev cycle17:55
<awoods>acoburn: exciting
<acoburn>awoods: activemq becomes optional (!)
<awoods>acoburn: for fcrepo-camel-toolbox?
<acoburn>awoods: yes17:56
<awoods>acoburn: nice
<acoburn>awoods: that way, once fcrepo supports other messaging protocols, the toolbox will be ready
awoods: obviously, one will need to install _some_ sort of connector (such as activemq), but it gets injected as an OSGi service
awoods: meaning there's not hard dependency on activemq any longer17:57
<awoods>acoburn: that is great
<acoburn>awoods: it also means that _all_ of the amq configuration goes into a single file
awoods: rather than being spread across all the different toolbox applications17:58
* mcritchlow leaves
<acoburn>awoods: and once this is reviewed/merged, the same thing will apply for the fcrepo component: https://github.com/fcrepo4-exts/fcrepo-camel/pull/110
awoods: meaning that it will be injected as an OSGi service rather than a hard-coded java bean in each and every toolbox app17:59
awoods: anyway, lots to look forward to… have a nice evening18:00
<awoods>acoburn: thanks, you too... and thanks for making Fedora a better place.
* acoburn leaves
* github-ff joins18:01
[fcrepo-camel-toolbox] awoods pushed 2 new commits to gh-pages: https://git.io/vrcfv
fcrepo-camel-toolbox/gh-pages f9e66af Aaron Coburn: update docs to include new release API site
fcrepo-camel-toolbox/gh-pages 3fe4192 Andrew Woods: Merge pull request #87 from acoburn/update_docs...
* github-ff leaves
* github-ff joins18:08
[fcrepo4] birkland opened pull request #1037: FCREPO-2025: Downgrade Jersey from 2.13 to 2.8 (master...FCREPO-2025) https://git.io/vrcJv
* github-ff leaves
* mcritchlow joins18:12
* apb18 leaves18:14
* mcritchlow leaves18:21
* jcoyne leaves18:42
* kefo leaves18:47
* dwilcox joins19:06
* dwilcox leaves20:02
* dwilcox joins20:09
* github-ff joins20:59
[fcrepo-camel-tests] acoburn opened pull request #3: update camel-toolbox version to latest snapshot (master...bump_version) https://git.io/vrcWJ
* github-ff leaves
* jcoyne joins
* dwilcox leaves21:00
* apb18 joins21:07
* dwilcox joins21:09
* github-ff joins21:36
[fcrepo4-vagrant] acoburn opened pull request #46: Omit ldp:contains triples from triplestore sync (master...fcrepo-2006) https://git.io/vrc8A
* github-ff leaves
* dwilcox leaves21:37
* cmills leaves21:57
* cmills joins
* mcritchlow joins22:04
* apb18 leaves
* apb18 joins22:51

Generated by Sualtam