The auto-generated helm charts for release used wrong image tag #4137
Assignees
Labels
kind/bug
Categorizes issue or PR as related to a bug.
Comments
antoninbas
added a commit
to antoninbas/antrea
that referenced
this issue
Aug 18, 2022
When installing a released version of a chart (for Antrea or the FlowAggregator), the image tag was not generated correctly. It was missing the leading "v". Fixes antrea-io#4137 Signed-off-by: Antonin Bas <abas@vmware.com>
antoninbas
added a commit
that referenced
this issue
Aug 19, 2022
When installing a released version of a chart (for Antrea or the FlowAggregator), the image tag was not generated correctly. It was missing the leading "v". Fixes #4137 Signed-off-by: Antonin Bas <abas@vmware.com>
yanjunz97
added a commit
to antrea-io/theia
that referenced
this issue
Aug 19, 2022
This PR is inspired by antrea-io/antrea#4138 and fixes the issue antrea-io/antrea#4137. It adds the missing leading `v` for ClickHouse monitor image tag in the released Helm chart. It also fixes the wrong ClickHouse monitor image tag when generating `flow-visibiltiy.yaml` for release. Signed-off-by: Yanjun Zhou <zhouya@vmware.com>
antoninbas
added a commit
to antoninbas/antrea
that referenced
this issue
Aug 20, 2022
When installing a released version of a chart (for Antrea or the FlowAggregator), the image tag was not generated correctly. It was missing the leading "v". Fixes antrea-io#4137 Signed-off-by: Antonin Bas <abas@vmware.com>
tnqn
pushed a commit
that referenced
this issue
Aug 24, 2022
When installing a released version of a chart (for Antrea or the FlowAggregator), the image tag was not generated correctly. It was missing the leading "v". Fixes #4137 Signed-off-by: Antonin Bas <abas@vmware.com>
heanlan
pushed a commit
to heanlan/antrea
that referenced
this issue
Mar 29, 2023
When installing a released version of a chart (for Antrea or the FlowAggregator), the image tag was not generated correctly. It was missing the leading "v". Fixes antrea-io#4137 Signed-off-by: Antonin Bas <abas@vmware.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
When I used helm to install Antrea 1.8.0, Antrea Pods can't be up and running because of missing image.
The image used by helm chart was "projects.registry.vmware.com/antrea/antrea-ubuntu:1.8.0" while it should be "projects.registry.vmware.com/antrea/antrea-ubuntu:v1.8.0".
It's because AppVersion is used as the tag, while it's set to 1.8.0.
antrea/build/charts/antrea/templates/agent/daemonset.yaml
Line 70 in 00e174c
To work around the issue, I hardcoded
.Values.image.tagto v1.8.0 when generating the chart and replaced the charts in release assets with the correct ones.Theia's image "theia-clickhouse-monitor" has the same issue, I didn't handle it as there are some unexpected difference between the auto-generated chart and the one I generated manually so it's still failing.
The text was updated successfully, but these errors were encountered: