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

Update dependencies #2374

Closed
wants to merge 31 commits into from
Closed

Update dependencies #2374

wants to merge 31 commits into from

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Nov 11, 2021

dependabot bot and others added 26 commits November 7, 2021 00:08
Bumps [github.com/benbjohnson/clock](https://github.com/benbjohnson/clock) from 1.1.0 to 1.2.0.
- [Release notes](https://github.com/benbjohnson/clock/releases)
- [Commits](benbjohnson/clock@v1.1.0...v1.2.0)

---
updated-dependencies:
- dependency-name: github.com/benbjohnson/clock
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [go.opentelemetry.io/proto/otlp](https://github.com/open-telemetry/opentelemetry-proto-go) from 0.9.0 to 0.10.0.
- [Release notes](https://github.com/open-telemetry/opentelemetry-proto-go/releases)
- [Commits](open-telemetry/opentelemetry-proto-go@v0.9.0...v0.10.0)

---
updated-dependencies:
- dependency-name: go.opentelemetry.io/proto/otlp
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [go.opentelemetry.io/proto/otlp](https://github.com/open-telemetry/opentelemetry-proto-go) from 0.9.0 to 0.10.0.
- [Release notes](https://github.com/open-telemetry/opentelemetry-proto-go/releases)
- [Commits](open-telemetry/opentelemetry-proto-go@v0.9.0...v0.10.0)

---
updated-dependencies:
- dependency-name: go.opentelemetry.io/proto/otlp
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.41.0 to 1.42.0.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.41.0...v1.42.0)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [go.opentelemetry.io/proto/otlp](https://github.com/open-telemetry/opentelemetry-proto-go) from 0.9.0 to 0.10.0.
- [Release notes](https://github.com/open-telemetry/opentelemetry-proto-go/releases)
- [Commits](open-telemetry/opentelemetry-proto-go@v0.9.0...v0.10.0)

---
updated-dependencies:
- dependency-name: go.opentelemetry.io/proto/otlp
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.41.0 to 1.42.0.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.41.0...v1.42.0)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [go.opentelemetry.io/proto/otlp](https://github.com/open-telemetry/opentelemetry-proto-go) from 0.9.0 to 0.10.0.
- [Release notes](https://github.com/open-telemetry/opentelemetry-proto-go/releases)
- [Commits](open-telemetry/opentelemetry-proto-go@v0.9.0...v0.10.0)

---
updated-dependencies:
- dependency-name: go.opentelemetry.io/proto/otlp
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.41.0 to 1.42.0.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.41.0...v1.42.0)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [go.opentelemetry.io/proto/otlp](https://github.com/open-telemetry/opentelemetry-proto-go) from 0.9.0 to 0.10.0.
- [Release notes](https://github.com/open-telemetry/opentelemetry-proto-go/releases)
- [Commits](open-telemetry/opentelemetry-proto-go@v0.9.0...v0.10.0)

---
updated-dependencies:
- dependency-name: go.opentelemetry.io/proto/otlp
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [go.opentelemetry.io/proto/otlp](https://github.com/open-telemetry/opentelemetry-proto-go) from 0.9.0 to 0.10.0.
- [Release notes](https://github.com/open-telemetry/opentelemetry-proto-go/releases)
- [Commits](open-telemetry/opentelemetry-proto-go@v0.9.0...v0.10.0)

---
updated-dependencies:
- dependency-name: go.opentelemetry.io/proto/otlp
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.41.0 to 1.42.0.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.41.0...v1.42.0)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [github.com/golangci/golangci-lint](https://github.com/golangci/golangci-lint) from 1.42.1 to 1.43.0.
- [Release notes](https://github.com/golangci/golangci-lint/releases)
- [Changelog](https://github.com/golangci/golangci-lint/blob/master/CHANGELOG.md)
- [Commits](golangci/golangci-lint@v1.42.1...v1.43.0)

---
updated-dependencies:
- dependency-name: github.com/golangci/golangci-lint
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.41.0 to 1.42.0.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.41.0...v1.42.0)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
…/tools/github.com/golangci/golangci-lint-1.43.0' into update-deps
…s/otlp/otlpmetric/google.golang.org/grpc-1.42.0' into update-deps
…s/otlp/otlpmetric/otlpmetrichttp/go.opentelemetry.io/proto/otlp-0.10.0' into update-deps
…s/otlp/otlpmetric/go.opentelemetry.io/proto/otlp-0.10.0' into update-deps
…s/otlp/otlptrace/go.opentelemetry.io/proto/otlp-0.10.0' into update-deps
…s/otlp/otlpmetric/otlpmetricgrpc/google.golang.org/grpc-1.42.0' into update-deps
…s/otlp/otlptrace/google.golang.org/grpc-1.42.0' into update-deps
…s/otlp/otlptrace/otlptracehttp/go.opentelemetry.io/proto/otlp-0.10.0' into update-deps
…s/otlp/otlptrace/otlptracegrpc/google.golang.org/grpc-1.42.0' into update-deps
…s/otlp/otlpmetric/otlpmetricgrpc/go.opentelemetry.io/proto/otlp-0.10.0' into update-deps
…s/otlp/otlptrace/otlptracegrpc/go.opentelemetry.io/proto/otlp-0.10.0' into update-deps
…ic/github.com/benbjohnson/clock-1.2.0' into update-deps
@Aneurysm9
Copy link
Member

Looks like golangci-lint requires Go 1.16+

We should discuss this at the SIG meeting today. I think we can't maintain compatibility with older versions indefinitely and I expect to see many projects require 1.17 once 1.16 isn't supported upstream due to the improvements in module handling.

@MrAlias
Copy link
Contributor Author

MrAlias commented Nov 11, 2021

Looks like golangci-lint requires Go 1.16+

We should discuss this at the SIG meeting today. I think we can't maintain compatibility with older versions indefinitely and I expect to see many projects require 1.17 once 1.16 isn't supported upstream due to the improvements in module handling.

Agreed.

@MrAlias
Copy link
Contributor Author

MrAlias commented Nov 11, 2021

/easycla

1 similar comment
@MrAlias
Copy link
Contributor Author

MrAlias commented Nov 11, 2021

/easycla

@MrAlias MrAlias closed this Nov 11, 2021
@MrAlias MrAlias reopened this Nov 11, 2021
@MrAlias
Copy link
Contributor Author

MrAlias commented Nov 11, 2021

/easycla

@MadVikingGod
Copy link
Contributor

I noted that lint did require 1.16 that, but that is only a part of our "build" process. Because the API or SDK don't import golangci-lint, go 1.15 should still be able to use them as a library. We just have to bump our test "build" platforms.

@MrAlias
Copy link
Contributor Author

MrAlias commented Nov 11, 2021

I noted that lint did require 1.16 that, but that is only a part of our "build" process. Because the API or SDK don't import golangci-lint, go 1.15 should still be able to use them as a library. We just have to bump our test "build" platforms.

Gotcha, I'll add that here. Not going anywhere real fast given EasyCLA looks stuck.

Our lint and race testers use this version of Go and we still explicitly
check version 1.15 with our compatibility tests. This will allow us to
update golangci-lint which requires Go 1.16.
…modules/internal/tools/github.com/golangci/golangci-lint-1.43.0' into update-deps""

This reverts commit 9d19c7d.
@codecov
Copy link

codecov bot commented Nov 11, 2021

Codecov Report

Merging #2374 (912d727) into main (3b9e067) will decrease coverage by 0.0%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #2374     +/-   ##
=======================================
- Coverage   73.8%   73.7%   -0.1%     
=======================================
  Files        175     175             
  Lines      12436   12436             
=======================================
- Hits        9182    9176      -6     
- Misses      3017    3023      +6     
  Partials     237     237             
Impacted Files Coverage Δ
sdk/trace/batch_span_processor.go 83.2% <0.0%> (-2.1%) ⬇️
exporters/jaeger/jaeger.go 93.4% <0.0%> (-0.9%) ⬇️

@MrAlias
Copy link
Contributor Author

MrAlias commented Nov 11, 2021

/easycla

@MrAlias
Copy link
Contributor Author

MrAlias commented Nov 11, 2021

/easycla

1 similar comment
@MrAlias
Copy link
Contributor Author

MrAlias commented Nov 11, 2021

/easycla

@MrAlias MrAlias closed this Nov 11, 2021
@MrAlias MrAlias reopened this Nov 11, 2021
@Aneurysm9
Copy link
Member

/easycla ಠ_ಠ

@MrAlias
Copy link
Contributor Author

MrAlias commented Nov 11, 2021

I really don't get how it worked for #2376 but not this ...

@Aneurysm9
Copy link
Member

I wonder if it is because Dependabot is still the author of commits on the branch. I pulled this into #2377 and it is doing the same thing there.

@MrAlias
Copy link
Contributor Author

MrAlias commented Nov 11, 2021

I wonder if it is because Dependabot is still the author of commits on the branch. I pulled this into #2377 and it is doing the same thing there.

Interesting, I'll try collapsing this and see if that works.

@MrAlias MrAlias mentioned this pull request Nov 11, 2021
@MrAlias
Copy link
Contributor Author

MrAlias commented Nov 11, 2021

It looks like I got a working version in #2378. It might be the merges of main from the GitHub console that are causing the problem. Not 100%, but in the linked PR when I did that it failed to run the EasyCI check. Likely relevant for #2373.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants