chore - Do not install playwright in CI build steps (#7786)

This commit is contained in:
Ray Myers
2025-04-10 11:55:02 -05:00
committed by GitHub
parent 513f7ab7e7
commit a5c57fbd3a
2 changed files with 24 additions and 15 deletions

View File

@@ -145,7 +145,7 @@ jobs:
- name: Install poetry via pipx - name: Install poetry via pipx
run: pipx install poetry run: pipx install poetry
- name: Install Python dependencies using Poetry - name: Install Python dependencies using Poetry
run: make install-python-dependencies run: make install-python-dependencies POETRY_GROUP=main INSTALL_PLAYWRIGHT=0
- name: Create source distribution and Dockerfile - name: Create source distribution and Dockerfile
run: poetry run python3 openhands/runtime/utils/runtime_build.py --base_image ${{ matrix.base_image.image }} --build_folder containers/runtime --force_rebuild run: poetry run python3 openhands/runtime/utils/runtime_build.py --base_image ${{ matrix.base_image.image }} --build_folder containers/runtime --force_rebuild
- name: Lowercase Repository Owner - name: Lowercase Repository Owner
@@ -220,7 +220,7 @@ jobs:
- name: Install poetry via pipx - name: Install poetry via pipx
run: pipx install poetry run: pipx install poetry
- name: Install Python dependencies using Poetry - name: Install Python dependencies using Poetry
run: make install-python-dependencies run: make install-python-dependencies POETRY_GROUP=main INSTALL_PLAYWRIGHT=0
- name: Get hash in App Image - name: Get hash in App Image
run: | run: |
echo "Hash from app image: ${{ needs.ghcr_build_app.outputs.hash_from_app_image }}" echo "Hash from app image: ${{ needs.ghcr_build_app.outputs.hash_from_app_image }}"
@@ -286,7 +286,7 @@ jobs:
- name: Install poetry via pipx - name: Install poetry via pipx
run: pipx install poetry run: pipx install poetry
- name: Install Python dependencies using Poetry - name: Install Python dependencies using Poetry
run: make install-python-dependencies run: make install-python-dependencies INSTALL_PLAYWRIGHT=0
- name: Lowercase Repository Owner - name: Lowercase Repository Owner
run: | run: |
echo REPO_OWNER=$(echo ${{ github.repository_owner }} | tr '[:upper:]' '[:lower:]') >> $GITHUB_ENV echo REPO_OWNER=$(echo ${{ github.repository_owner }} | tr '[:upper:]' '[:lower:]') >> $GITHUB_ENV
@@ -355,7 +355,7 @@ jobs:
- name: Install poetry via pipx - name: Install poetry via pipx
run: pipx install poetry run: pipx install poetry
- name: Install Python dependencies using Poetry - name: Install Python dependencies using Poetry
run: make install-python-dependencies run: make install-python-dependencies INSTALL_PLAYWRIGHT=0
- name: Lowercase Repository Owner - name: Lowercase Repository Owner
run: | run: |
echo REPO_OWNER=$(echo ${{ github.repository_owner }} | tr '[:upper:]' '[:lower:]') >> $GITHUB_ENV echo REPO_OWNER=$(echo ${{ github.repository_owner }} | tr '[:upper:]' '[:lower:]') >> $GITHUB_ENV

View File

@@ -133,20 +133,29 @@ install-python-dependencies:
export HNSWLIB_NO_NATIVE=1; \ export HNSWLIB_NO_NATIVE=1; \
poetry run pip install chroma-hnswlib; \ poetry run pip install chroma-hnswlib; \
fi fi
@poetry install @if [ -n "${POETRY_GROUP}" ]; then \
@if [ -f "/etc/manjaro-release" ]; then \ echo "Installing only POETRY_GROUP=${POETRY_GROUP}"; \
echo "$(BLUE)Detected Manjaro Linux. Installing Playwright dependencies...$(RESET)"; \ poetry install --only $${POETRY_GROUP}; \
poetry run pip install playwright; \
poetry run playwright install chromium; \
else \ else \
if [ ! -f cache/playwright_chromium_is_installed.txt ]; then \ poetry install; \
echo "Running playwright install --with-deps chromium..."; \ fi
poetry run playwright install --with-deps chromium; \ @if [ "${INSTALL_PLAYWRIGHT}" != "false" ] && [ "${INSTALL_PLAYWRIGHT}" != "0" ]; then \
mkdir -p cache; \ if [ -f "/etc/manjaro-release" ]; then \
touch cache/playwright_chromium_is_installed.txt; \ echo "$(BLUE)Detected Manjaro Linux. Installing Playwright dependencies...$(RESET)"; \
poetry run pip install playwright; \
poetry run playwright install chromium; \
else \ else \
echo "Setup already done. Skipping playwright installation."; \ if [ ! -f cache/playwright_chromium_is_installed.txt ]; then \
echo "Running playwright install --with-deps chromium..."; \
poetry run playwright install --with-deps chromium; \
mkdir -p cache; \
touch cache/playwright_chromium_is_installed.txt; \
else \
echo "Setup already done. Skipping playwright installation."; \
fi \
fi \ fi \
else \
echo "Skipping Playwright installation (INSTALL_PLAYWRIGHT=${INSTALL_PLAYWRIGHT})."; \
fi fi
@echo "$(GREEN)Python dependencies installed successfully.$(RESET)" @echo "$(GREEN)Python dependencies installed successfully.$(RESET)"