Skip to content

Conversation

@ianychoi
Copy link
Member

Description

Update Korean translation on ingress.md

Issue

Closes: #53828

@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. area/localization General issues or PRs related to localization labels Dec 28, 2025
@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 ianychoi for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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 language/ko Issues or PRs related to Korean 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. labels Dec 28, 2025
@netlify
Copy link

netlify bot commented Dec 28, 2025

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 79afab0
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/695c3cfdbc24630008428994
😎 Deploy Preview https://deploy-preview-53829--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 project configuration.

@developowl
Copy link
Member

/assign

Copy link
Member

@developowl developowl left a comment

Choose a reason for hiding this comment

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

@ianychoi Thank you for contribution.🥝
I have left some minor reviews. Please take a look and ping me when you're done!

[플래그](https://kubernetes.github.io/ingress-nginx/#what-is-the-flag-watch-ingress-without-class)를 이용하여 구성될 수 있다.
하지만 [아래](#default-ingress-class)에 나와 있는 것과 같이 기본 `IngressClass`를 명시하는 것을
[권장](https://kubernetes.github.io/ingress-nginx/#i-have-only-one-instance-of-the-ingresss-nginx-controller-in-my-cluster-what-should-i-do)한다.
몇몇 인그레스 컨트롤러는 기본 `IngressClass`가 정의되어 있지
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
몇몇 인그레스 컨트롤러는 기본 `IngressClass` 정의되어 있지
몇몇 인그레스 컨트롤러는 기본 IngressClass가 정의되어 있지

Comment on lines +261 to +278
IngressClass의 `.spec.parameters` 필드를 사용하여
해당 IngressClass와 연관있는 환경 설정을 제공하는 다른 리소스를 참조할 수 있다.

사용 가능한 파라미터의 상세한 타입은
인그레스클래스의 `.spec.parameters` 필드에 명시한 인그레스 컨트롤러의 종류에 따라 다르다.
IngressClass의 `.spec.parameters` 필드에 명시한 인그레스 컨트롤러의 종류에 따라 다르다.

### 인그레스클래스 범위
### IngressClass 범위

인그레스 컨트롤러의 종류에 따라, 클러스터 범위로 설정한 파라미터의 사용이 가능할 수도 있고,
또는 한 네임스페이스에서만 사용 가능할 수도 있다.

{{< tabs name="tabs_ingressclass_parameter_scope" >}}
{{% tab name="클러스터" %}}
인그레스클래스 파라미터의 기본 범위는 클러스터 범위이다.
IngressClass 파라미터의 기본 범위는 클러스터 범위이다.

`.spec.parameters` 필드만 설정하고 `.spec.parameters.scope` 필드는 지정하지 않거나,
`.spec.parameters.scope` 필드를 `Cluster`로 지정하면,
인그레스클래스는 클러스터 범위의 리소스를 참조한다.
IngressClass는 클러스터 범위의 리소스를 참조한다.
Copy link
Member

Choose a reason for hiding this comment

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

In line 252, it is written as 인그레스 클래스, but you noticed IngressClass is used below. I think it would be better to use consistent terminology throughout the document. Thank you.

Copy link
Member Author

Choose a reason for hiding this comment

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

@developowl It is because the original string on L252 is "Ingress class", not "IngressClass". Note that I translated from "Ingress class" to "인그레스 클래스", and from "IngressClass" to "IngressClass" not to be confused. You can also double-check by seeing between L252 and L267.

Copy link
Member

Choose a reason for hiding this comment

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

I'm sorry to confused between by "IngressClass" and "Ingress Class". Thank you for noted for me.

@ianychoi
Copy link
Member Author

ianychoi commented Jan 5, 2026

@developowl I have reflected all the comments except one - please take a look. Thank you.

@developowl
Copy link
Member

/lgtm

/cc @jongwooo I think this PR's looks good to me. Please check this PR. Thank you! 🥝

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

LGTM label has been added.

DetailsGit tree hash: 897b567090821da7aeed0f1edffe41ed06c7c7f1

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

Labels

area/localization General issues or PRs related to localization cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/ko Issues or PRs related to Korean 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.

[ko] Up-to-date Ingress doc

3 participants