Chat: Adjust rate limiting for @blenderbot:blender.org #120

Closed
opened 2024-08-20 21:10:33 +02:00 by Bart van der Braak · 3 comments

This bot (created by @LazyDodo) adds status updates to these chats:

any chance you can give blender bot a bit of a break on the API limit? those 8 lines were enough for

{"errcode":"M_LIMIT_EXCEEDED","error":"Too Many Requests","retry_after_ms":3021}
This bot (created by @LazyDodo) adds status updates to these chats: - [#blender-status-buildbot:blender.org](https://chat.blender.org/#/room/#blender-status-buildbot:blender.org) - [#blender-status-buildbot-docs:blender.org](https://chat.blender.org/#/room/#blender-status-buildbot-docs:blender.org) > any chance you can give blender bot a bit of a break on the API limit? those 8 lines were enough for > ``` > {"errcode":"M_LIMIT_EXCEEDED","error":"Too Many Requests","retry_after_ms":3021} > ```
Bart van der Braak added the
Service
Chat
Type
Config
labels 2024-08-20 21:10:33 +02:00
Author
Owner

Also something to investigate:

one thing i don't like in the new system, is the api token, is just listed in the help & about section (stupid place), i've seen it change (i think) and there's no way to tell what tokens are out there, or a way to revoke a token
i don't feel in control at all

Also something to investigate: > one thing i don't like in the new system, is the api token, is just listed in the help & about section (stupid place), i've seen it change (i think) and there's no way to tell what tokens are out there, or a way to revoke a token > i don't feel in control at all
Bart van der Braak added this to the DevOps Progress Board project 2024-08-20 21:12:53 +02:00

I'll be upfront, that may not have been 8 lines, i didn't notice it was also updating a second channel, which had a fair number of builds in it.

it's just spewing a lot of data right now, since i'm transitioning the bots from the old to the new system. once things settle i'll likely be fine

I'd say we ignore the rate limit issue for now, since in practice it may not be much of a problem (you rarely have a flood of 8+ builds coming in at the same time) if it in the long run turns out to be an issue we'll burn that bridge when we get there.

I'll be upfront, that may not have been 8 lines, i didn't notice it was also updating a second channel, which had a fair number of builds in it. it's just spewing a lot of data right now, since i'm transitioning the bots from the old to the new system. once things settle i'll likely be fine I'd say we ignore the rate limit issue for now, since in practice it may not be much of a problem (you rarely have a flood of 8+ builds coming in at the same time) if it in the long run turns out to be an issue we'll burn that bridge when we get there.
Author
Owner

@LazyDodo I'll close this issue for now. If you ever notice that the rate limit becomes a problem, we'll reopen this.

@LazyDodo I'll close this issue for now. If you ever notice that the rate limit becomes a problem, we'll reopen this.
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: infrastructure/blender-projects-platform#120
No description provided.