Skip to content

fix: Fix registry Rest API tests intermittent failure#6111

Merged
ntkathole merged 1 commit intofeast-dev:masterfrom
Srihari1192:fixRegistryRESTAPITests
Mar 16, 2026
Merged

fix: Fix registry Rest API tests intermittent failure#6111
ntkathole merged 1 commit intofeast-dev:masterfrom
Srihari1192:fixRegistryRESTAPITests

Conversation

@Srihari1192
Copy link
Contributor

@Srihari1192 Srihari1192 commented Mar 16, 2026

What this PR does / why we need it:

Fixes #5977

  • Wrapped the apply_project() call in a try-except block to catch IntegrityError
  • Increased HTTP readiness timeout from 180s to 300s
  • Added a 30-second initial delay before polling the endpoint to give the backend more time to initialize after dataset creation
  • Prevents premature connection attempts that result in 502 errors

Open with Devin

@Srihari1192 Srihari1192 requested a review from a team as a code owner March 16, 2026 07:10
@Srihari1192
Copy link
Contributor Author

@ntkathole Please review it Tested with this changes here multiple times

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@ntkathole ntkathole changed the title Fix registry Rest API tests intermittent failure fix: Fix registry Rest API tests intermittent failure Mar 16, 2026
@Srihari1192 Srihari1192 force-pushed the fixRegistryRESTAPITests branch from d78e931 to a28b151 Compare March 16, 2026 07:18
@ntkathole ntkathole merged commit d53a339 into feast-dev:master Mar 16, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

registry-rest-api-tests CI workflow sometimes fails due to missing HTTP readiness check

2 participants