Skip to content

grpc: add service accounts#5270

Merged
HofiOne merged 3 commits intosyslog-ng:developfrom
HofiOne:backport/grpc/grpc-service-accounts
Apr 4, 2025
Merged

grpc: add service accounts#5270
HofiOne merged 3 commits intosyslog-ng:developfrom
HofiOne:backport/grpc/grpc-service-accounts

Conversation

@HofiOne
Copy link
Collaborator

@HofiOne HofiOne commented Mar 17, 2025

It seems like gRPC sets the audience and scope options based on the gRPC service name, so they can't be set in the config.

Example usage:

destination {
    google-pubsub-grpc(
        project("test")
        topic("test")
        auth(service-account(key ("path_to_service_account_key.json")))
    );
};

Backport of 412 by @sodomelle

Depends on #5269

@HofiOne HofiOne changed the title Backport/grpc/grpc service accounts grpc: add service accounts Mar 17, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Mar 17, 2025

This Pull Request introduces config grammar changes

syslog-ng/23001ea8c2128ca427a928693fd512cd0e7b87b0 -> HofiOne/backport/grpc/grpc-service-accounts

Details
--- a/destination
+++ b/destination

 bigquery(
     KW_AUTH(
+        KW_SERVICE_ACCOUNT(
+            <empty>
+            KW_TOKEN_VALIDITY_DURATION(<number>)
+            key(<path>)
+        )
     )
 )

 clickhouse(
     KW_AUTH(
+        KW_SERVICE_ACCOUNT(
+            <empty>
+            KW_TOKEN_VALIDITY_DURATION(<number>)
+            key(<path>)
+        )
     )
 )

 google-pubsub-grpc(
     KW_AUTH(
+        KW_SERVICE_ACCOUNT(
+            <empty>
+            KW_TOKEN_VALIDITY_DURATION(<number>)
+            key(<path>)
+        )
     )
 )

 loki(
     KW_AUTH(
+        KW_SERVICE_ACCOUNT(
+            <empty>
+            KW_TOKEN_VALIDITY_DURATION(<number>)
+            key(<path>)
+        )
     )
 )

 opentelemetry(
     KW_AUTH(
+        KW_SERVICE_ACCOUNT(
+            <empty>
+            KW_TOKEN_VALIDITY_DURATION(<number>)
+            key(<path>)
+        )
     )
 )

 syslog-ng-otlp(
     KW_AUTH(
+        KW_SERVICE_ACCOUNT(
+            <empty>
+            KW_TOKEN_VALIDITY_DURATION(<number>)
+            key(<path>)
+        )
     )
 )

@HofiOne HofiOne force-pushed the backport/grpc/grpc-service-accounts branch 2 times, most recently from 0df8663 to f6a3748 Compare March 27, 2025 16:22
@kira-syslogng
Copy link
Contributor

Build FAILURE

@HofiOne HofiOne force-pushed the backport/grpc/grpc-service-accounts branch 2 times, most recently from aeced94 to 9d69aac Compare April 1, 2025 12:30
@HofiOne HofiOne requested a review from therandomstring April 3, 2025 08:28
Signed-off-by: Tamás Kosztyu <[email protected]>
Signed-off-by: Hofi <[email protected]>
Signed-off-by: Tamás Kosztyu <[email protected]>
Signed-off-by: Hofi <[email protected]>
@HofiOne HofiOne force-pushed the backport/grpc/grpc-service-accounts branch from 9d69aac to 36d972a Compare April 4, 2025 11:00
@HofiOne HofiOne merged commit d02e6b2 into syslog-ng:develop Apr 4, 2025
3 of 21 checks passed
@HofiOne HofiOne deleted the backport/grpc/grpc-service-accounts branch April 4, 2025 11:17
@kira-syslogng
Copy link
Contributor

Build FAILURE

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.

4 participants