• Iustin Pop's avatar
    Change behaviour of ConfigWriter._WriteConfig · d2231b8c
    Iustin Pop authored
    
    
    This patch changes the behaviour of _WriteConfig in case of
    configuration errors:
    
    - before, it used to abort the saving (even though the in-memory
      configuration used by current jobs has already changed)
    - now, we log it (both to the log and to the user) but continue, since
      we can't revert to a good version of the config anyway
    
    This should make the internal behaviour of the code more consistent with
    the external world, even though the config might be “wrong”; we leave
    the cleanup to the user. This should not be as bad as it sounds, since
    we haven't actually seen this case except for the ugly master candidates
    handling, and that was fixed recently by Guido's patch series.
    Signed-off-by: default avatarIustin Pop <iustin@google.com>
    Reviewed-by: default avatarMichael Hanselmann <hansmi@google.com>
    d2231b8c
config.py 44 KB