Skip to content

Add ability to modify probe settings#35

Merged
tananaev merged 2 commits intotraccar:mainfrom
rossigee:modify-probe-settings
Feb 22, 2026
Merged

Add ability to modify probe settings#35
tananaev merged 2 commits intotraccar:mainfrom
rossigee:modify-probe-settings

Conversation

@rossigee
Copy link
Contributor

Allows the setting of custom probe values (refs #34).

@tananaev
Copy link
Member

Can you please rebase.

@rossigee
Copy link
Contributor Author

Rebased. Thanks.

@rossigee rossigee force-pushed the modify-probe-settings branch 2 times, most recently from 6e66ad6 to e8ac319 Compare August 21, 2025 23:46
- Add readiness and liveness probe configuration options
- Allow customization of probe intervals, timeouts, and failure thresholds
- Enhance deployment flexibility for different environments
@rossigee rossigee force-pushed the modify-probe-settings branch from e8ac319 to 578dc70 Compare August 21, 2025 23:48
@rossigee
Copy link
Contributor Author

rossigee commented Oct 7, 2025

@tananaev - is this still of interest?

@tananaev
Copy link
Member

tananaev commented Oct 7, 2025

We would need someone else to review this because I'm not really an expert.

@rossigee
Copy link
Contributor Author

@tananaev - who would you expect to review this ?

@tananaev
Copy link
Member

Probably some other people who contributed here.

@rossigee
Copy link
Contributor Author

Probably some other people who contributed here.

Any people specifically?

I see @filippolmt has commented/approved other previously merged PRs.

@filippolmt
Copy link
Contributor

The processing seems correct to me. Move the logic to the values like this, so, if needed, you can directly modify the livenessProbe and readinessProbe.

@rossigee
Copy link
Contributor Author

rossigee commented Dec 1, 2025

Worth merging ?

@rossigee
Copy link
Contributor Author

If there's no interest, please let me know so I can close this out.

@tananaev tananaev merged commit 10a9128 into traccar:main Feb 22, 2026
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