-
Notifications
You must be signed in to change notification settings - Fork 206
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
Load generator docker #1260
base: master
Are you sure you want to change the base?
Load generator docker #1260
Changes from 26 commits
2d76c84
c10d500
59c21ef
e6e61a5
e2d27cf
85c7b6d
48f8f3f
0101460
65484f7
6f9463f
b9c1c58
5670b74
8f40403
2c6e1c0
43e9f5e
842d2fc
593b45f
944eb78
757e279
79781f2
633033d
89dcae7
6501fd8
9546cb9
3ba64fc
9b6e6ec
0907459
6a51993
d47783e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,14 +3,13 @@ package client | |
import ( | ||
"encoding/json" | ||
"fmt" | ||
"github.com/Layr-Labs/eigenda/common" | ||
"github.com/Layr-Labs/eigensdk-go/logging" | ||
"github.com/stretchr/testify/require" | ||
"os" | ||
"os/exec" | ||
"sync" | ||
"testing" | ||
|
||
"github.com/Layr-Labs/eigenda/common" | ||
"github.com/Layr-Labs/eigensdk-go/logging" | ||
"github.com/stretchr/testify/require" | ||
) | ||
|
||
var ( | ||
|
@@ -24,6 +23,13 @@ var ( | |
|
||
const ( | ||
PreprodEnv = "../config/environment/preprod.json" | ||
|
||
//G1URL = "https://eigenda.s3.amazonaws.com/srs/g1.point" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nit: remove comments There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. deleted |
||
//G2URL = "https://eigenda.s3.amazonaws.com/srs/g2.point" | ||
//G2PowerOf2URL = "https://eigenda.s3.amazonaws.com/srs/g2.point.powerOf2" | ||
G1URL = "https://srs-mainnet.s3.amazonaws.com/kzg/g1.point" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should we make There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure this makes sense as a config value. The purpose of these is to download the SRS files on a developer laptop the first time they run the test suite. Once those files are present, these constants aren't used for anything. So in kubernetes where we are using a persistent volume, these files are ignored. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sounds good, I don't think this URL will change anyway. |
||
G2URL = "https://srs-mainnet.s3.amazonaws.com/kzg/g2.point" | ||
G2PowerOf2URL = "https://srs-mainnet.s3.amazonaws.com/kzg/g2.point.powerOf2" | ||
) | ||
|
||
// GetConfig returns a TestClientConfig instance parsed from the config file. | ||
|
@@ -183,17 +189,6 @@ func setupFilesystem(logger logging.Logger, config *TestClientConfig) error { | |
return fmt.Errorf("failed to create SRS tables directory: %w", err) | ||
} | ||
|
||
// Check to see if the private key file exists. If not, stop the test. | ||
filePath, err := ResolveTildeInPath(config.KeyPath) | ||
if err != nil { | ||
return fmt.Errorf("failed to resolve tilde in path: %w", err) | ||
} | ||
_, err = os.Stat(filePath) | ||
if err != nil { | ||
return fmt.Errorf("private key file %s does not exist. This file should "+ | ||
"contain the private key for the account used in the test, in hex: %w", filePath, err) | ||
} | ||
|
||
// If any of the srs files do not exist, download them. | ||
err = ensureFileIsPresent(config, SRSPathG1, G1URL) | ||
if err != nil { | ||
|
This file was deleted.
This file was deleted.
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.
nit: Does your IDE use a different import ordering scheme?
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.
I think it's just ordering them alphabetically. Is there a different schema we should be using?
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.
https://pkg.go.dev/golang.org/x/tools/cmd/goimports