Specifying multiple triggers does not behave as expected

[bo0tzz]

Describe the bug
When specifying multiple triggers on a ReplicationSource, for example

spec:
  trigger:
    schedule: 0 23 * * *
    manual: foo

The schedule trigger gets ignored.

Steps to reproduce
Create an RS with the above triggers

Expected behavior
Either both triggers apply, or the resource is rejected by a webhook when applying.

Actual results
Only the manual trigger applies, and the schedule is ignored.

Additional context
I believe the relevant code that ends up ignoring the schedule trigger is this

volsync/controllers/statemachine/machine.go

Lines 83 to 92 in a5acd32

	func getTrigger(r ReplicationMachine) triggerType {
	switch {
	case len(r.ManualTag()) > 0:
	return manualTrigger
	case len(r.Cronspec()) > 0:
	return scheduleTrigger
	default:
	return noTrigger
	}
	}

[tesshuflower]

@bo0tzz VolSync doesn't support using manual and schedule triggers at the same time. If you use a manual trigger and want to go back to your schedule you'll need to remove the manual trigger.

[bo0tzz]

VolSync doesn't support using manual and schedule triggers at the same time

I am aware of that. The issue is that it still lets me set both, and then fails silently. I would expect the manifest to be rejected by a validating webhook.