Repository navigation
feat(portworx): add Portworx StackPack - #35
Open
abonillabeeche wants to merge 1 commit into
Open
abonillabeeche wants to merge 1 commit into
abonillabeeche wants to merge 1 commit into
Conversation
Adds a StackPack for Portworx Enterprise 3.x on Kubernetes. Metrics come from a ServiceMonitor the agent's OpenTelemetry scraper picks up; topology comes from the StorageCluster and StorageNode custom resources. Portworx nodes and volumes merge into the Kubernetes node and persistent volume components. Includes presentations, 20 monitors ported from the Portworx operator's alert rules plus pool and volume health checks, and three dashboards.
Contributor
|
@abonillabeeche Thanks for contributing. We are still getting used to the stackpacks flow and have to figure out in what way this will land in the product, who will review. Merging this as is would put it on equal footing with our current stackpacks, but prooably we want to first mark this experimental and list this under 3rd party technologies, we'll keep you posted. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a
portworxStackPack for Portworx Enterprise 3.x on Kubernetes.Data path. No Prometheus server is needed:
ServiceMonitorlabelledobservability.suse.com/agent: scrapeon the Portworx service (px-api,:9001/metrics).StorageClusterandStorageNodeCRs (core.libopenstorage.org).Topology.
portworx clustercomponent perStorageCluster.StorageNodemerges into its Kubernetes node, throughadditionalIdentifiersand ranked below the Kubernetes source, so the node keeps typenode.px_volume_replication_status, the only volume metric reported for every volume.member ofcluster, volumeprovided bycluster.Presentations. A Portworx main menu group with Clusters, Storage nodes and Volumes lists, plus a Portworx metrics tab on each of the three component kinds:
Monitors. 20 monitors, each with a remediation hint that uses
pxctlcommands checked against 3.7.1. Most port the Portworx operator'sPrometheusRulewith Portworx's thresholds:Dashboards. Cluster overview, Volumes and Nodes and pools, modelled on the Portworx Grafana dashboards.
Setup (in
resources/configuration.md):exportMetrics: trueon the StorageCluster.so-agent-portworxServiceMonitor. Itslabelkeepdrops the PVC labels Portworx copies onto every volume series, which would otherwise create many series.core.libopenstorage.orgtootel.k8sResourceCollector.crDiscovery.apiGroups.include.Testing
Tested on Portworx 3.7.1 (operator, internal KVDB) on a 3-node RKE2 / SUSE Virtualization cluster, with agent chart 1.6.12:
stack-pack-validatorfrom the CI image passes (33 nodes).sts stackpack test-deploy:Notes:
px_volume_reads_total/px_volume_writes_totalcarry onlyvolumeid, so per-volume IOPS use the*_iopsgauges.Coverage was checked against Portworx's alert reference (https://docs.portworx.com/portworx-enterprise/reference/portworx-alerts) and its Grafana dashboards. Stork, Autopilot and KVDB/etcd metrics (
:9091,:9628,:9019) need extra scrapes and are left for a follow-up.