Skip to content

test(acceptance): add setupApache configs to 10.16#41551

Merged
DeepDiver1975 merged 1 commit into
10.16from
backport-setupapache-to-10.16
May 19, 2026
Merged

test(acceptance): add setupApache configs to 10.16#41551
DeepDiver1975 merged 1 commit into
10.16from
backport-setupapache-to-10.16

Conversation

@DeepDiver1975
Copy link
Copy Markdown
Member

Summary

  • Backports tests/acceptance/setupApache/single-server.conf and dual-server.conf verbatim from master
  • No existing files modified

Why

owncloud/reusable-workflows acceptance.yml hardcodes a cp from tests/acceptance/setupApache/ relative to the checked-out core directory. When an
OC10-targeted app sets core-branch-for-test-code: '10.16', this step fails with exit code 1 because the directory does not exist on the 10.16 branch. The
configs were introduced on master after 10.16 was cut.

Test plan

  • Verify the two files are present and identical to their master counterparts
  • Confirm an app using core-branch-for-test-code: '10.16' can reach and pass the "Configure Apache" step in acceptance tests

🤖 Generated with Claude Code

Backport of tests/acceptance/setupApache/ from master (verbatim).
The owncloud/reusable-workflows acceptance.yml hardcodes:

  sudo cp tests/acceptance/setupApache/single-server.conf ...

when core-branch-for-test-code is set to 10.16, causing a fatal
exit-code-1 failure because the directory was absent on this branch.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
@DeepDiver1975 DeepDiver1975 requested a review from phil-davis May 18, 2026 23:02
@DeepDiver1975
Copy link
Copy Markdown
Member Author

needed for owncloud/richdocuments#600

@DeepDiver1975 DeepDiver1975 merged commit da530ed into 10.16 May 19, 2026
1 check passed
@DeepDiver1975 DeepDiver1975 deleted the backport-setupapache-to-10.16 branch May 19, 2026 07:56
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