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

Initial deb and rpm packaging #1278

Merged
merged 2 commits into from
Jul 8, 2020
Merged

Conversation

jchengsfx
Copy link
Contributor

@jchengsfx jchengsfx commented Jul 7, 2020

Description:

  • Uses https://github.com/jordansissel/fpm
  • Adds deb-package and rpm-package targets to Makefile
  • 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.
  • Adds deb and rpm packages to circleci workspace for github release
  • TODO: create service scripts

Link to tracking Issue:

Testing:

  • Added simple installation test after package build

Documentation:

  • Added packaging/deb/README.md and packaging/rpm/README.md

@codecov
Copy link

codecov bot commented Jul 7, 2020

Codecov Report

Merging #1278 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7d7e008...c11e912. Read the comment docs.

Copy link
Contributor

@flands flands left a 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
Copy link
Contributor

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?

Copy link
Member

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.

@@ -0,0 +1,13 @@
FROM debian:9

RUN apt-get update && \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

upgrade here as well?

@jchengsfx
Copy link
Contributor Author

Need to make similar changes according to review in open-telemetry/opentelemetry-collector-contrib#405.

Copy link
Member

@bogdandrutu bogdandrutu left a 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
@bogdandrutu bogdandrutu merged commit 1ecc3a1 into open-telemetry:master Jul 8, 2020
@jchengsfx jchengsfx deleted the deb-rpm branch July 9, 2020 00:18
wyTrivail pushed a commit to mxiamxia/opentelemetry-collector that referenced this pull request Jul 13, 2020
* 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
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this pull request Nov 11, 2021
hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this pull request Apr 27, 2023
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.

3 participants