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

October 16th attackaton feedback #656

Open
m3dwards opened this issue Oct 16, 2024 · 17 comments
Open

October 16th attackaton feedback #656

m3dwards opened this issue Oct 16, 2024 · 17 comments

Comments

@m3dwards
Copy link
Collaborator

No description provided.

@m3dwards
Copy link
Collaborator Author

We had the question about if it's remote, why do I need to install anything.

@m3dwards
Copy link
Collaborator Author

m3dwards commented Oct 16, 2024

  • People didn't understand that you had to clone the galen repo

fixed in bitcoin-dev-project/battle-of-galen-erso@97e2cf4 --MZ

@m3dwards
Copy link
Collaborator Author

People asked what the 99 nodes were based on, i.e. v28.

@m3dwards
Copy link
Collaborator Author

m3dwards commented Oct 16, 2024

  • People didn't know to expand the section on fork observer to see the full hostname

https://github.com/bitcoin-dev-project/battle-of-galen-erso?tab=readme-ov-file#attack-development

"Tanks in the kubernetes network have URIs that include a namespace. URIs for all tanks can be seen in the "description" field in the fork-observer web UI."

-- MZ

@m3dwards
Copy link
Collaborator Author

Update the warnet/app docker image

@m3dwards
Copy link
Collaborator Author

Sometimes the warnet run didn't work. Looked like the code didn't get uploaded. Also had an RPC command fail.

@m3dwards
Copy link
Collaborator Author

m3dwards commented Oct 16, 2024

  • update the node defaults to disable net logging by default?

done: bitcoin-dev-project/battle-of-galen-erso@b62c391 --MZ

@m3dwards
Copy link
Collaborator Author

See if logs work and give everyone the link

@m3dwards
Copy link
Collaborator Author

Shall we tell people that if they want to turn on specific logging that they can ask us?

@m3dwards
Copy link
Collaborator Author

The invalid op_code scenario is really hard to take down because the transactions have to pass standardness checks. Could we disable the standardness checks?

@m3dwards
Copy link
Collaborator Author

need to make it easier to find the hostnames

@m3dwards
Copy link
Collaborator Author

Add code snippets on how to do things in scenarios like sending a message, creating a transaction connecting to a peer.

@m3dwards
Copy link
Collaborator Author

m3dwards commented Oct 16, 2024

  • Someone asked if we could bundle the actual functional test suite or at least link to it

We did: https://github.com/bitcoin-dev-project/battle-of-galen-erso?tab=readme-ov-file#hints --MZ

@m3dwards
Copy link
Collaborator Author

bitcorncore.org has lots of references to bitcoincore which could be grep'd and changed

@m3dwards
Copy link
Collaborator Author

Perhaps rename miner_std to local_miner?

@pinheadmz
Copy link
Contributor

pinheadmz commented Oct 16, 2024

  • Change the exporter metrics. Utxo set size from one node and memory usage info from target nodes

done: bitcoin-dev-project/battle-of-galen-erso@b62c391

@m3dwards
Copy link
Collaborator Author

Write a guide in the documentation on how to switch back and forth between cluster and local.

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

No branches or pull requests

2 participants