Skip to content

Workload Identity: SPIFFE Federation and Join Method RFD#43348

Merged
strideynet merged 9 commits intomasterfrom
rfd/176-spiffe-federation
Sep 2, 2024
Merged

Workload Identity: SPIFFE Federation and Join Method RFD#43348
strideynet merged 9 commits intomasterfrom
rfd/176-spiffe-federation

Conversation

@strideynet
Copy link
Copy Markdown
Contributor

@strideynet strideynet commented Jun 21, 2024

Part of #36639
Part of #38927
Closes #44859

RFD for the introduction of SPIFFE Federation to Teleport Workload Identity and support for a SPIFFE SVID based join method.

@strideynet strideynet added the no-changelog Indicates that a PR does not require a changelog entry label Jun 21, 2024
@strideynet strideynet marked this pull request as ready for review June 24, 2024 13:06
@github-actions github-actions Bot added rfd Request for Discussion size/md labels Jun 24, 2024
@github-actions github-actions Bot requested review from camscale and rosstimothy June 24, 2024 13:07
Comment thread rfd/0175-spiffe-federation.md
Comment thread rfd/0175-spiffe-federation.md
Copy link
Copy Markdown
Contributor

@timothyb89 timothyb89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, just a few minor comments

Comment thread rfd/0175-spiffe-federation.md
Comment thread rfd/0175-spiffe-federation.md
Comment thread rfd/0175-spiffe-federation.md Outdated
Comment thread rfd/0175-spiffe-federation.md Outdated
Comment thread rfd/0175-spiffe-federation.md
### Federation

SPIFFE federation relationships are one-way. One trust domain can be configured
to trust identities issued by another trust domain, but the reverse may not
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds a lot like trusted clusters. I wonder if Teleport trusted clusters should automatically create a SPIFFE federation relationship..

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hah - I think technically this is already the case today as we currently just pull the CAs, so if you're in a leaf cluster, the root cluster SPIFFE CA is already there. But, we should definitely formalize that as a thing that happens. I think it should be possible to make the federation syncer automatically create a SPIFFEFederation resource in these cases as well so it's "visible" in the same way. I'll have a think about it.

Copy link
Copy Markdown
Contributor Author

@strideynet strideynet Sep 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So I've opted against this for now. I think there's a lot of cases where folks will want this relationship to be "two way" and I think providing an easier mechanism for configuring that (or the one way relationship) makes more sense rather than having a default that won't actually serve most people's needs

I think for now it makes sense to tackle this as a future improvement because the existing mechanism already works between two Teleport clusters and I don't want to expand the scope too much.

We'll achieve that through a custom "bundle source" that's optimised for Teleport clusters and will leverage the existing CA syncing that Teleport performs. It'll make it easier to setup, but leave the control in the hands of the operator.

Comment thread rfd/0175-spiffe-federation.md Outdated
@strideynet strideynet added this pull request to the merge queue Sep 2, 2024
Merged via the queue into master with commit f660cd5 Sep 2, 2024
@strideynet strideynet deleted the rfd/176-spiffe-federation branch September 2, 2024 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry rfd Request for Discussion size/md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SPIFFE Federation: RFD

5 participants