Skip to content

Conversation

rastislavs
Copy link
Contributor

@rastislavs rastislavs commented Jun 9, 2025

Once this PR is merged, a GitHub action will update the labels of these PRs:

 38537

[ upstream commit dd79107 ]

[ backporter's notes:
  - modified Watcher's New method to not
    take the logger arg and use the old logger
  - skipped changes in pkg/k8s/client/restConfig_provider.go
    (not present in the target branch)
]

Remove reliance on fsnotify by replacing with a polling
mechanism to check for modifications to tracked files.

There is a default interval that is configurable depending
on the usage, and it is also significantly shortened when
running tests.

Signed-off-by: Glib Smaga <code@gsmaga.com>
Signed-off-by: Rastislav Szabo <rastislav.szabo@isovalent.com>
@rastislavs rastislavs added kind/backports This PR provides functionality previously merged into master. backport/1.16 This PR represents a backport for Cilium 1.16.x of a PR that was merged to main. labels Jun 9, 2025
@rastislavs rastislavs requested a review from glibsm June 9, 2025 08:03
@rastislavs
Copy link
Contributor Author

/test

@rastislavs rastislavs marked this pull request as ready for review June 9, 2025 10:25
@rastislavs rastislavs requested a review from a team as a code owner June 9, 2025 10:25
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 11, 2025
@rastislavs
Copy link
Contributor Author

Conformance K8s Kind: #40004

@tklauser tklauser added this pull request to the merge queue Jun 12, 2025
Merged via the queue into v1.16 with commit 7dd44fe Jun 12, 2025
329 of 349 checks passed
@tklauser tklauser deleted the pr/v1.16-backport-2025-06-09-09-53 branch June 12, 2025 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.16 This PR represents a backport for Cilium 1.16.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants