Skip to content

🐛 fix(OscMachine/validation/webhook): loadbalancer disabling fixes#742

Open
jfbus wants to merge 1 commit intomainfrom
fix_nolbu
Open

🐛 fix(OscMachine/validation/webhook): loadbalancer disabling fixes#742
jfbus wants to merge 1 commit intomainfrom
fix_nolbu

Conversation

@jfbus
Copy link
Contributor

@jfbus jfbus commented Feb 16, 2026

Description

  • validate assertion against empty loadbalancer when disabled
  • prevent validation of osc loadbalancer attributes when disabled
  • prevent setdefault values in osc loadbalancer while reconciling oscmachine vms
  • prevent osc loadbalancer webhook validation in update webhook when disabled

Primary author: @ddavid-numspot

Refs: #740

Type of Change

Please check the relevant option(s):

  • 🐛 Bug fix
  • ✨ New feature
  • 🧹 Code cleanup or refactor
  • 📝 Documentation update
  • 🔧 Build or CI-related change
  • 🔒 Security fix
  • Other (specify):

How Has This Been Tested?

Please describe the test strategy:

  • Manual testing
  • Unit tests
  • Integration tests
  • Not tested yet

Checklist

  • I have followed the Contributing Guidelines
  • I have added tests or explained why they are not needed
  • I have updated relevant documentation (README, examples, etc.)
  • My changes follow the Conventional Commits specification
  • My commits include appropriate Gitmoji

Additional Context

- validate assertion against empty loadbalancer when disabled
- prevent validation of osc loadbalancer attributes when disabled
- prevent setdefault values in osc loadbalancer while reconciling oscmachine vms
- prevent osc loadbalancer webhook validation in update webhook when disabled

Co-Authored-By: Jean-François Bustarret
<jean-francois.bustarret@outscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants