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

fix(deps): update golang #259

Closed
wants to merge 2 commits into from
Closed

fix(deps): update golang #259

wants to merge 2 commits into from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Feb 24, 2022

Mend Renovate

This PR contains the following updates:

Package Type Update Change
github.com/jackc/pgconn require minor v1.10.0 -> v1.12.1
github.com/jackc/pgx/v4 require minor v4.13.0 -> v4.16.1
github.com/jetstack/cert-manager require minor v1.6.1 -> v1.8.2
github.com/open-telemetry/opentelemetry-operator require minor v0.39.0 -> v0.55.0
github.com/spf13/cobra require minor v1.2.1 -> v1.5.0
github.com/spf13/viper require minor v1.9.0 -> v1.12.0
helm.sh/helm/v3 require minor v3.7.0 -> v3.9.1
k8s.io/api require minor v0.22.4 -> v0.24.3
k8s.io/apiextensions-apiserver require minor v0.22.4 -> v0.24.3
k8s.io/apimachinery require minor v0.22.4 -> v0.24.3
k8s.io/cli-runtime require minor v0.22.4 -> v0.24.3

Configuration

📅 Schedule: Branch creation - "before 2am" in timezone Etc/UTC, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, click this checkbox. ⚠ Warning: custom changes will be lost.

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot requested a review from a team as a code owner February 24, 2022 01:21
@renovate renovate bot added the cli label Feb 24, 2022
@CLAassistant
Copy link

CLAassistant commented Feb 24, 2022

CLA assistant check
All committers have signed the CLA.

@paulfantom
Copy link
Contributor

Blocked by #260

@paulfantom paulfantom self-assigned this Feb 24, 2022
@paulfantom paulfantom added the dependencies Pull requests that update a dependency file label Feb 24, 2022
@renovate renovate bot force-pushed the renovate/golang branch 2 times, most recently from 77d7ceb to a059083 Compare March 2, 2022 17:15
@renovate renovate bot force-pushed the renovate/golang branch 3 times, most recently from dc97a9f to 32fb6ca Compare March 10, 2022 15:00
@cevian
Copy link
Collaborator

cevian commented Mar 11, 2022

@paulfantom should this get merged now?

Copy link
Member

@Harkishen-Singh Harkishen-Singh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some lint issues which i think is due to mismatch in go mod. Fixing that it should be good to merge.

@renovate renovate bot force-pushed the renovate/golang branch 2 times, most recently from 3f70bbd to dde9b01 Compare March 19, 2022 20:59
Copy link

@sumerman sumerman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The build has been red for weeks. I don't feel comfortable green-lighting it.

Copy link
Contributor

@paulfantom paulfantom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This cannot be merged until issues with cert-manager are solved. It might be better to wait for a Design Doc on the future of the project than to invest development cycles into solving issues from this PR.

@renovate
Copy link
Contributor Author

renovate bot commented Apr 8, 2022

⚠ Artifact update problem

Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.

♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

  • any of the package files in this branch needs updating, or
  • the branch becomes conflicted, or
  • you click the rebase/retry checkbox if found above, or
  • you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:

File name: cli/go.sum
Command failed: docker run --rm --name=renovate_go --label=renovate_child -v "/mnt/renovate/gh/timescale/tobs":"/mnt/renovate/gh/timescale/tobs" -v "/tmp/renovate-cache":"/tmp/renovate-cache" -e GOPATH -e GOPROXY -e GOFLAGS -e CGO_ENABLED -e GIT_CONFIG_KEY_0 -e GIT_CONFIG_VALUE_0 -e GIT_CONFIG_KEY_1 -e GIT_CONFIG_VALUE_1 -e GIT_CONFIG_KEY_2 -e GIT_CONFIG_VALUE_2 -e GIT_CONFIG_COUNT -e GIT_CONFIG_KEY_3 -e GIT_CONFIG_VALUE_3 -e GIT_CONFIG_KEY_4 -e GIT_CONFIG_VALUE_4 -e GIT_CONFIG_KEY_5 -e GIT_CONFIG_VALUE_5 -w "/mnt/renovate/gh/timescale/tobs/cli" docker.io/renovate/go:1.18.1 bash -l -c "go get -d -t ./..."
go: github.com/jetstack/cert-manager@v1.8.0: parsing go.mod:
	module declares its path as: github.com/cert-manager/cert-manager
	        but was required as: github.com/jetstack/cert-manager

@renovate renovate bot force-pushed the renovate/golang branch 2 times, most recently from fb8baa8 to d19c671 Compare April 13, 2022 17:37
@renovate renovate bot force-pushed the renovate/golang branch 2 times, most recently from c9f0aa0 to 726a6d8 Compare April 22, 2022 03:04
@renovate renovate bot requested a review from a team as a code owner April 22, 2022 03:04
@paulfantom paulfantom marked this pull request as draft May 3, 2022 13:30
@paulfantom paulfantom added this to the 0.11.0 milestone May 10, 2022
@paulfantom paulfantom modified the milestones: 0.11.0, 0.12.0 May 25, 2022
@paulfantom
Copy link
Contributor

Closing as we are deprecating CLI.

@paulfantom paulfantom closed this Jul 20, 2022
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants