[FrameworkBundle][HttpKernel] Introduce `$buildDir` argument to `WarmableInterface::warmup` to warm read-only artefacts in `build_dir` #50391

okhoshi · 2023-05-22T22:06:42Z

Q	A
Branch?	6.4
Bug fix?	no
New feature?	yes
Deprecations?	no
Tickets	Fix #50357
License	MIT
Doc PR	symfony/symfony-docs#...

See #50357 for the details and the reproduction steps. In this PR, I also moved ConfigBuilderCacheWarmer to use the buildDir argument.

nicolas-grekas

As discussed on Slack, I'd suggest adding a string $buildDir = null argument to the existing warmUp method (using a commented argument for BC). We'd pass null when only warming up, and the build-dir when building.

nicolas-grekas

Here are some comment, and my review as a patch :)

Now that we have this, I'm wondering about each warmers: are they doing something for build-time or compilte-time? what's their purpose, etc; E.g CompiledClassMetadataFactory added in #29117 by @fbourigault is never wired? We should likely investigate this in a follow up PR this and the others too.

diff --git a/src/Symfony/Bridge/Doctrine/CacheWarmer/ProxyCacheWarmer.php b/src/Symfony/Bridge/Doctrine/CacheWarmer/ProxyCacheWarmer.php
index b148b9347b..baedf9c033 100644
--- a/src/Symfony/Bridge/Doctrine/CacheWarmer/ProxyCacheWarmer.php
+++ b/src/Symfony/Bridge/Doctrine/CacheWarmer/ProxyCacheWarmer.php
@@ -40,9 +40,7 @@ class ProxyCacheWarmer implements CacheWarmerInterface
     }
 
     /**
-     * @param string|null $buildDir directory to put generated source code that can be marked as read-only at runtime.
-     *                              The directory is not provided when only cache_dir should be warmed up.
-     * @return string[] A list of files to preload on PHP 7.4+
+     * @param string|null $buildDir
      */
     public function warmUp(string $cacheDir /* , string $buildDir = null */): array
     {
diff --git a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/AbstractPhpFileCacheWarmer.php b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/AbstractPhpFileCacheWarmer.php
index 49fd7f035d..98c281276b 100644
--- a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/AbstractPhpFileCacheWarmer.php
+++ b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/AbstractPhpFileCacheWarmer.php
@@ -35,17 +35,16 @@ abstract class AbstractPhpFileCacheWarmer implements CacheWarmerInterface
     }
 
     /**
-     * @param string|null $buildDir directory to put generated source code that can be marked as read-only at runtime.
-     *                              The directory is not provided when only cache_dir should be warmed up.
-     * @return string[] A list of classes to preload on PHP 7.4+
+     * @param string|null $buildDir
      */
     public function warmUp(string $cacheDir /* , string $buildDir = null */): array
     {
+        $buildDir = 1 < \func_num_args() ? func_get_arg(1) : null;
         $arrayAdapter = new ArrayAdapter();
 
         spl_autoload_register([ClassExistenceResource::class, 'throwOnRequiredClass']);
         try {
-            if (!$this->doWarmUp($cacheDir, $arrayAdapter)) {
+            if (!$this->doWarmUp($cacheDir, $arrayAdapter, $buildDir)) {
                 return [];
             }
         } finally {
@@ -80,7 +79,9 @@ abstract class AbstractPhpFileCacheWarmer implements CacheWarmerInterface
     }
 
     /**
+     * @param string|null $buildDir
+     *
      * @return bool false if there is nothing to warm-up
      */
-    abstract protected function doWarmUp(string $cacheDir, ArrayAdapter $arrayAdapter): bool;
+    abstract protected function doWarmUp(string $cacheDir, ArrayAdapter $arrayAdapter /* , string $buildDir = null */): bool;
 }
diff --git a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/AnnotationsCacheWarmer.php b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/AnnotationsCacheWarmer.php
index 279dc4ec9e..20533bb60e 100644
--- a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/AnnotationsCacheWarmer.php
+++ b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/AnnotationsCacheWarmer.php
@@ -44,7 +44,10 @@ class AnnotationsCacheWarmer extends AbstractPhpFileCacheWarmer
         parent::__construct($phpArrayFile);
     }
 
