[v15] Add EKS auto discovery for dynamic integration based fetchers.#38772
Merged
AntonAM merged 1 commit intobranch/v15from Feb 29, 2024
Merged
[v15] Add EKS auto discovery for dynamic integration based fetchers.#38772AntonAM merged 1 commit intobranch/v15from
AntonAM merged 1 commit intobranch/v15from
Conversation
AntonAM
added
the
no-changelog
github-actions
Bot
requested review from
marcoandredinis,
rosstimothy and
tigrato
AntonAM
changed the title
tigrato
approved these changes
Feb 29, 2024
Collaborator
|
/excludeflake * |
AntonAM
force-pushed
the
anton/backport-38204-branch/v15
branch
from
b673d42 to
a08f3d0
Compare
* AWSOIDC: Move enrollment of EKS clusters to the gRPC service. * Fix after rebase. * Add EKS auto discovery for dynamic integration based fetchers. * Remove commented code. Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com> * Remove unneeded code. Co-authored-by: Marco André Dinis <marco.dinis@goteleport.com> * Add missing godoc. * Initiate map with length. Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com> * Fix typo in the interface name. * Introduce iam join method for EKS enrollment. * Correctly process DiscoveredEKSCluster in kube watcher. * Remove remaining CreateToken related code from Discovery service. * Change var initialization. Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com> * Remove dev version detection for EKS enrollment. Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com> * Revert "Introduce iam join method for EKS enrollment." This reverts commit 0eb3f4f1226239a5d4e768ebe9129535be318ccb. * Fix usage of a missing function. * Refactor cluster features to be a function. * Change discovery to use gRPC call for EKS enrollment. * Remove unused mock. * Correct a comment. * Remove token creation permission from Discovery service. * Fix after rebase. * Improve sorting clusters by region and integration. Co-authored-by: Tiago Silva <tiago.silva@goteleport.com> * Return and error if cluster features are missing. * Fix tests. --------- Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com> Co-authored-by: Marco André Dinis <marco.dinis@goteleport.com> Co-authored-by: Tiago Silva <tiago.silva@goteleport.com>
AntonAM
force-pushed
the
anton/backport-38204-branch/v15
branch
from
a08f3d0 to
2ec52fc
Compare
rosstimothy
approved these changes
Feb 29, 2024
public-teleport-github-review-bot
Bot
removed the request for review
from marcoandredinis
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
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.
Backport #38204 to branch/v15.
Depends on #38771
Manual backport - couple of phantom merge conflicts on neighbouring line because of TAG matchers.