Skip to content

Add Dockerfile for alpine#1

Merged
moonming merged 3 commits intoapache:masterfrom
vkill:alpine
Jul 15, 2019
Merged

Add Dockerfile for alpine#1
moonming merged 3 commits intoapache:masterfrom
vkill:alpine

Conversation

@vkill
Copy link
Contributor

@vkill vkill commented Jul 7, 2019

Hi there

The Dockerfile has been completed, but the official docker image 500 happened when require "ngx.re", I should wait openresty/docker-openresty/pull/113 merge.

--- 2019-07-12 updated
The official docker image was fixed pcre bug in 1.15.8.1-3 , and I was modified the Dockerfile.

autoconf \
libtool \
pkgconfig \
cmake \
Copy link
Member

Choose a reason for hiding this comment

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

I think we need luarocks too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The luarocks has been built the alpine-fat .

Copy link
Member

Choose a reason for hiding this comment

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

cool

@moonming
Copy link
Member

@vkill I am a newbie of alpine, is alpine has package management utility like yum? I want to install the package directly instead of compile in docker, what's do you think about that?

@vkill
Copy link
Contributor Author

vkill commented Jul 15, 2019

is alpine has package management utility like yum?

It's apk, you can search package name then apk add package-name

@vkill vkill changed the title [WIP]Add Dockerfile for alpine Add Dockerfile for alpine Jul 15, 2019
@vkill
Copy link
Contributor Author

vkill commented Jul 15, 2019

@moonming I was completed test. please review it. thx.

@moonming
Copy link
Member

@vkill I think we should EXPOSE the ports of APISIX: 9080, 9443

@vkill
Copy link
Contributor Author

vkill commented Jul 15, 2019

Yes, you are right, it's added.

@moonming moonming merged commit 683bf9b into apache:master Jul 15, 2019
@moonming
Copy link
Member

merged, thx

Akayeshmantha pushed a commit to Akayeshmantha/incubator-apisix-docker that referenced this pull request Mar 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments