fix(CSI-241): conflict in metrics between node and controller

[sergeyberezansky]

TL;DR

Updated metrics configuration for CSI WekaFS Plugin to support separate ports for different components.

What changed?

• Modified the prometheus.io/port annotation in controllerserver-statefulset.yaml to include separate ports for controller, provisioner, resizer, and snapshotter.
• Updated the metrics port configuration in both controllerserver-statefulset.yaml and nodeserver-daemonset.yaml to use component-specific ports.
• Added new metrics port configurations in values.yaml:
  • controllerPort: 9090
  • provisionerPort: 9091
  • resizerPort: 9092
  • snapshotterPort: 9093
  • nodePort: 9094

How to test?

1. Deploy the CSI WekaFS Plugin with metrics enabled.
2. Verify that the prometheus.io/port annotation in the controller pod includes all specified ports.
3. Check that the controller and node server pods are exposing metrics on their respective ports.
4. Attempt to scrape metrics from each component using the newly configured ports.

Why make this change?

This change allows for more granular control over metrics collection for different components of the CSI WekaFS Plugin. By using separate ports for each component, it becomes easier to monitor and troubleshoot specific parts of the plugin, enhancing observability and debugging capabilities.

Additional idea behind this change is avoiding port conflicts when running CSI plugin in hostNetwork mode (required for NFS)

---

[sergeyberezansky]

• chore(deps): update sidecar versions #326
• docs(CSI-254): update official docs link in Helm templates and README #323
• feat(CSI-252): implement kubelet PVC stats #322
• refactor(CSI-250): do not maintain redundant active mounts from node server after publishing volume #320
• feat(CSI-247): implement InterfaceGroup.GetRandomIpAddress() #319
• feat(CSI-249): optimize NFS mounter to use multiple targets #318
• fix(CSI-241): fix unmountWithOptions to use map key rather than options.String() #317
• feat(CSI-245): allow specifying client group for NFS #316
• chore(deps): update packages to latest versions and Go to 1.22.5 #314
• chore(deps): combine chmod with ADD in Dockerfile #313
• feat(CSI-239): moveToTrash does not return error to upper layers #312
• fix(CSI-243): service accounts for CSI plugin assume ImagePullSecret and cause error messages. #311
• feat(CSI-244): match subnets if existing in client rule #315
• fix(CSI-241): conflict in metrics between node and controller #325 👈
• feat(CSI-213): support NFS transport #299
• fix(CSI-241): disregard sync_on_close in mountmap per FS #310
• feat(CSI-253): support custom CA certificate in API secret #324
• dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sergeyberezansky and the rest of your teammates on Graphite

[graphite-app]

Graphite Automations

"Request reviewers once CI passes" took an action on this PR • (09/11/24)

1 reviewer was added to this PR based on Sergey Berezansky's automation.

[sergeyberezansky]

Merge activity

• Sep 12, 6:10 AM EDT: @sergeyberezansky started a stack merge that includes this pull request via Graphite.
• Sep 12, 6:36 AM EDT: Graphite rebased this pull request as part of a merge.
• Sep 12, 6:39 AM EDT: Graphite rebased this pull request as part of a merge.
• Sep 12, 6:54 AM EDT: @sergeyberezansky merged this pull request with Graphite.