Skip to content

feat: Adding new role type - mixed#191

Open
sanicheev wants to merge 406 commits into
kaasops:mainfrom
sanicheev:feat_mixed_role_type
Open

feat: Adding new role type - mixed#191
sanicheev wants to merge 406 commits into
kaasops:mainfrom
sanicheev:feat_mixed_role_type

Conversation

@sanicheev
Copy link
Copy Markdown

@sanicheev sanicheev commented Aug 27, 2025

This PR addresses the issue with wildcard processing in input parameters.

Vector supports wildcard identifiers: https://vector.dev/highlights/2021-01-20-wildcard-identifiers/
This is extremely useful in case you have to deal with multi-tenancy configurations(different projects store observability data in different accounts/resources)
Attaching a diagram with an example use-case:
vectorpipelinewildcard

In order to achieve this, following 2 issues needs to be solved:

  • Fix agent/aggregator role selection logic
  • Control prefix usage

I added required changes in my PR.

This PR includes following changes:

  • Adding new role type: mixed.
    In case pipeline has a mixed source (valid for both agent and aggregator).
  • New skipPrefix flag on vectorpipelines and clustervectorpipelines.
    If specified will skip adding a prefix to a pipeline name(namespace + object name).
    Required when using wildcard in input references from different pipeline.
  • Applying linter fixes which were missed before

zvlb and others added 30 commits November 19, 2022 12:17
Signed-off-by: Zemtsov Vladimir <vl.zemtsov@gmail.com>
Signed-off-by: Zemtsov Vladimir <vl.zemtsov@gmail.com>
Signed-off-by: Zemtsov Vladimir <vl.zemtsov@gmail.com>
Signed-off-by: Zemtsov Vladimir <vl.zemtsov@gmail.com>
Signed-off-by: Zemtsov Vladimir <vl.zemtsov@gmail.com>
Signed-off-by: Zemtsov Vladimir <vl.zemtsov@gmail.com>
Signed-off-by: Zemtsov Vladimir <vl.zemtsov@gmail.com>
Signed-off-by: Zemtsov Vladimir <vl.zemtsov@gmail.com>
Signed-off-by: Zemtsov Vladimir <vl.zemtsov@gmail.com>
Signed-off-by: Zemtsov Vladimir <vl.zemtsov@gmail.com>
Signed-off-by: Zemtsov Vladimir <vl.zemtsov@gmail.com>
Filter cache and disable time reconcile
do not reconсile vector if  vp check fail
merge vp and cvp reconcile funcs
aa1ex and others added 29 commits October 4, 2024 14:57
add cluster vector aggregator

Signed-off-by: Aleksandr Aleksandrov <aaleksandrov.cy@gmail.com>
add pre-v* to github action tags

Signed-off-by: Aleksandr Aleksandrov <aaleksandrov.cy@gmail.com>
Signed-off-by: zvlb <vl.zemtsov@gmail.com>
fix role print && add doc

Signed-off-by: Aleksandr Aleksandrov <aaleksandrov.cy@gmail.com>
Signed-off-by: zvlb <vl.zemtsov@gmail.com>
Signed-off-by: zvlb <vl.zemtsov@gmail.com>
* fix: add event access for operator

Signed-off-by: zvlb <vl.zemtsov@gmail.com>
* fix: add list access for events

Signed-off-by: zvlb <vl.zemtsov@gmail.com>
restruct event-collector

Signed-off-by: Aleksandr Aleksandrov <aaleksandrov.cy@gmail.com>
…-failed-resources

add retry mechanism for reconciling invalid vector pipelines
Signed-off-by: Aleksandr Aleksandrov <aaleksandrov.cy@gmail.com>
Add support for global option expire-metrics-secs

---------

Signed-off-by: Aleksandr Aleksandrov <aaleksandrov.cy@gmail.com>
Co-authored-by: an.makarov <an.makarov@tensor.ru>
Signed-off-by: Aleksandr Aleksandrov <aaleksandrov.cy@gmail.com>
improve performance with global vector params
---------

Signed-off-by: Aleksandr Aleksandrov <aaleksandrov.cy@gmail.com>
Signed-off-by: zvlb <vl.zemtsov@gmail.com>
* rebase PR133

Signed-off-by: Robert van Veelen <vanveele@users.noreply.github.com>

* Update specification

* Fix typo

---------

Signed-off-by: Robert van Veelen <vanveele@users.noreply.github.com>
Co-authored-by: Christopher Forkner <cforkner@hitachims.local>
envFrom for aggregator

Signed-off-by: Aleksandr Aleksandrov <aaleksandrov.cy@gmail.com>
Signed-off-by: Aleksandr Aleksandrov <aaleksandrov.cy@gmail.com>
Fixes kaasops#182

Update labels definition for all resources
Fixes kaasops#182

Update manifests and code containing DeepCopy, DeepCopyInto, and DeepCopyObject method implementations

Update documentation

Add individual method for matchLabels

Update description for labels field in CRDs

Fix specification.md

Update CRDs in helm chart

Create new function MergeLabels for merging of labels :)
P0lskay/issue182 Add support for customizable labels on Vector resources
Fix imports in vectoragent_controller.go
Signed-off-by: Hornwind <hornwind89@gmail.com>
This PR includes following changes:
- Adding new role type: mixed. In case pipline has a mixed source
  (valid for both agent and aggregator).
- New skipPrefix flag on vectorpipelines and clustervectorpipelines.
  If specified will skip adding a prefix to a pipeline name(namespace + object name).
  Required when using wildcard in input references from different
pipeline.
- Applying linter fixes which were missed before
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.

8 participants