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

Azure - member limitation of 100 #246

Closed
dortlii opened this issue Jan 17, 2023 · 7 comments · Fixed by #249
Closed

Azure - member limitation of 100 #246

dortlii opened this issue Jan 17, 2023 · 7 comments · Fixed by #249
Assignees
Labels
bug Something isn't working

Comments

@dortlii
Copy link

dortlii commented Jan 17, 2023

We are using the operator (version 0.0.23) on OpenShift 4.10.43 and syncing some Azure AD groups. We have 2 large groups with 100+ members (a big project and a common group) and in OpenShift we only find 100 from e.g. 123 members.

Is there a limit which we can adjust? If not, what could be a good solution for our 2 biggest groups? We are syncing a lot of groups (probably around 230) and only these 2 are causing problems.

The operator status is fine and syncing every 5 min as configured. The manager container from the group-sync-operator-controller-manager doesn't show any errors. Is this an already known issue / limitation?

@sabre1041
Copy link
Collaborator

@dortlii ive acknowledged the bug. looking into a resolution

@sabre1041 sabre1041 self-assigned this Feb 2, 2023
@sabre1041 sabre1041 added the bug Something isn't working label Feb 2, 2023
@sabre1041
Copy link
Collaborator

@dortlii looked into the issue. Ultimately, I was unable to find the foolproof solution at this time. However, an increase to the page size was made to the maximum allowed by the Azure graph API (999 items) which should cover your use case.

@dortlii
Copy link
Author

dortlii commented Feb 3, 2023

@sabre1041 perfect, as soon as we can test this with the page size increased, I will updated this issue. Thanks for taking a look at it.

@MoazHafeez
Copy link

Hi all, Did this get resolved and tested?
I am having the same issue:
operator (version 0.0.28) on OpenShift 4.12.51

@sabre1041
Copy link
Collaborator

@MoazHafeez yes it was tested at the time

@MoazHafeez
Copy link

@sabre1041 does anything need to be configured on either the operator, instance or Azure side to increase the page size?

@sabre1041
Copy link
Collaborator

@sabre1041 does anything need to be configured on either the operator, instance or Azure side to increase the page size?

#313 was just submitted to address providing a pagesize to all queries to Azure. It is the hope that it addresses the issues that you are currently facing

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

Successfully merging a pull request may close this issue.

3 participants