Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

fix: update local-topos docker setup #9

Merged
merged 10 commits into from
Mar 24, 2023

Conversation

sebastiendan
Copy link
Member

@sebastiendan sebastiendan commented Mar 23, 2023

Description

A few things are wrongly set in docker/local-topos, this PR fixes them.

Additions and Changes

Bug fix (non-breaking change which fixes an issue)

  • Replace data-dir=data- with data-dir=/data/data- with fit latest code from docker/local
  • Replace GENESIS_PATH from /genesis/genesis.json with /data/genesis.json
  • Replace COPY docker/local/polygon-edge.sh with COPY docker/local-topos/polygon-edge.sh
  • Make polygon-edge.sh executable
  • Move the pre-deployment of ConstAddressDeployer in polygon-edge.sh so that it is actually executed
  • Make data folder's content readable (755) so that the sequencer can read it

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

@sebastiendan sebastiendan changed the title fix: update wrong data-dir in topos docker fix: update wrong paths in local-topos docker setup Mar 23, 2023
@sebastiendan sebastiendan changed the title fix: update wrong paths in local-topos docker setup fix: update local-topos docker setup Mar 23, 2023
@sebastiendan sebastiendan force-pushed the fix/wrong-data-dir-docker branch from cebce72 to daeb89f Compare March 24, 2023 10:10
@sebastiendan sebastiendan marked this pull request as ready for review March 24, 2023 10:44
@sebastiendan sebastiendan requested a review from a team as a code owner March 24, 2023 10:44
@sebastiendan sebastiendan self-assigned this Mar 24, 2023
@sebastiendan sebastiendan merged commit 6221afd into develop Mar 24, 2023
@sebastiendan sebastiendan deleted the fix/wrong-data-dir-docker branch March 24, 2023 10:47
atanmarko pushed a commit that referenced this pull request Aug 18, 2023
atanmarko pushed a commit that referenced this pull request Sep 7, 2023
atanmarko pushed a commit that referenced this pull request Sep 15, 2023
atanmarko pushed a commit that referenced this pull request Oct 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants