-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Initial deb and rpm packaging #1278
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1278 +/- ##
=======================================
Coverage 89.40% 89.40%
=======================================
Files 212 212
Lines 15152 15152
=======================================
Hits 13546 13546
Misses 1171 1171
Partials 435 435 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
- run: | ||
name: Install fpm and dependencies | ||
command: | | ||
sudo apt-get update |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should an upgrade
happen here as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
upgrade
upgrades other deps, update just refreshes the repositories.
packaging/deb/Dockerfile
Outdated
@@ -0,0 +1,13 @@ | |||
FROM debian:9 | |||
|
|||
RUN apt-get update && \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
upgrade
here as well?
Need to make similar changes according to review in open-telemetry/opentelemetry-collector-contrib#405. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please move all the scripts in internal/buildscripts/packaging
- Uses https://github.com/jordansissel/fpm - Builds simple deb and rpm packages that installs the otelcol binary to `/usr/bin/otelcol` and `examples/otel-local-config.yaml` to `/etc/otel-collector/config.yaml`. - TODO: create systemd service scripts
- Add support for arm64 packages - Move files to internal/buildscripts/packaging/ - Use single Dockerfile for local deb and rpm builds - Explicitly set output dir for packages in circleci
* Initial deb and rpm packaging - Uses https://github.com/jordansissel/fpm - Builds simple deb and rpm packages that installs the otelcol binary to `/usr/bin/otelcol` and `examples/otel-local-config.yaml` to `/etc/otel-collector/config.yaml`. - TODO: create systemd service scripts * Packaging updates - Add support for arm64 packages - Move files to internal/buildscripts/packaging/ - Use single Dockerfile for local deb and rpm builds - Explicitly set output dir for packages in circleci
Description:
deb-package
andrpm-package
targets to Makefilebinary to
/usr/bin/otelcol
andexamples/otel-local-config.yaml
to
/etc/otel-collector/config.yaml
.Link to tracking Issue:
Testing:
Documentation:
packaging/deb/README.md
andpackaging/rpm/README.md