Skip to content

Commit

Permalink
network: clarify VMI IP reporting
Browse files Browse the repository at this point in the history
Make it clear that IP reported in VMI status always reflects the first
address from IPs list. This is already the case, but has not been
documented. This behavior matches the logic seen in Kubernetes' PodIPs
that were recently introduced to support dual-stack.

Signed-off-by: Petr Horacek <[email protected]>
  • Loading branch information
phoracek committed May 28, 2020
1 parent 276971c commit 9f7badb
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion api/openapi-spec/swagger.json
Original file line number Diff line number Diff line change
Expand Up @@ -6777,7 +6777,7 @@
"type": "string"
},
"ipAddress": {
"description": "IP address of a Virtual Machine interface",
"description": "IP address of a Virtual Machine interface. It is always the first item of IPs",
"type": "string"
},
"ipAddresses": {
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 2 additions & 1 deletion staging/src/kubevirt.io/client-go/api/v1/types.go
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,8 @@ func (m *VirtualMachineInstanceMigration) TargetIsHandedOff() bool {
//
// +k8s:openapi-gen=true
type VirtualMachineInstanceNetworkInterface struct {
// IP address of a Virtual Machine interface
// IP address of a Virtual Machine interface. It is always the first item of
// IPs
IP string `json:"ipAddress,omitempty"`
// Hardware address of a Virtual Machine interface
MAC string `json:"mac,omitempty"`
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 9f7badb

Please sign in to comment.