Skip to content

Use Docker format for containers - #992

Merged
giacomofiorin merged 26 commits into
masterfrom
use-docker
Nov 9, 2026
Merged

giacomofiorin merged 26 commits into
masterfrom
use-docker

Conversation

@giacomofiorin

@giacomofiorin giacomofiorin commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

Summary

This PR replaces Apptainer-format containers with Docker-format ones for use in CI jobs. The change is breaking for existing branches because they pull containers expecting them in Apptainer format. However, one can still use the containers with Apptainer by specifying that they are in Docker format.

The container images are now compatible with all major container software (Docker, Podman, Apptainer), and updated instructions for each are in the README.

Also included some small CI fixes and improvements:

  • Fixed the cache keys in a couple of jobs
  • Reduced the maximum ccache size per job, allowing more caches for more branches within the repo's allotted quota.
  • Switched the order of testing in backends CI jobs (interface tests now run first, because they take less and are arguably less redundant than the stand-alone library tests)
  • The website is now updated as part of the Version Maintenance job, i.e. it will only be updated after merging to master

Related issue(s)

Installing Apptainer from PPA sporadically fails, and it adds some extra time at startup. Using Docker is better integrated with GHA, but we can still use Apptainer offline.

Affected engine(s)

Author(s)

@giacomofiorin

Licensing

By submitting this pull request, I agree that my contribution will be included in Colvars repository and redistributed under the GNU Lesser General Public License (LGPL) version 3.

Note: NAMD, VMD and TinkerHP follow more restrictive licenses, e.g. the UIUC license. However, the terms of the LGPLv3 remain in effect for the copies of the Colvars code distributed alongside those packages.

Artificial Intelligence (AI)

Copilot converted the Apptainer definition files to Dockerfiles; it also drafted the fix for MathJax code, which I reviewed very quickly.

Further information / links

@giacomofiorin giacomofiorin changed the title Use docker Use Docker format for containers Nov 5, 2026
@giacomofiorin
giacomofiorin force-pushed the use-docker branch 8 times, most recently from 9eee0f8 to 2700066 Compare November 6, 2026 15:15
@HanatoK
HanatoK added this pull request to stack #994 November 6, 2026 21:25
@giacomofiorin
giacomofiorin removed this pull request from stack #994 November 6, 2026 21:37
@giacomofiorin
giacomofiorin force-pushed the use-docker branch 3 times, most recently from 11164a8 to 516e445 Compare November 9, 2026 17:08
@giacomofiorin
giacomofiorin requested a balanced review from Copilot November 9, 2026 18:14

This comment was marked as outdated.

@giacomofiorin
giacomofiorin requested a balanced review from Copilot November 9, 2026 19:17
@giacomofiorin giacomofiorin added testing Only affects CI; not listed in outside PRs maintenance No user-visible effects labels Nov 9, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comment thread .github/workflows/backend-template.yml Resolved
Comment thread .github/workflows/test-library.yml Resolved
giacomofiorin and others added 2 commits November 9, 2026 14:26
[skip ci]

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@giacomofiorin
giacomofiorin merged commit 4f8b3f2 into master Nov 9, 2026
23 checks passed
@giacomofiorin
giacomofiorin deleted the use-docker branch November 9, 2026 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance No user-visible effects testing Only affects CI; not listed in outside PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants