mirror of https://github.com/kubernetes/kops.git
fix(nodeup): set `MACAddressPolicy` to `none` when using AWS CNI and Ubuntu 22.04
This commit is contained in:
parent
6a1783b592
commit
1342fd1afa
|
@ -0,0 +1,59 @@
|
|||
/*
|
||||
Copyright 2024 The Kubernetes Authors.
|
||||
|
||||
Licensed under the Apache License, Version 2.0 (the "License");
|
||||
you may not use this file except in compliance with the License.
|
||||
You may obtain a copy of the License at
|
||||
|
||||
http://www.apache.org/licenses/LICENSE-2.0
|
||||
|
||||
Unless required by applicable law or agreed to in writing, software
|
||||
distributed under the License is distributed on an "AS IS" BASIS,
|
||||
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
See the License for the specific language governing permissions and
|
||||
limitations under the License.
|
||||
*/
|
||||
|
||||
package networking
|
||||
|
||||
import (
|
||||
"k8s.io/kops/nodeup/pkg/model"
|
||||
"k8s.io/kops/upup/pkg/fi"
|
||||
"k8s.io/kops/upup/pkg/fi/nodeup/nodetasks"
|
||||
)
|
||||
|
||||
// AmazonVPCRoutedENIBuilder writes the Amazon VPC CNI configuration
|
||||
type AmazonVPCRoutedENIBuilder struct {
|
||||
*model.NodeupModelContext
|
||||
}
|
||||
|
||||
var _ fi.NodeupModelBuilder = &AmazonVPCRoutedENIBuilder{}
|
||||
|
||||
// Build is responsible for configuring the network cni
|
||||
func (b *AmazonVPCRoutedENIBuilder) Build(c *fi.NodeupModelBuilderContext) error {
|
||||
if b.NodeupConfig.Networking.AmazonVPC == nil {
|
||||
return nil
|
||||
}
|
||||
|
||||
// Running Amazon VPC CNI on Ubuntu 22.04 and later requires setting MACAddressPolicy to `none` (ref: https://github.com/aws/amazon-vpc-cni-k8s/issues/2103 & https://github.com/kubernetes/kops/issues/16255)
|
||||
if b.Distribution.IsUbuntu() && b.Distribution.Version() >= 22.04 {
|
||||
contents := `
|
||||
[Match]
|
||||
OriginalName=*
|
||||
[Link]
|
||||
NamePolicy=keep kernel database onboard slot path
|
||||
AlternativeNamesPolicy=database onboard slot path
|
||||
MACAddressPolicy=none
|
||||
`
|
||||
|
||||
// Copy all the relevant entries and replace the one that contains MACAddressPolicy= with MACAddressPolicy=none
|
||||
c.AddTask(&nodetasks.File{
|
||||
Path: "/etc/systemd/network/99-default.link",
|
||||
Contents: fi.NewStringResource(contents),
|
||||
Type: nodetasks.FileType_File,
|
||||
OnChangeExecute: [][]string{{"systemctl", "restart", "systemd-networkd"}},
|
||||
})
|
||||
|
||||
}
|
||||
return nil
|
||||
}
|
|
@ -318,6 +318,7 @@ func (c *NodeUpCommand) Run(out io.Writer) error {
|
|||
loader.Builders = append(loader.Builders, &networking.CommonBuilder{NodeupModelContext: modelContext})
|
||||
loader.Builders = append(loader.Builders, &networking.CalicoBuilder{NodeupModelContext: modelContext})
|
||||
loader.Builders = append(loader.Builders, &networking.CiliumBuilder{NodeupModelContext: modelContext})
|
||||
loader.Builders = append(loader.Builders, &networking.AmazonVPCRoutedENIBuilder{NodeupModelContext: modelContext})
|
||||
loader.Builders = append(loader.Builders, &networking.KuberouterBuilder{NodeupModelContext: modelContext})
|
||||
|
||||
loader.Builders = append(loader.Builders, &model.BootstrapClientBuilder{NodeupModelContext: modelContext})
|
||||
|
|
Loading…
Reference in New Issue