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

Build and publish Docker container for Windows #2715

Closed
iNikem opened this issue Mar 16, 2021 · 5 comments
Closed

Build and publish Docker container for Windows #2715

iNikem opened this issue Mar 16, 2021 · 5 comments
Labels
ci-cd CI, CD, testing, build issues os:windows Windows specific issues

Comments

@iNikem
Copy link
Contributor

iNikem commented Mar 16, 2021

Is your feature request related to a problem? Please describe.
open-telemetry/opentelemetry-java-instrumentation wants to run a bench of smoke tests on Windows platform as well. We extensively use testcontainers and Docker images as part of our testing setup. Currently GitHub actions do not allow running Linux Docker images on Windows Server environment.

Describe the solution you'd like
This project provides Windows executable for the Collector and also Docker images for Linux platform. I would like to ask to also provide pre-built Windows docker images.

Describe alternatives you've considered
We currently are going to build our own Docker images to run Collector on Windows. But there is no sense for us to do it :)

@bogdandrutu
Copy link
Member

bogdandrutu commented Mar 17, 2021

Do you want to run the windows msi image there instead of docker? For the moment, until this is resolved

@iNikem
Copy link
Contributor Author

iNikem commented Mar 17, 2021

Do you want to run the windows msi image there instead of docker? For the moment, until this is resolved

I don't :) For now it is easier for me to build my own docker images for Windows that are good enough for our test infra. I prefer to have the same test setup both for Linux and Windows and prefer not to run msi from Java test class.

@Jmorjsm
Copy link

Jmorjsm commented Aug 17, 2022

I'm interested in picking this one up.

hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this issue Apr 27, 2023
…open-telemetry#2715)

Bumps [github.com/signalfx/golib/v3](https://github.com/signalfx/golib) from 3.3.48 to 3.3.49.
- [Release notes](https://github.com/signalfx/golib/releases)
- [Commits](signalfx/golib@v3.3.48...v3.3.49)

---
updated-dependencies:
- dependency-name: github.com/signalfx/golib/v3
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@mx-psi mx-psi added the os:windows Windows specific issues label Oct 16, 2023
@pjanotti
Copy link
Contributor

pjanotti commented Mar 20, 2024

I suggest closing this one and follow-up on the releaser repo via open-telemetry/opentelemetry-collector-releases#339

cc @open-telemetry/collector-triagers

@andrzej-stencel
Copy link
Member

Closed in favor of open-telemetry/opentelemetry-collector-releases#339.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-cd CI, CD, testing, build issues os:windows Windows specific issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants