Skip to content

Conversation

im-jinxinwang
Copy link
Contributor

Fix issue [259] Support set different image registry server

jinxinwang and others added 2 commits November 14, 2023 10:20
Signed-off-by: jinxinwang <g1561925068@163.com>
A modified version
@LinuxSuRen LinuxSuRen added enhancement New feature or request helm labels Nov 14, 2023
@LinuxSuRen LinuxSuRen linked an issue Nov 14, 2023 that may be closed by this pull request
Copy link
Owner

@LinuxSuRen LinuxSuRen left a comment

Choose a reason for hiding this comment

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

Thanks for your effort. I left some comments, please take a look at it.

Comment on lines 25 to 30
dependencies:
- name: common
repository: oci://registry-1.docker.io/bitnamicharts
tags:
- bitnami-common
version: 2.x.x
Copy link
Owner

Choose a reason for hiding this comment

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

What is reason to add this dependency?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There are a large number of public libraries in bitnami-common, which can be used to form a specification

Copy link
Owner

Choose a reason for hiding this comment

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

Got it. But how about using it when this chart is completed enough. In my view, it's pretty simple. I didn't find a solid reason to use a third-party chart.

@@ -5,10 +5,16 @@
replicaCount: 1

image:
registry: docker.io
Copy link
Owner

Choose a reason for hiding this comment

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

I prefer to use linuxsuren.docker.scarf.sh as the default registry. It is a mirror of docker.io. Because it could collect more information about the pulling data.

repository: linuxsuren/api-testing
pullPolicy: IfNotPresent
tag: "v0.0.2-helm"
Copy link
Owner

Choose a reason for hiding this comment

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

v0.0.2-helm is the tag of the helm image instead of the application. I perfer to use the chart appVersion as the default version instead of a fixed version.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The fixed version is indeed not dynamic enough, but it is not user-friendly because after I download the chart, I still need to look at the code to know which image tag I am using.

Copy link
Owner

Choose a reason for hiding this comment

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

But the tag v0.0.2-helm is incorrect.

Signed-off-by: jinxinwang <g1561925068@163.com>
Comment on lines 1 to 6
dependencies:
- name: common
repository: oci://registry-1.docker.io/bitnamicharts
version: 2.13.3
digest: sha256:9a971689db0c66ea95ac2e911c05014c2b96c6077c991131ff84f2982f88fb83
generated: "2023-11-14T09:58:21.488814356+08:00"
Copy link
Owner

Choose a reason for hiding this comment

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

Forgot to delete?

jinxinwang added 4 commits November 14, 2023 14:31
Signed-off-by: jinxinwang <g1561925068@163.com>
Signed-off-by: jinxinwang <g1561925068@163.com>
Signed-off-by: jinxinwang <g1561925068@163.com>
Signed-off-by: jinxinwang <g1561925068@163.com>
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Owner

@LinuxSuRen LinuxSuRen left a comment

Choose a reason for hiding this comment

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

LGTM

@LinuxSuRen LinuxSuRen merged commit f31cd40 into LinuxSuRen:master Nov 14, 2023
@LinuxSuRen LinuxSuRen changed the title Fix issue [259] Support set different image registry server feat: Support set different image registry server Nov 29, 2023
LinuxSuRen pushed a commit that referenced this pull request Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request helm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support set different image registry server
2 participants