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-390 - [reconciler] implement ReconcilerService.RetrieveIngestionDataSources #43

Merged
merged 11 commits into from
Feb 21, 2024
Merged

feat: OBS-390 - [reconciler] implement ReconcilerService.RetrieveIngestionDataSources #43

merged 11 commits into from
Feb 21, 2024

Conversation

mfiedorowicz
Copy link
Member

  • add proto for reconciler service and generate go code
  • implement ReconcilerService.RetrieveIngestionDataSources RPC
  • add client for reconciler service [go]
  • tidy up API key for ingestion (s/DATASOURCE_TO_DIODE/INGESTION)
  • distributor service utilizing reconciler client to retrieve ingestion data sources and check if incoming push requests are authorized

@mfiedorowicz
feat: OBS-390 - add proto for reconciler
719a384 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-390 - [reconciler] implement ReconcilerService.RetrieveInge…
286fc26 
…stionDataSources

Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-390 - [reconciler] add client
c8abed9 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
chore: [distributor] - remove redundant constant
2c20ad8 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
chore: tidy up ingestion api key provisioning
7bb781b 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-390 - [distributor] add dependency on reconciler
5b2c189 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-390 - [reconciler] add sdk name+version to the request via …
84933aa 
…client

Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-390 - [distributor] configure reconciler client
30e0962 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-390 - [reconciler] tidy up ingestion data sources and api keys
15290cf 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-390 - [distributor] handle ingestion data sources
d4b2223 
- retrieve ingestion data sources utilising reconciler svc client
- add auth unary interceptor
- tidy up

Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz
feat: OBS-390 - [distributor] don't log ingestion data sources
c37aafa 
Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz mfiedorowicz self-assigned this Feb 21, 2024
@mfiedorowicz mfiedorowicz requested a review from natm as a code owner February 21, 2024 22:01
@linear Linear
Copy link

linear bot commented Feb 21, 2024

OBS-390 Create and implement ReconcilerService.RetrieveIngestionDatasources RPC

Create reconciler service with RetrieveIngestionDataSources RPC returning DATASOURCE_TO_DIODE API key.

Distributor service to call ReconcilerService.RetrieveIngestionDatasources to retrieve and cache into its memory.

@mfiedorowicz mfiedorowicz merged commit 7e8117e into develop Feb 21, 2024
15 checks passed
@mfiedorowicz mfiedorowicz deleted the feat-OBS-390-retrieve-ingestion-data-sources branch February 21, 2024 22:07
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
diode-netbox-plugin diode-reconciler diode-server go python
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mfiedorowicz