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.

Sign up for GitHub

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

Jump to bottom

CAD-390 profiling chain-sync #401

Closed
CodiePP opened this issue Dec 12, 2019 · 3 comments
Closed

CAD-390 profiling chain-sync #401

CodiePP opened this issue Dec 12, 2019 · 3 comments
Assignees
@CodiePP
Labels
byron Required for a Byron mainnet: replace the old core nodes with cardano-node.
Milestone
S5 2020-01-30

Comments

@CodiePP
Copy link
Contributor

CodiePP commented Dec 12, 2019

a memory leak has been observed.
the chain-sync benchmark runs for one hour; should be done with profiling.
@CodiePP CodiePP added the test label Dec 12, 2019
@CodiePP CodiePP self-assigned this Dec 12, 2019
@karknu
Copy link
Contributor

karknu commented Dec 13, 2019

The memory leak seems to have been introduced with IntersectMBO/ouroboros-network@f2a050b

iohk-bors bot added a commit that referenced this issue Dec 16, 2019
@iohk-bors @CodiePP
Merge #408
05c55a7 
408: [#401] update chain sync config and scripts r=CodiePP a=CodiePP

updates to chain sync config and scripts.
replaces PR #385 

Co-authored-by: Alexander Diemand <[email protected]>
@CodiePP CodiePP mentioned this issue Dec 16, 2019
Closed
8 tasks
@mrBliss
Copy link
Contributor

mrBliss commented Dec 17, 2019

Fixed in IntersectMBO/ouroboros-network#1357

@CodiePP CodiePP added this to the S2 2019-12-19 milestone Dec 17, 2019
@CodiePP CodiePP changed the title profiling chain-sync CAD-390 profiling chain-sync Dec 23, 2019
@CodiePP CodiePP modified the milestones: S3 2020-01-02, S4 2020-01-16 Jan 6, 2020
@CodiePP
Copy link
Contributor Author

CodiePP commented Jan 6, 2020

needs to be reevaluated on new integration.

@CodiePP CodiePP added the byron Required for a Byron mainnet: replace the old core nodes with cardano-node. label Jan 13, 2020
@Jimbo4350 Jimbo4350 modified the milestones: S4 2020-01-16, S6 2020-02-13, S5 2020-01-30 Jan 16, 2020
@CodiePP CodiePP closed this as completed Jan 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CodiePP CodiePP

Labels
byron Required for a Byron mainnet: replace the old core nodes with cardano-node.
Projects
None yet
Milestone
S5 2020-01-30
Development

No branches or pull requests
4 participants
@mrBliss @karknu @Jimbo4350 @CodiePP