openclarity / apiclarity

An API security tool to capture and analyze API traffic, test API endpoints, reconstruct Open API specification, and identify API security risks. 
https://apiclarity.io
Apache License 2.0
501 stars 64 forks source link

Aduminuc cross build #260

Closed aduminuc closed 1 year ago

aduminuc commented 1 year ago

Add Makefile target that allows to build APIClarity from a Mac into a x86 image to use in k8s

FrimIdan commented 1 year ago

@aduminuc Not sure I understand why it is needed, the images that is being built via make push-docker-backend from Mac can run on k8s AFAIK. By default docker build build images for linux/amd64 platform.

aduminuc commented 1 year ago

@aduminuc Not sure I understand why it is needed, the images that is being built via make push-docker-backend from Mac can run on k8s AFAIK. By default docker build build images for linux/amd64 platform.

@FrimIdan I just got a new M1 Mac and in my case the docker images are built for arm and they won't run on k8s.

FrimIdan commented 1 year ago

@FrimIdan I just got a new M1 Mac and in my case the docker images are built for arm and they won't run on k8s.

@aduminuc export DOCKER_DEFAULT_PLATFORM=linux/amd64 is not working? https://docs.docker.com/engine/reference/commandline/cli/

aduminuc commented 1 year ago

@FrimIdan I just got a new M1 Mac and in my case the docker images are built for arm and they won't run on k8s.

@aduminuc export DOCKER_DEFAULT_PLATFORM=linux/amd64 is not working? https://docs.docker.com/engine/reference/commandline/cli/

@FrimIdan good point, I started this because I wanted to support multiple platforms at the same time (for which you do need buildx) , then I switched for a single one in which case you are right and there is no need of buildx. I am closing this, we may consider to do multiple platform builds in the future. Thanks!