github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-08-02 16:29:21 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #130512 from HirazawaUi/Clarify-cri-fields

Browse Source 
Clarify the behavior when the host_port value is set to 0 in CRI
This commit is contained in:
Kubernetes Prow Robot 2025-03-01 09:32:55 -08:00 committed by GitHub
commit b669e6cc9a
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 20 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
staging/src/k8s.io/cri-api/pkg/apis/runtime/v1/api.pb.go
View File

@ -555,7 +555,16 @@ type PortMapping struct {
Protocol Protocol `protobuf:"varint,1,opt,name=protocol,proto3,enum=runtime.v1.Protocol" json:"protocol,omitempty"` Protocol Protocol `protobuf:"varint,1,opt,name=protocol,proto3,enum=runtime.v1.Protocol" json:"protocol,omitempty"`
// Port number within the container. Default: 0 (not specified). // Port number within the container. Default: 0 (not specified).
ContainerPort int32 `protobuf:"varint,2,opt,name=container_port,json=containerPort,proto3" json:"container_port,omitempty"` ContainerPort int32 `protobuf:"varint,2,opt,name=container_port,json=containerPort,proto3" json:"container_port,omitempty"`
// Port number on the host. Default: 0 (not specified). // Port number on the host to map the container port to.
//
// - Valid host port range is 1-65535.
// - The value 0 has explicit semantic meaning: it indicates NO host port should be allocated.
// - The value 0 does NOT indicate dynamic port allocation. Future implementations
// of dynamic allocation will use different values/semantics.
// - Implementations MUST handle the case where this field is explicitly set to 0,
// This field SHOULD be omitted when no port is required.
//
// Default: If omitted, container port will not be exposed on the host.
HostPort int32 `protobuf:"varint,3,opt,name=host_port,json=hostPort,proto3" json:"host_port,omitempty"` HostPort int32 `protobuf:"varint,3,opt,name=host_port,json=hostPort,proto3" json:"host_port,omitempty"`
// Host IP. // Host IP.
HostIp string `protobuf:"bytes,4,opt,name=host_ip,json=hostIp,proto3" json:"host_ip,omitempty"` HostIp string `protobuf:"bytes,4,opt,name=host_ip,json=hostIp,proto3" json:"host_ip,omitempty"`

11
staging/src/k8s.io/cri-api/pkg/apis/runtime/v1/api.proto
View File

@ -201,7 +201,16 @@ message PortMapping {
Protocol protocol = 1; Protocol protocol = 1;
// Port number within the container. Default: 0 (not specified). // Port number within the container. Default: 0 (not specified).
int32 container_port = 2; int32 container_port = 2;
// Port number on the host. Default: 0 (not specified). // Port number on the host to map the container port to.
//
// * Valid host port range is 1-65535.
// * The value 0 has explicit semantic meaning: it indicates NO host port should be allocated.
// * The value 0 does NOT indicate dynamic port allocation. Future implementations
// of dynamic allocation will use different values/semantics.
// * Implementations MUST handle the case where this field is explicitly set to 0,
// This field SHOULD be omitted when no port is required.
//
// Default: If omitted, container port will not be exposed on the host.
int32 host_port = 3; int32 host_port = 3;
// Host IP. // Host IP.
string host_ip = 4; string host_ip = 4;