Skip to content

Conversation

@dennisvang
Copy link
Contributor

@dennisvang dennisvang commented May 19, 2025

fixes #691

@dennisvang dennisvang linked an issue May 19, 2025 that may be closed by this pull request
@dennisvang dennisvang force-pushed the backport/issue691 branch from 0a9b047 to a84e18e Compare May 19, 2025 19:29
@dennisvang dennisvang force-pushed the backport/issue691 branch from 5691238 to 3f35927 Compare May 19, 2025 20:10
to prevent 'Error: Command failed: sudo systemctl start mongod'

see ankane/setup-mongodb#6

looks like mongo 8 is the lowest version supported by ankane/setup-mongodb for ubuntu 24 runners:

https://github.com/ankane/setup-mongodb?tab=readme-ov-file#versions
@dennisvang dennisvang force-pushed the backport/issue691 branch from dd26a24 to 58bb369 Compare May 19, 2025 20:27
This reverts commit ce3177d.

the preliminary workflow resulted in successful tests, so we can patch the maven-verify workflow v2 (in the github-workflows repo)
@dennisvang
Copy link
Contributor Author

Updated maven-publish workflow to support mongodb on various runners, so tests should now succeed.

See FAIRDataTeam/github-workflows#24

to prevent error "no match for platform in manifest" when building for arm64 architecture
@dennisvang dennisvang force-pushed the backport/issue691 branch from 9df6f37 to 13a3d8f Compare May 20, 2025 09:02
because the option flags differ between alpine and focal

we can now use adduser to create both user and group in one go (see man adduser)
@dennisvang dennisvang force-pushed the backport/issue691 branch from 5c1d727 to 6e2b068 Compare May 20, 2025 09:49
@dennisvang
Copy link
Contributor Author

Tests pass and workflows run. These are otherwise equal to develop, so have been reviewed already.
The security check failure will be tackled in a separate PR for dependency upgrades.

@dennisvang dennisvang merged commit 1421927 into support/1.17.x May 20, 2025
12 of 13 checks passed
@dennisvang dennisvang deleted the backport/issue691 branch May 20, 2025 10:03
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.

Backport github/workflows cleanup to v1.17.x

2 participants