Skip to content

set up correct version for ci e2e job#43

Merged
cnvergence merged 1 commit intomainfrom
fix-ci
Oct 2, 2025
Merged

set up correct version for ci e2e job#43
cnvergence merged 1 commit intomainfrom
fix-ci

Conversation

@cnvergence
Copy link
Copy Markdown
Collaborator

@cnvergence cnvergence commented Oct 2, 2025

Summary by CodeRabbit

  • Tests
    • Improved kind-based testing: test setup now automatically uses the current chart version and force-reinstalls to reduce failures on repeated runs, remove manual updates to hardcoded package names, and ensure a consistent, predictable test environment—making CI and local testing more reliable and speeding up developer iteration.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Oct 2, 2025

Walkthrough

Makefile packaging now passes an explicit chart version to helm package and kind-test installs the versioned chart artifact dist/example-httpbin-operator-$(CHART_VERSION).tgz, adding --force to the Helm install command.

Changes

Cohort / File(s) Summary of Changes
Build & test automation
Makefile
Added --version $(CHART_VERSION) to helm package and updated kind-test to install dist/example-httpbin-operator-$(CHART_VERSION).tgz with --force to overwrite existing resources.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes


📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Free

📥 Commits

Reviewing files that changed from the base of the PR and between 0e243e9 and d02ad61.

📒 Files selected for processing (1)
  • Makefile (2 hunks)

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
@cnvergence cnvergence merged commit 1c77c8f into main Oct 2, 2025
8 checks passed
@cnvergence cnvergence deleted the fix-ci branch October 2, 2025 12:57
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.

2 participants