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

Unpin openjdk-11 version in blobstore package#64045

Merged
willdollman merged 2 commits into
mainfrom
will/blobstore-unpin
Jul 24, 2024
Merged

Unpin openjdk-11 version in blobstore package#64045
willdollman merged 2 commits into
mainfrom
will/blobstore-unpin

Conversation

@willdollman

Copy link
Copy Markdown
Contributor

We previously pinned the version of openjdk-11 in order to avoid a bad package signature. This is no longer required.

Test plan

  • CI
  • Manually verify updated package

Changelog

@willdollman willdollman self-assigned this Jul 24, 2024
@cla-bot cla-bot Bot added the cla-signed label Jul 24, 2024
@willdollman willdollman requested a review from a team July 24, 2024 15:40
@willdollman willdollman merged commit 092b8be into main Jul 24, 2024
@willdollman willdollman deleted the will/blobstore-unpin branch July 24, 2024 15:57
willdollman referenced this pull request Jul 24, 2024
<!-- PR description tips:
https://www.notion.so/sourcegraph/Write-a-good-pull-request-description-610a7fd3e613496eb76f450db5a49b6e
-->
Update the version of openjdk-11 we use in the blobstore image.

We updated the blobstore image earlier today to try and fix this issue,
but were thwarted by an old version pin. This has now been
[removed](https://github.com/sourcegraph/sourcegraph/pull/64045), so
updating packages fully resolves the issue.

This PR branch is not based off `main` as backporting package changes
often results in merge conflicts. It was generated by checking our
`5.5.x` and running `sg wolfi lock blobstore`.

## Test plan

- CI
- Run image locally

<!-- REQUIRED; info at
https://docs-legacy.sourcegraph.com/dev/background-information/testing_principles
-->

## Changelog

<!-- OPTIONAL; info at
https://www.notion.so/sourcegraph/Writing-a-changelog-entry-dd997f411d524caabf0d8d38a24a878c
-->
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants