Use the full PWB version docker tag

[cm421]

The tag for PWB image rstudio/rstudio-workbench:ubuntu2204-2024.04.0 doesn't exist on docker hub for some reason, which causes problems because that's the tag used in PWB's helm chart.

While platform team investigates why that tag wasn't created, we can use rstudio/rstudio-workbench:ubuntu2204-024.04.0-735.pro3 instead

[kfeinauer]

@cm421 Need to bump the chart version before merge.

[tnederlof]

I think those annotations are only for artifacthub.io, the values in the values.yaml file under image and session.image are what drive the images pulled by default. The chart is set up to take the app version + tagPrefix and pull automatically, to override we would need to set the following two tags with ubuntu2204-2024.04.0-735.pro3:

image.tag

helm/charts/rstudio-workbench/values.yaml

Line 141 in 90bc753

tag: ""

session.image.tag

helm/charts/rstudio-workbench/values.yaml

Line 29 in 90bc753

tag: ""

[cm421]

So changing appVersion: 2024.04.0 to appVersion: 2024.04.0+735.pro3 in charts/rstudio-workbench/Chart.yaml should set the default tag as well?

Anyone with more helm experience know if changing that will affect other areas as well?

[tnederlof]

So changing appVersion: 2024.04.0 to appVersion: 2024.04.0+735.pro3 in charts/rstudio-workbench/Chart.yaml should set the default tag as well?

Anyone with more helm experience know if changing that will affect other areas as well?

It would apply to the image, but it looks like the variable is used in a couple of spots. Since all the helm charts are consistent across products in this area, it would be nice to leave it as is if the image can be corrected on Monday.