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

Upgrade Fuseki used in Skosmos tests and containers #1775

Open
4 of 5 tasks
osma opened this issue Mar 13, 2025 · 1 comment · May be fixed by #1778
Open
4 of 5 tasks

Upgrade Fuseki used in Skosmos tests and containers #1775

osma opened this issue Mar 13, 2025 · 1 comment · May be fixed by #1778
Assignees
Labels
maintenance Dependency changes, security updates, infrastructure tweaks & general mainenance
Milestone

Comments

@osma
Copy link
Member

osma commented Mar 13, 2025

We are currently using Jena Fuseki 4.8.0 for the Fuseki container:

JENA_VERSION: 4.8.0

This is a rather old release (released 2023-04-23) and we should upgrade to a newer version to ensure that Skosmos remains compatible with recent Fuseki. The most recent release is currently 5.3.0.

With previous Fuseki upgrades, we've had occasional problems with SPARQL queries that no longer return the same results or that take much longer to run than before. Some testing is needed to ensure this doesn't happen again with this upgrade.

Things to check:

  • upgrade the version number in docker-compose.yml
  • upgrade the Dockerfile used for Fuseki
  • run PHPUnit and Cypress tests
  • measure test execution time before/after locally
  • measure test execution time before/after on GitHub Actions CI

Related to #1700 and PR #1731; the EasyRdf upgrade is a bit stuck because of incompatibilities between recent EasyRdf and the Fuseki version currently used in Skosmos. Let's hope that upgrading Fuseki will enable upgrading EasyRdf as well.

@osma osma converted this from a draft issue Mar 13, 2025
@osma osma changed the title Upgrade Fuseki Upgrade Fuseki used in Skosmos tests Mar 13, 2025
@osma osma added the maintenance Dependency changes, security updates, infrastructure tweaks & general mainenance label Mar 13, 2025
@osma osma changed the title Upgrade Fuseki used in Skosmos tests Upgrade Fuseki used in Skosmos tests and containers Mar 13, 2025
@osma osma added this to the 3.0 milestone Mar 13, 2025
@osma osma self-assigned this Mar 13, 2025
@osma
Copy link
Member Author

osma commented Mar 13, 2025

Current test runtimes on the main branch running the tests on my laptop (Ubuntu 24.04, PHP 8.3.6, no coverage extensions):

PHPUnit tests:

Time: 00:46.081, Memory: 14.00 MB

There was 1 PHPUnit test runner warning:

1) No code coverage driver available

WARNINGS!
Tests: 518, Assertions: 997, Warnings: 1, Skipped: 3.

Cypress tests:

✔  All specs passed!                        01:12      150      147        -        3        - 

@osma osma linked a pull request Mar 13, 2025 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Dependency changes, security updates, infrastructure tweaks & general mainenance
Projects
Status: In progress
Development

Successfully merging a pull request may close this issue.

1 participant