Skip to content

refactor(ansible): delete .env files and update README references#6962

Merged
mitsudome-r merged 1 commit intomainfrom
refactor/delete-env-files
Apr 1, 2026
Merged

refactor(ansible): delete .env files and update README references#6962
mitsudome-r merged 1 commit intomainfrom
refactor/delete-env-files

Conversation

@xmfcx
Copy link
Copy Markdown
Contributor

@xmfcx xmfcx commented Mar 28, 2026

Why

This is the final step of the .env file removal tracked in #6938. All consumers were already migrated in prior PRs:

With every consumer rewired, the .env files are dead code. Deleting them completes the migration to ansible role defaults as the single source of truth for version pins.


Test plan

  • CI passes -- no workflow, script, or Dockerfile references the deleted files
  • Grep confirms no remaining references: git grep -E 'amd64\.env|amd64_jazzy\.env|arm64\.env' returns nothing
  • setup-dev-env.sh --ros-distro humble still runs (uses --extra-vars rosdistro=... only, no .env sourcing)
  • docker/build.sh --ros-distro humble still runs (uses derive_images(), no .env sourcing)

@xmfcx xmfcx added the type:build Tooling and infrastructure around building the Autoware. label Mar 28, 2026
@xmfcx xmfcx requested a review from mitsudome-r as a code owner March 28, 2026 12:17
@xmfcx xmfcx self-assigned this Mar 28, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 28, 2026

Thank you for contributing to the Autoware project!

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

Please ensure:

Final step of #6938. All consumers were removed in prior PRs (#6947,
#6949, #6950, #6952, #6961). This deletes the now-unused .env files
and updates ansible role READMEs to extract versions directly from
defaults/main.yaml instead of sourcing the deleted files.

Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
@xmfcx xmfcx force-pushed the refactor/delete-env-files branch from f11a6b6 to 14a12b3 Compare March 29, 2026 19:27
Copy link
Copy Markdown
Contributor

@oguzkaganozt oguzkaganozt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mitsudome-r mitsudome-r merged commit 5dfb090 into main Apr 1, 2026
33 of 34 checks passed
@mitsudome-r mitsudome-r deleted the refactor/delete-env-files branch April 1, 2026 14:04
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 type:build Tooling and infrastructure around building the Autoware.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants