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

chore: add Fix repository path #126

Merged
merged 2 commits into from
Jan 19, 2024
Merged

chore: add Fix repository path #126

merged 2 commits into from
Jan 19, 2024

Conversation

svienot
Copy link
Contributor

@svienot svienot commented Jan 18, 2024

Description:

  • add REPOSITORY_PATH_HOST .env variable (cherry pick commit from upstream ethereum/sourcify and use it in hedera specific files)
  • align repository mount point inside the server container on what is used in (production) K8S deployment (i.e. /data instead of /home/data/repository)
  • adjust network reset script accordingly to enable its use in production.

Related issue(s):

Fixes #125

kuzdogan and others added 2 commits January 18, 2024 22:31
(cherry picked from commit 97952a9986e273140a60faa3e8519f715002b405)
Signed-off-by: Simon Viénot <[email protected]>
…a as repository mount point in server container.

Signed-off-by: Simon Viénot <[email protected]>
@svienot svienot requested review from a team as code owners January 18, 2024 22:42
@svienot svienot requested review from ebadiere, Sheng-Long, acuarica and rustyShacklefurd and removed request for a team, ebadiere and Sheng-Long January 18, 2024 22:42
Copy link
Contributor

@acuarica acuarica left a comment

Choose a reason for hiding this comment

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

lgtm

@svienot svienot merged commit 2f2d562 into main Jan 19, 2024
8 checks passed
@svienot svienot deleted the fix-repository-path branch January 19, 2024 15:25
svienot added a commit that referenced this pull request Feb 14, 2024
Signed-off-by: Simon Viénot <[email protected]>
Co-authored-by: Kaan Uzdoğan <[email protected]>
(cherry picked from commit 2f2d562)
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.

Add REPOSITORY_PATH_HOST environment variable
3 participants