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

Drop netcoreapp3.1 from CI & update testing framework in Instrumentation.GrpcCore to net6.0 #1287

Merged
merged 8 commits into from
Aug 1, 2023

Conversation

Kielek
Copy link
Contributor

@Kielek Kielek commented Aug 1, 2023

Towards #1031

Changes

  • update testing framework in Instrumentation.GrpcCore from netcoreapp3.1 to net6.0
  • Bum Grpc package in test to latest released. It is currently obsolete, unmaintained package so update to the latest release seems to be reasonable
    • mitigation of Unable to load shared library 'libdl.so' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: liblibdl.so: cannot open shared object file: No such file or directory
  • Drop netcoreapp3.1 from CI
  • Minor opportunistic changes

As part of working on this redis-test (netcoreapp3.1) was removed from required builds

While reviewing consider to review each commit separately. History is more descriptive.

For significant contributions please make sure you have completed the following items:

  • [ ] Appropriate CHANGELOG.md updated for non-trivial changes
  • [ ] Design discussion issue #
  • [ ] Changes in public API reviewed

@Kielek Kielek added infra Infra work - CI/CD, code coverage, linters comp:instrumentation.grpccore Things related to OpenTelemetry.Instrumentation.GrpcCore labels Aug 1, 2023
@Kielek Kielek requested a review from a team August 1, 2023 07:28
to mitigate Unable to load shared library 'libdl.so' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: liblibdl.so: cannot open shared object file: No such file or directory
@Kielek
Copy link
Contributor Author

Kielek commented Aug 1, 2023

@pcwiese, @vishweshbankwar please review.

@pcwiese, could you please create request based on #1201 ? it will help us maintaining reviewers automatically.

@pcwiese
Copy link
Contributor

pcwiese commented Aug 1, 2023

@pcwiese, @vishweshbankwar please review.

@pcwiese, could you please create request based on #1201 ? it will help us maintaining reviewers automatically.

Here is my request for membership
open-telemetry/community#1622

@Kielek Kielek merged commit 672db90 into open-telemetry:main Aug 1, 2023
@Kielek Kielek deleted the grpc-core-switch-tests-to-net6 branch August 1, 2023 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:instrumentation.grpccore Things related to OpenTelemetry.Instrumentation.GrpcCore infra Infra work - CI/CD, code coverage, linters
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants