[GitHub] [nifi-registry] bbende edited a comment on issue #169: NIFIREG-216 Add S3 BundlePersistenceProvider

[GitBox <gi...@apache.org>]

bbende edited a comment on issue #169: NIFIREG-216 Add S3 BundlePersistenceProvider
URL: https://github.com/apache/nifi-registry/pull/169#issuecomment-485455592
 
 
   I merged in the PR for the URL and docker work so this branch is now updated.
   
   I was playing around with the include-aws profile, and if we want to include it by default we can do something like:
   ```
   <id>include-aws</id>
   <activation>
       <property>
          <name>!skipAws</name>
       </property>
   </activation>
   ```
   The only downside I see is that users still need to go into nifi-registry.properties and add `nifi.registry.extension.dir.aws=./ext/aws/lib`. We could have that in the properties file commented out to make it easier, or we could try to do something during the build to conditionally inject it into the properties file, or just leave it up to reading documentation.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services