Fix SKU allocation validation bypass

[Sinck]

This prevents allow_allocation_on_validation_failure from allowing a machine without an assigned SKU to skip SKU assignment. The two policies now retain distinct semantics: explicitly ignoring unassigned machines is separate from allowing validation failures for an assigned SKU.

Related issues

None.

Type of Change

• Add - New feature or capability
• Change - Changes in existing functionality
• Fix - Bug fixes
• Remove - Removed features or deprecated functionality
• Internal - Internal changes (refactoring, tests, docs, etc.)

Breaking Changes

• This PR contains breaking changes

Testing

• Unit tests added/updated
• Integration tests added/updated
• Manual testing performed
• No testing required (docs, internal refactor, etc.)

Additional Notes

Review the policy split: ignore_unassigned_machines applies only when no SKU is assigned, while allow_allocation_on_validation_failure applies only after validation of an assigned SKU fails.

[coderabbitai]

Caution

Review failed

An error occurred during the review process. Please try again later.

---

_{Comment @coderabbitai help to get the list of available commands.}