Skip to content

Commit

Permalink
Merge pull request #233 from vertica/fix
Browse files Browse the repository at this point in the history
Fix VerticaPyLab start bug
  • Loading branch information
roypaulin authored Mar 4, 2024
2 parents d041f26 + 432eab6 commit 5ec20bd
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 1 deletion.
4 changes: 4 additions & 0 deletions .github/workflows/makefile.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,10 @@ jobs:
run: bash -ec "! make verticapylab-build"
- name: Create a VerticaPyLab container
run: sed -i s/#VERTICAPYLAB_IMG_VERSION=latest/VERTICAPYLAB_IMG_VERSION=v0.0/ etc/VerticaPyLab.conf && make verticapylab-build
- name: Create a VerticaPyLab container in test mode
run: TEST_MODE=yes make verticapylab-build
- name: Check VerticaPy container for test mode
run: docker image ls opentext/verticapy-jupyterlab:v0.2.0 | grep -q opentext/verticapy-jupyterlab
- name: Run the VerticaPyLab container
run: make verticapylab-start
- name: Create and run a grafana container
Expand Down
10 changes: 9 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -110,9 +110,17 @@ vsql: ## Run a basic sanity test (optional -DQUERY="select 'whatever')
.PHONY: verticapylab-start
verticapylab-start: etc/VerticaPyLab.conf ## Start a jupyterlab
@source etc/VerticaPyLab.conf; \
if [[ $${TEST_MODE^^} == "YES" ]] ; then \
VERTICAPYLAB_IMG_VERSION=$(VERSION); \
fi; \
if (($$(docker ps --no-trunc -q -f NAME="$$VERTICAPYLAB_CONTAINER_NAME" | wc -l)==0)); then \
if [[ -z $$(docker image ls -q "opentext/$$VERTICAPYLAB_IMG:$$VERTICAPYLAB_IMG_VERSION" 2>&1) ]]; then \
$(MAKE) verticapylab-install || exit 1; \
if [[ $${TEST_MODE^^} == "YES" ]] ; then \
echo "Building image opentext/$$VERTICAPYLAB_IMG:$$VERTICAPYLAB_IMG_VERSION"; \
TEST_MODE=yes $(MAKE) verticapylab-build; \
else \
$(MAKE) verticapylab-install || exit 1; \
fi; \
fi; \
docker container rm "$$VERTICAPYLAB_CONTAINER_NAME" >/dev/null 2>&1; \
bin/verticapylab; \
Expand Down

0 comments on commit 5ec20bd

Please sign in to comment.