cybozu-go / moco

Describe the bug
SetReadOnly waits for a running DML.
Therefore, if it waits for a long time, deleteGracePeriodSeconds may be reached.

moco/clustering/operations.go

Lines 154 to 156 in c545ebb

    
           if err := pdb.SetReadOnly(ctx, true); err != nil { 
        
           	return fmt.Errorf("failed to make instance %d read-only: %w", ss.Primary, err) 
        
           }

Expected behavior
If SetReadOnly does not complete in a short time, give up and run KillConnections.

The problem is that the switching is not completed within the PreStopSeconds time.
Solution will be considered separately.

	if err := pdb.SetReadOnly(ctx, true); err != nil {
	return fmt.Errorf("failed to make instance %d read-only: %w", ss.Primary, err)
	}

Long wait for SetReadOnly to complete