[DNM] ci: test buildkit features compat#45160
Conversation
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com> (cherry picked from commit 9635627)
6115268 to
bfb2c88
Compare
disable azblob and s3 cache backend tests Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
bfb2c88 to
dff80ca
Compare
|
I think we had to test this one in combination with #45153 |
|
@thaJeztah This PR was just to test for the s3 case: moby/buildkit#3713 (comment) but I can open another one with #45153 |
|
@crazy-max yeah, so on that PR we saw a failure, so I thought (assumed) we wanted to test the new changes in BuildKit to work with that PR (to exclude the things that need to be excluded with the new env-vars)? Looks like this PR is green, so I guess we can re-use this PR to include the other PR to test if it also works with those changes. |
|
🤦 OH! I see you already opened another PR; #45165 ? |
Yes that's the one for merge diff, and looks good: https://github.com/moby/moby/actions/runs/4425205314 |
|
It looks like we can close this as #44908 was merged? |
follow-up moby/buildkit#3713 (comment)
disable azblob and s3 cache backend tests