Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(vmip): add validating ip address for VMIP with type 'Static' #404

Merged
merged 5 commits into from
Sep 30, 2024

Conversation

LopatinDmitr
Copy link
Contributor

@LopatinDmitr LopatinDmitr commented Sep 25, 2024

Description

This PR introduces several fixes:

  • Fix the issue with creating duplicate VirtualMachineIPAddresses when creating a VirtualMachine with an automatically assigned IP address.
  • Fix the functionality of the 'need restart' option in the VirtualMachine status.
  • Add IP address validation when manually creating a VirtualMachineIPAddress of type 'Static' (the first and last addresses in the CIDR can no longer be specified).

Why do we need it, and what problem does it solve?

What is the expected result?

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Signed-off-by: dmitry.lopatin <[email protected]>
@LopatinDmitr LopatinDmitr force-pushed the fix/vmip/add-validating-static-ip-address branch from 2ce9a4e to 788e81d Compare September 26, 2024 19:34
@LopatinDmitr LopatinDmitr marked this pull request as ready for review September 30, 2024 08:00
@LopatinDmitr LopatinDmitr merged commit ddb60cc into main Sep 30, 2024
43 of 46 checks passed
@LopatinDmitr LopatinDmitr deleted the fix/vmip/add-validating-static-ip-address branch September 30, 2024 19:53
diafour pushed a commit that referenced this pull request Oct 2, 2024
universal-itengineer pushed a commit that referenced this pull request Oct 14, 2024
Features
api: move calculation of scratch filesystem overhead to cdi controller (91d8db3)
cvi,vi,vd: replace uploadCommand field with imageUploadURLs (70ccd2e)
kube-api-rewriter: add pprof and metrics, fix rewrite performance (#402) (a009052)
vd, vmbda: add _labels and _annotations metrics (#399) (7690d2b)
vd: resizing vd in filesystem mode (#385) (7857bd3)
vm: add custom secret types for provisioning (#390) (597d551)
vm: add failure message to condition if migration is failed (78b3a42)
vm: add new VM _labels and _annotations metrics and refactor existing ones (#398) (75e57ad)
vmclass: add validation for matching virtual machine sizing policies upon virtual machine class change (#389) (029c445)
vm: disable serial console log (21d0bec)
vmop: add metrics (#370) (2436b8e)
vmop: add operation to migrate vm (#386) (d61bab4)
vmrestore: add the ability to restore virtualmachines (0c59bff)
vmsnapshot: add the ability to snapshot virtualmachines (38b84d4)

Bug Fixes
vd,vi,cvi: fix vd uploader service creating (#409) (d012007)
vd: allow to change size in spec for not ready vd (#411) (38bb0eb)
vm,vmip: improve VMIP management (#374) (d6695ef)
vmbda: reconcile from virtual disk phase changes events (7157204)
vm: fix the virtual machine matching with the virtual machine class during validation when no sizing policy is provided (#431) (4704e31)
vmip: add validating ip address for VMIP with type 'Static' (#404) (2c95be9)
vmip: fix deleting unattached vmip (#405) (56cb6cd)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants