Skip to content

Commit

Permalink
Move Release Pipeline to OIDC (#387)
Browse files Browse the repository at this point in the history
## Changes
In #386, I added a
workflow for testing OIDC auth to TestPyPI. This failed due to an
existing package:
https://github.com/databricks/databricks-sdk-py/actions/runs/6418567925/job/17426619417.

This PR fixes that issue by adding `skip-existing: true` and at the same
time changes the production publishing pipeline to use OIDC as well.

## Tests
- [x] Updated test pipeline succeeds:
https://github.com/databricks/databricks-sdk-py/actions/runs/6418660406/job/17426893715
  • Loading branch information
mgyucht authored Oct 5, 2023
1 parent 16f7bc0 commit bb3095b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/release-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,3 +21,5 @@ jobs:
name: Publish package distributions to TestPyPI
with:
repository-url: https://test.pypi.org/legacy/
# See https://github.com/pypa/gh-action-pypi-publish#tolerating-release-package-file-duplicates
skip-existing: true
7 changes: 4 additions & 3 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@ on:
jobs:
publish:
runs-on: ubuntu-latest
environment: release
permissions:
id-token: write
steps:
- uses: actions/checkout@v3

Expand All @@ -25,6 +28,4 @@ jobs:
dist/databricks-*.tar.gz
- uses: pypa/gh-action-pypi-publish@release/v1
with:
user: __token__
password: ${{ secrets.PYPI_TOKEN }}
name: Publish package distributions to PyPI

0 comments on commit bb3095b

Please sign in to comment.