update advanced settings in create trigger page (close #654) #868
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
|
Review app available at: https://hge-ci-pull-868.herokuapp.com |
praveenweb
previously approved these changes
Oct 26, 2018
Contributor
|
Review app available at: https://hge-ci-pull-868.herokuapp.com |
Contributor
|
Review app available at: https://hge-ci-pull-868.herokuapp.com |
Contributor
|
Review app https://hge-ci-pull-868.herokuapp.com is deleted |
hasura-bot
pushed a commit
that referenced
this pull request
Jul 23, 2024
### What This PR enables the use of Opentelemetry Baggage. Every bit of baggage is then replicated on every span. The current implementation does not actually set any baggage itself - it only relays and outputs what it's getting. Crafting a request with a `baggage` header set:  Has it propagated (here, to `dev-auth-webhook`) and stamped onto every span:  ### How This PR actually makes the engine and auth-hook use the globally specified propagators (before they would only use an obsucre, concrete, re-exported one from opentelemetry_contrib), and adds the `BaggagePropagator` to the list. It also adds a `SpanProcessor` which outputs the baggage as span attributes. Currently it outputs all Baggage entries, but can be made more specific in the future if we want to treat Baggage differently. V3_GIT_ORIGIN_REV_ID: 3b5b8604b624c0b90c192e68b3b57fab7ca9b63e
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix #654
Description
Removed jsx portions for insert/update and changes section header and the tooltip.
What component does this PR affect?
Requires changes from other components? If yes, please mark the components:
Related Issue
Solution and Design
Type
Checklist: