Skip to content

Conversation

@tdonohue
Copy link
Member

@tdonohue tdonohue commented Jan 13, 2026

References

Description

During the manual backport of #4987, I realized we still had a few Docker Compose files with unnecessary tty and stdin_open configurations. I've removed these few remaining from all the backport PRs (#4994, #4995, #4996).

So, this PR is essentially a "forward port" of that tiny change back to main, as it's already been applied in #4994, #4995, #4996.

Instructions for Reviewers

  • We now have automated tests which cover these small changes.

@tdonohue tdonohue added this to the 10.0 milestone Jan 13, 2026
@tdonohue tdonohue added 1 APPROVAL pull request only requires a single approval to merge backend: Docker related to DSpace deployment via Docker labels Jan 13, 2026
@tdonohue tdonohue changed the title Remove more unnecessary tty/stdin_open settings from Docker scripts [Docker] Remove more unnecessary tty/stdin_open settings from Docker scripts Jan 13, 2026
@tdonohue tdonohue merged commit 3753418 into DSpace:main Jan 13, 2026
16 checks passed
@tdonohue tdonohue deleted the remove_tty branch January 13, 2026 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge backend: Docker related to DSpace deployment via Docker code task

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

1 participant