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

azblob: fails to build on Dragonfly #3266

Closed
bep opened this issue Jun 23, 2023 · 14 comments · Fixed by #3268
Closed

azblob: fails to build on Dragonfly #3266

bep opened this issue Jun 23, 2023 · 14 comments · Fixed by #3268
Assignees

Comments

@bep
Copy link

bep commented Jun 23, 2023

Even in v0.30.0

See #3261

@vangent
Copy link
Contributor

vangent commented Jun 23, 2023

What's the error? Is it still in github.com/Azure/azure-sdk-for-go/sdk/storage/azblob/blockblob ?

If so, please file a bug for them. All I can do is update the dependency.

In #3261 you said "This is fixed upstream" so I assumed that updating all dependencies would fix it. If it hasn't, can you be more specific about what "upstream" is?

@bep
Copy link
Author

bep commented Jun 23, 2023

So, the errors are the same

# github.com/Azure/azure-sdk-for-go/sdk/storage/azblob/blockblob
../gomodcache/github.com/!azure/azure-sdk-for-go/sdk/storage/[email protected]/blockblob/chunkwriting.go:273:15: undefined: mmb
../gomodcache/github.com/!azure/azure-sdk-for-go/sdk/storage/[email protected]/blockblob/chunkwriting.go:279:65: undefined: mmb
../gomodcache/github.com/!azure/azure-sdk-for-go/sdk/storage/[email protected]/blockblob/chunkwriting.go:281:22: undefined: mmb
../gomodcache/github.com/!azure/azure-sdk-for-go/sdk/storage/[email protected]/blockblob/chunkwriting.go:287:39: undefined: mmb
../gomodcache/github.com/!azure/azure-sdk-for-go/sdk/storage/[email protected]/blockblob/chunkwriting.go:293:18: undefined: newMMB
../gomodcache/github.com/!azure/azure-sdk-for-go/sdk/storage/[email protected]/blockblob/chunkwriting.go:303:37: undefined: mmb
• core: Total in 127.46s …
error running command: exit status 1

Exited with code exit status 1

When I said "fixed upstream" I meant that the issue I opened in the Azure repo above was closed as fixed with a commit, but I guess that fix hasn't made it into a release yet (I haven't checked).

So, the more general problem is that we (that includes me) don't test these build tags as part of the daily CI builds, so we end up getting these errors very late when we do a binary release. I will try to fix this on my end so I don't have to "revert go-cloud" so frequently.

@bep
Copy link
Author

bep commented Jun 23, 2023

Upstream issue: Azure/azure-sdk-for-go#20885

@vangent
Copy link
Contributor

vangent commented Jun 23, 2023

SGTM. Can you link to the Azure issue you're referring to?

I don't know how to test on Dragonfly; https://docs.github.com/en/actions/using-jobs/choosing-the-runner-for-a-job#choosing-github-hosted-runners doesn't include it.

@bep
Copy link
Author

bep commented Jun 24, 2023

@vangent the link is in my comment above (Azure/azure-sdk-for-go#20885).

As to testing, I'm more referring to the smoke tests in the form of GOOS=foo GOARCH=bar go build for all variants that you're supposed to support. That Hugo supports Dragonfly comes with some history, but I have kept it because it ... has a cool name. And it's just a BSD Unix variant.

@vangent
Copy link
Contributor

vangent commented Jun 24, 2023

Can you point me to the Hugo smoke tests that manage that?

I tried this locally and got an error but a different one than you quoted:

$ cd blob/azureblob
$ GOOS=dragonfly go build
# runtime
/usr/lib/google-golang/src/runtime/googleexit.go:81:6: exit defined in both Go and assembly

@bep
Copy link
Author

bep commented Jun 24, 2023

Can you point me to the Hugo smoke tests that manage that?

There's none. I will try to add some, but currently this fail on release. I have had to revert the go-cloud upgrade on the last 2 Hugo releases.

@vangent
Copy link
Contributor

vangent commented Jun 24, 2023

OK, how about how you reproduced the error above? I can't do it locally.

@bep
Copy link
Author

bep commented Jun 25, 2023

@vangent I'm out travelling, but will add one such test to the Hugo CLI tests when I get back home. And, looking at this issue, I should have done that before opening this issue, but I'm learning.

@bep
Copy link
Author

bep commented Jul 1, 2023

I have added GOARCH=amd64 GOOS=dragonfly go install to Hugo's CI build.

Which fails on "bump gocloud.dev from 0.24.0 to 0.30.0" gohugoio/hugo#11172

https://github.com/gohugoio/hugo/actions/runs/5432106017/jobs/9878929559?pr=11172

@vangent
Copy link
Contributor

vangent commented Jul 5, 2023

I tried to do the same, but it seemed to pass:

#3268

Any idea what I've done wrong?

@vangent
Copy link
Contributor

vangent commented Jul 5, 2023

Ah, I got it to fail, I needed go install ./....

It doesn't look like the Azure fix has been pushed yet.

@vangent vangent self-assigned this Jul 5, 2023
@vangent vangent changed the title Stil fails to build on Dragonfly azblob: fails to build on Dragonfly Jul 5, 2023
@bep
Copy link
Author

bep commented Jul 5, 2023

It doesn't look like the Azure fix has been pushed yet.

I looked, it's not in a released tag.

I'm closing this issue; Dragonfly is rather exotic, and I have a test for it and I can just wait for a version that goes green.

@vangent
Copy link
Contributor

vangent commented Jul 19, 2023

The next build should work.

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 a pull request may close this issue.

2 participants