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

Support changeable configuration file location #36

Merged
2 commits merged into from
Aug 12, 2022

Conversation

ghost
Copy link

@ghost ghost commented Aug 12, 2022

Summary

Configuration file location is hardcoded in the codebase, which prevent users to change it's location to somewhere else.

Dockerfile work dir is configured as /opt/ops-tool/bin which makes us to define every folder under bin folder. Docker folder structure:

/opt/ops-tool/bin
          ops-tool
          config/
              config.yaml
          commands/
              command_config.yaml
          scripts/ 
              script.sh

We need to map or create other folders under /opt/ops-tool. For that docker workdir is changed.

Ticket Link

Fixes: #33

We are using many folders like config, commands and scripts.
Changing workdir to `/opt/ops-tool` will allow us to access those
folders relatively.

Signed-off-by: Mustafa Kara <[email protected]>
@ghost ghost self-requested a review as a code owner August 12, 2022 09:36
@ghost ghost linked an issue Aug 12, 2022 that may be closed by this pull request
@ghost ghost self-assigned this Aug 12, 2022
@ghost ghost requested review from phoinixgrr, spirosoik and JulienTant August 12, 2022 09:36
@ghost ghost added this to the 0.0.2 milestone Aug 12, 2022
@ghost ghost force-pushed the feat/33-modify-dockerfile-working-directory branch from 10c37b5 to 53d13c1 Compare August 12, 2022 09:38
Copy link
Contributor

@phoinixgrr phoinixgrr left a comment

Choose a reason for hiding this comment

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

🎖️ :shipit:

@ghost ghost merged commit 12df07f into main Aug 12, 2022
@ghost ghost deleted the feat/33-modify-dockerfile-working-directory branch August 12, 2022 10:53
This pull request was closed.
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.

Modify Dockerfile Working Directory
1 participant