-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Enable E2E tests on GitHub Actions #9660
Conversation
Signed-off-by: Derek Nola <[email protected]>
Signed-off-by: Derek Nola <[email protected]>
Signed-off-by: Derek Nola <[email protected]>
Simplify node roles Signed-off-by: Derek Nola <[email protected]>
Signed-off-by: Derek Nola <[email protected]>
5898a7e
ab01cd7
to
5898a7e
Compare
Sorry some new tests were merged to master since I started working on all this so I had to add more to the |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #9660 +/- ##
===========================================
- Coverage 51.52% 39.36% -12.17%
===========================================
Files 154 151 -3
Lines 13595 13543 -52
===========================================
- Hits 7005 5331 -1674
- Misses 5253 7044 +1791
+ Partials 1337 1168 -169
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Proposed Changes
btrfs
testsudo vagrant
everywhere for CIAfterSuite
test cleanup, no longer have irrelevant failures around go coverage clogging up the logs.Types of Changes
Testing
Verification
New CI passes in my forked PR dereknola#20
Testing
Its testing all the way down
Linked Issues
#9659
User-Facing Change
Further Comments