Skip to content

release/helm/node:v0.4.3#45

Merged
inyellowbus merged 1 commit intomasterfrom
release/helm/node/v0.4.3
Apr 1, 2026
Merged

release/helm/node:v0.4.3#45
inyellowbus merged 1 commit intomasterfrom
release/helm/node/v0.4.3

Conversation

@inyellowbus
Copy link
Copy Markdown
Contributor

  • terminationGracePeriodSeconds — configurable grace period before SIGKILL on pod termination. Defaults to 300s (5 minutes). The Kubernetes default of 30s is too short for a TON node — an unclean kill may corrupt the database and forces a cold boot

Copilot AI review requested due to automatic review settings April 1, 2026 16:05
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request introduces a configurable terminationGracePeriodSeconds parameter to the TON Rust Node Helm chart (v0.4.3). The Kubernetes default of 30 seconds is too short for a TON node, as an unclean kill may corrupt the database and force a cold boot. The new parameter defaults to 300 seconds (5 minutes), allowing graceful shutdown time for the node process.

Changes:

  • Added terminationGracePeriodSeconds parameter to values.yaml with a default value of 300 seconds
  • Integrated the parameter into the StatefulSet pod specification
  • Updated Chart version from 0.4.2 to 0.4.3
  • Added CHANGELOG entry documenting the new feature

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
helm/ton-rust-node/values.yaml Added terminationGracePeriodSeconds parameter with comprehensive documentation explaining the rationale
helm/ton-rust-node/templates/statefulset.yaml Configured the StatefulSet to use the new parameter at the pod spec level
helm/ton-rust-node/Chart.yaml Bumped version to 0.4.3
helm/ton-rust-node/CHANGELOG.md Added release notes entry for version 0.4.3

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@inyellowbus inyellowbus merged commit 228770b into master Apr 1, 2026
9 checks passed
@inyellowbus inyellowbus deleted the release/helm/node/v0.4.3 branch April 1, 2026 16: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.

3 participants