Code Changes for Updates💣
Jaeger is a modified/customized version of an upstream chart. The below details the steps required to update to a new version of the Jaeger package.
-
Navigate to the upstream chart repo and folder and find the tag that corresponds with the new chart version for this image update. For example, if updating the Jaeger images to 1.28 you would check the chart values and switch Gitlab tags until you find the latest chart version that uses 1.28 images. In this case that is
jaeger-operator-2.27.0
(as of this doc construction). -
Checkout the
renovate/ironbank
branch. This branch will already have the updates you need for the images. -
From the root of the repo run
kpt pkg update chart@jaeger-operator-2.27.0 --strategy alpha-git-patch
replacingjaeger-operator-2.27.0
with the version tag you got in step 1. You may be prompted to resolve some conflicts - choose what makes sense (if there are BB additions/changes keep them, if there are upstream additions/changes keep them). -
Modify the
version
inChart.yaml
- you will want to append-bb.0
to the chart version from upstream. -
Update
CHANGELOG.md
adding an entry for the new version and noting all changes (at minimum should includeUpdated Jaeger to x.x.x
). -
Generate the
README.md
updates by following the guide in gluon. -
Push up your changes, validate that CI passes. If there are any failures follow the information in the pipeline to make the necessary updates and reach out to the team if needed.
-
Perform the steps below for manual testing. CI provides a good set of basic smoke tests but it is beneficial to run some additional checks.
Manual Testing for Updates💣
NOTE: For these testing steps it is good to do them on both a clean install and an upgrade. For clean install, point jaeger to your branch. For an upgrade do an install with jaeger pointing to the latest tag, then perform a helm upgrade with jaeger pointing to your branch.
You will want to install with:
- Jaeger, Logging (elastic, eck operator, and fluentbit), Kiali, Authservice and Monitoring packages enabled
- Istio enabled
- Dev SSO values for Kiali, Monitoring, and Jaeger
Testing Steps:
- Login with SSO to Jaeger (if you are not prompted for an SSO login, this could indicate a problem with the authservice connection)
- On the search fields on the left pick a service and click “find traces”. Validate that traces load.
- Navigate to Kiali and login with SSO, under applications find the namespace and service that corresponds with the service you picked in Jaeger earlier. Validate that traces show under the traces tab.
- Navigate to Prometheus and validate that the Jaeger operator targets show as UP.
When in doubt with any testing or upgrade steps ask one of the CODEOWNERS for assistance.
Modifications made to upstream chart💣
This is a high-level list of modifications that Big Bang has made to the upstream helm chart. You can use this as as cross-check to make sure that no modifications were lost during the upgrade process.
chart/templates/bigbang💣
- Files added to support networkPolicies, cert generation, monitoring, mTLS enforcement, VirtualService, etc
chart/Chart.yaml💣
- Append
-bb.x
versioning to version - Add gluon dependency chart for helm tests (also run
helm dependency update chart/
to store this):dependencies: - name: gluon version: 0.2.9 repository: oci://registry.dso.mil/platform-one/big-bang/apps/library-charts/gluon
- Add bigbang dev annotation for release automation:
annotations: bigbang.dev/applicationVersions: | - Jaeger: 1.34.1
chart/templates/_helpers.tpl💣
- Added selector label template to support upgrades
{{/* Generate selector labels -- see issue #512. This allows helm upgrades to happen */}} {{- define "jaeger-operator.selector.labels" }} app.kubernetes.io/name: {{ include "jaeger-operator.name" . }} {{- end }}
- Added
name: {{ include "jaeger-operator.fullname" . }}
to"jaeger-operator.labels"
template
chart/templates/deployment.yaml💣
spec.selector.matchLabels
changed to{{ include "jaeger-operator.selector.labels" . | nindent 6 }}
- Upgrade strategy added below
spec.replicas
:{{- if .Values.operatorUpdateStrategy }} strategy: {{- toYaml .Values.operatorUpdateStrategy | nindent 4 }} {{- end }}
- Annotations values added below
extraLabels
:{{- if .Values.annotations }} annotations: {{ toYaml .Values.annotations | nindent 8 }} {{- end }}
chart/templates/jaeger.yaml💣
- Changed name to
jaeger
- Refactored to support certain parts of the spec rather than a simple toYaml (should we re-evaluate this?)
chart/templates/tests💣
- Gluon cypress test template added
chart/tests💣
- Cypress config and test added
chart/values.yaml💣
- Substantial values additions/changes to use IB images, support BB core interactions, etc
- When in doubt ask about these changes