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.

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: v2/ in Containerd mirror path #363

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

dkoshkin
Copy link
Contributor

Found a bug(?) in how Containerd handles mirrors with a path and it incorrectly appends /v2 to the end of mirror, and not between the provided host and path.

See containerd/containerd#6634 (comment) for a longer explanation and example output.

@dkoshkin dkoshkin requested review from jimmidyson and faiq February 15, 2024 01:01
@dkoshkin dkoshkin self-assigned this Feb 15, 2024
@github-actions github-actions bot added the fix label Feb 15, 2024
Copy link
Member

@jimmidyson jimmidyson left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the detailed explanation.

@jimmidyson jimmidyson merged commit cbd2fb5 into main Feb 15, 2024
16 checks passed
@jimmidyson jimmidyson deleted the dkoshkin/fix-containerd-mirror-path branch February 15, 2024 13:48
@github-actions github-actions bot mentioned this pull request Feb 15, 2024
dkoshkin pushed a commit that referenced this pull request Feb 15, 2024
🤖 I have created a release *beep* *boop*
---


## 0.4.2 (2024-02-15)

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

## What's Changed
### Fixes 🔧
* fix: remove word konvoy/ and use cre/ by @dkoshkin in
#365
* fix: v2/ in Containerd mirror path by @dkoshkin in
#363
* fix: set config_path in Containerd config by @dkoshkin in
#364
* fix: generate config with only globalImageRegistryMirror set by
@dkoshkin in
#362
### Other Changes
* refactor: Fix formatting issue by @jimmidyson in
#368
* build: Include CAPX APis by @jimmidyson in
#367
* build: Upgrade golangci-lint to 1.56.1 by @jimmidyson in
#369
* docs: Update examples to be clusterctl templates by @jimmidyson in
#361


**Full Changelog**:
v0.4.1...v0.4.2

---
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>
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.

2 participants