Skip to content

Initialize Secret in Bahasa Indonesia.#14792

Merged
k8s-ci-robot merged 1 commit intokubernetes:masterfrom
irvifa:id/secret
Mar 26, 2020
Merged

Initialize Secret in Bahasa Indonesia.#14792
k8s-ci-robot merged 1 commit intokubernetes:masterfrom
irvifa:id/secret

Conversation

@irvifa
Copy link
Copy Markdown
Member

@irvifa irvifa commented Jun 9, 2019

This address #13929

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/id Issues or PRs related to Indonesian language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Jun 9, 2019
@irvifa
Copy link
Copy Markdown
Member Author

irvifa commented Jun 9, 2019

/assign @girikuncoro

@k8s-ci-robot k8s-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jun 9, 2019
@irvifa irvifa removed the request for review from jimangel June 9, 2019 02:45
@netlify
Copy link
Copy Markdown

netlify bot commented Jun 9, 2019

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

Built with commit d11e062

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

Copy link
Copy Markdown
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

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

A few suggestions. I hope these are useful; if they're not, please feel free to skip them.


Sebagai contoh, untuk menghasilan sebuah Secret dari _file-file_ `./username.txt` dan `./password.txt`
```shell
# Create a kustomization.yaml file with SecretGenerator
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should this comment be localized into Bahasa Indonesia?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@sftim Have difficulties on the glossary_tooltip doesn't rendered properly even though I already created all needed glossaries

Sebagai contoh, untuk membuat sebuah Secret dari literal `username=admin` dan `password=secret`,
kamu dapat menspesifikasikan _generator_ Secret pada _file_ `kustomization.yaml` sebagai
```shell
# Create a kustomization.yaml file with SecretGenerator
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should this comment be localized into Bahasa Indonesia?

Copy link
Copy Markdown
Contributor

@girikuncoro girikuncoro left a comment

Choose a reason for hiding this comment

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

Thanks for your great work, please address my comments! 🎉 🎉 🎉

Objek `secret` pada Kubernetes mengizinkan kamu menyimpan dan mengatur informasi yang sifatnya sensitif, seperti
_passwords_, token OAuth, dan ssh _keys_. Menyimpan informasi yang sifatnya sensitif ini ke dalam `secret`
cenderung lebih aman dan fleksible jika dibandingkan dengan menyimpan informasi tersebut secara apa adanya pada
definisi {{< glossary_tooltip term_id="pod" >}} atau di dalam {{< glossary_tooltip text="container image" term_id="image" >}}.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

pada sebuah Pod:

1. Buatlah sebuah Secret, atau gunakan sebuah Secret yang sudah kamu buat sebelumnya. Beberapa Pod dapat merujuk pada sebuah Secret yang sama.
1. Modifikasi definisi Pod pada setiap kontainer dimana kamu menginginkan kontainer tersebut dapat mengonsumsi your Pod definition in each container that you wish to consume the value of a secret key to add an environment variable for each secret key you wish to consume. The environment variable that consumes the secret key should populate the secret's name and key in `env[].valueFrom.secretKeyRef`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

missing translation

## Menggunakan Secret

Secret dapat di-_mount_ sebagai _volume_ data atau dapat diekspos sebagai
{{< glossary_tooltip text="variabel-variabel environment" term_id="container-env-variabeles" >}}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

typo:

Suggested change
{{< glossary_tooltip text="variabel-variabel environment" term_id="container-env-variabeles" >}}
{{< glossary_tooltip text="variabel-variabel environment" term_id="container-env-variables" >}}

then

add https://github.com/kubernetes/website/blob/master/content/en/docs/reference/glossary/container-env-variables.md to id/docs/references/glossary/container-env-variables.md

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think you also need to add more files to the id/docs/references/glossary

@irvifa
Copy link
Copy Markdown
Member Author

irvifa commented Jun 24, 2019

/retest

@raelga
Copy link
Copy Markdown
Member

raelga commented Jul 13, 2019

Hi @irvifa ,

I got it working locally:

image

By replacing the tabs per spaces at glossary-terms partial #15408:

https://github.com/kubernetes/website/blob/master/layouts/partials/docs/glossary-terms.html

@rgarcia:~/gh/irvifa/website
$ git diff
diff --git a/layouts/partials/docs/glossary-terms.html b/layouts/partials/docs/glossary-terms.html
index fcc5171c2..b0153d49f 100644
--- a/layouts/partials/docs/glossary-terms.html
+++ b/layouts/partials/docs/glossary-terms.html
@@ -1,13 +1,13 @@
 {{ $glossaryBundle := site.GetPage "page" "docs/reference/glossary" }}
 {{- if $glossaryBundle -}}
-       {{ $pages := $glossaryBundle.Resources.ByType "page" }}
-       {{- range site.Params.language_alternatives -}}
-               {{- with (where $glossaryBundle.Translations ".Lang" . ) -}}
-               {{ $p := (index . 0) }}
-               {{ $pages = $pages | lang.Merge ($p.Resources.ByType "page") }}
-               {{ end }}
-       {{ end }}
-       {{- $.Scratch.Set "glossary_items"  $pages -}}
+  {{ $pages := $glossaryBundle.Resources.ByType "page" }}
+  {{- range site.Params.language_alternatives -}}
+    {{- with (where $glossaryBundle.Translations ".Lang" . ) -}}
+    {{ $p := (index . 0) }}
+    {{ $pages = $pages | lang.Merge ($p.Resources.ByType "page") }}
+    {{ end }}
+  {{ end }}
+  {{- $.Scratch.Set "glossary_items"  $pages -}}
 {{- else -}}
 {{- errorf "[%s] Glossary Bundle not found for language. Create at least an index.md file inside docs/reference/glossary" site.Language.Lang -}}
 {{- end -}}

Maybe @bep can share some insights on this issue.

/cc @bep

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

@raelga: GitHub didn't allow me to request PR reviews from the following users: bep.

Note that only kubernetes members and repo collaborators can review this PR, and authors cannot review their own PRs.

Details

In response to this:

Hi @irvifa ,

I got it working locally:

image

By replacing the tabs per spaces at:

https://github.com/kubernetes/website/blob/master/layouts/partials/docs/glossary-terms.html

@rgarcia:~/gh/irvifa/website
$ git diff
diff --git a/layouts/partials/docs/glossary-terms.html b/layouts/partials/docs/glossary-terms.html
index fcc5171c2..b0153d49f 100644
--- a/layouts/partials/docs/glossary-terms.html
+++ b/layouts/partials/docs/glossary-terms.html
@@ -1,13 +1,13 @@
{{ $glossaryBundle := site.GetPage "page" "docs/reference/glossary" }}
{{- if $glossaryBundle -}}
-       {{ $pages := $glossaryBundle.Resources.ByType "page" }}
-       {{- range site.Params.language_alternatives -}}
-               {{- with (where $glossaryBundle.Translations ".Lang" . ) -}}
-               {{ $p := (index . 0) }}
-               {{ $pages = $pages | lang.Merge ($p.Resources.ByType "page") }}
-               {{ end }}
-       {{ end }}
-       {{- $.Scratch.Set "glossary_items"  $pages -}}
+  {{ $pages := $glossaryBundle.Resources.ByType "page" }}
+  {{- range site.Params.language_alternatives -}}
+    {{- with (where $glossaryBundle.Translations ".Lang" . ) -}}
+    {{ $p := (index . 0) }}
+    {{ $pages = $pages | lang.Merge ($p.Resources.ByType "page") }}
+    {{ end }}
+  {{ end }}
+  {{- $.Scratch.Set "glossary_items"  $pages -}}
{{- else -}}
{{- errorf "[%s] Glossary Bundle not found for language. Create at least an index.md file inside docs/reference/glossary" site.Language.Lang -}}
{{- end -}}

Maybe @bep can share some insights on this issue.

/cc @bep

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@irvifa
Copy link
Copy Markdown
Member Author

irvifa commented Jul 14, 2019

@raelga Thanks a lot 👍

k8s-ci-robot pushed a commit that referenced this pull request Jul 14, 2019
Using tabs in this partials seems to cause Hugo to parse the partial
as a code block instead of rendering the HTML code.

Solves #14792 rendering issues.
@irvifa
Copy link
Copy Markdown
Member Author

irvifa commented Jul 14, 2019

Rebased it after #15408 and it's working like a charm..

Objek `secret` pada Kubernetes mengizinkan kamu menyimpan dan mengatur informasi yang sifatnya sensitif, seperti
_password_, token OAuth, dan ssh _keys_. Menyimpan informasi yang sifatnya sensitif ini ke dalam `secret`
cenderung lebih aman dan fleksible jika dibandingkan dengan menyimpan informasi tersebut secara apa adanya pada
definisi {{< glossary_tooltip term_id="pod" >}} atau di dalam {{< glossary_tooltip text="container image" term_id="image" >}}.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Some lines are still breaking for the glossary, not sure why
image

any clue @raelga ?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looking into it! 🔍

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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


Untuk menggunakan Secret, sebuah Pod haruslah merujuk pada Secret tersebut.
Sebuah Secret dapat digunakan di dalam sebuah Pod melalui dua cara:
sebagai _file_ yang ada di dalam _volume_ {{< glossary_tooltip text="volume" term_id="volume" >}}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same with this, glossary lines are breaking

image

@sftim
Copy link
Copy Markdown
Contributor

sftim commented Oct 7, 2019

I don't speak Bahasa Indonesia
/uncc

@fejta-bot
Copy link
Copy Markdown

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jan 5, 2020
@irvifa
Copy link
Copy Markdown
Member Author

irvifa commented Jan 5, 2020

/remove-lifecycle stale

@k8s-ci-robot k8s-ci-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jan 5, 2020
@sftim
Copy link
Copy Markdown
Contributor

sftim commented Jan 14, 2020

@irvifa you're still looking for reviews on this?

@irvifa
Copy link
Copy Markdown
Member Author

irvifa commented Jan 14, 2020 via email

@girikuncoro
Copy link
Copy Markdown
Contributor

let's triage this together and help @irvifa getting this merged
/cc @kubernetes/sig-docs-id-reviews

@wahyuoi
Copy link
Copy Markdown
Contributor

wahyuoi commented Feb 7, 2020

Hi @irvifa @girikuncoro
I just tried to build from id/secret branch, and found few errors. But it solved by merging from upstream/master.

Maybe we need to update something by merging master 🤔

@kbhawkey
Copy link
Copy Markdown
Contributor

Ping @irvifa , Just checking if you are working on this pull request?

@irvifa
Copy link
Copy Markdown
Member Author

irvifa commented Mar 20, 2020

@kbhawkey Yes, I'm working on this

@irvifa
Copy link
Copy Markdown
Member Author

irvifa commented Mar 20, 2020

/cc @wahyuoi

@k8s-ci-robot k8s-ci-robot requested a review from wahyuoi March 20, 2020 12:13
@girikuncoro
Copy link
Copy Markdown
Contributor

@irvifa finally you made it, nicely done! thanks @wahyuoi for helping out

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 24, 2020
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: girikuncoro

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details 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 Mar 24, 2020
@sftim
Copy link
Copy Markdown
Contributor

sftim commented Mar 24, 2020

/hold
pending v1.18 Kubernetes release

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 24, 2020
@VineethReddy02
Copy link
Copy Markdown
Contributor

/hold cancel
v1.18 release is out.

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 26, 2020
@k8s-ci-robot k8s-ci-robot merged commit 884fc46 into kubernetes:master Mar 26, 2020
fancc pushed a commit to fancc/website that referenced this pull request Apr 26, 2020
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/id Issues or PRs related to Indonesian 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.

9 participants