Skip to content
This repository was archived by the owner on Jun 22, 2024. It is now read-only.

small addition to readme to confirm graviton compatibility #13

Merged
merged 1 commit into from
Jul 10, 2022

Conversation

triesztui
Copy link

No description provided.

@jamesmortensen
Copy link

Hi @triesztui Thanks for the contribution. I just created an AWS Graviton instance. Do you mind documenting the steps on how to set it up, and I can validate it?

Haven't tried yet, but I assume you're doing something like:

$ docker run --rm -it -p 4444:4444 -p 7900:7900 --shm-size 2g seleniarm/standalone-chromium:latest

and then opening up the necessary ports 4444/7900 through the firewall?

Also, are you doing anything to secure the instance from outside abuse?

Thanks again,
James

@triesztui
Copy link
Author

Hi @jamesmortensen ,

I actually use an EKS cluster with additional graviton worker node group. Then I used the "official" selenium grid helm chart:
https://github.com/SeleniumHQ/docker-selenium/tree/trunk/charts/selenium-grid

and just updated the images to point to yours.

I only tested the hub and firefox ones, as I would like to keep covering the google chrome browser, not the chromium one. (I used taints and node selectors to make sure these containers will run on the graviton workers and nothing else.)

relevant extract from the values

    hub:
      # Selenium Hub image name
      # imageName: selenium/hub
      imageName: seleniarm/hub

...

    firefoxNode:
      # Enable firefox nodes
      enabled: true
      # Number of firefox nodes
      replicas: 3
      # Image of firefox nodes
      # imageName: selenium/node-firefox
      imageName: seleniarm/node-firefox

thank YOU very much for producing these images. I believe the arm use case is important cost and carbon footprint saving opportunity so I do hope the selenium folks will start publishing these images soon as well on the official channel.

@jamesmortensen
Copy link

Hi @triesztui do you think it would make sense to also add the above sample configuration (For an arm hub and 3 arm Firefox) to the readme, or do you think it is enough to just mention it works on AWS?

@jamesmortensen jamesmortensen merged commit 37b1e2b into seleniumhq-community:trunk Jul 10, 2022
@jamesmortensen
Copy link

Hi @triesztui I'll go ahead and merge this. Thank you!

@triesztui
Copy link
Author

Hi @triesztui I'll go ahead and merge this. Thank you!

Thank you James. I was thinking the easiest way to include example but I think it would be outside of the scope of this repo to elaborate on a k8s setup here. Maybe a dedicated example repo somewhere else we can just refer to.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants