Skip to content

Commit

Permalink
chore: change image tag retrieval
Browse files Browse the repository at this point in the history
allow fallback to chat appVersion
  • Loading branch information
evegufy committed Mar 18, 2024
1 parent e8dfdcb commit 49541c9
Show file tree
Hide file tree
Showing 8 changed files with 34 additions and 18 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/chart-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ jobs:
run: ct lint --validate-maintainers=false --check-version-increment=false --target-branch ${{ github.event.repository.default_branch }}

- name: Run service chart-testing (install)
run: ct install --charts charts/ssi-credential-issuer --config charts/chart-testing-config.yaml --helm-extra-set-args "--set=issuer.image=kind-registry:5000/credential-issuer-service:testing --set=issuermigrations.image=kind-registry:5000/credential-issuer-migrations:testing --set=processesworker.image=kind-registry:5000/credential-issuer-processes-worker --set=credentialExpiry.image=kind-registry:5000/credential-expiry-app"
run: ct install --charts charts/ssi-credential-issuer --config charts/chart-testing-config.yaml --helm-extra-set-args "--set=issuer.image.name=kind-registry:5000/credential-issuer-service --set=issuer.image.tag=testing --set=issuermigrations.image.name=kind-registry:5000/credential-issuer-migrations --set=issuermigrations.image.tag=testing --set=processesworker.image.name=kind-registry:5000/credential-issuer-processes-worker --set=processesworker.image.tag=testing --set=credentialExpiry.image.name=kind-registry:5000/credential-expiry-app --set=credentialExpiry.image.tag=testing"
if: github.event_name != 'pull_request' || steps.list-changed.outputs.changed == 'true'

# TODO: re-add the step after the first version release
Expand All @@ -133,5 +133,5 @@ jobs:
# helm repo add tractusx-dev https://eclipse-tractusx.github.io/charts/dev
# helm install ssi-credential-issuer tractusx-dev/ssi-credential-issuer --version ${{ github.event.inputs.upgrade_from || 'tbd' }} --namespace upgrade --create-namespace
# helm dependency update charts/ssi-credential-issuer
# helm upgrade ssi-credential-issuer charts/ssi-credential-issuer --set issuer.image=kind-registry:5000/service:testing --set=issuermigrations.image=kind-registry:5000/migrations:testing --namespace upgrade
# helm upgrade ssi-credential-issuer charts/ssi-credential-issuer --set issuer.image.name=kind-registry:5000/service:testing --set=issuermigrations.image.name=kind-registry:5000/migrations:testing --namespace upgrade
# if: github.event_name != 'pull_request' || steps.list-changed.outputs.changed == 'true'
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ spec:
- ALL
readOnlyRootFilesystem: true
runAsNonRoot: true
image: "{{ .Values.credentialExpiry.image }}"
image: "{{ .Values.credentialExpiry.image.name }}:{{ .Values.credentialExpiry.image.tag | default .Chart.AppVersion }}"
imagePullPolicy: "{{ .Values.credentialExpiry.imagePullPolicy }}"
env:
- name: DOTNET_ENVIRONMENT
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ spec:
- ALL
readOnlyRootFilesystem: true
runAsNonRoot: true
image: "{{ .Values.processesworker.image }}"
image: "{{ .Values.processesworker.image.name }}:{{ .Values.processesworker.image.tag | default .Chart.AppVersion }}"
imagePullPolicy: "{{ .Values.processesworker.imagePullPolicy }}"
env:
- name: DOTNET_ENVIRONMENT
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ spec:
- ALL
readOnlyRootFilesystem: true
runAsNonRoot: true
image: "{{ .Values.issuer.image }}"
image: "{{ .Values.issuer.image.name }}:{{ .Values.issuer.image.tag | default .Chart.AppVersion }}"
imagePullPolicy: "{{ .Values.issuer.imagePullPolicy }}"
env:
- name: DOTNET_ENVIRONMENT
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ spec:
- ALL
readOnlyRootFilesystem: true
runAsNonRoot: true
image: "{{ .Values.issuermigrations.image }}"
image: "{{ .Values.issuermigrations.image.name }}:{{ .Values.issuermigrations.image.tag | default .Chart.AppVersion }}"
imagePullPolicy: "{{ .Values.issuermigrations.imagePullPolicy }}"
env:
- name: DOTNET_ENVIRONMENT
Expand Down
16 changes: 12 additions & 4 deletions charts/ssi-credential-issuer/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,9 @@
###############################################################

