Skip to content

resolves #1881 NLP: add healthcheck for Sagemaker classifiers#1882

Merged
vsct-jburet merged 4 commits intotheopenconversationkit:masterfrom
Pazuzzu:feat/nlp-healtcheck
May 2, 2025
Merged

resolves #1881 NLP: add healthcheck for Sagemaker classifiers#1882
vsct-jburet merged 4 commits intotheopenconversationkit:masterfrom
Pazuzzu:feat/nlp-healtcheck

Conversation

@Pazuzzu
Copy link
Copy Markdown
Contributor

@Pazuzzu Pazuzzu commented Apr 8, 2025

This issue addresses our need for a healthcheck functionality for Sagemaker intent and entity classifiers to ensure proper monitoring of the NLP services. The implementation will:

  • Group clients by name and verify their health status
  • Include the healtcheck result to the existing detailed healthcheck

@Pazuzzu Pazuzzu force-pushed the feat/nlp-healtcheck branch from cbc71d0 to 5bf5d08 Compare April 8, 2025 16:34
Comment thread nlp/model/sagemaker/src/main/kotlin/SagemakerEngineProvider.kt Outdated
Comment thread nlp/core/service/src/main/kotlin/NlpCoreService.kt Outdated
Comment thread nlp/model/sagemaker/src/main/kotlin/SagemakerAwsClient.kt Outdated
@Pazuzzu Pazuzzu force-pushed the feat/nlp-healtcheck branch from 5bf5d08 to 2bd1fa3 Compare April 10, 2025 17:24
@vsct-jburet vsct-jburet self-requested a review April 29, 2025 09:22
@vsct-jburet vsct-jburet added this to the 25.3.3 milestone May 2, 2025
@vsct-jburet vsct-jburet merged commit a853eba into theopenconversationkit:master May 2, 2025
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.

4 participants