• Michael Taufen's avatar
    Don't compare interface values in dynamic config controller · 1089cb62
    Michael Taufen authored
    This previously caused a panic when moving lastKnownGood between two
    non-nil values, because we were comparing the interface wrapper instead
    of comparing the NodeConfigSources. The case of moving from one non-nil
    lastKnownGood config to another doesn't appear to be tested by the e2e
    node tests. I added a unit test and an e2e node test to help catch bugs
    with this case in the future.
    1089cb62