Skip to content

rename confusing variable#219

Merged
simonLeary42 merged 1 commit intomainfrom
remove-group-ou
Jan 26, 2026
Merged

rename confusing variable#219
simonLeary42 merged 1 commit intomainfrom
remove-group-ou

Conversation

@simonLeary42
Copy link
Copy Markdown
Member

No description provided.

@simonLeary42 simonLeary42 marked this pull request as draft May 8, 2025 03:07
@bryank-cs
Copy link
Copy Markdown
Contributor

Isn't this the ou that is used for the gid=uid groups?

@simonLeary42 simonLeary42 changed the title remove useless group ou rename confusing variable May 8, 2025
@simonLeary42
Copy link
Copy Markdown
Member Author

@bryank-cs you're right. changes are now to rename "group ou" to "user group ou"

@simonLeary42 simonLeary42 force-pushed the main branch 5 times, most recently from 021a1af to 969c0ad Compare June 16, 2025 15:18
@simonLeary42
Copy link
Copy Markdown
Member Author

simonLeary42 commented Jun 16, 2025

TODO: differentiate between "unityusers posixGroup" and "OU that contains posixGroups for each unityuser"

edit: unityusers is now the "qualified users group"

@simonLeary42 simonLeary42 marked this pull request as ready for review January 26, 2026 15:30
@simonLeary42 simonLeary42 merged commit a05f471 into main Jan 26, 2026
3 checks passed
@simonLeary42 simonLeary42 deleted the remove-group-ou branch January 26, 2026 15:32
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