Skip to content

Conversation

@nimmolo
Copy link
Contributor

@nimmolo nimmolo commented Dec 1, 2022

Heavily commented new Vagrantfile for building an upgraded box, a work in progress.

Adds Ubuntu Firefox for Capybara Selenium webdriver testing!
Needs proofreading.

@mo-nathan It seems that Virtualbox may now work on M1
https://osxdaily.com/2022/10/22/you-can-now-run-virtualbox-on-apple-silicon-m1-m2/
https://download.virtualbox.org/virtualbox/7.0.4/VirtualBox-7.0.4_BETA4-154605-macOSArm64.dmg

@nimmolo nimmolo requested a review from mo-nathan December 1, 2022 02:48
@JoeCohen
Copy link
Member

JoeCohen commented Dec 1, 2022

  1. Is the reason for the upgrade to use Selenium for testing?
  2. Does it make sense/is it possible to grab the Ruby version from the production server (instead of freezing it at 3.1.2)?
  3. Do we need to make changes to the firewall? Also note Docker issue:

nftables is now the default backend for the firewall. All applications on the system must agree on whether they will use the legacy xtables backend or the newer nftables backend. Bug 1968608 96 provides some context that may be helpful. Docker may not be ready for the new nftables backend 342.

https://discourse.ubuntu.com/t/jammy-jellyfish-release-notes/24668

@pellaea
Copy link
Member

pellaea commented Dec 1, 2022

  1. Yes, Selenium is what prompted this.
  2. We shouldn't need to make changes to the firewall, since the browser will be headless. No need to, for example, tunnel X windows through the firewall to give the VM access to our display.

@JoeCohen
Copy link
Member

JoeCohen commented Dec 1, 2022 via email

@pellaea
Copy link
Member

pellaea commented Dec 1, 2022 via email

@nimmolo
Copy link
Contributor Author

nimmolo commented Dec 1, 2022 via email

@pellaea
Copy link
Member

pellaea commented Dec 1, 2022 via email

@JoeCohen
Copy link
Member

JoeCohen commented Dec 1, 2022 via email

@nimmolo nimmolo marked this pull request as draft December 2, 2022 02:11
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