Skip to content
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

Refactor lib/tbot/spiffe to use WorkloadAttrs proto #50833

Merged
merged 2 commits into from
Jan 15, 2025

Conversation

strideynet
Copy link
Contributor

@strideynet strideynet commented Jan 7, 2025

Part of: #49986

As per RFD191: #49133

Depends on #50812

A wholly internal refactor of tbot to use the new Workload Identity attribute protos. This sets the stage for the next PR which will introduce the new tbot service for the SPIFFE Workload API and the new Workload Identity UX.

This PR shouldn't introduce any changes in behaviour.

@strideynet strideynet added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v16 backport/branch/v17 labels Jan 8, 2025
@strideynet strideynet requested a review from timothyb89 January 8, 2025 10:55
@strideynet strideynet force-pushed the strideynet/refactor-tbot-internals-around-attrs branch 2 times, most recently from 85accc5 to 68eebba Compare January 8, 2025 14:38
@strideynet strideynet marked this pull request as ready for review January 9, 2025 09:37
@github-actions github-actions bot requested a review from vapopov January 9, 2025 09:38
@strideynet strideynet force-pushed the strideynet/refactor-tbot-internals-around-attrs branch from 68eebba to 467a4cb Compare January 14, 2025 13:08
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from vapopov January 15, 2025 09:58
Base automatically changed from strideynet/new-x509-output-tbot to master January 15, 2025 10:25
@strideynet strideynet force-pushed the strideynet/refactor-tbot-internals-around-attrs branch from 467a4cb to bc2ce53 Compare January 15, 2025 10:44
@strideynet strideynet enabled auto-merge January 15, 2025 10:50
@strideynet strideynet added this pull request to the merge queue Jan 15, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 15, 2025
@strideynet strideynet added this pull request to the merge queue Jan 15, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 15, 2025
@strideynet strideynet added this pull request to the merge queue Jan 15, 2025
Merged via the queue into master with commit 228b392 Jan 15, 2025
41 checks passed
@strideynet strideynet deleted the strideynet/refactor-tbot-internals-around-attrs branch January 15, 2025 12:42
@public-teleport-github-review-bot

@strideynet See the table below for backport results.

Branch Result
branch/v16 Failed
branch/v17 Failed

strideynet added a commit that referenced this pull request Jan 15, 2025
* Refactor `lib/tbot/spiffe` to use WorkloadAttrs proto

* Refactor existing Workload API and SDS services to use new attr protos
strideynet added a commit that referenced this pull request Jan 15, 2025
* Refactor `lib/tbot/spiffe` to use WorkloadAttrs proto

* Refactor existing Workload API and SDS services to use new attr protos
github-merge-queue bot pushed a commit that referenced this pull request Jan 15, 2025
* Refactor `lib/tbot/spiffe` to use WorkloadAttrs proto

* Refactor existing Workload API and SDS services to use new attr protos
github-merge-queue bot pushed a commit that referenced this pull request Jan 15, 2025
* Refactor `lib/tbot/spiffe` to use WorkloadAttrs proto

* Refactor existing Workload API and SDS services to use new attr protos
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants