#24353 Fix Varnish 6 Too many restarts issue

[ihor-sviziev]

Description (*)

This PR applies fix from #24353 (comment), it was confirmed that it fixes issue in #24353 (comment).

Related Pull Requests

Fixed Issues (if relevant)

1. Fixes Magento varnish 6 Too many restarts #24353

Manual testing scenarios (*)

1. See Magento varnish 6 Too many restarts #24353

Questions or comments

Contribution checklist (*)

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/integration tests (if applicable)
• All automated tests passed successfully (all builds are green)

[m2-assistant]

Hi @ihor-sviziev. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

• @magento give me test instance - deploy test instance based on PR changes
• @magento give me 2.4-develop instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Guide documentation.

[Hanhan1989]

The functional Tests B2B failed!

[ihor-sviziev]

The functional Tests B2B failed!

Looks like false positive

[Hanhan1989]

The functional Tests B2B failed!

Looks like false positive

I think so. The error was the following:

SUITE PRECONDITION FAILED:
Can't connect to Webdriver at http://localhost:5555/wd/hub. Please make sure that Selenium Server or PhantomJS is running.

[ihor-sviziev]

@lbajsarowicz maybe you could review my PR?

[lbajsarowicz]

✔️ According to the documentation, this change is fine.

I'd love to see review from someone who has more expertise with Varnish,.

[magento-engcom-team]

Hi @lbajsarowicz, thank you for the review.
ENGCOM-7761 has been created to process this Pull Request
✳️ @lbajsarowicz, could you please add one of the following labels to the Pull Request?

Label	Description
Auto-Tests: Covered	All changes in Pull Request is covered by auto-tests
Auto-Tests: Not Covered	Changes in Pull Request requires coverage by auto-tests
Auto-Tests: Not Required	Changes in Pull Request does not require coverage by auto-tests

[engcom-Alfa]

✔️ QA Passed

Preconditions:

Magento 2.4-develop;
Sample Data;
Php 7.3;
Varnish v. 6.2

Manual testing scenario:

1. Configure Varnish and Web server (Apache 2 in my case) devdocs;
2. Configure Magento to use Varnish

Go to Admin->Stores->Configuration->System->Full Page Cache:

• set Cache Application to Varnish Cache;
• set TTL for public content to 120 sec;
  

3. Export Varnish 6 VCL, configure and restart varnish service;
4. Run varnishlog in console;
5. Go to Storefront and try browsing the site for longer than 10min.

Before: ✖️ VCL keeps restarting forever resulting in: - VCL_Error Too many restarts after a few tries
503 Response status and VCL error: Too many restarts

After: ✔️ No 503 errors caused by restart. Everything works as expected

[m2-assistant]

Hi @ihor-sviziev, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.