Skip to content

Conversation

johnstcn
Copy link
Member

@johnstcn johnstcn commented Aug 8, 2024

No description provided.

maxbrunet and others added 30 commits July 5, 2024 18:03
Closes #260

- Removes manually vendored codersdk, we're back on the real deal now
- Moves logging setup to package internal/eblog
- Modifies logging to use new agent api v2 methods, falling back to PatchLogs
- Adds ENVBUILDER_VERBOSE

Tested manually against Coder version 2.8.0, 2.9.0, and 2.13.0.

(cherry picked from commit 5be0611)
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
(cherry picked from commit 309464b)
…y default (#274)

Signed-off-by: Cian Johnston <[email protected]>
(cherry picked from commit 244c3ec)
…a6fdf5a28c to 0.2.0 (#270)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Cian Johnston <[email protected]>
(cherry picked from commit ac3c34c)
Signed-off-by: Cian Johnston <[email protected]>
(cherry picked from commit cc0b4c5)
Signed-off-by: Cian Johnston <[email protected]>
(cherry picked from commit 09ce456)
Builds on top of #282 and #283:

- Extracts the logic for --get-cached-image to a separate function
- Also pulls out some other common logic shared between Run and RunCacheProbe.

(cherry picked from commit cacbcb8)
(cherry picked from commit 9073748)
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
(cherry picked from commit 2b457fe)
- Extracts 'magic directives' to constants.go.
- Adds 'magic image' file to signify envbuilder should skip the destructive 'build image' stage.
- Modifies logic for copying binary into built image: we now copy to build context and remove it after build finishes to avoid leaving around files owned by root:root. Also ensures files are created with consistent permissions.

(cherry picked from commit 6afe89e)
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…0.2 (#302)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
mafredri and others added 28 commits September 26, 2024 10:28
…incompatible (#329)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
(cherry picked from commit bb53f2f)
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
(cherry picked from commit 754b9dd)
- Unexports constants.MagicDir and moves it to an internal package.
- Moves other const/var declarations in `constants` package to where they are used
- Removes constants package
- Adds MagicDirBase to options (intentionally not configurable via CLI)
- Removes usages of default MagicDir to instead use path provided by options

Co-authored-by: Danny Kopping <[email protected]>
Co-authored-by: Mathias Fredriksson <[email protected]>
(cherry picked from commit e6c8c66)
Co-authored-by: Cian Johnston <[email protected]>
(cherry picked from commit df8ea67)
Fixes #364

Adds the capability for Run() to run defers defined outside of Run().
Currently the only use-case for this is to close the Coder logger.
Also adds an integration test that validates this behaviour and ensures that closeLogs gets called.

Co-authored-by: Mathias Fredriksson <[email protected]>
(cherry picked from commit 481e3a9)
Co-authored-by: Mathias Fredriksson <[email protected]>
(cherry picked from commit 7fe1900)
Co-authored-by: Cian Johnston <[email protected]>
Co-authored-by: Muhammad Atif Ali <[email protected]>
(cherry picked from commit 63a383b)
(cherry picked from commit 4f3e9cd)
Co-authored-by: Marcin Tojek <[email protected]>
(cherry picked from commit 845f112)
@mafredri mafredri closed this Oct 2, 2024
@mafredri
Copy link
Member

mafredri commented Oct 2, 2024

Closing this as we're releasing from the release/1.0 branch. When it's time for v1.1, we'll fork main and start a new release branch (release/1.1 as we do with coder/coder).

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