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 module github.com/imdario/mergo to v1 - autoclosed #219

Closed
wants to merge 1 commit into from

Conversation

openstack-k8s-ci-robot
Copy link

@openstack-k8s-ci-robot openstack-k8s-ci-robot commented Jun 24, 2023

This PR contains the following updates:

Package Type Update Change
github.com/imdario/mergo require major v0.3.16 -> v1.0.0

Release Notes

imdario/mergo (github.com/imdario/mergo)

v1.0.0: 1.0.0 released with new module URL: dario.cat/mergo

Compare Source

This PR is a release containing 0.3.15 features but changing the module URL to dario.cat/mergo.

What's Changed

Full Changelog: darccio/mergo@v0.3.16...v1.0.0


Configuration

📅 Schedule: Branch creation - "every weekend" in timezone America/New_York, Automerge - At any time (no schedule defined).

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

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

@openstack-k8s-ci-robot
Copy link
Author

openstack-k8s-ci-robot commented Jun 24, 2023

⚠ Artifact update problem

Renovate failed to update artifacts 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: go.sum
Command failed: go get -d -t ./...
go: github.com/imdario/[email protected]: parsing go.mod:
	module declares its path as: dario.cat/mergo
	        but was required as: github.com/imdario/mergo

File name: api/go.sum
Command failed: go get -d -t ./...
go: github.com/imdario/[email protected]: parsing go.mod:
	module declares its path as: dario.cat/mergo
	        but was required as: github.com/imdario/mergo

File name: go.mod
Command failed: make gowork
Makefile:76: Command "oc" and "kubectl" missing, some targets may fail
go: github.com/imdario/[email protected]: parsing go.mod:
	module declares its path as: dario.cat/mergo
	        but was required as: github.com/imdario/mergo
make: *** [Makefile:151: gowork] Error 1

File name: go.mod
Command failed: make tidy
Makefile:76: Command "oc" and "kubectl" missing, some targets may fail
go: downloading github.com/imdario/mergo v1.0.0
go: github.com/imdario/[email protected]: parsing go.mod:
	module declares its path as: dario.cat/mergo
	        but was required as: github.com/imdario/mergo
make: *** [Makefile:132: tidy] Error 1

File name: go.mod
Command failed: make manifests generate
Makefile:76: Command "oc" and "kubectl" missing, some targets may fail
/go/pkg/mod/k8s.io/[email protected]/tools/clientcmd/client_config.go:33:2: missing go.sum entry for module providing package github.com/imdario/mergo (imported by k8s.io/client-go/tools/clientcmd); to add:
	go get k8s.io/client-go/tools/[email protected]
/go/pkg/mod/k8s.io/[email protected]/tools/clientcmd/client_config.go:33:2: missing go.sum entry for module providing package github.com/imdario/mergo (imported by k8s.io/client-go/tools/clientcmd); to add:
	go get k8s.io/client-go/tools/[email protected]
Error: not all generators ran successfully
run `controller-gen rbac:roleName=manager-role crd:maxDescLen=0 webhook paths=./... output:crd:artifacts:config=config/crd/bases -w` to see all available markers, or `controller-gen rbac:roleName=manager-role crd:maxDescLen=0 webhook paths=./... output:crd:artifacts:config=config/crd/bases -h` for usage
make: *** [Makefile:112: manifests] Error 1

File name: api/go.mod
Command failed: make gowork
Makefile:76: Command "oc" and "kubectl" missing, some targets may fail
go: github.com/imdario/[email protected]: parsing go.mod:
	module declares its path as: dario.cat/mergo
	        but was required as: github.com/imdario/mergo
make: *** [Makefile:151: gowork] Error 1

File name: api/go.mod
Command failed: make tidy
Makefile:76: Command "oc" and "kubectl" missing, some targets may fail
go: github.com/imdario/[email protected]: parsing go.mod:
	module declares its path as: dario.cat/mergo
	        but was required as: github.com/imdario/mergo
make: *** [Makefile:132: tidy] Error 1

File name: api/go.mod
Command failed: make manifests generate
Makefile:76: Command "oc" and "kubectl" missing, some targets may fail
/go/pkg/mod/k8s.io/[email protected]/tools/clientcmd/client_config.go:33:2: missing go.sum entry for module providing package github.com/imdario/mergo (imported by k8s.io/client-go/tools/clientcmd); to add:
	go get k8s.io/client-go/tools/[email protected]
/go/pkg/mod/k8s.io/[email protected]/tools/clientcmd/client_config.go:33:2: missing go.sum entry for module providing package github.com/imdario/mergo (imported by k8s.io/client-go/tools/clientcmd); to add:
	go get k8s.io/client-go/tools/[email protected]
Error: not all generators ran successfully
run `controller-gen rbac:roleName=manager-role crd:maxDescLen=0 webhook paths=./... output:crd:artifacts:config=config/crd/bases -w` to see all available markers, or `controller-gen rbac:roleName=manager-role crd:maxDescLen=0 webhook paths=./... output:crd:artifacts:config=config/crd/bases -h` for usage
make: *** [Makefile:112: manifests] Error 1

@openshift-ci openshift-ci bot requested review from ASBishop and viroel June 24, 2023 04:55
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 24, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: openstack-k8s-ci-robot
Once this PR has been reviewed and has the lgtm label, please assign eharney for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@stuggi
Copy link
Contributor

stuggi commented Jul 20, 2023

/hold lib-common v0.1.0 still uses v0.3.16 so we should not bump this to 1.0 now

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 21, 2023

@openstack-k8s-ci-robot: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/unit a8a6c7b link true /test unit
ci/prow/precommit-check a8a6c7b link true /test precommit-check
ci/prow/cinder-operator-build-deploy-tempest a8a6c7b link false /test cinder-operator-build-deploy-tempest
ci/prow/cinder-operator-build-deploy-kuttl a8a6c7b link true /test cinder-operator-build-deploy-kuttl

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@openstack-k8s-ci-robot openstack-k8s-ci-robot changed the title Update module github.com/imdario/mergo to v1 Update module github.com/imdario/mergo to v1 - autoclosed Jul 24, 2023
@openstack-k8s-ci-robot openstack-k8s-ci-robot deleted the renovate/major-misc branch July 24, 2023 13:50
ASBishop pushed a commit to ASBishop/cinder-operator that referenced this pull request Mar 11, 2024
…rators/renovate/openstack-k8s-operators

Update openstack-k8s-operators
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants