-
Notifications
You must be signed in to change notification settings - Fork 9
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
Added support for creating linux based docker image #908
base: main
Are you sure you want to change the base?
Conversation
Wiz Scan Summary
To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension. |
@@ -15,7 +15,7 @@ on: | |||
|
|||
env: | |||
DOCKER_IMAGE_NAME: ghcr.io/ansys/prime | |||
DOCKER_IMAGE_TAG: '25.1.0.dev19' | |||
DOCKER_IMAGE_TAG: '25.1.0-test' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rchopade7 I would expect this not to be changed for cicd.
# Build the docker image | ||
print(">>> Building docker image. This might take some time...") | ||
out = subprocess.run( | ||
["docker", "build", "-f", "linux/Dockerfile", "-t", "ghcr.io/ansys/prime:25.1.0-test", "."], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
["docker", "build", "-f", "linux/Dockerfile", "-t", "ghcr.io/ansys/prime:25.1.0-test", "."], | |
["docker", "build", "-f", "linux/Dockerfile", "-t", "ghcr.io/ansys/prime:latest", "."], |
# copy parasolid schema | ||
shutil.copytree( | ||
os.path.join( | ||
AWP_ROOT, "commonfiles", "CAD", "Siemens", "Parasolid36.1.212", "linx64", "schema" | ||
), | ||
os.path.join( | ||
dest_package_path, | ||
"commonfiles", | ||
"CAD", | ||
"Siemens", | ||
"Parasolid36.1.212", | ||
"linx64", | ||
"schema", | ||
), | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rchopade7 this looks version specific, can we consider modifying so this will work between all versions going forward?
Added support for creating linux based docker image