Context
.github/workflows/dt4acc-integration.yml, step "Install pyAML and control-system backends", installs the two companion control-system packages from fixed git refs instead of PyPI:
python -m pip install --no-deps
"tango-pyaml @ git+https://github.com/python-accelerator-middle-layer/tango-pyaml.git@1f010735bd675ad2f918958abd0396cb6766f14e"
"pyaml-cs-oa @ git+https://github.com/python-accelerator-middle-layer/pyaml-cs-oa.git@0.2.0"
- tango-pyaml is stuck on a specific commit hash.
- pyaml-cs-oa is stuck on git tag 0.2.0, fetched via git+https://... rather than PyPI.
Problem
These pins go stale: every time a new release of tango-pyaml or pyaml-cs-oa is published, the dt4acc integration pipeline keeps testing against the old, frozen ref until someone remembers to manually bump the commit/tag in the workflow. This defeats the purpose of the smoke test, which is meant to catch incompatibilities between pyaml and the current published companion packages — not a snapshot from whenever the pin was last touched.
Expected behavior
The workflow always installs the latest published PyPI release of tango-pyaml and pyaml-cs-oa, with no fixed version/commit pin, so the integration test naturally stays current as new releases come out.
Proposal
Replace the --no-deps git install with an unpinned PyPI install:
python -m pip install --no-deps tango-pyaml pyaml-cs-oa
(keep --no-deps if it's intentional here to avoid pulling unwanted transitive deps into this job; otherwise reconsider it too).
Acceptance criteria
Context
.github/workflows/dt4acc-integration.yml, step "Install pyAML and control-system backends", installs the two companion control-system packages from fixed git refs instead of PyPI:
python -m pip install --no-deps
"tango-pyaml @ git+https://github.com/python-accelerator-middle-layer/tango-pyaml.git@1f010735bd675ad2f918958abd0396cb6766f14e"
"pyaml-cs-oa @ git+https://github.com/python-accelerator-middle-layer/pyaml-cs-oa.git@0.2.0"
Problem
These pins go stale: every time a new release of tango-pyaml or pyaml-cs-oa is published, the dt4acc integration pipeline keeps testing against the old, frozen ref until someone remembers to manually bump the commit/tag in the workflow. This defeats the purpose of the smoke test, which is meant to catch incompatibilities between pyaml and the current published companion packages — not a snapshot from whenever the pin was last touched.
Expected behavior
The workflow always installs the latest published PyPI release of tango-pyaml and pyaml-cs-oa, with no fixed version/commit pin, so the integration test naturally stays current as new releases come out.
Proposal
Replace the --no-deps git install with an unpinned PyPI install:
python -m pip install --no-deps tango-pyaml pyaml-cs-oa
(keep --no-deps if it's intentional here to avoid pulling unwanted transitive deps into this job; otherwise reconsider it too).
Acceptance criteria