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

[chore] Bump github.com/aerospike/aerospike-client-go from 6.13.0 to 7.2.0 #32108

Closed
wants to merge 2 commits into from
Closed

[chore] Bump github.com/aerospike/aerospike-client-go from 6.13.0 to 7.2.0 #32108

wants to merge 2 commits into from

Conversation

TylerHelmuth
Copy link
Member

@TylerHelmuth TylerHelmuth commented Apr 2, 2024 •
edited
Loading

Closes #26265

@TylerHelmuth TylerHelmuth requested a review from djaglowski as a code owner April 2, 2024 14:32
@TylerHelmuth TylerHelmuth requested a review from a team April 2, 2024 14:32
@github-actions github-actions bot added cmd/configschema configschema command cmd/otelcontribcol otelcontribcol command receiver/aerospike labels Apr 2, 2024
@github-actions github-actions bot requested a review from antonblock April 2, 2024 14:32
@songy23 songy23 added the dependencies Pull requests that update a dependency file label Apr 2, 2024
@TylerHelmuth
Copy link
Member Author

Still getting the proto clash: https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/8524693103/job/23351136703?pr=32108

@TylerHelmuth TylerHelmuth mentioned this pull request Apr 2, 2024
Closed
@TylerHelmuth
Copy link
Member Author

@songy23 @mx-psi I am able to run make gotest GROUP=cmd-0 locally no problem, if you checkout this branch does it work for you? I'm not sure why it is failing in GitHub actions.

@songy23
Copy link
Member

songy23 commented Apr 2, 2024

I saw some error, don't know if it's related

% make gotest GROUP=cmd-0
...
# github.com/open-telemetry/opentelemetry-collector-contrib/cmd/configschema/cfgmetadatagen/cfgmetadatagen.test
ld: warning: '/private/var/folders/zs/578nl21x5wj5tjws_80nd81r0000gp/T/go-link-780598431/000020.o' has malformed LC_DYSYMTAB, expected 98 undefined symbols to start at index 1626, found 95 undefined symbols starting at index 1626
# github.com/open-telemetry/opentelemetry-collector-contrib/cmd/configschema.test
ld: warning: '/private/var/folders/zs/578nl21x5wj5tjws_80nd81r0000gp/T/go-link-809219092/000089.o' has malformed LC_DYSYMTAB, expected 98 undefined symbols to start at index 1626, found 95 undefined symbols starting at index 1626
∅  cfgmetadatagen
∅  docsgen
✓  cfgmetadatagen/cfgmetadatagen (cached)
✓  . (cached)
# github.com/open-telemetry/opentelemetry-collector-contrib/cmd/configschema/docsgen/docsgen.test
ld: warning: '/private/var/folders/zs/578nl21x5wj5tjws_80nd81r0000gp/T/go-link-2423132131/000089.o' has malformed LC_DYSYMTAB, expected 98 undefined symbols to start at index 1626, found 95 undefined symbols starting at index 1626
✓  docsgen/docsgen (cached)

=== Skipped
=== SKIP: docsgen/docsgen TestHandleCLI_All (0.00s)
    cli_test.go:70: this test takes > 5m when -race is used

=== Errors
ld: warning: '/private/var/folders/zs/578nl21x5wj5tjws_80nd81r0000gp/T/go-link-780598431/000020.o' has malformed LC_DYSYMTAB, expected 98 undefined symbols to start at index 1626, found 95 undefined symbols starting at index 1626
ld: warning: '/private/var/folders/zs/578nl21x5wj5tjws_80nd81r0000gp/T/go-link-809219092/000089.o' has malformed LC_DYSYMTAB, expected 98 undefined symbols to start at index 1626, found 95 undefined symbols starting at index 1626
ld: warning: '/private/var/folders/zs/578nl21x5wj5tjws_80nd81r0000gp/T/go-link-2423132131/000089.o' has malformed LC_DYSYMTAB, expected 98 undefined symbols to start at index 1626, found 95 undefined symbols starting at index 1626

DONE 27 tests, 1 skipped, 3 errors in 2.563s

@TylerHelmuth
Copy link
Member Author

I think unreleated since you got past docsgen stuff

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@songy23 songy23 songy23 approved these changes

@mx-psi mx-psi mx-psi approved these changes

@djaglowski djaglowski Awaiting requested review from djaglowski

@dmitryax dmitryax Awaiting requested review from dmitryax

@antonblock antonblock Awaiting requested review from antonblock

Assignees

@mx-psi mx-psi

Labels
cmd/configschema configschema command cmd/otelcontribcol otelcontribcol command dependencies Pull requests that update a dependency file receiver/aerospike
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Bump github.com/aerospike/aerospike-client-go/v6 from 6.13.0 to 6.14.0 causes issues
3 participants
@TylerHelmuth @songy23 @mx-psi