Skip to content

[v17] Vendor gravitational/trace/trail in api#56034

Merged
tigrato merged 2 commits intobranch/v17from
tigrato/bump-trace-v17
Jun 25, 2025
Merged

[v17] Vendor gravitational/trace/trail in api#56034
tigrato merged 2 commits intobranch/v17from
tigrato/bump-trace-v17

Conversation

@tigrato
Copy link
Copy Markdown
Contributor

@tigrato tigrato commented Jun 24, 2025

Backport #51032 to branch/v17

* Vendor gravitational/trace/trail in api

Pulling in the trail package directly in api will allow the trace
module to shed the grpc-go dependency. This needs to land prior
to gravitational/trace#112 being included
in a new version of trace.

There should be no noticable change in the api depdency tree since
it already depends on grpc-go. Some additional items from the
trace/internal package were also vendored within trail as needed.
Additionally, some of the public api of trail that was not being
consumed has been made private.

* fix: appease linters
@tigrato tigrato requested a review from rosstimothy June 24, 2025 13:36
@tigrato tigrato added the no-changelog Indicates that a PR does not require a changelog entry label Jun 24, 2025
@github-actions github-actions Bot added backport database-access Database access related issues and PRs kubernetes-access size/md tctl tctl - Teleport admin tool labels Jun 24, 2025
@tigrato tigrato force-pushed the tigrato/bump-trace-v17 branch from 9a802ea to 7d6a027 Compare June 25, 2025 14:53
@tigrato tigrato enabled auto-merge June 25, 2025 15:00
@tigrato tigrato added this pull request to the merge queue Jun 25, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 25, 2025
@tigrato tigrato added this pull request to the merge queue Jun 25, 2025
Merged via the queue into branch/v17 with commit fea20dd Jun 25, 2025
41 checks passed
@tigrato tigrato deleted the tigrato/bump-trace-v17 branch June 25, 2025 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport database-access Database access related issues and PRs kubernetes-access no-changelog Indicates that a PR does not require a changelog entry size/md tctl tctl - Teleport admin tool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants