Skip to content

pass component prerelease image tag to E2E tests#86

Open
bvolovat wants to merge 1 commit intomainfrom
fix/pass-component-image-tags
Open

pass component prerelease image tag to E2E tests#86
bvolovat wants to merge 1 commit intomainfrom
fix/pass-component-image-tags

Conversation

@bvolovat
Copy link
Copy Markdown
Contributor

@bvolovat bvolovat commented Mar 31, 2026

Summary

  • Pass the prerelease image tag via component_image_tags in the E2E dispatch payload so system tests run against the actual built image instead of the default release
  • Removes charts_name from the dispatch (receiver already defaults it to kubescape-operator) to stay within GitHub's 10 top-level key limit on client_payload
  • Companion PRs: armosec/shared-workflows (forward COMPONENT_IMAGE_TAGS in receiver) and kubescape/node-agent (same change in its local workflow copy)

How it works

The dispatch now includes component_image_tags=<component>-tag=<prerelease-version> (e.g., operator-tag=v0.2.42-prerelease). The receiver forwards it to b-run-system-tests-self-hosted.yaml which already supports COMPONENT_IMAGE_TAGS and appends it to --kwargs for systest-cli.py.

Test plan

  • Merge armosec/shared-workflows companion PR first (receiver must forward the new field)
  • Trigger a component PR merge and verify the dispatch includes component_image_tags
  • Verify system tests install the helm chart with the prerelease image override

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated CI/CD workflow to modify the payload structure used when dispatching system tests. The payload now includes component image tags and removes unnecessary fields, improving test execution efficiency.

Replace the hardcoded charts_name (already defaulted in the receiver)
with component_image_tags so system tests run against the actual
pre-release image instead of the default release.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 31, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 48ed6ca7-ba7c-4385-94f0-d94fedb424c3

📥 Commits

Reviewing files that changed from the base of the PR and between 889c5f6 and c0db9f3.

📒 Files selected for processing (1)
  • .github/workflows/incluster-comp-pr-merged.yaml

📝 Walkthrough

Walkthrough

Modified .github/workflows/incluster-comp-pr-merged.yaml to change the GitHub repository_dispatch payload for system tests dispatch. Removed the charts_name field while keeping charts_repo, and added a new component_image_tags field dynamically composed from the component name and prerelease image tag.

Changes

Cohort / File(s) Summary
GitHub Workflow Dispatch Configuration
.github/workflows/incluster-comp-pr-merged.yaml
Modified the client_payload in the "Dispatch system tests to private repo" step: removed charts_name field, added component_image_tags field with value dynamically constructed from COMPONENT_NAME and IMAGE_TAG_PRERELEASE.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested reviewers

  • Naor-Armo

Poem

🐰 A rabbit hops through workflows fine,
Removing charts, new tags align,
The payload shifts with nimble grace,
Components tagged in perfect place! 🏷️

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'pass component prerelease image tag to E2E tests' directly and clearly describes the main change: adding the ability to pass a prerelease image tag to E2E tests via the dispatch payload.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/pass-component-image-tags

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions
Copy link
Copy Markdown

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: failure

1 similar comment
@github-actions
Copy link
Copy Markdown

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: failure

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants