Skip to content

[docker] Incorporate amqp ext compilation to docker build process. #275

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 21, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions bin/build-enqueue-dev-image.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
#!/usr/bin/env bash

set -e
set -x

(cd docker && docker build --rm --force-rm --no-cache --pull --squash --tag "enqueue/dev:latest" -f Dockerfile .)
(cd docker && docker login --username="$DOCKER_USER" --password="$DOCKER_PASSWORD")
(cd docker && docker push "enqueue/dev:latest")
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@
"config": {
"bin-dir": "bin",
"platform": {
"ext-amqp": "1.7",
"ext-amqp": "1.9.3",
"ext-gearman": "1.1",
"ext-rdkafka": "3.3"
}
Expand Down
2 changes: 0 additions & 2 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ version: '2'
services:
dev:
image: enqueue/dev:latest
# build: { context: docker, dockerfile: Dockerfile }
depends_on:
- rabbitmq
- mysql
Expand Down Expand Up @@ -48,7 +47,6 @@ services:

rabbitmq:
image: 'enqueue/rabbitmq:latest'
# build: { context: docker, dockerfile: Dockerfile.rabbitmq }
environment:
- RABBITMQ_DEFAULT_USER=guest
- RABBITMQ_DEFAULT_PASS=guest
Expand Down
27 changes: 20 additions & 7 deletions docker/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -3,21 +3,34 @@ FROM formapro/nginx-php-fpm:latest-all-exts
## libs
RUN set -x && \
apt-get update && \
apt-get install -y wget curl openssl ca-certificates nano netcat php-dev php-redis git python
apt-get install -y --no-install-recommends --no-install-suggests wget curl openssl ca-certificates nano netcat php-dev php-redis git python

RUN set -x && \
apt-get update && \
apt-get install -y --no-install-recommends --no-install-suggests php-dev librabbitmq-dev make && \
mkdir -p $HOME/php-amqp && \
cd $HOME/php-amqp && \
git clone https://github.com/pdezwart/php-amqp.git . && git checkout v1.9.3 && \
phpize --clean && phpize && ./configure && make install

## confis

# RUN rm -f /etc/php/7.0/cli/conf.d/*xdebug.ini

## librdkafka
RUN git clone https://github.com/edenhill/librdkafka.git /root/librdkafka
RUN cd /root/librdkafka && git checkout v0.11.0-RC2 && ./configure && make && make install
RUN pecl install rdkafka
RUN echo "extension=rdkafka.so" > /etc/php/7.1/cli/conf.d/10-rdkafka.ini
RUN echo "extension=rdkafka.so" > /etc/php/7.1/fpm/conf.d/10-rdkafka.ini
RUN set -x && \
apt-get update && \
apt-get install -y --no-install-recommends --no-install-suggests g++ php-pear php-dev && \
mkdir -p $HOME/librdkafka && \
cd $HOME/librdkafka && \
git clone https://github.com/edenhill/librdkafka.git . && \
git checkout v0.11.1 && \
./configure && make && make install && \
pecl install rdkafka && \
echo "extension=rdkafka.so" > /etc/php/7.1/cli/conf.d/10-rdkafka.ini && \
echo "extension=rdkafka.so" > /etc/php/7.1/fpm/conf.d/10-rdkafka.ini

COPY ./php/cli.ini /etc/php/7.1/cli/conf.d/1-dev_cli.ini
COPY ./php/amqp.so /usr/lib/php/20160303/amqp.so
COPY ./bin/dev_entrypoiny.sh /usr/local/bin/entrypoint.sh
RUN chmod u+x /usr/local/bin/entrypoint.sh

Expand Down
Binary file removed docker/php/amqp.so
Binary file not shown.
10 changes: 8 additions & 2 deletions pkg/amqp-ext/AmqpConnectionFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -125,11 +125,17 @@ private function establishConnection()

$this->connection = new \AMQPConnection($extConfig);

$this->config->isPersisted() ? $this->connection->pconnect() : $this->connection->connect();
$this->config->isPersisted() ?
$this->connection->pconnect() :
$this->connection->connect()
;
}

if (false == $this->connection->isConnected()) {
$this->config->isPersisted() ? $this->connection->preconnect() : $this->connection->reconnect();
$this->config->isPersisted() ?
$this->connection->preconnect() :
$this->connection->reconnect()
;
}

return $this->connection;
Expand Down
2 changes: 1 addition & 1 deletion pkg/amqp-ext/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
"license": "MIT",
"require": {
"php": ">=5.6",
"ext-amqp": "^1.6",
"ext-amqp": "^1.9.3",

"queue-interop/amqp-interop": "^0.7@dev",
"enqueue/amqp-tools": "^0.8.4@dev"
Expand Down