Skip to content

M2 / ARM builds - #112

Merged
fils merged 1 commit into
devfrom
dev-mm--arm-build
Oct 22, 2022
Merged

fils merged 1 commit into
devfrom
dev-mm--arm-build

Conversation

@nein09

@nein09 nein09 commented Sep 27, 2022

Copy link
Copy Markdown
Collaborator

I'm trying a private image out (pushed to my own Docker repo) right now and it's working great. I'll edit the GitHub action here too, if this looks good.

@nein09
nein09 requested review from fils and valentinedwv September 27, 2022 16:26
@nein09
nein09 changed the base branch from master to dev September 27, 2022 16:26
@nein09

nein09 commented Sep 27, 2022

Copy link
Copy Markdown
Collaborator Author

I thought Alpine wouldn't work for ARM64 but that turns out to be an issue specific to a different stack on Alpine (NodeJS) - Golang seems to be fine!

@nein09

nein09 commented Sep 27, 2022

Copy link
Copy Markdown
Collaborator Author

wait a second, it looks like @valentinedwv may have beat me to the punch with an multiarch action? Is that working now or should I have a look?

@nein09

nein09 commented Sep 27, 2022

Copy link
Copy Markdown
Collaborator Author

However, it turns out that chrome-headless-shell is doing miserably on ARM64, so I'll see what I can do about that..

@valentinedwv

valentinedwv commented Sep 27, 2022 •

Copy link
Copy Markdown
Member

However, it turns out that chrome-headless-shell is doing miserably on ARM64, so I'll see what I can do about that..

Don't worry, headless doing miserable on AMD64. wondering if browser less/chrome is a something to try: https://hub.docker.com/r/browserless/chrome

also looking at: https://stackoverflow.com/questions/30210362/how-to-increase-the-size-of-the-dev-shm-in-docker-container

@nein09

nein09 commented Sep 28, 2022

Copy link
Copy Markdown
Collaborator Author

Hmm, it looks like Browserless does support chromedp (as a Beta, but it does it): https://docs.browserless.io/docs/go.html

@fils fils left a comment

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.

cool! anxious to hear how it goes!

@fils
fils merged commit 9726a1c into dev Oct 22, 2022
@nein09
nein09 deleted the dev-mm--arm-build branch January 23, 2023 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants