-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
[pt-br] Add /docs/tasks/access-application-cluster/configure-access-m… #44755
[pt-br] Add /docs/tasks/access-application-cluster/configure-access-m… #44755
Conversation
…ultiple-clusters.md
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Hello there, @holgers66! 👋
Thank you very much for your contribution! 😃
I found a few small bits to be adjusted. If you have any questions or disagree with anything I pointed out, please feel free to reach out through the #kubernetes-docs-pt
channel of Kubernetes Slack.
When you're done addressing the feedback, please feel free to request a new review from me.
|
||
|
||
{{< warning >}} | ||
Use somente arquivos kubeconfig de fontes confiáveis. O uso de um arquivo kubeconfig especialmente criado, pode resultar em execução de código malicioso ou exposição de arquivos. Se você preciso usar um arquivo kubeconfig não confiável, inspecione-o cuidadosamente antes, da mesma forma que faria com um script de shell. |
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.
A couple of very minor typos:
Use somente arquivos kubeconfig de fontes confiáveis. O uso de um arquivo kubeconfig especialmente criado, pode resultar em execução de código malicioso ou exposição de arquivos. Se você preciso usar um arquivo kubeconfig não confiável, inspecione-o cuidadosamente antes, da mesma forma que faria com um script de shell. | |
Use somente arquivos kubeconfig de fontes confiáveis. O uso de um arquivo kubeconfig especialmente criado pode resultar em execução de código malicioso ou exposição de arquivos. Se você preciso usar um arquivo kubeconfig não-confiável, inspecione-o cuidadosamente antes, da mesma forma que faria com um script de shell. |
I appreciate the use of "especialmente criado" here but I don't feel it is a particularly good translation to deliver the intent of creating something manually for the purpose of running something malicious, for instance. What do you think of using "artificialmente criado" instead?
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.
I agree. I think "artificialmente criado" shows the maliciousness better. I'll add that change.
Para verificar se {{< glossary_tooltip text="kubectl" term_id="kubectl" >}} está instalado, | ||
execute `kubectl version --client`. A versão do kubectl deve ser [dentro de uma versão secundária](/releases/version-skew-policy/#kubectl) do servidor de API do seu cluster. |
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.
We currently use "maior"/"menor"/"correção" to define major/minor/patch versions of Kubernetes in the versions page. This is just a suggestion to make it clearer to the reader, but feel free to suggest a different way of stating this if you feel it could be better:
Para verificar se {{< glossary_tooltip text="kubectl" term_id="kubectl" >}} está instalado, | |
execute `kubectl version --client`. A versão do kubectl deve ser [dentro de uma versão secundária](/releases/version-skew-policy/#kubectl) do servidor de API do seu cluster. | |
Para verificar se {{< glossary_tooltip text="kubectl" term_id="kubectl" >}} está instalado, | |
execute `kubectl version --client`. A versão do kubectl deve ter [no máximo uma versão menor de diferença](/releases/version-skew-policy/#kubectl) da versão do servidor de API do seu cluster. |
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.
That does sound much better, and I've noted it for future localizations.
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.
That does sound much better, and I've noted it for future localizations.
|
||
## Defina clusters, usuários e contextos | ||
|
||
Suponha que você tenha dois clusters, um para o trabalho de desenvolvimento e outro para o trabalho de teste. |
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.
How do you feel about establishing the cluster names here?
Something like:
Suponha que você tenha dois clusters, um para o trabalho de desenvolvimento e outro para o trabalho de teste. | |
Suponha que você tenha dois clusters, um para o trabalho de desenvolvimento, chamado `development`, e outro para o trabalho de teste, chamado `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.
That does make it clearer. I'll use that.
name: exp-test | ||
``` | ||
|
||
Um arquivo de configuração descreve clusters, usuários e contextos. Seu arquivo `config-demo |
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.
Um arquivo de configuração descreve clusters, usuários e contextos. Seu arquivo `config-demo | |
Um arquivo de configuração descreve clusters, usuários e contextos. Seu arquivo `config-demo` |
Small typo
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.
Fixed
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.
Fixed
Adicione detalhes do usuário ao seu arquivo de configuração: | ||
|
||
{{< caution >}} | ||
O salvamento de senhas na configuração do cliente do Kubernetes é arriscado. Uma alternativa melhor seria usar um plug-in de credenciais e salvá-las separadamente. Veja: [client-go credential plugins](/docs/reference/access-authn-authz/authentication/#client-go-credential-plugins) |
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.
Small suggestion for this paragraph and a correction for the link:
O salvamento de senhas na configuração do cliente do Kubernetes é arriscado. Uma alternativa melhor seria usar um plug-in de credenciais e salvá-las separadamente. Veja: [client-go credential plugins](/docs/reference/access-authn-authz/authentication/#client-go-credential-plugins) | |
O armazenamento de senhas na configuração do cliente do Kubernetes é arriscado. Uma alternativa melhor seria usar um plug-in de credenciais e salvá-las separadamente. Veja: [plugins de credenciais client-go](/pt-br/docs/reference/access-authn-authz/authentication/#plugins-de-credenciais-client-go) |
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.
Armazenamento does sound much better than salvamento. Done.
- Para excluir um usuário, é possível executar `kubectl --kubeconfig=config-demo config unset users.<name>` | ||
- Para remover um cluster, é possível executar `kubectl --kubeconfig=config-demo config unset clusters.<name>` | ||
- Para remover um contexto, é possível executar `kubectl --kubeconfig=config-demo config unset contexts.<name>` |
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.
It's fine to use active voice here:
- Para excluir um usuário, é possível executar `kubectl --kubeconfig=config-demo config unset users.<name>` | |
- Para remover um cluster, é possível executar `kubectl --kubeconfig=config-demo config unset clusters.<name>` | |
- Para remover um contexto, é possível executar `kubectl --kubeconfig=config-demo config unset contexts.<name>` | |
- Para excluir um usuário, você pode executar `kubectl --kubeconfig=config-demo config unset users.<name>` | |
- Para remover um cluster, você pode executar `kubectl --kubeconfig=config-demo config unset clusters.<name>` | |
- Para remover um contexto, você pode executar `kubectl --kubeconfig=config-demo config unset contexts.<name>` |
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.
Done.
# Documentation note (this comment is NOT part of the command output). | ||
# Storing passwords in Kubernetes client config is risky. | ||
# A better alternative would be to use a credential plugin | ||
# and store the credentials separately. | ||
# See https://kubernetes.io/docs/reference/access-authn-authz/authentication/#client-go-credential-plugins |
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.
What do you think of translating the comment here? It provides important context about what's going on.
as credenciais do usuário listado no contexto `dev-frontend`. | ||
|
||
Para ver apenas as informações de configuração associadas ao | ||
o contexto atual, use o sinalizador `--minify`. |
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.
What do you think of saying "opção" instead of "sinalizador"?
o contexto atual, use o sinalizador `--minify`. | |
o contexto atual, use a opção `--minify`. |
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.
Agreed. Sounds better.
``` | ||
|
||
Para obter mais informações sobre como os arquivos kubeconfig são mesclados, consulte | ||
[Organizando o acesso ao cluster usando arquivos kubeconfig](/docs/concepts/configuration/organize-cluster-access-kubeconfig/) |
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.
[Organizando o acesso ao cluster usando arquivos kubeconfig](/docs/concepts/configuration/organize-cluster-access-kubeconfig/) | |
[Organizando o acesso ao cluster usando arquivos kubeconfig](/pt-br/docs/concepts/configuration/organize-cluster-access-kubeconfig/) |
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.
Fixed
|
||
## {{% heading "whatsnext" %}} | ||
|
||
* [Organizando o acesso ao cluster usando arquivos kubeconfig](/docs/concepts/configuration/organize-cluster-access-kubeconfig/) |
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.
* [Organizando o acesso ao cluster usando arquivos kubeconfig](/docs/concepts/configuration/organize-cluster-access-kubeconfig/) | |
* [Organizando o acesso ao cluster usando arquivos kubeconfig](/pt-br/docs/concepts/configuration/organize-cluster-access-kubeconfig/) |
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.
Fixed
Fixed typos, anchor links and made some of the recommended changes.
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
/cc @edsoncelio @MrErlison
for a second-pass review
# Armazenar senhas na configuração do cliente Kubernetes é arriscado. | ||
# Uma alternativa melhor seria usar um plugin de credenciais | ||
# e armazenar as credenciais separadamente. | ||
# Veja https://kubernetes.io/pt-br/docs/reference/access-authn-authz/authentication/#client-go-credential-plugins |
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.
Tiny detail
# Veja https://kubernetes.io/pt-br/docs/reference/access-authn-authz/authentication/#client-go-credential-plugins | |
# Veja https://kubernetes.io/pt-br/docs/reference/access-authn-authz/authentication/#plugins-de-credenciais-client-go |
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.
Fixed
LGTM label has been added. Git tree hash: ef2f7aef3245f3da90f988e475668c3b12241d28
|
Fixed incorrect link
/lgtm |
LGTM label has been added. Git tree hash: 2f26d1592fbc22a96ca0d0276d04ff7d9928e34b
|
/approve Thanks @holgers66! 🙏 |
/label tide/merge-method-squash |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: stormqueen1990 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 |
This PR is to add the Brazilian Portuguese localization of /docs/tasks/access-application-cluster/configure-access-multiple-clusters.md