Skip to content

Fixed configcheck validates ClusterVectorPipeline (CVP) against all ClusterVectorAggregator instances instead of only matching ones#202

Closed
P0lskay wants to merge 1 commit into
kaasops:mainfrom
P0lskay:P0lskay/issue201
Closed

Fixed configcheck validates ClusterVectorPipeline (CVP) against all ClusterVectorAggregator instances instead of only matching ones#202
P0lskay wants to merge 1 commit into
kaasops:mainfrom
P0lskay:P0lskay/issue201

Conversation

@P0lskay
Copy link
Copy Markdown
Contributor

@P0lskay P0lskay commented Nov 16, 2025

Fixes #201

Added a check for matching pipeline labels to the vector selector.

@P0lskay P0lskay marked this pull request as ready for review November 23, 2025 09:11
author an.makarov <gg.lol.2264@gmail.com> 1763274744 +0300
committer an.makarov <gg.lol.2264@gmail.com> 1763904905 +0300

`configcheck` validates ClusterVectorPipeline (CVP) against all ClusterVectorAggregator instances instead of only matching ones
Fixes kaasops#201

feat: add e2e test framework and CI workflow (kaasops#206)

* feat: add e2e test framework and CI workflow

- Add comprehensive e2e test framework with Ginkgo/Gomega
- Add GitHub Actions workflow for automated e2e testing
- Add artifact collection for test failures
- Add HTML report generation script
- Add normal mode tests as baseline
- Update Makefile with test-e2e, test-report, deploy-helm-e2e targets
- Update Ginkgo to v2.20.2 and Gomega to v1.34.1
- Add CI/CD documentation

* fix(makefile): add ginkgo dependency for test-e2e target

- Add GINKGO binary to tool binaries section
- Add GINKGO_VERSION (v2.20.2) to tool versions
- Add ginkgo target to download ginkgo if necessary
- Update test-e2e target to depend on ginkgo and use $(GINKGO)

This fixes CI failure where ginkgo command was not found.

* chore(ci): remove event-collector from e2e workflow

Event collector is not used in current e2e tests.

* fix(makefile): use absolute path for ginkgo binary

Formatted k8s package

Fixed imports for linting :)

Fix imports again :)
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.

configcheck validates ClusterVectorPipeline (CVP) against all ClusterVectorAggregator instances instead of only matching ones

1 participant