water-hole / ansible-operator

POC Code for the operator backed by ansible
48 stars 29 forks source link

Latest Build is Broken #70

Open alexsorkin opened 5 years ago

alexsorkin commented 5 years ago

Image Digest 2a6df59805b8 is broken.

Anything I'm trying to deploy using my operator build on top of 2a6df59805b8 ends up in: {"level":"error","ts":1545062584.6350603,"logger":"kubebuilder.controller","msg":"Reconciler error","controller":"blueocean-controller","request":"kube-system/example-blueocean","error":"the server could not find the requested resource","stacktrace":"github.com/water-hole/ansible-operator/vendor/github.com/go-logr/zapr.(zapLogger).Error\n\t/home/travis/gopath/src/github.com/water-hole/ansible-operator/vendor/github.com/go-logr/zapr/zapr.go:128\ngithub.com/water-hole/ansible-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller.(Controller).processNextWorkItem\n\t/home/travis/gopath/src/github.com/water-hole/ansible-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:215\ngithub.com/water-hole/ansible-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func1\n\t/home/travis/gopath/src/github.com/water-hole/ansible-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:158\ngithub.com/water-hole/ansible-operator/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil.func1\n\t/home/travis/gopath/src/github.com/water-hole/ansible-operator/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:133\ngithub.com/water-hole/ansible-operator/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil\n\t/home/travis/gopath/src/github.com/water-hole/ansible-operator/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:134\ngithub.com/water-hole/ansible-operator/vendor/k8s.io/apimachinery/pkg/util/wait.Until\n\t/home/travis/gopath/src/github.com/water-hole/ansible-operator/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:88"}

Havilland commented 5 years ago

I found a way to fix this error.

You have to add the subresources to your CRD.

They added this in the last commit. https://github.com/water-hole/ansible-operator/commit/c069cb9af8dc82758fbea1240723c3fec3762d61#diff-2801c16178697c1c264821ad53fbd762

I found that after updating the operator-sdk and creating a new ansible-operator to compare it to my existing one.

MarSik commented 5 years ago

Same here. I noticed the CHANGELOG but this never occured to me. Maybe an operator-sdk update or at least operator-sdk lint would be nice.

MarSik commented 5 years ago

@Havilland thanks for figuring it out.