Skip to content

Conversation

@nguyer
Copy link
Contributor

@nguyer nguyer commented Jan 19, 2023

This PR updates the manifest.json with the latest version of each dependency service, the latest CLI, the latest UI, and sets evmconnect to be the new default for E2E tests. Ethconnect is still tested, but it now takes a back seat compared to evmconnect.

Signed-off-by: Nicko Guyer <nicko.guyer@kaleido.io>
Signed-off-by: Nicko Guyer <nicko.guyer@kaleido.io>
Signed-off-by: Nicko Guyer <nicko.guyer@kaleido.io>
Signed-off-by: Nicko Guyer <nicko.guyer@kaleido.io>
@codecov-commenter
Copy link

codecov-commenter commented Jan 19, 2023

Codecov Report

Merging #1154 (105aba3) into main (a23fcc1) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1154   +/-   ##
=======================================
  Coverage   99.98%   99.98%           
=======================================
  Files         305      305           
  Lines       19966    19967    +1     
=======================================
+ Hits        19964    19965    +1     
  Misses          1        1           
  Partials        1        1           
Impacted Files Coverage Δ
internal/database/sqlcommon/pin_sql.go 100.00% <100.00%> (ø)
internal/events/webhooks/webhooks.go 99.06% <0.00%> (-0.94%) ⬇️
internal/apiserver/server.go 100.00% <0.00%> (+0.91%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Signed-off-by: Nicko Guyer <nicko.guyer@kaleido.io>
Signed-off-by: Nicko Guyer <nicko.guyer@kaleido.io>
Signed-off-by: Nicko Guyer <nicko.guyer@kaleido.io>
Signed-off-by: Nicko Guyer <nicko.guyer@kaleido.io>
s.callbacks.OrderedCollectionNSEvent(database.CollectionPins, core.ChangeEventTypeCreated, pin.Namespace, pin.Sequence)
},
)
}, requestConflictEmptyResult)
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we log anything when the insert is a no-op? Seems good to at least be aware for debugging purposes.

"image": "ghcr.io/hyperledger/firefly-fabconnect",
"tag": "v.0.9.16",
"sha": "4f4798ee34eee0b67886f1baa37f4d50a33e2932c94312f189fe5f015f6564a4"
"tag": "null-20230119-30",
Copy link
Contributor

Choose a reason for hiding this comment

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

null?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah interestingly enough that's the tag on the latest fabconnect image: https://github.com/hyperledger/firefly-fabconnect/pkgs/container/firefly-fabconnect/64423633?tag=null-20230119-30

It will get updated before this is released anyway.

@nguyer nguyer merged commit 9ffba93 into hyperledger:main Jan 20, 2023
@nguyer nguyer deleted the manifest-update branch January 20, 2023 19:51
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.

3 participants