Skip to content

cuopt service add healthcheck for /#114

Merged
rapids-bot[bot] merged 5 commits intoNVIDIA:branch-25.08from
tmckayus:healthcheck
Jul 17, 2025
Merged

cuopt service add healthcheck for /#114
rapids-bot[bot] merged 5 commits intoNVIDIA:branch-25.08from
tmckayus:healthcheck

Conversation

@tmckayus
Copy link
Copy Markdown
Contributor

Some deployment platforms like NVIDIA Brev may expect a healthcheck at the root url (/). Add the / path for the standard healthcheck response.

@tmckayus tmckayus requested a review from a team as a code owner June 18, 2025 20:13
@tmckayus tmckayus requested review from Iroy30 and rgsl888prabhu June 18, 2025 20:13
@tmckayus tmckayus added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Jun 18, 2025
Copy link
Copy Markdown
Collaborator

@rgsl888prabhu rgsl888prabhu left a comment

Choose a reason for hiding this comment

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

@tmckayus Lets add a test for this endpoint

@anandhkb anandhkb added this to the 25.08 milestone Jul 1, 2025
@rgsl888prabhu
Copy link
Copy Markdown
Collaborator

/merge

@rapids-bot rapids-bot bot merged commit 34f11f8 into NVIDIA:branch-25.08 Jul 17, 2025
141 of 142 checks passed
@tmckayus tmckayus deleted the healthcheck branch February 5, 2026 14:51
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
Some deployment platforms like NVIDIA Brev may expect a healthcheck at the root url (/).  Add the / path for the standard healthcheck response.

Authors:
  - Trevor McKay (https://github.com/tmckayus)
  - Ramakrishnap (https://github.com/rgsl888prabhu)

Approvers:
  - Ishika Roy (https://github.com/Iroy30)

URL: NVIDIA#114
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants