updateCheckpoint can run concurrent with the construction of a blocks
iterator. When this happens, serialization is needed between the update
of the checkpoint info and the read done by the construction of the
iterator.

Change-Id: I4fba0400820abd2c6d7c511286a8382b801563e9
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

Turns out when you have global singleton without serialization backing
all of the configuration for the system, it's not safe to update it
concurrently from multiple tests that have opted into parallel
execution.

It also turns out that relying on package initialization to setup a
global singleton works well until you use some other mechanism to clean
it up and remove it; then you just panic because going through
initialization again doesn't do anything.

Change-Id: Iba5f953885b05c7354a68ade9e2afad4b4f1b82c
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

Change-Id: Ifc4e66de1bf0ce3c67d84e736f5871b67a4439e9
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

Change-Id: I71c6d063e8aa81fc018299af6cc3beee2f4e4dff
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

Ticks are not guaranteed to be delivered in the event of slow receivers:

<quote ref="https://godoc.org/time#NewTicker

">
It adjusts the intervals or drops ticks to make up for slow receivers.
</quote>

Also remove unnecessary casts of numeric constants.

Change-Id: I7c85cf2435bd044231e542de106098e0eadd5312
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

The mock block cutter has no serialization but needs it.

Change-Id: I538789c4152279b6da1f73e5059525b7bbe25335
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

Addresses flake in CI related to process exit completion taking longer
than 1 second.

Applied the pattern via rewrite across all integration packages.

Change-Id: Ia2b2ce0f97acad66ade61b71c836a3a3ad418ad7
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

The go routine launched to get the next block from the iterator could
execute the happy-path setup before the error channel was closed.

Change-Id: Iad57413e207a0905de4101ba718697d33bb2c652
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

The test TestShuffle uses a slice of 1 million elements in order the
shuffling to fail with a negligible probability.

This runs in 2 seconds without race detector, but -
with race detector enabled it runs very very slow (20 seconds).

I changed the million to a thousand, and the number of
permutations should be still high enough to have the test
fail with low probability.

FAB-12209 #done

Change-Id: I57ccfbe0051055bb470f1c87d44eb813ecd6196e
Signed-off-by: yacovm <yacovm@il.ibm.com>

FAB-12083 #done

This change set adds a call to the context cancel function
when the stream cannot be established.

Change-Id: Ie38d5e59e6f6a4635dcf427ea4c2a4cf4d9ff44c
Signed-off-by: yacovm <yacovm@il.ibm.com>

This change set adds cancelations of the stream cancel function
on all exit paths to appease go vet.

FAB-12082 #done

Change-Id: Ib94d5baaa28ae65b96fbff8e1837b770ed7c6531
Signed-off-by: yacovm <yacovm@il.ibm.com>

Remove incorrect information about chaincodes.
Chaincodes must have the exact same fingerprint
across nodes on the channel.

Change-Id: I1710401468e8534d57ea8470f426ec0f6282b57e
Signed-off-by: David Enyeart <enyeart@us.ibm.com>

Pick up fix for EINVAL from os.File#Sync in goleveldb.

Change-Id: I5bc4dd64ab02430730851cfbe6464e80a5a8ad9f
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

Missed an underscore to end the link.

Change-Id: Ia41b813acf647a16237e85075bf2aeedf1f1a735
Signed-off-by: joe-alewine <Joe.Alewine@ibm.com>

This CR addresses the remaining comments in the CR
(https://gerrit.hyperledger.org/r/#/c/21775/

)

Primarily - renames the package valinternal to internal
and merges the two files with related content into one.

FAB-12189 #done

Change-Id: I6c61a48188aa6e4489cd7f58fc9cd49f922d889a
Signed-off-by: manish <manish.sethi@gmail.com>

Release note links need to formatted
correctly and changed.

Change-Id: I474b66c49e775812f169ab5a0e9a6628233097cf
Signed-off-by: joe-alewine <Joe.Alewine@ibm.com>

had to refactor a bit, as membership provider creation was
dependant on collectionStore, which is created per channel,
therefore membership provider was also created per channel.

changed it such that we can create a single membership provider

FAB-11780 - #done

Change-Id: I2c44ebf00d6eea1fc1d713b3168328f95074c6b5
Signed-off-by: nirro <nirro@il.ibm.com>

This fix lets the gossip private data coordinator recognize not only
regular writes to private data keys, but also updates of the metadata.

Change-Id: I41ac4dcdc929e1d2390a1eb75684d01c37264f4c
Signed-off-by: Matthias Neugschwandtner <eug@zurich.ibm.com>

Change-Id: Ia0d33bfd72132e1ede6c60d63416ae7262fdbfb5
Signed-off-by: David Enyeart <enyeart@us.ibm.com>

Note - also fixed changelog to remove v1.1.1
changes, as those changes are in release-1.1
changelog and are not in master.

Change-Id: I330353a138a944f9fd8cd78ce46fababef9adffc
Signed-off-by: David Enyeart <enyeart@us.ibm.com>

Change-Id: I91dd6c42b6470b84704b18ca9926f7948e5319c8
Signed-off-by: rameshthoomu <rameshbabu.thoomu@gmail.com>

Change-Id: I9cca5f8fbf0c3cfd0f19549952db9eb13bb43793
Signed-off-by: Christopher Ferris <chrisfer@us.ibm.com>

Change-Id: I869411c275412fa21084f62487e494cb4bcfc855
Signed-off-by: gennady <gennady@il.ibm.com>

This reverts commit 17b9da87

.

Reason for revert: relative links are incorrect

Change-Id: I83a8dbacd5f38efad338b36d059c569d29c7fc38
Signed-off-by: Christopher Ferris <chrisfer@us.ibm.com>

A page with links to new documentation
for v1.3 features. Add to TOC.

Change-Id: I46e21cf75b0fd7185eabf915db1ae22d528033e9
Signed-off-by: joe-alewine <Joe.Alewine@ibm.com>

From doc on Understanding the Fabcar
Network.

Change-Id: I688d4b4c22f03a1d20d8eb09625c613920a38ee9
Signed-off-by: joe-alewine <Joe.Alewine@ibm.com>

This allows us to run tests with and without the race detector.

Change-Id: If20a850c6e9d5484bf9f4b39415e0cabe8410d1c
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

Change-Id: I9ca389f711165dc8f447bbd28f14e923f1317113
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

Change-Id: I621467df3c699c39d84317297863bee688dc1cb9
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

* changes:
  [FAB-12085] lint out golang.org/x/net/context
  [FAB-12084] address simple lostcancel vet issues

Fix misspelling of "Singed" to "Signed" in testutil method name.

Change-Id: Ib9239ae507af6a92620299411ba58adb955c920c
Signed-off-by: Daisuke IIZUKA <daisuke.iizuka.ag@hitachi.com>

Exercise the validation for 1.3 at a block-level with blocks that contain
transactions requiring key-level validation. Also test with private
data.

Change-Id: Ib8fda0b52ac7da55b1aea407c3442a3437636a70
Signed-off-by: Matthias Neugschwandtner <eug@zurich.ibm.com>
Signed-off-by: Alessandro Sorniotti <ale.linux@sopit.net>