ci: Add external postgresql logs

[dsavineau]

Only the internal managed postgresql pod get logs gathered at the end of the CI run.
When using an external database (like one of the CI scenario) then those logs aren't gathered.

Summary by CodeRabbit

• Chores
  • Enhanced CI/CD pipeline logging capabilities to better support different database scenarios during automated testing.
  • Improved database log collection mechanisms to ensure comprehensive diagnostics across multiple test deployment configurations.
  • Streamlined log retrieval processes for various environment setups to improve debugging efficiency.

[coderabbitai]

📝 Walkthrough

Walkthrough

Both CI and PR workflow files are updated to conditionally fetch database logs based on the matrix.SCENARIO parameter. When the scenario is externaldb, logs are retrieved from a locally-run PostgreSQL Docker container; for all other scenarios, Kubernetes logs are used.

Changes

Cohort / File(s)	Summary
GitHub Workflows - Log Collection .github/workflows/ci.yml, .github/workflows/pr.yml	Updated "Get logs" step to conditionally fetch PostgreSQL logs. For externaldb scenario, switches to Docker environment and retrieves logs from local postgresql container via docker logs. For other scenarios, retains existing kubectl logs behavior with label selector.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'ci: Add external postgresql logs' clearly and concisely describes the main change—adding PostgreSQL log collection for external databases in CI workflows.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/ci.yml:
- Around line 126-128: The eval command in the "Get logs" step is currently not
tolerant to failure, so if eval $(minikube -p minikube docker-env) fails it
aborts log collection; change that invocation to be best-effort (e.g., make the
eval non-fatal by appending a failure-tolerant operator such as "|| true" or
otherwise ensuring the shell does not exit on failure) so the subsequent docker
logs postgresql || true still runs; apply the same change to the matching line
in .github/workflows/pr.yml where eval $(minikube -p minikube docker-env)
appears.

In @.github/workflows/pr.yml:
- Around line 120-122: The unguarded eval $(minikube -p minikube docker-env) can
fail and abort the always() log-collection step; modify the step to guard that
command (e.g., append "|| true" or otherwise tolerate failure) so the eval
cannot cause the job to exit before running docker logs postgresql --tail 1000;
locate the eval invocation and update it to be a no-fail invocation (eval
$(minikube -p minikube docker-env) || true) so docker logs postgresql still
runs.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 397ce906-b9e5-4813-bcd7-b1d677f6197e

📥 Commits

Reviewing files that changed from the base of the PR and between 3ab62d8 and 3415718.

📒 Files selected for processing (2)

• .github/workflows/ci.yml
• .github/workflows/pr.yml