Well, it is consistent with around code but I think that some more readable code would be better, especially separation of formatting and values like. Also you can use operator .=
```
if ($cluster_present) {
  $content .= "\t" . "clustering = yes" . "\n";
  $content .= "\t" . "rctdbd socket =" . "\n";
}
```

This makes me also curious why empty value in tctdbd socket is right. Maybe some explanation why it is right should be here. Enabled clustering looks obvious for me, but second option is slightly confusing so it should be explained.