Skip to content
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

Lighthouse image for AET #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

malaskowski
Copy link
Owner

Work in progress

AET Lighthouse Docker image

Since there is still no official Lighthouse Docker image,
this one is forked https://github.com/GoogleChromeLabs/lighthousebot/tree/master/builder
and adjusted to AET needs.

Running

Execute:

docker run -dit -p 5000:5000 --rm --name lighthouse_aet --cap-add=SYS_ADMIN skejven/aet_lighthouse

Limitations

Lighthouse and Docker

Currently Lighthouse does not have official Docker image.
aet_lighthouse image is inspired by https://github.com/GoogleChromeLabs/lighthousebot/tree/master/builder.
What more, this image can't be deployed with Docker Swarm, because it requires SYS_ADMIN capabilities
which is not supported in swarm mode.

What is more, Lighthouse on Docker runs quite unstable... If you can use another Lighthouse instance, it will
probably be a good idea.

Max 1 url

Yes... This is due to lack of scaling Lighthouse instance. If you play with that module, please remember
to have max 1 url running it in your suite.

@malaskowski malaskowski added the help wanted Extra attention is needed label Aug 27, 2019
#
# AET Docker
#
# Copyright (C) 2019 Maciej Laskowski
Copy link
Owner Author

Choose a reason for hiding this comment

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

Suggested change
# Copyright (C) 2019 Maciej Laskowski
# Copyright (C) 2022 Maciej Laskowski

@malaskowski malaskowski force-pushed the master branch 5 times, most recently from 3e0cce0 to 9e7e3e0 Compare March 13, 2023 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant