Skip to content

Canary tests should use operator release image for full upgrade testing #189

@mjudeikis-bot

Description

@mjudeikis-bot

Summary

Currently, canary tests only swap the kcp image but use the current operator codebase. This means the test doesn't validate a real operator upgrade path — only that the new image works with the existing operator binary.

Problem

As highlighted in PR #188, this approach breaks when there are operator-level incompatibilities (e.g., unknown flags introduced in newer versions, like the cache server flag). The test passes for the wrong reasons and fails for reasons unrelated to the image swap.

Proposed Solution

Canary tests should test a full operator upgrade, which means:

  1. Start with the released operator image (e.g., the last stable release from the registry)
  2. Upgrade the operator itself (not just the kcp image) to the new version being tested
  3. Then validate that kcp runs correctly with the new operator

This ensures we catch operator-level regressions, not just kcp image-level ones.

Context

Raised in PR #188 review: #188 (comment)

/cc @mjudeikis

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions