diff --git a/Gopkg.lock b/Gopkg.lock index 53c079c..f50c898 100644 --- a/Gopkg.lock +++ b/Gopkg.lock @@ -291,12 +291,12 @@ version = "v1.0.0" [[projects]] - digest = "1:c37d9d96de085f7a82451b6f62352d32f92f93663373535a8f429ea1981e4a00" + digest = "1:f01e38ee53d9eab9295424e4b5c67c652ac65f4f69bf8c765a375dee02d60df5" name = "github.com/google/go-github" packages = ["github"] pruneopts = "NT" - revision = "b041e285fc4f7a936a81027c21c810cbaac2dc13" - version = "v25.0.0" + revision = "1fef44b9b427e6c43f92b2f20918e496c275393f" + version = "v25.0.2" [[projects]] digest = "1:a63cff6b5d8b95638bfe300385d93b2a6d9d687734b863da8e09dc834510a690" @@ -336,7 +336,7 @@ [[projects]] branch = "master" - digest = "1:f620c969301c9543b5a311a662615f69fbaeeb6864865df9e86a3eca560445e6" + digest = "1:36e1d8cbdf6853c7abf45742d49d38948e8d5d02b7c18ebf0dd07c59b233ec23" name = "github.com/gophercloud/gophercloud" packages = [ ".", @@ -348,7 +348,7 @@ "pagination", ] pruneopts = "NT" - revision = "bb98932a7b3a106e1814e0c6b7f7d48c0a71a919" + revision = "60507118a582789eaa9c8dcefc1374c7f5431b9b" [[projects]] branch = "master" @@ -386,11 +386,11 @@ [[projects]] branch = "master" - digest = "1:c13aea0b77243f1fcd0253583c90e180010a3a5bbdf43f61a3daddc48d88b0b2" + digest = "1:b0b1cae9ff30d8c219dc64e5f8dcfbb3f5f3f10f9e67dec737ce0bc04bf31a54" name = "github.com/iancoleman/strcase" packages = ["."] pruneopts = "NT" - revision = "3605ed457bf7f8caa1371b4fafadadc026673479" + revision = "e506e3ef73653e84c592ba44aab577a46678f68c" [[projects]] digest = "1:aaa38889f11896ee3644d77e17dc7764cc47f5f3d3b488268df2af2b52541c5f" @@ -530,7 +530,8 @@ revision = "84c2b942258aea2462e675e03aeb8eb4cb5f3c29" [[projects]] - digest = "1:674610d54812d3c36ab7861fc826176bf581a9426cc09abec0107414c17f89cd" + branch = "controller-runtime-0.2.0-alpha0-vendor" + digest = "1:e36c3a2b15891f2b517400bf85f7c0d7efa07588ecfe9101535068777bbad4ae" name = "github.com/operator-framework/operator-sdk" packages = [ "internal/pkg/scaffold", @@ -549,8 +550,8 @@ "version", ] pruneopts = "NT" - revision = "7e07b20c9e22560fb22a3295b612eae7652b8b14" - version = "v0.7.0" + revision = "74bb44b203f4f70e9ed8b9a084806fd6d5fd96e8" + source = "github.com/corinnekrych/operator-sdk" [[projects]] digest = "1:93b1d84c5fa6d1ea52f4114c37714cddd84d5b78f151b62bb101128dd51399bf" @@ -577,12 +578,12 @@ revision = "53be0d36a84c2a886ca057d34b6aa4468df9ccb4" [[projects]] - digest = "1:e4e9e026b8e4c5630205cd0208efb491b40ad40552e57f7a646bb8a46896077b" + digest = "1:d8139b87c97ab035e546f58bb812ec6eb11518ec7dff03694e5427bc67ca7c0f" name = "github.com/peterbourgon/diskv" packages = ["."] pruneopts = "NT" - revision = "5f041e8faa004a95c88a202771f4cc3e991971e6" - version = "v2.0.1" + revision = "0be1b92a6df0e4f5cb0a5d15fb7f643d0ad93ce6" + version = "v3.0.0" [[projects]] digest = "1:14715f705ff5dfe0ffd6571d7d201dd8e921030f8070321a79380d8ca4ec1a24" @@ -634,21 +635,23 @@ [[projects]] branch = "master" - digest = "1:75f1b9f32d4d204eac68f3427bc4642453fb4fe20f1596ec058757f01dc16e7b" + digest = "1:55ff1480a5e2c800636fb8971c08e407acfb321b1002b876e3acc6725402f542" name = "github.com/prometheus/procfs" packages = ["."] pruneopts = "NT" - revision = "8368d24ba045f26503eb745b624d930cbe214c79" + revision = "87a4384529e0652f5035fb5cc8095faf73ea9b0b" [[projects]] - digest = "1:16678f9916ec29322beba0678d0e1564082261e7fdc4b9ae8cb97c39e053627b" + branch = "update.version.runtime-controller" + digest = "1:4e7e746fb11821543e9616e1bdb6c91f6b8a7b3324cd4bdeffba0b678836882c" name = "github.com/redhat-developer/devconsole-api" packages = [ "pkg/apis", "pkg/apis/devconsole/v1alpha1", ] pruneopts = "NT" - revision = "e9de16b0f6bbc760546eb19e3cec6e043855f8fa" + revision = "daa8af1daeccb296ec43858844c08061a573c2ba" + source = "github.com/corinnekrych/devconsole-api" [[projects]] digest = "1:d08555f5becd60863db2ec355b4611fa0590524ee0fca69cb2282a33e84ec3f1" @@ -841,7 +844,7 @@ [[projects]] branch = "master" - digest = "1:f8cb3167bd2a33ac306e159a8d88534dbf973569e848cb5f34de94d9bad552e8" + digest = "1:5f92996be2964f7a413cc8c6b7ff02e88e02db6042371b7acadc06cd9dc5a658" name = "golang.org/x/crypto" packages = [ "cast5", @@ -864,11 +867,11 @@ "ssh/terminal", ] pruneopts = "NT" - revision = "f416ebab96af27ca70b6e5c23d6a0747530da626" + revision = "a29dc8fdc73485234dbef99ebedb95d2eced08de" [[projects]] branch = "master" - digest = "1:89cf4e2680b312eb7a1b76f32c29089145b639f578b5fcef106c71ee867921f6" + digest = "1:940590250ff6229f11678ea802c15e75768bf39b6440d24025bc5bede1f40367" name = "golang.org/x/net" packages = [ "context", @@ -881,7 +884,7 @@ "trace", ] pruneopts = "NT" - revision = "1da14a5a36f220ea3f03470682b737b1dfd5de22" + revision = "4829fb13d2c62012c17688fa7f629f371014946d" [[projects]] branch = "master" @@ -899,15 +902,15 @@ [[projects]] branch = "master" - digest = "1:5a0a5ba0bb4fadf7e1c3eaaa48923e0efef23e43243e6038c426288aeef92c42" + digest = "1:bde1844fd8dff0a4ed259e24bf0e3cb91c2bc7bc3cd0a43118c6ef8f8be1b9ca" name = "golang.org/x/sync" packages = ["semaphore"] pruneopts = "NT" - revision = "56d357773e8497dfd526f0727e187720d1093757" + revision = "112230192c580c3556b8cee6403af37a4fc5f28c" [[projects]] branch = "master" - digest = "1:aded639d831ffb2df2531a0298f4bd094b56cb3a79bb212c3b7ebdf5b5f40eb0" + digest = "1:6b6dcecf1cf3271a2b8902cab8e2fd119b68eebaacf4eb06e709def6d51bbac7" name = "golang.org/x/sys" packages = [ "cpu", @@ -915,16 +918,18 @@ "windows", ] pruneopts = "NT" - revision = "12500544f89f9420afe9529ba8940bf72d294972" + revision = "2cc0cad0ac78f3ede1f4d2b6ca16ea3038641af4" [[projects]] - digest = "1:8c74f97396ed63cc2ef04ebb5fc37bb032871b8fd890a25991ed40974b00cd2a" + digest = "1:348fa8283a7c60b5b71ce04d27b37f7c0fce552d4d0b463b5b3ebbd1840d3f1a" name = "golang.org/x/text" packages = [ "collate", "collate/build", "internal/colltab", "internal/gen", + "internal/language", + "internal/language/compact", "internal/tag", "internal/triegen", "internal/ucd", @@ -938,8 +943,8 @@ "width", ] pruneopts = "NT" - revision = "f21a4dfb5e38f5895301dc265a8def02365cc3d0" - version = "v0.3.0" + revision = "342b2e1fbaa52c93f31447ad2c6abc048c63e475" + version = "v0.3.2" [[projects]] branch = "master" @@ -951,7 +956,7 @@ [[projects]] branch = "master" - digest = "1:3176402f98ede82da36c998ab557f60da76a1e95b9347bf00ac5336581a65b87" + digest = "1:10bc5604a6f43dde4d6609ddf58820f77f1c0f99a4b6bb559a6c2a98abc3505f" name = "golang.org/x/tools" packages = [ "go/ast/astutil", @@ -967,15 +972,15 @@ "internal/semver", ] pruneopts = "NT" - revision = "a5870b40385977769b3e1d711864fcd2d82bb9fd" + revision = "550556f78a900afacb9e8b22219f5e5b5c6106db" [[projects]] - digest = "1:568f70cf0f1cacd5fc9b9ad58aed1a42cf5348b001080dc0be69832f78819c9b" + digest = "1:81d407dcb2347a544350b7fb394e0eb065a34c75dc235fd4c6061ea72024a485" name = "google.golang.org/api" packages = ["support/bundler"] pruneopts = "NT" - revision = "0cbcb99a9ea0c8023c794b2693cbe1def82ed4d7" - version = "v0.3.2" + revision = "067bed655e9cbc26f4dbac8f8897b30756d90990" + version = "v0.4.0" [[projects]] digest = "1:711179d92dbef98932acd7afd0dab80b4cf2d97f099f3285c9ba8fa451809fec" @@ -998,7 +1003,7 @@ [[projects]] branch = "master" - digest = "1:a252437ce08a185d70c3fc019a015b175a13c2c8e5fe5c61bbb16ed87ea5857d" + digest = "1:08676f4e20fd831dae5113b3aa175595e83275956128a090bc19b1c88987e6b7" name = "google.golang.org/genproto" packages = [ "googleapis/api/httpbody", @@ -1006,10 +1011,10 @@ "protobuf/field_mask", ] pruneopts = "NT" - revision = "d1146b9035b912113a38af3b138eb2af567b2c67" + revision = "357c62f0e4bbba7e6cc403ae09edcf3e2b9028fe" [[projects]] - digest = "1:d0908c315cebc69c64e6a21df389f50de5458c9693d95cd60c1d65b3d0806c39" + digest = "1:5761833c144990cd4ce4136a99284f5323116c9c063a1953f0bda38483a9c3cb" name = "google.golang.org/grpc" packages = [ ".", @@ -1046,8 +1051,8 @@ "tap", ] pruneopts = "NT" - revision = "236199dd5f8031d698fb64091194aecd1c3895b2" - version = "v1.20.0" + revision = "25c4f928eaa6d96443009bd842389fb4fa48664e" + version = "v1.20.1" [[projects]] digest = "1:2d1fbdc6777e5408cabeb02bf336305e724b925ff4546ded0fa8715a7267922a" @@ -1161,7 +1166,7 @@ version = "v2.2.2" [[projects]] - digest = "1:6fa82ea248029bbbdddade20c06ab177ff6e485e5e45e48b045707415b7efd34" + digest = "1:18b9b70bdf29da610c0df032cf60ee7526a48756fa44012cc639dcb557335cbc" name = "k8s.io/api" packages = [ "admission/v1beta1", @@ -1199,7 +1204,8 @@ "storage/v1beta1", ] pruneopts = "NT" - revision = "05914d821849570fba9eacfb29466f2d8d3cd229" + revision = "5cb15d34447165a97c76ed5a60e4e99c8a01ecfe" + version = "kubernetes-1.13.4" [[projects]] digest = "1:c6f23048e162e65d586c809fd02e263e180ad157f110df17437c22517bb59a4b" @@ -1210,10 +1216,11 @@ "pkg/client/clientset/clientset/scheme", ] pruneopts = "NT" - revision = "0fe22c71c47604641d9aa352c785b7912c200562" + revision = "d002e88f6236312f0289d9d1deab106751718ff0" + version = "kubernetes-1.13.4" [[projects]] - digest = "1:15b5c41ff6faa4d0400557d4112d6337e1abc961c65513d44fce7922e32c9ca7" + digest = "1:f44b5a32a31245ea062cd397cc343bda06d75c731596589996c1f77634e653af" name = "k8s.io/apimachinery" packages = [ "pkg/api/errors", @@ -1261,10 +1268,11 @@ "third_party/forked/golang/reflect", ] pruneopts = "NT" - revision = "2b1284ed4c93a43499e781493253e2ac5959c4fd" + revision = "86fb29eff6288413d76bd8506874fddd9fccdff0" + version = "kubernetes-1.13.4" [[projects]] - digest = "1:c904a3d70131b33df36e4e51b574226b82308fc1ea66964aa21095a95d453fc9" + digest = "1:7f28bbfb5889618ab038d4d250d7adb6762aa43b2fc120a95a1cd47c68af6dd4" name = "k8s.io/client-go" packages = [ "discovery", @@ -1344,7 +1352,8 @@ "util/workqueue", ] pruneopts = "NT" - revision = "8d9ed539ba3134352c586810e749e58df4e94e4f" + revision = "b40b2a5939e43f7ffe0028ad67586b7ce50bb675" + version = "kubernetes-1.13.4" [[projects]] digest = "1:dc1ae99dcab96913d81ae970b1f7a7411a54199b14bfb17a7e86f9a56979c720" @@ -1442,7 +1451,7 @@ revision = "0cf8f7e6ed1d2e3d47d02e3b6e559369af24d803" [[projects]] - digest = "1:06035489efbd51ccface65fc878ceeb849aba05b2f9443c8993f363fc96e80ac" + digest = "1:f27c48732f0350be071e81d971a62afb9f5f1ff686ee6cf2b5e14c9266f50741" name = "sigs.k8s.io/controller-runtime" packages = [ "pkg/cache", @@ -1457,28 +1466,31 @@ "pkg/handler", "pkg/internal/controller", "pkg/internal/controller/metrics", + "pkg/internal/log", + "pkg/internal/objectutil", "pkg/internal/recorder", "pkg/leaderelection", + "pkg/log", + "pkg/log/zap", "pkg/manager", + "pkg/manager/signals", "pkg/metrics", - "pkg/patch", "pkg/predicate", "pkg/reconcile", "pkg/recorder", "pkg/runtime/inject", "pkg/runtime/log", - "pkg/runtime/scheme", "pkg/runtime/signals", + "pkg/scheme", "pkg/source", "pkg/source/internal", + "pkg/webhook", "pkg/webhook/admission", - "pkg/webhook/admission/types", "pkg/webhook/internal/metrics", - "pkg/webhook/types", ] pruneopts = "NT" - revision = "12d98582e72927b6cd0123e2b4e819f9341ce62c" - version = "v0.1.10" + revision = "4276f3895df0acc9249f817eb86a47a3db6b7a9e" + version = "v0.2.0-alpha.0" [[projects]] digest = "1:0a14ea9a2647d064bb9d48b2de78306e74b196681efd7b654eb0b518d90c2e8d" @@ -1530,7 +1542,6 @@ "k8s.io/api/core/v1", "k8s.io/apimachinery/pkg/api/errors", "k8s.io/apimachinery/pkg/apis/meta/v1", - "k8s.io/apimachinery/pkg/labels", "k8s.io/apimachinery/pkg/runtime", "k8s.io/apimachinery/pkg/runtime/schema", "k8s.io/apimachinery/pkg/types", diff --git a/Gopkg.toml b/Gopkg.toml index 48b1a0f..73e6641 100644 --- a/Gopkg.toml +++ b/Gopkg.toml @@ -31,23 +31,19 @@ required = [ [[override]] name = "k8s.io/api" - # revision for tag "kubernetes-1.13.1" - revision = "05914d821849570fba9eacfb29466f2d8d3cd229" + version = "kubernetes-1.13.4" [[override]] name = "k8s.io/apiextensions-apiserver" - # revision for tag "kubernetes-1.13.1" - revision = "0fe22c71c47604641d9aa352c785b7912c200562" + version = "kubernetes-1.13.4" [[override]] name = "k8s.io/apimachinery" - # revision for tag "kubernetes-1.13.1" - revision = "2b1284ed4c93a43499e781493253e2ac5959c4fd" + version = "kubernetes-1.13.4" [[override]] name = "k8s.io/client-go" - # revision for tag "kubernetes-1.13.1" - revision = "8d9ed539ba3134352c586810e749e58df4e94e4f" + version = "kubernetes-1.13.4" [[override]] name = "github.com/coreos/prometheus-operator" @@ -55,13 +51,13 @@ required = [ [[override]] name = "sigs.k8s.io/controller-runtime" - version = "=v0.1.10" + version = "v0.2.0-alpha.0" [[constraint]] name = "github.com/operator-framework/operator-sdk" - # The version rule is used for a specific release and the master branch for in between releases. - # branch = "master" #osdk_branch_annotation - version = "=v0.7.0" #osdk_version_annotation + source = "github.com/corinnekrych/operator-sdk" + #version = "=v0.7.0" #osdk_version_annotation + branch = "controller-runtime-0.2.0-alpha0-vendor" [[constraint]] name = "github.com/openshift/client-go" @@ -87,10 +83,6 @@ required = [ name = "github.com/stretchr/testify" version = "1.3.0" -[[constraint]] - branch = "master" - name = "github.com/wadey/gocovmerge" - [[override]] name = "github.com/golang/protobuf" @@ -102,8 +94,6 @@ required = [ [[constraint]] name = "github.com/redhat-developer/devconsole-api" - revision = "e9de16b0f6bbc760546eb19e3cec6e043855f8fa" - packages = [ - "pkg/apis", - "pkg/apis/devconsole/v1alpha1", - ] + source = "github.com/corinnekrych/devconsole-api" + branch = "update.version.runtime-controller" + diff --git a/deploy/crds/devconsole_v1alpha1_component_crd.yaml b/deploy/crds/devconsole_v1alpha1_component_crd.yaml index 8067f76..8f5739d 100644 --- a/deploy/crds/devconsole_v1alpha1_component_crd.yaml +++ b/deploy/crds/devconsole_v1alpha1_component_crd.yaml @@ -12,6 +12,8 @@ spec: shortNames: - cp scope: Namespaced + subresources: + status: {} validation: openAPIV3Schema: properties: diff --git a/make/test.mk b/make/test.mk index 62a1c9b..9ad7a94 100644 --- a/make/test.mk +++ b/make/test.mk @@ -178,7 +178,7 @@ test-e2e-local: build-image-local $(Q)-oc create -f ./deploy/crds/devconsole_v1alpha1_gitsource_crd.yaml $(Q)-oc create -f ./deploy/service_account.yaml --namespace $(TEST_NAMESPACE) $(Q)-oc create -f ./deploy/role.yaml --namespace $(TEST_NAMESPACE) -ifeq ($(UNAME_S),Darwin ) +ifeq ($(UNAME_S),Darwin) $(Q)sed -i "" 's|REPLACE_NAMESPACE|$(TEST_NAMESPACE)|g' ./deploy/test/role_binding_test.yaml else $(Q)sed -i 's|REPLACE_NAMESPACE|$(TEST_NAMESPACE)|g' ./deploy/test/role_binding_test.yaml diff --git a/pkg/controller/component/component_controller.go b/pkg/controller/component/component_controller.go index d3c0424..b30e93a 100644 --- a/pkg/controller/component/component_controller.go +++ b/pkg/controller/component/component_controller.go @@ -13,7 +13,6 @@ import ( corev1 "k8s.io/api/core/v1" "k8s.io/apimachinery/pkg/api/errors" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" - "k8s.io/apimachinery/pkg/labels" "k8s.io/apimachinery/pkg/runtime" "k8s.io/apimachinery/pkg/runtime/schema" "k8s.io/apimachinery/pkg/types" @@ -116,7 +115,6 @@ func (r *ReconcileComponent) Reconcile(request reconcile.Request) (reconcile.Res // Error reading the object - requeue the request/* */. return reconcile.Result{}, err } - // Checking and logging secondary resource lifecycle dcList := &v1.DeploymentConfigList{} err = r.ObserveDeploymentConfig(cp, dcList) @@ -196,15 +194,9 @@ func (r *ReconcileComponent) Reconcile(request reconcile.Request) (reconcile.Res // ObserveBuildConfig watches for secondary resource BuildConfig. func (r *ReconcileComponent) ObserveBuildConfig(cp *devconsoleapi.Component, bcList *buildv1.BuildConfigList) error { lbls := map[string]string{ - "app": cp.Name, - } - opts := client.ListOptions{ - Namespace: cp.Namespace, - LabelSelector: labels.SelectorFromSet(lbls), + "app.kubernetes.io/instance": cp.Name, } - err := r.client.List(context.TODO(), - &opts, - bcList) + err := r.client.List(context.TODO(), bcList, client.InNamespace(cp.Namespace), client.MatchingLabels(lbls)) if err != nil { log.Error(err, "failed to list existing BuildConfig") return err @@ -222,15 +214,9 @@ func (r *ReconcileComponent) ObserveBuildConfig(cp *devconsoleapi.Component, bcL // ObserveDeploymentConfig watches for secondary resource DeploymentConfig. func (r *ReconcileComponent) ObserveDeploymentConfig(cp *devconsoleapi.Component, dcList *v1.DeploymentConfigList) error { lbls := map[string]string{ - "app": cp.Name, - } - opts := client.ListOptions{ - Namespace: cp.Namespace, - LabelSelector: labels.SelectorFromSet(lbls), + "app.kubernetes.io/instance": cp.Name, } - err := r.client.List(context.TODO(), - &opts, - dcList) + err := r.client.List(context.TODO(), dcList, client.InNamespace(cp.Namespace), client.MatchingLabels(lbls)) if err != nil { log.Error(err, "failed to list existing DeploymentConfig") return err @@ -252,7 +238,7 @@ func (r *ReconcileComponent) ObserveDeploymentConfig(cp *devconsoleapi.Component func (r *ReconcileComponent) UpdateStatus(cp *devconsoleapi.Component, status string) error { if cp.Status.Phase != status { cp.Status.Phase = status - err := r.client.Update(context.TODO(), cp) + err := r.client.Status().Update(context.TODO(), cp) if err != nil { log.Error(err, "** failed to update component status **") return err