Skip to content

[v18] Fix kube discovery service poll_interval value not set correctly#61791

Merged
kshi36 merged 5 commits intobranch/v18from
kevin/backport-60982-branch/v18
Dec 1, 2025
Merged

[v18] Fix kube discovery service poll_interval value not set correctly#61791
kshi36 merged 5 commits intobranch/v18from
kevin/backport-60982-branch/v18

Conversation

@kshi36
Copy link
Copy Markdown
Contributor

@kshi36 kshi36 commented Nov 25, 2025

Backport #60982 to branch/v18

changelog: Fixed bug where Kubernetes App Discovery poll_interval is not set correctly

…60982)

* Fix kube discovery service poll interval

* Add unit test, pass down clocks in discovery configs

* Leverage synctest, pass down clocks properly

* Refactor
@kshi36 kshi36 changed the title [v18] Fix kube discovery service poll_interval value not set correctly [v18] Fix kube discovery service poll_interval value not set correctly Nov 25, 2025
@kshi36
Copy link
Copy Markdown
Contributor Author

kshi36 commented Nov 25, 2025

I omitted test changes in discovery_test.go and discovery_eks_test.go due to discrepancies with synctest usage on branch/v18.

@kshi36 kshi36 marked this pull request as ready for review November 26, 2025 02:59
@rosstimothy
Copy link
Copy Markdown
Contributor

I omitted test changes in discovery_test.go and discovery_eks_test.go due to discrepancies with synctest usage on branch/v18.

Lets use testutils.synctest so that we can backport test coverage.

Copy link
Copy Markdown
Collaborator

@r0mant r0mant left a comment

Choose a reason for hiding this comment

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

lgtm once Tim's comment is addressed

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from bernardjkim December 1, 2025 09:51
@kshi36
Copy link
Copy Markdown
Contributor Author

kshi36 commented Dec 1, 2025

@rosstimothy Will need exclude flake, same for v17 backport #61792

@rosstimothy
Copy link
Copy Markdown
Contributor

@rosstimothy Will need exclude flake, same for v17 backport #61792

It's only a required check on master 🙃.

@kshi36 kshi36 added this pull request to the merge queue Dec 1, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 1, 2025
@kshi36 kshi36 added this pull request to the merge queue Dec 1, 2025
Merged via the queue into branch/v18 with commit 8479272 Dec 1, 2025
38 of 40 checks passed
@kshi36 kshi36 deleted the kevin/backport-60982-branch/v18 branch December 1, 2025 18:28
@aadc-dev aadc-dev mentioned this pull request Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants