Skip to content

Comments

ctb: Fix Holocene upgrade script#13015

Merged
mslipper merged 4 commits intodevelopfrom
seb/fix-holocene-upgrade-script
Nov 22, 2024
Merged

ctb: Fix Holocene upgrade script#13015
mslipper merged 4 commits intodevelopfrom
seb/fix-holocene-upgrade-script

Conversation

@sebastianst
Copy link
Member

@sebastianst sebastianst commented Nov 21, 2024

Description

A couple of fixes to the Holocene upgrade script.

  • bumped the contracts release to use to rc.2.
  • removed '' from sed -i '' invocations - this is a MacOS quirk, but doesn't work in a Linux Docker env.
  • deployment script is only run if deployments aren't present yet.
  • deploy-config.json was made a parameter to just run.
  • output folder is an optional second parameter to just run. Defaults to output/.
  • scripts are copied from the checked out repo, not the contracts revision of the monorepo, allowing to use newer scripts (and possibly modify them locally).
  • improved documentation in README.

@sebastianst sebastianst requested a review from clabby November 21, 2024 17:12
@sebastianst sebastianst marked this pull request as ready for review November 22, 2024 07:06
@sebastianst sebastianst requested a review from a team as a code owner November 22, 2024 07:06
@sebastianst sebastianst added this pull request to the merge queue Nov 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 22, 2024
@sebastianst sebastianst added this pull request to the merge queue Nov 22, 2024
github-merge-queue bot pushed a commit that referenced this pull request Nov 22, 2024
* ctb: Fix Holocene upgrade script

* improve README

* pass deploy-config as parameter to just run, other improvements

* remove commented-out variables
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 22, 2024
@mslipper
Copy link
Collaborator

The issue breaking the merge queue here has been fixed - should be good to merge once the above convo is resolved.

@mslipper mslipper added this pull request to the merge queue Nov 22, 2024
Merged via the queue into develop with commit 974ee6f Nov 22, 2024
@mslipper mslipper deleted the seb/fix-holocene-upgrade-script branch November 22, 2024 19:20
geoknee pushed a commit that referenced this pull request Nov 25, 2024
* ctb: Fix Holocene upgrade script

* improve README

* pass deploy-config as parameter to just run, other improvements

* remove commented-out variables
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.

5 participants