Repository navigation
PennyLane Backline: Add qec-backline-head node and --qpu=backline-local for the Backline QPM - #64
Merged
Merged
Conversation
Adds a backline-head node, the backline-local service and --qpu mapping, device and user entries, a backline environment module and a script that creates the shared Backline venv. qfw-site-services gains an opt-in backline target; 'all' is unchanged except that stop also stops a running Backline QPM.
DougSO
reviewed
Oct 22, 2026
DougSO
left a comment
Collaborator
There was a problem hiding this comment.
Thanks for this. The wiring is consistent: the new mapping is unique, plugin.conf and resources.lua agree, and it merges cleanly with #63 and #67. Keeping backline out of all is the right call for sites without Catalyst. As you say, it needs QFw#159 and #63 merged first.
Couple of notes:
docs/backline.mdcovers a fresh cluster only. On an existing one, three things bite after a pull.modulefiles/backlinereaches the containers only through the image (COPY modulefiles /etc/modulefiles), so it needs a rebuild. Theetc_slurmvolume keeps its oldslurm.conf, so slurmd onqec-backline-headwon't find its NodeName until./update_slurmfiles.sh slurm.conf. Andtools/provision-qfw-cluster.shnow listsqec-backline-head, so provisioning stops with "container is not available" until that container is up. A short "existing cluster" section with those steps would save the next person some digging.tools/install-backline-venv.shclonesjosephleekl/qri-backlineat whatever its default branch holds, and jobs then run that code. Could it check out a fixed commit or tag until the demo moves under openQSE?
Smaller points:
- The run section follows a root shell from
./do_ssh.sh. We've seen root batch jobs take slurmd down on this cluster before, so showing thesbatchas a provisioned test user (user-a, accountqfw-test) would be safer. - As noted on QFw#159, the executor accepts code from anyone who can reach it, and here it runs as root on
qec-backline-head. A line indocs/backline.mdsaying so would help until that's addressed after the SC26 demo.
…t, run demo as user-a, executor security note
Contributor
Author
|
Thanks @DougSO - 24355f7 addressed your comments:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds the cluster side of the Backline integration. It deploys the Backline QPM from openQSE/QFw#159 on a new
qec-backline-headnode and exposes it to Slurm jobs as--qpu=backline-local. It also adds the shared Catalyst/PennyLane environment, service controls and setup docs needed to run a Backline QEC program on the cluster.Merge order and external references
docs/backline.mdbuilds the QFw override from openQSE/QFwmain, and the cluster clones openQSE/QFw-SLURM-Clustermain, so the setup only works once both are merged. Until then, use thebackline-qpmbranches of josephleekl/QFw and josephleekl/QFw-SLURM-Cluster.tools/install-backline-venv.shclones the demo from josephleekl/qri-backline (personal repo; there is no openQSE copy). Change the URL if the demo moves under openQSE (QRI_BACKLINE_DIRalready overrides the checkout path).