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

[pt-br] Add /docs/tasks/access-application-cluster/configure-access-m… #44755

Merged

Conversation

holgers66
Copy link
Contributor

This PR is to add the Brazilian Portuguese localization of /docs/tasks/access-application-cluster/configure-access-multiple-clusters.md

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jan 15, 2024
@k8s-ci-robot k8s-ci-robot added the language/pt Issues or PRs related to Portuguese language label Jan 15, 2024
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Jan 15, 2024
Copy link

netlify bot commented Jan 15, 2024

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit f6a07a2
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/65c32404e82ec60008a94fb0
😎 Deploy Preview https://deploy-preview-44755--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@stormqueen1990 stormqueen1990 left a 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.
Copy link
Member

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:

Suggested change
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?

Copy link
Contributor Author

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.

Comment on lines 29 to 30
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.
Copy link
Member

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:

Suggested 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.
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.

Copy link
Contributor Author

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.

Copy link
Contributor Author

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.
Copy link
Member

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:

Suggested change
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`.

Copy link
Contributor Author

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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

Copy link
Contributor Author

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)
Copy link
Member

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:

Suggested change
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)

Copy link
Contributor Author

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.

Comment on lines 93 to 95
- 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>`
Copy link
Member

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:

Suggested change
- 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>`

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

Comment on lines 151 to 155
# 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
Copy link
Member

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`.
Copy link
Member

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"?

Suggested change
o contexto atual, use o sinalizador `--minify`.
o contexto atual, use a opção `--minify`.

Copy link
Contributor Author

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/)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[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/)

Copy link
Contributor Author

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/)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* [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/)

Copy link
Contributor Author

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.
Copy link
Member

@stormqueen1990 stormqueen1990 left a 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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tiny detail

Suggested change
# 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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 7, 2024
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: ef2f7aef3245f3da90f988e475668c3b12241d28

@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 7, 2024
@edsoncelio
Copy link
Member

/lgtm
cc @stormqueen1990

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 7, 2024
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 2f26d1592fbc22a96ca0d0276d04ff7d9928e34b

@stormqueen1990
Copy link
Member

/approve

Thanks @holgers66! 🙏

@stormqueen1990
Copy link
Member

/label tide/merge-method-squash

@k8s-ci-robot k8s-ci-robot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Feb 12, 2024
@k8s-ci-robot
Copy link
Contributor

[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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 12, 2024
@k8s-ci-robot k8s-ci-robot merged commit fdc49a6 into kubernetes:main Feb 12, 2024
6 checks passed
@holgers66 holgers66 deleted the hschneid66_ptbr_l10n_20240115_2 branch February 12, 2024 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/pt Issues or PRs related to Portuguese language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants