-
Notifications
You must be signed in to change notification settings - Fork 369
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.
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
[datadog_synthetics_test] Add plain_proto_file and deprecate compressed_json_descriptor #2273
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
2023-05-17T17:05:32.491613-04:00 | ||
2024-02-08T12:25:53.842767+01:00 |
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
2023-10-09T14:55:35.802692+02:00 | ||
2024-02-08T12:07:18.4908+01:00 |
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -344,11 +344,11 @@ resource "datadog_synthetics_test" "grpc" { | |
type = "api" | ||
subtype = "grpc" | ||
request_definition { | ||
method = "GET" | ||
host = "google.com" | ||
port = 50050 | ||
service = "Hello" | ||
proto_json_descriptor = <<EOT | ||
method = "GET" | ||
host = "google.com" | ||
port = 50050 | ||
service = "Hello" | ||
plain_proto_file = <<EOT | ||
syntax = "proto3"; | ||
option java_multiple_files = true; | ||
option java_package = "io.grpc.examples.helloworld"; | ||
|
@@ -609,8 +609,9 @@ Optional: | |
- `no_saving_response_body` (Boolean) Determines whether or not to save the response body. | ||
- `number_of_packets` (Number) Number of pings to use per test for ICMP tests (`subtype = "icmp"`) between 0 and 10. | ||
- `persist_cookies` (Boolean) Persist cookies across redirects. | ||
- `plain_proto_file` (String) The content of a proto file as a string | ||
- `port` (Number) Port to use when performing the test. | ||
- `proto_json_descriptor` (String) A protobuf JSON descriptor. | ||
- `proto_json_descriptor` (String, Deprecated) A protobuf JSON descriptor. **Deprecated.** Use `plain_proto_file` instead | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add period at the end of this line for consistency |
||
- `servername` (String) For SSL tests, it specifies on which server you want to initiate the TLS handshake, allowing the server to present one of multiple possible certificates on the same IP address and TCP port number. | ||
- `service` (String) The gRPC service on which you want to perform the gRPC call. | ||
- `should_track_hops` (Boolean) This will turn on a traceroute probe to discover all gateways along the path to the host destination. For ICMP tests (`subtype = "icmp"`). | ||
|
@@ -954,8 +955,9 @@ Optional: | |
- `no_saving_response_body` (Boolean) Determines whether or not to save the response body. | ||
- `number_of_packets` (Number) Number of pings to use per test for ICMP tests (`subtype = "icmp"`) between 0 and 10. | ||
- `persist_cookies` (Boolean) Persist cookies across redirects. | ||
- `plain_proto_file` (String) The content of a proto file as a string | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add period |
||
- `port` (Number) Port to use when performing the test. | ||
- `proto_json_descriptor` (String) A protobuf JSON descriptor. | ||
- `proto_json_descriptor` (String, Deprecated) A protobuf JSON descriptor. **Deprecated.** Use `plain_proto_file` instead | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add period |
||
- `servername` (String) For SSL tests, it specifies on which server you want to initiate the TLS handshake, allowing the server to present one of multiple possible certificates on the same IP address and TCP port number. | ||
- `service` (String) The gRPC service on which you want to perform the gRPC call. | ||
- `should_track_hops` (Boolean) This will turn on a traceroute probe to discover all gateways along the path to the host destination. For ICMP tests (`subtype = "icmp"`). | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add period at the end of this line for consistency
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made another pr #2274 for these comment, feel free to give it a 👍