Skip to content

Postgres Pipeline#408

Merged
eric-pSAP merged 17 commits intomainfrom
postgresPipeline
Mar 24, 2026
Merged

Postgres Pipeline#408
eric-pSAP merged 17 commits intomainfrom
postgresPipeline

Conversation

@eric-pSAP
Copy link
Contributor

Adding Postgres to the Pipeline

@eric-pSAP eric-pSAP requested a review from a team as a code owner March 23, 2026 11:48
@hyperspace-insights
Copy link
Contributor

Summary

The following content is AI-generated and provides a summary of the pull request:


Add Postgres Integration Tests to CI Pipeline

New Feature

✨ Introduces a dedicated CI pipeline job for running integration tests against a PostgreSQL database, alongside the existing HANA-based integration tests.

Changes

  • .github/workflows/test.yml:
    • Renamed the existing Integration tests step to Integration tests (HANA) for clarity.
    • Added a new integration-tests-postgres job that:
      • Runs on ubuntu-latest for Node.js versions 20.x and 22.x.
      • Spins up a postgres:14.5 service container with health checks configured.
      • Installs dependencies, deploys the schema via cds deploy --to postgres, and executes the test suite.
      • Uses environment variables (PG_USER, PG_PASSWORD, PG_HOST, PG_PORT) to connect to the local Postgres service.
      • Follows the same approval gating (requires-approval) as the existing integration test jobs.

  • 🔄 Regenerate and Update Summary
  • ✏️ Insert as PR Description (deletes this comment)
  • 🗑️ Delete comment
PR Bot Information

Version: 1.19.1 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback

  • Summary Prompt: Default Prompt
  • Correlation ID: 3b9c4ca0-26ae-11f1-8e24-7bade27c381e
  • LLM: anthropic--claude-4.6-sonnet
  • Event Trigger: pull_request.opened
  • Output Template: Default Template

💌 Have ideas or want to contribute? Create an issue and share your thoughts with us!
📑 Check out the documentation for more information.
📬 Subscribe to the Hyperspace PR Bot DL to get the latest announcements and pilot features!

Made with ❤️ by Hyperspace.

Copy link
Contributor

@hyperspace-insights hyperspace-insights bot left a comment

Choose a reason for hiding this comment

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

The new integration-tests-postgres job has several issues that need to be addressed before it can work correctly: a hardcoded plain-text password, a stale actions/setup-node version, cds deploy being run from the wrong directory (likely causing an empty/incorrect schema), and npm run test running the full test suite instead of scoping to Postgres-relevant integration tests only.

PR Bot Information

Version: 1.19.1 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback

  • Correlation ID: 3b9c4ca0-26ae-11f1-8e24-7bade27c381e
  • Event Trigger: pull_request.opened
  • LLM: anthropic--claude-4.6-sonnet

@KoblerS KoblerS enabled auto-merge (squash) March 24, 2026 11:36
@eric-pSAP eric-pSAP disabled auto-merge March 24, 2026 11:46
@eric-pSAP eric-pSAP enabled auto-merge (squash) March 24, 2026 15:26
@eric-pSAP eric-pSAP merged commit 7ef34a3 into main Mar 24, 2026
18 of 19 checks passed
@eric-pSAP eric-pSAP deleted the postgresPipeline branch March 24, 2026 15:45
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