ci: automate upgrade testing#1019
Open
nabim777 wants to merge 22 commits into
Open
Conversation
nabim777
force-pushed
the
upgrade-testing-local-setup
branch
2 times, most recently
from
b714a55 to
48467b1
Compare
nabim777
force-pushed
the
upgrade-testing-local-setup
branch
18 times, most recently
from
e7f22dd to
c696b82
Compare
nabim777
requested review from
Kharonus,
ba1ash,
dominic-braeunlein,
individual-it,
julien-nc,
lindenthal and
psatyal
as code owners
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
nabim777
force-pushed
the
upgrade-testing-local-setup
branch
from
1479aa0 to
d3fd391
Compare
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
nabim777
force-pushed
the
upgrade-testing-local-setup
branch
from
d3fd391 to
4e77495
Compare
saw-jan
reviewed
May 25, 2026
saw-jan
reviewed
May 25, 2026
Collaborator
individual-it
left a comment
individual-it
left a comment
There was a problem hiding this comment.
LGTM except for the things already addressed by @saw-jan
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
nabim777
force-pushed
the
upgrade-testing-local-setup
branch
from
b7bcc14 to
c2bd49f
Compare
There was a problem hiding this comment.
Pull request overview
Adds a new GitHub Actions workflow to exercise the Nextcloud app upgrade path in CI, including building an “upgradable” app artifact, upgrading it on a running Nextcloud container, and running API tests post-upgrade.
Changes:
- Added a new
App Upgradeworkflow that provisions a Nextcloud service, builds an upgrade artifact, performs an in-container upgrade, and runs API tests. - Added a new script to build/sign/package an “upgradable”
integration_openprojectapp and generate local-appstore metadata (apps.json) for upgrade testing. - Updated acceptance test cleanup to skip Docker-based user data deletion when the Nextcloud container is not present.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
tests/acceptance/features/bootstrap/FeatureContext.php |
Adds a guard to skip Docker cleanup when the nextcloud container is absent. |
.github/workflows/app-upgrade.yml |
Introduces the upgrade-testing workflow and job matrix. |
.github/scripts/build-upgradable-app.sh |
Builds/signs a new app version for upgrade testing and generates local appstore metadata. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
saw-jan
reviewed
May 25, 2026
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
nabim777
force-pushed
the
upgrade-testing-local-setup
branch
from
9a413da to
1bfe875
Compare
nabim777
force-pushed
the
upgrade-testing-local-setup
branch
3 times, most recently
from
6e33eac to
c97dbea
Compare
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
PHP Code CoverageCoverage after merging upgrade-testing-local-setup into release/2.11 will be
Coverage Report
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds an upgrade testing workflow.
It validates the full upgrade path by:
Tests run across multiple Nextcloud and PHP version combinations.
Available env:
NEXTCLOUD_VERSIONS: Nextcloud versions to test.
PHP_VERSIONS: PHP versions to test.
For now, this workflow runs on every pull request.
The open question is:
Related Issue or Workpackage
Screenshots (if appropriate):
Types of changes
Checklist:
CHANGELOG.mdfile