Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

O365Group Keys need to Reflect Identity #85

Closed
NikCharlebois opened this issue Apr 5, 2019 · 8 comments
Closed

O365Group Keys need to Reflect Identity #85

NikCharlebois opened this issue Apr 5, 2019 · 8 comments
Assignees
Labels
Bug Something isn't working Question Further information is requested

Comments

@NikCharlebois
Copy link
Collaborator

Details of the scenario you tried and the problem that is occurring

The O365Group resource currently uses DisplayName as the primary key, however I have recently ran into a scenario where multiple groups have the exact same display name, but different Identity. We should change the Key of the resource to rely on Identity to make sure the group is unique.

Verbose logs showing the problem

N/A

Suggested solution to the issue

Change the Resource's Key to be Identity

The DSC configuration that is used to reproduce the issue (as detailed as possible)

N/A

@NikCharlebois NikCharlebois self-assigned this Apr 5, 2019
@NikCharlebois NikCharlebois added Bug Something isn't working Question Further information is requested labels Apr 5, 2019
@NikCharlebois
Copy link
Collaborator Author

Off course the challenge is that Security Groups don't have Identities. Changing this as a question:

Should we split O365Group into 2 resources:
O365Groups
O365SecurityGroups

Pinging my EXO folks: @poiriersimon @RobBiddle @SammyKrosoft

@RobBiddle
Copy link
Contributor

I agree, those should be split into separate resources.

Also, I was planning on adding a separate resource for O365 Unified Groups, which has some potential overlap with the TeamsTeam resource since a new Team creates a Unified Group if an existing one isn't specified. We'll need to cover the caveats of configuration resource order (e.g. for things like Outlook group visibility) in the documentation.

@RobBiddle
Copy link
Contributor

Another thing to consider. Should the group related resources be able to change the group type on an existing group?

@NikCharlebois
Copy link
Collaborator Author

@RobBiddle very good point. I believe we don't want to go to the extent of allowing a Security Group to become an Office 365 Group. Under the cover those really are 2 different entities and quite honestly, having both entity type handled by the same resources gives the impression that these GroupType are interchangeable when they are not. Having them handled by two different resources will stop giving the impression that we can easily flip back and forth between types I think.

@markroloff
Copy link

Hey guys!

Anybody actively working on this? I've been using my own scripts to handle migrating groups between tenants but have been meaning to cut my teeth on writing DSC resources, as I feel like it'd work better. Happy to take a stab at it!

@jeff29922
Copy link
Collaborator

@markroloff I don't think anyone is actively working on this, please feel free to take a shot at it.

@NikCharlebois
Copy link
Collaborator Author

@markroloff I will actually go ahead and close this. Now that Azure AD is part of M365DSC, we have a new AADMSGroup resource that will handle this. More than happy to have your look through the backlog and pick any resource you'd like to help working on!

@markroloff
Copy link

@NikCharlebois no worries! I got stuck in a bit of analysis-paralysis and then work took a dump on me, so no big loss 😂

Will definitely be scouring the backlog for anything I think I can jump on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants