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

Using timezone: Eastern Standard Time
* dwilcox leaves03:21
* dwilcox joins06:39
* dwilcox leaves06:44
* dwilcox joins08:26
* ksclarke joins09:32
<pivotal-bot_>A. "Ropsamingo" Soroka added comment: "That looks really nice." https://www.pivotaltracker.com/story/show/6970991009:36
A. "Ropsamingo" Soroka added comment: "That looks really nice. +1" https://www.pivotaltracker.com/story/show/69709910
* ermadmix joins09:58
* jcoyne joins10:26
<awoods>ermadmix: standup?11:02
* github-ff joins11:04
[fcrepo4] cbeer created repository-sitemap (+1 new commit): https://github.com/futures/fcrepo4/commit/be8ad2943fbc
fcrepo4/repository-sitemap be8ad29 Chris Beer: move FedoraSitemap to repository.FedoraRepositorySitemap
* github-ff leaves
* github-ff joins
[fcrepo4] cbeer opened pull request #327: move FedoraSitemap to repository.FedoraRepositorySitemap (master...repository-sitemap) http://git.io/ZdK4Vg
* github-ff leaves
<pivotal-bot_>Chris Beer added comment: "What's the use case driving this? Off-hand, it seems to conflict with the namespace-driven approach, but I gu..." https://www.pivotaltracker.com/story/show/7004586211:07
Justin Coyne added comment: "I believe I raised this issue with Esm´┐Ż. It was more of a hypothetical question, so if this is in reference..." https://www.pivotaltracker.com/story/show/7004586211:16
Justin Coyne added comment: "I think the first point is a higher priority, it is confusing to get a 4xx error when you try to use a path..." https://www.pivotaltracker.com/story/show/7004586211:17
<jcoyne>pivotal-bot_ why you no like unicode?11:18
* travis-ci joins11:23
[travis-ci] futures/fcrepo4#1841 (repository-sitemap - be8ad29 : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/commit/be8ad2943fbc
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24024090
* travis-ci leaves
<pivotal-bot_>Andrew Woods added comment: "Yes, this issue is the one you raised @justincoyne4." https://www.pivotaltracker.com/story/show/7004586211:27
Eric James added "node delete fails silently under access control" https://www.pivotaltracker.com/story/show/7037122011:53
Kevin Clarke started "Tomcat Roles PrincipalProvider" https://www.pivotaltracker.com/story/show/6944465612:10
Andrew Woods accepted "Raise HTTP 404 when a POST is performed against a node that doesn't exist" https://www.pivotaltracker.com/story/show/7017455612:15
Kevin Clarke finished "Tomcat Roles PrincipalProvider" https://www.pivotaltracker.com/story/show/6944465612:21
* dwilcox leaves12:38
* edInCo joins12:44
* nikhiltri joins12:52
* ermadmix leaves13:03
* ermadmix joins13:17
* dwilcox joins13:22
<awoods>afk13:42
* ermadmix leaves13:53
* ermadmix joins13:58
<ksclarke>awoods question for you once you're back14:19
<pivotal-bot_>Kevin Clarke added comment: "@stefanoc Do you have a cnd file that you've used to load these? I'm new to the project, and it would help ..." https://www.pivotaltracker.com/story/show/6934690414:34
* gregjansen leaves14:58
<awoods>ksclarke: what's cooking?15:11
<pivotal-bot_>Stefano Cossu added comment: "@ksclarke The relevant definition is in https://github.com/scossu/artic_fedora_cnd/blob/master/classes/cnd..." https://www.pivotaltracker.com/story/show/6934690415:14
Andrew Woods added comment: "@ksclarke, I suspect you can simply create a Fedora object (or two) and add some properties to the object(s..." https://www.pivotaltracker.com/story/show/69346904
<ksclarke>awoods: just needed a little push on that ticket, but think yours and stefano's responses will do it... thanks15:16
<awoods>ksclarke: good luck15:17
<pivotal-bot_>Chris Beer started "Update to Modeshape 4 / ISPN 6" https://www.pivotaltracker.com/story/show/7038790415:19
Chris Beer added "Update to Modeshape 4 / ISPN 6" https://www.pivotaltracker.com/story/show/70387904
<awoods>cbeer: I assume this is a placeholder for when ModeShape 4 is actually released?15:20
<cbeer>awoods: they cut an alpha2 recently15:21
<awoods>cbeer: yes
* gregjansen joins
<awoods>cbeer: I am not sure we want to be exactly *that* bleeding edge.15:22
* scossu joins
<awoods>ksclarke: one more comment to address on the ContainerRoles PR.15:45
* ksclarke goes to look15:46
hmm, it works as is... isn't that ref'ed to authenticationProvider?15:51
I'm new to spring so may be missing something obvious
awoods ^
<awoods>ksclarke: you are defining two spring-beans with the same name15:54
ksclarke: one will win
ksclarke: the goal is to have the principalProviderSet contain both PrincipalProviders.15:55
ksclarke: what do you mean by "it works as is"?
<ksclarke>right, just my lack of understanding of spring then... thought it was doing that
well, I only tested mine of course :-)
so I guess mine overwrote the other?
thought there was magic in: <property name="principalProviders" ref="principalProviderSet"/>15:56
will look deeper, thanks
* dwilcox leaves15:57
* dwilcox joins16:11
* dwilcox leaves16:16
<ksclarke>darn, and it looks like I lose the ability to use util:set :-( https://jira.spring.io/browse/SPR-652316:27
<awoods>ksclarke/ermadmix: Are you able to get a clean build on: fcrepo-jms-indexer-pluggable ?16:31
* ksclarke checks16:32
yes, with `mvn clean install` checking `mvn install` now16:35
yes, with `mvn install` too, awoods16:37
<awoods>thank you, ksclarke.16:38
* edInCo leaves16:40
<awoods>ksclarke: I am getting IndexerGroupIT 404 errors16:41
* martinjd joins16:42
<ksclarke>hmm, I do have fcrepo running at 8080... let me stop it and try again
maybe it's finding that?
* ksclarke tries again
no, it worked fine again...16:46
<awoods>ksclarke: I see the issue, it is due to: https://github.com/futures/fcrepo4/commit/7eb2a63f9dbaf68bd13eea58e231d06f6b4b48ef16:47
which is the most recent commit.16:48
<ksclarke>ah, yeah; I pulled earlier today but I haven't installed the whole project again16:49
looks like he fixed a lot of the same things in those tests as I did16:55
<ermadmix>Just pulled master down 20 minutes ago, and just got 3 failed tests on clean install:16:56
IndexerGroupIT.testIndexerGroupReindex:126->doIndexerGroupUpdateTest:8716:57
IndexerGroupIT.testIndexerGroupDelete:98->doIndexerGroupUpdateTest:87
IndexerGroupIT.testIndexerGroupUpdate:61->doIndexerGroupUpdateTest:87
<awoods>ermadmix: Thanks, I am working on a fix.16:58
<pivotal-bot_>Andrew Woods added "Fix indexer build from propagated F4 changes" https://www.pivotaltracker.com/story/show/7039879817:17
Andrew Woods started "Fix indexer build from propagated F4 changes" https://www.pivotaltracker.com/story/show/70398798
* github-ff joins17:18
[fcrepo-jms-indexer-pluggable] awoods pushed 1 new commit to master: https://github.com/futures/fcrepo-jms-indexer-pluggable/commit/e7b82cbebf7847cd85fdc96f05b7cd8a75eaa1d9
fcrepo-jms-indexer-pluggable/master e7b82cb Andrew Woods: Fix build from propagated changes in fcrepo4...
* github-ff leaves
<pivotal-bot_>Andrew Woods added comment: "Resolved with: https://github.com/futures/fcrepo-jms-indexer-pluggable/commit/e7b82cbebf7847cd85fdc96f05b7c..." https://www.pivotaltracker.com/story/show/7039879817:19
<awoods>ermadmix: pull down the latest from fcrepo-jms-indexer-pluggable, and try the build again.
<pivotal-bot_>Andrew Woods delivered "Fix indexer build from propagated F4 changes" https://www.pivotaltracker.com/story/show/70398798
* github-ff joins17:27
[fcrepo-jms-indexer-pluggable] awoods pushed 1 new commit to master: https://github.com/futures/fcrepo-jms-indexer-pluggable/commit/800759c637317c2c9ae75d5de7e4d3a6fafa574d
fcrepo-jms-indexer-pluggable/master 800759c Kevin S. Clarke: Add sonatype snapshots repo so fcrepo snapshots can be resolved...
* github-ff leaves
<pivotal-bot_>Andrew Woods added comment: "Resolved with: https://github.com/futures/fcrepo-jms-indexer-pluggable/commit/800759c637317c2c9ae75d5de7e4d..." https://www.pivotaltracker.com/story/show/6914805217:28
Andrew Woods delivered "Pick-up fcrepo4 snapshot dependencies in fcrepo-jms-indexer-pluggable" https://www.pivotaltracker.com/story/show/69148052
* github-ff joins
[fcrepo-jms-indexer-pluggable] awoods closed pull request #37: added sonatype snapshots repo so fcrepo snapshots can be resolved (master...add-snapshots-repo-69148052) https://github.com/futures/fcrepo-jms-indexer-pluggable/pull/37
* github-ff leaves
<ermadmix>awoods: same 3 failed tests17:29
* travis-ci joins17:30
[travis-ci] futures/fcrepo-jms-indexer-pluggable#105 (master - e7b82cb : Andrew Woods): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo-jms-indexer-pluggable/compare/35ac685e5327...e7b82cbebf78
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo-jms-indexer-pluggable/builds/24052839
* travis-ci leaves
<awoods>ermadmix: you are working off of the latest fcrepo-jms-indexer-pluggable?
<ermadmix>... and double checking, the latest pull pulled IndexerGroupIT and SolrMappingsIT
<awoods>ermadmix: and your working branch is "master"?17:31
<ermadmix>awoods: yes
<awoods>how about you, ksclarke?17:32
* travis-ci joins17:38
[travis-ci] futures/fcrepo-jms-indexer-pluggable#106 (master - 800759c : Kevin S. Clarke): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo-jms-indexer-pluggable/compare/e7b82cbebf78...800759c63731
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo-jms-indexer-pluggable/builds/24053408
* travis-ci leaves
<pivotal-bot_>Andrew Woods accepted "URN predicates expose _ref variants, Internal Server Error when removed" https://www.pivotaltracker.com/story/show/6996580417:44
Andrew Woods accepted "Preconditions failures should return 412 Precondition Failed and a useful error message" https://www.pivotaltracker.com/story/show/7017647417:46
Andrew Woods rejected "Tomcat Roles PrincipalProvider" https://www.pivotaltracker.com/story/show/6944465617:48
Chris Beer added "Upgrade to Modeshape 3.7.2" https://www.pivotaltracker.com/story/show/7040187218:00
Chris Beer added comment: "Blocked by MODE-2200." https://www.pivotaltracker.com/story/show/70401872
Chris Beer added comment: "(also blocks https://www.pivotaltracker.com/story/show/70387904)" https://www.pivotaltracker.com/story/show/70401872
* jcoyne leaves18:01
<pivotal-bot_>Kevin Clarke started "Tomcat Roles PrincipalProvider" https://www.pivotaltracker.com/story/show/6944465618:03
<ksclarke>awoods, working branch for fcrepo4 or fcrepo-jms-indexer?18:04
<awoods>ksclarke: indexer18:05
<ksclarke>yes, master
should I pull and test again?
<awoods>ksclarke: and your build is successful?
<ksclarke>it was but I hadn't built the latest fcrepo4 changes... just pulled them
I can update both and build everything18:06
if that would help
<awoods>ksclarke: That would be great.
<ksclarke>underway...18:07
* nikhiltri leaves18:18
* ermadmix leaves18:24
<ksclarke>awoods, successfully built18:27
<awoods>ksclarke: nice. thanks.
ksclarke: I am just resolving yours and cbeer's merge conflicts.18:28
<ksclarke>I noticed there were going to be a lot there :-)
<awoods>ksclarke: I thought it better I do it than to make you give it a third pass.18:29
<ksclarke>thank you... my eyes are blistered from pulling out all the checkstyle stuff
:-)
(totally my fault... I approached that from the wrong direction)18:30
* ksclarke leaves18:31
* jcoyne joins19:22
* jcoyne leaves19:23
* scossu leaves19:36
<pivotal-bot_>Andrew Woods accepted "Fix indexer build from propagated F4 changes" https://www.pivotaltracker.com/story/show/7039879819:44
* github-ff joins19:58
[fcrepo4] awoods pushed 1 new commit to master: https://github.com/futures/fcrepo4/commit/ba01f4342e10100c5de0cfc95f9fa14dcdc9a97e
fcrepo4/master ba01f43 Kevin S. Clarke: Fix integration tests which were breaking in non-clean builds...
* github-ff leaves
<pivotal-bot_>Andrew Woods added comment: "Resolved with: https://github.com/futures/fcrepo4/commit/ba01f4342e10100c5de0cfc95f9fa14dcdc9a97e" https://www.pivotaltracker.com/story/show/69740244
Andrew Woods delivered "HTTP API Module fails when not installed with clean" https://www.pivotaltracker.com/story/show/69740244
* github-ff joins19:59
[fcrepo4] awoods closed pull request #321: fixed integration tests which were breaking in non-clean builds (master...fix-tests-for-nonclean-build-69740244) https://github.com/futures/fcrepo4/pull/321
* github-ff leaves
* travis-ci joins20:14
[travis-ci] futures/fcrepo4#1845 (master - ba01f43 : Kevin S. Clarke): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/7eb2a63f9dba...ba01f4342e10
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24063332
* travis-ci leaves
<pivotal-bot_>Andrew Woods added comment: "This PR looks good, except when creating a version of a datastream, two new versions are displayed.20:26
For ex..." https://www.pivotaltracker.com/story/show/69026914
Andrew Woods rejected "A link to the endpoint fcr:versions should not be presented in the UI for unversionable nodes since it will alway..." https://www.pivotaltracker.com/story/show/69026914
* jcoyne joins21:35
* scossu joins23:50
* jcoyne leaves00:02
* scossu leaves00:59
* scossu joins01:01
* scossu leaves01:12

Generated by Sualtam