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

oximeter should refresh its producer list more frequently #6916

Closed
bnaecker opened this issue Oct 22, 2024 · 0 comments · Fixed by #6926
Closed

oximeter should refresh its producer list more frequently #6916

bnaecker opened this issue Oct 22, 2024 · 0 comments · Fixed by #6926
Assignees
Labels

Comments

@bnaecker
Copy link
Collaborator

See #6905 and #6895 for more context.

This issue tracks making oximeter only learn about producers by fetching its list from Nexus, and for doing so more periodically. Today, it does so every 10 minutes, but that should be much smaller, 10-30 seconds. Secondly, we should remove entirely the API that Nexus uses to notify oximeter of a new producer. Instead, oximeter will only know about that by fetching this list.

We should have a specific configuration for integration tests, which makes the interval much smaller, say 1-5 seconds. That's only an optimization, not strictly required.

@bnaecker bnaecker self-assigned this Oct 22, 2024
bnaecker added a commit that referenced this issue Oct 24, 2024
- Remove `oximeter` producer HTTP endpoint for registering individual
  producers.
- Dramatically reduce interval on which `oximeter` collector refreshes
  its list of producers. This is now the only way the collector learns
  of producers. The interval is also much smaller in tests to ensure
  pretty snappy registrations
- Remove calls from both Nexus and the `oximeter` standalone mock Nexus
  for registering producers
- Have `oximeter` collector start polling producers immediately, rather
  than waiting for the first polling interval to expire.
- Closes #6916, #6895, and #6901
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant