Skip to content

Conversation

@xamberl
Copy link
Contributor

@xamberl xamberl commented Nov 14, 2024

also added a section for building on local apptainer images

@xamberl
Copy link
Contributor Author

xamberl commented Nov 14, 2024

Copy link
Contributor

@aowen-uwmad aowen-uwmad left a comment

Choose a reason for hiding this comment

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

Some suggestions, good to merge when you are ready.

Copy link
Contributor

@ChristinaLK ChristinaLK left a comment

Choose a reason for hiding this comment

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

lgtm. 1-2 things to check.

```

> If you just want to convert an existing Docker container into an Apptainer container, you do not need to use a definition file.
> Instead, you can directly run the `apptainer build` command using the Docker address, as described [below](#converting-a-docker-image-to-an-apptainer-container-image).
Copy link
Contributor

Choose a reason for hiding this comment

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

are these instructions still below? or on the new page?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I left the instructions on this page

@xamberl xamberl merged commit 5af4cb4 into master Nov 15, 2024
2 of 3 checks passed
@xamberl xamberl deleted the preview-docker-to-apptainer branch November 29, 2024 17:36
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.

4 participants