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.

Sign up for GitHub

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

Jump to bottom

feat: enable tenant routing #86

Merged
merged 10 commits into from
Oct 28, 2024
Merged

feat: enable tenant routing #86

merged 10 commits into from
Oct 28, 2024

Conversation

csatib02
Copy link
Member

@csatib02 csatib02 commented Oct 21, 2024 •
edited
Loading

  • Adds support for routing specific logs from a tenant to another tenant using connectors and the transform processor based on statements.

TODO

  • Use Bridge's status in route controller.

kép

  • Docs example.

@csatib02
feat: add transform processor to tenant crd
79f2c52 
Signed-off-by: Bence Csati <bence.csati@axoflow.com>
@csatib02 csatib02 added the enhancement New feature or request label Oct 21, 2024
@csatib02 csatib02 self-assigned this Oct 21, 2024
@csatib02 csatib02 marked this pull request as ready for review October 21, 2024 15:44
OverOrion
OverOrion reviewed Oct 22, 2024
View reviewed changes
Copy link
Collaborator

@OverOrion OverOrion left a comment

Choose a reason for hiding this comment

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

I like where this is going! 🚀

@csatib02 csatib02 force-pushed the feat/enable-tenant-routing branch 8 times, most recently from 5c05dcc to 7f3ad78 Compare October 24, 2024 12:05
@csatib02 csatib02 requested a review from OverOrion October 24, 2024 12:07
@csatib02
feat: introduce bridge crd
4dbab3e 
Signed-off-by: Bence Csati <bence.csati@axoflow.com>
@csatib02
feat: enable tenant routing
d898db3 
Signed-off-by: Bence Csati <bence.csati@axoflow.com>
@csatib02
feat: add rbac for new crd
625ff9b 
Signed-off-by: Bence Csati <bence.csati@axoflow.com>
@csatib02
feat(test): add unit tests for tenant routing
df26416 
Signed-off-by: Bence Csati <bence.csati@axoflow.com>
@csatib02
chore: yamllint, tidy up makefile, add bridge crd to kustomize configs
d6af85f 
Signed-off-by: Bence Csati <bence.csati@axoflow.com>
@csatib02
refactor: route-reconciler
f724235 
Signed-off-by: Bence Csati <bence.csati@axoflow.com>
@csatib02
feat: add state field for subs, enable routing reconciler for bridges
1273a8e 
Signed-off-by: Bence Csati <bence.csati@axoflow.com>
@csatib02
docs: add guide for tenant to tenant routing
ed5852e 
Signed-off-by: Bence Csati <bence.csati@axoflow.com>
@csatib02 csatib02 force-pushed the feat/enable-tenant-routing branch from 7f3ad78 to 045e1b4 Compare October 25, 2024 07:44
OverOrion
OverOrion reviewed Oct 25, 2024
View reviewed changes
Makefile Outdated Show resolved Hide resolved
internal/controller/telemetry/route_controller.go Outdated Show resolved Hide resolved
@csatib02 csatib02 force-pushed the feat/enable-tenant-routing branch from 045e1b4 to c11a3fa Compare October 25, 2024 12:17
@csatib02 csatib02 force-pushed the feat/enable-tenant-routing branch 2 times, most recently from c47814e to 6c92d72 Compare October 25, 2024 12:42
@csatib02 csatib02 requested a review from OverOrion October 25, 2024 12:52
@csatib02
chore: tidy up
094701c 
Signed-off-by: Bence Csati <bence.csati@axoflow.com>
@csatib02 csatib02 force-pushed the feat/enable-tenant-routing branch from 6c92d72 to 094701c Compare October 28, 2024 08:43
OverOrion
OverOrion approved these changes Oct 28, 2024
View reviewed changes
Copy link
Collaborator

@OverOrion OverOrion left a comment

Choose a reason for hiding this comment

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

LGTM, nicely done!

@csatib02 csatib02 merged commit 00a9177 into main Oct 28, 2024
13 checks passed
@csatib02 csatib02 deleted the feat/enable-tenant-routing branch October 28, 2024 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pepov pepov pepov approved these changes

@OverOrion OverOrion OverOrion approved these changes

@tarokkk tarokkk Awaiting requested review from tarokkk

@kristofgyuracz kristofgyuracz Awaiting requested review from kristofgyuracz

Assignees

@csatib02 csatib02

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@csatib02 @pepov @OverOrion