Skip to content

Set cluster-agent replicas to 2 #1492

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Set cluster-agent replicas to 2 #1492

wants to merge 1 commit into from

Conversation

zhuminyi
Copy link
Contributor

@zhuminyi zhuminyi commented Mar 28, 2025

What does this PR do?

Set cluster agent replicas to 2 so that k8s cluster has one leader dca and one follower dca pod.

Which scenarios this will impact?

k8s_test.go
eks_test.go

Motivation

Cluster agent leader election test has been added to E2E test inPR

Additional Notes

Test run is here

@zhuminyi zhuminyi force-pushed the setup_dca_replicas branch 2 times, most recently from d1b6437 to c32f269 Compare April 3, 2025 17:22
@zhuminyi zhuminyi force-pushed the setup_dca_replicas branch 2 times, most recently from d03224b to 731b7ba Compare April 5, 2025 13:34
@zhuminyi zhuminyi force-pushed the setup_dca_replicas branch from 731b7ba to dfcc0ab Compare April 9, 2025 18:00
@zhuminyi zhuminyi marked this pull request as ready for review April 9, 2025 18:14
@zhuminyi zhuminyi requested a review from a team as a code owner April 9, 2025 18:14
@zhuminyi zhuminyi changed the title Set dca replicas to 2 Set cluster-agent replicas to 2 Apr 9, 2025
@@ -429,6 +429,7 @@ func buildLinuxHelmValues(baseName, agentImagePath, agentImageTag, clusterAgentI
"tag": pulumi.String(clusterAgentImageTag),
"doNotCheckTag": pulumi.Bool(true),
},
"replicas": pulumi.Int(2),
Copy link
Member

Choose a reason for hiding this comment

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

Is there any impact expected on other Kubernetes based tests?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, only cluster agent status is affected. datadog-agent side test has been adjusted accordingly and merged.

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