Skip to content

Commit

Permalink
[FAB-14908] Support latest images for java/node cc
Browse files Browse the repository at this point in the history
Allow java chaincode javaenv and node chaincode baseimage
to be at different version level than Fabric.

Similar updates were made for v2.0 in FAB-11096.

Change-Id: Ia7706a07aa77f781e5e3199c54194a82b9850820
Signed-off-by: David Enyeart <[email protected]>
  • Loading branch information
denyeart committed Mar 29, 2019
1 parent edabb1b commit 29433f0
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
7 changes: 6 additions & 1 deletion core/chaincode/chaincodetest.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -315,7 +315,12 @@ chaincode:
# tools added for java shim layer packaging.
# This image is packed with shim layer libraries that are necessary
# for Java chaincode runtime.
runtime: $(DOCKER_NS)/fabric-javaenv:$(ARCH)-$(PROJECT_VERSION)
runtime: $(DOCKER_NS)/fabric-javaenv:latest

node:
# need node.js engine at runtime, currently available in baseimage
# but not in baseos
runtime: $(BASE_DOCKER_NS)/fabric-baseimage:latest

# timeout in millisecs for starting up a container and waiting for Register
# to come through. 1sec should be plenty for chaincode unit tests
Expand Down
4 changes: 2 additions & 2 deletions sampleconfig/core.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -505,12 +505,12 @@ chaincode:
# tools added for java shim layer packaging.
# This image is packed with shim layer libraries that are necessary
# for Java chaincode runtime.
runtime: $(DOCKER_NS)/fabric-javaenv:$(ARCH)-$(PROJECT_VERSION)
runtime: $(DOCKER_NS)/fabric-javaenv:latest

node:
# need node.js engine at runtime, currently available in baseimage
# but not in baseos
runtime: $(BASE_DOCKER_NS)/fabric-baseimage:$(ARCH)-$(BASE_VERSION)
runtime: $(BASE_DOCKER_NS)/fabric-baseimage:latest

# Timeout duration for starting up a container and waiting for Register
# to come through. 1sec should be plenty for chaincode unit tests
Expand Down

0 comments on commit 29433f0

Please sign in to comment.