Skip to content

Commit

Permalink
fix(uplink): use tag instead of sha256 for the Docker image (#903)
Browse files Browse the repository at this point in the history
* chore(updatecli): don't hide `uplink` Docker image tag

* don't trim sha256 prefix

* add current pinned tag to uplink chart values

* bump uplink chart version

* fix: use only named tag, no sha256 preventing the deployment of other architecture images

* fix: remove sha256 from the repo

* bump chart version

* docker image tag with amd64 and arm64
  • Loading branch information
lemeurherve authored Oct 31, 2023
1 parent 773d4f5 commit 58f5f2d
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
2 changes: 1 addition & 1 deletion charts/uplink/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v1
appVersion: "1.0"
description: A Helm chart for uplink.jenkins.io
name: uplink
version: 0.3.2
version: 0.3.3
maintainers:
- name: olblak
- name: dduportal
Expand Down
4 changes: 2 additions & 2 deletions charts/uplink/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@
# Declare variables to be passed into your templates.
replicaCount: 1
image:
repository: jenkinsciinfra/uplink@sha256
tag: 88605c3cfcb9018b270c340d8426955f13aa72708c01a25c75f99f8710af3729
repository: jenkinsciinfra/uplink
tag: 0.1.0
pullPolicy: IfNotPresent
imagePullSecrets: []
nameOverride: ""
Expand Down
3 changes: 0 additions & 3 deletions updatecli/updatecli.d/uplink.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,6 @@ sources:
image: "jenkinsciinfra/uplink"
tag: "latest"
architecture: "amd64"
hidetag: true
transformers:
- trimprefix: '@sha256:'

# no condition to test docker image availability as we're using a digest from docker hub

Expand Down

0 comments on commit 58f5f2d

Please sign in to comment.