[GitHub] nifi-registry issue #89: NIFIREG-120 Basic Docker Image Support

2018-04-10 Thread Chaffelson
Github user Chaffelson commented on the issue: https://github.com/apache/nifi-registry/pull/89 Great to see this finished, thanks for chasing it through @kevdoran ---

[GitHub] nifi-registry issue #89: NIFIREG-120 Basic Docker Image Support

2018-04-10 Thread kevdoran
Github user kevdoran commented on the issue: https://github.com/apache/nifi-registry/pull/89 FYI to those following this PR. Thanks to @apiri an image built using these scripts has been published to DockerHub under the Apache group: https://hub.docker.com/r/apache/nifi-registry/tags/

[GitHub] nifi-registry issue #89: NIFIREG-120 Basic Docker Image Support

2018-03-18 Thread kevdoran
Github user kevdoran commented on the issue: https://github.com/apache/nifi-registry/pull/89 The branch is working if you don’t need to run it securely. I have some fixes that enable an LDAP secured instance to work. I’ll work with @Chaffelson to get this PR merged in the next wee

[GitHub] nifi-registry issue #89: NIFIREG-120 Basic Docker Image Support

2018-03-18 Thread qcho
Github user qcho commented on the issue: https://github.com/apache/nifi-registry/pull/89 Love to see this PR. I was about to start a similar project myself! How is it going? Can I help in anything? ---

[GitHub] nifi-registry issue #89: NIFIREG-120 Basic Docker Image Support

2018-01-31 Thread Chaffelson
Github user Chaffelson commented on the issue: https://github.com/apache/nifi-registry/pull/89 I think this is now fixed, apologies @kevdoran ---

[GitHub] nifi-registry issue #89: NIFIREG-120 Basic Docker Image Support

2018-01-31 Thread kevdoran
Github user kevdoran commented on the issue: https://github.com/apache/nifi-registry/pull/89 @Chaffelson, In order to review the PR and merge it to master, we'll need to be able to rebase on ASF master. When I tried this, I got failures: ``` First, rewinding head to replay

[GitHub] nifi-registry issue #89: NIFIREG-120 Basic Docker Image Support

2018-01-31 Thread Chaffelson
Github user Chaffelson commented on the issue: https://github.com/apache/nifi-registry/pull/89 Argh I think I've broken this PR by rebaselineing my fork to test something else. Let me know if I need to recreate this. ---

[GitHub] nifi-registry issue #89: NIFIREG-120 Basic Docker Image Support

2018-01-31 Thread abinet
Github user abinet commented on the issue: https://github.com/apache/nifi-registry/pull/89 @Chaffelson Just got secured image working with some small changes: rename login provider file to `identity-providers.xml` https://github.com/Chaffelson/nifi-registry/blob

[GitHub] nifi-registry issue #89: NIFIREG-120 Basic Docker Image Support

2018-01-30 Thread kevdoran
Github user kevdoran commented on the issue: https://github.com/apache/nifi-registry/pull/89 Thanks @abinet and @Chaffelson. Will review. @apiri and @bbende, I looked into the comment above about the jdk and the error log @Chaffelson provided. It looks like a requirement for j

[GitHub] nifi-registry issue #89: NIFIREG-120 Basic Docker Image Support

2018-01-30 Thread Chaffelson
Github user Chaffelson commented on the issue: https://github.com/apache/nifi-registry/pull/89 Hi @abinet that is an excellent point. I did not include it originally because I do not have the test harness to ensure that I did it correctly, but I have spoken with @kevdoran and I be

[GitHub] nifi-registry issue #89: NIFIREG-120 Basic Docker Image Support

2018-01-29 Thread abinet
Github user abinet commented on the issue: https://github.com/apache/nifi-registry/pull/89 Just some remark. This code in ./nifi-registry-docker/dockerhub/sh/start.sh ``` # Establish baseline properties prop_replace 'nifi.registry.web.http.port' '18080' pro