crashappsec / crashappsec/chalk

add support for docker commit command

Open
#175 0 comments 0 reactions 0 assignees View on GitHub
docker
Dominant language
Nim
Stars
436
Forks
25
PR merge metrics
No merged PRs in 30d

Description

## Description

There are 2 ways to build an image in docker:

* ```
docker build ...
```
* ```
container=$(docker run ...)
# do stuff in container
docker stop $container
docker commit $container
```

Currently chalk only supports first approach which adds chalk metadata to the built image. We should add support for doing something similar for the second case.

## Result

Created images from running containers will be chalked which will include repo metadata/etc

## Wrapping

The complexity will be how to make it work with entrypoint wrapping/etc. But that can be done with intermediate docker build internally:

```
container=$(docker run ...)
# do stuff in container
docker stop $container
chalk docker commit $container

# internally:
docker commit $container chalk-tmp
docker build . -f - < <(cat <

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.