Skip to content

GitHub registry#6

Open
PhilWun wants to merge 6 commits into
OpenTOSCA:propUifrom
PhilWun:propUi
Open

GitHub registry#6
PhilWun wants to merge 6 commits into
OpenTOSCA:propUifrom
PhilWun:propUi

Conversation

@PhilWun

@PhilWun PhilWun commented Mar 23, 2023

Copy link
Copy Markdown

This PR replaces Docker Hub with the GitHub registry in the GitHub workflow and updates the used GitHub actions.

Needed for OpenTOSCA/opentosca-docker#57

@PhilWun PhilWun requested a review from wederbn March 23, 2023 16:18
Comment thread Dockerfile
FROM node:12.7-alpine AS build
WORKDIR /usr/src/app
COPY package.json package-lock.json ./
RUN apk add --no-cache python2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need additional dependencies just because we change the registry?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is a fix to be able to build it on the arm64 architecture. I can't remember why it was necessary, because that commit is almost a year old.

uses: docker/build-push-action@v4
with:
context: .
platforms: linux/amd64 #,linux/arm64

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any idea what goes wrong for ARM?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On my machine building for arm64 works, but in the GitHub action it hangs. Maybe their emulation has some problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants