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

[VSSL-11995] VESSL_NGINX_ENABLED 추가 #75

Merged
merged 5 commits into from
Feb 3, 2025
Merged

[VSSL-11995] VESSL_NGINX_ENABLED 추가 #75

merged 5 commits into from
Feb 3, 2025

Conversation

thomasgim
Copy link
Contributor

No description provided.

@thomasgim thomasgim requested a review from NEM-NE February 3, 2025 02:02
@@ -2,6 +2,7 @@ agent:
accessToken: # required
clusterName:
providerType: on-premise # on-premise, aws, gcp, oci, azure
nginxEnabled: true
Copy link
Contributor

Choose a reason for hiding this comment

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

value만 추가하는 것이 아니라 helm chart에 sub-chart로 nginx-ingress-controller를 설치하고 싶어요

+) 지금은 aron-onprem에서 설치하고 있을겁니다.
+) nginx-ingress-controller 추가 설정이 필요한 것으로 알고있습니다. 관련해서 @synthdnb 에게 여쭤보시지요

Copy link
Contributor

Choose a reason for hiding this comment

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

Comment on lines +401 to +411
vessl-nginx:
enabled: true
controller:
ingressClass:
name: "vessl-nginx"
service:
# This will create nodePort service on cluster biding port 80 and 443.
# Set this to false if cluster already has other ingress controller, such as ingress-nginx
create: false
# enableSnippets must be true to run VESSL service in nodeport mode
enableSnippets: true
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@synthdnb @NEM-NE 우선 aron-onprem 에 있는 설정 그대로 가져왔습니다.

@thomasgim thomasgim requested review from synthdnb and NEM-NE February 3, 2025 06:02
Copy link
Contributor

@synthdnb synthdnb left a comment

Choose a reason for hiding this comment

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

👍

@thomasgim thomasgim removed the request for review from NEM-NE February 3, 2025 09:55
@thomasgim thomasgim added the release 차트 배포 라벨입니다. label Feb 3, 2025
@thomasgim thomasgim merged commit 452cc51 into main Feb 3, 2025
@thomasgim thomasgim deleted the vssl-11995 branch February 3, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release 차트 배포 라벨입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants