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.

Sign up for GitHub

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

Jump to bottom

Update github.com/aws/aws-sdk-go-v2/service/s3 to v1.51.2 breaks build #31592

Closed
songy23 opened this issue Mar 5, 2024 · 3 comments
Closed

Update github.com/aws/aws-sdk-go-v2/service/s3 to v1.51.2 breaks build #31592

songy23 opened this issue Mar 5, 2024 · 3 comments
Labels
cmd/configschema configschema command dependency issue

Comments

@songy23
Copy link
Member

songy23 commented Mar 5, 2024

Component(s)

cmd/configschema

Describe the issue you're reporting

See https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/8149526970/job/22274417890?pr=31557

Error: /home/runner/go/pkg/mod/github.com/aws/aws-sdk-go-v2/service/s3@v1.31.0/api_client.go:766:25: undefined: presignedurlcust.AddAsIsPresigingMiddleware
@songy23 songy23 added needs triage New item requiring triage dependency issue labels Mar 5, 2024
@songy23 songy23 mentioned this issue Mar 5, 2024
Merged
1 task
@github-actions github-actions bot added the cmd/configschema configschema command label Mar 5, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Mar 5, 2024

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@crobert-1
Copy link
Member

Issue: aws/aws-sdk-go-v2#2529

This should be fixed by upgrading to v1.51.2, but it looks like the renovate PR is failing using v1.31.0. Looks like the indirect go.mod references are still on the old version.

@crobert-1 crobert-1 removed the needs triage New item requiring triage label Mar 5, 2024
@github-actions github-actions bot mentioned this issue Mar 12, 2024
Closed
@songy23 songy23 closed this as completed Mar 20, 2024
@songy23
Copy link
Member Author

songy23 commented Mar 20, 2024

#31557 bumps to a newer version, build succeeds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cmd/configschema configschema command dependency issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@songy23 @crobert-1