Skip to content

Prevent trusted clusters in Cloud#31815

Merged
rosstimothy merged 1 commit intomasterfrom
tross/cloud_tc
Sep 13, 2023
Merged

Prevent trusted clusters in Cloud#31815
rosstimothy merged 1 commit intomasterfrom
tross/cloud_tc

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread lib/auth/auth_with_roles.go Outdated
@rosstimothy rosstimothy requested a review from dboslee September 13, 2023 17:51
Copy link
Copy Markdown
Contributor

@codingllama codingllama left a comment

Choose a reason for hiding this comment

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

Do you think it's worth adding a test for it?

Comment thread lib/auth/auth_with_roles.go Outdated
Comment thread lib/auth/trustedcluster_test.go Outdated
Comment thread lib/auth/trustedcluster_test.go Outdated
Comment thread lib/auth/trustedcluster_test.go Outdated
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from lxea September 13, 2023 21:05
@rosstimothy rosstimothy added this pull request to the merge queue Sep 13, 2023
Merged via the queue into master with commit 0c5992a Sep 13, 2023
@rosstimothy rosstimothy deleted the tross/cloud_tc branch September 13, 2023 21:45
@public-teleport-github-review-bot
Copy link
Copy Markdown

@rosstimothy See the table below for backport results.

Branch Result
branch/v13 Create PR
branch/v14 Create PR

rosstimothy added a commit that referenced this pull request Dec 13, 2023
#31815 already prevented a Cloud tenant from being a root cluster
but it did not stop a Cloud tenant from being a leaf. This prevents
the creating of `trusted_cluster` resources which effectively blocks
any attempts to make Cloud be a leaf cluster.
github-merge-queue Bot pushed a commit that referenced this pull request Dec 13, 2023
#31815 already prevented a Cloud tenant from being a root cluster
but it did not stop a Cloud tenant from being a leaf. This prevents
the creating of `trusted_cluster` resources which effectively blocks
any attempts to make Cloud be a leaf cluster.
github-actions Bot pushed a commit that referenced this pull request Dec 13, 2023
#31815 already prevented a Cloud tenant from being a root cluster
but it did not stop a Cloud tenant from being a leaf. This prevents
the creating of `trusted_cluster` resources which effectively blocks
any attempts to make Cloud be a leaf cluster.
github-actions Bot pushed a commit that referenced this pull request Dec 13, 2023
#31815 already prevented a Cloud tenant from being a root cluster
but it did not stop a Cloud tenant from being a leaf. This prevents
the creating of `trusted_cluster` resources which effectively blocks
any attempts to make Cloud be a leaf cluster.
rosstimothy added a commit that referenced this pull request Dec 13, 2023
#31815 already prevented a Cloud tenant from being a root cluster
but it did not stop a Cloud tenant from being a leaf. This prevents
the creating of `trusted_cluster` resources which effectively blocks
any attempts to make Cloud be a leaf cluster.
github-merge-queue Bot pushed a commit that referenced this pull request Dec 13, 2023
#31815 already prevented a Cloud tenant from being a root cluster
but it did not stop a Cloud tenant from being a leaf. This prevents
the creating of `trusted_cluster` resources which effectively blocks
any attempts to make Cloud be a leaf cluster.
github-merge-queue Bot pushed a commit that referenced this pull request Dec 13, 2023
#31815 already prevented a Cloud tenant from being a root cluster
but it did not stop a Cloud tenant from being a leaf. This prevents
the creating of `trusted_cluster` resources which effectively blocks
any attempts to make Cloud be a leaf cluster.
github-merge-queue Bot pushed a commit that referenced this pull request Dec 13, 2023
#31815 already prevented a Cloud tenant from being a root cluster
but it did not stop a Cloud tenant from being a leaf. This prevents
the creating of `trusted_cluster` resources which effectively blocks
any attempts to make Cloud be a leaf cluster.
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.

4 participants