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

build: set helmRepository tag in list-images target #968

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

dkoshkin
Copy link
Contributor

@dkoshkin dkoshkin commented Nov 7, 2024

What problem does this PR solve?:
Noticed in the last release this image tag was set to v0.0.0-dev, fixing it to get the release version.

$ make list-images
...
ghcr.io/nutanix-cloud-native/caren-helm-reg:v0.22.1

Which issue(s) this PR fixes:
Fixes #

How Has This Been Tested?:

Special notes for your reviewer:

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.

Oops thanks!

@dkoshkin dkoshkin merged commit 52f106a into main Nov 7, 2024
25 checks passed
@dkoshkin dkoshkin deleted the dkoshkin/build-set-helmRepository-image-tag branch November 7, 2024 18:31
jimmidyson pushed a commit that referenced this pull request Nov 13, 2024
**What problem does this PR solve?**:
While looking at
#968
I noticed the image name.
The `caren-helm-reg` image is no longer used as a registry. The word
registry was also recently changed to repository in the templates.

Renamed it to `cluster-api-runtime-extensions-bundle-initializer`, but
open to suggestions!

**Which issue(s) this PR fixes**:
Fixes #

**How Has This Been Tested?**:
<!--
Please describe the tests that you ran to verify your changes.
Provide output from the tests and any manual steps needed to replicate
the tests.
-->

**Special notes for your reviewer**:
<!--
Use this to provide any additional information to the reviewers.
This may include:
- Best way to review the PR.
- Where the author wants the most review attention on.
- etc.
-->
jimmidyson added a commit that referenced this pull request Nov 13, 2024
🤖 I have created a release *beep* *boop*
---


## 0.23.0 (2024-11-13)

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

## What's Changed
### Exciting New Features 🎉
* feat: Build with go 1.23.3 and upgrade all tools by @jimmidyson in
#967
### Fixes 🔧
* fix: Update mindthegap to fix cert rotation by @dlipovetsky in
#971
* fix: image registries with no credentials but with a CA by @dkoshkin
in
#927
### Other Changes
* build: set helmRepository tag in list-images target by @dkoshkin in
#968
* build: rename caren-helm-reg to better match role by @dkoshkin in
#969


**Full Changelog**:
v0.22.0...v0.23.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>
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.

3 participants