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

Dockerfile: allow specifying s3gw directory #804

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

jecluis
Copy link
Contributor

@jecluis jecluis commented Nov 17, 2023

When requiring the copy of the entrypoint.sh script, we did not account for actions on other repositories relying on checking out the s3gw.git repository and building images using the Dockerfile from within an s3gw directory. This causes the script to not be found, because it's instead in s3gw/tools/entrypoint.sh, versus what we expected, tools/entrypoint.sh.

This helps addressing that by allowing the caller to specify on which directory we can find the s3gw repository.

Signed-off-by: Joao Eduardo Luis [email protected]

@jecluis jecluis added kind/bug Something isn't working area/CI Continuous Integration labels Nov 17, 2023
@jecluis jecluis added this to the v0.23.0 milestone Nov 17, 2023
@jecluis jecluis requested a review from m-ildefons November 17, 2023 14:31
@jecluis jecluis self-assigned this Nov 17, 2023
When requiring the copy of the `entrypoint.sh` script, we did not
account for actions on other repositories relying on checking out the
`s3gw.git` repository and building images using the Dockerfile from
within an `s3gw` directory. This causes the script to not be found,
because it's instead in `s3gw/tools/entrypoint.sh`, versus what we
expected, `tools/entrypoint.sh`.

This helps addressing that by allowing the caller to specify on which
directory we can find the s3gw repository.

Signed-off-by: Joao Eduardo Luis <[email protected]>
Copy link
Contributor

@m-ildefons m-ildefons left a comment

Choose a reason for hiding this comment

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

Approved.

@jecluis jecluis merged commit e73f70d into s3gw-tech:main Nov 17, 2023
2 checks passed
@jecluis jecluis deleted the wip-fix-dockerfile branch November 17, 2023 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration kind/bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants