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.

Sign up for GitHub

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

Jump to bottom

[pt-br] Update docs/tasks/tools/included/verify-kubectl #48056

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[pt-br] Update docs/tasks/tools/included/verify-kubectl #48056

wants to merge 1 commit into from

Conversation

moreirayokoyama
Copy link
Contributor

Description

This PR solves #48020, updating the included page verify-kubectl.md

Issue

Closes: #48020

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign edsoncelio for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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 area/localization General issues or PRs related to localization label Sep 23, 2024
@k8s-ci-robot k8s-ci-robot added language/pt Issues or PRs related to Portuguese language cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 23, 2024
@netlify Netlify
Copy link

netlify bot commented Sep 23, 2024

✅ Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 783b812
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/66f19281f9f6d7000874d07f
😎 Deploy Preview https://deploy-preview-48056--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.

@stormqueen1990
Copy link
Member

/retitle [pt-br] Update docs/tasks/tools/included/verify-kubectl

@k8s-ci-robot k8s-ci-robot changed the title [pt-br] Update content/pt-br/docs/tasks/tools/included/verify-kubectl.md doc [pt-br] Update docs/tasks/tools/included/verify-kubectl Oct 31, 2024
stormqueen1990
stormqueen1990 reviewed Oct 31, 2024
View reviewed 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.

Olá @moreirayokoyama! Muito obrigada pela tua contribuição!

Encontrei apenas dois ajustes pequenos que seria legal efetuarmos antes de mesclar essa alteração. Assim que fizeres estes ajustes podes me marcar para uma nova revisão.

Obrigada novamente!

content/pt-br/docs/tasks/tools/included/verify-kubectl.md

No Kubernetes 1.26, o kubectl removeu a autenticação incorporada para as serviços de Kubernetes
gerenciados ofertados por provedores de nuvem a seguir. Estes provedores lançaram plugins
para o kubectl para fornecer a autenticação específica da nuvem. Para instruções, dirija-se à documentação do provedor:
Copy link
Member

Choose a reason for hiding this comment

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

O que tu achas de usar "verifique a documentação" ao invés de "dirija-se"? Acho uma tradução mais aproximada de "refer".

Suggested change
para o kubectl para fornecer a autenticação específica da nuvem. Para instruções, dirija-se à documentação do provedor:
para o kubectl para fornecer a autenticação específica da nuvem. Para instruções, verifique a documentação do provedor:

Copy link
Member

Choose a reason for hiding this comment

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

+1

content/pt-br/docs/tasks/tools/included/verify-kubectl.md

Se o kubectl cluster-info retornar a URL de resposta, mas você não conseguir acessar seu cluster, para verificar se ele está configurado corretamente, use:
Se o comando `kubectl cluster-info` retornar a URL de resposta, mas você não conseguir
acessar seu cluster, para verificar se ele está configurado corretamente, use:
Copy link
Member

Choose a reason for hiding this comment

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

Sugestão de ajuste pequeno pois achei que a versão original tem excesso de vírgulas aqui:

Suggested change
acessar seu cluster, para verificar se ele está configurado corretamente, use:
acessar seu cluster, verifique se ele está configurado corretamente usando:

Copy link
Member

Choose a reason for hiding this comment

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

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stormqueen1990 stormqueen1990 stormqueen1990 left review comments

@MrErlison MrErlison Awaiting requested review from MrErlison

Assignees
No one assigned
Labels
area/localization General issues or PRs related to localization cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/pt Issues or PRs related to Portuguese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[pt-br] Page tasks/tools/included/verify-kubectl.md requires content update
4 participants
@moreirayokoyama @k8s-ci-robot @stormqueen1990 @MrErlison