* Add link to image in Dockerhub
Mainly for consistency with other samples.
* also README
* Fixes indentation issue causing build test failure
* fix bullet list indentation
* attempt #2 -- fixing build failure
This is a 15 line trivial hello world, a 15 line license header adds unnecessary complexity.
Many other samples do not include a license header.
The license is already captured at the root of the repository.
This is a 30 line trivial hello world, a 15 line license header adds unnecessary complexity.
Many other samples do not include a license header.
The license is already captured at the root of the repository.
Addresses #66.
Addresses #67.
* Update test-infra to use the default build/unit test runners, which includes building go code and linting markdown files.
* Remove the integration tests, as they only start Knative Serving, which is irrelevant for this repo.
Bonuses:
* fix unbuildable `grpc-ping-go` sample
* remove test-infra import hack from `Gopack.lock` and `update-deps.sh`
We've been having issues with prettier.io on one of the autoscaling markdown files, so I manually fixed that file so that it wouldn't trigger the issue and ran prettier.io over the repo using the normal @mattmoor-sockpuppet command.
* Set target concurrency to 10 for more consistent testing.
* Use hey instead of /load-generator.
* Add panic mode to Algorithm section.
* Add Customization section with annotations.
* Add link to Kubecon demo.
* Update Other Experiments with hey commands.
* Clarify customized examples.
* Update Other Experiments comments to reflect limited test duration.
* samples/autoscale-go: simplify instructions
This patch simplifies the autoscaling tutorial by eliminating steps to:
- go-get the repo
- set a $REPO
- enable GCR API
- show how to authenticate to GCR via docker client
- push image
- update service.yaml
Instead we just do:
- deploy service.yaml that has a pre-built public image
I just published this image to `gcr.io/knative-samples/autoscale-go:0.1` and I
have eliminated excessive Dockerfile instructions that cause the entire docs/
repo to be sent to docker-engine as part of the "docker build".
Fixes#633.
Signed-off-by: Ahmet Alp Balkan <ahmetb@google.com>
* fix: bundle load generator into img, update docs
Signed-off-by: Ahmet Alp Balkan <ahmetb@google.com>
* Fluentd DaemonSet needs a specific Label on Nodes
The Fluentd DaemonSet will only run on nodes labeled with
`beta.kubernetes.io/fluentd-ds-ready="true"`. This label isn't set on
Nodes out of the box on some platforms, including minikube. It appears
to be set by default on GKE.
* Add steps to ensure the Fluentd DaemonSet is up
* Fix markdown after resolving merge conflict
This will yield the same result (single-line output as -w 0 disables line
breaks). However note that, -w flag doesn't exist in BSD base64, but this PR
is not about that.
Signed-off-by: Ahmet Alp Balkan <ahmetb@google.com>
* serving/ssl: add 90d warning about letsencrypt
I don't think certbot method listed here is renewing LE certs. It's especially
important as LE certs expire every 90 days, so I'm adding a warning.
Also using the Let's Encrypt spelling.
Signed-off-by: Ahmet Alp Balkan <ahmetb@google.com>
* move warning up
* Update using-an-ssl-cert.md