Skip to content

Add active column to seed CSV and update dev setup docs#76

Merged
brianclemens merged 1 commit intoresf:mainfrom
rockythorn:fix/seed-csv-active-column
Mar 2, 2026
Merged

Add active column to seed CSV and update dev setup docs#76
brianclemens merged 1 commit intoresf:mainfrom
rockythorn:fix/seed-csv-active-column

Conversation

@rockythorn
Copy link
Collaborator

Summary

  • The active column was added to supported_products_rh_mirrors in schema.sql but the seed CSV was not updated, causing \copy to fail with missing data for column "active" during local dev setup. The FK error on supported_products_rpm_repomds is a cascade from the mirrors copy failing first.
  • Added active to all 20 rows: true for current products, false for Legacy entries to match their intended inactive state.
  • Updated apollo/README.md to fix several issues in the dev setup guide:
    • Add Docker as an alternative to Podman for the PostgreSQL container step
    • Use PGPASSWORD env var instead of interactive password prompts (-W)
    • Start Temporal before Apollo server and workers (correct dependency order)
    • Move the last_indexed_at optimization hint to Tips and Tricks as an optional step

Testing

Tested by tearing down a local dev environment completely and following the README setup guide from scratch. All four \copy statements completed without error, supported_products_rh_mirrors seeded with 20 rows with correct active values (Legacy rows false, all others true), and all Apollo components came up successfully.

The schema added an active column to supported_products_rh_mirrors with
DEFAULT true, but the seed CSV was not updated, causing \copy to fail
with "missing data for column" during local dev setup. Legacy entries
seed as false to match their intended inactive state.

Updates to the Apollo README:
- Add Docker as an alternative to Podman for the PostgreSQL container
- Use PGPASSWORD env var instead of interactive password prompts
- Start Temporal before Apollo server and workers (correct order)
- Move last_indexed_at hint to Tips and Tricks as an optional step
Copy link

@thefossguy-ciq thefossguy-ciq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It works on my end with this updated seed, thanks for the work!

@brianclemens brianclemens merged commit d35fde3 into resf:main Mar 2, 2026
1 check passed
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.

4 participants