Skip to content

Conversation

@onelapahead
Copy link
Contributor

  • Templates blockchain.fabric config section of firefly.core with new Helm values
  • Chart CI values broken into two tests: it-eth and it-fab which tests setting connector values for both Ethereum and Fabric

@onelapahead onelapahead changed the title [helm] Fabrconnect Support [helm] Fabconnect Support Dec 12, 2021
@codecov-commenter
Copy link

codecov-commenter commented Dec 12, 2021

Codecov Report

Merging #356 (3735920) into main (b2678eb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #356   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          239       239           
  Lines        12888     12888           
=========================================
  Hits         12888     12888           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b2678eb...3735920. Read the comment docs.

Copy link
Contributor

@peterbroadhurst peterbroadhurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One suggestion @hfuss to consider, with approval

Signed-off-by: hfuss <haydenfuss@gmail.com>
Signed-off-by: hfuss <haydenfuss@gmail.com>
Signed-off-by: hfuss <haydenfuss@gmail.com>
Signed-off-by: hfuss <haydenfuss@gmail.com>
Signed-off-by: hfuss <haydenfuss@gmail.com>
Signed-off-by: hfuss <haydenfuss@gmail.com>
Signed-off-by: hfuss <haydenfuss@gmail.com>
Signed-off-by: hfuss <haydenfuss@gmail.com>
@onelapahead
Copy link
Contributor Author

Checks are failing and will need to be rerun due to ongoing DockerHub outage.

@peterbroadhurst peterbroadhurst merged commit 7065837 into hyperledger:main Dec 21, 2021
@peterbroadhurst peterbroadhurst deleted the helm-fabric-support branch December 21, 2021 03:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants