-
Notifications
You must be signed in to change notification settings - Fork 83
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
testutil/compose: add support for real vcs #595
Conversation
b, err := static.ReadFile(path.Join(staticRoot, d.Name(), f.Name())) | ||
if err != nil { | ||
return errors.Wrap(err, "read file") | ||
} | ||
|
||
if err := os.WriteFile(path.Join(dir, d.Name(), f.Name()), b, info.Mode()); err != nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't use info.Mode()
since that is 0o444
for embedded files.
@@ -9,23 +9,23 @@ x-node-base: &node-base | |||
{{if .Bootnode }}depends_on: [bootnode] {{end}} | |||
|
|||
services: | |||
{{ range $i, $node := .Nodes}} | |||
{{- range $i, $node := .Nodes}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix spacing issues
var vcByType = map[vcType]vc{ | ||
vcLighthouse: { | ||
Label: string(vcLighthouse), | ||
Build: "lighthouse", | ||
}, | ||
vcTeku: { | ||
Label: string(vcTeku), | ||
Image: "consensys/teku:latest", | ||
Command: `| | ||
validator-client | ||
--network=auto | ||
--beacon-node-api-endpoint="http://node1:16002" | ||
--validator-keys="/compose/node1:/compose/node1" | ||
--validators-proposer-default-fee-recipient="0x0000000000000000000000000000000000000000"`, | ||
}, | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the meat of the PR.
Codecov Report
@@ Coverage Diff @@
## main #595 +/- ##
==========================================
+ Coverage 54.16% 54.41% +0.24%
==========================================
Files 95 95
Lines 8922 8929 +7
==========================================
+ Hits 4833 4859 +26
+ Misses 3389 3376 -13
+ Partials 700 694 -6
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM🚀
Adds support for real VCs.
category: feature
ticket: #568