Miloš Prchlík
Miloš Prchlík
See https://github.com/teemtee/tmt/pull/3645, seems like you and @martinhoyer are dancing around the same problem.
> > See #3645, seems like you and @martinhoyer are dancing around the same problem. > > hmmm, Martin's mr is trying to fix [my issue](https://github.com/teemtee/tmt/issues/3549)? While this mr is...
Sounds like a good tool. There are some ideas and caveats, and there may be a nice MVP to begin with & improve later.
@thrix that script path was bugging me, I wasn't sure why, but eventually, I decided to give it a try. It turned out it was the multiple places calling it,...
@thrix do you plan to include https://github.com/teemtee/tmt/pull/3229/commits/e2db1d03ec5b9eeb0cc1bc307561833d03c32a86 in this PR? It would be lost between squashes, and such a change deserves its own spot in the sun :)
> > @dcermak also sounds like maybe we could change something in the code to use multiple `Container` instances rather, to also avoid the problem? > > Chiming in @happz...
AFAICT, we're using the fixtures correctly. We do have two pairs of them: `guest` + `container`, and `guest_per_test` + `container_per_test`. The first set is used for read-only-ish tests, the second...
Adding an `error` result for the test that "killed" the `execute` should be well within our reach. It was also discussed (briefly, IIRC, feel free to correct me) that tmt...
/packit build
One thing to consider: `prepare -h feature --buildroot=enabled`, if present in a plan that's being split into buildroot and non-buildroot piles of tests, needs to be dropped from one of...