Skip to content

CI: fix DEBIAN_FRONTEND for arm apt-install step - #6044

Merged
kholia merged 1 commit into
openwall:bleeding-jumbofrom
kholia:fix-ci-arm-debian-frontend
Sep 30, 2026
Merged

kholia merged 1 commit into
openwall:bleeding-jumbofrom
kholia:fix-ci-arm-debian-frontend

Conversation

@kholia

@kholia kholia commented Sep 30, 2026

Copy link
Copy Markdown
Member

The environment: key was nested under machine: which CircleCI ignores for shell execution. Move it to the job level. Also, sudo drops the caller's environment by default, so inline DEBIAN_FRONTEND=noninteractive directly in each sudo apt invocation so the service-restart prompt never appears.

This should fix the CI related problems seen in #6029 and other PRs.

The environment: key was nested under machine: which CircleCI ignores
for shell execution. Move it to the job level. Also, sudo drops the
caller's environment by default, so inline DEBIAN_FRONTEND=noninteractive
directly in each sudo apt invocation so the service-restart prompt never
appears.
@kholia
kholia requested review from solardiz and a balanced review from Copilot September 30, 2026 09:19
@kholia kholia self-assigned this Sep 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The configuration correctly prevents interactive APT prompts in the ARM CI job.

Review effort: Balanced
Findings: None

What changed in this PR

Corrects the ARM CircleCI job’s noninteractive package installation setup.

Changes:

  • Moves DEBIAN_FRONTEND to the job-level environment.
  • Preserves it explicitly across sudo for APT commands.
File Description
.circleci/​config.yml Fixes ARM job environment and APT invocation.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@kholia

kholia commented Sep 30, 2026

Copy link
Copy Markdown
Member Author

I will wait for all the CI(s) to pass and then merge this PR.

@kholia
kholia merged commit 302f527 into openwall:bleeding-jumbo Sep 30, 2026
34 of 35 checks 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.

3 participants