Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use ksonnet to generate examples #52

Merged
merged 1 commit into from
Aug 21, 2017
Merged

Conversation

chuckha
Copy link
Contributor

@chuckha chuckha commented Aug 17, 2017

Closes issue #5.

  • Adds jsonnet to generate the quickstart examples
  • Removes the YAML
  • Adds the generated JSON
  • Docs are updated to reflect JSON files vs YAML
  • Adds a Makefile target, generate-examples

Signed-off-by: Chuck Ha chuck@heptio.com

More details:

The only dependency for building examples is docker. We use the official ksonnet image.

Closes issue vmware-tanzu#5.

- Adds jsonnet to generate the quickstart examples
- Removes the YAML
- Adds the generated JSON
- Docs are updated to reflect JSON files vs YAML
- Adds a Makefile target, `generate-examples`

Signed-off-by: Chuck Ha <chuck@heptio.com>
@chuckha chuckha added the kind/enhancement New or improved functionality label Aug 17, 2017
@chuckha chuckha added this to the v0.9.0 milestone Aug 17, 2017
@chuckha
Copy link
Contributor Author

chuckha commented Aug 17, 2017

To add a little more detail:

This does not support an RBAC flag. That is fairly trivial to add, but I need more time to test it or a volunteer to test the example on a non-rbac enabled cluster.

I believe this PR addresses concerns about config map readability.

@timothysc
Copy link
Contributor

@kensimon could you take this one.

Copy link

@kensimon kensimon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@kensimon kensimon merged commit 6b2b102 into vmware-tanzu:master Aug 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New or improved functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants