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

feat: Introduction of DDE_DATA_HOME and Path Adjustments #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sbaerlocher
Copy link
Contributor

@sbaerlocher sbaerlocher commented Jan 26, 2024

Added

  • New Script createDataHome.sh:

    • Introduced a new _createDataHome function for creating the DDE_DATA_HOME directory and migrating data from DATA_DIR.
    • Setup of configuration files in the DDE_DATA_HOME.
  • Transition from DATA_DIR to DDE_DATA_HOME in checkUpdate.sh:

    • Adjusted the script to use DDE_DATA_HOME directory instead of DATA_DIR for update checks.
    • No functional changes to script behavior.

Modified

  • Updates in destroy.sh and up.sh:

    • Changed from CERT_DIR to DDE_CERT_PATH for handling SSL certificates.
    • Adjusted path variables without changing the functionality.
  • Environment Variable Updates in system Directory (install.sh, env.sh, up.sh):

    • Included DDE_DATA_HOME and DDE_CERT_PATH environment variables, marking DATA_DIR and CERT_DIR as deprecated.
    • Integrated _createDataHome into both install.sh and update.sh.

Removed

  • Empty File data/.gitkeep:
    • Deleted the empty placeholder file, as it's no longer needed.

Refactored

  • dde.sh and docker-compose.yml Adjustments:
    • Implemented new environment variables DDE_DATA_HOME and DDE_CERT_PATH.
    • Updated volume paths in docker-compose.yml to align with new directory structures.

Related Issue

@robinlehrmann
Copy link
Contributor

Hello @sbaerlocher I think you need to check the nuke command in this file as well. :)

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