Skip to content

Check/issue 11052 sagemaker models monitor enabled#11278

Open
RishiWig3 wants to merge 4 commits into
prowler-cloud:masterfrom
RishiWig3:check/issue_11052_sagemaker_models_monitor_enabled
Open

Check/issue 11052 sagemaker models monitor enabled#11278
RishiWig3 wants to merge 4 commits into
prowler-cloud:masterfrom
RishiWig3:check/issue_11052_sagemaker_models_monitor_enabled

Conversation

@RishiWig3
Copy link
Copy Markdown

Context

Please include relevant motivation and context for this PR.

If fixes an issue please add it with Fix #XXXX

Description

Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.

Steps to review

Please add a detailed description of how to review this PR.

Checklist

Community Checklist
  • This feature/issue is listed in here or roadmap.prowler.com
  • Is it assigned to me, if not, request it via the issue/feature in here or Prowler Community Slack

SDK/CLI

  • Are there new checks included in this PR? Yes / No
    • If so, do we need to update permissions for the provider? Please review this carefully.

UI

  • All issue/task requirements work as expected on the UI
  • If this PR adds or updates npm dependencies, include package-health evidence (maintenance, popularity, known vulnerabilities, license, release age) and explain why existing/native alternatives are insufficient.
  • Screenshots/Video of the functionality flow (if applicable) - Mobile (X < 640px)
  • Screenshots/Video of the functionality flow (if applicable) - Table (640px > X < 1024px)
  • Screenshots/Video of the functionality flow (if applicable) - Desktop (X > 1024px)
  • Ensure new entries are added to CHANGELOG.md, if applicable.

API

  • All issue/task requirements work as expected on the API
  • Endpoint response output (if applicable)
  • EXPLAIN ANALYZE output for new/modified queries or indexes (if applicable)
  • Performance test results (if applicable)
  • Any other relevant evidence of the implementation (if applicable)
  • Verify if API specs need to be regenerated.
  • Check if version updates are required (e.g., specs, uv, etc.).
  • Ensure new entries are added to CHANGELOG.md, if applicable.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@RishiWig3 RishiWig3 requested a review from a team as a code owner May 21, 2026 08:41
@github-actions github-actions Bot added provider/aws Issues/PRs related with the AWS provider metadata-review community Opened by the Community labels May 21, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

Conflict Markers Resolved

All conflict markers have been successfully resolved in this pull request.

@RishiWig3 RishiWig3 force-pushed the check/issue_11052_sagemaker_models_monitor_enabled branch from 1040f09 to ecd1abc Compare May 21, 2026 08:45
@RishiWig3
Copy link
Copy Markdown
Author

Hi @danibarranqueroo, I believe I have completed the functionality of the [New Check]: SageMaker model monitoring schedules are active #11052. Worked on the service layer, the check and the three test cases. Would be happy to step back in to fix anything.

Also want to note: with the nature of the check only returning a singular report, it would be returning the name of the first passing monitoring schedule (i.e. of status "Scheduled") or the name last in its set of failing monitoring schedules (i.e.e "Stopped, Pending, Failed")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Opened by the Community metadata-review provider/aws Issues/PRs related with the AWS provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants