Skip to content

Commit d5231e6

Browse files
authored
Add step to initialize faiss submodule (#107)
Signed-off-by: Rajvaibhav Rahane <rrahane@amazon.com>
1 parent 8ac91d7 commit d5231e6

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

.github/workflows/publish_faiss_base_image.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,11 @@ jobs:
2525
uses: actions/checkout@v4
2626
with:
2727
submodules: 'recursive'
28+
path: 'rvib'
2829

2930
- name: Build Docker Image
3031
run : |
32+
cd rvib
3133
docker build -f ./base_image/build_scripts/Dockerfile . -t opensearchstaging/remote-vector-index-builder:faiss-base-snapshot
3234
3335
- name: Configure AWS Credentials
@@ -56,8 +58,11 @@ jobs:
5658
- name: Runner Cleanups
5759
if: always()
5860
run: |
61+
# Docker cleanup
5962
docker logout
6063
docker system prune -a -f
64+
65+
# Workspace cleanup
6166
rm -rf ${{ github.workspace }}/*
6267
6368
# Trigger build of core image since base image is changed
@@ -66,4 +71,4 @@ jobs:
6671
uses: ./.github/workflows/publish_remote_core_image.yml
6772
secrets:
6873
REMOTE_VECTOR_DOCKER_ROLE: ${{ secrets.REMOTE_VECTOR_DOCKER_ROLE }}
69-
REMOTE_VECTOR_DOCKER_USERNAME: ${{ secrets.REMOTE_VECTOR_DOCKER_USERNAME }}
74+
REMOTE_VECTOR_DOCKER_USERNAME: ${{ secrets.REMOTE_VECTOR_DOCKER_USERNAME }}

base_image/build_scripts/Dockerfile

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,10 @@ RUN chown -R appuser:appuser /tmp
2525
USER appuser
2626
# Put conda in path so we can use conda activate
2727
ENV PATH=$CONDA_DIR/bin:$PATH
28+
29+
# Accept Conda Terms of Service
30+
ENV CONDA_PLUGINS_AUTO_ACCEPT_TOS=yes
31+
2832
#install some necessary dependencies
2933
RUN conda install -c conda-forge -y -q python=3.11 cmake=3.30.4 make=4.2 swig=4.0 "numpy<2" scipy=1.14 pytest=7.4 gflags=2.2
3034
RUN cmake --version

0 commit comments

Comments
 (0)