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
asset/manifests/infrastructure.go: Set the network resource group for the Azure cluster #2541
asset/manifests/infrastructure.go: Set the network resource group for the Azure cluster #2541
Conversation
The k8s libs need to be updated because the openshift/api now uses 1.16.0 version. Without this the build fails with error: ``` vendor/github.com/openshift/api/route/v1/generated.pb.go:448:28: m.ObjectMeta.MarshalToSizedBuffer undefined (type "github.com/openshift/installer/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".ObjectMeta has no field or method MarshalToSizedBuffer) vendor/github.com/openshift/api/route/v1/generated.pb.go:539:37: m.LastTransitionTime.MarshalToSizedBuffer undefined (type *"github.com/openshift/installer/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".Time has no field or method MarshalToSizedBuffer) vendor/github.com/openshift/api/route/v1/generated.pb.go:607:26: m.ListMeta.MarshalToSizedBuffer undefined (type "github.com/openshift/installer/vendor/k8s.io/apimachinery/pkg/apis/meta/v1".ListMeta has no field or method MarshalToSizedBuffer) vendor/github.com/openshift/api/route/v1/generated.pb.go:640:28: m.TargetPort.MarshalToSizedBuffer undefined (type intstr.IntOrString has no field or method MarshalToSizedBuffer) vendor/github.com/openshift/client-go/config/clientset/versioned/typed/config/v1/config_client.go:132:45: scheme.Codecs.WithoutConversion undefined (type serializer.CodecFactory has no field or method WithoutConversion) ```
/test e2e-azure |
1820066
to
972691c
Compare
/test e2e-azure |
With this change, the byo-vnet settings originally from #2441 (comment) no longer require the additional step of adding machine-api-operator permissions on the network resource group.
|
/lgtm |
/retest |
/test e2e-gcp |
The bump was done using ``` $ dep version dep: version : v0.5.0 build date : 2018-07-26 git hash : 224a564 go version : go1.10.3 go compiler : gc platform : linux/amd64 features : ImportDuringSolve=false $ dep ensure -update -v github.com/openshift/api github.com/openshift/client-go ```
29bba80 [1] switched off the glog output to highest and had the flag trick to prevent glog flags from showing up. With switch to kubernetes-1.16.0, all the k8s libs are using klog [2], that doesn't add the flags and also allows to set the default Ouput writer for all levels. Setting the klog output to ioutil.Discard keeps the current behavior [1]: openshift@29bba80 [2]: https://godoc.org/k8s.io/klog
… the Azure cluster The network resource group is the same as the cluster resource group, unless the Vnet and subnets were provided by the user to the installer.
972691c
to
3165e5f
Compare
/test e2e-azure |
/test e2e-gcp |
/retest |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhinavdahiya, jhixson74 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 |
The network resource group is the same as the cluster resource group, unless the Vnet and subnets were provided by the user to the installer.
The new field was added by openshift/api#463
Keeping the network resource group empty is same setting it to cluster resource group, but i think setting it explicitly is more accurate and visible.
/assign @jhixson74