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

tests: run all integration tests during nightly builds #10555

Closed
18 tasks done
powersj opened this issue Feb 1, 2022 · 2 comments
Closed
18 tasks done

tests: run all integration tests during nightly builds #10555

powersj opened this issue Feb 1, 2022 · 2 comments
Assignees
@powersj
Labels
testing request An issue that requires the tests to be updated

Comments

@powersj
Copy link
Contributor

powersj commented Feb 1, 2022 •
edited
Loading

Feature Request

Currently, all CircleCI testing runs only the short unit tests. There are many longer-running tests or tests that require internet connectivity that the short tests skip. This is great for developers and local testing, however, we should run the longer tests somewhere in Telegraf.

Proposal:

The nightly jobs are a perfect place to add additional testing. These jobs are run once a day and act as a good gate for the current quality of Telegraf. Therefore, the proposal is to update the nightly CircleCI jobs to run the complete set of unit tests, not just short tests.

Migrating from docker-compose -> test-containers
@powersj powersj added the feature request Requests for new plugin and for new features to existing plugins label Feb 1, 2022
@Hipska Hipska added testing request An issue that requires the tests to be updated and removed feature request Requests for new plugin and for new features to existing plugins labels Feb 1, 2022
@powersj
Copy link
Contributor Author

powersj commented May 10, 2022

next steps: add nightly test runner for all tests

@powersj powersj changed the title tests: run all tests during nightly builds tests: run all integration tests during nightly builds May 18, 2022
@powersj powersj self-assigned this May 18, 2022
powersj added a commit to powersj/telegraf that referenced this issue Jun 2, 2022
@powersj
test: integration testing with circleci
aa30301 
This adds an integration test run with every circleci run. It does not
take that long to run and would be helpful to have during runs to catch
things in advanced.

fixes: influxdata#10555
@powersj
Copy link
Contributor Author

powersj commented Jun 6, 2022

This is officially done and in place. Running integration tests on both nightly and every PR.

@powersj powersj closed this as completed Jun 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@powersj powersj

Labels
testing request An issue that requires the tests to be updated
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Hipska @powersj