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

Add Kubernetes examples #117

Merged
merged 5 commits into from
Aug 17, 2018
Merged

Add Kubernetes examples #117

merged 5 commits into from
Aug 17, 2018

Conversation

hausdorff
Copy link
Contributor

@hausdorff hausdorff commented Aug 16, 2018

Fixes #115. Fixes #111.

@hausdorff hausdorff force-pushed the hausdorff/examples branch 2 times, most recently from cab3b28 to 43f8031 Compare August 16, 2018 22:40
@hausdorff hausdorff changed the title Add example showing rollout triggered on ConfigMap data Add Kubernetes examples Aug 16, 2018
@hausdorff hausdorff added this to the 0.16 milestone Aug 16, 2018
@hausdorff hausdorff force-pushed the hausdorff/examples branch 2 times, most recently from e2aec9b to 635ac5e Compare August 17, 2018 00:39
Copy link
Member

@lukehoban lukehoban left a comment

Choose a reason for hiding this comment

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

Love it!

Enter a stack name: configmap-rollout-dev
```

This example will attempt to expose the `nginx` deployment Internet with a `Service` of type
Copy link
Member

Choose a reason for hiding this comment

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

"Internet"?

Preview the deployment of the application:

```sh
$ pulumi preview
Copy link
Member

Choose a reason for hiding this comment

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

We can skip the preview - the update will do it anyway.

and `grep` to retrieve the `<title>` of the site the proxy points at.

```sh
$ curl -sL $(pulumi stack output frontendIp):80 | grep -C 1 "<title>"
Copy link
Member

Choose a reason for hiding this comment

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

You don't need :80 do you?

> 1. Update the `PodTemplate` of the `Deployment` to point at the new `ConfigMap`. This update
> triggers the `Deployment` controller to try to roll out a new set of containers with mounts
> that contain this new data.
> 1. Only once that succeeds, delete teh old `ConfigMap`.
Copy link
Member

Choose a reason for hiding this comment

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

teh > the

```sh
$ curl -sL $(pulumi stack output frontendIp):80 | grep -o "<title>Google</title>"
<title>Google</title>
```
Copy link
Member

Choose a reason for hiding this comment

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

Add cleanup steps at the end?


Preview the deployment of the application.

> **TIP:** This example installs the Wordpress Chart. You can use `helm search` to find other Helm
Copy link
Member

Choose a reason for hiding this comment

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

This TIP doesn't apply to this example, does it?

Notice also that if you provide an image that does not exist, Pulumi will report errors as it sees
them. You should see something similar in principle to this:

![Diff](images/error.gif "Error reporting")
Copy link
Member

Choose a reason for hiding this comment

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

Cleanup steps?

// If your cluster config does not include a dns service, then to instead access an environment
// variable to find the master service's host, comment out the 'value: dns' line above, and
// uncomment the line below:
env: [{ name: "GET_HOSTS_FROM", value: "dns" /*value: "env"*/ }],
Copy link
Member

Choose a reason for hiding this comment

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

The comment is a little disconnected from the program text now.

@hausdorff hausdorff merged commit 8d9b0ac into master Aug 17, 2018
@pulumi-bot pulumi-bot deleted the hausdorff/examples branch August 17, 2018 18:37
ramene pushed a commit to ramene/pulumi-kubeflow-ml that referenced this pull request Sep 7, 2019
ramene pushed a commit to ramene/pulumi-kubeflow-ml that referenced this pull request Sep 13, 2019
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.

2 participants