lifecycle-toolkit icon indicating copy to clipboard operation
lifecycle-toolkit copied to clipboard

[Tracking] Add collectors to Integration tests

Open RealAnna opened this issue 2 years ago • 6 comments

Given the good first example 🚀 of how to add a collectors to kuttl tests #1834, we should add collectors in each test that:

  1. collect logs of operators
  2. run a kubectl describe for each involved CRDS

Listing the tests here with what would be nice to collect so that we can have multiple contributions 🥇

  • [x] #2697
  • [ ] #2698
  • [x] #2699
  • [x] #2700
  • [ ] #2702

RealAnna avatar Aug 11 '23 14:08 RealAnna

As for the previous task of adding collectors to testasserts, I would like to take this up.

prakrit55 avatar Aug 11 '23 14:08 prakrit55

I believe there can be multiple contributors to this particular issue. Anyone can work on this and raise a separate PR for a separate test so that it becomes easy to review and easy to track. Just mention here before what test/tests you are currently working on.

rakshitgondwal avatar Aug 13 '23 15:08 rakshitgondwal

I believe there can be multiple contributors to this particular issue. Anyone can work on this and raise a separate PR for a separate test so that it becomes easy to review and easy to track. Just mention here before what test/tests you are currently working on.

So, I committed for the tests

  • annotated-namespace-not-enabled
  • simple-daemonset-annotated

and wd like to improve other tests too.

prakrit55 avatar Aug 13 '23 15:08 prakrit55

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Oct 16 '23 03:10 github-actions[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Dec 16 '23 03:12 github-actions[bot]

Updated the ticket to reflect the current status of our Integration tests, this is now divided into 4. I believe this would reduce our test time of 2min ca. in case of test failure if not more

RealAnna avatar Dec 19 '23 09:12 RealAnna