-    protected function doWarmUp(string $cacheDir, ArrayAdapter $arrayAdapter): bool
+    /**
+     * @param string|null $buildDir
+     */
+    protected function doWarmUp(string $cacheDir, ArrayAdapter $arrayAdapter /* , string $buildDir = null */): bool
     {
         $annotatedClassPatterns = $cacheDir.'/annotations.map';
 
diff --git a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/CachePoolClearerCacheWarmer.php b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/CachePoolClearerCacheWarmer.php
index 7cd02d1cc5..7498a82d1f 100644
--- a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/CachePoolClearerCacheWarmer.php
+++ b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/CachePoolClearerCacheWarmer.php
@@ -37,10 +37,7 @@ final class CachePoolClearerCacheWarmer implements CacheWarmerInterface
         $this->pools = $pools;
     }
 
-    /**
-     * @return string[]
-     */
-    public function warmUp(string $cacheDirectory, string $buildDir = null): array
+    public function warmUp(string $cacheDir, string $buildDir = null): array
     {
         foreach ($this->pools as $pool) {
             if ($this->poolClearer->hasPool($pool)) {
diff --git a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/ConfigBuilderCacheWarmer.php b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/ConfigBuilderCacheWarmer.php
index 12e24bd77c..225fea11b7 100644
--- a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/ConfigBuilderCacheWarmer.php
+++ b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/ConfigBuilderCacheWarmer.php
@@ -38,11 +38,13 @@ class ConfigBuilderCacheWarmer implements CacheWarmerInterface
     }
 
     /**
-     * @return string[]
+     * @param string|null $buildDir
      */
-    public function warmUp(string $cacheDir, string $buildDir = null): array
+    public function warmUp(string $cacheDir /* , string $buildDir = null */): array
     {
-        if (null === $buildDir) {
+        $buildDir = 1 < \func_num_args() ? func_get_arg(1) : null;
+
+        if (!$buildDir) {
             return [];
         }
 
diff --git a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/RouterCacheWarmer.php b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/RouterCacheWarmer.php
index 82a16c01d3..2af9a2fe80 100644
--- a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/RouterCacheWarmer.php
+++ b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/RouterCacheWarmer.php
@@ -34,16 +34,12 @@ class RouterCacheWarmer implements CacheWarmerInterface, ServiceSubscriberInterf
         $this->container = $container;
     }
 
-    /**
-     * @param string|null $buildDir directory to put generated source code that can be marked as read-only at runtime.
-     *                              The directory is not provided when only cache_dir should be warmed up.
-     */
-    public function warmUp(string $cacheDir /* , string $buildDir = null */): array
+    public function warmUp(string $cacheDir, string $buildDir = null): array
     {
         $router = $this->container->get('router');
 
         if ($router instanceof WarmableInterface) {
-            return (array) $router->warmUp($cacheDir);
+            return (array) $router->warmUp($cacheDir, $buildDir);
         }
 
         throw new \LogicException(sprintf('The router "%s" cannot be warmed up because it does not implement "%s".', get_debug_type($router), WarmableInterface::class));
diff --git a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/SerializerCacheWarmer.php b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/SerializerCacheWarmer.php
index 35595474ef..b47a48ce69 100644
--- a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/SerializerCacheWarmer.php
+++ b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/SerializerCacheWarmer.php
@@ -39,7 +39,10 @@ class SerializerCacheWarmer extends AbstractPhpFileCacheWarmer
         $this->loaders = $loaders;
     }
 
-    protected function doWarmUp(string $cacheDir, ArrayAdapter $arrayAdapter): bool
+    /**
+     * @param string|null $buildDir
+     */
+    protected function doWarmUp(string $cacheDir, ArrayAdapter $arrayAdapter /* , string $buildDir = null */): bool
     {
         if (!$this->loaders) {
             return true;
diff --git a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/TranslationsCacheWarmer.php b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/TranslationsCacheWarmer.php
index d3abbaca5b..39b1444b0e 100644
--- a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/TranslationsCacheWarmer.php
+++ b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/TranslationsCacheWarmer.php
@@ -34,16 +34,16 @@ class TranslationsCacheWarmer implements CacheWarmerInterface, ServiceSubscriber
     }
 
     /**
-     * @param string|null $buildDir directory to put generated source code that can be marked as read-only at runtime.
-     *                              The directory is not provided when only cache_dir should be warmed up.
-     * @return string[]
+     * @param string|null $buildDir
      */
     public function warmUp(string $cacheDir /* , string $buildDir = null */): array
     {
         $this->translator ??= $this->container->get('translator');
 
         if ($this->translator instanceof WarmableInterface) {
-            return (array) $this->translator->warmUp($cacheDir);
+            $buildDir = 1 < \func_num_args() ? func_get_arg(1) : null;
+
+            return (array) $this->translator->warmUp($cacheDir, $buildDir);
         }
 
         return [];
diff --git a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/ValidatorCacheWarmer.php b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/ValidatorCacheWarmer.php
index 3119c9942a..224e90985e 100644
--- a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/ValidatorCacheWarmer.php
+++ b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/ValidatorCacheWarmer.php
@@ -39,7 +39,10 @@ class ValidatorCacheWarmer extends AbstractPhpFileCacheWarmer
         $this->validatorBuilder = $validatorBuilder;
     }
 
-    protected function doWarmUp(string $cacheDir, ArrayAdapter $arrayAdapter): bool
+    /**
+     * @param string|null $buildDir
+     */
+    protected function doWarmUp(string $cacheDir, ArrayAdapter $arrayAdapter /* , string $buildDir = null */): bool
     {
         $loaders = $this->validatorBuilder->getLoaders();
         $metadataFactory = new LazyLoadingMetadataFactory(new LoaderChain($loaders), $arrayAdapter);
diff --git a/src/Symfony/Bundle/FrameworkBundle/Routing/Router.php b/src/Symfony/Bundle/FrameworkBundle/Routing/Router.php
index fe89fef6d6..3367ecec2b 100644
--- a/src/Symfony/Bundle/FrameworkBundle/Routing/Router.php
+++ b/src/Symfony/Bundle/FrameworkBundle/Routing/Router.php
@@ -81,9 +81,7 @@ class Router extends BaseRouter implements WarmableInterface, ServiceSubscriberI
     }
 
     /**
-     * @param string|null $buildDir directory to put generated source code that can be marked as read-only at runtime.
-     *                              The directory is not provided when only cache_dir should be warmed up.
-     * @return string[] A list of classes to preload on PHP 7.4+
+     * @param string|null $buildDir
      */
     public function warmUp(string $cacheDir /* , string $buildDir = null */): array
     {
diff --git a/src/Symfony/Bundle/FrameworkBundle/Translation/Translator.php b/src/Symfony/Bundle/FrameworkBundle/Translation/Translator.php
index 70f5874307..04b56308f3 100644
--- a/src/Symfony/Bundle/FrameworkBundle/Translation/Translator.php
+++ b/src/Symfony/Bundle/FrameworkBundle/Translation/Translator.php
@@ -95,9 +95,7 @@ class Translator extends BaseTranslator implements WarmableInterface
     }
 
     /**
-     * @param string|null $buildDir directory to put generated source code that can be marked as read-only at runtime.
-     *                              The directory is not provided when only cache_dir should be warmed up.
-     * @return string[]
+     * @param string|null $buildDir
      */
     public function warmUp(string $cacheDir /* , string $buildDir = null */): array
     {
diff --git a/src/Symfony/Bundle/SecurityBundle/CacheWarmer/ExpressionCacheWarmer.php b/src/Symfony/Bundle/SecurityBundle/CacheWarmer/ExpressionCacheWarmer.php
index c63c973e41..1cbb681c2d 100644
--- a/src/Symfony/Bundle/SecurityBundle/CacheWarmer/ExpressionCacheWarmer.php
+++ b/src/Symfony/Bundle/SecurityBundle/CacheWarmer/ExpressionCacheWarmer.php
@@ -35,9 +35,7 @@ class ExpressionCacheWarmer implements CacheWarmerInterface
     }
 
     /**
-     * @param string|null $buildDir directory to put generated source code that can be marked as read-only at runtime.
-     *                              The directory is not provided when only cache_dir should be warmed up.
-     * @return string[]
+     * @param string|null $buildDir
      */
     public function warmUp(string $cacheDir /* , string $buildDir = null */): array
     {
diff --git a/src/Symfony/Bundle/TwigBundle/CacheWarmer/TemplateCacheWarmer.php b/src/Symfony/Bundle/TwigBundle/CacheWarmer/TemplateCacheWarmer.php
index 1adabe6636..2ab801130b 100644
--- a/src/Symfony/Bundle/TwigBundle/CacheWarmer/TemplateCacheWarmer.php
+++ b/src/Symfony/Bundle/TwigBundle/CacheWarmer/TemplateCacheWarmer.php
@@ -36,9 +36,7 @@ class TemplateCacheWarmer implements CacheWarmerInterface, ServiceSubscriberInte
     }
 
     /**
-     * @param string|null $buildDir directory to put generated source code that can be marked as read-only at runtime.
-     *                              The directory is not provided when only cache_dir should be warmed up.
-     * @return string[] A list of template files to preload on PHP 7.4+
+     * @param string|null $buildDir
      */
     public function warmUp(string $cacheDir /* , string $buildDir = null */): array
     {
diff --git a/src/Symfony/Component/HttpKernel/CacheWarmer/CacheWarmerAggregate.php b/src/Symfony/Component/HttpKernel/CacheWarmer/CacheWarmerAggregate.php
index 638d9ad2c4..a672956e0f 100644
--- a/src/Symfony/Component/HttpKernel/CacheWarmer/CacheWarmerAggregate.php
+++ b/src/Symfony/Component/HttpKernel/CacheWarmer/CacheWarmerAggregate.php
@@ -48,8 +48,17 @@ class CacheWarmerAggregate implements CacheWarmerInterface
         $this->onlyOptionalsEnabled = $this->optionalsEnabled = true;
     }
 
-    public function warmUp(string $cacheDir, string $buildDir = null, SymfonyStyle $io = null): array
+    /**
+     * @param string|null $buildDir
+     */
+    public function warmUp(string $cacheDir, string|SymfonyStyle $buildDir = null, SymfonyStyle $io = null): array
     {
+        if ($buildDir instanceof SymfonyStyle) {
+            trigger_deprecation('symfony/http-kernel', '6.4', 'Passing a "%s" as second argument of "%s()" is deprecated, pass it as third argument instead, after the build directory.', SymfonyStyle::class, __METHOD__);
+            $io = $buildDir;
+            $buildDir = null;
+        }
+
         if ($collectDeprecations = $this->debug && !\defined('PHPUNIT_COMPOSER_INSTALL')) {
             $collectedLogs = [];
             $previousHandler = set_error_handler(function ($type, $message, $file, $line) use (&$collectedLogs, &$previousHandler) {
@@ -96,8 +105,8 @@ class CacheWarmerAggregate implements CacheWarmerInterface
                 }
 
                 $start = microtime(true);
-                foreach ((array) $warmer->warmUp($cacheDir, null === $buildDir || $warmer->isOptional() ? null : $buildDir) as $item) {
-                    if (is_dir($item) || (str_starts_with($item, \dirname($cacheDir)) && !is_file($item))) {
+                foreach ((array) $warmer->warmUp($cacheDir, $buildDir) as $item) {
+                    if (is_dir($item) || (str_starts_with($item, \dirname($cacheDir)) && !is_file($item)) || ($buildDir && str_starts_with($item, \dirname($buildDir)) && !is_file($item))) {
                         throw new \LogicException(sprintf('"%s::warmUp()" should return a list of files or classes but "%s" is none of them.', $warmer::class, $item));
                     }
                     $preload[] = $item;
diff --git a/src/Symfony/Component/HttpKernel/CacheWarmer/WarmableInterface.php b/src/Symfony/Component/HttpKernel/CacheWarmer/WarmableInterface.php
index 799705e42d..cd051b1add 100644
--- a/src/Symfony/Component/HttpKernel/CacheWarmer/WarmableInterface.php
+++ b/src/Symfony/Component/HttpKernel/CacheWarmer/WarmableInterface.php
@@ -21,8 +21,8 @@ interface WarmableInterface
     /**
      * Warms up the cache.
      *
-     * @param string|null $buildDir directory to put generated source code that can be marked as read-only at runtime.
-     *                              The directory is not provided when only cache_dir should be warmed up.
+     * @param string      $cacheDir Where warm-up artifacts should be stored
+     * @param string|null $buildDir Where read-only artifacts should go; null when called after compile-time
      *
      * @return string[] A list of classes or files to preload on PHP 7.4+
      */
diff --git a/src/Symfony/Component/Translation/DataCollectorTranslator.php b/src/Symfony/Component/Translation/DataCollectorTranslator.php
index bdd0e6ce4d..785e5b89ed 100644
--- a/src/Symfony/Component/Translation/DataCollectorTranslator.php
+++ b/src/Symfony/Component/Translation/DataCollectorTranslator.php
@@ -72,14 +72,14 @@ class DataCollectorTranslator implements TranslatorInterface, TranslatorBagInter
     }
 
     /**
-     * @param string|null $buildDir directory to put generated source code that can be marked as read-only at runtime.
-     *                              The directory is not provided when only cache_dir should be warmed up.
-     * @return string[]
+     * @param string|null $buildDir
      */
     public function warmUp(string $cacheDir /* , string $buildDir = null */): array
     {
+        $buildDir = 1 < \func_num_args() ? func_get_arg(1) : null;
+
         if ($this->translator instanceof WarmableInterface) {
-            return (array) $this->translator->warmUp($cacheDir);
+            return (array) $this->translator->warmUp($cacheDir, $buildDir);
         }
 
         return [];

src/Symfony/Component/HttpKernel/CacheWarmer/CacheWarmerAggregate.php

nicolas-grekas

please rebase

src/Symfony/Bundle/FrameworkBundle/Command/CacheWarmupCommand.php

…ableInterface::warmup` to warm read-only artefacts in `build_dir`

nicolas-grekas · 2023-10-17T11:35:59Z

Thank you @okhoshi.

…bois) This PR was merged into the 6.4 branch. Discussion ---------- [HttpKernel] Fix CacheWarmerAggregateTest | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT #50391 seems to bring broken tests. Particularly, the `testWarmupOnOptionalWarmerPassBuildDir` test doesn't seem relevant given the actual code of `CacheWarmerAggregate`. Indeed, nothing in the code is setting `buildDir` to null when using an optional warmer. I suggest a changes to fix this test on 6.4. Also for the first one, given `optionalsEnabled` is enabled, `isOptional` is never called. I suggest to remove this expectation. Commits ------- 945d5cd [HttpKernel] Fix CacheWarmerAggregateTest

…bois) This PR was merged into the 6.4 branch. Discussion ---------- [HttpKernel] Fix CacheWarmerAggregateTest | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT symfony/symfony#50391 seems to bring broken tests. Particularly, the `testWarmupOnOptionalWarmerPassBuildDir` test doesn't seem relevant given the actual code of `CacheWarmerAggregate`. Indeed, nothing in the code is setting `buildDir` to null when using an optional warmer. I suggest a changes to fix this test on 6.4. Also for the first one, given `optionalsEnabled` is enabled, `isOptional` is never called. I suggest to remove this expectation. Commits ------- 945d5cd5e1 [HttpKernel] Fix CacheWarmerAggregateTest

…el.build_dir` (Okhoshi) This PR was squashed before being merged into the 7.1 branch. Discussion ---------- [FrameworkBundle] Move Router cache directory to `kernel.build_dir` | Q | A | ------------- | --- | Branch? | 7.1 | Bug fix? | no | New feature? | yes | Deprecations? | no | Issues | none | License | MIT Follow up to #50391, set up Router cache directory to `kernel.build_dir` instead of `kernel.cache_dir` by default, and only warm the cache on read-only resources phase. #SymfonyHackday Commits ------- 1f031f8 [FrameworkBundle] Move Router cache directory to `kernel.build_dir`

…-optional (nicolas-grekas) This PR was merged into the 6.4 branch. Discussion ---------- [FrameworkBundle] `ConfigBuilderCacheWarmer` should be non-optional | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | Fix #53496 | License | MIT Replaces #53512 This became apparent after #50391, where the warmer is run only when compiling the container. Before, it didn't really matter. Commits ------- 8bd2ff8 [FrameworkBundle] ConfigBuilderCacheWarmer should be non-optional

…ates known at build time (Okhoshi) This PR was merged into the 7.3 branch. Discussion ---------- [TwigBundle] Use `kernel.build_dir` to store the templates known at build time | Q | A | ------------- | --- | Branch? | 7.3 | Bug fix? | no | New feature? | yes | Deprecations? | no | Issues | none | License | MIT Follow up to #50391, set up the Twig `TemplateCacheWarmer` to use the `kernel.build_dir` instead of `kernel.cache_dir`. A new argument is added to its constructor to specify the directory to use for the cache. Commits ------- 0f841d2 [TwigBundle] Use `kernel.build_dir` to store the templates known at build time

…ializeCacheWarmer` use `kernel.build_dir` instead of `kernel.cache_dir` (Okhoshi) This PR was squashed before being merged into the 7.3 branch. Discussion ---------- [FrameworkBundle] Make `ValidatorCacheWarmer` and `SerializeCacheWarmer` use `kernel.build_dir` instead of `kernel.cache_dir` | Q | A | ------------- | --- | Branch? | 7.3 | Bug fix? | no | New feature? | yes | Deprecations? | no | Issues | none | License | MIT Follow up to #50391, set up the `ValidatorCacheWarmer` and `SerializerCacheWarmer` to use the `kernel.build_dir` instead of `kernel.cache_dir`. The value conveyed by their constructor parameter has been changed to prevent developers to configure the cache file outside of the build_dir. `AbstractPhpFileCacheWarmer` has been reworked a bit too, to reduce the risk of misconfiguration. #SymfonyHackday Commits ------- dc59817 [FrameworkBundle] Make `ValidatorCacheWarmer` and `SerializeCacheWarmer` use `kernel.build_dir` instead of `kernel.cache_dir`

carsonbot added Status: Needs Review Feature FrameworkBundle HttpKernel labels May 22, 2023

carsonbot added this to the 6.3 milestone May 22, 2023

okhoshi force-pushed the ticket_50357 branch 2 times, most recently from a575a7e to 9890af2 Compare May 22, 2023 22:21

nicolas-grekas modified the milestones: 6.3, 6.4 May 23, 2023

okhoshi force-pushed the ticket_50357 branch 5 times, most recently from 28b878f to 4b9ed49 Compare August 7, 2023 07:25

nicolas-grekas reviewed Oct 6, 2023

View reviewed changes

nicolas-grekas added the ❄️ Feature Freeze Important Pull Requests to finish before the next Symfony "feature freeze" label Oct 6, 2023

okhoshi force-pushed the ticket_50357 branch 3 times, most recently from 92ba4e8 to d62544e Compare October 9, 2023 13:40

okhoshi changed the title ~~[FrameworkBundle][HttpKernel] Introduce BuildWarmerInterface to warm read-only artefacts in build_dir~~ [FrameworkBundle][HttpKernel] Introduce $buildDir argument to WarmableInterface::warmup to warm read-only artefacts in build_dir Oct 9, 2023

okhoshi force-pushed the ticket_50357 branch 3 times, most recently from 03a16b2 to 3b80633 Compare October 9, 2023 14:55

okhoshi requested review from yceruto, welcoMattic, chalasr and dunglas as code owners October 10, 2023 09:36

okhoshi force-pushed the ticket_50357 branch 2 times, most recently from 931cc08 to dc5d8f9 Compare October 10, 2023 13:25

nicolas-grekas reviewed Oct 10, 2023

View reviewed changes

src/Symfony/Component/HttpKernel/CacheWarmer/CacheWarmerAggregate.php Outdated Show resolved Hide resolved

nicolas-grekas reviewed Oct 13, 2023

View reviewed changes

src/Symfony/Bundle/FrameworkBundle/Command/CacheWarmupCommand.php Outdated Show resolved Hide resolved

okhoshi force-pushed the ticket_50357 branch 3 times, most recently from 38fa985 to d643c9f Compare October 16, 2023 21:19

nicolas-grekas approved these changes Oct 17, 2023

View reviewed changes

carsonbot added Status: Reviewed and removed Status: Needs Review labels Oct 17, 2023

[FrameworkBundle][HttpKernel] Introduce $buildDir argument to `Warm…

8b604ff

…ableInterface::warmup` to warm read-only artefacts in `build_dir`

nicolas-grekas force-pushed the ticket_50357 branch from d643c9f to 8b604ff Compare October 17, 2023 11:35

nicolas-grekas mentioned this pull request Oct 17, 2023

[FrameworkBundle][HttpKernel] Introduce $buildDir argument to WarmableInterface::warmup to warm read-only artefacts in build_dir symfony/symfony-docs#19034

Closed

nicolas-grekas merged commit aa5ba21 into symfony:6.4 Oct 17, 2023

okhoshi deleted the ticket_50357 branch October 17, 2023 11:42

alexandre-daubois mentioned this pull request Oct 17, 2023

[HttpKernel] Fix CacheWarmerAggregateTest #52111

Merged

This was referenced Oct 21, 2023

Release v6.4.0-BETA1 #52220

Merged

Release v7.0.0-BETA1 #52224

Merged

franmomu mentioned this pull request Oct 29, 2023

Remove deprecations using Symfony 6.4 doctrine/DoctrineMongoDBBundle#789

Merged

This was referenced Dec 9, 2023

[FrameworkBundle] Move Router cache directory to kernel.build_dir #52962

Merged

[FrameworkBundle] Make ValidatorCacheWarmer and SerializeCacheWarmer use kernel.build_dir instead of kernel.cache_dir #52981

Merged

xabbuh mentioned this pull request Dec 18, 2023

Change ProxyCacheWarmer::warmUp signature #53128

Merged

nicolas-grekas mentioned this pull request Jan 29, 2024

[FrameworkBundle] ConfigBuilderCacheWarmer should be non-optional #53672

Merged

okhoshi mentioned this pull request Mar 23, 2024

[TwigBundle] Use kernel.build_dir to store the templates known at build time #54384

Merged

This was referenced Jun 27, 2024

[HttpKernel] Enable optional cache-warmers when cache-dir != build-dir #57553

Merged

[FrameworkBundle] Split "var/build" from "var/cache" when compiling for the "prod" env #57556

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

[FrameworkBundle][HttpKernel] Introduce `$buildDir` argument to `WarmableInterface::warmup` to warm read-only artefacts in `build_dir` #50391

[FrameworkBundle][HttpKernel] Introduce `$buildDir` argument to `WarmableInterface::warmup` to warm read-only artefacts in `build_dir` #50391

Uh oh!

okhoshi commented May 22, 2023 •

edited

Loading

Uh oh!

nicolas-grekas left a comment •

edited

Loading

Uh oh!

nicolas-grekas left a comment

Uh oh!

Uh oh!

nicolas-grekas left a comment

Uh oh!

Uh oh!

nicolas-grekas commented Oct 17, 2023

Uh oh!

Uh oh!

Uh oh!

[FrameworkBundle][HttpKernel] Introduce $buildDir argument to WarmableInterface::warmup to warm read-only artefacts in build_dir #50391

[FrameworkBundle][HttpKernel] Introduce $buildDir argument to WarmableInterface::warmup to warm read-only artefacts in build_dir #50391

Uh oh!

Conversation

okhoshi commented May 22, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nicolas-grekas left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nicolas-grekas left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

nicolas-grekas left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

nicolas-grekas commented Oct 17, 2023

Uh oh!

Uh oh!

[FrameworkBundle][HttpKernel] Introduce `$buildDir` argument to `WarmableInterface::warmup` to warm read-only artefacts in `build_dir` #50391

[FrameworkBundle][HttpKernel] Introduce `$buildDir` argument to `WarmableInterface::warmup` to warm read-only artefacts in `build_dir` #50391

okhoshi commented May 22, 2023 •

edited

Loading

nicolas-grekas left a comment •

edited

Loading