Fix: Overwriting tls.crt and tls.key when using secret type kubernetes.io/tls for VaultPKISecret (#658)

[jacob-faber]

Fixes #658

[hashicorp-cla]

All committers have signed the CLA.

[benashz]

Thanks for the contribution @jacob-faber - I would be more in favour of making this change a bit simpler. We could just say that if there are any transformations defined for the destination then skip the hard coded data rendering.

[jacob-faber]

@benashz, Do you think

vault-secrets-operator/controllers/vaultpkisecret_controller.go

Lines 232 to 234 in 33c087f

	if len(data["ca_chain"]) > 0 {
	data["ca_chain"] = []byte(strings.Join(certResp.CAChain, "\n"))
	}

should be skipped too?

[benashz]

LGTM!