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

Dev: Started using the trusted publisher concept of Pypi #1693

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

andy-maier
Copy link
Member

@andy-maier andy-maier commented Oct 9, 2024

When trusted publishing was enabled for the exporter, the upload to Pypi worked, but the subsequent step in the publish.yml workflow to create a release failed. GitHub ticket https://support.github.com/ticket/personal/0/3042595 has been opened for that. Need to wait for clarity on that.

The trust of GitHub has already been established on the Pypi side for all zhmcclient repos.

Once trusted publishing works, the PYPI_API_KEY secrets can be removed in GitHub and the corresponding keys in Pypi.

@andy-maier andy-maier self-assigned this Oct 9, 2024
@andy-maier andy-maier added this to the 1.19.0 milestone Oct 9, 2024
@coveralls
Copy link
Collaborator

coveralls commented Oct 9, 2024

Coverage Status

coverage: 82.453%. remained the same
when pulling 45154d4 on andy/fix-publish-warning
into 1b91074 on master.

@andy-maier andy-maier force-pushed the andy/fix-publish-warning branch from 4d1e4ed to 511d223 Compare October 9, 2024 19:57
@andy-maier andy-maier force-pushed the andy/fix-publish-warning branch from 511d223 to 58ddf3a Compare October 10, 2024 07:29
@andy-maier andy-maier force-pushed the andy/fix-publish-warning branch from 58ddf3a to 45154d4 Compare October 10, 2024 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants