Make test pods optional and allow image override #360
No reviewers
Labels
No Label
has
backport
in progress
invalid
kind
breaking
kind
bug
kind
build
kind
dependency
kind
deployment
kind
docs
kind
enhancement
kind
feature
kind
lint
kind
proposal
kind
question
kind
refactor
kind
security
kind
testing
kind
translation
kind
ui
need
backport
priority
critical
priority
low
priority
maybe
priority
medium
reviewed
duplicate
reviewed
invalid
reviewed
wontfix
skip-changelog
status
blocked
status
needs-feedback
status
needs-reviews
status
wip
upstream
gitea
upstream
other
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: gitea/helm-chart#360
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "ooms97/helm-chart:main"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Description of the change
Make the test-connection Pod optional and override the wget container's image.
Benefits
Allows users to enable/disabled the test-connection Pod and override the wget container's image.
Possible drawbacks
Checklist
values.yaml
and added to theREADME.md
using readme-generator-for-helmREADME.md
Make test pods optional and allow image override (#1)to Make test pods optional and allow image override@ -473,2 +473,4 @@
## @param checkDeprecation Set it to false to skip this basic validation check.
checkDeprecation: true
## @section Test
A separate section is not necessary, IMO. Modifying the helm tests can be considered "advanced". ?
Great point, changes have been added! ?
@ -10,3 +11,3 @@
containers:
- name: wget
image: busybox
image: {{ .Values.test.image | default "busybox" }}
Should we split this in
image.name
andimage.tag
to follow canonical conventions?LGTM
@justusbunsi Should we get that in if LGTM to you?
LGTM. Thanks for your contribution.