-
Notifications
You must be signed in to change notification settings - Fork 165
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
Add nightly linux package tests #659
Comments
i'd like to work on this, pls assign me :) |
I started implementing this but hit some road blocks it seems. I tried building the otelcol-contrib on the spot while setting all component versions to There are 2 options on where to do this:
would be cool to get some feedback on this, maybe from @andrzej-stencel or @mx-psi or @codeboten |
Do we know (or can we know) why that is the case? Is it downloading dependencies that takes too long? Cloning the repo? The build process itself?
One downside of this is that these artifacts do not necessarily reflect the build time options we use here, but for testing it feels like they should be a net positive
I would prefer this, I'd like to keep anything related to linux package files on this repository |
From what I could gather, it's trying to fetch the latest main version from the Go proxy and waits for minutes until finally getting back a 404, then going to the github repo and downloading the whole thing from there, which is not a small download either. And I don't think lots of things are cached either in-between dependencies, so it does the whole process over and over.
sounds good, then I'll continue in this direction |
New status on this: the only downside is that maintainability will suffer a bit since the contrib goreleaser config is split into 2 files. But I think this is the only sensible approach after also speaking with goreleaser maintainers (goreleaser/goreleaser#5234). |
Description
Create a nightly version of the linux package tests that use the latest snapshots of main branch of core and contrib distributions from the respective repos.
This is a follow-up to open-telemetry/opentelemetry-collector-contrib#34372
The text was updated successfully, but these errors were encountered: