-
Notifications
You must be signed in to change notification settings - Fork 31
Conversation
Signed-off-by: Ernesto Ojeda <[email protected]>
@cloudxxx8 @cherrycl please review this one at your earliest convenience. I don't have permissions to add reviewers on this repo. |
I setup a test job on the sandbox for this PR and it is still failing with the |
…hat command fails, use kong migrations up Signed-off-by: Ernesto Ojeda <[email protected]>
Signed-off-by: Ernesto Ojeda <[email protected]>
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.
LGTM
curious why EdgeX couldn't use |
Signed-off-by: Ernesto Ojeda <[email protected]>
I am not sure how to answer this except maybe that at the time that this repository was setup the arm64 image did not exist and someone pointed us to this Dockerfile by someone in the community: https://github.com/svenwal/kong-arm/blob/master/Dockerfile. Anyway, I switched everything over to the Here is the job running on the sandbox: |
I am updating the arm64_env file to update the docker kong image to the latest arm64 based version. This is an attempt to fix the
exec format error
due to the previous image being based on 32bit arm. The build of the image can be found here: https://jenkins.edgexfoundry.org/view/CI-Build-Images/job/ci-build-images-kong-merge-pipeline/64/Signed-off-by: Ernesto Ojeda [email protected]