Skip to content

CI: isolate LTS container packages - #7781

Open
Stardust0831 wants to merge 1 commit into
deepmodeling:LTSfrom
Stardust0831:fix/lts-container-package-names
Open

CI: isolate LTS container packages#7781
Stardust0831 wants to merge 1 commit into
deepmodeling:LTSfrom
Stardust0831:fix/lts-container-package-names

Conversation

@Stardust0831

Copy link
Copy Markdown
Collaborator

Reminder

  • Have you linked an issue with this pull request?
  • Have you added adequate unit tests and/or case tests for your pull request?
  • Have you noticed possible changes of behavior below or in the linked issue?
  • Have you explained the changes of codes in core modules of ESolver, HSolver, ElecState, Hamilt, Operator or Psi? (not applicable)

Linked Issue

Fix #7780

Unit Tests and/or Case Tests for my changes

  • ruby -e 'require "yaml"; YAML.parse_file(".github/workflows/devcontainer.yml")' -- passed.
  • A focused assertion verified that all three registry publish/cache references use the -lts suffix -- passed (3/3 isolated).
  • git diff --check -- passed.
  • A live container publication was not run locally because it requires the upstream X64 runner and registry credentials.

What's changed?

  • Publish LTS images as abacus-{gnu,intel,cuda}-lts in both GHCR and the AliCloud registry.
  • Read the BuildKit cache from the corresponding abacus-*-lts:latest package.
  • Preserve the existing unsuffixed abacus-*:latest names for develop and existing consumers, avoiding a breaking migration of legacy workflows and user scripts.
  • Keep LTS CI consumers on their current image references for now; the new packages do not exist until this PR is merged and the LTS Container workflow runs.

This prevents future LTS pushes from overwriting the develop images. The first LTS build may be a cold build because the suffixed cache packages do not exist yet. After this PR is merged, the develop Container workflow still needs to be rerun once to restore the currently overwritten unsuffixed latest images.

Any changes of core modules? (ignore if not applicable)

  • None. This PR only changes the LTS container publication destinations.

@Growl1234

Copy link
Copy Markdown

Testing workflows also need modification, otherwise they will use containers from develop branch in the future if a PR to LTS branch is submitted. But this will definitely break the current CI workflow in this PR.

@Stardust0831

Stardust0831 commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator Author

Agreed. The LTS test workflows should move to abacus-*-lts in a follow-up PR after #7781 creates those packages; switching them in this bootstrap PR would prevent its container jobs from starting. The unsuffixed abacus-*:latest names will remain develop-owned for backward compatibility.

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