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

Add content/es/docs/reference/glossary/kubelet.md #14556

Merged
merged 1 commit into from
Aug 15, 2019
Merged

Add content/es/docs/reference/glossary/kubelet.md #14556

merged 1 commit into from
Aug 15, 2019

Conversation

javisabalete
Copy link
Contributor

Kubelet in Spanish

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 27, 2019
@k8s-ci-robot k8s-ci-robot requested review from alexbrand and raelga May 27, 2019 16:22
@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. language/es Issues or PRs related to Spanish language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels May 27, 2019
@javisabalete javisabalete mentioned this pull request May 27, 2019
Closed
@netlify
Copy link

netlify bot commented May 27, 2019 •
edited
Loading

Deploy preview for kubernetes-io-master-staging ready!

Built with commit b795580

https://deploy-preview-14556--kubernetes-io-master-staging.netlify.com

@raelga
Copy link
Member

raelga commented May 27, 2019

Muchas gracias!

Tracked at #13948

@raelga raelga added this to the KubeCon EU 2019 milestone May 27, 2019
alexbrand
alexbrand reviewed May 28, 2019
View reviewed changes
content/es/docs/reference/glossary/kubelet.md Outdated

<!--more-->

El kubelet toma un conjunto de PodSpecs proporcionados a través de varios mecanismos y garantiza que los contenedores descritos en esos PodSpecs estén funcionando y en buen estado. El kubelet no administra aquellos que no hayan sido creados por Kubernetes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Que te parece este cambio?
El kubelet no administra aquellos contenedores que no hayan sido creadeos por Kubernetes

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Creo que se sobre entiende que aquellos hace referencia a contenedores, pero si ves que es poco claro, lo añado 😃

@raelga raelga self-assigned this Jun 6, 2019
raelga
raelga reviewed Jun 12, 2019
View reviewed changes
Copy link
Member

@raelga raelga left a comment

Choose a reason for hiding this comment

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

Muchas gracias @javisabalete!

He añadido mis two cents a la conversación.

content/es/docs/reference/glossary/kubelet.md Outdated Show resolved Hide resolved
@raelga
Copy link
Member

raelga commented Jun 22, 2019

Muchas gracias @javisabalete, a ver si logramos un par de lgtms a parte del mío y lo pulblicamos!
/lgtm
/cc @glo-pena @electrocucaracha

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 22, 2019
@alexbrand
Copy link
Contributor

Me suena bastante bien!
/lgtm

@raelga
Copy link
Member

raelga commented Jul 27, 2019

@javisabalete puedes echarle un ojo a los comentarios de @electrocucaracha ?

Muchas gracias!

@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 8, 2019
@javisabalete
Copy link
Contributor Author

Cambios aplicados y squash listo

@glo-pena
Copy link
Contributor

glo-pena commented Aug 8, 2019

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 8, 2019
@raelga
Copy link
Member

raelga commented Aug 9, 2019

Gracias!

A ver si @electrocucaracha puede validar los cambios, por mi parte,

/lgtm

glo-pena
glo-pena approved these changes Aug 10, 2019
View reviewed changes
Copy link
Contributor

@glo-pena glo-pena left a comment

Choose a reason for hiding this comment

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

seems like it still asks for my review.
/lgtm from the review menu

@xiangpengzhao
Copy link
Contributor

Can @alexbrand approve this PR if it's ready to merge?

@electrocucaracha
Copy link
Contributor

/lgtm

@raelga
Copy link
Member

raelga commented Aug 15, 2019

Gracias @electrocucaracha, @glo-pena y @javisabalete !!

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: raelga

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 Aug 15, 2019
@k8s-ci-robot k8s-ci-robot merged commit 559e25d into kubernetes:master Aug 15, 2019
wahyuoi pushed a commit to wahyuoi/website that referenced this pull request Sep 9, 2019
@javisabalete @wahyuoi
Add content/es/docs/reference/glossary/kubelet.md (kubernetes#14556)
de2a793
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@electrocucaracha electrocucaracha electrocucaracha left review comments

@raelga raelga raelga left review comments

@alexbrand alexbrand alexbrand left review comments

@glo-pena glo-pena glo-pena approved these changes

Assignees

@glo-pena glo-pena

@alexbrand alexbrand

@raelga raelga

@electrocucaracha electrocucaracha

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/es Issues or PRs related to Spanish 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/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
Spanish M1
Development

Successfully merging this pull request may close these issues.
7 participants
@javisabalete @raelga @alexbrand @glo-pena @xiangpengzhao @electrocucaracha @k8s-ci-robot