Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove other mirror maker 2 topics from test #628

Conversation

RafalKorepta
Copy link
Contributor

After test the following tests are left behind:

redpanda@redpanda-xkdkx8z0cl-0:/$ rpk topic list
NAME                              PARTITIONS  REPLICAS
_internal_connectors_configs      1           1
_internal_connectors_offsets      25          1
_internal_connectors_status       5           1
_schemas                          1           3
mm2-offset-syncs.target.internal  1           3
source.test-topic                 1           1

Copy link
Contributor

@alejandroEsc alejandroEsc left a comment

Choose a reason for hiding this comment

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

lgtm

@RafalKorepta RafalKorepta force-pushed the rk/remove-leftovers-from-mm2-test branch from b22249b to 85d0c66 Compare July 31, 2023 21:38
After test the following tests are left behind:
```
redpanda@redpanda-xkdkx8z0cl-0:/$ rpk topic list
NAME                              PARTITIONS  REPLICAS
_internal_connectors_configs      1           1
_internal_connectors_offsets      25          1
_internal_connectors_status       5           1
_schemas                          1           3
mm2-offset-syncs.target.internal  1           3
source.test-topic                 1           1
```
In the Github action there is issue of existing mirror maker 2 connector
already exist after helm upgrade is invoked. As the Pods are named the
same we lose logs from previous execution. This change will make unique
Pod name by appending 3 numbers at the end of mirror maker 2 test Pod.
@RafalKorepta RafalKorepta force-pushed the rk/remove-leftovers-from-mm2-test branch from 6c1d248 to 05bd6df Compare August 1, 2023 10:46
@RafalKorepta RafalKorepta force-pushed the rk/remove-leftovers-from-mm2-test branch from 05bd6df to 5115a04 Compare August 1, 2023 11:45
@RafalKorepta RafalKorepta merged commit 6c2b26e into redpanda-data:main Aug 2, 2023
16 of 17 checks passed
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