Skip to content
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

Watcher query timeout Backport Microk8s 1.28 #163

Merged
merged 8 commits into from
Aug 30, 2024

Conversation

louiseschmidtgen
Copy link
Contributor

@louiseschmidtgen louiseschmidtgen commented Aug 28, 2024

Description

Back-port watcher query timeout fix: #162
Depends on canonical/kine#33.

@louiseschmidtgen louiseschmidtgen requested a review from a team as a code owner August 28, 2024 11:37
@louiseschmidtgen louiseschmidtgen changed the base branch from master to 1.28 August 28, 2024 11:37
Copy link
Contributor

@bschimke95 bschimke95 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, minor nits

go.mod Outdated Show resolved Hide resolved
app/app.go Outdated
@@ -109,4 +111,5 @@ func init() {
dqliteCmd.Flags().StringVar(&opts.ProfilingListen, "profiling-listen", opts.ProfilingListen, "listen address for pprof endpoint")
dqliteCmd.Flags().BoolVar(&opts.DiskMode, "disk-mode", opts.DiskMode, "(experimental) run dqlite store in disk mode")
dqliteCmd.Flags().UintVar(&opts.ClientSessionCacheSize, "tls-client-session-cache-size", opts.ClientSessionCacheSize, "ClientCacheSession size for dial TLS config")
dqliteCmd.Flags().DurationVar(&opts.WatchQueryTimeout, "watch-query-timeout", opts.WatchQueryTimeout, "timeout for querying events in the watch poll loop. If timeout is reached, the poll loop will be re-triggered. The minimum value is 5 seconds.")
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: the timeout

Copy link
Contributor

@bschimke95 bschimke95 left a comment

Choose a reason for hiding this comment

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

LGTM

@louiseschmidtgen
Copy link
Contributor Author

Cool, Microk8s doesn't break and the timeout gets passed: Connecting to kine endpoint: dqlite://k8s?peer-file=/var/snap/microk8s/current/var/kubernetes/backend/localnode.yaml&driver-name=dqlite-1&watch-query-timeout=10s

@louiseschmidtgen louiseschmidtgen merged commit ab6b2aa into 1.28 Aug 30, 2024
1 check passed
@berkayoz berkayoz deleted the KU-1429/watcher-query-timeout-1.28 branch September 2, 2024 07:48
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