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 openstack-k8s-operators #280

Merged
merged 3 commits into from
Apr 4, 2023

Conversation

openstack-k8s-ci-robot
Copy link

@openstack-k8s-ci-robot openstack-k8s-ci-robot commented Mar 4, 2023

This PR contains the following updates:

Package Type Update Change
github.com/openstack-k8s-operators/infra-operator/apis require digest d3cedda -> d1ac945
github.com/openstack-k8s-operators/keystone-operator/api require digest 37346b6 -> 279efbd
github.com/openstack-k8s-operators/lib-common/modules/common require digest 39ed0da -> 2c21535
github.com/openstack-k8s-operators/lib-common/modules/database require digest 39ed0da -> 2c21535
github.com/openstack-k8s-operators/lib-common/modules/storage require digest c4fe7a2 -> 2c21535
github.com/openstack-k8s-operators/lib-common/modules/test require digest 39ed0da -> 2c21535
github.com/openstack-k8s-operators/mariadb-operator/api require digest e49b65b -> 3a320b9
github.com/openstack-k8s-operators/openstack-ansibleee-operator/api require digest a27b51d -> 2ffc926

Configuration

📅 Schedule: Branch creation - "every weekend" (UTC), 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.

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


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

This PR has been generated by Renovate Bot.

@openshift-ci openshift-ci bot requested review from kk7ds and SeanMooney March 4, 2023 00:42
@openstack-k8s-ci-robot openstack-k8s-ci-robot force-pushed the renovate/openstack-k8s-operators branch from 9ea65c2 to 26c2de4 Compare March 6, 2023 15:53
@gibizer
Copy link
Contributor

gibizer commented Mar 7, 2023

/hold
The lib-common bump needs manual adaptation as in #283

@openstack-k8s-ci-robot openstack-k8s-ci-robot force-pushed the renovate/openstack-k8s-operators branch 7 times, most recently from c6be75e to 1819d25 Compare March 10, 2023 13:11
@gibizer
Copy link
Contributor

gibizer commented Mar 10, 2023

/unhold
#283 has been merged

@openstack-k8s-ci-robot openstack-k8s-ci-robot force-pushed the renovate/openstack-k8s-operators branch 16 times, most recently from 095a2c2 to b846b72 Compare March 17, 2023 11:09
@openstack-k8s-ci-robot openstack-k8s-ci-robot force-pushed the renovate/openstack-k8s-operators branch 6 times, most recently from da46b4a to 57d7f3f Compare April 3, 2023 11:49
@gibizer
Copy link
Contributor

gibizer commented Apr 3, 2023

This needs a functional fix to adapt to the lib-common changes

@gibizer
Copy link
Contributor

gibizer commented Apr 3, 2023

pushed test adaptation commits to align with the new lib-common version

@openstack-k8s-ci-robot
Copy link
Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.

Warning: custom changes will be lost.

@gibizer gibizer mentioned this pull request Apr 3, 2023
@gibizer gibizer force-pushed the renovate/openstack-k8s-operators branch from beb9223 to 186a5a4 Compare April 3, 2023 16:59
@gibizer
Copy link
Contributor

gibizer commented Apr 4, 2023

We need a version bump of github.com/go-task/slim-sprig in openstack-operator before we can deploy this:
https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openstack-k8s-operators_nova-operator/280/pull-ci-openstack-k8s-operators-nova-operator-master-nova-operator-build-deploy/1642934978801045504/artifacts/nova-operator-build-deploy/openstack-k8s-operators-build/build-log.txt

+ pushd /alabama/openstack-operator
~/openstack-operator ~/openstack-operator ~ /go/src/github.com/openstack-k8s-operators/nova-operator
+ GOWORK=
+ make build bundle
mkdir -p /alabama/openstack-operator/bin
test -s /alabama/openstack-operator/bin/controller-gen || GOBIN=/alabama/openstack-operator/bin go install sigs.k8s.io/controller-tools/cmd/[email protected]
/alabama/openstack-operator/bin/controller-gen object:headerFile="hack/boilerplate.go.txt" paths="./..."
go fmt ./...
test -f go.work || go work init
go work use .
go work use ./apis
go work sync
go: downloading github.com/emicklei/go-restful v2.9.5+incompatible
go: version constraints conflict:
	github.com/onsi/ginkgo/[email protected] requires github.com/go-task/[email protected], but github.com/go-task/[email protected] is requested
	github.com/onsi/[email protected] requires github.com/go-task/[email protected], but github.com/go-task/[email protected] is requested
	github.com/openstack-k8s-operators/nova-operator/[email protected] requires github.com/go-task/[email protected], but github.com/go-task/[email protected] is requested
go: version constraints conflict:
	github.com/onsi/ginkgo/[email protected] requires github.com/go-task/[email protected], but github.com/go-task/[email protected] is requested
	github.com/onsi/[email protected] requires github.com/go-task/[email protected], but github.com/go-task/[email protected] is requested
	github.com/openstack-k8s-operators/nova-operator/[email protected] requires github.com/go-task/[email protected], but github.com/go-task/[email protected] is requested
make: *** [Makefile:332: gowork] Error 1

gibizer added a commit to gibizer/openstack-operator that referenced this pull request Apr 4, 2023
To be able to build in recent nova-operator changes into
openstack-operator we need to resolve a version conflict in go-task.

See openstack-k8s-operators/nova-operator#280 (comment)
@gibizer
Copy link
Contributor

gibizer commented Apr 4, 2023

The deploy job depends on openstack-k8s-operators/openstack-operator#263

@gibizer
Copy link
Contributor

gibizer commented Apr 4, 2023

The build in github action fails due to golang version conflict:

go: go.mod file indicates go 1.19, but maximum version supported by tidy is 1.18

@gibizer
Copy link
Contributor

gibizer commented Apr 4, 2023

The build in github action fails due to golang version conflict:

go: go.mod file indicates go 1.19, but maximum version supported by tidy is 1.18

It is being fixed in #327

@gibizer
Copy link
Contributor

gibizer commented Apr 4, 2023

/test nova-operator-build-deploy
openstack-k8s-operators/openstack-operator#263 has been merged.

OpenStack K8s CI and others added 3 commits April 4, 2023 11:47
In openstack-k8s-operators/lib-common#231
lib-common fixed typos in condition messages. This patch adapts our test
to the changed messages
In openstack-k8s-operators/lib-common#214
lib-common added  new network annotations to the pods. Adapt our tests
to the extra annotation.
@gibizer
Copy link
Contributor

gibizer commented Apr 4, 2023

#327 is merged

@gibizer gibizer force-pushed the renovate/openstack-k8s-operators branch from 186a5a4 to 159270d Compare April 4, 2023 10:27
@gibizer
Copy link
Contributor

gibizer commented Apr 4, 2023

/test nova-operator-build-deploy

It seems we got hit by a rate limit

[2/16] STEP 1/1: FROM quay.io/openstack-k8s-operators/infra-operator-bundle:d1ac945900ed9062c3358c9b45b88d36456a93b2 AS infra-bundle
Trying to pull quay.io/openstack-k8s-operators/infra-operator-bundle:d1ac945900ed9062c3358c9b45b88d36456a93b2...
error: build error: error creating build container: 4 errors occurred while pulling:
 * initializing source docker://registry.redhat.io/golang:1.19: reading manifest 1.19 in registry.redhat.io/golang: unknown: Not Found
 * initializing source docker://registry.access.redhat.com/golang:1.19: reading manifest 1.19 in registry.access.redhat.com/golang: name unknown: Repo not found
 * initializing source docker://quay.io/golang:1.19: reading manifest 1.19 in quay.io/golang: StatusCode: 404, <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final/...
 * initializing source docker://golang:1.19: reading manifest 1.19 in docker.io/library/golang: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
+ BASE_BUNDLE=quay.rdoproject.org/openstack-k8s-operators/openstack-operator-bundle:159270dbe5dbcaf458cd07c4019d1aca3781f42c

Copy link
Contributor

@gibizer gibizer left a comment

Choose a reason for hiding this comment

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

finally

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 4, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gibizer, openstack-k8s-ci-robot

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

The pull request process is described 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

@openshift-ci openshift-ci bot added the approved label Apr 4, 2023
@openshift-merge-robot openshift-merge-robot merged commit 7e9aa75 into master Apr 4, 2023
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.

3 participants