Skip to content

Try OIDC #3639

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Try OIDC #3639

wants to merge 6 commits into from
+44 −0

Conversation

@shreyas-goenka
Copy link
Contributor

Changes

Why

Tests

@eng-dev-ecosystem-bot
Copy link
Collaborator

eng-dev-ecosystem-bot commented Sep 22, 2025 •
edited
Loading

Run: 17919707197

Env ✅​pass 🙈​skip
✅​ aws linux 311 529
✅​ aws windows 312 528
✅​ aws-ucws linux 423 427
✅​ aws-ucws windows 424 426
✅​ azure linux 311 528
✅​ azure windows 312 527
✅​ azure-ucws linux 423 426
✅​ azure-ucws windows 424 425
✅​ gcp linux 310 530
✅​ gcp windows 311 529

@shreyas-goenka
Refactor OIDC workflow into composite action
910a894 
- Move Databricks CLI steps to .github/actions/foo/bar/action.yml
- Update workflow to use composite action with inputs
- Makes the OIDC testing logic reusable
@shreyas-goenka
Fix local action checkout issue
ae4f2ab 
- Add checkout step to main workflow before calling local action
- Remove redundant checkout from composite action
- Fixes 'Can't find action.yml' error
@pietern
Copy link
Contributor

pietern commented Sep 29, 2025

@shreyas-goenka Can this be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andrewnester andrewnester Awaiting requested review from andrewnester andrewnester will be requested when the pull request is marked ready for review andrewnester is a code owner

@anton-107 anton-107 Awaiting requested review from anton-107 anton-107 will be requested when the pull request is marked ready for review anton-107 is a code owner

@denik denik Awaiting requested review from denik denik will be requested when the pull request is marked ready for review denik is a code owner

@pietern pietern Awaiting requested review from pietern pietern will be requested when the pull request is marked ready for review pietern is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@shreyas-goenka @eng-dev-ecosystem-bot @pietern