Skip to content

AI rule: aws.sagemaker.training_job.long_running#151

Merged
javvaji-devops merged 2 commits intomainfrom
ai_rules_9
Apr 15, 2026
Merged

AI rule: aws.sagemaker.training_job.long_running#151
javvaji-devops merged 2 commits intomainfrom
ai_rules_9

Conversation

@sureshcsdp
Copy link
Copy Markdown
Collaborator

No description provided.

try:
findings = find_long_running_sagemaker_training_jobs(session, "us-east-1")
except PermissionError as e:
pytest.skip(f"Missing IAM permissions: {e}")
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why skip ? the test should fail right ? so we know some rules are missing

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

thanks, fixed

@javvaji-devops javvaji-devops merged commit 332db67 into main Apr 15, 2026
18 checks passed
@javvaji-devops javvaji-devops deleted the ai_rules_9 branch April 15, 2026 19:47
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.

2 participants