Skip to content

Conversation

@mikezupper
Copy link
Contributor

@mikezupper mikezupper commented Jan 22, 2025

What does this pull request do? Explain your changes. (required)
This provides a docker build for Windows binaries. This was in response to the Bounty published by @rickstaa
livepeer/bounties#62

Specific updates (required)

  • added a Makefile entry for docker-windows
  • added a Dockerfile.windows file

How did you test each of these updates (required)
The build ran successfully on Livepeer.Cloud's CI/CD environment. The windows binary was deployed to a windows server and properly started. (testing on the actual transcoding was NOT done)

Does this pull request close any open issues?
No

Checklist:

@github-actions github-actions bot added the docker Pull requests that update Docker code label Jan 22, 2025
@mikezupper mikezupper force-pushed the tasks/livepeer.cloud/add-windows-support branch from a5f7933 to 164ee83 Compare December 29, 2025 16:05
@codecov
Copy link

codecov bot commented Dec 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 31.91887%. Comparing base (8a37cb2) to head (f18c976).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##              master       #3366   +/-   ##
=============================================
  Coverage   31.91887%   31.91887%           
=============================================
  Files            169         169           
  Lines          41217       41217           
=============================================
  Hits           13156       13156           
  Misses         27071       27071           
  Partials         990         990           

see 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a37cb2...f18c976. Read the comment docs.
see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mikezupper mikezupper force-pushed the tasks/livepeer.cloud/add-windows-support branch from 164ee83 to f18c976 Compare December 29, 2025 16:32
@rickstaa rickstaa requested a review from hjpotter92 December 29, 2025 16:38
@hjpotter92 hjpotter92 closed this in 06d50a9 Jan 1, 2026
@hjpotter92
Copy link
Member

we were installing libz-mingw-w64-dev in our workflow causing that to take precedence over self-compiled zlib. fixed this behaviour in our build workflow now in #3848

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Pull requests that update Docker code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants