Skip to content

AI rule 1 for Azure: azure.aml.compute.idle#123

Merged
sureshcsdp merged 1 commit into
mainfrom
ai_rule_2_azure
Apr 1, 2026
Merged

AI rule 1 for Azure: azure.aml.compute.idle#123
sureshcsdp merged 1 commit into
mainfrom
ai_rule_2_azure

Conversation

@sureshcsdp
Copy link
Copy Markdown
Collaborator

No description provided.

@@ -0,0 +1,371 @@
from datetime import datetime, timedelta, timezone
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.

this logic looks alright to me 👍

Copy link
Copy Markdown
Collaborator

@javvaji-devops javvaji-devops left a comment

Choose a reason for hiding this comment

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

LGTM 👍
I see all docs are correctly updated this time ,

@@ -0,0 +1,371 @@
from datetime import datetime, timedelta, timezone
from typing import List, Optional
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.

did we test this rule logic using read azure creds/perms just to make sure it works or breaks?

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.

i tested with real creds but the endpoint has not returned any findings due to no ML Compute resources - but for sure it didnt break

@sureshcsdp sureshcsdp merged commit 1297d48 into main Apr 1, 2026
19 checks passed
@sureshcsdp sureshcsdp deleted the ai_rule_2_azure branch April 1, 2026 14:37
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