Skip to content

fix: preserve apt temporary directory permissions - #73

Merged
fqjony merged 4 commits into
latestfrom
fix/apt-update-temp-permissions
Jul 22, 2026
Merged

fix: preserve apt temporary directory permissions#73
fqjony merged 4 commits into
latestfrom
fix/apt-update-temp-permissions

Conversation

@fqjony

@fqjony fqjony commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Preserve /tmp as the standard root-owned sticky directory so apt can verify repository signatures.
  • Avoid recursively changing permissions on application and log files.
  • Upgrade the worker base image from 0.45.0 to 0.47.0.

Validation

  • Rebuilt worker-php locally and verified a signed apt refresh and expected runtime directory modes.
  • Rebuilt worker-site against the local parent; the full image build succeeded with current Ubuntu dependencies.

Copilot AI review requested due to automatic review settings July 22, 2026 12:50

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.

Pull request overview

Updates the image build’s apt workflow to avoid signature verification issues caused by non-standard /tmp ownership/permissions and to prevent successful builds from proceeding with stale apt metadata.

Changes:

  • Make apt-get update fail the build on any index refresh error (APT::Update::Error-Mode=any).
  • Stop recreating and chown/chmoding /tmp, preserving the base image’s root-owned sticky directory permissions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings July 22, 2026 12:52
@fqjony fqjony self-assigned this Jul 22, 2026
@fqjony
fqjony requested a review from a team July 22, 2026 12:53
@fqjony fqjony added bug Something isn't working docker Pull requests that update Docker code labels Jul 22, 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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread Dockerfile Outdated
Bumps usabilitydynamics/udx-worker from 0.45.0 to 0.47.0.

---
updated-dependencies:
- dependency-name: usabilitydynamics/udx-worker
  dependency-version: 0.47.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Copilot AI review requested due to automatic review settings July 22, 2026 12:56

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings July 22, 2026 13:07

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread Dockerfile
@fqjony
fqjony enabled auto-merge July 22, 2026 13:17
@fqjony
fqjony merged commit 64fe09a into latest Jul 22, 2026
18 checks passed
@fqjony
fqjony deleted the fix/apt-update-temp-permissions branch July 22, 2026 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working docker Pull requests that update Docker code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants