Skip to content

feat(opensearch): add external service for client nodes#873

Merged
joluc merged 16 commits intomainfrom
feat/enable-lb-service
May 27, 2025
Merged

feat(opensearch): add external service for client nodes#873
joluc merged 16 commits intomainfrom
feat/enable-lb-service

Conversation

@joluc
Copy link
Contributor

@joluc joluc commented May 14, 2025

Pull Request Details

Added an external LoadBalancer service to expose client nodes. This is used for data ingestion from tools like OpenTelemetry.

@joluc joluc requested a review from a team as a code owner May 14, 2025 13:54
Copy link
Member

@olandr olandr left a comment

Choose a reason for hiding this comment

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

Two questions on the choice of LoadBalancer. Maybe I am a bit overly cautious here...

@joluc joluc force-pushed the feat/enable-lb-service branch from e7a83cb to 570e966 Compare May 22, 2025 12:46
@joluc joluc force-pushed the feat/enable-lb-service branch from 570e966 to 39a41a0 Compare May 26, 2025 09:43
@joluc joluc requested review from a team and olandr May 26, 2025 12:54
Copy link
Member

@olandr olandr left a comment

Choose a reason for hiding this comment

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

Great stuff! Looks good to me.

@joluc joluc merged commit 3f30f1a into main May 27, 2025
8 checks passed
@joluc joluc deleted the feat/enable-lb-service branch May 27, 2025 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants