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

Adding the Azure sync module functions along with new cloud client functionality #50366

Open
wants to merge 27 commits into
base: master
Choose a base branch
from

Conversation

mvbrock
Copy link
Contributor

@mvbrock mvbrock commented Dec 17, 2024

This PR is a result of splitting #48843 into smaller PRs, and provides the Azure fetching functions and new cloud client functionality specifically for role assignments and definitions. This PR also incorporates the existing lib/msgraph client and removes the extraneous client code from the original PR.

@mvbrock mvbrock changed the base branch from master to mvbrock/azure-integration-disco-proto December 17, 2024 23:42
@mvbrock mvbrock force-pushed the mvbrock/azure-integration-disco-azure branch from a817501 to e88e148 Compare December 17, 2024 23:43
Base automatically changed from mvbrock/azure-integration-disco-proto to master December 18, 2024 20:24
@mvbrock mvbrock force-pushed the mvbrock/azure-integration-disco-azure branch from e88e148 to 308bc2c Compare December 18, 2024 20:29
@mvbrock mvbrock marked this pull request as ready for review December 19, 2024 07:07
@github-actions github-actions bot requested a review from hugoShaka December 19, 2024 07:07
@mvbrock mvbrock added the no-changelog Indicates that a PR does not require a changelog entry label Dec 19, 2024
@mvbrock mvbrock force-pushed the mvbrock/azure-integration-disco-azure branch from 02bcb5e to 0a4c1f4 Compare December 19, 2024 23:59
@mvbrock mvbrock force-pushed the mvbrock/azure-integration-disco-azure branch from 0a4c1f4 to 672ae8d Compare December 20, 2024 00:01
Copy link

🤖 Vercel preview here: https://docs-hx7av502y-goteleport.vercel.app/docs

@mvbrock mvbrock requested a review from tigrato December 20, 2024 02:58
lib/msgraph/paginated.go Outdated Show resolved Hide resolved
lib/cloud/azure/roledefinitions.go Show resolved Hide resolved
lib/srv/discovery/fetchers/azure-sync/principals.go Outdated Show resolved Hide resolved
@mvbrock mvbrock force-pushed the mvbrock/azure-integration-disco-azure branch from 2894408 to 0dc6820 Compare December 30, 2024 17:16
lib/msgraph/paginated.go Outdated Show resolved Hide resolved
@mvbrock mvbrock requested a review from zmb3 December 30, 2024 17:51
lib/msgraph/paginated.go Outdated Show resolved Hide resolved
e Outdated Show resolved Hide resolved
lib/msgraph/paginated.go Outdated Show resolved Hide resolved
lib/msgraph/paginated.go Outdated Show resolved Hide resolved
lib/msgraph/paginated.go Outdated Show resolved Hide resolved
@mvbrock mvbrock requested review from zmb3 and tigrato January 7, 2025 05:17
@mvbrock mvbrock force-pushed the mvbrock/azure-integration-disco-azure branch from 932b2e8 to a20ac6f Compare January 7, 2025 20:19
@mvbrock mvbrock force-pushed the mvbrock/azure-integration-disco-azure branch from a1cfe02 to 801ef67 Compare January 9, 2025 04:40
go.mod Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discovery no-changelog Indicates that a PR does not require a changelog entry size/lg
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants