Skip to content

refactor(ansible): mark legacy entrypoints for removal#7059

Merged
xmfcx merged 1 commit intomainfrom
refactor/mark-deletion
Apr 24, 2026
Merged

refactor(ansible): mark legacy entrypoints for removal#7059
xmfcx merged 1 commit intomainfrom
refactor/mark-deletion

Conversation

@xmfcx
Copy link
Copy Markdown
Contributor

@xmfcx xmfcx commented Apr 24, 2026

Why

Phase 1 of the ansible entrypoint consolidation in #7052: give callers a migration window before Phase 2 deletes these files together.


Test plan

  • Banner fires: bash setup-dev-env.sh --help 2>&1 >/dev/null | head -3 shows 3 yellow [DEPRECATED] lines.
    image
  • Marked playbook still parses: ansible-playbook --syntax-check ansible/playbooks/universe.yaml.

Phase 1 deprecation-marker step for the ansible entrypoint consolidation
tracked in #7052.
Every file below remains fully functional; Phase 2 (no earlier than
2026-05-24) deletes them together.

- setup-dev-env.sh: adds a DEPRECATED header comment and prints a yellow
  warning banner to stderr at every invocation.
- ansible/playbooks/universe.yaml, openadkit.yaml: superseded by
  install_dev_env.yaml.
- ansible/playbooks/role_rmw_implementation.yaml: duplicate of
  install_rmw.yaml (was rmw.yaml).
- ansible/playbooks/setup_acados.yaml, download_artifacts.yaml,
  install_rviz_theme.yaml, install_spconv.yaml: replace with
  install_dev_env.yaml + --tags.
- ansible/playbooks/telegraf.yaml: no known callers.

Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
@github-actions
Copy link
Copy Markdown

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@xmfcx xmfcx added the run:health-check Run health-check label Apr 24, 2026
@xmfcx xmfcx merged commit ffd9034 into main Apr 24, 2026
35 of 38 checks passed
@xmfcx xmfcx deleted the refactor/mark-deletion branch April 24, 2026 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run:health-check Run health-check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants