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

add kiln instructions (do not merge WIP) #3590

Merged
merged 3 commits into from
Apr 18, 2022
Merged

add kiln instructions (do not merge WIP) #3590

merged 3 commits into from
Apr 18, 2022

Conversation

unixpi
Copy link
Contributor

@unixpi unixpi commented Apr 13, 2022

No description provided.

@unixpi unixpi requested a review from tersec April 13, 2022 10:56
@@ -0,0 +1,120 @@
# Run Kiln

Kiln is the latest long-running merge testnet (the previous one was known as Kintsugi).
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure how useful it is to mention Kintsugi -- people who've been following this presumably know, and for those who haven't, it's just one more name to remember, introduced in the beginning and never revisited or otherwise used. My impression is that this isn't meant as primarily a historical document, but rather one aimed at current circumstances.

Also, Kintsugi isn't even available really to test anymore, so there's not any chance of confusion.


Kiln is the latest long-running merge testnet (the previous one was known as Kintsugi).

Kiln is the perfect opportunity to verify your setup works as expected through the proof-of-stake transition and in a post-merge context. If you come across any issues, please [report them here](https://github.com/eth-clients/merge-testnets/).
Copy link
Contributor

Choose a reason for hiding this comment

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

That GitHub URL is for the EF side of things, basically -- presumably it'd be better for Nimbus if they reported them to the Nimbus issue tracker.


```
git clone https://github.com/eth-clients/merge-testnets.git
cd merge-testnets/kiln
Copy link
Contributor

Choose a reason for hiding this comment

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

There's a question of how much to mirror the Kiln launchpad instructions, which this is obviously doing, and reasonably so.

Mostly, it's fine, and they're good instructions, but I'm not sure that their methods of using merge-testnets/kiln as the current working directory is ideal. It kind of pollutes the git repo.

It might be better to adjust the instructions later to refer to merge-testnets/kiln/genesis.json or similar in various places where they currently assume the current working directory.

Or, maybe not -- there's an argument for keeping this as parallel and interchangeable with the Launchpad instructions as feasible, in which case, well.


#### 2.1G Clone and build

Clone and build the `merge-kiln-v2` branch from Marius' fork of Geth:
Copy link
Contributor

Choose a reason for hiding this comment

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

I know that's what the launchpad instructions say, and this is currently mostly a copy of those, but https://github.com/MariusVanDerWijden/go-ethereum/commits/merge-kiln-v2 was abandoned a month ago, and now one can use the master Geth repo branch with the --kiln instructions.

Copy link
Contributor

Choose a reason for hiding this comment

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

The other advantage is that it'd mean that the Geth instructions wouldn't be dependent on the merge-testnets/kiln location, because it'd be using its own internal copy of that.

- [Ethereum On Arm Kiln RP4 image](https://ethereum-on-arm-documentation.readthedocs.io/en/latest/kiln/kiln-testnet.html): Run Nimbus on a raspberry pi or using an AWS AMI

<br/>

Copy link
Contributor

Choose a reason for hiding this comment

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

A bunch of empty lines at the end?

@unixpi unixpi merged commit c8260a4 into unstable Apr 18, 2022
@unixpi unixpi deleted the kiln-instructions branch April 18, 2022 10:23
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.

2 participants