Skip to content

fix: e2e not able to connect to db#5750

Merged
khvn26 merged 2 commits intomainfrom
fix/e2e-name-resolution-failure
Jul 14, 2025
Merged

fix: e2e not able to connect to db#5750
khvn26 merged 2 commits intomainfrom
fix/e2e-name-resolution-failure

Conversation

@tiagoapolo
Copy link
Copy Markdown
Contributor

@tiagoapolo tiagoapolo commented Jul 14, 2025

Thanks for submitting a PR! Please check the boxes below:

  • I have added information to docs/ if required so people know about the feature!
  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have used a Conventional Commit title for this Pull Request

Changes

Screenshot 2025-07-14 at 12 00 49

We were using postgresql:15 and not specifying a minor, a breaking change was introduced in 15.13-13
cloudnative-pg/postgres-containers#248

How did you test this code?

  1. Go to frontend folder
  2. Run docker-compose -f docker-compose-e2e-tests.yml up
  3. Check if migrations are being applied, meaning the app was able to connect to DB

A breaking change was introduced in 15.13-13
cloudnative-pg/postgres-containers#248
@tiagoapolo tiagoapolo requested a review from a team as a code owner July 14, 2025 19:42
@tiagoapolo tiagoapolo requested review from Zaimwa9 and removed request for a team July 14, 2025 19:42
@vercel
Copy link
Copy Markdown

vercel Bot commented Jul 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
flagsmith-frontend-preview ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2025 8:53pm
flagsmith-frontend-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2025 8:53pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Visit Preview Jul 14, 2025 8:53pm

@github-actions github-actions Bot added the front-end Issue related to the React Front End Dashboard label Jul 14, 2025
@tiagoapolo tiagoapolo self-assigned this Jul 14, 2025
@github-actions github-actions Bot added the fix label Jul 14, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 14, 2025

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-api-test:pr-5750 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-5750 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-api:pr-5750 Finished ✅ Results
ghcr.io/flagsmith/flagsmith:pr-5750 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-private-cloud:pr-5750 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-frontend:pr-5750 Finished ✅ Results

@tiagoapolo tiagoapolo requested review from khvn26 and removed request for Zaimwa9 July 14, 2025 19:43
@tiagoapolo tiagoapolo changed the title fix: e2e name resolution failure fix: e2e not able to connect to db Jul 14, 2025
@github-actions github-actions Bot added fix and removed fix labels Jul 14, 2025
@tiagoapolo tiagoapolo force-pushed the fix/e2e-name-resolution-failure branch from a440111 to 11c738e Compare July 14, 2025 20:35
@github-actions github-actions Bot added fix and removed fix labels Jul 14, 2025
Comment thread frontend/docker-compose-e2e-tests.yml Outdated
Co-authored-by: Evandro Myller <22429+emyller@users.noreply.github.com>
@github-actions github-actions Bot added fix and removed fix labels Jul 14, 2025
Copy link
Copy Markdown
Contributor

@emyller emyller left a comment

Choose a reason for hiding this comment

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

docker.io seems like the right option for now.

@khvn26 khvn26 merged commit d30de50 into main Jul 14, 2025
25 checks passed
@khvn26 khvn26 deleted the fix/e2e-name-resolution-failure branch July 14, 2025 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix front-end Issue related to the React Front End Dashboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants