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

Pragma header not sent when using additional_groups with auto_create: True #813

Open
Luci2015 opened this issue Mar 17, 2023 · 0 comments · May be fixed by #825
Open

Pragma header not sent when using additional_groups with auto_create: True #813

Luci2015 opened this issue Mar 17, 2023 · 0 comments · May be fixed by #825

Comments

@Luci2015
Copy link
Collaborator

Description
Possibility to get throttled when using additional_groups if first call is missed for different reasons

  • Relevant config options
    additional_groups enabled
    groups are set for:
    auto_create: True

Expected behavior
not get throttled

Screenshots

023-03-13 04:00:09 94799 INFO processor - ---------- Start Sync with UMAPI -------------------------------------
2023-03-13 04:01:38 94799 WARNING umapi - UMAPI request limit reached (code 504 on try 1)
2023-03-13 04:01:38 94799 WARNING umapi - waiting 16 seconds to continue...
2023-03-13 04:01:54 94799 WARNING umapi - UMAPI request limit reached (code 429 on try 2)
2023-03-13 04:01:54 94799 WARNING umapi - waiting 1938 seconds to continue...
2023-03-13 04:39:38 94799 INFO umapi - Sending start_sync signal
2023-03-13 04:39:52 94799 INFO umapi - 1999/5785 (34.6%) 

The call to get all groups gets 504, retry kicks-in and second call to get groups gets throttled. GET users page 0 starts and Pragma header is sent along, then sync continues as usual.

Environment

  • UST version: all
  • OS type and version: all
  • Any other relevant info: n/a
Luci2015 added a commit to Luci2015/user-sync.py that referenced this issue Oct 9, 2023
@Luci2015 Luci2015 linked a pull request Oct 9, 2023 that will close this issue
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 a pull request may close this issue.

1 participant