contracts: gas oracle script

[tynes]

Description

Create a simple script to set the scalar
in the OVM_GasPriceOracle using cast
as using the hardhat task requires 4gb+
of memory when running.

[changeset-bot]

🦋 Changeset detected

Latest commit: 6863a69

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages

Name	Type
@eth-optimism/contracts	Patch
@eth-optimism/data-transport-layer	Patch
@eth-optimism/sdk	Patch
@eth-optimism/message-relayer	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[mslipper]

What do you think of removing the ability to update the scalar from the other script? Can have it raise an error if the scalar is passed in directing users to use this instead.

[tynes]

What do you think of removing the ability to update the scalar from the other script? Can have it raise an error if the scalar is passed in directing users to use this instead.

I don't see much of a need to brick the usage of the other script, there are no plans to alter the OVM_GasPriceOracle so there is no risk that it becomes stale, the code will all be deleted post bedrock

[tynes]

I'll update the functionality so its the same as the hardhat task

[mslipper]

I don't see much of a need to brick the usage of the other script, there are no plans to alter the OVM_GasPriceOracle so there is no risk that it becomes stale, the code will all be deleted post bedrock

The idea is to make it clear which script is canonical, so we don't have to maintain two ways of doing the same thing.

[tynes]

I don't see much of a need to brick the usage of the other script, there are no plans to alter the OVM_GasPriceOracle so there is no risk that it becomes stale, the code will all be deleted post bedrock

The idea is to make it clear which script is canonical, so we don't have to maintain two ways of doing the same thing.

I've deleted the old script now

[tynes]

Now the script can be used in "view only" mode by not passing a key

[tynes]

I don't see much of a need to brick the usage of the other script, there are no plans to alter the OVM_GasPriceOracle so there is no risk that it becomes stale, the code will all be deleted post bedrock

The idea is to make it clear which script is canonical, so we don't have to maintain two ways of doing the same thing.

Now that the script is 1:1 with functionality of the other script, I have deleted the other script

[mslipper]

@mergify refresh

[mergify]

refresh

✅ Pull request refreshed

Hey, I reacted but my real name is @Mergifyio

[mergify]

This PR has been added to the merge queue, and will be merged soon.

[mergify]

This PR is next in line to be merged, and will be merged as soon as checks pass.