Skip to content

chore: v1.2.1 release prep — version bump, docs update, and Makefile automation - #119

Open
yuva29 wants to merge 1 commit into
release-v1.2.1from
user/yuva29/v1.2.1-release-prep-rocm-release
Open

chore: v1.2.1 release prep — version bump, docs update, and Makefile automation#119
yuva29 wants to merge 1 commit into
release-v1.2.1from
user/yuva29/v1.2.1-release-prep-rocm-release

Conversation

@yuva29

@yuva29 yuva29 commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

Cherry-pick of pensando/network-operator#358 to release-v1.2.1.

Full v1.2.1 release prep including:

  • Version bump to v1.2.1 across Go code, CI, Helm charts, OLM bundle/catalog
  • AINIC_VERSIONS, update-docs-version, and release-prep Makefile targets
  • Docs: image tags bumped, compat matrix v1.2.1 row, OpenShift 4.21 in OS support matrix
  • Fix update-version-in-ci sed to preserve shell variable refs in bundle build flow
  • Fix cni-pluginsk8s-cni-plugins typo in networkconfig.md

🤖 Generated with Claude Code

@yuva29
yuva29 force-pushed the user/yuva29/v1.2.1-release-prep-rocm-release branch 6 times, most recently from 174ef27 to 9d0b72d Compare August 28, 2026 22:28
…automation

Cherry-pick from pensando/network-operator#358.

- Bump version to v1.2.1 across Go code, CI, Helm charts, OLM bundle/catalog
- Add AINIC_VERSIONS, update-docs-version, and release-prep Makefile targets
- Bump docs image tags and helm version refs from v1.1.0 to v1.2.1
- Add v1.2.1 compat matrix row and OpenShift 4.21 to OS support matrix
- Fix update-version-in-ci sed pattern to preserve shell variable refs
- Fix cni-plugins image name typo in networkconfig.md

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
@yuva29
yuva29 force-pushed the user/yuva29/v1.2.1-release-prep-rocm-release branch from 9d0b72d to 9ac413f Compare August 28, 2026 22:54
@yuva29
yuva29 requested review from sundar-pds and a lite review from Copilot August 28, 2026 22:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Prepares the v1.2.1 release by updating version references across the operator codebase, Helm artifacts, docs, and OLM bundle assets, and by adding Makefile automation to streamline the release-prep workflow.

Changes:

  • Bump default image/version strings to v1.2.1 across Go code, example manifests, Helm chart metadata, and docs.
  • Add/extend Makefile automation for release prep (docs version updates, CI version updates, docs lint target, bundle build tweaks).
  • Add/update OLM bundle assets (CSV/CRD/manifests, bundle annotations, scorecard config, device-plugin ConfigMap).

Reviewed changes

Copilot reviewed 33 out of 34 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
Makefile Version bump, new release-prep/docs-lint/CI+docs update targets, bundle-build tweak, and registry/version update expansions
internal/utils.go Bump default utils image tag to v1.2.1
internal/secondarynetwork/cniplugins.go Bump default CNI plugins image tag to v1.2.1
internal/nodelabeller/nodelabeller.go Bump default node-labeller image tag to v1.2.1
internal/metricsexporter/exporter.go Bump default metrics exporter image tag to v1.2.1
internal/deviceplugin/deviceplugin.go Bump default device-plugin image tag to v1.2.1
internal/controllers/upgrademgr.go Bump default utils image tag to v1.2.1
helm-charts-openshift/values.yaml Add OpenShift chart values with controller image tag set to v1.2.1
helm-charts-k8s/README.md Update chart version badge to v1.2.1
helm-charts-k8s/crds/networkconfig-crd.yaml Update chart label version to v1.2.1
helm-charts-k8s/Chart.yaml Update chart version to v1.2.1 and adjust icon/keywords
helm-charts-k8s/Chart.lock Regenerate lock metadata timestamp
hack/openshift-patch/metadata-patch/Chart.yaml Bump patch chart version to v1.2.1
hack/k8s-patch/metadata-patch/Chart.yaml Bump patch chart version to v1.2.1
hack/device-plugin-configmap.yaml Add device-plugin ConfigMap source used during bundle build
example/networkconfig.yaml Update example image tags to v1.2.1
docs/upgrades/upgrade.md Update Helm upgrade examples to v1.2.1
docs/upgrades/componentupgrades.md Update example image tags to v1.2.1
docs/metrics/exporter.md Update metrics exporter image tag to v1.2.1
docs/installation/networkconfig.md Update documented defaults (including k8s-cni-plugins name) to v1.2.1
docs/installation/networkconfig-full.md Update full example defaults to v1.2.1
docs/installation/kubernetes-helm.md Update Helm install/upgrade examples and defaults to v1.2.1
docs/installation/kubernetes-helm-operators.md Update Helm co-install example version to v1.2.1
docs/index.md Update OS/Kubernetes/OpenShift support matrix and add v1.2.1 compat row
docs/device_plugin/deviceplugin.md Update documented default images to v1.2.1
docs/_static/cluster-validation-job.yaml Update utils image tag to v1.2.1 in validation job YAML
config/samples/amd.com_networkconfigs.yaml Update sample CR image defaults to v1.2.1
config/manifests/bases/amd-network-operator.clusterserviceversion.yaml Update CSV base annotations/images to v1.2.1
bundle/tests/scorecard/config.yaml Add scorecard configuration for bundle tests
bundle/metadata/annotations.yaml Add standard bundle annotations, including scorecard test config pointer
bundle/manifests/device-plugin-configmap.yaml Add ConfigMap manifest to the bundle
bundle/manifests/amd.com_networkconfigs.yaml Add CRD manifest to the bundle
bundle/manifests/amd-network-operator.clusterserviceversion.yaml Add generated CSV for v1.2.1 bundle
bundle/manifests/amd-network-operator-manager-config_v1_configmap.yaml Add manager config ConfigMap manifest to the bundle
Suppressed comments (6)

bundle/tests/scorecard/config.yaml:27

  • mountPath is set to {} (a map), but Scorecard storage mountPath should be a string path; leaving it as {} is likely to break scorecard config parsing.
        mountPath: {}

bundle/tests/scorecard/config.yaml:37

  • mountPath is set to {} (a map), but Scorecard storage mountPath should be a string path; leaving it as {} is likely to break scorecard config parsing.
        mountPath: {}

bundle/tests/scorecard/config.yaml:47

  • mountPath is set to {} (a map), but Scorecard storage mountPath should be a string path; leaving it as {} is likely to break scorecard config parsing.
        mountPath: {}

bundle/tests/scorecard/config.yaml:57

  • mountPath is set to {} (a map), but Scorecard storage mountPath should be a string path; leaving it as {} is likely to break scorecard config parsing.
        mountPath: {}

bundle/tests/scorecard/config.yaml:67

  • mountPath is set to {} (a map), but Scorecard storage mountPath should be a string path; leaving it as {} is likely to break scorecard config parsing.
        mountPath: {}

bundle/tests/scorecard/config.yaml:70

  • Top-level Scorecard storage.spec.mountPath is set to {} (a map), but it should be a string path; leaving it as {} is likely to break scorecard config parsing.
    mountPath: {}

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread Makefile
sed -i '0,/BUNDLE_VERSION=/s|BUNDLE_VERSION=[^ ]*|BUNDLE_VERSION=$${RELEASE:-${PROJECT_VERSION}-dev}|' .job.yml
sed -i 's|network-operator-helm-k8s-[^$$].*\.tgz|network-operator-helm-k8s-${HELM_CHARTS_VERSION}.tgz|' .job.yml
sed -i 's|network-operator-helm-openshift-.*\.tgz|network-operator-helm-openshift-${HELM_CHARTS_VERSION}.tgz|' .job.yml
sed -i 's|PROJECT_VERSION:-.*$$|PROJECT_VERSION:-${PROJECT_VERSION}\}|' asset-build/networkoperator-asset-push.sh
Comment thread Makefile

.PHONY: release-prep
release-prep: IMAGE_TAG=$(PROJECT_VERSION)
release-prep: update-version-in-ci update-docs-version helm-k8s bundle-build catalog ## Prepare release: bump versions in code, CI, docs, helm charts, and OLM bundle/catalog
test: basic-check-spec-test
storage:
spec:
mountPath: {}

@sundar-pds sundar-pds left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dont we need these changes in main branch as well. ?
Can you update the branch_policy.yml as well in this PR , to trigger the CP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants