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

Bug: Release download URL fix (#553) #31

Merged
merged 2 commits into from
Mar 3, 2025

Conversation

CodeBreaker444
Copy link
Contributor

@CodeBreaker444 CodeBreaker444 commented Mar 16, 2024

Description

While building the docker image from https://github.com/gitpod-io/openvscode-releases, the github release url for downloading the release of openvscode-server is not properly getting parsed (wrong release name) and throwing 404 not found error from github. Like in the below image!

1f355f1820cc02f6371b481760845d73

Related Issue(s)

Fixes #553

How to test

  1. Run docker build --tag "vscode" --build-arg RELEASE_TAG=v1.88.0 --build-arg RELEASE_TYPE=insiders .
  2. After this you run the docker image as usual.

Note: You don't need to pass RELEASE_TYPE argument if the release is a stable release.

Documentation

Copy link

@dav3yblaz3 dav3yblaz3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets get this to test thank you team this needed to be fixed

Copy link
Member

@filiptronicek filiptronicek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @CodeBreaker444 and thanks a lot for your contribution. Although we no longer use the Dockerfile for building OpenVSCode Server (and use https://github.com/gitpod-io/openvscode-releases/blob/main/.github/workflows/release.yml instead), this looks like a small enough change to ship, if it helps you build it (I unfortunately couldn't get your test command to work).

@filiptronicek filiptronicek merged commit 5b3afd2 into gitpod-io:main Mar 3, 2025
jeanp413 added a commit that referenced this pull request Mar 7, 2025
jeanp413 added a commit that referenced this pull request Mar 7, 2025
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.

3 participants