-
Notifications
You must be signed in to change notification settings - Fork 1.6k
zh-translation: /blog/2019/data-plane-setup/index.md #1087 #6240
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
Conversation
Hi @gorda. 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 Once the patch is verified, the new status will be reflected by the 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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ok-to-test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need change
|
||
In simple terms, sidecar injection is adding the configuration of additional containers to the pod template. The added containers needed for the Istio service mesh are: | ||
简单来说,Sidecar 注入会将额外容器的配置添加到 Pod 模板中。Istio 服务网格目前所需的容器有: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
en 没删
|
||
{{< text bash >}} | ||
$ kubectl get pods | grep demo-red | ||
demo-red-pod-8b5df99cc-pgnl7 2/2 Running 0 3d | ||
{{< /text >}} | ||
|
||
The count is not 3 because the `istio-init` container is an init type container that exits after doing what it supposed to do, which is setting up the `iptable` rules within the pod. To confirm the init container exit, let’s look at the output of `kubectl describe`: | ||
该计数不是 3,因为 `istio-init` 容器是一个 init 类型的容器,它在完成应做的操作后退出,其用于在 pod 中设置 `iptable` 规则。为了确认 init 容器已退出,让我们看一下 `kubectl describe` 的输出: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
en 没删
{{< /tip >}} | ||
|
||
For automatic sidecar injection, Istio relies on `Mutating Admission Webhook`. Let’s look at the details of the `istio-sidecar-injector` mutating webhook configuration. | ||
对于 sidecar 自动注入,Istio 依赖于 `更改准入 Webhook`。让我们来看看 `istio-sidecar-injector` 更改 webhook 配置的细节。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mutating Admission Webhook 这个是不是不用翻译比较好?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mutating Admission Webhook 这个是不是不用翻译比较好?
@ilylia ,我同意你的看法,我认为都可以。
这一段文字的前面一行,还包含了类似的单词,validating admission Webhook and mutating admission webhook.
对此你有什么建议吗,都翻译或者都不翻译,我觉得统一就行。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我觉得看语境吧,比如这儿是用 `` 包裹的,我觉得可以归类到专有名词,所以不翻译
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我觉得看语境吧,比如这儿是用 `` 包裹的,我觉得可以归类到专有名词,所以不翻译
@ilylia 感谢细心 review,相关问题已修改
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/LGTM
Ref: ServiceMesher#1087