Code review comment for lp://staging/~benji/landscape-client/bug-1548946-xenial-test-failures

Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

I'm basically +1 with a non-blocking nitpick.

Just marking as Needs information because of the following question.

Since it seems you investigated the issue in depth and possible looked at the changed behavior in the apt module, do you think it would be possible to avoid using the TestCache and change the test to adapt to the new behavior somehow? Or is it now plain impossible for calling code to have visibility on that?

review: Needs Information

« Back to merge proposal