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: OBS-381 - implement DistributorService.Push RPC #9

Merged
merged 12 commits into from
Feb 2, 2024
Merged

feat: OBS-381 - implement DistributorService.Push RPC #9

merged 12 commits into from
Feb 2, 2024

Conversation

mfiedorowicz
Copy link
Member

@mfiedorowicz mfiedorowicz commented Feb 2, 2024 •
edited
Loading

  • basic implementation of DistributorService.Push RPC appending data into Redis stream
  • distributor service with Redis client
  • unit tests for server package (note: GitHub Action for running go test 🔜 )
  • minor fixes

@mfiedorowicz
feat: OBS-381 - use same logger for consistency
9b19350 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-381 - add redis to distributor config
b95b194 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-381 - add redis client to distributor component
66b0850 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-381 - set correct default redis port
1c3651b 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-381 - fix server error messages formatting
0a94db3 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-381 - append ingestion data into redis stream
b452aa0 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-381 - fix typo
cae3de2 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-381 - add unit tests for server package
150fd61 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-381 - tidy up server tests
79cbafa 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-381 - basic push request validation
5d057d4 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-381 - extract push request validation into func
a065f49 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz mfiedorowicz self-assigned this Feb 2, 2024
@linear Linear
Copy link

linear bot commented Feb 2, 2024

OBS-381 Implement DistributorService.Push RPC

@mfiedorowicz
feat: OBS-381 - revive if-return
2aac012 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz mfiedorowicz merged commit 01bad08 into develop Feb 2, 2024
7 checks passed
@mfiedorowicz mfiedorowicz deleted the OBS-381-implement-distributor-push branch February 2, 2024 21:20
@orb-ci
Copy link

orb-ci commented Feb 15, 2024

🎉 This PR is included in version diode-ingester-v1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@orb-ci
Copy link

orb-ci commented Feb 15, 2024

🎉 This PR is included in version diode-reconciler-v1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@orb-ci
Copy link

orb-ci commented Feb 15, 2024

🎉 This PR is included in version diode-distributor-v1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@natm natm Awaiting requested review from natm

Assignees

@mfiedorowicz mfiedorowicz

Labels
dependencies diode-ingester diode-reconciler diode-server go released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mfiedorowicz @orb-ci