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

Addressing rocket-pool/smartnode#601 on v2 branch #617

Conversation

SolezOfScience
Copy link
Contributor

Porting the changes made as part of rocket-pool/smartnode-install#135 to the v2 branch.

@jshufro
Copy link
Contributor

jshufro commented Aug 27, 2024

I think you accidentally included an extra commit in this PR. It should just be the mev-boost issue, but has a commit intended to implement partial wallet recovery.

@SolezOfScience
Copy link
Contributor Author

You're right - not sure how that happened. I'll fix this.

@SolezOfScience SolezOfScience force-pushed the feature/v2/feature/smartnode-issues-601 branch from 5085b10 to 0ef750f Compare August 31, 2024 00:12
@@ -19,6 +19,7 @@ services:
- NETWORK={{.Network}}
- MEV_BOOST_PORT={{.MevBoost.Port}}
- MEV_BOOST_RELAYS={{.MevBoost.GetRelayString}}
- MEV_BOOST_ADDITIONAL_FLAGS={{.MevBoost.AdditionalFlags}}
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't see this being added to the config struct in this pr. I'm assuming it was already cherry-picked in to v2?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it's already being passed, unless I'm misunderstanding. Code Link

Also looks like it gets passed here. I also checked the UI on my node and confirmed the parameter box is present but it won't be used without this change.

@0xfornax 0xfornax merged commit 9a2d682 into rocket-pool:v2 Sep 24, 2024
5 checks passed
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

Successfully merging this pull request may close these issues.

3 participants