Skip to content

Fix build - use only apache allowed actions#58

Merged
benweidig merged 5 commits intoapache:masterfrom
derkoe:ci/change-firefox-setup
May 7, 2026
Merged

Fix build - use only apache allowed actions#58
benweidig merged 5 commits intoapache:masterfrom
derkoe:ci/change-firefox-setup

Conversation

@derkoe
Copy link
Copy Markdown
Contributor

@derkoe derkoe commented May 7, 2026

Firefox is already installed in ubuntu-latest so we just have to install the geckodriver

This will remove the dependency on browser-actions which are not in the allowlist. See apache/infrastructure-actions#813

Firefox is already installed in ubuntu-latest so we just have to install the geckodriver
@benweidig
Copy link
Copy Markdown
Contributor

Hi @derkoe!

Just yesterday at the Apache Infrastructure roundtable one of the topics was reviewing GitHub actions.

According to the runner-images the gecko driver 0.36.0 is already available in ubuntu-latest at /usr/local/share/gecko_driver: https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2404-Readme.md (around in the middle of the document)

I have to check if it's correctly loaded or if we have to do some configuration in SeleniumTestCase.

@derkoe
Copy link
Copy Markdown
Contributor Author

derkoe commented May 7, 2026

@benweidig done that - also verified that in my testing repo https://github.com/derkoe/testing-firefox-install

@benweidig
Copy link
Copy Markdown
Contributor

Quick ask before I start building a test setup: do you have an PR with Actions active, to see if it works with the existing Selenium tests?

If not, I'd test it later today and merge if successful.

@derkoe
Copy link
Copy Markdown
Contributor Author

derkoe commented May 7, 2026

checking that here: derkoe#2

The action in this repo is not running with "Startup failure": https://github.com/apache/tapestry-5/actions/runs/25489205311

Seems like soma Apache rules are blocking that.

@derkoe derkoe changed the title build - remove depenendecy on browser-actions Fix build - use only apache allowed actions May 7, 2026
@derkoe
Copy link
Copy Markdown
Contributor Author

derkoe commented May 7, 2026

Okay - found it: used the pinned versions of the actions and now the build works.

@benweidig benweidig merged commit 38df86a into apache:master May 7, 2026
2 checks passed
potiuk pushed a commit to raboof/infrastructure-actions that referenced this pull request May 7, 2026
As Tapestry was the only one using them, and they
moved away from it.

apache#813
apache/tapestry-5#58
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