lightbend / fdp-sample-applications

All sample applications for Fast Data Platform
Apache License 2.0
15 stars 6 forks source link

Remove DC/OS information and improve the build structure #19

Closed deanwampler closed 5 years ago

deanwampler commented 5 years ago

This is a big PR, so I don't expect a comprehensive review.

I reworked all the READMEs to remove references to DC/OS and update them, as best I know how, for building and deploying locally (where applicable) and to OpenShift/Kubernetes.

I also eliminated a number of gratuitous differences, like ensuring all the Helm chart directories are named helm or helm-* (when there's more than one).

The build.sh scripts are now more consistent and now do not push the Docker images by default, but instead accept an argument that tells them to do that. I made this change so people can easily use these scripts instead of having to run SBT all the time, without having to deal with image pushing. (I'll modify the Jenkins jobs accordingly to pass the new flag for pushing...)

I also removed the remaining release/fdp-package.sh still inside a few sample apps. There is still the top-level one, but I may remove it in a subsequent PR, as I don't think it's really used anymore, not even by fdp-release.