hyperledger-archives / fabric

THIS IS A READ-ONLY historic repository. Current development is at https://gerrit.hyperledger.org/r/#/admin/projects/fabric . pull requests not accepted

Home Page:https://gerrit.hyperledger.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Behave tests are failing after merge the commit# `ead0af7 `

rameshthoomu opened this issue · comments

Description

Following behave tests are failing in Jenkins build after merge commit# ead0af7 in master repo.

peer_logging.Peer Logging.Invoke is attempted after deploy in Dev Mode
peer_logging.Peer Logging.Query is attempted after deploy in Dev Mode

error log:

File "steps/peer_logging_impl.py", line 61, in step_impl
          assert logHasError(stderr) is False
      AssertionError

Jenkins Build reference log files:

https://jenkins.hyperledger.org/view/fabric/job/fabric-merge-x86_64/12/
https://jenkins.hyperledger.org/view/fabric/job/fabric-merge-x86_64/13/
https://jenkins.hyperledger.org/view/fabric/job/fabric-merge-x86_64/14/

Describe How to Reproduce

Run behave tests in Jenkins..

@jeffgarratt @tuand27613 : Could you please check this?

@rameshthoomu I wrote these tests. For these tests the Chaincode is deployed manually and I suspect the containers aren't ready in time. I'm working on a more robust method of starting the containers which will fail fast and give a better logging of the container state: https://gerrit.hyperledger.org/r/#/c/421/.
I hope this helps.

@juliancarrivick-ibm thanks for the fix. Will re-verify once this patch set is merged in master.

The patch-set in question was the one that was reverted because it broke master. Since I have rebased it on the latest master a test has started failing. I am working on a fix now. The new change request is https://gerrit.hyperledger.org/r/#/c/563/

ok, closing this then since we have it being worked in Gerrit thanks