The current SDAM update implementation returns a Result, but this is only due to an implementation quirk rather than the method actually possibly failing (the fields of ServerDescription are accessed through a Result, though the parts where this is done in update are guaranteed to not fail). We should update this method to be infallible to avoid confusion and eliminate the need to ignore its return value.