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.

Sign up for GitHub

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

Support for non-root containers - cat file.sh | sh #35

Closed
wants to merge 9 commits into from
Closed

Support for non-root containers - cat file.sh | sh #35

Changes from 4 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
205018b
Update job.yaml
hibohra1398 Jul 19, 2024
0b1e92d
Apply suggestions from code review
hibohra1398 Jul 19, 2024
b93ec71
Update job.yaml
hibohra1398 Jul 22, 2024
511a68a
Merge branch 'main' into patch-1
abeytom07 Aug 8, 2024
f521985
Update run.sh
hibohra1398 Aug 8, 2024
1c80754
Update Dockerfile
hibohra1398 Aug 8, 2024
b03fdcb
Update job.yaml
hibohra1398 Aug 8, 2024
5dd6968
Merge branch 'main' into patch-1
hibohra1398 Sep 5, 2024
5d4cd40
Merge branch 'main' into patch-1
ravisingal Nov 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions kstreams-app-version-checker/helm/templates/job.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ spec:
- name: kstreams-app-version-checker
image: {{ .Values.image.repository }}:{{ default .Chart.AppVersion .Values.image.tagOverride }}
imagePullPolicy: {{ .Values.image.pullPolicy }}
command: ["/bin/sh", "-ce", "cat /tmp/updated_workloads.sh | /bin/sh"]
Copy link
Contributor

Choose a reason for hiding this comment

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

instead make this change in Dockerfile itself.

env:
- name: NAMESPACE
valueFrom:
Expand Down
Loading