Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set TERM for docker worker images #464

Merged
merged 2 commits into from
Feb 28, 2024
Merged

Set TERM for docker worker images #464

merged 2 commits into from
Feb 28, 2024

Conversation

bhearsum
Copy link
Collaborator

@bhearsum bhearsum commented Feb 22, 2024

This is to improve issues on docker worker interactive tasks where curses applications don't work correctly.

This is not quite perfect - but it's an improvement. There's seems to be a UI issue in Taskcluster still, where it mixes up x and y size for docker-worker sessions, which leads to a wrongly sized vim window and weird wrapping. I intend to fix that, but that's outside of the scope of this repo.

@bhearsum bhearsum force-pushed the term branch 3 times, most recently from 9afa5ca to 2fe28f1 Compare February 22, 2024 18:53
@bhearsum bhearsum marked this pull request as ready for review February 22, 2024 19:33
@bhearsum bhearsum requested a review from a team as a code owner February 22, 2024 19:33
@bhearsum
Copy link
Collaborator Author

I ended up cancel the taskgraph here to save resources - no need to run the entire pipeline to verify this small change to the base docker image.

@bhearsum
Copy link
Collaborator Author

This is to improve issues on docker worker interactive tasks where curses applications don't work correctly.

This is not quite perfect - but it's an improvement. There's seems to be a UI issue in Taskcluster still, where it mixes up x and y size for docker-worker sessions, which leads to a wrongly sized vim window and weird wrapping. I intend to fix that, but that's outside of the scope of this repo.

I fixed the Taskcluster issue in taskcluster/taskcluster#6856; that will take a couple of weeks to make it to the Firefox CI instance.

@gabrielBusta
Copy link
Member

Nice.

@bhearsum bhearsum merged commit ed41ca2 into mozilla:main Feb 28, 2024
28 of 32 checks passed
nordzilla pushed a commit that referenced this pull request Sep 19, 2024
Bumps [3rd_party/marian-dev](https://github.com/browsermt/marian-dev) from `300a50f` to `780df27`.
- [Commits](browsermt/marian-dev@300a50f...780df27)

---
updated-dependencies:
- dependency-name: 3rd_party/marian-dev
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

2 participants