Code review comment for lp:~fcorrea/landscape-client/wait-for-valid-session-id-on-resynchronize

Revision history for this message
Chad Smith (chad.smith) wrote :

Good work here and additional unit tests. It took a while for me to go through upgrade path testing w/ broken released client TO patched trunk client.

Looks like a plain upgrade of a broken existing released client will still sit missing the initial delta of all system users. But, those broken clients will properly send any users or groups that were modified after the initial registration of that client.

To truly "fix" the broken clients, the customer would have to upgrade landscape-client and click the resynch button for that computer as a support person.

+1. This was tough.

review: Approve

« Back to merge proposal