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

[WIP] Deps update - forked controller-runtime #1446

Closed
wants to merge 6 commits into from

Conversation

OverOrion
Copy link
Contributor

@OverOrion OverOrion commented Aug 29, 2023

This PR is used for running the on the current state.

@OverOrion
Copy link
Contributor Author

There are (at least) two things that are prerequisites for the dependencies update to work:

The latter is required, because there is an upstream bug (kubernetes/client-go#1269) that causes a panic:

cd e2e && \
        LOGGING_OPERATOR_IMAGE="controller:local" \
        KIND_PATH="/home/orion/axoflow/logging-operator/bin/kind" \
        KIND_IMAGE="kindest/node:v1.23.17@sha256:f77f8cf0b30430ca4128cc7cfafece0c274a118cd0cdb251049664ace0dee4ff" \
        PROJECT_DIR="/home/orion/axoflow/logging-operator" \
        go test -v -timeout 20m .//...
# sigs.k8s.io/kustomize/kyaml/openapi
../../../go/pkg/mod/sigs.k8s.io/kustomize/[email protected]/openapi/openapi.go:683:33: cannot use doc (variable of type *"github.com/google/gnostic/openapiv2".Document) as *"github.com/google/gnostic-models/openapiv2".Document value in argument to swagger.FromGnostic
# k8s.io/kubectl/pkg/util/openapi
../../../go/pkg/mod/k8s.io/[email protected]/pkg/util/openapi/openapi_getter.go:36:42: cannot use &CachedOpenAPIGetter{} (value of type *CachedOpenAPIGetter) as discovery.OpenAPISchemaInterface value in variable declaration: *CachedOpenAPIGetter does not implement discovery.OpenAPISchemaInterface (wrong type for method OpenAPISchema)
                have OpenAPISchema() (*"github.com/google/gnostic/openapiv2".Document, error)
                want OpenAPISchema() (*"github.com/google/gnostic-models/openapiv2".Document, error)
../../../go/pkg/mod/k8s.io/[email protected]/pkg/util/openapi/openapi_getter.go:49:28: cannot use g.openAPIClient.OpenAPISchema() (value of type *"github.com/google/gnostic-models/openapiv2".Document) as *"github.com/google/gnostic/openapiv2".Document value in assignment
../../../go/pkg/mod/k8s.io/[email protected]/pkg/util/openapi/openapi_getter.go:78:46: cannot use oapi (variable of type *"github.com/google/gnostic-models/openapiv2".Document) as *"github.com/google/gnostic/openapiv2".Document value in argument to NewOpenAPIData
../../../go/pkg/mod/k8s.io/[email protected]/pkg/util/openapi/openapi.go:52:38: cannot use doc (variable of type *"github.com/google/gnostic/openapiv2".Document) as *"github.com/google/gnostic-models/openapiv2".Document value in argument to proto.NewOpenAPIData
?       github.com/kube-logging/logging-operator/e2e/common     [no test files]
?       github.com/kube-logging/logging-operator/e2e/common/cond        [no test files]
?       github.com/kube-logging/logging-operator/e2e/common/kind        [no test files]
FAIL    github.com/kube-logging/logging-operator/e2e/fluentd-aggregator [build failed]
FAIL    github.com/kube-logging/logging-operator/e2e/syslog-ng-aggregator [build failed]
FAIL    github.com/kube-logging/logging-operator/e2e/volumedrain [build failed]
FAIL
make[1]: *** [Makefile:166: test-e2e-nodeps] Error 1

@pepov pepov closed this Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants