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

DevOps: Regen net.json for scenario2 test to include api endpoints for nodes. #3654

Merged
merged 3 commits into from Feb 23, 2022

Conversation

egieseke
Copy link
Contributor

Summary

Regen scenario 2 net.json file to include API endpoints for nodes.

Test Plan

Testing with scenario2 recipe with cicd perf pipeline.

@codecov-commenter
Copy link

codecov-commenter commented Feb 18, 2022

Codecov Report

Merging #3654 (de2e615) into master (468d34b) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3654      +/-   ##
==========================================
- Coverage   48.12%   48.12%   -0.01%     
==========================================
  Files         381      381              
  Lines       62214    62214              
==========================================
- Hits        29939    29938       -1     
+ Misses      28847    28844       -3     
- Partials     3428     3432       +4     
Impacted Files Coverage Δ
crypto/merkletrie/trie.go 66.42% <0.00%> (-2.19%) ⬇️
crypto/merkletrie/node.go 91.62% <0.00%> (-1.87%) ⬇️
ledger/acctupdates.go 65.27% <0.00%> (-1.14%) ⬇️
catchup/peerSelector.go 98.95% <0.00%> (-1.05%) ⬇️
network/wsPeer.go 68.61% <0.00%> (-0.28%) ⬇️
network/wsNetwork.go 62.99% <0.00%> (+0.19%) ⬆️
catchup/service.go 70.12% <0.00%> (+1.48%) ⬆️
ledger/blockqueue.go 85.05% <0.00%> (+2.87%) ⬆️
cmd/algoh/blockWatcher.go 80.95% <0.00%> (+3.17%) ⬆️

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 468d34b...de2e615. Read the comment docs.

ghost
ghost previously approved these changes Feb 18, 2022
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Looks good.

algojack
algojack previously approved these changes Feb 18, 2022
@algobarb
Copy link
Contributor

Looks good, but does opening these API endpoints impact the amount of traffic going to telemetry? Just trying to still understand why Prometheus recently went down.

@egieseke egieseke dismissed stale reviews from algojack and ghost via 6af7573 February 18, 2022 16:32
@algobarb
Copy link
Contributor

Thanks for removing netaddress for scenario2. Should we also remove it everywhere else that we recently added it to, before we forget?

Copy link
Contributor

@algobarb algobarb left a comment

Choose a reason for hiding this comment

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

Thanks for removing the NetAddress from custom and scenario2.

@egieseke
Copy link
Contributor Author

Looks good, but does opening these API endpoints impact the amount of traffic going to telemetry? Just trying to still understand why Prometheus recently went down.

These endpoints allow the bandwidth tool to communicate with the nodes and collect the bandwidth data.

@egieseke egieseke requested review from algojack and a user February 23, 2022 16:29
Copy link
Contributor

@algojack algojack left a comment

Choose a reason for hiding this comment

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

lgtm

@tsachiherman tsachiherman merged commit 1bb149d into algorand:master Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants