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

Workbench session image promotion #876

Merged
merged 32 commits into from
Nov 27, 2024
Merged

Workbench session image promotion #876

merged 32 commits into from
Nov 27, 2024

Conversation

bschwedler
Copy link
Contributor

@bschwedler bschwedler commented Nov 26, 2024

Build the workbench-session image.

skyeturriff and others added 30 commits November 1, 2024 15:50
…uild-fix-tests

Use env var for WORKBENCH_SESSION_INIT_VERSION or default to `get-version` to fetch version
…uild

Add release target for workbench-session-init image
Adds the WORKBENCH_SESSION_INIT variable to the manual build flow
Merge versions job into workbench-session-init job
* add go program

* add multi-stage build dockerfile

* add positron

* fix path

* update err msgs and reuse file info

* merge comments

* enhance file copying with otiai10/copy package and update Dockerfile

* remove uneeded components

* add shared_run to common deps

* fix component filename

* update Dockerfile to use ubuntu:22.04 base image, as the scratch image doesn't work nicely with GOSS

* remove test var

---------

Co-authored-by: Ian Pittwood <[email protected]>
@bschwedler bschwedler requested review from zachhannum and removed request for costrouc and ianpittwood November 26, 2024 19:46
@bschwedler bschwedler marked this pull request as draft November 26, 2024 20:08
skyeturriff and others added 2 commits November 27, 2024 09:06
temp removal of workbench-session-init from github actions release build
@bschwedler bschwedler marked this pull request as ready for review November 27, 2024 16:10
@bschwedler
Copy link
Contributor Author

@zachhannum is it OK if the preview/daily builds still run for the session init container? (as they did in the above workflow)

@zachhannum
Copy link
Contributor

@zachhannum is it OK if the preview/daily builds still run for the session init container? (as they did in the above workflow)

Yep, preview/daily builds are still OK! We are using them for QA prior to the release. Thanks!

1 similar comment
@zachhannum
Copy link
Contributor

@zachhannum is it OK if the preview/daily builds still run for the session init container? (as they did in the above workflow)

Yep, preview/daily builds are still OK! We are using them for QA prior to the release. Thanks!

@bschwedler bschwedler merged commit 94d4b6b into main Nov 27, 2024
100 of 102 checks passed
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.

4 participants