Skip to content

Conversation

BarryWu0812
Copy link
Contributor

@BarryWu0812 BarryWu0812 commented Aug 29, 2025

Tracking issue

Why are the changes needed?

Recently, the integration test often fail at test_attr_access_sd with user runtime error.
Thus, just extending the waiting time to test whether the integration test pass easily.

What changes were proposed in this pull request?

Extend the waiting time.

How was this patch tested?

Setup process

Screenshots

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

Docs link

Summary by Bito

This pull request extends the waiting time in the integration test for the `test_attr_access_sd` function to reduce frequent test failures caused by user runtime errors. The adjustment aims to enhance the reliability of the integration tests.

@BarryWu0812 BarryWu0812 marked this pull request as draft August 29, 2025 17:38
@BarryWu0812 BarryWu0812 marked this pull request as ready for review August 29, 2025 17:38
Signed-off-by: Barry Wu <[email protected]>
@flyte-bot
Copy link
Contributor

Bito Automatic Review Skipped - Files Excluded

Bito didn't auto-review this change because all changed files are in the exclusion list for automatic reviews. No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the excluded files settings here, or contact your Bito workspace admin at [email protected].

Signed-off-by: Barry Wu <[email protected]>
@flyte-bot
Copy link
Contributor

Bito Automatic Review Skipped - Files Excluded

Bito didn't auto-review this change because all changed files are in the exclusion list for automatic reviews. No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the excluded files settings here, or contact your Bito workspace admin at [email protected].

@pingsutw pingsutw merged commit 4d21550 into flyteorg:master Sep 27, 2025
228 checks passed
Atharva1723 pushed a commit to Atharva1723/flytekit that referenced this pull request Oct 5, 2025
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.

3 participants