Skip to content
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.

feat: apply wasm opts #23

Merged
merged 1 commit into from
Aug 24, 2021
Merged

feat: apply wasm opts #23

merged 1 commit into from
Aug 24, 2021

Conversation

brew0722
Copy link
Contributor

@brew0722 brew0722 commented Aug 23, 2021

Description

Closes: Finschia/finschia-sdk#308
Add wasm opts to enable telemetry.

Motivation and context

https://github.com/CosmWasm/wasmd/blob/93e2e669402dd96b99f223bc6176abef6e794455/cmd/wasmd/root.go#L204
The telemetry metrics is included :1317(API Server metrics).

e.g) recorded instantiate
{"Name":"wasm.contract.instantiate","Count":1,"Rate":2.222800064086914,"Sum":22.22800064086914,"Min":22.22800064086914,"Max":22.22800064086914,"Mean":22.22800064086914,"Stddev":0,"Labels":{}}]}

How has this been tested?

Screenshots (if appropriate):

Checklist:

  • I followed the contributing guidelines.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.

@brew0722 brew0722 self-assigned this Aug 23, 2021
@loloicci
Copy link

Closes: Finschia/finschia-sdk#308

I think this PR is a part of this issue and does not directly close the issue. Is it right?

@brew0722
Copy link
Contributor Author

brew0722 commented Aug 24, 2021

Closes: line/lfb-sdk#308

I think this PR is a part of this issue and does not directly close the issue. Is it right?

I think This PR resolve the entire issue.
Instantiate time includes compilation time, and Execute time includes execution time.
It seems useful for judge JIT bomb or gas consumption is appropriate.
So it looks like we don't need to add additional metrics.

Are there any more metrics that need to be added?

@loloicci
Copy link

Ah, Sorry. I missed our version of wasm has the metric feature. Thanks.

@brew0722
Copy link
Contributor Author

Me too, I didn't even know it was added. thanks for the review!

@brew0722 brew0722 merged commit 1670395 into Finschia:main Aug 24, 2021
@brew0722 brew0722 deleted the apply-wasm-opts branch August 24, 2021 04:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add VM Metrics
3 participants