Skip to content

Conversation

@dkoshkin
Copy link
Contributor

What problem does this PR solve?:
Updates chart version that has a fix that correctly sets headless service port equal to pod's port.
Usually clients would explicitly set the port directly when connecting to the headless service, but this does fix using kubectl port-forward

Which issue(s) this PR fixes:
Fixes #

How Has This Been Tested?:

$ kubectl port-forward --address=127.0.0.1 --namespace registry-system service/cncf-distribution-registry-docker-registry-headless https
Forwarding from 127.0.0.1:5000 -> 5000
Handling connection for 5000

$ crane copy nginx:stable 127.0.0.1:5000/library/nginx:dkoshkin --insecure                                                                      
2025/06/13 13:51:08 Copying from nginx:stable to 127.0.0.1:5000/library/nginx:dkoshkin
2025/06/13 13:51:11 pushed blob: sha256:6b8146f9616cc4005c1b785be80e6c75d2fc745cdf9d7f12e1aacf0449faada7
...

Special notes for your reviewer:

Updates chart version that has a fix that correctly sets headless service port
equal to pod's port.
@jimmidyson jimmidyson enabled auto-merge (squash) June 16, 2025 09:33
@jimmidyson jimmidyson merged commit e0ec4be into main Jun 16, 2025
24 checks passed
@jimmidyson jimmidyson deleted the dkoshkin/fix-registry-headless-service branch June 16, 2025 09:35
supershal added a commit that referenced this pull request Jun 24, 2025
🤖 I have created a release *beep* *boop*
---


## 0.30.0 (2025-06-24)

<!-- Release notes generated using configuration in .github/release.yaml
at main -->

## What's Changed
### Exciting New Features 🎉
* feat: Build with Go 1.24.4 to fix CVEs by @jimmidyson in
#1157
* feat: add requests and limits to registry containers by @dkoshkin in
#1158
* feat: Add preflight checks framework by @dlipovetsky in
#1129
* feat: Preflight check opt-out by @dlipovetsky in
#1156
* feat: Nutanix VM image preflight check by @dlipovetsky in
#1130
* feat: update addons by @dkoshkin in
#1168
* feat: Enforce MD replicas within cluster autoscaler bounds by
@jimmidyson in
#1169
* feat(preflight): Storage container checks for Nutanix by
@thunderboltsid in
#1136
* feat: update Nutanix CSI to 3.3.4 by @dkoshkin in
#1179
### Fixes 🔧
* fix: update CNCF registry version to 2.3.4, app version 2.8.3 by
@dkoshkin in
#1150
* fix: registry addon headless service port by @dkoshkin in
#1159
* fix: preserve registry addon root CA on move by @dkoshkin in
#1155
* fix: Add noderegistration patch to previous handler by @jimmidyson in
#1177
### Other Changes
* build: include regclient/regsync image for registry addon by @dkoshkin
in
#1148
* test: Add update test helpers by @jimmidyson in
#1162
* test(e2e): Nutanix 1.33.1 testing by @jimmidyson in
#1164
* build: Update all tools by @jimmidyson in
#1165
* refactor: add global feature.Gates variable by @dkoshkin in
#1167
* ci: new env variable to set --feature-gates by @dkoshkin in
#1166
* build: github.com/hashicorp/[email protected] to fix CVE by
@jimmidyson in
#1170
* docs: Update link to default Cilium values in cni.md by
@yannickstruyf3 in
#1173
* docs: Fix up Cilium config link (again) & icons by @jimmidyson in
#1176

## New Contributors
* @yannickstruyf3 made their first contribution in
#1173

**Full Changelog**:
v0.29.0...v0.30.0

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Shalin Patel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants