Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(thanos) labelselector specify http port name #627

Merged
merged 6 commits into from
Feb 5, 2025
Merged

fix(thanos) labelselector specify http port name #627

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
5a5be0c
fix svc label selector
trouaux Feb 5, 2025
87a76df
bump version
trouaux Feb 5, 2025
d59c3fe
unnecessary label
trouaux Feb 5, 2025
8b0bd44
specify port name to expose
trouaux Feb 5, 2025
59aae46
Merge branch 'main' into fix-thanos-ruler-svc-labelselector
trouaux Feb 5, 2025
ef1e7f8
bump version
trouaux Feb 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion thanos/charts/Chart.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ maintainers:
name: thanos
sources:
- https://github.com/cloudoperators/greenhouse-extensions
version: 0.1.8
version: 0.1.9
# thanos-release
appVersion: v0.35.0
keywords:
Expand Down
5 changes: 3 additions & 2 deletions thanos/plugindefinition.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ spec:
helmChart:
name: thanos
repository: "oci://ghcr.io/cloudoperators/greenhouse-extensions/charts"
version: 0.1.8
version: 0.1.9
options:
- default: null
description: CLI param for Thanos Query
Expand Down Expand Up @@ -101,6 +101,7 @@ spec:
type: string
- default:
greenhouse.sap/expose: "true"
greenhouse.sap/exposeNamedPort: http
description: Needed for exposing Thanos Ruler to the service proxy. UI is accessible via http, exposing this particular port.
name: thanos.ruler.serviceLabels
required: false
Expand All @@ -118,4 +119,4 @@ spec:
description: TLS key for communication with Alertmanager
required: false
type: secret
version: 0.2.8
version: 0.2.9
Loading