Commit Graph

275 Commits

Author SHA1 Message Date
Nghia Tran 6a52303145 Update SSL and IP address docs to work with istio-ingressgateway. (#776)
* Update SSL and IP address docs to work with istio-ingressgateway.

* Change quoting per PR feedback.
2019-01-16 14:11:31 -08:00
Nghia Tran 9048eebd45 Instruction to make a Route/KService visible only inside cluster. (#694)
* Instruction to make a Route/KService visible only inside cluster.

* Address PR feedbacks.

* Remove redundant punctuations.
2019-01-16 10:34:31 -08:00
Nghia Tran 01b818c33f Update docs to use correct ingress gateway service. (#700)
* Update docs to use correct ingress gateway service.

* Update wording of knative-ingressgateway deprecation.

* Update wording of knative-ingressgateway deprecation per feeeback.
2019-01-16 08:42:31 -08:00
Mustafa Demirhan 7766388b8f Update installation instructions for logs, metrics and traces (#763)
* Update instructions to install logging and monitoring components.

* Update serving/installing-logging-metrics-traces.md

Co-Authored-By: mdemirhan <4033879+mdemirhan@users.noreply.github.com>
2019-01-15 11:16:30 -08:00
Steren 2078ae3723 Add link to image in Dockerhub (#747)
* 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
2019-01-15 10:18:31 -08:00
Ville Aikas caed7ee6d0 use consistent env variables from install section (#759)
* use consistent env variables from install section

* fix existing lint problems causing test failures
2019-01-14 11:54:31 -08:00
Steren 5de45fb22d Remove license header for Node.js sample (#745)
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.
2019-01-10 15:16:29 -08:00
Steren 67eef0f91b
Remove license header (#744)
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.
2019-01-10 15:06:26 -08:00
Adriano Cunha 49126fcc43 Add real build/unit presubmit tests (#734)
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`
2019-01-10 14:23:42 -08:00
mattmoor-sockpuppet a6e62ae1ad Format markdown (#721)
Produced via: `prettier --write --prose-wrap=always $(find -name '*.md' | grep -v vendor | grep -v .github)`
2019-01-09 09:51:42 -08:00
Steren 1ef00e4f9e Swift sample (#710)
* wift sample

* nodejs -> swift

* newline

* newline

* newline

* Update README.md

Minor wording changes, including:
-Using the present tense (instead of future).
-Using articles and nouns for file names ("kubectl" > "the kubectl command") - better for translation.
2019-01-07 13:37:40 -08:00
Matt Moore fc6ccd7024 Manually run prettier.io (#713)
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.
2019-01-07 09:45:41 -08:00
Adam Ross 72f1777628 serving/samples/helloworld-php: use development php.ini (#699)
* serving/samples/helloworld-php: use development php.ini

* serving/samples/helloworld-php: update README
2019-01-03 15:19:39 -08:00
Joseph Burnett 69f5e6cd94 Add annotations to autoscale-go sample. (#685)
* 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.
2018-12-28 10:10:37 -08:00
mattmoor-sockpuppet 3f9959f289 Format markdown (#675)
Produced via: `prettier --write --prose-wrap=always $(find -name '*.md' | grep -v vendor | grep -v .github)`
2018-12-21 08:49:36 -08:00
RichieEscarez 15709fb082 Update old monitoring namespaces (#674)
* monitoring namespace to knative-monitoring

fixes: #407

* update to serving 0.2.3
2018-12-20 15:33:35 -08:00
Gaurav Abbi a13a629e42 Add refernce to helloworld haskell section (#672) 2018-12-20 09:12:37 -08:00
Adam Ross 2a5147bd9b serving/samples: update helloworld-nodejs to nodejs v10 (#651) 2018-12-19 11:39:35 -08:00
mattmoor-sockpuppet 58b37a4d18 Format markdown (#669)
Produced via: `prettier --write --prose-wrap=always $(find -name '*.md' | grep -v vendor | grep -v .github)`
2018-12-19 10:35:35 -08:00
RichieEscarez 8be95b2089 Add link to source code (#666) 2018-12-18 17:33:35 -08:00
csamak 07bcdda117 Restore code in the helloworld-haskell README (#661)
The code was deleted in 41462c6
2018-12-17 21:40:34 -08:00
Ying Chun Guo df75e7e488 fix an error in README of serving/sample/telemetry-go (#659) 2018-12-17 13:45:34 -08:00
mattmoor-sockpuppet ef7ef10833 Format markdown (#658)
Produced via: `prettier --write --prose-wrap=always $(find -name '*.md' | grep -v vendor | grep -v .github)`
2018-12-12 08:47:16 -08:00
Yifan Ge 7f185c4d08 Update service.yaml in the gitwebhook sample (#648)
Change the image name to match the instructions in README.
2018-12-11 13:44:16 -08:00
Sam Foo b75f11ab4a Remove repeated words (#654)
Signed-off-by: GuessWhoSamFoo <sfoohei@gmail.com>
2018-12-11 12:47:16 -08:00
mattmoor-sockpuppet 4f0ffb8d31 Format markdown (#650)
Produced via: `prettier --write --prose-wrap=always $(find -name '*.md' | grep -v vendor | grep -v .github)`
2018-12-10 12:50:15 -08:00
George Harley fb93f7e89b Add PKS information to configuring outbound access setup (#640)
* This is connected to #639
2018-12-07 14:49:27 -08:00
Ahmet Alp Balkan 1df8f9e434 samples/autoscale-go: simplify instructions (#634)
* 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>
2018-12-07 13:35:27 -08:00
Zhimin Xiang b0e83685e9 Add more content about granting CloudDNS permission to GKE cluster (#607)
* update external dns doc

* rename file

* address comment

* fix typos

* address comment
2018-12-07 10:20:26 -08:00
mattmoor-sockpuppet c64c63a160 Format markdown (#626)
Produced via: `prettier --write --prose-wrap=always $(find -name '*.md' | grep -v vendor | grep -v .github)`
2018-12-04 14:34:25 -08:00
zxxa 1e779dca20 Fix spelling error (#617) 2018-12-04 10:07:25 -08:00
mattmoor-sockpuppet 41462c69e1 Format markdown (#620)
Produced via: `prettier --write --proseWrap=always $(find -name '*.md' | grep -v vendor)`
2018-12-03 13:54:25 -08:00
Guang Ya Liu b83769e72d Use go build to build helloworld-go image. (#571) 2018-12-03 12:26:25 -08:00
Ben Browning f92e52857c Fluentd DaemonSet needs a specific Label on Nodes (#332)
* 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
2018-11-30 16:41:24 -08:00
Jonas Finnemann Jensen b11a0d4de3 Fix example response (#451)
We specify `TARGET='Node.js Sample v1'` in `service.yaml`, so it would be surprising if this isn't printed in the response.
2018-11-29 13:48:23 -08:00
mattmoor-sockpuppet df893ed169 Fix spelling errors (#605)
Produced via: `github.com/client9/misspell`
2018-11-28 10:40:23 -08:00
RichieEscarez 5682c7ca9d Fix code indentation (#602)
* Fix code indentation

Fixes: #592 

/assign @averikitsch

* remove extra tabs

* try again
2018-11-27 14:25:22 -08:00
Guang Ya Liu e57450a66b Use golang:alpine instead of golang for example Dockerfile. (#572)
* Use golang:alpine instead of golang for build-private-repo-go Dockerfile.

* wording tweaks
2018-11-27 13:38:22 -08:00
Ahmet Alp Balkan 33b7fbc9f7 use base64 -w0 instead of some random high number (#384)
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>
2018-11-27 13:34:23 -08:00
Guang Ya Liu 47bef7c63f [sample/build-private-repo-go] Unify the format for build private repo. (#578) 2018-11-27 13:14:22 -08:00
Guang Ya Liu 26fb00f42a Updated helloworld-go by clarifying the `IP_ADDRESS` address. (#570)
* Updated helloworld-go by clarifying the `IP_ADDRESS` address.

* Quick edit of the deploy steps
2018-11-27 11:55:22 -08:00
Averi Kitsch c41f9193e8 update Docker file to respect PORT (#523) 2018-11-27 11:51:23 -08:00
Averi Kitsch 803a5e5a94 update dockerfile to be consistent (#566) 2018-11-27 11:33:22 -08:00
Guang Ya Liu 871d3c007d [samples/knative-routing-go] Updated README for some keywords. (#590) 2018-11-27 10:43:23 -08:00
Victor Agababov 648cbb70db Fix the documentation for the knative go example, where the docs refer to the original golang docs. (#588)
The app name is different and it might be confusing (also incorrect).
2018-11-27 10:28:22 -08:00
Ahmet Alp Balkan cfb2e9dc58 serving/ssl: add 90d warning about letsencrypt (#393)
* 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
2018-11-27 10:23:23 -08:00
Morgan Hallmon d0fcb6e2bc Update gke-assigning-static-ip-address.md (#598)
Removed console output from command
2018-11-27 10:12:23 -08:00
Guang Ya Liu 92a92ede21 [samples/grpc-ping-go] Minor typo fix for this example. (#589) 2018-11-27 09:22:22 -08:00
Yanwei Guo 10e43caf81 Replace Bazel commands with ko (#406)
* replace bazel commands with ko

* remove ko commands

* add a command
2018-11-26 16:41:22 -08:00
Ahmet Alp Balkan fb91667928 source-to-url: remove unnecessary yaml indentation (#583) 2018-11-26 14:59:24 -08:00