Description of problem: ovirt_env_name - must be a valid OpenShift namespace identifier - alphanum + "-" character, cannot begin with "-" or a number, and cannot end with a “-”, up to 49 characters. The length is opentshift limitation for the total text length. Total length should be up to 63 characters. "ovirt-metrics-" prefix is 14 characters. We should add a validation that checks ovirt_env_name length and fails the ansible script if name does not follow the above limitation
Also you should note somewhere (comment in config.yml) about the prefix, without it people wont be aware there is any, except for logs in metrics, and would have to change namespaces.
The user does not need to add a prefix. We add it. I will add a comment with the prefixes that we add.
Lukas, Please review the PR: https://github.com/oVirt/ovirt-site/pull/1097/files
Commented inside PR. Few more comments: - inside the document you are have diferent z-stream versions, I would prefer just to use y-stream version 4.1 so we wont have to change it. - fqdn/of/engine/machine -> should rather be fqdn-of-engine-machine - its a hostname not path - In the end "It should finish without errors." I would also mention "collectd and fluentd services should be running", this does not make user to check it however in case of issue they are able to debug it.
verified in ovirt-engine-metrics-1.0.6-1.el7ev.noarch
> ovirt_env_name must be a valid OpenShift namespace identifier in order for > the records to be kept to the elasticsearch that is running in OpenShift. Hi Shirly, looking at this again before we publish the Release Notes book, I'm not quite sure what the technical meaning is meant to be here (I'm not familiar with Elasticsearch, but looking it up suggests it searches rather than stores - I may be wrong). Does this mean: * The name must be valid in order for the records to be kept by the Elasticsearch running on OpenShift"? * "The name must be valid in order for metrics to be gathered by the Elasticsearch running on Openshift"? * Something else?
(In reply to Tahlia Richardson from comment #6) > > ovirt_env_name must be a valid OpenShift namespace identifier in order for > > the records to be kept to the elasticsearch that is running in OpenShift. > > Hi Shirly, looking at this again before we publish the Release Notes book, > I'm not quite sure what the technical meaning is meant to be here (I'm not > familiar with Elasticsearch, but looking it up suggests it searches rather > than stores - I may be wrong). > > Does this mean: > * The name must be valid in order for the records to be kept by the > Elasticsearch running on OpenShift"? That is correct. > * "The name must be valid in order for metrics to be gathered by the > Elasticsearch running on Openshift"? > * Something else?