Skip to content

ci: fix upgrade testing on 2.11#1041

Open
nabim777 wants to merge 2 commits into
masterfrom
fix-ci/upgrade-testing
Open

ci: fix upgrade testing on 2.11#1041
nabim777 wants to merge 2 commits into
masterfrom
fix-ci/upgrade-testing

Conversation

@nabim777
Copy link
Copy Markdown
Collaborator

@nabim777 nabim777 commented Jun 8, 2026

Description

This PR fixes the failure in the Upgrade testing CI job.
In order to fix the CI, the following changes are done:

  • Added the missing checkout step with the input ref.
  • Skipped upgrade testing for the master branch because no app versions compatible with the Nextcloud master branch are available in the App Store, which is causing the following failure:
 occ app:enable groupfolders
    App "Team Folders" cannot be installed because it is not compatible with this version of the server.
 🔄 retrying
 occ app:enable user_oidc
    App "OpenID Connect user backend" cannot be installed because it is not compatible with this version of the server.
 🔄 retrying
 occ app:enable integration_openproject
    App "OpenProject Integration" cannot be installed because it is not compatible with this version of the server.
 🔄 retrying

Related Issue or Workpackage

  • Fixes <issue_link>

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Updated CHANGELOG.md file

@nabim777 nabim777 force-pushed the fix-ci/upgrade-testing branch from 8031ecf to f473e27 Compare June 8, 2026 06:37
@nabim777 nabim777 self-assigned this Jun 8, 2026
@nabim777 nabim777 force-pushed the fix-ci/upgrade-testing branch 3 times, most recently from 5dfc6af to 50e36cc Compare June 8, 2026 07:11
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Comment thread .github/workflows/app-upgrade.yml Outdated
- name: Wait for Nextcloud server to be ready
run: |
if ! timeout 5m bash -c '
if ! timeout 8m bash -c '
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to check why server not ready within timeout. I see these in the logs

 occ app:enable groupfolders
    App "Team Folders" cannot be installed because it is not compatible with this version of the server.
 🔄 retrying
 occ app:enable user_oidc
    App "OpenID Connect user backend" cannot be installed because it is not compatible with this version of the server.
 🔄 retrying
 occ app:enable integration_openproject
    App "OpenProject Integration" cannot be installed because it is not compatible with this version of the server.
 🔄 retrying

Copy link
Copy Markdown
Collaborator Author

@nabim777 nabim777 Jun 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No app versions compatible with the Nextcloud master branch are available in the App Store, which is causing this ⬆️ failure.

I think we need to build the apps (similar to integration_openproject) for all required apps to support upgrade testing against the master branch.

For now, let's skip this for the master branch and work in seperate PR.

What do you prefer, @saw-jan dai?

@nabim777 nabim777 requested a review from saw-jan June 8, 2026 10:32
@nabim777 nabim777 force-pushed the fix-ci/upgrade-testing branch from e36596d to 7fa1f73 Compare June 8, 2026 11:29
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
@nabim777 nabim777 force-pushed the fix-ci/upgrade-testing branch from 7fa1f73 to b951e0a Compare June 8, 2026 11:33
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 8, 2026

JS Code Coverage

Coverage after merging fix-ci/upgrade-testing into master will be
92.50%
Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   adminSettings.js0%0%0%0%1, 1, 10–19, 2–9
   bootstrap.js0%0%0%0%1, 1, 10–12, 2–9
   dashboard.js0%0%0%0%1, 1, 10–19, 2–9
   personalSettings.js0%0%0%0%1, 1, 10–19, 2–9
   projectTab.js0%0%0%0%1, 1, 10–19, 2, 20–29, 3, 30–37, 4–9
   reference.js0%0%0%0%1, 1, 10–19, 2, 20–29, 3, 30–39, 4, 40–45, 5–9
   utils.js85.92%33.33%50%87.59%12–20, 23–32
src/api
   endpoints.js100%100%100%100%
   settings.js64.71%100%0%73.33%10–11, 14–15
src/components
   AdminSettings.vue96.06%94.74%97.10%96.19%1003–1005, 1025–1028, 568, 568–569, 569, 606–612, 715–716, 720–721, 724–725, 729–730, 740–745, 788–790, 802–805, 818–820, 842–844, 927–929, 964–967
   ErrorLabel.vue100%100%100%100%
   OAuthConnectButton.vue91.85%75%100%93.28%64–69, 72–76
   PersonalSettings.vue92.02%95.65%90%91.71%133–134, 144–149, 152–161
src/components/admin
   FieldValue.vue100%100%100%100%
   FormAuthMethod.vue98.12%96.88%100%98.12%222–224, 247–250
   FormHeading.vue100%100%100%100%
   FormOpenProjectHost.vue98.87%94.74%100%99.34%167–169, 279
   FormSSOSettings.vue98.57%97.18%96.55%98.91%231–233, 242–243, 358–359
   TermsOfServiceUnsigned.vue100%100%100%100%
   TextInput.vue100%100%100%100%
src/components/icons
   ClippyIcon.vue100%100%100%100%
   OpenProjectIcon.vue100%100%100%100%
src/components/settings
   CheckBox.vue100%100%100%100%
   ErrorNote.vue100%100%100%100%
   SettingsTitle.vue96.91%85.71%100%97.67%51–53
src/components/tab
   EmptyContent.vue98.99%95.65%100%99.41%107–108
   SearchInput.vue95.31%92.96%94.74%95.78%138–139, 192, 203–208, 267–269, 285–287, 291–296
   WorkPackage.vue86.25%73.17%93.33%87.62%107–116, 129–131, 142–146, 156–158, 176–182, 220, 220–225, 225, 225–236, 81–82
src/constants
   appID.js100%100%100%100%
   links.js100%100%100%100%
   messages.js100%100%100%100%
src/filesPlugin
   filesPlugin.js0%0%0%0%1, 1, 10, 100–109, 11, 110–113, 12–19, 2, 20–29, 3, 30–39, 4, 40–49, 5, 50–59, 6, 60–69, 7, 70–79, 8, 80–89, 9, 90–99
src/utils
   workpackageHelper.js93.80%93.10%88.89%94.24%100–102, 23–27, 54, 54–56, 97–99
src/views
   CreateWorkPackageModal.vue94.18%86.32%90.48%95.43%367–372, 375, 391, 508–511, 516–521, 526–531, 537–540, 543, 559, 559, 600–604, 614–616, 639–640, 648–650, 679–681, 703–705, 714–718
   Dashboard.vue92.96%92.86%82.61%93.77%120–125, 134, 144, 147, 158–160, 214–217, 220–221, 228–232, 67
   LinkMultipleFilesModal.vue99.14%97.56%100%99.32%157–159
   ProjectsTab.vue94.95%94.34%93.75%95.10%104–107, 143, 154–155, 189–199, 248–250

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.

2 participants