-
Notifications
You must be signed in to change notification settings - Fork 21
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
E2E Testing for Vector #823
Comments
Evaluating the following:
Noting that I will have to deploy a privileged pod to generate node logs as there are no node logs being generated by the time vector is deployed in CI. I will also have to expose a new zarf var on the vector package to toggle enabling the API. |
This is surprising to me - there should be a very short log file for k3d startup? |
There is - about a dozen lines. But from what I found in testing, vector only collects logs that were generated after it starts up. I could be missing something though. Maybe there's a parameter I can add that allows me to search back in time before it was deployed. |
Discussed offline - found there is no need to manually generate node logs at this time. |
Should also note - log generation for testing could be a good path if we want to test additional log scrape targets in the future, I've definitely seen similar tests in other projects where a pod will just generate some noise to ensure there is always something happening. |
Noting here context - we switched the vector e2e tests to querying loki directly during the loki e2e tests. We noticed that it took a while for vector metrics to populate which was causing tests to fail. Running the tests from loki ensures that same cases are validated and gives vector adequate time to collect logs. |
## Description Add E2E prometheus, loki, and vector testing. ## Related Issues Fixes #824 #823 #822 ## Type of change - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [x] Other (security config, docs update, etc) ## Checklist before merging - [x] Test, docs, adr added or updated as needed - [x] [Contributor Guide](https://github.com/defenseunicorns/uds-template-capability/blob/main/CONTRIBUTING.md) followed --------- Co-authored-by: Rob Ferguson <[email protected]> Co-authored-by: Noah <[email protected]> Co-authored-by: Noah Birrer <[email protected]> Co-authored-by: Micah Nagel <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
E2E testing for Vector should only be run on the all package test only since it focuses on interactions between components - see #211 for some thoughts on this.
Initial thoughts on testing areas:
Vector's API is not enabled by default so we may need to enable it in CI at minimum and use it to access some of these data points.
The text was updated successfully, but these errors were encountered: