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

[v16] [entraid] add setup script for offline clusters. #48089

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

tigrato
Copy link
Contributor

@tigrato tigrato commented Oct 29, 2024

Backport #47863 to branch/v16

changelog: Added support for Entra ID directory synchronization for clusters without public internet access.

@github-actions github-actions bot added backport size/md tctl tctl - Teleport admin tool labels Oct 29, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-48089.d212ksyjt6y4yg.amplifyapp.com

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from mvbrock October 30, 2024 08:44
@tigrato tigrato force-pushed the bot/backport-47863-branch/v16 branch from 6d30e67 to 97c44e8 Compare October 30, 2024 09:25
@tigrato tigrato enabled auto-merge October 30, 2024 09:26
* [entraid] add setup script for offline clusters.

This PR adds a cli configuration for Entra ID where it's possible to default to system credentials instead of relying on OIDC for authentication in EntraID. OIDC is not always a possibility specially when the cluster is private and not internet acessible.

The UX is the following:

```text

Step 1: Run the Setup Script

1. Open **Azure Cloud Shell** (Bash) using **Google Chrome** or **Safari** for the best compatibility.
2. Upload the setup script using the **Upload** button in the Cloud Shell toolbar.
3. Once uploaded, execute the script by running the following command:
   $ bash entraid.sh

**Important Considerations**:
- You must have **Azure privileged administrator permissions** to complete the integration.
- Ensure you're using the **Bash** environment in Cloud Shell.
- During the script execution, you'll be prompted to run 'az login' to authenticate with Azure. **Teleport** does not store or persist your credentials.
- **Mozilla Firefox** users may experience connectivity issues in Azure Cloud Shell; using Chrome or Safari is recommended.

Once the script completes, type 'continue' to proceed, 'exit' to quit: continue

Step 2: Input Tenant ID and Client ID

With the output of Step 1, please copy and paste the following information:
Enter the Tenant ID: 1056b571-0390-4b08-86c8-2edba8d9ae79
Enter the Client ID: 1056b571-0390-4b08-86c8-2edba8d9ae79

Successfully created EntraID plugin "name".
```

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>

* move function to api

* handle code review comments

* Apply suggestions from code review

Co-authored-by: Marco Dinis <marco.dinis@goteleport.com>

* fix url

* enable group claims

* add godoc

* handle code review comments

* fix gomod

---------

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
Co-authored-by: Marco Dinis <marco.dinis@goteleport.com>
@tigrato tigrato force-pushed the bot/backport-47863-branch/v16 branch from 97c44e8 to 8c8f8a8 Compare October 30, 2024 09:38
@tigrato tigrato added this pull request to the merge queue Oct 30, 2024
Merged via the queue into branch/v16 with commit 331d8a3 Oct 30, 2024
43 checks passed
@tigrato tigrato deleted the bot/backport-47863-branch/v16 branch October 30, 2024 10:13
@camscale camscale mentioned this pull request Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport size/md tctl tctl - Teleport admin tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants