Skip to content

statefulset/gitaly: define port names for the internal and the metrics

Carlos Panato requested to merge cpanato/gitlab-charts:add-gitaly-port-name into master

What does this MR do?

when using Prometheus operator and using the CRDS to define which services/pods we want to monitor, it has a field to define the port name for the metrics endpoint instead of the port number. Also, it is a good practice to define the port name to avoid use the port number in other resources, if that change and using the port name we don't need to replace it

Related issues

N/A

Checklist

See Definition of done.

For anything in this list which will not be completed, please provide a reason in the MR discussion.

Required

  • Merge Request Title and Description are up to date, accurate, and descriptive
  • MR targeting the appropriate branch
  • MR has a green pipeline on GitLab.com

Expected (please provide an explanation if not completing)

  • Test plan indicating conditions for success has been posted and passes
  • Documentation created/updated
  • Tests added
  • Integration tests added to GitLab QA
  • Equivalent MR/issue for omnibus-gitlab opened

Merge request reports