Skip to content

feat: copy Linux packages to sign them with a new key #252529

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

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

rzhao271
Copy link
Contributor

@rzhao271 rzhao271 commented Jun 26, 2025

This PR changes the main publishing pipeline to create copies of the built Debian and RPM packages so that it can sign them with a new key from packages.microsoft.com (PMC). The new key is required for us to upload the packages to new PMC repositories, which in turn would allow Linux users on newer platforms to install VS Code without security warnings.

However, this change will not immediately make newer packages available to users because the artifacts will have to be made available on the builds and/or VS Code website.

Questions

  • Considering that this change will slow down the publishing pipeline, how do we want to handle newer package keys going forward?
  • When do we want to deprecate/remove support for the older package key?
  • How do we want to update the builds and/or VS Code websites to support these packages signed with the newer key?

@rzhao271 rzhao271 requested a review from joaomoreno June 26, 2025 18:19
@rzhao271 rzhao271 self-assigned this Jun 26, 2025
@rzhao271 rzhao271 added this to the June 2025 milestone Jun 26, 2025
@deepak1556 deepak1556 self-requested a review June 30, 2025 15:50
@deepak1556
Copy link
Collaborator

Will have a detailed review in debt week, but primarily we cannot republish the el8 package as el10, the release parameter needs to be bumped

Release: @@RELEASE@@.el8

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