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

update spec-requirements doc page #2190

Merged
merged 1 commit into from
Aug 8, 2018
Merged

Conversation

jmazzitelli
Copy link
Member

Indicate that the "app" label (along with "version") is recommended because it is also used for metric telemetry, not just distributed tracing.

indicate "app" label is also used for metric telemetry, not just distributed tracing.
@istio-testing
Copy link
Contributor

Hi @jmazzitelli. Thanks for your PR.

I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@jmazzitelli
Copy link
Member Author

/assign @rcaballeromx

@geeknoid
Copy link
Contributor

geeknoid commented Aug 8, 2018

/lgtm

@istio-testing
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: geeknoid

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@istio-testing istio-testing merged commit d52bf01 into istio:master Aug 8, 2018
@jmazzitelli jmazzitelli deleted the patch-1 branch August 8, 2018 22:07
geeknoid added a commit that referenced this pull request Aug 10, 2018
* translate /help/glossary/ page (#2163)

* translate /help/glossary/ page

* add period to description

(cherry picked from commit f6628e4)

* Translate to chinese: content/help/faq/_index.md #189 (#2165)

(cherry picked from commit eb680ee)

* Translate to chinese: content/help/faq (#2166)

* Translate to chinese: content/help/faq/_index.md #189

* Translate to chinese: content/help/faq/setup/index.md #224

* Translate to chinese: content/help/faq/setup/consul-app-not-working.md #222

* Translate to chinese: content/help/faq/setup/k8s-sidecar-injection-not-working.md #226

* improve format

* Translate to chinese: content/help/faq/setup/consul-unset-context.md #223

* fix lint error: spell check

* Translate to chinese: content/help/faq/setup/k8s-migrating.md #225

* improve format

* improve format

* improve format

(cherry picked from commit 0e88c05)

* Translate Istio 1.0 canary into Chinese (#2167)

(cherry picked from commit 7821bd9)

* translate /help/glossary/#attribute (#2164)

* translate /help/glossary/#attribute /help/glossary/#workload

* commit one file in a time, for passing ci

(cherry picked from commit 070f298)

* Translate into Chinese: Multiple files Sync/update (#2156)

* Translate into Chinese: blog/2017/0.2-announcement/index.md

* Update index.md

* Update _index.md

* Translate into Chinese: Multiple files

* Update index.md

* Update index.md

(cherry picked from commit 55fcce6)

* Translate Istio 1.0 canary into Chinese (#2170)

(cherry picked from commit 399c685)

* Translate installation options (#2157)

(cherry picked from commit b9fc21a)

* translate /help/glossary/#adapter (#2171)

* translate /help/glossary/#adapter

* add pauses between chinese words (#2171)

(cherry picked from commit 663d240)

* Fix up some aliases. (#2174)

(cherry picked from commit be92dc7)

* fix auth lint issues (#2179)

(cherry picked from commit d8eec80)

* translate /help/glossary/#micro-segmentation (#2183)

(cherry picked from commit 75267fc)

* Revise the health check faq (#2175)

(cherry picked from commit 9b9f7b2)

* add alicloud (#2125)

(cherry picked from commit 1303ef3)

* [ImgBot] optimizes images (#2186)

*Total -- 559.34kb -> 377.43kb (32.52%)

/content/docs/setup/kubernetes/platform-setup/alicloud/csconsole.png -- 375.59kb -> 251.11kb (33.14%)
/content/docs/setup/kubernetes/quick-start-alicloud-ack/app-catalog-istio-1.0.0.png -- 183.75kb -> 126.33kb (31.25%)

(cherry picked from commit c3e9a72)

* A few editorial fixes.

(cherry picked from commit 3e3bffa)

* Translate into Chinese: Multiple files  Sync & update (#2187)

* Translate into Chinese: blog/2017/0.2-announcement/index.md

* Update index.md

* Update _index.md

* Translate into Chinese: Multiple files

* Update index.md

* Update index.md

* Sync

* Sync

* Sync/Update

* Sync/Update

* Sync

* Fix no space problem.

(cherry picked from commit 714a335)

* translate /help/glossary/#mutual-tls (#2178)

(cherry picked from commit a7d4a48)

* translate content/help/ops/security/_index.md (#2188)

(cherry picked from commit 65cdf96)

* Translate help/ops/traffic-management into Chinese (#2184)

(cherry picked from commit a2901dd)

* Revise the health check faq (#2191)

* Revise the health check faq

* Fix format

* Fix format

(cherry picked from commit 481e58c)

* Update index.md (#2190)

indicate "app" label is also used for metric telemetry, not just distributed tracing.

(cherry picked from commit d52bf01)

* translate /help/glossary/#workload (#2172)

* translate /help/glossary/#workload

* try fix chinese archoar (#2172)

(cherry picked from commit 332514a)

* Update Terms (#2189)

(cherry picked from commit 83efe7d)

* fix typo (#2192)

(cherry picked from commit 9961b52)

* add IKS-ICP hybrid multicluster example (#2169)

(cherry picked from commit 65a1638)

* Couple FAQ fixes (#2195)

(cherry picked from commit f8f6491)

* Translate reference istioctl (#2182)

* Translate: reference/istioctl/

* spell checking.

* ci

* re-lint

* accept comments

(cherry picked from commit 3e25501)

* translate /help/glossary/#mixer-instance /help/glossary/#mixer-handler (#2194)

* translate /help/glossary/#mixer-instance /help/glossary/#mixer-handler

* preserve `handler``instance` word

(cherry picked from commit e0b8c6f)

* Trans help glossary mixer and more (#2197)

* translate /help/glossary/#envoy #mixer #service-endpoint #service-mesh #service-name #service-version #service #workload-instance-principal #workload-instance

* fix typo

(cherry picked from commit 93864e8)

* fix typesetting error (#2200)

(cherry picked from commit f9571e6)

* Remove misplaced "the" (#2198)

(cherry picked from commit 6c4b750)
charili pushed a commit to charili/istio.io that referenced this pull request Nov 10, 2019
* translate /help/glossary/ page (istio#2163)

* translate /help/glossary/ page

* add period to description

(cherry picked from commit f6628e4)

* Translate to chinese: content/help/faq/_index.md istio#189 (istio#2165)

(cherry picked from commit eb680ee)

* Translate to chinese: content/help/faq (istio#2166)

* Translate to chinese: content/help/faq/_index.md istio#189

* Translate to chinese: content/help/faq/setup/index.md istio#224

* Translate to chinese: content/help/faq/setup/consul-app-not-working.md istio#222

* Translate to chinese: content/help/faq/setup/k8s-sidecar-injection-not-working.md istio#226

* improve format

* Translate to chinese: content/help/faq/setup/consul-unset-context.md istio#223

* fix lint error: spell check

* Translate to chinese: content/help/faq/setup/k8s-migrating.md istio#225

* improve format

* improve format

* improve format

(cherry picked from commit 0e88c05)

* Translate Istio 1.0 canary into Chinese (istio#2167)

(cherry picked from commit 7821bd9)

* translate /help/glossary/#attribute (istio#2164)

* translate /help/glossary/#attribute /help/glossary/#workload

* commit one file in a time, for passing ci

(cherry picked from commit 070f298)

* Translate into Chinese: Multiple files Sync/update (istio#2156)

* Translate into Chinese: blog/2017/0.2-announcement/index.md

* Update index.md

* Update _index.md

* Translate into Chinese: Multiple files

* Update index.md

* Update index.md

(cherry picked from commit 55fcce6)

* Translate Istio 1.0 canary into Chinese (istio#2170)

(cherry picked from commit 399c685)

* Translate installation options (istio#2157)

(cherry picked from commit b9fc21a)

* translate /help/glossary/#adapter (istio#2171)

* translate /help/glossary/#adapter

* add pauses between chinese words (istio#2171)

(cherry picked from commit 663d240)

* Fix up some aliases. (istio#2174)

(cherry picked from commit be92dc7)

* fix auth lint issues (istio#2179)

(cherry picked from commit d8eec80)

* translate /help/glossary/#micro-segmentation (istio#2183)

(cherry picked from commit 75267fc)

* Revise the health check faq (istio#2175)

(cherry picked from commit 9b9f7b2)

* add alicloud (istio#2125)

(cherry picked from commit 1303ef3)

* [ImgBot] optimizes images (istio#2186)

*Total -- 559.34kb -> 377.43kb (32.52%)

/content/docs/setup/kubernetes/platform-setup/alicloud/csconsole.png -- 375.59kb -> 251.11kb (33.14%)
/content/docs/setup/kubernetes/quick-start-alicloud-ack/app-catalog-istio-1.0.0.png -- 183.75kb -> 126.33kb (31.25%)

(cherry picked from commit c3e9a72)

* A few editorial fixes.

(cherry picked from commit 3e3bffa)

* Translate into Chinese: Multiple files  Sync & update (istio#2187)

* Translate into Chinese: blog/2017/0.2-announcement/index.md

* Update index.md

* Update _index.md

* Translate into Chinese: Multiple files

* Update index.md

* Update index.md

* Sync

* Sync

* Sync/Update

* Sync/Update

* Sync

* Fix no space problem.

(cherry picked from commit 714a335)

* translate /help/glossary/#mutual-tls (istio#2178)

(cherry picked from commit a7d4a48)

* translate content/help/ops/security/_index.md (istio#2188)

(cherry picked from commit 65cdf96)

* Translate help/ops/traffic-management into Chinese (istio#2184)

(cherry picked from commit a2901dd)

* Revise the health check faq (istio#2191)

* Revise the health check faq

* Fix format

* Fix format

(cherry picked from commit 481e58c)

* Update index.md (istio#2190)

indicate "app" label is also used for metric telemetry, not just distributed tracing.

(cherry picked from commit d52bf01)

* translate /help/glossary/#workload (istio#2172)

* translate /help/glossary/#workload

* try fix chinese archoar (istio#2172)

(cherry picked from commit 332514a)

* Update Terms (istio#2189)

(cherry picked from commit 83efe7d)

* fix typo (istio#2192)

(cherry picked from commit 9961b52)

* add IKS-ICP hybrid multicluster example (istio#2169)

(cherry picked from commit 65a1638)

* Couple FAQ fixes (istio#2195)

(cherry picked from commit f8f6491)

* Translate reference istioctl (istio#2182)

* Translate: reference/istioctl/

* spell checking.

* ci

* re-lint

* accept comments

(cherry picked from commit 3e25501)

* translate /help/glossary/#mixer-instance /help/glossary/#mixer-handler (istio#2194)

* translate /help/glossary/#mixer-instance /help/glossary/#mixer-handler

* preserve `handler``instance` word

(cherry picked from commit e0b8c6f)

* Trans help glossary mixer and more (istio#2197)

* translate /help/glossary/#envoy #mixer #service-endpoint #service-mesh #service-name #service-version #service #workload-instance-principal #workload-instance

* fix typo

(cherry picked from commit 93864e8)

* fix typesetting error (istio#2200)

(cherry picked from commit f9571e6)

* Remove misplaced "the" (istio#2198)

(cherry picked from commit 6c4b750)
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.

None yet

5 participants