Skip to content

Geo: Add container registry sync test for missing config

Michael Kozono requested to merge mk/add-container-registry-sync-test into master

What does this MR do and why?

Follow up for !153914 (comment 1926896198):

question (non-blocking) I am wondering if it would be hard to write a test to cover the case of this error bubbling up to last_sync_failure?

This MR adds such a test.

It also refactors the code a tiny bit.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Michael Kozono

Merge request reports