[16:36] <blackboxsw> falcojr: rjschwei thanks for the pyversion checks PR. Just landed https://github.com/canonical/cloud-init/pull/857
[17:01] <rjschwei> falcojr: blackboxsw thanks for the help
[17:33] <blackboxsw> np... better late than never.
[17:34] <blackboxsw> falcojr: just waiting on CI for the network activators PR https://github.com/canonical/cloud-init/pull/919
[18:10] <blackboxsw> falcojr: one CI fix on the above as the available_mocks don't seem to have worked on the NetworkManagerActitivator.available check
[18:13] <blackboxsw> I think we are missing  the following https://paste.ubuntu.com/p/S9yxyk4CR5/
[18:17] <falcojr> thanks, I'll make the update 
[18:25] <blackboxsw> falcojr: BTW nice use of pytest.yield_fixture for the consolidating reused sets of mocks it makes reading the unit tests much easier. that was a TIL moment for me on this PR
[18:26] <blackboxsw> in my mind, definitely beats a set of  class-level @mock.patch decorators queued, plus duplicating all those mocked params on every test
[18:33] <falcojr> yeah, all of the mocking stuff can get pretty complicated. Glad you find this better, but it all still feels a little janky to me :)
[18:37] <blackboxsw> yes  toomanymocks == bad smell
[18:37] <blackboxsw> but we can happily sweep it under the carpet with fixtures
[18:37] <blackboxsw> :)
[18:37] <falcojr> heh
[19:23] <blackboxsw> falcojr: one more review for you https://github.com/canonical/cloud-init/pull/932/files
[19:25] <blackboxsw> also will let you squash merge the approved https://github.com/canonical/cloud-init/pull/919