-
Notifications
You must be signed in to change notification settings - Fork 17
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
add build push docker image image in the publish ci #311
Changes from 18 commits
6c0eac5
b19aec9
b24192e
710b8f4
795fa87
bd5cddf
c97d6f0
7226fc1
0e70af2
feb24d2
e8d6fc4
15b0d65
4826f19
3e67674
34f36a1
a2ee00b
12af96a
808ee37
f4ab110
10eba9b
c033dfb
562f7b3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,11 +3,8 @@ version: '3.9' | |
services: | ||
app-elasticsearch: | ||
container_name: stac-fastapi-es | ||
image: stac-utils/stac-fastapi-es | ||
image: ghcr.io/stac-utils/stac-fastapi-es:latest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you revert this to pulling from the local code like it was doing before ( |
||
restart: always | ||
build: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Revert this back to use the local Dockerfile.dev.es |
||
context: . | ||
dockerfile: dockerfiles/Dockerfile.dev.es | ||
environment: | ||
- STAC_FASTAPI_TITLE=stac-fastapi-elasticsearch | ||
- STAC_FASTAPI_DESCRIPTION=A STAC FastAPI with an Elasticsearch backend | ||
|
@@ -35,11 +32,8 @@ services: | |
|
||
app-opensearch: | ||
container_name: stac-fastapi-os | ||
image: stac-utils/stac-fastapi-os | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you revert this to pulling from the local code like it was doing before ( |
||
image: ghcr.io/stac-utils/stac-fastapi-os:latest | ||
restart: always | ||
build: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Revert this back to use the local Dockerfile.dev.os |
||
context: . | ||
dockerfile: dockerfiles/Dockerfile.dev.os | ||
environment: | ||
- STAC_FASTAPI_TITLE=stac-fastapi-opensearch | ||
- STAC_FASTAPI_DESCRIPTION=A STAC FastAPI with an Opensearch backend | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,34 @@ | ||
FROM python:3.12-slim | ||
|
||
ENV APP_HOST="0.0.0.0" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sorry but could you remove the ENV lines here? The environment variables should be set when you run the Docker image. Either from an ,env file or as |
||
ENV APP_PORT="8080" | ||
ENV WEB_CONCURRENCY="10" | ||
ENV RELOAD="true" | ||
ENV ES_HOST="localhost" | ||
ENV ES_PORT="9200" | ||
ENV ES_USE_SSL="false" | ||
ENV ES_VERIFY_CERTS="false" | ||
ENV STAC_FASTAPI_TITLE="stac-fastapi-elasticsearch" | ||
ENV STAC_FASTAPI_DESCRIPTION="A STAC FastAPI with an Elasticsearch backend" | ||
ENV STAC_FASTAPI_VERSION="2.1" | ||
ENV ENVIRONMENT="local" | ||
ENV BACKEND="elasticsearch" | ||
ENV STAC_FASTAPI_RATE_LIMIT="200/minute" | ||
|
||
WORKDIR /app | ||
|
||
RUN apt-get update && \ | ||
apt-get install -y --no-install-recommends \ | ||
gcc \ | ||
curl \ | ||
&& apt-get clean && \ | ||
rm -rf /var/lib/apt/lists/* | ||
|
||
COPY . /app/ | ||
|
||
RUN pip3 install --no-cache-dir -e ./stac_fastapi/core && \ | ||
pip3 install --no-cache-dir ./stac_fastapi/elasticsearch[server] | ||
|
||
USER root | ||
|
||
CMD ["python", "-m", "stac_fastapi.elasticsearch.app"] |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,34 @@ | ||
FROM python:3.12-slim | ||
|
||
ENV STAC_FASTAPI_TITLE="stac-fastapi-opensearch" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sorry but could you remove the ENV lines here? The environment variables should be set when you run the Docker image. Either from an ,env file or as |
||
ENV STAC_FASTAPI_DESCRIPTION="A STAC FastAPI with an Opensearch backend" | ||
ENV STAC_FASTAPI_VERSION="3.0.0a2" | ||
ENV APP_HOST="0.0.0.0" | ||
ENV APP_PORT="8082" | ||
ENV RELOAD="true" | ||
ENV ENVIRONMENT="local" | ||
ENV WEB_CONCURRENCY="10" | ||
ENV ES_HOST="localhost" | ||
ENV ES_PORT="9202" | ||
ENV ES_USE_SSL="false" | ||
ENV ES_VERIFY_CERTS="false" | ||
ENV BACKEND="opensearch" | ||
ENV STAC_FASTAPI_RATE_LIMIT="200/minute" | ||
|
||
WORKDIR /app | ||
|
||
RUN apt-get update && \ | ||
apt-get install -y --no-install-recommends \ | ||
gcc \ | ||
curl \ | ||
&& apt-get clean && \ | ||
rm -rf /var/lib/apt/lists/* | ||
|
||
COPY . /app/ | ||
|
||
RUN pip3 install --no-cache-dir -e ./stac_fastapi/core && \ | ||
pip3 install --no-cache-dir ./stac_fastapi/opensearch[server] | ||
|
||
USER root | ||
|
||
CMD ["python", "-m", "stac_fastapi.opensearch.app"] |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
"""library version.""" | ||
__version__ = "3.2.0" | ||
__version__ = "3.2.1" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
"""library version.""" | ||
__version__ = "3.2.0" | ||
__version__ = "3.2.1" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
"""library version.""" | ||
__version__ = "3.2.0" | ||
__version__ = "3.2.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great