Skip to content

resolves "does not have a named port 'https' error#888

Merged
RafalKorepta merged 1 commit intomainfrom
jbarnes/https-fix
Jun 10, 2025
Merged

resolves "does not have a named port 'https' error#888
RafalKorepta merged 1 commit intomainfrom
jbarnes/https-fix

Conversation

@RafalKorepta
Copy link
Contributor

Fixes error for missing named port https

kubectl --namespace redpanda port-forward svc/redpanda-controller-operator-metrics-service 8443:8443
error: Pod 'redpanda-controller-operator-c897f85f9-sprxh' does not have a named port 'https'

@@ -0,0 +1,4 @@
project: operator
kind: Fixed
body: Operator chart had missing 'https' named port in operator container ports list
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
body: Operator chart had missing 'https' named port in operator container ports list
body: Added the missing `https` port to the operator Pod that was referenced by the [`ServiceMonitor`](https://github.com/redpanda-data/redpanda-operator/blob/4e34c5ea79b00fa0caeda64955e3291666194274/operator/chart/servicemonitor.go#L42)

@RafalKorepta RafalKorepta enabled auto-merge (rebase) June 10, 2025 08:26
Fixes error for missing named port https

```
kubectl --namespace redpanda port-forward svc/redpanda-controller-operator-metrics-service 8443:8443
error: Pod 'redpanda-controller-operator-c897f85f9-sprxh' does not have a named port 'https'
```
@RafalKorepta RafalKorepta merged commit 59f6af7 into main Jun 10, 2025
10 checks passed
@RafalKorepta RafalKorepta deleted the jbarnes/https-fix branch June 10, 2025 10:29
@david-yu
Copy link
Contributor

Does this perhaps need a backport?

@RafalKorepta
Copy link
Contributor Author

@david-yu Yes, I will create backports today.

@RafalKorepta
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
release/v25.1.x
release/v2.4.x
release/v2.3.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants