Skip to content

operator: Point to local operator helm chart in kuttle tests#912

Merged
RafalKorepta merged 1 commit intorelease/v2.4.xfrom
rk/2.4.x/fix-kuttl-v2-tests
Jun 16, 2025
Merged

operator: Point to local operator helm chart in kuttle tests#912
RafalKorepta merged 1 commit intorelease/v2.4.xfrom
rk/2.4.x/fix-kuttl-v2-tests

Conversation

@RafalKorepta
Copy link
Contributor

@RafalKorepta RafalKorepta commented Jun 12, 2025

As the #675 changed the
set up of the operator installation the helm repository was missing. The disable
helm controllers test case depends on official helm repository index existence.
From that point the kuttle v2 end to end test suite start to fail. In this
change the test case will use local operator helm chart definition.

Reference
9c9aa06#diff-25902a4a01139dc3d8d70d5086b8cdb53d4b68b46185b3a5dc37c8585e801181L16-L18

@chrisseto
Copy link
Contributor

chrisseto commented Jun 12, 2025

@RafalKorepta can we instead refer to the local helm chart? That command should have been pointing there anyways As I see, this is part of the test suite. It should still point at the local helm chart (operator/chart) if possible.

@chrisseto
Copy link
Contributor

chrisseto commented Jun 12, 2025

Same comment as in #912 Whoops, I must have had too many tabs open.

@RafalKorepta RafalKorepta force-pushed the rk/2.4.x/fix-kuttl-v2-tests branch from d605e57 to de643c3 Compare June 14, 2025 20:03
@RafalKorepta RafalKorepta changed the title operator: Add missing redpanda helm chart repository in disable helm cotnrollers test case operator: Point to local operator helm chart in kuttle tests Jun 14, 2025
@RafalKorepta RafalKorepta force-pushed the rk/2.4.x/fix-kuttl-v2-tests branch from de643c3 to 5856c63 Compare June 14, 2025 20:12
As the #675 changed the
set up of the operator installation the helm repository was missing. The disable
helm controllers test case depends on official helm repository index existence.
From that point the kuttle v2 end to end test suite start to fail. In this
change the test case will use local operator helm chart definition.

Reference
9c9aa06#diff-25902a4a01139dc3d8d70d5086b8cdb53d4b68b46185b3a5dc37c8585e801181L16-L18
@RafalKorepta RafalKorepta force-pushed the rk/2.4.x/fix-kuttl-v2-tests branch from 5856c63 to 07b528f Compare June 14, 2025 22:13
@RafalKorepta RafalKorepta merged commit 13c3941 into release/v2.4.x Jun 16, 2025
11 checks passed
@RafalKorepta RafalKorepta deleted the rk/2.4.x/fix-kuttl-v2-tests branch June 16, 2025 14:02
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