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.

Sign up for GitHub

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

Jump to bottom

Add smoke tests to verify packaged artifacts #10467

Closed
sspaink opened this issue Jan 19, 2022 · 1 comment · Fixed by #11377
Closed

Add smoke tests to verify packaged artifacts #10467

sspaink opened this issue Jan 19, 2022 · 1 comment · Fixed by #11377
Assignees
@powersj
Labels
feature request Requests for new plugin and for new features to existing plugins

Comments

@sspaink
Copy link
Contributor

sspaink commented Jan 19, 2022 •
edited
Loading

General Process follow this https://github.com/influxdata/telegraf#package-repository:

  1. Test repo during release
    • Make sure the release was a success
  2. Test packages nightly
    • Make sure there hasn't been any major breaking changes (panics at startup) introduced

This could help avoid potential app breaking changes getting released by testing nightly (e.g. #10296) and also make sure if we change anything to the packaging that things will stay the same.
@sspaink sspaink added the feature request Requests for new plugin and for new features to existing plugins label Jan 19, 2022
@sspaink sspaink self-assigned this Jan 19, 2022
@sspaink sspaink changed the title Add smoke tests to verify packaged artifacts during nightly Add smoke tests to verify packaged artifacts Jan 19, 2022
@sjwang90 sjwang90 assigned powersj and unassigned sspaink Feb 22, 2022
@powersj
Copy link
Contributor

powersj commented May 10, 2022

ack - have test scripts for individual RPM/DEB, which satisfy #1, but need to work on nightly.

powersj added a commit to powersj/telegraf that referenced this issue Jun 28, 2022
@powersj
test: add nightly package testing to amd64/linux
f482562 
This adds nightly testing of the generated amd64/linux RPM and DEB
packages. This uses LXD in a VM on CircleCI to launch a number of
different distros. The official InfluxData repo is added per the
official instructions, then the new RPM/DEB is installed. This
method verifies that the upgrade works as expected and the service
continues to run.

The package_lxd_test can be used to also locally when doing development
or changes to the DEB and RPM scripts. LXD is required for it to
correctly works. This is done as LXD let's us use system containers to
verify the systemd connection with the services versus docker, which
only lets us sort of test that the install works, but not the end-to-end
service.

fixes: influxdata#10467
@powersj powersj mentioned this issue Jun 28, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@powersj powersj

Labels
feature request Requests for new plugin and for new features to existing plugins
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: add nightly package testing to amd64/linux powersj/telegraf
2 participants
@sspaink @powersj