Skip to content

Commit 8c44a7d

Browse files
kibbles-n-bytespmorie
authored andcommitted
update OSB client to 2.13 (openshift#1392)
* change alpha resources to non-alpha versions * update OSB client to 2.13
1 parent e64bbd1 commit 8c44a7d

35 files changed

+764
-204
lines changed

glide.lock

+3-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

glide.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -88,4 +88,4 @@ import:
8888
- package: code.cloudfoundry.org/lager
8989
version: dfcbcba2dd4a5228c43b0292d219d5c010daed3a
9090
- package: github.com/pmorie/go-open-service-broker-client
91-
version: 962a416798366b37a20a06fdb0a09180589ca964
91+
version: 530248ec023d52217c43db2f06fa03ce844c40cc

pkg/controller/controller.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -584,7 +584,7 @@ func convertClusterServicePlans(plans []osb.Plan, serviceClassID string) ([]*v1b
584584
servicePlans[i].Spec.ExternalMetadata = &runtime.RawExtension{Raw: metadata}
585585
}
586586

587-
if schemas := plan.AlphaParameterSchemas; schemas != nil {
587+
if schemas := plan.ParameterSchemas; schemas != nil {
588588
if instanceSchemas := schemas.ServiceInstances; instanceSchemas != nil {
589589
if instanceCreateSchema := instanceSchemas.Create; instanceCreateSchema != nil && instanceCreateSchema.Parameters != nil {
590590
schema, err := json.Marshal(instanceCreateSchema.Parameters)

pkg/controller/controller_binding_test.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -1743,7 +1743,7 @@ func TestReconcileBindingUsingOriginatingIdentity(t *testing.T) {
17431743
t.Errorf("%v: unexpected request type; expected %T, got %T", tc.name, &osb.BindRequest{}, actualRequest)
17441744
return
17451745
}
1746-
var expectedOriginatingIdentity *osb.AlphaOriginatingIdentity
1746+
var expectedOriginatingIdentity *osb.OriginatingIdentity
17471747
if tc.expectedOriginatingIdentity {
17481748
expectedOriginatingIdentity = testOriginatingIdentity
17491749
}
@@ -1794,7 +1794,7 @@ func TestReconcileBindingDeleteUsingOriginatingIdentity(t *testing.T) {
17941794
t.Errorf("%v: unexpected request type; expected %T, got %T", tc.name, &osb.UnbindRequest{}, actualRequest)
17951795
return
17961796
}
1797-
var expectedOriginatingIdentity *osb.AlphaOriginatingIdentity
1797+
var expectedOriginatingIdentity *osb.OriginatingIdentity
17981798
if tc.expectedOriginatingIdentity {
17991799
expectedOriginatingIdentity = testOriginatingIdentity
18001800
}

pkg/controller/controller_instance.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -692,7 +692,7 @@ func (c *controller) reconcileServiceInstance(instance *v1beta1.ServiceInstance)
692692
UserInfo: instance.Spec.UserInfo,
693693
}
694694

695-
var originatingIdentity *osb.AlphaOriginatingIdentity
695+
var originatingIdentity *osb.OriginatingIdentity
696696
if utilfeature.DefaultFeatureGate.Enabled(scfeatures.OriginatingIdentity) {
697697
originatingIdentity, err = buildOriginatingIdentity(instance.Spec.UserInfo)
698698
if err != nil {

pkg/controller/controller_instance_test.go

+3-3
Original file line numberDiff line numberDiff line change
@@ -2716,7 +2716,7 @@ func TestReconcileInstanceUsingOriginatingIdentity(t *testing.T) {
27162716
t.Errorf("%v: unexpected request type; expected %T, got %T", tc.name, &osb.ProvisionRequest{}, actualRequest)
27172717
return
27182718
}
2719-
var expectedOriginatingIdentity *osb.AlphaOriginatingIdentity
2719+
var expectedOriginatingIdentity *osb.OriginatingIdentity
27202720
if tc.expectedOriginatingIdentity {
27212721
expectedOriginatingIdentity = testOriginatingIdentity
27222722
}
@@ -2770,7 +2770,7 @@ func TestReconcileInstanceDeleteUsingOriginatingIdentity(t *testing.T) {
27702770
t.Errorf("%v: unexpected request type; expected %T, got %T", tc.name, &osb.DeprovisionRequest{}, actualRequest)
27712771
return
27722772
}
2773-
var expectedOriginatingIdentity *osb.AlphaOriginatingIdentity
2773+
var expectedOriginatingIdentity *osb.OriginatingIdentity
27742774
if tc.expectedOriginatingIdentity {
27752775
expectedOriginatingIdentity = testOriginatingIdentity
27762776
}
@@ -2817,7 +2817,7 @@ func TestPollInstanceUsingOriginatingIdentity(t *testing.T) {
28172817
t.Errorf("%v: unexpected request type; expected %T, got %T", tc.name, &osb.LastOperationRequest{}, actualRequest)
28182818
return
28192819
}
2820-
var expectedOriginatingIdentity *osb.AlphaOriginatingIdentity
2820+
var expectedOriginatingIdentity *osb.OriginatingIdentity
28212821
if tc.expectedOriginatingIdentity {
28222822
expectedOriginatingIdentity = testOriginatingIdentity
28232823
}

pkg/controller/controller_test.go

+3-3
Original file line numberDiff line numberDiff line change
@@ -350,7 +350,7 @@ const testOriginatingIdentityValue = `{
350350
"fakekey": ["fakevalue"]
351351
}`
352352

353-
var testOriginatingIdentity = &osb.AlphaOriginatingIdentity{
353+
var testOriginatingIdentity = &osb.OriginatingIdentity{
354354
Platform: originatingIdentityPlatform,
355355
Value: testOriginatingIdentityValue,
356356
}
@@ -775,7 +775,7 @@ func TestCatalogConversion(t *testing.T) {
775775
checkPlan(servicePlans[1], "0f4008b5-XXXX-XXXX-XXXX-dace631cd648", "fake-plan-2", "Shared fake Server, 5tb persistent disk, 40 max concurrent connections. 100 async", t)
776776
}
777777

778-
func TestCatalogConversionWithAlphaParameterSchemas(t *testing.T) {
778+
func TestCatalogConversionWithParameterSchemas(t *testing.T) {
779779
catalog := &osb.CatalogResponse{}
780780
err := json.Unmarshal([]byte(alphaParameterSchemaCatalogBytes), &catalog)
781781
if err != nil {
@@ -2473,7 +2473,7 @@ func assertUnbind(t *testing.T, action fakeosb.Action, request *osb.UnbindReques
24732473
}
24742474
}
24752475

2476-
func assertOriginatingIdentity(t *testing.T, expected *osb.AlphaOriginatingIdentity, actual *osb.AlphaOriginatingIdentity) {
2476+
func assertOriginatingIdentity(t *testing.T, expected *osb.OriginatingIdentity, actual *osb.OriginatingIdentity) {
24772477
if e, a := expected, actual; (e != nil) != (a != nil) {
24782478
fatalf(t, "unexpected originating identity in request: expected %q, got %q", e, a)
24792479
}

pkg/controller/originating_identity.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ const (
3030
originatingIdentityGroups = "groups"
3131
)
3232

33-
func buildOriginatingIdentity(userInfo *v1beta1.UserInfo) (*osb.AlphaOriginatingIdentity, error) {
33+
func buildOriginatingIdentity(userInfo *v1beta1.UserInfo) (*osb.OriginatingIdentity, error) {
3434
if userInfo == nil {
3535
return nil, nil
3636
}
@@ -45,7 +45,7 @@ func buildOriginatingIdentity(userInfo *v1beta1.UserInfo) (*osb.AlphaOriginating
4545
if err != nil {
4646
return nil, err
4747
}
48-
oi := &osb.AlphaOriginatingIdentity{
48+
oi := &osb.OriginatingIdentity{
4949
Platform: originatingIdentityPlatform,
5050
Value: string(oiValue),
5151
}

vendor/github.com/pmorie/go-open-service-broker-client/.travis.yml

+3-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/pmorie/go-open-service-broker-client/README.md

+3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)