◐ Shell
reader mode source ↗
Skip to content

fix: Fix registry Rest API tests intermittent failure#6111

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

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

Conversation

@Srihari1192

@Srihari1192 Srihari1192 commented Mar 16, 2026 •
edited by devin-ai-integration Bot
Loading

Copy link
Copy Markdown
Contributor

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

Srihari1192 commented Mar 16, 2026

Copy link
Copy Markdown
Contributor Author

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

devin-ai-integration[bot]
devin-ai-integration Bot reviewed Mar 16, 2026
View reviewed changes

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hide comment

✅ 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 Mar 16, 2026
@Srihari1192
fix: Fix regstry Rest API tests intermittent failure …
a28b151 
Signed-off-by: Srihari <svenkata@redhat.com>
@Srihari1192 Srihari1192 force-pushed the fixRegistryRESTAPITests branch from d78e931 to a28b151 Compare March 16, 2026 07:18
Hide details View details @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

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments

@ntkathole ntkathole ntkathole approved these changes

Assignees

No one assigned

Labels

ok-to-test

Projects

None yet

Milestone

No milestone

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

@Srihari1192 @ntkathole