-
Notifications
You must be signed in to change notification settings - Fork 49
Document the reasons why servicecontrol.throughput queue exists #4992
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
Merged
Merged
Changes from 1 commit
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| From version 5.4.0 we have introduce a new feature to collect usage data as part of the Platform. | ||
|
|
||
| Usage data is collected from 3 different sources: | ||
| - Audited messages | ||
| - Monitoring metrics | ||
|
johnsimons marked this conversation as resolved.
Outdated
|
||
| - Directly from the broker | ||
|
|
||
| The error instance is the orchestrator for all the collection from all these different endpoints. | ||
|
johnsimons marked this conversation as resolved.
Outdated
|
||
|
|
||
| Audited messages usage is collected by querying the audit instances every so often. | ||
| Broker collection is also collected by querying the borker directly every 24 hours. | ||
| The monitoring metrics collection is collected by the metrics intance sending a direct message to a well known queue that by default is named "servicecontrol.throughput". | ||
|
johnsimons marked this conversation as resolved.
Outdated
|
||
|
|
||
|
|
||
| ### Why is the "servicecontrol.throughput" queue hardcoded? | ||
| The TF that initially created this new feature decided that by hardcoding the queue name, it would be simpler for the customers to get started because they don't need to configure anything up front. | ||
| This was decided based on simplicity of upgrade vs maintanability. | ||
|
|
||
| ### Why is the "servicecontrol.throughput" not a sub queue of the error instance? | ||
| Because this would mean customer would have to configure the sending endpoint (monitoring instance) with the name of this queue, which we wanted to avoid. | ||
|
johnsimons marked this conversation as resolved.
Outdated
|
||
|
|
||
| #### But we have SCMU, can't that make sure the queue names match? | ||
| SCMU is a Windows only tool, as well as, this would only work if they are installing all instances on the same machine. | ||
| Same reasons why, we do not configure the remote audit instances. | ||
|
|
||
| ### Can the "servicecontrol.throughput" queue be renamed? | ||
| Yes, see https://docs.particular.net/servicecontrol/monitoring-instances/configuration#usage-reporting | ||
|
johnsimons marked this conversation as resolved.
Outdated
|
||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.