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.

Sign up for GitHub

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

Jump to bottom

Added Health Probes #867

Merged
merged 45 commits into from
Apr 1, 2025
Merged

Added Health Probes #867

merged 45 commits into from
Apr 1, 2025

Conversation

njaved
Copy link
Collaborator

@njaved njaved commented Mar 27, 2025

No description provided.

njaved added 30 commits March 18, 2025 09:14
@njaved
Added NBS6 Health Probes
751c385
@njaved
Added SAS Health Probe for Port
14c47da
@njaved
ignore values yaml files.
3fdb545
@njaved
ignore values yaml files.
03a4496
@njaved
Added defaults
6ab96a7
@njaved
Added parameters in example values file
718626b
@njaved
Added port health checks for nbs-gateway
5dd35cf
@njaved
Added health check to keycloak
97f6fac
@njaved
Added probe as default enabled
da174d1
@njaved
correct probe syntax location
3c1d07e
@njaved
shorter time nbs-gateway
e3d9a8e
@njaved
Merge branch 'main' into njaved-health-check
1ad1f4d
@njaved
Added porbe default as true
06b0ef4
@njaved
Merge branch 'njaved-health-check' of https://github.com/CDCgov/NEDSS…
0bb9d6c 
…-Helm into njaved-health-check
@njaved
Merge branch 'main' into njaved-health-check
c7089ee
@njaved
Updated to use port container port variable
072b5f8
@njaved
Added probes for nnd and dataingestion
2834af0
@njaved
added probe in values file
cd829fd
@njaved
Updated to port probe
0d9a396
@njaved
Updated probe enabled logic
c8114d4
@njaved
probe logic update
64e43c7
@njaved
Updated probe logic
b883d99
@njaved
Updated probe logic
b0a388a
@njaved
Updated wait time
c42a372
@njaved
Removed parem from values
def294c
@njaved
removed if statement
ba95ce6
@njaved
revert changes
a0cfb9a
@njaved
Added probe for obser, person and post
eae762d
@njaved
if Statement update
ebb031b
@njaved
syntax update
d8eb8f0
szamfir
szamfir reviewed Mar 27, 2025
View reviewed changes
Copy link
Collaborator

@szamfir szamfir left a comment

Choose a reason for hiding this comment

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

nbs-gateway seems to have different values for its probe in main

szamfir
szamfir approved these changes Apr 1, 2025
View reviewed changes
Copy link
Collaborator

@szamfir szamfir left a comment

Choose a reason for hiding this comment

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

LGTM

@njaved njaved merged commit 09021bf into main Apr 1, 2025
2 checks passed
@njaved njaved deleted the njaved-health-check branch April 1, 2025 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@szamfir szamfir szamfir approved these changes

@pvijay-enquizit pvijay-enquizit Awaiting requested review from pvijay-enquizit pvijay-enquizit is a code owner

@cmoss-enq cmoss-enq Awaiting requested review from cmoss-enq cmoss-enq is a code owner

@tukhi tukhi Awaiting requested review from tukhi tukhi is a code owner

@kramakur kramakur Awaiting requested review from kramakur kramakur is a code owner

@akhamraev akhamraev Awaiting requested review from akhamraev akhamraev is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@njaved @szamfir