mirror of
https://github.com/open-metadata/OpenMetadata
synced 2026-05-24 09:39:11 +00:00
* ci: reduce checkout history footprint in PR workflows Optimize actions/checkout usage to avoid downloading the full repo blob history on every PR run. The repo is large, so cloning everything just to run tests wastes minutes of CI time per job. - py-operator-build-test.yml: drop fetch-depth: 0 (no history needed) - openmetadata-service-unit-tests.yml: drop fetch-depth: 0 (Sonar is explicitly skipped via -Dsonar.skip=true); shallow-fetch PR base ref - airflow-apis-tests.yml, py-tests.yml, yarn-coverage.yml: add filter: blob:none to Sonar jobs so commits/trees remain available for blame while blobs are fetched lazily on demand - ui-checkstyle.yml: add filter: blob:none to all jobs that rely on tj-actions/changed-files (needs commit/tree metadata, not blobs) * ci: drop fetch-depth: 0 from jobs that don't walk history Follow-up audit after the initial pass. Four jobs were still declaring fetch-depth: 0 (plus filter: blob:none in two cases) without actually needing any history beyond HEAD. ui-checkstyle.yml - i18n-sync: runs 'yarn i18n' then 'git status --porcelain'. git status compares the working tree to HEAD; no history walk. Default depth 1 is sufficient. - app-docs: same pattern with 'yarn generate:app-docs'. py-sonarcloud-nightly.yml - py-unit-tests: only uploads a coverage artifact, no Sonar invocation. - py-integration-tests: same. - py-combine-coverage: does run SonarSource/sonarqube-scan-action, so it genuinely needs the commit graph — added filter: blob:none for parity with the PR Sonar jobs. * ci: remove unused 'Fetch PR base branch' step from service unit tests Copilot review flagged that the step was using --depth=1 while the main checkout is also shallow, which would break any merge-base operation. On investigation, nothing downstream actually uses the base ref: the only command that runs after the checkout is 'mvn ... -Dsonar.skip=true', which has no git dependency. The step was preserved defensively in the previous commit, but it's dead code — cleanest fix is to delete it.
227 lines
6.6 KiB
YAML
227 lines
6.6 KiB
YAML
# Copyright 2021 Collate
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
|
|
name: SonarCloud Ingestion Nightly
|
|
|
|
on:
|
|
workflow_dispatch:
|
|
schedule:
|
|
- cron: "0 2 * * *"
|
|
|
|
permissions:
|
|
contents: read
|
|
|
|
concurrency:
|
|
group: sonarcloud-ingestion-nightly
|
|
cancel-in-progress: true
|
|
|
|
env:
|
|
PYTHON_VERSION: "3.10"
|
|
BRANCH_NAME: "main"
|
|
PROJECT_BASE_DIR: "ingestion"
|
|
|
|
jobs:
|
|
py-unit-tests:
|
|
name: Unit Tests
|
|
timeout-minutes: 60
|
|
runs-on: ubuntu-latest
|
|
steps:
|
|
- name: Free Disk Space (Ubuntu)
|
|
uses: jlumbroso/free-disk-space@main
|
|
with:
|
|
tool-cache: false
|
|
android: true
|
|
dotnet: true
|
|
haskell: true
|
|
large-packages: false
|
|
swap-storage: true
|
|
docker-images: false
|
|
|
|
- name: Checkout
|
|
uses: actions/checkout@v4
|
|
with:
|
|
ref: ${{ env.BRANCH_NAME }}
|
|
|
|
- name: Setup Openmetadata Test Environment
|
|
uses: ./.github/actions/setup-openmetadata-test-environment
|
|
with:
|
|
python-version: ${{ env.PYTHON_VERSION }}
|
|
install-server: 'false'
|
|
|
|
- name: Run Unit Tests
|
|
run: |
|
|
source env/bin/activate
|
|
cd ingestion
|
|
nox --no-venv -s unit-tests
|
|
shell: bash
|
|
|
|
- name: Upload coverage artifact
|
|
if: ${{ !cancelled() }}
|
|
uses: actions/upload-artifact@v4
|
|
with:
|
|
name: coverage-unit
|
|
path: ingestion/.coverage
|
|
include-hidden-files: true
|
|
|
|
py-integration-tests:
|
|
name: "Integration Tests (${{ matrix.shard.name }})"
|
|
timeout-minutes: 180
|
|
runs-on: ubuntu-latest
|
|
strategy:
|
|
fail-fast: false
|
|
matrix:
|
|
shard:
|
|
- name: "shard-1"
|
|
nox-args: >-
|
|
tests/integration/ometa
|
|
tests/integration/postgres
|
|
tests/integration/mysql
|
|
tests/integration/profiler
|
|
tests/integration/data_quality
|
|
- name: "shard-2"
|
|
nox-args: >-
|
|
--ignore=tests/integration/ometa
|
|
--ignore=tests/integration/postgres
|
|
--ignore=tests/integration/mysql
|
|
--ignore=tests/integration/profiler
|
|
--ignore=tests/integration/data_quality
|
|
steps:
|
|
- name: Free Disk Space (Ubuntu)
|
|
uses: jlumbroso/free-disk-space@main
|
|
with:
|
|
tool-cache: false
|
|
android: true
|
|
dotnet: true
|
|
haskell: true
|
|
large-packages: false
|
|
swap-storage: true
|
|
docker-images: false
|
|
|
|
- name: Checkout
|
|
uses: actions/checkout@v4
|
|
with:
|
|
ref: ${{ env.BRANCH_NAME }}
|
|
|
|
- name: Setup Openmetadata Test Environment
|
|
uses: ./.github/actions/setup-openmetadata-test-environment
|
|
with:
|
|
python-version: ${{ env.PYTHON_VERSION }}
|
|
args: "-m no-ui"
|
|
ingestion_dependency: "mysql,elasticsearch,sample-data"
|
|
|
|
- name: Run Integration Tests
|
|
run: |
|
|
source env/bin/activate
|
|
cd ingestion
|
|
nox --no-venv -s integration-tests -- --standalone --durations=5 ${{ matrix.shard.nox-args }}
|
|
env:
|
|
TESTCONTAINERS_RYUK_DISABLED: true
|
|
shell: bash
|
|
|
|
- name: Upload coverage artifact
|
|
if: ${{ !cancelled() }}
|
|
uses: actions/upload-artifact@v4
|
|
with:
|
|
name: coverage-integration-${{ matrix.shard.name }}
|
|
path: ingestion/.coverage
|
|
include-hidden-files: true
|
|
|
|
- name: Clean Up
|
|
if: ${{ !cancelled() }}
|
|
run: |
|
|
cd ./docker/development
|
|
docker compose down --remove-orphans
|
|
sudo rm -rf ${PWD}/docker-volume
|
|
|
|
py-combine-coverage:
|
|
if: ${{ !cancelled() }}
|
|
needs: [py-unit-tests, py-integration-tests]
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 10
|
|
steps:
|
|
- name: Checkout
|
|
uses: actions/checkout@v4
|
|
with:
|
|
ref: ${{ env.BRANCH_NAME }}
|
|
fetch-depth: 0
|
|
filter: blob:none
|
|
|
|
- name: Setup Python
|
|
uses: actions/setup-python@v5
|
|
with:
|
|
python-version: ${{ env.PYTHON_VERSION }}
|
|
|
|
- name: Install uv
|
|
run: pip install uv
|
|
shell: bash
|
|
|
|
- name: Install coverage
|
|
run: |
|
|
python3 -m venv env
|
|
source env/bin/activate
|
|
uv pip install "coverage[toml]" nox
|
|
shell: bash
|
|
|
|
- name: Download coverage artifacts
|
|
uses: actions/download-artifact@v4
|
|
with:
|
|
pattern: coverage-*
|
|
path: ingestion/coverage-data/
|
|
|
|
- name: Prepare coverage files
|
|
run: |
|
|
cd ingestion
|
|
[ -f coverage-data/coverage-unit/.coverage ] && mv coverage-data/coverage-unit/.coverage .coverage.unit
|
|
for dir in coverage-data/coverage-integration-*/; do
|
|
shard=$(basename "$dir" | sed 's/coverage-integration-//')
|
|
[ -f "$dir/.coverage" ] && mv "$dir/.coverage" ".coverage.integration-$shard"
|
|
done
|
|
shell: bash
|
|
|
|
- name: Combine coverage
|
|
run: |
|
|
source env/bin/activate
|
|
cd ingestion
|
|
nox --no-venv -s combine-coverage
|
|
shell: bash
|
|
|
|
- name: Remove pom.xml
|
|
run: rm pom.xml
|
|
shell: bash
|
|
|
|
- name: Push Results To Sonar
|
|
if: ${{ !cancelled() }}
|
|
id: push-to-sonar
|
|
continue-on-error: true
|
|
uses: SonarSource/sonarqube-scan-action@v7
|
|
env:
|
|
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
|
SONAR_TOKEN: ${{ secrets.INGESTION_SONAR_SECRET }}
|
|
with:
|
|
projectBaseDir: ${{ env.PROJECT_BASE_DIR }}/
|
|
args: >
|
|
-Dsonar.branch.name=${{ env.BRANCH_NAME }}
|
|
|
|
- name: Wait to retry 'Push Results to Sonar'
|
|
if: ${{ !cancelled() && steps.push-to-sonar.outcome != 'success' }}
|
|
run: sleep 20s
|
|
shell: bash
|
|
|
|
- name: Retry 'Push Results to Sonar'
|
|
if: ${{ !cancelled() && steps.push-to-sonar.outcome != 'success' }}
|
|
uses: SonarSource/sonarqube-scan-action@v7
|
|
env:
|
|
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
|
SONAR_TOKEN: ${{ secrets.INGESTION_SONAR_SECRET }}
|
|
with:
|
|
projectBaseDir: ${{ env.PROJECT_BASE_DIR }}/
|
|
args: >
|
|
-Dsonar.branch.name=${{ env.BRANCH_NAME }}
|