Skip to content

Conversation

@santigimeno
Copy link
Member

No description provided.

In some cases, they can take a negative value (don't really know why
because supposedly v8 uses a monotonic clock). We don't want to cast
them to unsigned to avoid overflows.

PR-URL: #254
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Add `app` field to ZMQ messages.
Update appName when it's dynamically changed.

PR-URL: #264
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Increase a bit the keepalive period and timeout.
Also, set `GRPC_ARG_HTTP2_MAX_PINGS_WITHOUT_DATA` to `0` to make sure we
can send `PING` frames indefinitelly even if there's no activity in the
channel, to make sure the channel is up for as long as possible.

PR-URL: #265
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
PR-URL: #256
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #256
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #260
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
PR-URL: #180
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-by: Trevor Norris <trev.norris@gmail.com>
PR-URL: #260
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
To retrieve the opentelemetry-proto version from the MODULE.bazel file.

PR-URL: #260
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
PR-URL: #266
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
So we can subscribe to the `flagsUpdated` event to receive notifications
when the tracing flags have changed.
Use it in the otel/core.js:register() method.

PR-URL: #216
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #245
PR-URL: #266
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Make sure the type of the JSON field matches with what's expected to
avoid unwanted crashes.

PR-URL: #267
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
@santigimeno santigimeno self-assigned this Feb 14, 2025
santigimeno added a commit that referenced this pull request Feb 14, 2025
In some cases, they can take a negative value (don't really know why
because supposedly v8 uses a monotonic clock). We don't want to cast
them to unsigned to avoid overflows.

PR-URL: #254
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
PR-URL: #269
santigimeno added a commit that referenced this pull request Feb 14, 2025
Add `app` field to ZMQ messages.
Update appName when it's dynamically changed.

PR-URL: #264
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
PR-URL: #269
santigimeno added a commit that referenced this pull request Feb 14, 2025
Increase a bit the keepalive period and timeout.
Also, set `GRPC_ARG_HTTP2_MAX_PINGS_WITHOUT_DATA` to `0` to make sure we
can send `PING` frames indefinitelly even if there's no activity in the
channel, to make sure the channel is up for as long as possible.

PR-URL: #265
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
PR-URL: #269
santigimeno added a commit that referenced this pull request Feb 14, 2025
PR-URL: #256
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #269
santigimeno added a commit that referenced this pull request Feb 14, 2025
PR-URL: #256
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #269
santigimeno added a commit that referenced this pull request Feb 14, 2025
PR-URL: #260
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
PR-URL: #269
santigimeno added a commit that referenced this pull request Feb 14, 2025
PR-URL: #180
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-by: Trevor Norris <trev.norris@gmail.com>
PR-URL: #260
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
PR-URL: #269
santigimeno added a commit that referenced this pull request Feb 14, 2025
To retrieve the opentelemetry-proto version from the MODULE.bazel file.

PR-URL: #260
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
PR-URL: #269
santigimeno added a commit that referenced this pull request Feb 14, 2025
PR-URL: #266
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #269
santigimeno added a commit that referenced this pull request Feb 14, 2025
PR-URL: #269
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
santigimeno added a commit that referenced this pull request Feb 14, 2025
So we can subscribe to the `flagsUpdated` event to receive notifications
when the tracing flags have changed.
Use it in the otel/core.js:register() method.

PR-URL: #216
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #245
PR-URL: #269
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
santigimeno added a commit that referenced this pull request Feb 14, 2025
PR-URL: #266
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #269
santigimeno added a commit that referenced this pull request Feb 14, 2025
PR-URL: #269
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
santigimeno added a commit that referenced this pull request Feb 14, 2025
Make sure the type of the JSON field matches with what's expected to
avoid unwanted crashes.

PR-URL: #267
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #269
@santigimeno
Copy link
Member Author

Landed in c733e9c...d2ee311

@santigimeno santigimeno deleted the santi/backports_18.x branch February 14, 2025 19:58
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.

3 participants