Skip to content

Commit 5b29286

Browse files
author
Igor Vitol
committed
Disabling blackfire by default. Will be enabled automatically in compose when added to .magento.app.yaml config
1 parent 71847ed commit 5b29286

File tree

7 files changed

+4
-19
lines changed

7 files changed

+4
-19
lines changed

src/Command/BuildCompose.php

-5
Original file line numberDiff line numberDiff line change
@@ -252,11 +252,6 @@ protected function configure(): void
252252
null,
253253
InputOption::VALUE_NONE,
254254
'Enables XDebug'
255-
)->addOption(
256-
Source\CliSource::OPTION_WITHOUT_BLACKFIRE,
257-
null,
258-
InputOption::VALUE_NONE,
259-
'Do not load blackfire php extension by default'
260255
)->addOption(
261256
Source\CliSource::OPTION_ENV_VARIABLES,
262257
null,

src/Compose/Php/ExtensionResolver.php

+1-2
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,7 @@ class ExtensionResolver
4646
'sysvsem',
4747
'sysvshm',
4848
'opcache',
49-
'zip',
50-
'blackfire'
49+
'zip'
5150
];
5251

5352
/**

src/Config/Source/CliSource.php

-7
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ class CliSource implements SourceInterface
5353
public const OPTION_NO_TMP_MOUNTS = 'no-tmp-mounts';
5454
public const OPTION_SYNC_ENGINE = 'sync-engine';
5555
public const OPTION_WITH_XDEBUG = 'with-xdebug';
56-
public const OPTION_WITHOUT_BLACKFIRE = 'without-blackfire';
5756
public const OPTION_SET_DOCKER_HOST_XDEBUG = 'set-docker-host';
5857
public const OPTION_WITH_ENTRYPOINT = 'with-entrypoint';
5958
public const OPTION_WITH_MARIADB_CONF = 'with-mariadb-conf';
@@ -145,7 +144,6 @@ public function __construct(InputInterface $input)
145144
public function read(): Repository
146145
{
147146
$repository = new Repository();
148-
$disabledExtensions = [];
149147

150148
if ($mode = $this->input->getOption(self::OPTION_MODE)) {
151149
$repository->set([
@@ -240,10 +238,6 @@ public function read(): Repository
240238
]);
241239
}
242240

243-
if ($this->input->getOption(self::OPTION_WITHOUT_BLACKFIRE)) {
244-
$disabledExtensions['blackfire'] = 'blackfire';
245-
}
246-
247241
if ($this->input->getOption(self::OPTION_SET_DOCKER_HOST_XDEBUG)) {
248242
$repository->set(self::SYSTEM_SET_DOCKER_HOST, true);
249243
}
@@ -316,7 +310,6 @@ public function read(): Repository
316310
$repository->set(self::SYSTEM_NGINX_WORKER_CONNECTIONS, $nginxWorkerConnections);
317311
}
318312

319-
$repository->set(self::PHP_DISABLED_EXTENSIONS, $disabledExtensions);
320313
return $repository;
321314
}
322315
}

src/Test/Integration/BuildComposeTest.php

-2
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,6 @@ public function buildDataProvider(): array
8787
[CliSource::OPTION_WITH_TEST, true],
8888
[CliSource::OPTION_WITH_CRON, true],
8989
[CliSource::OPTION_WITH_XDEBUG, true],
90-
[CliSource::OPTION_WITHOUT_BLACKFIRE, true],
9190
[CliSource::OPTION_ES, '5.2'],
9291
[CliSource::OPTION_NO_ES, true],
9392
[CliSource::OPTION_DB_INCREMENT_INCREMENT, 3],
@@ -107,7 +106,6 @@ public function buildDataProvider(): array
107106
[CliSource::OPTION_WITH_TEST, true],
108107
[CliSource::OPTION_WITH_CRON, true],
109108
[CliSource::OPTION_WITH_XDEBUG, true],
110-
[CliSource::OPTION_WITHOUT_BLACKFIRE, true],
111109
[CliSource::OPTION_ES, '5.2'],
112110
[CliSource::OPTION_NO_ES, true],
113111
[CliSource::OPTION_DB_INCREMENT_INCREMENT, 3],

src/Test/Integration/_files/cloud_base/docker-compose.exp.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ services:
127127
image: 'magento/magento-cloud-docker-php:7.3-cli-1.2.0'
128128
env_file: ./.docker/config.env
129129
environment:
130-
- 'PHP_EXTENSIONS=bcmath bz2 calendar exif gd gettext intl mysqli pcntl pdo_mysql soap sockets sysvmsg sysvsem sysvshm opcache zip blackfire redis xsl sodium'
130+
- 'PHP_EXTENSIONS=bcmath bz2 calendar exif gd gettext intl mysqli pcntl pdo_mysql soap sockets sysvmsg sysvsem sysvshm opcache zip redis xsl sodium'
131131
build:
132132
hostname: build.magento2.docker
133133
image: 'magento/magento-cloud-docker-php:7.3-cli-1.2.0'

src/Test/Integration/_files/custom_cloud_base/docker-compose.exp.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ services:
101101
image: 'magento/magento-cloud-docker-php:7.2-cli-1.2.0'
102102
env_file: ./.docker/config.env
103103
environment:
104-
- 'PHP_EXTENSIONS=bcmath bz2 calendar exif gd gettext intl mysqli pcntl pdo_mysql soap sockets sysvmsg sysvsem sysvshm opcache zip blackfire xsl'
104+
- 'PHP_EXTENSIONS=bcmath bz2 calendar exif gd gettext intl mysqli pcntl pdo_mysql soap sockets sysvmsg sysvsem sysvshm opcache zip xsl'
105105
build:
106106
hostname: build.magento2.test
107107
image: 'magento/magento-cloud-docker-php:7.2-cli-1.2.0'

src/Test/Integration/_files/custom_cloud_custom_images/docker-compose.exp.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ services:
152152
image: 'php-v1:7.4-cli'
153153
env_file: ./.docker/config.env
154154
environment:
155-
- 'PHP_EXTENSIONS=bcmath bz2 calendar exif gd gettext intl mysqli pcntl pdo_mysql soap sockets sysvmsg sysvsem sysvshm opcache zip blackfire xsl'
155+
- 'PHP_EXTENSIONS=bcmath bz2 calendar exif gd gettext intl mysqli pcntl pdo_mysql soap sockets sysvmsg sysvsem sysvshm opcache zip xsl'
156156
build:
157157
hostname: build.magento2.test
158158
image: 'php-v1:7.4-cli'

0 commit comments

Comments
 (0)