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

Remove pgp signing for pypi artifacts #222

Closed
gaiksaya opened this issue May 25, 2023 · 13 comments
Closed

Remove pgp signing for pypi artifacts #222

gaiksaya opened this issue May 25, 2023 · 13 comments
Assignees
Labels
enhancement New feature or request

Comments

@gaiksaya
Copy link
Member

Is your feature request related to a problem? Please describe

PyPi recently announced that they will be Removing PGP from PyPI. See https://blog.pypi.org/posts/2023-05-23-removing-pgp/

Even though the upload is allowed it will be silently ignored.

Describe the solution you'd like

Remove https://github.com/opensearch-project/opensearch-build-libraries/blob/main/vars/publishToPyPi.groovy#L19-L23

Describe alternatives you've considered

Continue publishing but it will throw an error then the support is completely removed.

Additional context

No response

@gaiksaya gaiksaya added enhancement New feature or request untriaged labels May 25, 2023
@prudhvigodithi
Copy link
Member

[Untriage]
Yes we can remove the .asc signing after confirming that if the uploaded tar package with twine does have some signing handled by PyPi.

@gaiksaya
Copy link
Member Author

Maybe remove PyPi library completely https://blog.pypi.org/posts/2023-04-20-introducing-trusted-publishers/

@zelinh
Copy link
Member

zelinh commented Nov 21, 2023

Since Pypi no longer requires PGP signing, we plan to move the Pypi publishing from Jenkins to GHA.
According to my research, here are some steps:

@bbarani
Copy link
Member

bbarani commented Jan 29, 2024

@zelinh can you please check all the repos publishing the signatures and create a campaign to on-board to new workflow?

@zelinh zelinh moved this from Not started to Done in OpenSearch Engineering Effectiveness Jan 29, 2024
@zelinh
Copy link
Member

zelinh commented Jan 29, 2024

This task is completed. We have added onboarding doc here.

@zelinh zelinh closed this as completed Jan 29, 2024
@gaiksaya
Copy link
Member Author

gaiksaya commented Jan 30, 2024

We need to create a campaign for all existing pypi artifacts. Please keep this issue open until we have that.

We might also need to deprecate the library from the vars folder for upcoming versions.
Thanks!

@zelinh
Copy link
Member

zelinh commented Jan 30, 2024

We need to create a campaign for all existing pypi artifacts. Please keep this issue open until we have that.

We might also need to deprecate the library from the vars folder for upcoming versions. Thanks!

I have created issue here in benchmark repo. opensearch-project/opensearch-benchmark#451

I think that's the only active pypi artifact we released using our jenkins library.

@gaiksaya
Copy link
Member Author

I see few more: https://github.com/search?q=org%3Aopensearch-project+publishToPyPi&type=code
Maybe list down in this issue what are migrated to new process and remaining ones to be migrated.

@zelinh
Copy link
Member

zelinh commented Jan 30, 2024

I see few more: https://github.com/search?q=org%3Aopensearch-project+publishToPyPi&type=code Maybe list down in this issue what are migrated to new process and remaining ones to be migrated.

We have addressed for migration on all repos from the link you shared. Not sure which else you talk about.

@gaiksaya
Copy link
Member Author

If we have already migrated then we need to off-board all the repos. Remove webhooks to jenkins and related jenkins file from the repos.

@zelinh
Copy link
Member

zelinh commented Jan 30, 2024

If we have already migrated then we need to off-board all the repos. Remove webhooks to jenkins and related jenkins file from the repos.

I think we should leave it to the repo owner. Neither of the team has released yet with the new onboarding GHA. We have done steps to help them onboard and they could remove it in the future since they won't be using it anymore.

@gaiksaya
Copy link
Member Author

gaiksaya commented Feb 5, 2024

We will open new issue to track the lib removal and off-boarding existing components.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

No branches or pull requests

5 participants