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

feat: enable building images in docker [image building pt. 1] #1911

Merged
merged 18 commits into from
Dec 8, 2023

Conversation

tedim52
Copy link
Contributor

@tedim52 tedim52 commented Dec 7, 2023

Description:

This is pt. 1 of the image building work. Specifically, this PR enables building images in container-engine-lib by adding BuildImage to the KurtosisBackend and implements it in DockerKurtosisBackend.

Is this change user facing?

NO

@tedim52
Copy link
Contributor Author

tedim52 commented Dec 7, 2023

@laurentluce builds are failing in CI, but everything builds locally rn bc I'm using a package that relies on go 1.20 , @lostbean is updating us to go 1.20 right now so this should be fixed in about an hour. I requested you if u want to get started in the meantime

@tedim52 tedim52 requested a review from laurentluce December 7, 2023 19:43
@laurentluce
Copy link
Contributor

laurentluce commented Dec 7, 2023

@tedim52 tedim52 changed the title feat: enable building images in docker feat: enable building images in docker [image building pt. 1] Dec 7, 2023
@tedim52 tedim52 added this pull request to the merge queue Dec 8, 2023
Merged via the queue into main with commit c153873 Dec 8, 2023
11 checks passed
@tedim52 tedim52 deleted the tedi/build-image branch December 8, 2023 01:34
github-merge-queue bot pushed a commit that referenced this pull request Dec 11, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.85.43](0.85.42...0.85.43)
(2023-12-11)


### Features

* Add new ports view to the EM UI
([#1919](#1919))
([027d74b](027d74b))
* add REST API bindings for TS and Golang
([#1907](#1907))
([97b9b80](97b9b80))
* add support for public ports
([#1905](#1905))
([97a3d95](97a3d95))
* enable building images in docker [image building pt. 1]
([#1911](#1911))
([c153873](c153873))
* Reverse proxy lifecycle management and connectivity on Docker
([#1906](#1906))
([69c5b27](69c5b27))
* service logs full download
([#1895](#1895))
([b91333f](b91333f))
* Unified REST API
([c3911f6](c3911f6))


### Bug Fixes

* add installation description for oapi-codegen
([#1917](#1917))
([8f2427b](8f2427b))
* Fix doc checker CI
([#1912](#1912))
([cc2696d](cc2696d))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: kurtosisbot <[email protected]>
h4ck3rk3y added a commit that referenced this pull request Dec 12, 2023
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