Skip to content
DataPusher+ Integration CI

DataPusher+ Integration CI #279

Sign in to view logs

DataPusher+ Integration CI

DataPusher+ Integration CI #279

Workflow file for this run

.github/workflows/ci.yml at 474ce97
name: DataPusher+ Integration CI
on:
push:
branches: [main, dev]
pull_request:
branches: [main]
schedule:
- cron: '0 2 * * *' # nightly at 02:00 UTC
workflow_dispatch:
inputs:
testing_directory:
description: 'Test files directory (in datapusher-plus_testing/tests/)'
required: false
default: 'quick'
type: string
qsv_version:
description: 'qsv version to install'
required: false
default: '20.1.0'
type: string
polling_timeout_seconds:
description: 'Max seconds to wait for DataPusher to process each file'
required: false
default: '90'
type: string
concurrency:
group: dp-ci-${{ github.ref }}
cancel-in-progress: true
permissions:
contents: read
env:
FILES_DIR: ${{ github.event.inputs.testing_directory || 'quick' }}
QSV_VER: ${{ github.event.inputs.qsv_version || '20.1.0' }}
# 90s ceiling (45 polls x 2s): a Prefect flow run spawns a fresh interpreter
# and bootstraps the full CKAN app context, so per-job latency is ~20-30s --
# well above the old RQ path. Matches main.yml's polling window.
POLLING_TIMEOUT: ${{ github.event.inputs.polling_timeout_seconds || '90' }}
TESTING_REPO_RAW: 'https://raw.githubusercontent.com/dathere/datapusher-plus_testing/main'
TESTING_REPO_API: 'https://api.github.com/repos/dathere/datapusher-plus_testing/contents'
CKAN_VERSION: "2.11"
POSTGRES_PASSWORD: postgres
CKAN_DB_PASSWORD: pass
CKAN_SITE_URL: http://localhost:5000
CKAN_SITE_ID: default
CKAN_SITE_TITLE: "CKAN Test Instance"
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: true
jobs:
integration-test:
runs-on: ubuntu-latest
timeout-minutes: 90
container:
image: ckan/ckan-dev:2.11
options: --user root
services:
solr:
image: ckan/ckan-solr:2.11-solr9
ports: ["8983:8983"]
postgres:
image: postgres:15
env:
POSTGRES_USER: postgres
POSTGRES_PASSWORD: postgres
POSTGRES_DB: postgres
ports: ["5432:5432"]
options: >-
--health-cmd "pg_isready -h 127.0.0.1 -U postgres -p 5432"
--health-interval 10s
--health-timeout 5s
--health-retries 10
--health-start-period 10s
redis:
image: redis:3
ports: ["6379:6379"]
env:
CKAN_SQLALCHEMY_URL: postgresql://ckan_default:pass@postgres/ckan_test
CKAN_DATASTORE_WRITE_URL: postgresql://datastore_write:pass@postgres/datastore_test
CKAN_DATASTORE_READ_URL: postgresql://datastore_read:pass@postgres/datastore_test
CKAN_SOLR_URL: http://solr:8983/solr/ckan
CKAN_REDIS_URL: redis://redis:6379/1
CKAN_SITE_URL: http://localhost:5000
steps:
- name: Fix permissions and install all system dependencies
run: |
mkdir -p /__w/_temp
chmod -R 777 /__w/_temp
chmod -R 777 /__w/
apt-get update -y
apt-get install -y \
curl wget net-tools procps postgresql-client jq \
python3-virtualenv python3-dev python3-pip python3-wheel \
build-essential libxslt1-dev libxml2-dev libffi-dev libpq-dev \
zlib1g-dev git uchardet unzip \
gdal-bin libgdal-dev libspatialindex-dev libgeos-dev libproj-dev \
libc6 libgcc-s1 libstdc++6
echo "System dependencies installed"
- name: Checkout datapusher-plus
uses: actions/checkout@v4
- name: Fetch log analyzer from testing repo
run: |
curl -fsSL "${TESTING_REPO_RAW}/tests/log_analyzer.py" -o /tmp/log_analyzer.py
echo "log_analyzer.py downloaded"
- name: Wait for PostgreSQL
run: |
timeout=90
until PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "SELECT 1;" >/dev/null 2>&1; do
[ $timeout -le 0 ] && echo "Timeout waiting for PostgreSQL" && exit 1
echo "Postgres not ready yet ($timeout s left)..."
sleep 3
timeout=$((timeout - 3))
done
echo "PostgreSQL ready"
- name: Setup database users and permissions
run: |
set -eu
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -Atc "SELECT 1 FROM pg_roles WHERE rolname='ckan_default'" | grep -q 1 || \
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "CREATE USER ckan_default WITH PASSWORD '$CKAN_DB_PASSWORD';"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -Atc "SELECT 1 FROM pg_roles WHERE rolname='datastore_write'" | grep -q 1 || \
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "CREATE USER datastore_write WITH PASSWORD '$CKAN_DB_PASSWORD';"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -Atc "SELECT 1 FROM pg_roles WHERE rolname='datastore_read'" | grep -q 1 || \
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "CREATE USER datastore_read WITH PASSWORD '$CKAN_DB_PASSWORD';"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -Atc "SELECT 1 FROM pg_database WHERE datname='ckan_test'" | grep -q 1 || \
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "CREATE DATABASE ckan_test OWNER ckan_default;"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -Atc "SELECT 1 FROM pg_database WHERE datname='datastore_test'" | grep -q 1 || \
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "CREATE DATABASE datastore_test OWNER ckan_default;"
# Prefect server backing database (shared Postgres; separate logical DB).
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -Atc "SELECT 1 FROM pg_database WHERE datname='prefect_test'" | grep -q 1 || \
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "CREATE DATABASE prefect_test OWNER postgres;"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "GRANT ALL PRIVILEGES ON DATABASE ckan_test TO ckan_default;"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "GRANT ALL PRIVILEGES ON DATABASE datastore_test TO datastore_write;"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "GRANT CONNECT ON DATABASE datastore_test TO datastore_read;"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -d ckan_test -c "GRANT USAGE, CREATE ON SCHEMA public TO ckan_default;"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -d datastore_test -c "GRANT USAGE, CREATE ON SCHEMA public TO datastore_write;"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -d datastore_test -c "GRANT USAGE ON SCHEMA public TO datastore_read;"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -d datastore_test -c "GRANT USAGE ON SCHEMA public TO ckan_default;"
echo "Database setup completed"
- name: Install Python dependencies and datapusher-plus
run: |
set -eu
export GDAL_VERSION=$(gdal-config --version)
echo "GDAL version: $GDAL_VERSION"
export CPLUS_INCLUDE_PATH=/usr/include/gdal
export C_INCLUDE_PATH=/usr/include/gdal
python3 -m pip install --upgrade pip setuptools wheel
pip install "GDAL==$GDAL_VERSION"
# Install datapusher-plus from local checkout (this repo)
echo "Installing datapusher-plus from local checkout: $GITHUB_WORKSPACE"
cd "$GITHUB_WORKSPACE"
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
if [ -f requirements-dev.txt ]; then pip install -r requirements-dev.txt; fi
pip install -e .
pip install --upgrade ckanapi
pip install -e 'git+https://github.com/ckan/ckanext-scheming.git#egg=ckanext-scheming'
echo "Installation complete (branch: ${{ github.ref_name }}, sha: ${{ github.sha }})"
- name: Check GLIBC version
run: |
echo "=== System Information ==="
lsb_release -a 2>/dev/null || cat /etc/os-release
echo "=== GLIBC Version ==="
ldd --version | head -n1
- name: Install qsv
run: |
set -eu
QSV_DEB_REPO_URL="https://dathere.github.io/qsv-deb-releases"
QSV_BIN_TARGET="/usr/local/bin/qsvdp"
# Try the official Debian package first (qsv-datapusher-plus, which
# ships the qsvdp binary). The apt repo at $QSV_DEB_REPO_URL only
# carries the latest version per package, so we fall back to the
# GitHub release zip when the requested QSV_VER doesn't match the
# repo's candidate (or when anything else about the deb path fails).
# See https://github.com/dathere/qsv#debian-package
# On any failure inside install_via_deb, drop the qsv apt source +
# keyring so subsequent apt-get invocations in later steps don't
# keep hitting (and possibly erroring on) the qsv repo.
_cleanup_qsv_apt_state() {
rm -f /etc/apt/sources.list.d/qsv.list /usr/share/keyrings/qsv-deb.gpg
}
install_via_deb() {
echo "=== Attempting Debian package install (qsv-datapusher-plus) ==="
if _install_via_deb_inner; then
return 0
fi
_cleanup_qsv_apt_state
return 1
}
_install_via_deb_inner() {
# Refresh the apt cache up front so the gnupg install below
# can't 404 on stale package files even if no earlier step has
# run apt-get update recently.
apt-get update -y \
|| { echo "apt-get update failed before gnupg install"; return 1; }
# gpg isn't in the ckan-dev base image; install it if missing so we
# can use the documented signed-by approach.
if ! command -v gpg >/dev/null 2>&1; then
apt-get install -y --no-install-recommends gnupg \
|| { echo "Failed to install gnupg"; return 1; }
fi
install -d -m 0755 /usr/share/keyrings
# Split the wget|gpg pipeline into two checked steps: this
# workflow's "Install qsv" step runs under `sh -e {0}` (dash on
# the ckan-dev image), where `set -o pipefail` isn't reliably
# available, so a piped wget failure would otherwise be masked
# by gpg --dearmor's exit status.
wget -qO /tmp/qsv-deb.gpg "${QSV_DEB_REPO_URL}/qsv-deb.gpg" \
|| { echo "Failed to download qsv repo GPG key"; return 1; }
gpg --dearmor -o /usr/share/keyrings/qsv-deb.gpg < /tmp/qsv-deb.gpg \
|| { echo "Could not dearmor qsv repo GPG key"; return 1; }
rm -f /tmp/qsv-deb.gpg
echo "deb [signed-by=/usr/share/keyrings/qsv-deb.gpg] ${QSV_DEB_REPO_URL} ./" \
> /etc/apt/sources.list.d/qsv.list
apt-get update -y \
|| { echo "apt-get update failed after adding qsv repo"; return 1; }
# Only proceed if the repo's candidate version matches QSV_VER —
# the apt repo carries one version per package, so older QSV_VER
# values must use the zip flow.
candidate=$(apt-cache policy qsv-datapusher-plus 2>/dev/null | awk '/Candidate:/ {print $2; exit}')
echo "qsv-datapusher-plus repo candidate: ${candidate:-<none>}"
echo "Requested QSV_VER: ${QSV_VER}"
case "$candidate" in
"${QSV_VER}"|"${QSV_VER}-"*) ;;
*)
echo "Repo candidate does not match QSV_VER; skipping deb install"
return 1
;;
esac
apt-get install -y qsv-datapusher-plus \
|| { echo "apt-get install qsv-datapusher-plus failed"; return 1; }
# qsv-datapusher-plus 20.0.0-1 installs the binary at
# /usr/local/bin/qsvdp directly (= ${QSV_BIN_TARGET}). Accept
# /usr/bin/qsvdp too in case a future package revision moves to
# the more conventional Debian path, and symlink it to the path
# the rest of this workflow + the CKAN config (qsv_bin) hard-code.
if [ -x "${QSV_BIN_TARGET}" ]; then
installed="${QSV_BIN_TARGET}"
elif [ -x /usr/bin/qsvdp ]; then
installed=/usr/bin/qsvdp
ln -sf /usr/bin/qsvdp "${QSV_BIN_TARGET}"
else
echo "ERROR: qsvdp not found on PATH after deb install. Package contents:"
dpkg -L qsv-datapusher-plus || true
return 1
fi
"${QSV_BIN_TARGET}" --version \
|| { echo "deb-installed qsvdp failed to run (libc/libstdc++ mismatch?)"; return 1; }
echo "qsv installed via Debian package successfully (qsv-datapusher-plus ${candidate}, binary at ${installed})"
}
install_via_zip() {
QSV_ZIP="qsv-${QSV_VER}-x86_64-unknown-linux-gnu.zip"
QSV_URL="https://github.com/dathere/qsv/releases/download/${QSV_VER}/${QSV_ZIP}"
echo "Downloading qsv GNU version $QSV_VER"
mkdir -p /tmp/qsv-install && cd /tmp/qsv-install
wget -q "$QSV_URL" -O "$QSV_ZIP" || { echo "Failed to download qsv"; return 1; }
unzip -q "$QSV_ZIP"
ls -lh
# Always install as /usr/local/bin/qsvdp so the CKAN config (qsv_bin)
# path is stable regardless of whether the archive ships qsvdp or qsv.
if [ -f "qsvdp" ]; then SRC_BIN="qsvdp"
elif [ -f "qsv" ]; then SRC_BIN="qsv"
else echo "ERROR: No qsv binary found"; ls -la; return 1; fi
chmod +x "$SRC_BIN"
mv -f "$SRC_BIN" "${QSV_BIN_TARGET}"
if ! "${QSV_BIN_TARGET}" --version; then
echo "GNU version failed. Falling back to musl..."
rm -f "${QSV_BIN_TARGET}" "$QSV_ZIP"
QSV_ZIP="qsv-${QSV_VER}-x86_64-unknown-linux-musl.zip"
wget -q "https://github.com/dathere/qsv/releases/download/${QSV_VER}/${QSV_ZIP}" -O "$QSV_ZIP" || return 1
unzip -qo "$QSV_ZIP"
if [ -f "qsvdp" ]; then SRC_BIN="qsvdp"
elif [ -f "qsv" ]; then SRC_BIN="qsv"
else echo "ERROR: No qsv binary found in musl archive"; ls -la; return 1; fi
chmod +x "$SRC_BIN"
mv -f "$SRC_BIN" "${QSV_BIN_TARGET}"
"${QSV_BIN_TARGET}" --version || { echo "ERROR: musl qsv also failed"; return 1; }
else
echo "qsv GNU version installed successfully!"
fi
cd / && rm -rf /tmp/qsv-install
}
if install_via_deb; then
echo "qsv install method: Debian package"
elif install_via_zip; then
echo "qsv install method: GitHub release zip"
else
echo "ERROR: all qsv install methods failed"
exit 1
fi
- name: Run qsv contract regression tests
env:
QSV_BIN: /usr/local/bin/qsvdp
# The ckan-dev container's site-packages registers a pytest plugin
# (ckan.tests.pytest_ckan.ckan_setup) that auto-loads on every
# pytest invocation and calls make_app(conf) in pytest_sessionstart,
# which requires a fully-configured CKAN (SECRET_KEY, etc.). That
# config isn't built until the later "Setup CKAN configuration"
# step. Disable pytest plugin autoload so the CKAN plugin can't
# hijack this standalone regression run — our tests only need core
# pytest (subprocess + skipif), not CKAN or any other plugin.
PYTEST_DISABLE_PLUGIN_AUTOLOAD: "1"
run: |
set -eu
# Fast pre-flight: assert the installed qsv satisfies the contracts
# DP+ relies on (safe_header_names byte-cap, safenames verify-mode
# JSON shape, stats percentile-label-prefix format, etc.). If qsv
# ever ships a version that breaks these, fail here in <1 minute
# rather than after spinning up the full CKAN + datastore + worker.
# See tests/test_qsv_v20_regression.py and
# docs/qsv-20.0.0-upgrade-test-plan.md.
echo "Asserting qsv ${QSV_VER} satisfies DP+'s contract expectations..."
python3 -m pytest tests/test_qsv_v20_regression.py -v
- name: Setup CKAN configuration
run: |
set -eu
if ! grep -q "^solr_url" /srv/app/src/ckan/test-core.ini; then
echo "solr_url = ${CKAN_SOLR_URL}" >> /srv/app/src/ckan/test-core.ini
fi
if ! grep -q "^ckan.redis.url" /srv/app/src/ckan/test-core.ini; then
echo "ckan.redis.url = ${CKAN_REDIS_URL}" >> /srv/app/src/ckan/test-core.ini
fi
REPLACE_FILE="$(mktemp)"
ADD_FILE="$(mktemp)"
MISSING_ADD_FILE="$(mktemp)"
: > "$REPLACE_FILE"; : > "$ADD_FILE"; : > "$MISSING_ADD_FILE"
printf '%s\n' \
"ckan.site_url|${CKAN_SITE_URL}" \
"sqlalchemy.url|${CKAN_SQLALCHEMY_URL}" \
"ckan.datastore.write_url|${CKAN_DATASTORE_WRITE_URL}" \
"ckan.datastore.read_url|${CKAN_DATASTORE_READ_URL}" \
"solr_url|${CKAN_SOLR_URL}" \
"ckan.redis.url|${CKAN_REDIS_URL}" \
> "$REPLACE_FILE"
cat > "$ADD_FILE" <<'EOF'
ckan.site_id = default
ckan.site_title = CKAN Test
ckan.auth.create_default_api_keys = true
ckanext.datapusher_plus.qsv_bin = /usr/local/bin/qsvdp
scheming.dataset_schemas = ckanext.datapusher_plus:dataset-druf.yaml
scheming.presets = ckanext.scheming:presets.json
scheming.dataset_fallback = false
ckanext.datapusher_plus.download_proxy =
ckanext.datapusher_plus.ssl_verify = false
ckanext.datapusher_plus.upload_log_level = INFO
ckanext.datapusher_plus.formats = csv tsv tab ssv xls xlsx xlsb xlsm ods geojson shp qgis zip
# CKAN's bundled test-core.ini sets the legacy `ckan.datapusher.formats` key, which DPP's
# _submit_to_datapusher checks FIRST (`tk.config.get("ckan.datapusher.formats") or ...`).
# Without overriding it here, the legacy list (csv/xls/xlsx/tsv only) wins and DPP silently
# refuses to auto-submit ssv/tab/ods/geojson/shp/qgis/zip resources.
ckan.datapusher.formats = csv tsv tab ssv xls xlsx xlsb xlsm ods geojson shp qgis zip
ckanext.datapusher_plus.pii_screening = false
ckanext.datapusher_plus.pii_found_abort = false
ckanext.datapusher_plus.pii_regex_resource_id_or_alias =
ckanext.datapusher_plus.pii_show_candidates = false
ckanext.datapusher_plus.pii_quick_screen = false
ckanext.datapusher_plus.preview_rows = 100
ckanext.datapusher_plus.download_timeout = 300
ckanext.datapusher_plus.max_content_length = 1256000000000
ckanext.datapusher_plus.chunk_size = 16384
ckanext.datapusher_plus.default_excel_sheet = 0
ckanext.datapusher_plus.sort_and_dupe_check = true
ckanext.datapusher_plus.dedup = false
ckanext.datapusher_plus.unsafe_prefix = unsafe_
ckanext.datapusher_plus.reserved_colnames = _id
ckanext.datapusher_plus.prefer_dmy = false
ckanext.datapusher_plus.ignore_file_hash = true
ckanext.datapusher_plus.auto_index_threshold = 3
ckanext.datapusher_plus.auto_index_dates = true
ckanext.datapusher_plus.auto_unique_index = true
ckanext.datapusher_plus.summary_stats_options =
ckanext.datapusher_plus.add_summary_stats_resource = false
ckanext.datapusher_plus.summary_stats_with_preview = false
ckanext.datapusher_plus.qsv_stats_string_max_length = 32767
ckanext.datapusher_plus.qsv_dates_whitelist = date,time,due,open,close,created
ckanext.datapusher_plus.qsv_freq_limit = 10
ckanext.datapusher_plus.auto_alias = true
ckanext.datapusher_plus.auto_alias_unique = false
ckanext.datapusher_plus.copy_readbuffer_size = 1048576
ckanext.datapusher_plus.type_mapping = {"String": "text", "Integer": "numeric","Float": "numeric","DateTime": "timestamp","Date": "date","NULL": "text"}
ckanext.datapusher_plus.auto_spatial_simplification = true
ckanext.datapusher_plus.spatial_simplification_relative_tolerance = 0.1
ckanext.datapusher_plus.latitude_fields = latitude,lat
ckanext.datapusher_plus.longitude_fields = longitude,long,lon
ckanext.datapusher_plus.jinja2_bytecode_cache_dir = /tmp/jinja2_bytecode_cache
ckanext.datapusher_plus.auto_unzip_one_file = true
# v3.0: ingestion runs on Prefect instead of RQ.
ckanext.datapusher_plus.prefect_deployment_name = datapusher-plus/datapusher-plus
ckanext.datapusher_plus.prefect_work_pool = datapusher-plus
ckanext.datapusher_plus.prefect_ui_base = http://localhost:4200
ckanext.datapusher_plus.flow_timeout = 600
ckanext.datapusher_plus.max_quarantine_pct = 5.0
EOF
if [ -f /srv/app/src/ckan/test-core.ini ]; then
awk 'BEGIN{in_sect=0}
/^\[DEFAULT\]/{ print; in_sect=1; next }
/^\[.*\]/{ if(in_sect){ print "debug = true"; in_sect=0 } }
{ if(in_sect){ if($1 == "debug") next; print } else { print } }
END { if(in_sect) print "debug = true" }' /srv/app/src/ckan/test-core.ini > /srv/app/src/ckan/test-core.ini.tmp \
&& mv /srv/app/src/ckan/test-core.ini.tmp /srv/app/src/ckan/test-core.ini
while IFS= read -r entry || [ -n "$entry" ]; do
key="$(printf '%s' "$entry" | cut -d'|' -f1)"
value="$(printf '%s' "$entry" | cut -d'|' -f2-)"
esc_value="$(printf '%s' "$value" | sed -e 's/[\/&]/\\&/g')"
if grep -q -E "^[[:space:]]*$(printf '%s' "$key" | sed 's/[][^$.*/]/\\&/g')[[:space:]]*=" /srv/app/src/ckan/test-core.ini; then
sed -i -E "s|^[[:space:]]*$(printf '%s' "$key" | sed 's/[][^$.*/]/\\&/g')[[:space:]]*=.*|${key} = ${esc_value}|g" /srv/app/src/ckan/test-core.ini
else
printf '%s\n' "${key} = ${value}" >> "$MISSING_ADD_FILE"
fi
done < "$REPLACE_FILE"
while IFS= read -r ln || [ -n "$ln" ]; do
case "$ln" in
\#*)
grep -Fq "$ln" /srv/app/src/ckan/test-core.ini || printf '%s\n' "$ln" >> "$MISSING_ADD_FILE"
;;
*)
key="$(printf '%s' "$ln" | cut -d'=' -f1 | sed 's/[[:space:]]*$//')"
value="$(printf '%s' "$ln" | cut -d'=' -f2- | sed 's/^[[:space:]]*//')"
esc_value="$(printf '%s' "$value" | sed -e 's/[\/&]/\\&/g')"
if grep -q -E "^[[:space:]]*$(printf '%s' "$key" | sed 's/[][^$.*/]/\\&/g')[[:space:]]*=" /srv/app/src/ckan/test-core.ini; then
sed -i -E "s|^[[:space:]]*$(printf '%s' "$key" | sed 's/[][^$.*/]/\\&/g')[[:space:]]*=.*|${key} = ${esc_value}|g" /srv/app/src/ckan/test-core.ini
else
printf '%s\n' "${key} = ${value}" >> "$MISSING_ADD_FILE"
fi
;;
esac
done < "$ADD_FILE"
if [ -s "$MISSING_ADD_FILE" ]; then
awk -v addfile="$MISSING_ADD_FILE" '
BEGIN{ inserted=0; while ((getline line < addfile) > 0) { add[++na]=line }; close(addfile) }
{ print; if(!inserted && $0=="[app:main]") { for(i=1;i<=na;i++) print add[i]; inserted=1 } }
END{ if(!inserted){ print "[app:main]"; for(i=1;i<=na;i++) print add[i] } }
' /srv/app/src/ckan/test-core.ini > /srv/app/src/ckan/test-core.ini.new \
&& mv /srv/app/src/ckan/test-core.ini.new /srv/app/src/ckan/test-core.ini
fi
sed -i "s|^sqlalchemy.url.*|sqlalchemy.url = ${CKAN_SQLALCHEMY_URL}|g" /srv/app/src/ckan/test-core.ini
sed -i "s|^ckan.datastore.write_url.*|ckan.datastore.write_url = ${CKAN_DATASTORE_WRITE_URL}|g" /srv/app/src/ckan/test-core.ini
sed -i "s|^ckan.datastore.read_url.*|ckan.datastore.read_url = ${CKAN_DATASTORE_READ_URL}|g" /srv/app/src/ckan/test-core.ini
fi
REQUIRED_PLUGINS="datastore datapusher_plus scheming_datasets"
if grep -q "^ckan.plugins" /srv/app/src/ckan/test-core.ini; then
current=$(grep "^ckan.plugins" /srv/app/src/ckan/test-core.ini | head -n1 | cut -d'=' -f2-)
for p in $REQUIRED_PLUGINS; do
echo "$current" | grep -qw "$p" || current="$current $p"
done
awk -v new="ckan.plugins = $current" 'BEGIN{done=0} {if(!done && $1=="ckan.plugins") {print new; done=1} else print $0}' /srv/app/src/ckan/test-core.ini > /srv/app/src/ckan/test-core.ini.new \
&& mv /srv/app/src/ckan/test-core.ini.new /srv/app/src/ckan/test-core.ini
else
echo "ckan.plugins = $REQUIRED_PLUGINS" >> /srv/app/src/ckan/test-core.ini
fi
echo "---- /srv/app/src/ckan/test-core.ini ----"
cat /srv/app/src/ckan/test-core.ini
echo "---- end ----"
- name: Initialize CKAN database
run: |
if ! PGPASSWORD=$CKAN_DB_PASSWORD psql -h postgres -U ckan_default -d ckan_test -c "SELECT 1;" >/dev/null 2>&1; then
# PostgreSQL does not support `CREATE USER/DATABASE IF NOT EXISTS`; guard with explicit existence checks.
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -Atc "SELECT 1 FROM pg_roles WHERE rolname='ckan_default'" | grep -q 1 || \
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "CREATE USER ckan_default WITH PASSWORD '$CKAN_DB_PASSWORD';"
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -Atc "SELECT 1 FROM pg_database WHERE datname='ckan_test'" | grep -q 1 || \
PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres -c "CREATE DATABASE ckan_test OWNER ckan_default;"
fi
ckan -c /srv/app/src/ckan/test-core.ini db init
echo "CKAN DB initialized."
ckan -c /srv/app/src/ckan/test-core.ini datastore set-permissions 2>/dev/null | grep -v "^[0-9]\{4\}-[0-9]\{2\}-[0-9]\{2\}" | PGPASSWORD=$POSTGRES_PASSWORD psql -h postgres -U postgres --set ON_ERROR_STOP=1
echo "Datastore permissions set."
- name: Initialize DataPusher Plus database
run: |
set -eu
# Fresh CI environment should always migrate cleanly; let migration failures fail the job
# rather than masking schema-init problems and breaking later steps silently.
ckan -c /srv/app/src/ckan/test-core.ini db upgrade -p datapusher_plus
echo "DataPusher Plus database initialized successfully"
- name: Start CKAN server
run: |
set -eu
nohup ckan -c /srv/app/src/ckan/test-core.ini run --host 0.0.0.0 --port 5000 --disable-reloader > /tmp/ckan_stdout.log 2>&1 &
CKAN_PID=$!
echo "CKAN PID=$CKAN_PID"
echo "CKAN_PID=$CKAN_PID" >> $GITHUB_ENV
timeout=120
until curl -fsS "${CKAN_SITE_URL}/api/3/action/status_show" >/dev/null 2>&1; do
if ! kill -0 "$CKAN_PID" >/dev/null 2>&1; then
echo "CKAN process died. Logs:"; tail -n 200 /tmp/ckan_stdout.log; exit 1
fi
[ $timeout -le 0 ] && echo "Timeout waiting for CKAN." && tail -n 200 /tmp/ckan_stdout.log && exit 1
echo "Waiting for CKAN API... ($timeout s left)"
sleep 3; timeout=$((timeout - 3))
done
echo "CKAN started successfully"
- name: Create sysadmin user and get API key
run: |
set -eu
user_response=$(ckanapi action user_create --config /srv/app/src/ckan/test-core.ini \
name=admin_ckan email=admins@example.com password=test1234 \
fullname="CKAN Administrator" with_apitoken=true \
about="Created by CI" 2>/dev/null) || echo "user_create returned non-zero (may already exist)"
ckan -c /srv/app/src/ckan/test-core.ini sysadmin add admin_ckan
json_response=$(echo "$user_response" | sed -n '/{/,/}/p')
api_key=$(echo "$json_response" | jq -r '.token // empty')
if [ -z "$api_key" ] || [ "$api_key" = "null" ]; then
# user_create may have failed (eg user already exists on a re-run) and not returned a token.
# Always mint a fresh token so downstream authenticated calls have a working CKAN_API_KEY.
echo "Minting fresh API token for admin_ckan..."
token_output=$(ckan -c /srv/app/src/ckan/test-core.ini user token add admin_ckan ci-fallback 2>&1)
api_key=$(echo "$token_output" | tail -n 1 | tr -d '\t ')
fi
if [ -z "$api_key" ] || [ "$api_key" = "null" ]; then
echo "ERROR: could not obtain a CKAN API token for admin_ckan"; exit 1
fi
echo "CKAN_API_KEY=$api_key" >> $GITHUB_ENV
echo "User admin_ckan ready"
- name: Create API token for datapusher-plus
run: |
set -eu
dp_token_output=$(ckan -c /srv/app/src/ckan/test-core.ini user token add admin_ckan dpplus 2>&1)
dp_token=$(echo "$dp_token_output" | tail -n 1 | tr -d '\t')
if [ -n "$dp_token" ] && [ "$dp_token" != "null" ]; then
ckan config-tool /srv/app/src/ckan/test-core.ini "ckanext.datapusher_plus.api_token=$dp_token"
echo "DATAPUSHER_PLUS_API_TOKEN=$dp_token" >> $GITHUB_ENV
else
echo "Falling back to main API key..."
ckan config-tool /srv/app/src/ckan/test-core.ini "ckanext.datapusher_plus.api_token=$CKAN_API_KEY"
fi
- name: Create organization and dataset
run: |
set -eu
ckanapi action organization_create --config /srv/app/src/ckan/test-core.ini \
name=demo-organization title="Demo Organization" \
description="CI test org." || echo "May already exist"
ckanapi action organization_member_create --config /srv/app/src/ckan/test-core.ini \
id=demo-organization username=admin_ckan role=admin || echo "May already be member"
ckanapi action package_create \
name=my-first-dataset title="CI Test Dataset" \
owner_org=demo-organization license_id=cc-by \
private:false state=active \
-c /srv/app/src/ckan/test-core.ini || echo "May already exist"
- name: Test datastore functionality
run: |
set -eu
metadata_response=$(curl -s "http://localhost:5000/api/3/action/datastore_search?resource_id=_table_metadata")
if ! echo "$metadata_response" | jq -e '.success == true' >/dev/null 2>&1; then
echo "Datastore read access failed"; exit 1
fi
echo "Datastore read access working"
test_response=$(curl -s -X POST \
-H "Content-Type: application/json" -H "Authorization: $CKAN_API_KEY" \
-d '{"resource":{"package_id":"my-first-dataset"},"fields":[{"id":"col","type":"text"}],"records":[{"col":"test"}]}' \
"http://localhost:5000/api/3/action/datastore_create")
if echo "$test_response" | jq -e '.success == true' >/dev/null 2>&1; then
echo "Datastore write access working"
test_rid=$(echo "$test_response" | jq -r '.result.resource_id')
curl -s -X POST -H "Content-Type: application/json" -H "Authorization: $CKAN_API_KEY" \
-d "{\"resource_id\":\"$test_rid\"}" \
"http://localhost:5000/api/3/action/datastore_delete" >/dev/null
else
echo "Datastore write access failed"; exit 1
fi
- name: Start Prefect server
env:
PREFECT_API_DATABASE_CONNECTION_URL: postgresql+asyncpg://postgres:${{ env.POSTGRES_PASSWORD }}@postgres:5432/prefect_test
PREFECT_SERVER_API_HOST: 0.0.0.0
PREFECT_API_URL: http://localhost:4200/api
run: |
set -eu
echo "PREFECT_API_URL=http://localhost:4200/api" >> $GITHUB_ENV
echo "Starting Prefect server (Postgres-backed) on :4200..."
nohup prefect server start > /tmp/prefect_server.log 2>&1 &
PREFECT_SERVER_PID=$!
echo "PREFECT_SERVER_PID=$PREFECT_SERVER_PID" >> $GITHUB_ENV
# Server start can take 30-60s on first run while Alembic migrations apply.
for i in $(seq 1 60); do
if curl -fsS http://localhost:4200/api/health > /dev/null 2>&1; then
echo "Prefect server is healthy after ${i}s"
break
fi
if ! kill -0 "$PREFECT_SERVER_PID" >/dev/null 2>&1; then
echo "Prefect server died:"
tail -n 80 /tmp/prefect_server.log
exit 1
fi
sleep 1
done
curl -fsS http://localhost:4200/api/health || (tail -n 80 /tmp/prefect_server.log; exit 1)
- name: Register DataPusher+ deployment
env:
PREFECT_API_URL: http://localhost:4200/api
run: |
set -eu
ckan -c /srv/app/src/ckan/test-core.ini datapusher_plus prefect-deploy
- name: Start Prefect worker
env:
PREFECT_API_URL: http://localhost:4200/api
CKAN_INI: /srv/app/src/ckan/test-core.ini
run: |
set -eu
echo "Starting Prefect worker on the datapusher-plus pool..."
nohup prefect worker start --pool datapusher-plus > /tmp/prefect_worker.log 2>&1 &
PREFECT_WORKER_PID=$!
echo "PREFECT_WORKER_PID=$PREFECT_WORKER_PID" >> $GITHUB_ENV
sleep 5
if kill -0 "$PREFECT_WORKER_PID" >/dev/null 2>&1; then
echo "Prefect worker started"
head -n 30 /tmp/prefect_worker.log || true
else
echo "Prefect worker failed to start"
cat /tmp/prefect_worker.log
exit 1
fi
- name: Start CKAN background job worker
run: |
set -eu
# As of DP+ v3.0 the ingestion pipeline runs on Prefect, not RQ.
# CKAN's RQ worker is still started because other CKAN extensions
# (search reindex, mailer, etc.) rely on it.
nohup ckan -c /srv/app/src/ckan/test-core.ini jobs worker > /tmp/ckan_worker.log 2>&1 &
WORKER_PID=$!
echo "CKAN Worker PID=$WORKER_PID"
echo "CKAN_WORKER_PID=$WORKER_PID" >> $GITHUB_ENV
for _i in $(seq 1 15); do
kill -0 "$WORKER_PID" 2>/dev/null && break
sleep 1
done
if kill -0 "$WORKER_PID" >/dev/null 2>&1; then
echo "Worker started successfully"
else
echo "Worker failed to start"; cat /tmp/ckan_worker.log; exit 1
fi
- name: Run DataPusher Plus integration tests
run: |
set -eu
echo "=== DataPusher Plus Integration Tests ==="
echo "Branch: ${{ github.ref_name }} | SHA: ${{ github.sha }}"
echo "Test directory: $FILES_DIR"
echo ""
echo "timestamp,file_name,upload_status,resource_id,datapusher_status,datastore_active,rows_imported,processing_time,error_message" > /tmp/test_results.csv
echo "file_name,reason_skipped" > /tmp/skipped_files.csv
# Discover test files via GitHub Contents API — no clone needed
REPO_DIR_URL="${TESTING_REPO_API}/tests/${FILES_DIR}"
echo "Fetching file list from: $REPO_DIR_URL"
# Authenticated requests get 5000/hr (vs 60/hr unauthenticated) — avoids flaky scheduled runs.
file_list=$(curl -fsSL \
-H "Authorization: Bearer ${{ secrets.GITHUB_TOKEN }}" \
-H "Accept: application/vnd.github+json" \
"$REPO_DIR_URL" | jq -r '.[].name' 2>/dev/null)
if [ -z "$file_list" ]; then
echo "ERROR: Could not list files from $REPO_DIR_URL"
echo "Check FILES_DIR value ($FILES_DIR) and GitHub API availability"
exit 1
fi
echo "Files found in tests/$FILES_DIR:"
echo "$file_list"
echo ""
max_attempts=$(( ${POLLING_TIMEOUT:-20} / 2 ))
[ $max_attempts -lt 1 ] && max_attempts=1
echo "Per-file polling: ${max_attempts} attempts x 2s = ${POLLING_TIMEOUT:-20}s max"
: > /tmp/test_files.txt
echo "$file_list" | while IFS= read -r filename; do
[ -z "$filename" ] && continue
name=$(echo "$filename" | sed 's/\.[^.]*$//')
extension=$(echo "$filename" | sed 's/.*\.//' | tr '[:upper:]' '[:lower:]')
file_url="${TESTING_REPO_RAW}/tests/${FILES_DIR}/${filename}"
case "$extension" in
csv) echo "$name|$file_url|CSV|text/csv|CSV: $filename" >> /tmp/test_files.txt ;;
tsv) echo "$name|$file_url|TSV|text/tab-separated-values|TSV: $filename" >> /tmp/test_files.txt ;;
tab) echo "$name|$file_url|TAB|text/tab-separated-values|TAB: $filename" >> /tmp/test_files.txt ;;
ssv) echo "$name|$file_url|SSV|text/csv|SSV: $filename" >> /tmp/test_files.txt ;;
xls) echo "$name|$file_url|XLS|application/vnd.ms-excel|XLS: $filename" >> /tmp/test_files.txt ;;
xlsx) echo "$name|$file_url|XLSX|application/vnd.openxmlformats-officedocument.spreadsheetml.sheet|XLSX: $filename" >> /tmp/test_files.txt ;;
xlsb) echo "$name|$file_url|XLSB|application/vnd.ms-excel.sheet.binary.macroEnabled.12|XLSB: $filename" >> /tmp/test_files.txt ;;
xlsm) echo "$name|$file_url|XLSM|application/vnd.ms-excel.sheet.macroEnabled.12|XLSM: $filename" >> /tmp/test_files.txt ;;
ods) echo "$name|$file_url|ODS|application/vnd.oasis.opendocument.spreadsheet|ODS: $filename" >> /tmp/test_files.txt ;;
geojson) echo "$name|$file_url|GEOJSON|application/geo+json|GeoJSON: $filename" >> /tmp/test_files.txt ;;
shp) echo "$name|$file_url|SHP|application/x-shp|SHP: $filename" >> /tmp/test_files.txt ;;
qgis) echo "$name|$file_url|QGIS|application/x-qgis|QGIS: $filename" >> /tmp/test_files.txt ;;
zip) echo "$name|$file_url|ZIP|application/zip|ZIP: $filename" >> /tmp/test_files.txt ;;
*)
echo "$filename,Unsupported format: .$extension" >> /tmp/skipped_files.csv
;;
esac
done
if [ ! -s /tmp/test_files.txt ]; then
echo "ERROR: No supported test files found in tests/$FILES_DIR"
exit 1
fi
ckanapi action package_create \
name=dp-ci-test-local-http \
title="DataPusher Plus CI Test (HTTP)" \
owner_org=demo-organization \
-c /srv/app/src/ckan/test-core.ini >/dev/null 2>&1 || true
total_files=0; passed_files=0; failed_files=0; skipped_files=0
while IFS='|' read -r file_name file_url file_format file_mimetype file_desc || [ -n "$file_name" ]; do
[ -z "$file_name" ] && continue
case "$file_name" in '#'*) continue ;; esac
# -f makes 4xx/5xx return non-zero so inaccessible files are reliably skipped.
if ! curl -fsI "$file_url" > /dev/null; then
echo "SKIP: Not accessible: $file_url"
echo "$(basename "$file_url"),Not accessible via HTTP" >> /tmp/skipped_files.csv
skipped_files=$((skipped_files + 1))
continue
fi
total_files=$((total_files + 1))
echo ""
echo "=========================================="
echo "File #${total_files}: $file_name | $file_format"
echo "=========================================="
start_time=$(date +%s)
upload_status="FAILED"; resource_id=""
datapusher_status="N/A"; datastore_active="false"
rows_imported="0"; error_message=""
if resource_response=$(ckanapi action resource_create \
package_id=dp-ci-test-local-http \
url="$file_url" name="CI: $file_name" \
description="$file_desc" format="$file_format" \
mimetype="$file_mimetype" \
-c /srv/app/src/ckan/test-core.ini 2>&1); then
upload_status="SUCCESS"
resource_id=$(echo "$resource_response" | grep -o '"id"[[:space:]]*:[[:space:]]*"[^"]*"' | sed 's/.*"id"[[:space:]]*:[[:space:]]*"\([^"]*\)".*/\1/')
[ -z "$resource_id" ] && resource_id=$(echo "$resource_response" | sed -n 's/.*"id"[[:space:]]*:[[:space:]]*"\([a-f0-9-]*\)".*/\1/p')
echo "Resource ID: $resource_id"
if [ -n "$resource_id" ] && [ "$resource_id" != "null" ]; then
for attempt in $(seq 1 $max_attempts); do
sleep 2
if dp_status_response=$(curl -s -H "Authorization: $CKAN_API_KEY" \
"http://localhost:5000/api/3/action/datapusher_status?resource_id=$resource_id" 2>/dev/null); then
if echo "$dp_status_response" | grep -q '"success"[[:space:]]*:[[:space:]]*true'; then
datapusher_status=$(echo "$dp_status_response" | grep -o '"status"[[:space:]]*:[[:space:]]*"[^"]*"' | head -1 | sed 's/.*"status"[[:space:]]*:[[:space:]]*"\([^"]*\)".*/\1/')
[ -z "$datapusher_status" ] && datapusher_status="unknown"
datapusher_status=$(echo "$datapusher_status" | tr -d '\n\r\t ' | cut -c1-10)
echo " Attempt $attempt/$max_attempts: $datapusher_status"
if [ "$datapusher_status" = "complete" ]; then break
elif [ "$datapusher_status" = "error" ]; then
error_info=$(echo "$dp_status_response" | grep -o '"message"[[:space:]]*:[[:space:]]*"[^"]*"' | sed 's/.*"message"[[:space:]]*:[[:space:]]*"\([^"]*\)".*/\1/' | head -1)
error_message="DataPusher error: ${error_info:-unknown}"
break
fi
fi
fi
[ $((attempt % 15)) -eq 0 ] && echo " Still processing... ($attempt/$max_attempts)"
done
if final_resource=$(curl -s "http://localhost:5000/api/3/action/resource_show?id=$resource_id" 2>/dev/null); then
if echo "$final_resource" | grep -q '"datastore_active"[[:space:]]*:[[:space:]]*true'; then
datastore_active="true"
if datastore_data=$(curl -s "http://localhost:5000/api/3/action/datastore_search?resource_id=$resource_id&limit=1" 2>/dev/null); then
rows_imported=$(echo "$datastore_data" | grep -o '"total"[[:space:]]*:[[:space:]]*[0-9]*' | sed 's/.*"total"[[:space:]]*:[[:space:]]*\([0-9]*\).*/\1/')
[ -z "$rows_imported" ] && rows_imported="0"
fi
echo " DataStore active | rows: $rows_imported"
else
echo " DataStore NOT active"
fi
fi
else
error_message="No valid resource ID"
fi
else
error_message="Resource creation failed: $(echo "$resource_response" | head -1)"
fi
end_time=$(date +%s)
processing_time=$((end_time - start_time))
timestamp=$(date '+%Y-%m-%d %H:%M:%S')
echo "$timestamp,$file_name,$upload_status,$resource_id,$datapusher_status,$datastore_active,$rows_imported,$processing_time,\"$error_message\"" >> /tmp/test_results.csv
if [ "$upload_status" = "SUCCESS" ] && [ "$datapusher_status" = "complete" ] && [ "$datastore_active" = "true" ]; then
passed_files=$((passed_files + 1))
echo " PASS: $file_name (${processing_time}s)"
else
failed_files=$((failed_files + 1))
echo " FAIL: $file_name | status=$datapusher_status | datastore=$datastore_active | error=$error_message"
fi
sleep 3
done < /tmp/test_files.txt
if [ -f /tmp/skipped_files.csv ]; then
skipped_files=$(tail -n +2 /tmp/skipped_files.csv | wc -l)
fi
echo ""
echo "=========================================="
echo "=== CI TEST RESULTS ==="
echo " Branch: ${{ github.ref_name }}"
echo " SHA: ${{ github.sha }}"
echo " Total: $((total_files + skipped_files))"
echo " Tested: $total_files"
echo " Skipped: $skipped_files"
echo " Passed: $passed_files"
echo " Failed: $failed_files"
[ $total_files -gt 0 ] && echo " Rate: $(( passed_files * 100 / total_files ))%"
echo "=========================================="
cat /tmp/test_results.csv
echo ""
# Save counts for gate step — do not exit 1 here so all steps always run
echo "CI_FAILED_FILES=$failed_files" >> $GITHUB_ENV
echo "CI_TOTAL_FILES=$total_files" >> $GITHUB_ENV
echo "CI_PASSED_FILES=$passed_files" >> $GITHUB_ENV
- name: Generate combined test results and worker analysis
if: always()
run: |
set -eu
echo "=== Processing DataPusher Plus Worker Logs ==="
if [ ! -f /tmp/ckan_worker.log ]; then
echo "No worker log file found"
echo "timestamp,job_id,file_name,status,qsv_version,file_format,encoding,normalized,valid_csv,sorted,db_safe_headers,analysis,records,total_time,download_time,analysis_time,copying_time,indexing_time,formulae_time,metadata_time,rows_copied,columns_indexed,error_type,error_message,data_quality_score,processing_efficiency" > /tmp/worker_analysis.csv
else
echo "Worker log size: $(du -h /tmp/ckan_worker.log | cut -f1)"
python3 /tmp/log_analyzer.py analyze /tmp/ckan_worker.log /tmp/worker_analysis.csv
fi
if [ ! -f /tmp/test_results.csv ] && [ ! -f /tmp/worker_analysis.csv ]; then
echo "# DataPusher+ CI Results" >> $GITHUB_STEP_SUMMARY
echo "No test data available." >> $GITHUB_STEP_SUMMARY
exit 0
fi
total_tests=0; skipped_count=0; total_files_in_dir=0
passed=0; failed=0; error_count=0
tested_success_rate=0; overall_success_rate=0
if [ -f /tmp/test_results.csv ]; then
total_lines=$(wc -l < /tmp/test_results.csv)
total_tests=$((total_lines - 1))
if [ -f /tmp/skipped_files.csv ]; then
skipped_lines=$(wc -l < /tmp/skipped_files.csv)
skipped_count=$((skipped_lines - 1))
fi
total_files_in_dir=$((total_tests + skipped_count))
if [ $total_tests -gt 0 ]; then
# `grep -c` always prints the count to stdout, then exits 1 when
# there are zero matches. `|| true` swallows that exit status
# while keeping grep's "0"; `|| echo "0"` would instead append a
# SECOND line, yielding a two-line value that breaks $(( )) below.
passed=$(grep -c ",SUCCESS,.*,complete,true," /tmp/test_results.csv 2>/dev/null || true)
failed=$(tail -n +2 /tmp/test_results.csv | grep -v ",SUCCESS,.*,complete,true," | wc -l)
error_count=$(grep -c ",error," /tmp/test_results.csv 2>/dev/null || true)
error_count=$(echo "$error_count" | tr -d '\n')
tested_success_rate=$(( passed * 100 / total_tests ))
[ $total_files_in_dir -gt 0 ] && overall_success_rate=$(( passed * 100 / total_files_in_dir ))
fi
fi
worker_analysis_available=false
[ -f /tmp/worker_analysis.csv ] && worker_analysis_available=true
{
echo "# DataPusher+ CI Results"
echo ""
echo "**Branch:** \`${{ github.ref_name }}\` | **SHA:** \`${{ github.sha }}\`"
echo ""
echo "## Summary"
echo ""
echo "| Metric | Value |"
echo "|--------|-------|"
echo "| Total Files in Directory | $total_files_in_dir |"
echo "| Files Tested | $total_tests |"
echo "| Files Skipped | $skipped_count |"
echo "| Passed | $passed |"
echo "| Failed | $failed |"
echo "| Errors | $error_count |"
echo "| Success Rate (Tested Files) | ${tested_success_rate}% |"
echo "| Success Rate (All Files) | ${overall_success_rate}% |"
echo ""
if [ $skipped_count -gt 0 ] && [ -f /tmp/skipped_files.csv ]; then
echo "## Skipped Files"
echo ""
echo "| File Name | Reason Skipped |"
echo "|-----------|----------------|"
tail -n +2 /tmp/skipped_files.csv | while IFS=',' read -r file_name reason; do
echo "| $file_name | $reason |"
done
echo ""
fi
if [ -f /tmp/test_results.csv ] && [ $total_tests -gt 0 ]; then
echo "## Test Run Results"
echo ""
echo "| # | File | Upload | DPP Status | DataStore | Rows | Time (s) | Error |"
echo "|---|------|--------|------------|-----------|------|----------|-------|"
counter=1
tail -n +2 /tmp/test_results.csv | while IFS=',' read -r ts file_name upload_status resource_id dp_status ds_active rows proc_time error_msg; do
clean_error=$(echo "$error_msg" | sed 's/^"//;s/"$//')
[ -z "$clean_error" ] && clean_error="-"
[ -z "$rows" ] && rows="0"
echo "| $counter | $file_name | $upload_status | $dp_status | $ds_active | $rows | $proc_time | $clean_error |"
counter=$((counter + 1))
done
echo ""
fi
if [ "$worker_analysis_available" = true ]; then
total_jobs=$(tail -n +2 /tmp/worker_analysis.csv | wc -l)
if [ $total_jobs -gt 0 ]; then
echo "## Complete Job Analysis"
echo ""
echo "| # | File Name | Status | Records | Columns | Time (s) | Valid CSV | Headers Safe | Error Type | Quality Score |"
echo "|---|-----------|--------|---------|---------|----------|-----------|--------------|------------|---------------|"
counter=1
tail -n +2 /tmp/worker_analysis.csv | while IFS=',' read timestamp job_id file_name status qsv_version file_format encoding normalized valid_csv sorted db_safe_headers analysis records total_time download_time analysis_time copying_time indexing_time formulae_time metadata_time rows_copied columns_indexed error_type error_message data_quality_score processing_efficiency; do
[ -z "$records" ] && records="0"
[ -z "$columns_indexed" ] && columns_indexed="0"
[ -z "$total_time" ] && total_time="0"
[ -z "$data_quality_score" ] && data_quality_score="-"
[ -z "$error_type" ] && error_type="-"
case "$status" in
"SUCCESS") status_display="SUCCESS" ;;
"ERROR") status_display="ERROR" ;;
"INCOMPLETE") status_display="INCOMPLETE" ;;
*) status_display="$status" ;;
esac
echo "| $counter | $file_name | $status_display | $records | $columns_indexed | $total_time | $valid_csv | $db_safe_headers | $error_type | $data_quality_score |"
counter=$((counter + 1))
done
echo ""
success_jobs=$(grep -c ",SUCCESS," /tmp/worker_analysis.csv || true)
error_jobs=$(grep -c ",ERROR," /tmp/worker_analysis.csv || true)
echo "## File Analysis"
echo ""
if [ $success_jobs -gt 0 ]; then
echo "### File Formats Processed"
echo ""
formats=$(tail -n +2 /tmp/worker_analysis.csv | grep ",SUCCESS," | cut -d',' -f6 | sort | uniq -c)
if [ -n "$formats" ]; then
echo "| Format | Files | Percentage |"
echo "|--------|-------|------------|"
echo "$formats" | while read count format; do
percentage=$((count * 100 / success_jobs))
echo "| $format | $count | $percentage% |"
done
else
echo "No format data available"
fi
echo ""
echo "### Encoding Distribution"
echo ""
encodings=$(tail -n +2 /tmp/worker_analysis.csv | grep ",SUCCESS," | cut -d',' -f7 | sort | uniq -c)
if [ -n "$encodings" ]; then
echo "| Encoding | Files | Status |"
echo "|----------|-------|--------|"
echo "$encodings" | while read count encoding; do
if [ -n "$encoding" ]; then
echo "| $encoding | $count | Compatible |"
else
echo "| Unknown | $count | Needs Review |"
fi
done
else
echo "No encoding data available"
fi
echo ""
fi
echo "## Error Analysis"
echo ""
if [ $error_jobs -gt 0 ]; then
echo "### Failed Files Details"
echo ""
echo "| File | Error Type | Error Message |"
echo "|------|------------|---------------|"
tail -n +2 /tmp/worker_analysis.csv | grep ",ERROR," | cut -d',' -f3,23,24 | while IFS=',' read file error_type error_msg; do
clean_error=$(echo "$error_msg" | sed 's/^"//;s/"$//')
clean_file=$(echo "$file" | sed 's/\.\.\.//')
echo "| $clean_file | $error_type | $clean_error |"
done
echo ""
else
echo "No errors found in worker logs - all processed jobs completed successfully."
echo ""
fi
echo "## Performance Anomalies"
echo ""
anomalies_output=$(python3 /tmp/log_analyzer.py anomalies /tmp/worker_analysis.csv 2>/dev/null || echo "")
if [ -z "$anomalies_output" ]; then
echo "No performance anomalies detected."
else
echo "Performance issues detected:"
echo ""
echo "$anomalies_output"
fi
echo ""
fi
fi
if [ $total_tests -eq 0 ] && [ $skipped_count -gt 0 ]; then
echo "## No Testable Files"
echo "All files in the test directory were skipped."
elif [ $total_tests -eq 0 ]; then
echo "## No Files Found"
echo "No files found in test directory."
elif [ $passed -eq $total_tests ]; then
echo "## All Tested Files Passed"
echo "DataPusher Plus is working correctly with all testable files."
[ $skipped_count -gt 0 ] && echo "" && echo "**Note:** $skipped_count file(s) skipped."
elif [ $passed -gt 0 ]; then
echo "## Partial Success"
echo "DataPusher Plus works with some files but has issues with others."
else
echo "## All Tested Files Failed"
echo "DataPusher Plus is not working correctly with any tested files."
if [ -f /tmp/test_results.csv ]; then
echo ""
tail -n +2 /tmp/test_results.csv | while IFS=',' read -r timestamp file_name upload_status resource_id datapusher_status datastore_active rows_imported processing_time error_message; do
clean_error=$(echo "$error_message" | sed 's/^"//;s/"$//')
echo "- **$file_name**: $clean_error"
done
fi
fi
echo ""
echo "---"
echo "**Completed:** $(date '+%A, %B %d, %Y at %I:%M %p %Z')"
} >> $GITHUB_STEP_SUMMARY
echo "Summary written to workflow step summary"
- name: Upload CI artifacts
uses: actions/upload-artifact@v4
if: always()
with:
# github.ref_name is "<PR>/merge" on pull_request events (rejected by upload-artifact@v4
# because of the "/"); use the PR number directly when available, fall back to ref_name otherwise.
name: dp-ci-${{ github.event.pull_request.number && format('pr{0}', github.event.pull_request.number) || github.ref_name }}-${{ github.run_id }}
path: |
/tmp/test_results.csv
/tmp/ckan_stdout.log
/tmp/ckan_worker.log
/tmp/worker_analysis.csv
/tmp/prefect_server.log
/tmp/prefect_worker.log
retention-days: 7
- name: Cleanup
if: always()
run: |
kill $CKAN_PID 2>/dev/null || true
kill $CKAN_WORKER_PID 2>/dev/null || true
echo "Stopping Prefect server and worker..."
# The start steps launch the servers with a plain `nohup ... &`, so the
# exported PIDs are ordinary PIDs (not process-group leaders) -- `kill`
# signals only the parent. The `pkill -f` lines sweep the
# uvicorn/scheduler children.
for pid in "${PREFECT_WORKER_PID:-}" "${PREFECT_SERVER_PID:-}"; do
[ -n "$pid" ] && kill "$pid" 2>/dev/null || true
done
pkill -f "prefect server start" || true
pkill -f "prefect worker start" || true
echo "Cleanup completed"
- name: Check test results
if: always()
run: |
failed=${CI_FAILED_FILES:-0}
total=${CI_TOTAL_FILES:-0}
passed=${CI_PASSED_FILES:-0}
if [ "$total" -eq 0 ]; then
echo "No files were tested. Check FILES_DIR ($FILES_DIR) and GitHub API availability."
exit 1
elif [ "$failed" -gt 0 ]; then
echo "$failed of $total file(s) failed DataPusher+ processing. $passed passed."
echo "See step summary and artifacts for details."
exit 1
else
echo "All $passed file(s) passed."
fi