Skip to content

ocl 0.5.0: load-test an export on a real OpenMRS database before putting it into pihemr - #9

Merged
mogoodrich merged 1 commit into
mainfrom
ocl-skill-updates
Nov 13, 2026
Merged

mogoodrich merged 1 commit into
mainfrom
ocl-skill-updates

Conversation

@mogoodrich

Copy link
Copy Markdown
Member

Adds a load test to workflow 6 (putting a released version's export into a PIH distro), to go with PIH/pihemr#783.

Changes

  • SKILL.md, workflow 6, new step 3: before creating any branch, run pihemr's scripts/ocl-load-test.sh on the new export. Claude:

    • asks first (about 10–20 minutes),
    • works in a throwaway detached worktree from origin/master, so the user's pihemr checkout isn't touched,
    • stops if the export doesn't load (exit 1), showing the failing items, or asks whether to continue if the test couldn't run (exit 2),
    • leaves removing the test instance to the user (openmrs-docker <instance> destroy asks for confirmation), and says that the test installed a pihemr snapshot into ~/.m2.

    If origin/master doesn't have the script yet (before Load-test new OCL exports on a real OpenMRS database pihemr#783 merges), it says the load test isn't available and carries on. Steps 3–5 are renumbered 4–6.

  • SKILL.md: the Update OCL export Action note mentions the new Skip the load test checkbox.

  • README: a paragraph on the load test, next to the release-check one.

  • Version 0.5.0.

Testing

The script itself was tested locally with a good export (passed) and a deliberately broken one (failed, naming the concept). Details are in PIH/pihemr#783. The skill wording hasn't been exercised end to end yet, since that needs a new release to put into the distro.

🤖 Generated with Claude Code

… into pihemr

Workflow 6 gets a new step 3: before creating any branch, run pihemr's
scripts/ocl-load-test.sh (PIH/pihemr#783) in a throwaway worktree from master.
It starts the ZL EMR with the new export on a copy of the nightly ZL seed
database and checks the OCL import finished without errors. Asks first
(about 10-20 minutes), stops if the export doesn't load, and leaves removing
the test instance to the user.

Also documents the Action's new "Skip the load test" checkbox, and bumps the
plugin to 0.5.0.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mogoodrich
mogoodrich merged commit 430eb11 into main Nov 13, 2026
1 check passed
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.

1 participant