Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

WIP: DevX Alpine package dependencies versioning Tracking issue #27942

Closed
jhchabran opened this issue Nov 19, 2021 · 4 comments · Fixed by #29493
Closed

WIP: DevX Alpine package dependencies versioning Tracking issue #27942

jhchabran opened this issue Nov 19, 2021 · 4 comments · Fixed by #29493
Assignees
Labels

Comments

@jhchabran
Copy link
Contributor

jhchabran commented Nov 19, 2021

Plan

Problem

We are pinning Alpine package versions down to the minor level, which can lead to breakage when the Alpine releases bump a given package.

See https://wiki.alpinelinux.org/wiki/Enable_Community_Repository#Using_specific_package_versions

Related documents

Availability

If you have planned unavailability this iteration (e.g., vacation), you can note that here.

Tracked issues

Legend

  • 🐛 Bug
  • 🧶 Technical debt
  • 🎩 Quality of life
  • 🔒 Security issue
@github-actions
Copy link
Contributor

Heads up @taylorsperry @pdubroy - the "team/dev-experience" label was applied to this issue.

@github-actions
Copy link
Contributor

Heads up @taylorsperry @pdubroy - the "team/dev-experience" label was applied to this issue.

@bobheadxi
Copy link
Member

I've gone ahead and relaxed the requirements here based on what I believe might be the "right" policy: https://github.com/sourcegraph/sourcegraph/pull/29493

Alpine packages only keep the most recent few releases, with older releases being dropped on a regular basis. We enforce versions, it seems, mostly to ensure the inclusion of particular security patches. Since not upgrading will only break our builds, and when we run into issues we just upgrade anyway, we should just only ever pin minimum versions.

I think we should make this official and close it out, WDYT @jhchabran @davejrt ?

@bobheadxi bobheadxi linked a pull request Jan 11, 2022 that will close this issue
@jhchabran
Copy link
Contributor Author

I think we should make this official and close it out

Agreed! Thank you so much for taking over this 🙏

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants