How to upgrade📜
Upgrading the minio instance can be a a little tricky since there are a number of changes that are unique to the minio instance deployed by Big Bang. You can not simply do a kpt pull and replace since there are unique changes that must be maintained. This is because of the way the minio tenant was originally deployed by in the upstream minio operator charts.
- Sync with new chart. This can be done with kpt or meld: Be careful not to overwrite the Big Bang specific changes.
kpt pkg update chart/@{TAG} --strategy alpha-git-patch
orkpt pkg update chart/@{TAG} --strategy force-delete-replace
or Meld UI - Move
minio.min.io_tenants.yaml
tominio-operator-crds/templates/
- Update
minio-operator-crds/Chart.yaml
with new version - Update
chart/Chart.yaml
dependency version - Rebuild dependency .tgz:
export HELM_EXPERIMENTAL_OCI=1
helm dependency update chart
Deploying📜
Prerequisites: - Make sure to do git pull to get the latest code from bigbang
Deployment Steps: - The following is an example override:
addons:
minio:
enabled: true
git:
tag: ""
branch: "renovate/ironbank"
minioOperator:
enabled: true
# git:
# tag: ""
# branch: "renovate/ironbank"
values:
console:
enabled: true
helm upgrade chart -f <registryCredentials overrides> -f chart/ingress-certs.yaml -f docs/assets/configs/example/dev-sso-values.yaml -f docs/assets/configs/example/policy-overrides-k3d.yaml -f <minio override file>
- Verify /etc/hosts file contains minio.bigbang.dev
targeting your K3D instance’s public IP
How to test the upgrade📜
Local install: 1. Clean install: install minio-operator and minio from your branch using deployment steps above. 1. Upgrade: install current versions of minio-operator and minio, and upgrade to your branch. Ensure both minio-operator and minio have upgraded to the newer versions. 1. Test: navigate to minio.bigbang.dev. Log in (credentials can be found in /chart/values.yaml). Create test bucket and verify creation. Delete test bucket and verify deletion.
Pipeline: modify MinIO package dependency to point to your branch/version and create an MR
automountServiceAccountToken📜
The mutating Kyverno policy named update-automountserviceaccounttokens
is leveraged to harden all ServiceAccounts in this package with automountServiceAccountToken: false
. This policy is configured by namespace in the Big Bang umbrella chart repository at chart/templates/kyverno-policies/values.yaml.
This policy revokes access to the K8s API for Pods utilizing said ServiceAccounts. If a Pod truly requires access to the K8s API (for app functionality), the Pod is added to the pods:
array of the same mutating policy. This grants the Pod access to the API, and creates a Kyverno PolicyException to prevent an alert.
Deploying with modifications to tenant pool definitions📜
At this time, tenant definitions are replaced by the last array provided in the merge order. This means any user override will completely replace default values provided in the chart which may result in key configurations being removed.
To resolve this issue, when modifying or applying new tenant pool definitions, all values must be duplicated in the user override chart (ex: securityContext
, containerSecurityContext
, automountServceAccountToken
, etc.).
Initial basic values for a single minio tenant pool can be found in chart/values.yaml
.
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.
When performing the helm update the following items should be maintained.📜
- in chart/templates/_helpers.tpl: keep the BB specific _helpers
{{/* Big Bang Added Helpers Start Here */}} {{/* Create the name of the service used to access the Minio object UI. Note: the Minio operator has a fixed name of "minio" for the service it creates. */}} {{- define "minio-operator.serviceName" }} {{- if .Values.upgradeTenants.enabled -}} minio {{- else -}} {{- default (include "minio.fullname" .) .Values.service.nameOverride }} {{- end }} {{- end }}
- in chart/templates/tenant-configuration.yaml: keep the secrets data access key changes._helpers
{{- if dig "secrets" true (.Values | merge (dict)) }} apiVersion: v1 kind: Secret metadata: name: {{ dig "secrets" "name" "" (.Values | merge (dict)) }} type: Opaque stringData: config.env: |- export MINIO_ROOT_USER={{ .Values.secrets.accessKey | quote }} export MINIO_ROOT_PASSWORD={{ .Values.secrets.secretKey | quote }} data: ## Access Key for MinIO Tenant accesskey: {{ dig "secrets" "accessKey" "" (.Values | merge (dict)) | b64enc }} ## Secret Key for MinIO Tenant secretkey: {{ dig "secrets" "secretKey" "" (.Values | merge (dict)) | b64enc }} {{- end }}
- in charts/templates/tenant.yaml: keep the changes associated with…_helpers
- assigning the metadata/name
- spec/image name
- spec/configuration secret and name
- metadata/annotations
chart/values.yaml📜
- Bigbang additions at the end of the file
- images to ironbank
chart/templates/bigbang/*📜
- networkpolicies/*
- minio-operaotor-exception.yaml
- peer-authentication.yaml
- tenant-patch-job.yaml
chart/charts/*.tgz📜
-
run
helm dependency update ./chart
and commit the downloaded archives -
commit the tar archives that were downloaded from the helm dependency update command. And also commit the requirements.lock that was generated.
Files that need integration testing📜
If you modify any of these things, you should perform an integration test with your branch against the rest of bigbang. Some of these files have automatic tests already defined, but those automatic tests may not model corner cases found in full integration scenarios.
./chart/templates/bigbang/*
./chart/templates/api-ingress.yaml
./chart/templates/peer-authentication.yaml
./chart/values.yaml
if it involves any of:- monitoring changes
- network policy changes
- kyverno policy changes
- istio hardening rule changes
- service definition changes
- TLS settings
- server ingress settings
- headless server settings (especially port or other comms settings)
Follow the standard process for performing an integration test against bigbang.