chore(release): v0.1.10 - #520
Conversation
| repository: public.ecr.aws/devzeroinc/zxporter | ||
| pullPolicy: IfNotPresent | ||
| tag: "v0.1.6" | ||
| tag: "v0.1.9" |
There was a problem hiding this comment.
⚠️ Bug: v0.1.10 release ships v0.1.9 chart versions and image tags
This PR is titled/tagged as release v0.1.10, but every Helm chart version, appVersion, and container image tag was bumped only to 0.1.9 / v0.1.9. Since this PR's headline feature is the new SchedulerMetricsCollector, users installing the v0.1.10 chart would pull v0.1.9 images that likely do not contain this code, so the feature would not actually be delivered. Bump the chart versions, appVersions, and image tags to v0.1.10 (or confirm the intended lag) so the released artifacts match the release name.
Align all chart versions, appVersions and image tags with the v0.1.10 release.:
# helm-chart/zxporter/values.yaml
tag: "v0.1.10"
# helm-chart/zxporter/Chart.yaml
version: 0.1.10
appVersion: "0.1.10"
# and dependency zxporter-nodemon version: "0.1.10"
# helm-chart/zxporter-netmon & zxporter-nodemon Chart.yaml / values.yaml: v0.1.10 likewise
- Apply fix
Check the box to apply the fix or reply for a change | Was this helpful? React with 👍 / 👎
Code Review
|
| if err == nil && pool.AppendCertsFromPEM(caData) { | ||
| tlsConfig.RootCAs = pool | ||
| } else { | ||
| tlsConfig.InsecureSkipVerify = true //nolint:gosec // see schedulerMetricsCAFileEnv doc comment. |
| tlsConfig.InsecureSkipVerify = true //nolint:gosec // see schedulerMetricsCAFileEnv doc comment. | ||
| } | ||
| } else { | ||
| tlsConfig.InsecureSkipVerify = true //nolint:gosec // see schedulerMetricsCAFileEnv doc comment. |
Release v0.1.10 of zxporter, published from the internal services monorepo.
The
v0.1.10tag has already been created and points at this branch's tip — ArgoCD and other tag consumers can use it immediately. This PR is only to advance public main to the released commit for browser convenience.