Skip to content

Re-iterate status evaluation.#86

Merged
Gerrit91 merged 11 commits intofirewall-health-checkfrom
timeout-review
Feb 17, 2026
Merged

Re-iterate status evaluation.#86
Gerrit91 merged 11 commits intofirewall-health-checkfrom
timeout-review

Conversation

@Gerrit91
Copy link
Contributor

Description

None

@metal-robot metal-robot bot added the area: gardener Affects the gardener area. label Feb 10, 2026
@Gerrit91
Copy link
Contributor Author

I tried this out now in the test environment by stopping the firewall-controller on the firewall (at 09:15:50):

firewall-controller-manager-7c8979f8bd-z2rqc firewall-controller-manager {"time":"2026-02-17T09:17:40.885753867Z","level":"INFO","msg":"scheduled check for next health timeout","logger":"controllers/timeout","firewall-name":"shoot--pcfgbt--gerrit-firewall-aa3f7","reason":"not all health conditions are true: [SeedConnected]","in":"9m59.114256585s"}
firewall-controller-manager-7c8979f8bd-z2rqc firewall-controller-manager {"time":"2026-02-17T09:27:40.000739902Z","level":"INFO","msg":"firewall timeout exceeded, deleting from set","logger":"controllers/timeout","reason":"10m0s health timeout exceeded, seed connection lost","firewall-name":"shoot--pcfgbt--gerrit-firewall-aa3f7"}

I also did a test when I started the firewall-controller again during the time window and the firewall remained intact.

It would be nice to have one more sophisticated integration test for this but it seems like ni general it works.

@Gerrit91 Gerrit91 marked this pull request as ready for review February 17, 2026 09:29
@Gerrit91 Gerrit91 requested a review from a team as a code owner February 17, 2026 09:29
@Gerrit91 Gerrit91 merged commit f69eef7 into firewall-health-check Feb 17, 2026
@Gerrit91 Gerrit91 deleted the timeout-review branch February 17, 2026 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: gardener Affects the gardener area.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants