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

adjust script path names in the Dockerfile so that no additional options are required to build the docker image #30

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

ustorbeck
Copy link
Contributor

Building the docker image fails for me (on rocky linux 8.9). This PR adjusts the script path names in the docker file so that the image build succeeds.

@ustorbeck
Copy link
Contributor Author

I see the CI is using custom options to build the docker image. Perhaps these options should be documented in the README file (or the script path names should be adjusted so that additional options are not required).

@ustorbeck ustorbeck force-pushed the docker_path_names branch from 10241f0 to ee927b7 Compare July 4, 2024 21:18
No additional options are required to build the docker image.
@ustorbeck ustorbeck force-pushed the docker_path_names branch from ee927b7 to f62fa00 Compare July 4, 2024 21:33
@ustorbeck
Copy link
Contributor Author

I've updated the PR to also adjust the CI command so that everything fits together again.

@ustorbeck ustorbeck changed the title fix path names in Dockerfile adjust script path names in the Dockerfile so that no additional options are required to build the docker image Jul 4, 2024
Copy link
Collaborator

@scuzzilla scuzzilla left a comment

Choose a reason for hiding this comment

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

@ustorbeck thanks for the optimization

@scuzzilla scuzzilla merged commit c493058 into network-analytics:main Jul 5, 2024
1 check failed
@ustorbeck ustorbeck deleted the docker_path_names branch July 5, 2024 20:40
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