Skip to content

Commit

Permalink
Fixes #17221: Setuptools Version Fix, Resolve Ingestion Setup Failure (
Browse files Browse the repository at this point in the history
  • Loading branch information
ayush-shah authored Jul 29, 2024
1 parent c522f14 commit 1962ffa
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 7 deletions.
4 changes: 2 additions & 2 deletions ingestion/Dockerfile.ci
Original file line number Diff line number Diff line change
Expand Up @@ -76,13 +76,13 @@ ENV PIP_QUIET=1
RUN pip install --upgrade pip

WORKDIR /home/airflow/openmetadata-airflow-apis
RUN pip install "."
RUN PIP_CONSTRAINT=/home/airflow/openmetadata-airflow-apis/constraints.txt pip install "."

WORKDIR /home/airflow/ingestion

# Argument to provide for Ingestion Dependencies to install. Defaults to all
ARG INGESTION_DEPENDENCY="all"
RUN pip install ".[${INGESTION_DEPENDENCY}]"
RUN PIP_CONSTRAINT=/home/airflow/ingestion/constraints.txt pip install ".[${INGESTION_DEPENDENCY}]"

# Temporary workaround for https://github.com/open-metadata/OpenMetadata/issues/9593
RUN echo "Image built for $(uname -m)"
Expand Down
10 changes: 5 additions & 5 deletions ingestion/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -11,23 +11,23 @@ endif

.PHONY: install
install: ## Install the ingestion module to the current environment
python -m pip install $(INGESTION_DIR)/
PIP_CONSTRAINT=$(INGESTION_DIR)/constraints.txt python -m pip install $(INGESTION_DIR)/

.PHONY: install_dev
install_dev: ## Install the ingestion module with dev dependencies
python -m pip install "$(INGESTION_DIR)[dev]/"
PIP_CONSTRAINT=$(INGESTION_DIR)/constraints.txt python -m pip install "$(INGESTION_DIR)[dev]/"

.PHONY: install_test
install_test: ## Install the ingestion module with test dependencies
python -m pip install "$(INGESTION_DIR)[test]/"
PIP_CONSTRAINT=$(INGESTION_DIR)/constraints.txt python -m pip install "$(INGESTION_DIR)[test]/"

.PHONY: install_all
install_all: ## Install the ingestion module with all dependencies
python -m pip install "$(INGESTION_DIR)[all]/"
PIP_CONSTRAINT=$(INGESTION_DIR)/constraints.txt python -m pip install "$(INGESTION_DIR)[all]/"

.PHONY: install_apis
install_apis: ## Install the REST APIs module to the current environment
python -m pip install $(ROOT_DIR)/openmetadata-airflow-apis/ setuptools==69.0.2
PIP_CONSTRAINT=$(ROOT_DIR)/openmetadata-airflow-apis/constraints.txt python -m pip install $(ROOT_DIR)/openmetadata-airflow-apis/ setuptools==69.0.2

.PHONY: lint
lint: ## Run pylint on the Python sources to analyze the codebase
Expand Down
1 change: 1 addition & 0 deletions ingestion/constraints.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
setuptools<72
1 change: 1 addition & 0 deletions openmetadata-airflow-apis/constraints.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
setuptools<72

0 comments on commit 1962ffa

Please sign in to comment.