Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: bump opentelemetry to 0.17 #720

Merged
merged 2 commits into from
Jun 8, 2022

Conversation

jjyr
Copy link
Collaborator

@jjyr jjyr commented Jun 7, 2022

No description provided.

@jjyr jjyr requested review from blckngm and zeroqn June 7, 2022 03:21
@gw-bot
Copy link

gw-bot bot commented Jun 7, 2022

Running integration test

Workflow Run Id: 2451890334

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

failure

@blckngm
Copy link
Contributor

blckngm commented Jun 7, 2022

We can enable the “auto split batch” option to avoid “message too large” errors:

79d9d93

@jjyr
Copy link
Collaborator Author

jjyr commented Jun 8, 2022

@sopium Please review

@jjyr jjyr merged commit 1b5c0e1 into godwokenrises:develop Jun 8, 2022
@jjyr jjyr deleted the bump-opentelemetry branch June 8, 2022 06:48
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