Skip to content

Conversation

dd-octo-sts[bot]
Copy link
Contributor

@dd-octo-sts dd-octo-sts bot commented Oct 17, 2025

Backport 8310fe0 from #42005.


What does this PR do?

Fixes flakey deployment test on windows

Motivation

=== Failed
=== FAIL: pkg/collector/corechecks/cluster/ksm/customresources TestDeploymentRolloutGeneration_OngoingRollout (0.00s)
    deployment_test.go:101: 
        	Error Trace:	C:/buildroot/datadog-agent/pkg/collector/corechecks/cluster/ksm/customresources/deployment_test.go:101
        	Error:      	"0" is not greater than "0"
        	Test:       	TestDeploymentRolloutGeneration_OngoingRollout
        	Messages:   	Rollout duration should be greater than 0 for ongoing rollout
=== FAIL: pkg/collector/corechecks/cluster/ksm/customresources TestDeploymentRolloutGeneration_OngoingRollout (re-run 1) (0.00s)
    deployment_test.go:101: 
        	Error Trace:	C:/buildroot/datadog-agent/pkg/collector/corechecks/cluster/ksm/customresources/deployment_test.go:101
        	Error:      	"0" is not greater than "0"
        	Test:       	TestDeploymentRolloutGeneration_OngoingRollout
        	Messages:   	Rollout duration should be greater than 0 for ongoing rollout
=== FAIL: pkg/collector/corechecks/cluster/ksm/customresources TestDeploymentRolloutGeneration_OngoingRollout (re-run 2) (0.00s)
    deployment_test.go:101: 
        	Error Trace:	C:/buildroot/datadog-agent/pkg/collector/corechecks/cluster/ksm/customresources/deployment_test.go:101
        	Error:      	"0" is not greater than "0"
        	Test:       	TestDeploymentRolloutGeneration_OngoingRollout
        	Messages:   	Rollout duration should be greater than 0 for ongoing rollout

Describe how you validated your changes

Additional Notes

### What does this PR do?
Fixes flakey deployment test on windows

### Motivation
```
=== Failed
=== FAIL: pkg/collector/corechecks/cluster/ksm/customresources TestDeploymentRolloutGeneration_OngoingRollout (0.00s)
    deployment_test.go:101:
        	Error Trace:	C:/buildroot/datadog-agent/pkg/collector/corechecks/cluster/ksm/customresources/deployment_test.go:101
        	Error:      	"0" is not greater than "0"
        	Test:       	TestDeploymentRolloutGeneration_OngoingRollout
        	Messages:   	Rollout duration should be greater than 0 for ongoing rollout
=== FAIL: pkg/collector/corechecks/cluster/ksm/customresources TestDeploymentRolloutGeneration_OngoingRollout (re-run 1) (0.00s)
    deployment_test.go:101:
        	Error Trace:	C:/buildroot/datadog-agent/pkg/collector/corechecks/cluster/ksm/customresources/deployment_test.go:101
        	Error:      	"0" is not greater than "0"
        	Test:       	TestDeploymentRolloutGeneration_OngoingRollout
        	Messages:   	Rollout duration should be greater than 0 for ongoing rollout
=== FAIL: pkg/collector/corechecks/cluster/ksm/customresources TestDeploymentRolloutGeneration_OngoingRollout (re-run 2) (0.00s)
    deployment_test.go:101:
        	Error Trace:	C:/buildroot/datadog-agent/pkg/collector/corechecks/cluster/ksm/customresources/deployment_test.go:101
        	Error:      	"0" is not greater than "0"
        	Test:       	TestDeploymentRolloutGeneration_OngoingRollout
        	Messages:   	Rollout duration should be greater than 0 for ongoing rollout
```
### Describe how you validated your changes

### Additional Notes

(cherry picked from commit 8310fe0)
@dd-octo-sts dd-octo-sts bot requested a review from a team as a code owner October 17, 2025 14:59
@dd-octo-sts dd-octo-sts bot added changelog/no-changelog qa/done QA done before merge and regressions are covered by tests backport bot short review PR is simple enough to be reviewed quickly team/container-integrations labels Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport bot changelog/no-changelog qa/done QA done before merge and regressions are covered by tests short review PR is simple enough to be reviewed quickly team/container-integrations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant