Skip to content
This repository has been archived by the owner on Oct 20, 2022. It is now read-only.

[Fix/Helm] Operator metrics configuration #156

Merged
merged 2 commits into from
Nov 12, 2021

Conversation

erdrix
Copy link
Contributor

@erdrix erdrix commented Nov 12, 2021

Q A
Bug fix? yes
New feature? no
API breaks? no
Deprecations? no
Related tickets fixes #145
License Apache 2.0

What's in this PR?

It fix the port configuration mistmach for the operator pod in the helm chart and add a way to configure it using : metrics.enabled and metrics.port fields.

Checklist

  • Implementation tested
  • Error handling code meets the guideline
  • Logging code meets the guideline
  • User guide and development docs updated (if needed)
  • Append changelog with changes

@erdrix erdrix added the bug Something isn't working label Nov 12, 2021
@erdrix erdrix added this to the 0.7.3 milestone Nov 12, 2021
@erdrix erdrix self-assigned this Nov 12, 2021
@erdrix erdrix added this to In progress in nifikop via automation Nov 12, 2021
@erdrix erdrix merged commit 4684da5 into Orange-OpenSource:master Nov 12, 2021
nifikop automation moved this from In progress to Done Nov 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
nifikop
  
Done
Development

Successfully merging this pull request may close these issues.

metrics port mismatch
1 participant