Skip to content

Fix pipeline-smoke-test artifact check to align with Titanic DB output#3

Closed
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-pipeline-smoke-test
Closed

Fix pipeline-smoke-test artifact check to align with Titanic DB output#3
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-pipeline-smoke-test

Conversation

Copilot AI commented Jun 6, 2026

Copy link
Copy Markdown

The pipeline-smoke-test job failed because the workflow still referenced the old database artifact path. The pipeline now emits data/titanic.db, but CI was asserting data/demo.db.

  • Root cause

    • CI smoke test contained a stale file assertion (data/demo.db) after the pipeline/database rename.
  • Workflow alignment

    • Updated .github/workflows/ci.yml to use Titanic artifact names consistently in cleanup and output verification steps.
    • Replaced legacy demo.db references with titanic.db in the smoke-test path checks.
  • Behavioral impact

    • CI now validates the actual outputs produced by pipelines/01_ingest_to_sqlite.py (data/titanic.db and reports/titanic_summary.csv) instead of a non-existent legacy artifact.
- name: Verify generated outputs
  run: |
    test -f data/titanic.db
    test -f reports/titanic_summary.csv

Copilot AI changed the title [WIP] Fix failing GitHub Actions job pipeline-smoke-test Fix pipeline-smoke-test artifact check to align with Titanic DB output Jun 6, 2026
Copilot AI requested a review from Bmowville June 6, 2026 05:33
@Bmowville

Copy link
Copy Markdown
Owner

Closing this stale draft because the Titanic database workflow alignment was superseded by later merged CI/reporting work, and this PR no longer has a file diff against main.

@Bmowville Bmowville closed this Jun 6, 2026
@Bmowville Bmowville deleted the copilot/fix-pipeline-smoke-test branch June 6, 2026 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants