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

feat(pypi): update release workflow to use OIDC auth #900

Merged
merged 3 commits into from
Sep 2, 2024

Conversation

esolitos
Copy link
Contributor

What

Update the "PyPi publishing" workflow to use OIDC authentication instead of personal-token.

Why

This removes the need of storing a secret with PYPI_TOKEN_PYVESPA in it by using the Trusted publishing feature offered by PyPi.


I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@esolitos esolitos self-assigned this Aug 29, 2024
@esolitos esolitos requested review from tokle and thomasht86 August 29, 2024 11:23
@esolitos esolitos marked this pull request as draft August 29, 2024 11:30
@esolitos
Copy link
Contributor Author

Set to Draft as it requires adding the pypa/gh-action-pypi-publish action to the allow list.

@thomasht86
Copy link
Collaborator

We should also do this for vespacli

@esolitos esolitos marked this pull request as ready for review August 30, 2024 06:53
@esolitos
Copy link
Contributor Author

pypa/gh-action-pypi-publish has been approved.

I will update also vespacli workflow.

@esolitos esolitos marked this pull request as draft August 30, 2024 06:58
@esolitos esolitos force-pushed the update-pyvespa-version branch 4 times, most recently from d62d535 to 3969a83 Compare August 30, 2024 08:57
@esolitos esolitos marked this pull request as ready for review August 30, 2024 08:58
@esolitos esolitos force-pushed the update-pyvespa-version branch from 3969a83 to fa2fe63 Compare August 30, 2024 09:04
thomasht86
thomasht86 previously approved these changes Sep 2, 2024
Copy link
Collaborator

@thomasht86 thomasht86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice!

Co-authored-by: Thomas Hjelde Thoresen <[email protected]>
Co-authored-by: Thomas Hjelde Thoresen <[email protected]>
@esolitos esolitos enabled auto-merge (squash) September 2, 2024 09:08
Copy link
Collaborator

@thomasht86 thomasht86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@thomasht86 thomasht86 disabled auto-merge September 2, 2024 13:32
@thomasht86 thomasht86 merged commit 7dd14dd into master Sep 2, 2024
29 checks passed
@thomasht86 thomasht86 deleted the update-pyvespa-version branch September 2, 2024 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants