Skip to content

IBX-12173: Added Varnish 9 support for Platform.sh/Upsun - #107

Open
vidarl wants to merge 3 commits into
4.6from
IBX-12173_support_for_varnish9
Open

IBX-12173: Added Varnish 9 support for Platform.sh/Upsun#107
vidarl wants to merge 3 commits into
4.6from
IBX-12173_support_for_varnish9

Conversation

@vidarl

@vidarl vidarl commented Aug 4, 2026

Copy link
Copy Markdown
Contributor
Question Answer
JIRA issue IBX-12173
Type feature
Target Ibexa version v4.6
BC breaks no

Caution

This PR cannot be merged up as-is. Talked to @barw4, and we agree to not merge this into 5.0 and up as it is deprecated and replaced by ibexa/cloud package

Adds .platform/varnish9.vcl and a commented-out varnish:9.0 service block in
.platform/services.yaml, under resources/platformsh/common/4.6/.

Related PRs:

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Provided automated test coverage.
  • Checked that target branch is set correctly (4.6, matching the existing
    resources/platformsh/common/4.6/ directory this feature extends).
  • Ran PHP CS Fixer for new PHP code. (N/A — no PHP changes)
  • Asked for a review (ping @ibexa/engineering).

@vidarl vidarl changed the title IBX-12173: Added Varnish 9 support for Platform.sh/Upsun (opt-in) IBX-12173: Added Varnish 9 support for Platform.sh/Upsun Aug 5, 2026
@vidarl
vidarl force-pushed the IBX-12173_support_for_varnish9 branch from 9f18621 to 55d45f6 Compare August 5, 2026 13:29
@vidarl
vidarl requested a review from a team August 5, 2026 13:30
Comment thread resources/platformsh/common/4.6/.platform/varnish9.vcl Outdated
@vidarl
vidarl requested a review from konradoboza August 6, 2026 07:31

@bnowak bnowak left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved to ublock topic, based on it's 1:1 copy between versions.
I don't know vcl much, so if you want to have more confidence, please wait for some more reviewers 😉

@vidarl

vidarl commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

This PR was approved before I added a new commit today, However, that new commit is only a cherry-pick of ibexa/http-cache#86, so I'll skip re-requesting a review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants