-
Notifications
You must be signed in to change notification settings - Fork 14
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
Fix doit tests #458
Fix doit tests #458
Conversation
@@ -55,21 +55,21 @@ The build system will attempt to detect the local architecture and automatically | |||
All commands `build`, `tests`, and `up` will use the locally detected platform and use a version tag based on the state of the local git repository. | |||
However, you can also specify a custom platform or version with the `--platform` and `--version` parameters, example: `doit build --arch=arm64 --version=my-version`. | |||
|
|||
You can specify target image variants to build with `--target`, example: `doit build --target base --target lab`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To simplify the code, you can now only provide a single --target
argument (and we now automatically validate that you choose one of the four valid images. If no target is specified, then all images are built by default.
@@ -15,23 +15,25 @@ | |||
|
|||
_ARCH_MAPPING = { | |||
"x86_64": "amd64", | |||
"amd64": "amd64", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is what podman returns on Fedora 39 at least.
I've merged this since changes here don't influence the build / image itself, and we can always tweak later. |
Fixes #451. You can now build images locally with e..g
doit build
and then run tests as
doit tests -t full-stack
I've also made some tweaks to support podman (and podman-compose) via extra
--compose-cmd
arguments. I've also fixed thedoit up
anddoit down
commands, which now have to include the-t/--target
argument.