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

Tweak the pitch deck a little #21

Merged
merged 1 commit into from
Oct 29, 2015
Merged

Tweak the pitch deck a little #21

merged 1 commit into from
Oct 29, 2015

Conversation

adamhjk
Copy link
Contributor

@adamhjk adamhjk commented Oct 29, 2015

  • Tweaks the pitch deck to be less specific on the order of partnership
    conversations.
  • Tweaks docker-etcd to automatically add the etcd link

* Tweaks the pitch deck to be less specific on the order of partnership
  conversations.
* Tweaks docker-etcd to automatically add the etcd link
@chef-delivery
Copy link
Contributor

This PR has passed 'Verify' and is ready for review and approval!
Use: '@delivery approve' when code review is complete.

@@ -1,2 +1,2 @@
#!/bin/bash -x
docker run --link=bldr_etcd_1:etcd $@
docker run -e BLDR_CONFIG_ETCD=http://etcd:4001 --link=bldr_etcd_1:etcd $@
Copy link
Collaborator

Choose a reason for hiding this comment

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

Yep, that seems more portable and better

@fnichol
Copy link
Collaborator

fnichol commented Oct 29, 2015

excellent!

@delivery approve

@chef-delivery
Copy link
Contributor

Change: 83bd68e8-7885-4362-af3b-4014537e50aa approved by: @fnichol

chef-delivery added a commit that referenced this pull request Oct 29, 2015
Merged change 83bd68e8-7885-4362-af3b-4014537e50aa

From review branch demo-update into master

Signed-off-by: fnichol <[email protected]>
@chef-delivery chef-delivery merged commit 42d4a41 into master Oct 29, 2015
@chef-delivery chef-delivery deleted the demo-update branch October 29, 2015 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants