-
Notifications
You must be signed in to change notification settings - Fork 2
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
chore(e2e): run babylond in docker #52
Changes from 6 commits
5043070
afbff88
01b8768
4d4f9e3
89d4d1a
32ca015
5dde9d7
1234dee
c588cc3
cc084c7
df8784e
8e24435
ce91fe1
29dfef7
97a5357
b8b1675
34b22f5
1f87ba3
3457e68
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 |
---|---|---|
|
@@ -5,19 +5,24 @@ package container | |
type ImageConfig struct { | ||
BitcoindRepository string | ||
BitcoindVersion string | ||
BabylonRepository string | ||
BabylonVersion string | ||
} | ||
|
||
//nolint:deadcode | ||
const ( | ||
dockerBitcoindRepository = "lncm/bitcoind" | ||
dockerBitcoindVersionTag = "v27.0" | ||
dockerBabylondRepository = "babylonlabs/babylond" | ||
dockerBabylondVersionTag = "8e0222804ed19b18d74d599b80baa18f05e87d8a" // this is built from commit b1e255a | ||
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. so this should be manually updated to be consistent with the version in 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. It should be a tag, like v0.9.3, but our CI wasn't setup at the time to build for arm64. So yes, it should reflect go mod version |
||
) | ||
|
||
// NewImageConfig returns ImageConfig needed for running e2e test. | ||
func NewImageConfig() ImageConfig { | ||
config := ImageConfig{ | ||
return ImageConfig{ | ||
BitcoindRepository: dockerBitcoindRepository, | ||
BitcoindVersion: dockerBitcoindVersionTag, | ||
BabylonRepository: dockerBabylondRepository, | ||
BabylonVersion: dockerBabylondVersionTag, | ||
} | ||
return config | ||
} |
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.
ensure we fail the test as soon as one of them fails, no need to waste CI