search

sensu / catalog

Monitoring as code for Sensu Go. "There's a template for that!"
8 stars 4 forks source link

[HTTP Endpoint Monitoring (Remote)] Incorrect check and entity names in readme #259

Open hillaryfraley opened 2 years ago

hillaryfraley commented 2 years ago

The check listed as http-endpoint-monitoring in the readme is in fact named [entity-name]-http-endpoint-healthcheck in the output (code link). I'm not sure which name is more instructive to users, but the name listed in the readme and the name of the check once created should match.

The entity name listed in the readme is also out of sync with the actual name of the entity once created. The readme says it will create an http-endpoint entity, but the entity name seems to be generated based on whatever URL you enter in the HTTP Endpoint Host field in the first page of the config wizard.

jspaleta commented 2 years ago

This is tough... the integration templating logic is allowed to change the names of the base resources.
There's no way to statically document the final resource names. All you can statically document are the starting resource names prior to the wizard templating.