Skip to content

Conversation

@Baune8D
Copy link

@Baune8D Baune8D commented Oct 2, 2025

What this PR does / why we need it:

Adds a configuration option for defining tunnel timeout value

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • DCO signed
  • Chart Version bumped
  • Variables are documented in the README.md
  • Title of the PR starts with chart name (e.g. [stable/mychartname])

@DandyDeveloper
Copy link
Owner

@Baune8D I'd recommend we don't add a conditional check and instead defer to a reasonable default that we introduce for others consuming the chart.

Signed-off-by: Morten Larsen <[email protected]>
@Baune8D Baune8D force-pushed the add-tunnel-timeout branch from 5573be7 to 4079f4a Compare December 8, 2025 08:16
@Baune8D
Copy link
Author

Baune8D commented Dec 8, 2025

@DandyDeveloper Set default tunnel timeout to 1h since it seems to be a quite commen default.

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