-
Notifications
You must be signed in to change notification settings - Fork 20
clarify call-home service and requirements #564
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
clarify call-home service and requirements #564
Conversation
src/server/administer/index.md
Outdated
| It is legally required by <EnterprisePlatformName /> servers to send usage information to monitor the usage of the License. | ||
| It is legally required by <EnterprisePlatformName /> servers to send usage information (`call-home` service) to monitor the usage of the License. | ||
|
|
||
| This service runs on background via `TCP` port `443`, hence it needs to be enabled on firewall side in order to comply with <EnterprisePlatformName /> terms. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not be part of the warning - warning is about legal bit - port is description how to set it up
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In terms of the technical requirement, it should say that it requires outgoing https access to https://api.merginmaps.com/monitoring on TCP port 443
I think this page has some wider issues though - it looks like it's written as a call to action to disable monitoring (which we don't want). I would instead recommend:
- changing the title so it describes monitoring, not how to disable it
- update the example to show monitoring enabled
- detail the technical requirements above
- say what it's for (making mm better and required for monitoring of enterprise edition licence usage)
- describe how to disable it if desired in the community edition while noting that this is now allowed in EE
src/server/administer/index.md
Outdated
| It is legally required by <EnterprisePlatformName /> servers to send usage information to monitor the usage of the License. | ||
| It is legally required by <EnterprisePlatformName /> servers to send usage information (`call-home` service) to monitor the usage of the License. | ||
|
|
||
| This service runs on background via `TCP` port `443`, hence it needs to be enabled on firewall side in order to comply with <EnterprisePlatformName /> terms. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In terms of the technical requirement, it should say that it requires outgoing https access to https://api.merginmaps.com/monitoring on TCP port 443
I think this page has some wider issues though - it looks like it's written as a call to action to disable monitoring (which we don't want). I would instead recommend:
- changing the title so it describes monitoring, not how to disable it
- update the example to show monitoring enabled
- detail the technical requirements above
- say what it's for (making mm better and required for monitoring of enterprise edition licence usage)
- describe how to disable it if desired in the community edition while noting that this is now allowed in EE
mostlyAtNight
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks much better - thank you.
One small change requested to make things clearer by removing the double-negative
|
Hey guys. Just wondering about this page https://merginmaps.com/docs/server/administer/environment/ and COLLECT_STATISTICS variable description. Could we add here some link to this Telemetry section or some short explanation of licensing related to 🔝 comments? @mostlyAtNight @fernandinand |
Sounds like a good idea - it would be good if everything reflected our recent discussions |
…thub.com:MerginMaps/docs into 548-adding-callback-network-requirements-to-docs
I added some info and link to the Telemetry section page @MarcelGeo |
alex-cit
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have left two comment to consider :)
| <MainPlatformName /> telemetry service helps us gather some information about the usage of the platform, namely on-premise deployment. This information is very valuable to us to help us improve <MainPlatformName />. | ||
| Please consider enabling this functionality, even if you are using <CommunityPlatformName /> edition. For <EnterprisePlatformName /> deployments, as you'll check down below, this is <b>mandatory</b>. | ||
|
|
||
| The <MainPlatformName /> telemetry service (`call-home`) requires https access to `https://api.merginmaps.com/monitoring` on `TCP` port `443`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this service has stable IP address so when client want's to whitelists the api?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's finish the discussion on Slack and create a new PR if needed
PR for #548