Add READMEs for deploying from Bigtop repo #2

Merged
merged 1 commit into from May 18, 2016

Conversation

Projects
None yet
3 participants
Owner

johnsca commented May 18, 2016

No description provided.

See `juju quickstart --help` for deployment options, including machine
-constraints and how to deploy a locally modified version of `bundle.yaml`.
+constraints.
+
@arosales

arosales May 18, 2016

We also need Juju 2.0 instructions for juju deploy <bundle-name synatax.

@@ -160,6 +169,10 @@ However, each benchmark is also an action that can be called manually:
Charms can be deployed in environments with limited network access. To deploy
in this environment, you will need a local mirror to serve required packages.
+
+## Building and Deploying from Charm Source
@kwmonroe

kwmonroe May 18, 2016

Member

this section header feels out of place.. this info is covered better in the ./charm/README

@@ -50,8 +50,13 @@ units. To deploy this bundle, simply use:
juju quickstart bigtop-processing-mapreduce
@arosales

arosales May 18, 2016

Is the bundle name "bigtop-processing-mapreduce" or "hadoop-mapreduce?" I would prefer a shorter name, and we may not need to include "bigtop" in the name as that is already in the path. I think there was a comment on the list to exchange "mapreduce" for "yarn." I am not partial to either of these, I think they are both fair names. I do think mapreduce does give a better indicator what the end objective is though.

@johnsca

johnsca May 18, 2016

Owner

I think it's important for the directory structure here to match the bundle name in the store, so that the association is clear. And I think it's important for the store to include at least "bigtop" and "yarn" (or "mapreduce"), so my preference is to make it "bigtop-processing-yarn" even though it's a bit longer.

bigtop-packages/src/charm/README.md
+
+You can then deploy the locally built charms individually:
+
+ juju deploy local:trusty/apache-bigtop-namenode
@kwmonroe

kwmonroe May 18, 2016

Member

just hadoop-namenode now, right?

bigtop-packages/src/charm/README.md
+
+You can also use the local version of a bundle:
+
+ juju deploy bigtop-deploy/juju/hadoop-mapreduce/bundle-local.yaml
@kwmonroe

kwmonroe May 18, 2016

Member

s/mapreduce/processing

Member

kwmonroe commented May 18, 2016

Minor edits to update charm/bundle names, but mostly LGTM

@johnsca johnsca merged commit ecaf053 into master May 18, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment