Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(ci): fix invalid Behavior Test Integration Cloud Filter trigger #5414

Merged
merged 2 commits into from
Dec 15, 2024

Conversation

Zheaoli
Copy link
Member

@Zheaoli Zheaoli commented Dec 14, 2024

When I try to release the v0.51.0-rc.1, I found "Behavior Test Integration Cloud Filter" failed and it's not been triggered when the file changed.

https://github.com/apache/opendal/actions/runs/12332075431/job/34419665126

Fix the CI first

@Zheaoli
Copy link
Member Author

Zheaoli commented Dec 14, 2024

cc @ho-229 @Xuanwo. I think this would be a release blocker

@Zheaoli
Copy link
Member Author

Zheaoli commented Dec 15, 2024

@ho-229 the test still fail on windows, do you have any ideas?

@ho-229
Copy link
Contributor

ho-229 commented Dec 15, 2024

It seems like the behavior of Cloud Filter API has changed, I am trying to figure it out.

@ho-229
Copy link
Contributor

ho-229 commented Dec 15, 2024

Fix #5416

@Zheaoli
Copy link
Member Author

Zheaoli commented Dec 15, 2024

It seems like the behavior of Cloud Filter API has changed, I am trying to figure it out.

Thanks! I will release 0.51.0-rc.2 after your fix

@Zheaoli Zheaoli force-pushed the manjusaka/fix-cloud-filter-intergration-test branch from 1840748 to 7ef4cd8 Compare December 15, 2024 13:28
Signed-off-by: Manjusaka <[email protected]>
@Zheaoli Zheaoli force-pushed the manjusaka/fix-cloud-filter-intergration-test branch from 9159362 to 8b7422b Compare December 15, 2024 13:58
@Zheaoli Zheaoli merged commit 9b64d96 into main Dec 15, 2024
30 checks passed
@Zheaoli Zheaoli deleted the manjusaka/fix-cloud-filter-intergration-test branch December 15, 2024 14:01
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.

2 participants