Skip to content

Commit 21a4c6a

Browse files
author
Srinivasan Muralidharan
committed
FAB-1128 finalize protos - remove discovery and devops
https://jira.hyperledger.org/browse/FAB-1128 This change set is a follow up of the previous one to cleanup protos and addresses Gari's comments . removes core/discovery (not used) . removes devops.proto. The only function used from devops.go has been copied into bddtests/endorser_test.go. Change-Id: I096a45a8eb898e07c02a840839754f5c8af7cfcf Signed-off-by: Srinivasan Muralidharan <[email protected]>
1 parent 77c7323 commit 21a4c6a

19 files changed

+156
-627
lines changed

bddtests/endorser_test.go

+43
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ limitations under the License.
1717
package bddtests
1818

1919
import (
20+
"errors"
2021
"fmt"
2122
"strconv"
2223
"time"
@@ -25,13 +26,49 @@ import (
2526

2627
"github.com/DATA-DOG/godog"
2728
"github.com/DATA-DOG/godog/gherkin"
29+
"github.com/hyperledger/fabric/core/chaincode/platforms"
30+
"github.com/hyperledger/fabric/core/container"
2831
"github.com/hyperledger/fabric/core/util"
2932
pb "github.com/hyperledger/fabric/protos/peer"
3033
"github.com/hyperledger/fabric/protos/utils"
3134
"golang.org/x/net/context"
3235
"google.golang.org/grpc"
3336
)
3437

38+
// checkSpec to see if chaincode resides within current package capture for language.
39+
func (*BDDContext) checkSpec(spec *pb.ChaincodeSpec) error {
40+
// Don't allow nil value
41+
if spec == nil {
42+
return errors.New("Expected chaincode specification, nil received")
43+
}
44+
45+
platform, err := platforms.Find(spec.Type)
46+
if err != nil {
47+
return fmt.Errorf("Failed to determine platform type: %s", err)
48+
}
49+
50+
return platform.ValidateSpec(spec)
51+
}
52+
53+
// Build builds the supplied chaincode image
54+
func (b *BDDContext) build(spec *pb.ChaincodeSpec) (*pb.ChaincodeDeploymentSpec, error) {
55+
var codePackageBytes []byte
56+
if err := b.checkSpec(spec); err != nil {
57+
return nil, err
58+
}
59+
60+
vm, err := container.NewVM()
61+
if err != nil {
62+
return nil, fmt.Errorf("Error getting vm")
63+
}
64+
65+
codePackageBytes, err = vm.BuildChaincodeContainer(spec)
66+
if err != nil {
67+
return nil, err
68+
}
69+
chaincodeDeploymentSpec := &pb.ChaincodeDeploymentSpec{ChaincodeSpec: spec, CodePackage: codePackageBytes}
70+
return chaincodeDeploymentSpec, nil
71+
}
3572
func (b *BDDContext) weCompose(composeFiles string) error {
3673
if b.composition != nil {
3774
return fmt.Errorf("Already have composition in BDD context (%s)", b.composition.projectName)
@@ -118,6 +155,7 @@ func (b *BDDContext) userCreatesADeploymentSpecUsingChaincodeSpecAndDevopsOnPeer
118155
return errRetFunc()
119156
}
120157

158+
/****** Let us get the deployment spec directly from this machine like SDK would
121159
// Now use the devops client to create the deployment spec
122160
var grpcClient *grpc.ClientConn
123161
if grpcClient, err = b.getGrpcClientForComposeService(devopsPeerComposeService); err != nil {
@@ -129,6 +167,11 @@ func (b *BDDContext) userCreatesADeploymentSpecUsingChaincodeSpecAndDevopsOnPeer
129167
if ccDeploymentSpec, err = devopsClient.Build(context.Background(), ccSpec); err != nil {
130168
return errRetFunc()
131169
}
170+
********/
171+
var ccDeploymentSpec *pb.ChaincodeDeploymentSpec
172+
if ccDeploymentSpec, err = b.build(ccSpec); err != nil {
173+
return errRetFunc()
174+
}
132175
// Now store the chaincode deployment spec
133176
if _, err = userRegistration.SetTagValue(ccDeploymentSpecAlias, ccDeploymentSpec); err != nil {
134177
return errRetFunc()

core/devops.go

-83
This file was deleted.

core/devops_test.go

-57
This file was deleted.

core/discovery/discovery.go

-119
This file was deleted.

0 commit comments

Comments
 (0)