issuer:
image: "docker.io/tractusx/credential-issuer-service:0.1.0-rc.1"
image:

Check failure on line 21 in charts/ssi-credential-issuer/values.yaml

View workflow job for this annotation

GitHub Actions / lint-test

21:9 [trailing-spaces] trailing spaces
name: "docker.io/tractusx/credential-issuer-service"
tag: ""
imagePullPolicy: "IfNotPresent"
# -- We recommend not to specify default resource limits and to leave this as a conscious choice for the user.
# If you do want to specify resource limits, uncomment the following lines and adjust them as necessary.
Expand Down Expand Up @@ -64,7 +66,9 @@ issuer:

issuermigrations:
name: "migrations"
image: "docker.io/tractusx/credential-issuer-migrations:0.1.0-rc.1"
image:

Check failure on line 69 in charts/ssi-credential-issuer/values.yaml

View workflow job for this annotation

GitHub Actions / lint-test

69:9 [trailing-spaces] trailing spaces
name: "docker.io/tractusx/credential-issuer-migrations"
tag: ""
imagePullPolicy: "IfNotPresent"
# -- We recommend not to specify default resource limits and to leave this as a conscious choice for the user.
# If you do want to specify resource limits, uncomment the following lines and adjust them as necessary.
Expand All @@ -83,7 +87,9 @@ issuermigrations:

processesworker:
name: "processesworker"
image: "docker.io/tractusx/credential-issuer-processes-worker:0.1.0-rc.1"
image:

Check failure on line 90 in charts/ssi-credential-issuer/values.yaml

View workflow job for this annotation

GitHub Actions / lint-test

90:9 [trailing-spaces] trailing spaces
name: "docker.io/tractusx/credential-issuer-processes-worker"
tag: ""
imagePullPolicy: "IfNotPresent"
# -- We recommend not to specify default resource limits and to leave this as a conscious choice for the user.
# If you do want to specify resource limits, uncomment the following lines and adjust them as necessary.
Expand Down Expand Up @@ -124,7 +130,9 @@ processesworker:

credentialExpiry:
name: "expiry"
image: "docker.io/tractusx/credential-expiry-app:0.1.0-rc.1"
image:

Check failure on line 133 in charts/ssi-credential-issuer/values.yaml

View workflow job for this annotation

GitHub Actions / lint-test

133:9 [trailing-spaces] trailing spaces
name: "docker.io/tractusx/credential-expiry-app"
tag: ""
imagePullPolicy: "IfNotPresent"
# -- We recommend not to specify default resource limits and to leave this as a conscious choice for the user.
# If you do want to specify resource limits, uncomment the following lines and adjust them as necessary.
Expand Down
12 changes: 8 additions & 4 deletions consortia/environments/values-dev.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,24 +38,28 @@ ingress:
port: 8080

issuer:
image: "docker.io/tractusx/ssi-credential-issuer-service:dev"
image:
tag: "dev"
imagePullPolicy: "Always"
swaggerEnabled: true

issuermigrations:
image: "docker.io/tractusx/ssi-credential-issuer-migrations:dev"
image:
tag: "dev"
imagePullPolicy: "Always"
logging:
default: "Debug"

processesworker:
image: "docker.io/tractusx/ssi-credential-issuer-processes-worker:dev"
image:
tag: "dev"
imagePullPolicy: "Always"
logging:
default: "Debug"

credentialexpiry:
image: "docker.io/tractusx/ssi-credential-expiry-app:dev"
image:
tag: "dev"
imagePullPolicy: "Always"
logging:
default: "Debug"
Expand Down
12 changes: 8 additions & 4 deletions consortia/environments/values-rc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,24 +38,28 @@ ingress:
port: 8080

issuer:
image: "docker.io/tractusx/ssi-credential-issuer-service:rc"
image:
tag: "rc"
imagePullPolicy: "Always"
swaggerEnabled: true

issuermigrations:
image: "docker.io/tractusx/ssi-credential-issuer-migrations:rc"
image:
tag: "rc"
imagePullPolicy: "Always"
logging:
default: "Debug"

processesworker:
image: "docker.io/tractusx/ssi-credential-issuer-processes-worker:rc"
image:
tag: "rc"
imagePullPolicy: "Always"
logging:
default: "Debug"

credentialexpiry:
image: "docker.io/tractusx/ssi-credential-expiry-app:rc"
image:
tag: "rc"
imagePullPolicy: "Always"
logging:
default: "Debug"
Expand Down

0 comments on commit 49541c9

Please sign in to comment.