Skip to content

Conversation

@syrull
Copy link
Contributor

@syrull syrull commented Jun 28, 2025

Issue: #5728

@syrull syrull self-assigned this Jun 28, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jun 28, 2025

Test Results

4 333 tests  ±0   4 329 ✅ ±0   2m 7s ⏱️ -7s
  407 suites ±0       4 💤 ±0 
   30 files   ±0       0 ❌ ±0 

Results for commit da6b7ec. ± Comparison against base commit d159ef9.

♻️ This comment has been updated with latest results.

@syrull syrull changed the title Restructure microsoft.identityAndAccess Restructure microsoft.identityAndAccess and add organization Jun 28, 2025
@syrull
Copy link
Contributor Author

syrull commented Jun 28, 2025

Accidentally added organization to this branch, which resolves #5727

This should be a new branch but I didn't realise that I committed into this one, apologies.

@syrull syrull requested a review from glower July 7, 2025 08:23
@syrull syrull closed this Jul 15, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jul 15, 2025
@syrull syrull reopened this Jul 15, 2025
// A container resource for identity and access management
microsoft.identityAndAccess {
[]microsoft.identityAndAccess.policy
// Get the instances of role eligibilities
Copy link
Contributor

Choose a reason for hiding this comment

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

this pr needs a rebase - roleEligibilityScheduleInstances() []microsoft.identityAndAccess.roleEligibilityScheduleInstance already exists on main


// A rule defined for a PIM role management policy
private microsoft.identityAndAccess.policy.rule @defaults("id") {
private microsoft.identityAndAccess.privilegedIdentityManagement.policy.rule @defaults("id") {
Copy link
Contributor

Choose a reason for hiding this comment

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

why is this change to the resource being made?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants