Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Add setup arguments to enable specifying alt tenants on the CLI #900

Merged
1 commit merged into from
May 19, 2021

Conversation

bmc-msft
Copy link
Contributor

This enables specifying the endpoint configuration for alternate tenants purely on the command line.

Previously, on a single tenant you could use the following:

onefuzz --endpoint https://INSTANCE.azurewebsites.net info get

For multi-tenant installs, we need to expose more than just endpoint.

This enables:

onefuzz --endpoint https://INSTANCE.azurewebsites.net --client_id CLIENT_ID  --authority https://login.microsoftonline.com/common --tenant_domain TENANT_DOMAIN info get

@ghost
Copy link

ghost commented May 18, 2021

Hello @bmc-msft!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 18 hours, a condition that will be fulfilled in about 10 hours 50 minutes. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 776e8fa into microsoft:main May 19, 2021
@bmc-msft bmc-msft deleted the add-setup-params-for-alternate-tenant branch May 24, 2021 19:27
@ghost ghost locked as resolved and limited conversation to collaborators Jun 24, 2021
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants