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 /content/pt-br/docs/concepts/configuration/secrets.md #30583

Merged
merged 4 commits into from
Jan 7, 2022

Conversation

stormqueen1990
Copy link
Member

Create the translation for the Secrets page in Brazilian Portuguese.

@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 21, 2021
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Nov 21, 2021

CLA Signed

The committers are authorized under a signed CLA.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Nov 21, 2021
@k8s-ci-robot k8s-ci-robot added language/pt Issues or PRs related to Portuguese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Nov 21, 2021
@netlify
Copy link

netlify bot commented Nov 21, 2021

✔️ Deploy Preview for kubernetes-io-main-staging ready!

🔨 Explore the source changes: 1d115b1

🔍 Inspect the deploy log: https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/619a77dbda29b10008468c37

😎 Browse the preview: https://deploy-preview-30583--kubernetes-io-main-staging.netlify.app

@netlify
Copy link

netlify bot commented Nov 21, 2021

✔️ Deploy Preview for kubernetes-io-main-staging ready!

🔨 Explore the source changes: 6708e86

🔍 Inspect the deploy log: https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/61d78a875b6009000750acec

😎 Browse the preview: https://deploy-preview-30583--kubernetes-io-main-staging.netlify.app

@edsoncelio
Copy link
Member

/assign edsoncelio

@devlware
Copy link
Contributor

Looks good @stormqueen1990.

@WillsonHG
Copy link

/easycla

@yagonobre
Copy link
Member

Thanks! @stormqueen1990
I'll take a look in a bit.
/assign

@edsoncelio edsoncelio removed their assignment Nov 24, 2021
@RA489
Copy link

RA489 commented Nov 25, 2021

/easycla

@stormqueen1990 stormqueen1990 marked this pull request as ready for review November 30, 2021 02:00
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 30, 2021
Copy link
Member

@edsoncelio edsoncelio left a comment

Choose a reason for hiding this comment

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

Hello @stormqueen1990, I left some comments (small changes), thanks for your PR!

weight: 30
---

<!--
Copy link
Member

Choose a reason for hiding this comment

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

I think we can remove these comments with the original content :)

Copy link
Member Author

Choose a reason for hiding this comment

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

Will do!

Copy link
Member Author

Choose a reason for hiding this comment

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

Removed!

consegue utilizar este privilégio para ler qualquer Secret naquele namespace. Isso
inclui acesso indireto, como por exemplo a permissão para criar Deployments.

Para utilizar Secrets de forma segura, siga pelo menos as instruções abaixo:
Copy link
Member

Choose a reason for hiding this comment

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

This enumerated list is repeated, needs to be 1, 2, 3...

Copy link
Member Author

Choose a reason for hiding this comment

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

About this, the enumerated lists are rendered correctly with the use of 1. repeated (due to markdown). I tend to use this style to avoid re-numbering all items when a new item is added. Do you think it would be better to have the numbers in code reflect the rendered numbers?

Copy link
Member

Choose a reason for hiding this comment

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

About this, the enumerated lists are rendered correctly with the use of 1. repeated (due to markdown). I tend to use this style to avoid re-numbering all items when a new item is added. Do you think it would be better to have the numbers in code reflect the rendered numbers?

Ohh okay, I've learned something new haha, If everything it's rendering right I don't see any problem with this way, thanks for the explanation!

This is an example of a Pod that mounts a Secret in a volume:
-->

Para consumir um Secret em um volume em um Pod:
Copy link
Member

Choose a reason for hiding this comment

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

This enumerated list is repeated, needs to be 1, 2, 3...

Para utilizar um secret em uma {{< glossary_tooltip text="variável de ambiente" term_id="container-env-variables" >}}
em um Pod:

1. Crie um Secret ou utilize um já existente. Múltiplos Pods podem referenciar o
Copy link
Member

Choose a reason for hiding this comment

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

This enumerated list is repeated, needs to be 1, 2, 3...

Mauren Berti added 3 commits January 6, 2022 18:46
@edsoncelio
Copy link
Member

/lgtm

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

LGTM label has been added.

Git tree hash: 7345e14c1bc9308e3e904b773342f7e6126e1b92

@yagonobre
Copy link
Member

/approve
@stormqueen1990 Sorry for the long delay
@edsoncelio thanks for the review

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: yagonobre

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 Jan 7, 2022
@k8s-ci-robot k8s-ci-robot merged commit ce5896d into kubernetes:main Jan 7, 2022
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/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants