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

[ar] Translate tutorials/_index and tutorials/hello-minikube.md pages to arabic #45047

Merged

Conversation

seifrajhi
Copy link
Member

This PR adds the translation related to the tutorials directory /docs/tutorials. It is a placeholder for the remaining commits.

Part of #44682

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Feb 6, 2024
@k8s-ci-robot k8s-ci-robot added 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. labels Feb 6, 2024
@seifrajhi seifrajhi marked this pull request as draft February 6, 2024 11:40
@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 Feb 6, 2024
@dipesh-rawat
Copy link
Member

/language ar

@k8s-ci-robot k8s-ci-robot added the language/ar Issues or PRs related to Arabic language label Feb 6, 2024
Copy link
Member

@mboukhalfa mboukhalfa left a comment

Choose a reason for hiding this comment

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

Great work here are some nits

content/ar/docs/tutorials/_index.md Outdated Show resolved Hide resolved
content/ar/docs/tutorials/_index.md Outdated Show resolved Hide resolved
content/ar/docs/tutorials/_index.md Outdated Show resolved Hide resolved
content/ar/docs/tutorials/_index.md Outdated Show resolved Hide resolved
content/ar/docs/tutorials/_index.md Outdated Show resolved Hide resolved
content/ar/docs/tutorials/hello-minikube.md Outdated Show resolved Hide resolved
content/ar/docs/tutorials/hello-minikube.md Outdated Show resolved Hide resolved
content/ar/docs/tutorials/hello-minikube.md Outdated Show resolved Hide resolved
content/ar/docs/tutorials/hello-minikube.md Outdated Show resolved Hide resolved
content/ar/docs/tutorials/kubernetes-basics/_index.html Outdated Show resolved Hide resolved
@mboukhalfa
Copy link
Member

you can also move the PR from draft to ready for review

@mboukhalfa
Copy link
Member

/retitle [ar] Translate tutorials/_index and tutorials/hello-minikube.md pages to arabic

@k8s-ci-robot k8s-ci-robot changed the title Translate tutorials pages to arabic [ar] Translate tutorials/_index and tutorials/hello-minikube.md pages to arabic Feb 6, 2024
@seifrajhi seifrajhi force-pushed the dev-1.29-ar.1-tutorials branch 2 times, most recently from 607e317 to 36bd4d9 Compare February 6, 2024 20:28
@seifrajhi seifrajhi marked this pull request as ready for review February 6, 2024 20:29
@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 Feb 6, 2024
@seifrajhi
Copy link
Member Author

Hey @mboukhalfa , changes are made + PR is in review state, let me know if something else needed

@mboukhalfa
Copy link
Member

LGTM for Arabic

Great work @seifrajhi

@seifrajhi
Copy link
Member Author

@mboukhalfa , Thanks 😄
Can you please resolve the conversation then ?

@mboukhalfa
Copy link
Member

@mboukhalfa , Thanks 😄 Can you please resolve the conversation then ?

I do not have the option to resolve the comments can you do it from your side

@a-mccarthy
Copy link
Contributor

adding the LGTM label based on localization reviews

/lgtm

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

@seokho-son seokho-son left a comment

Choose a reason for hiding this comment

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

Hello @seifrajhi
Please check the following comments. :)

Comment on lines 3 to 5
main_menu: true
weight: 60
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
main_menu: true
weight: 60
main_menu: true
no_list: true
weight: 60

Please ensure that most parameters in the header match those in the English document to maintain synced. 😊

For reference, you can view the English document's header at:
https://raw.githubusercontent.com/kubernetes/website/dev-1.29-ar.1/content/en/docs/tutorials/_index.md


يُظهر البرنامج التعليمي كيفية تحقيق هدف يتجاوز مجرد [مهمة](/docs/tasks/). عادة يحتوي البرنامج التعليمي على أقسام مختلفة، ويحتوي كل قسم على خطوات مختلفة.

قبل استكشاف كل درس، قد يكون من الفائدة الاحتفاظ بعلامة مرجعية للـ [المعجم الموحد](/docs/reference/glossary/) لتسهيل الرجوع إليه لاحقًا.
Copy link
Member

Choose a reason for hiding this comment

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

I hope to check [ ] is correctly used. (to me it looks like ] [)

Copy link
Member Author

Choose a reason for hiding this comment

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

We use it like this and it should be ok

minikube dashboard
```

الآن، قم بالتبديل إلى سطر الأوامر حيث قمت بتشغيل `minikube start`.
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 there is remaining content in the English doc.

PTAL: after ## Open the Dashboard in https://raw.githubusercontent.com/kubernetes/website/dev-1.29-ar.1/content/en/docs/tutorials/hello-minikube.md

{{< note >}}
The `dashboard` command enables the dashboard add-on and opens the proxy in the default web browser.
You can create Kubernetes resources on the dashboard such as Deployment and Service.

To find out how to avoid directly invoking the browser from the terminal and get a URL for the web dashboard, see the "URL copy and paste" tab.

By default, the dashboard is only accessible from within the internal Kubernetes virtual network.
The `dashboard` command creates a temporary proxy to make the dashboard accessible from outside the Kubernetes virtual network.

To stop the proxy, run `Ctrl+C` to exit the process.
After the command exits, the dashboard remains running in the Kubernetes cluster.
You can run the `dashboard` command again to create another proxy to access the dashboard.
{{< /note >}}


....

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

Hello @a-mccarthy , can you please check this PR, and if all ok, can you please add LGTM label ?

Then we need @seokho-son to review and approve it.

@mboukhalfa
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 Feb 23, 2024
@mboukhalfa
Copy link
Member

/cancel lgtm
/assign @adowair

@mboukhalfa
Copy link
Member

/lgtm cancel

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

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mboukhalfa

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 Apr 13, 2024
Copy link

@adowair adowair left a comment

Choose a reason for hiding this comment

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

Thank you @seifrajhi for the big effort. As always, I've left a bunch of little grammar/language suggestions and you have the final judgement 😄. There are also a few newline addition/removal suggestions.

Mainly I want to highlight that these docs introduce some K8s terms and other terms for the first time in the work so far. Namely:

  • Deployment
  • Redis
  • Cassandra
  • Zookeeper

I notice that each of these is expressed differently in the text:

  • Some are expressed in English
  • Some are transliterated to Arabic only

It'd be good to unify this so that reading the page is predictable. Might I suggest?:

  • The first time a term is used, transliterate it to Arabic and include the English term in parentheses
  • From thereon, use just the Arabic
    OR just use the English only as these terms are quite recognizable in English 😄

Thanks again!!

<!-- overview -->

يحتوي هذا القسم من وثائق كوبيرنيتيس على برامج تعليمية.

Copy link

Choose a reason for hiding this comment

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

In the English version it seems there are no newlines in this paragraph.

Suggested change


يحتوي هذا القسم من وثائق كوبيرنيتيس على برامج تعليمية.

يُظهر البرنامج التعليمي كيفية تحقيق هدف يتجاوز مجرد [مهمة](/docs/tasks/). عادة يحتوي البرنامج التعليمي على أقسام مختلفة، ويحتوي كل قسم على خطوات مختلفة.
Copy link

Choose a reason for hiding this comment

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

Splitting over two lines and some minor grammar suggestions

Suggested change
يُظهر البرنامج التعليمي كيفية تحقيق هدف يتجاوز مجرد [مهمة](/docs/tasks/). عادة يحتوي البرنامج التعليمي على أقسام مختلفة، ويحتوي كل قسم على خطوات مختلفة.
يعرض الدرس التعليمي كيفية تحقيق هدف أكبر من [مهمة](/docs/tasks/) واحدة.
عادةً ما يحتوي الدرس التعليمي على عدة أقسام، يحتوي كل منها على تسلسل من الخطوات.


يُظهر البرنامج التعليمي كيفية تحقيق هدف يتجاوز مجرد [مهمة](/docs/tasks/). عادة يحتوي البرنامج التعليمي على أقسام مختلفة، ويحتوي كل قسم على خطوات مختلفة.

قبل استكشاف كل درس، قد يكون من الفائدة الاحتفاظ بعلامة مرجعية للـ [المعجم الموحد](/docs/reference/glossary/) لتسهيل الرجوع إليه لاحقًا.
Copy link

Choose a reason for hiding this comment

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

Splitting line along comma and some language suggestions

Suggested change
قبل استكشاف كل درس، قد يكون من الفائدة الاحتفاظ بعلامة مرجعية للـ [المعجم الموحد](/docs/reference/glossary/) لتسهيل الرجوع إليه لاحقًا.
قبل استكشاف الدروس،
ينصح حفظ صفحة [معجم المعاني الموحد](/docs/reference/glossary/) لتسهيل مراجعتها لاحقًا.


* [أساسيات كوبيرنيتيس](/docs/tutorials/kubernetes-basics/) هو برنامج تعليمي تفاعلي مفصل سيساعدك على فهم نظام كوبيرنيتيس والبدء باستخدام ميزات كوبيرنيتيس الأساسية.

* [تمهيد إلى كوبيرنيتيس (edX) (EN)](https://www.edx.org/course/introduction-kubernetes-linuxfoundationx-lfs158x#)
Copy link

Choose a reason for hiding this comment

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

Suggested change
* [تمهيد إلى كوبيرنيتيس (edX) (EN)](https://www.edx.org/course/introduction-kubernetes-linuxfoundationx-lfs158x#)
* [مقدمة إلى كوبيرنيتيس (edX) (EN)](https://www.edx.org/course/introduction-kubernetes-linuxfoundationx-lfs158x#)


## الإعدادات

* [مثال: كوبيرنيتيسإعداد خدمة مصغرة جافا](/docs/tutorials/configuration/configure-java-microservice/)
Copy link

Choose a reason for hiding this comment

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

Suggested change
* [مثال: كوبيرنيتيسإعداد خدمة مصغرة جافا](/docs/tutorials/configuration/configure-java-microservice/)
* [مثال: إعداد خدمة برمجية مصغرة للجافا](/docs/tutorials/configuration/configure-java-microservice/)

The 'metrics-server' addon is enabled
```

1. عرض الحجيرة والخدمة التي أنشأتها من خلال تثبيت تلك الإضافة:
Copy link

Choose a reason for hiding this comment

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

Suggested change
1. عرض الحجيرة والخدمة التي أنشأتها من خلال تثبيت تلك الإضافة:
1. عرض الحجيرة والخدمة التي أُنشِئت نتيجة تفعيل تلك الإضافة:

minikube delete
```

إذا كنت ترغب في استخدام مينيكيوب مرة أخرى للتعرف على المزيد حول كوبيرنيتيس، فلا حاجة لحذفه.
Copy link

Choose a reason for hiding this comment

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

Suggested change
إذا كنت ترغب في استخدام مينيكيوب مرة أخرى للتعرف على المزيد حول كوبيرنيتيس، فلا حاجة لحذفه.
إذا كنت ترغب في استخدام مينيكيوب مرة أخرى لتعلم المزيد عن كوبيرنيتيس، فلا حاجة لحذفه.

## {{% heading "whatsnext" %}}

* البرنامج التعليمي لـ _[نشر التطبيق الأول على كوبيرنيتيس باستخدام kubectl](/docs/tutorials/kubernetes-basics/deploy-app/deploy-intro/)_.
* تعلم المزيد حول [النشر](/docs/concepts/workloads/controllers/deployment/).
Copy link

Choose a reason for hiding this comment

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

Suggested change
* تعلم المزيد حول [النشر](/docs/concepts/workloads/controllers/deployment/).
* تعلم المزيد عن [النشر](/docs/concepts/workloads/controllers/deployment/).

* البرنامج التعليمي لـ _[نشر التطبيق الأول على كوبيرنيتيس باستخدام kubectl](/docs/tutorials/kubernetes-basics/deploy-app/deploy-intro/)_.
* تعلم المزيد حول [النشر](/docs/concepts/workloads/controllers/deployment/).
* تعلم المزيد حول [نشر التطبيقات](/docs/tasks/run-application/run-stateless-application-deployment/).
* تعلم المزيد حول [الخدمة](/docs/concepts/services-networking/service/).
Copy link

Choose a reason for hiding this comment

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

Suggested change
* تعلم المزيد حول [الخدمة](/docs/concepts/services-networking/service/).
* تعلم المزيد عن [الخدمة](/docs/concepts/services-networking/service/).


* البرنامج التعليمي لـ _[نشر التطبيق الأول على كوبيرنيتيس باستخدام kubectl](/docs/tutorials/kubernetes-basics/deploy-app/deploy-intro/)_.
* تعلم المزيد حول [النشر](/docs/concepts/workloads/controllers/deployment/).
* تعلم المزيد حول [نشر التطبيقات](/docs/tasks/run-application/run-stateless-application-deployment/).
Copy link

Choose a reason for hiding this comment

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

Suggested change
* تعلم المزيد حول [نشر التطبيقات](/docs/tasks/run-application/run-stateless-application-deployment/).
* تعلم المزيد عن [نشر التطبيقات](/docs/tasks/run-application/run-stateless-application-deployment/).

Copy link

@adowair adowair left a comment

Choose a reason for hiding this comment

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

Thank you @seifrajhi for the quick turnaround. One more nit and a comment which I think hasn't been addressed. Otherwise LGTM


## نشر التطبيق

[*الحجيرة*](/ar/docs/concepts/workloads/pods/)كوبيرنيتيس هي مجموعة تتكون من حاوية واحدة أو عدة حاويات، مرتبطة معاً لأغراض الإدارة والشبكات. الحجيرة في هذا البرنامج التعليمي تحتوي على حاوية واحدة فقط. [*النشر*](/ar/docs/concepts/workloads/controllers/deployment/) في كوبيرنيتيس هو وحدة تتولى إنشاء وإدارة وتحجيم الحجيرات، وهي الأداة الموصى باستخدامها لتلك الأغراض. يحافظ النشر على صحة حجيرتك ويعيد تشغيل الحاويات داخلها إذا توقفت.
Copy link

Choose a reason for hiding this comment

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

Suggested change
[*الحجيرة*](/ar/docs/concepts/workloads/pods/)كوبيرنيتيس هي مجموعة تتكون من حاوية واحدة أو عدة حاويات، مرتبطة معاً لأغراض الإدارة والشبكات. الحجيرة في هذا البرنامج التعليمي تحتوي على حاوية واحدة فقط. [*النشر*](/ar/docs/concepts/workloads/controllers/deployment/) في كوبيرنيتيس هو وحدة تتولى إنشاء وإدارة وتحجيم الحجيرات، وهي الأداة الموصى باستخدامها لتلك الأغراض. يحافظ النشر على صحة حجيرتك ويعيد تشغيل الحاويات داخلها إذا توقفت.
[*الحجيرة*](/ar/docs/concepts/workloads/pods/) في كوبيرنيتيس هي مجموعة تتكون من حاوية واحدة أو عدة حاويات، مرتبطة معاً لأغراض الإدارة والشبكات. الحجيرة في هذا البرنامج التعليمي تحتوي على حاوية واحدة فقط. [*النشر*](/ar/docs/concepts/workloads/controllers/deployment/) في كوبيرنيتيس هو وحدة تتولى إنشاء وإدارة وتحجيم الحجيرات، وهي الأداة الموصى باستخدامها لتلك الأغراض. يحافظ النشر على صحة حجيرتك ويعيد تشغيل الحاويات داخلها إذا توقفت.

Copy link
Member Author

Choose a reason for hiding this comment

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

I thought it was a typo a first but it looks like it is legit rectification

Copy link

Choose a reason for hiding this comment

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

Yeah thank you, it confused me at first too

@adowair
Copy link

adowair commented Apr 17, 2024

/lgtm

Thank you again @seifrajhi for the heavy lift 🚀

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

LGTM label has been added.

Git tree hash: 8e695dc942181c0cb02a38be1f1ca63d5c84bb05

@k8s-ci-robot k8s-ci-robot merged commit 64b8f87 into kubernetes:dev-1.29-ar.1 Apr 17, 2024
2 checks passed
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/ar Issues or PRs related to Arabic 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants