logsearch / logsearch-boshrelease

A BOSH-scalable Elasticsearch+Logstash+Kibana release
http://www.logsearch.io
Apache License 2.0
57 stars 46 forks source link

Curator setup as a bosh job #170

Closed cloud-dude closed 9 years ago

dpb587 commented 9 years ago

Please remove the blobs from src/ and squash them out of existence from this PR. I've cloned them locally so I can upload them to the public bucket, but won't merge the PR while they still exist in git history.

dpb587 commented 9 years ago

Thanks for sending this! I'm excited to merge and be able to use this in our own clusters and deprecate our Jenkins job. I've left a few line/regular comments as I reviewed it - let me know if you have any questions!

dpb587 commented 9 years ago

Looks like you might have already squashed the binaries out. I uploaded the blobs and you or I will want to include the following eventually during merge...

diff --git a/config/blobs.yml b/config/blobs.yml
index b3cf3f8..ff79c63 100644
--- a/config/blobs.yml
+++ b/config/blobs.yml
@@ -43,3 +43,23 @@ elasticsearch/elasticsearch-1.7.1.tar.gz:
   object_id: 5ba2c3ed-1730-40fc-9646-aafcab7d9c39
   sha: 0984ae27624e57c12c33d4a559c3ebae25e74508
   size: 28500556
+curator/click-3.3.tar.gz:
+  object_id: 5a17f7ca-b4dc-467a-b5a3-9181dfba17b6
+  sha: d716a932b930d71059e49465b6b42e833808369a
+  size: 259855
+curator/elasticsearch-curator.tar.gz:
+  object_id: dbacd365-5287-439b-8d48-006be450ee5b
+  sha: 28fd2f073b92b33e01db0df08adfc16699969e81
+  size: 82745
+curator/elasticsearch-py.tar.gz:
+  object_id: 44bd1583-2ea7-463d-97f8-a0860da085eb
+  sha: d6d61ddb5c20b0f806978174f75754bce1c53e41
+  size: 77205
+curator/urllib3-1.8.3.tar.gz:
+  object_id: f5601b3b-e555-4ca0-b955-c01e4081d0ee
+  sha: 00c92b0c3b0756ae3c4e6262d5bc80023dfe9ca3
+  size: 85466
+python/Python-3.4.3.tgz:
+  object_id: 34ff2b80-c2ca-4075-81a7-164805059d89
+  sha: 94cae02a334c72ea6ba8c056bb2864e2838878d3
+  size: 19554643
cloud-dude commented 9 years ago

@dpb587 Take a look at the last refactor, if you need any additional changes please let us know. Would you mind doing the merge of the config/blobs.yml?

@bonzofenix && @cloud-dude

dpb587 commented 9 years ago

Sure thing; will take a look soon and @you if anything else comes up.

dpb587 commented 9 years ago

Merged - thanks!