mirror of
https://github.com/siderolabs/talos.git
synced 2025-11-09 21:01:30 +01:00
New multi-doc way to apply static configuration to network links. Fixes #10957 Also implements related changes: * new-style auto-DHCP mode - run DHCP as long as there is no new-style config document * refactoring of controllers/tests which were touched in this PR * implemented embedded config structs handling in docgen -> Markdown and docgen -> JSON schema Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
236 lines
7.3 KiB
Go
236 lines
7.3 KiB
Go
// This Source Code Form is subject to the terms of the Mozilla Public
|
|
// License, v. 2.0. If a copy of the MPL was not distributed with this
|
|
// file, You can obtain one at http://mozilla.org/MPL/2.0/.
|
|
|
|
//go:build integration_api
|
|
|
|
package api
|
|
|
|
import (
|
|
"bufio"
|
|
"context"
|
|
"fmt"
|
|
"math/rand/v2"
|
|
"net/netip"
|
|
"strings"
|
|
"time"
|
|
|
|
"github.com/cosi-project/runtime/pkg/resource"
|
|
"github.com/cosi-project/runtime/pkg/resource/rtestutils"
|
|
"github.com/cosi-project/runtime/pkg/safe"
|
|
"github.com/siderolabs/go-pointer"
|
|
"github.com/stretchr/testify/assert"
|
|
|
|
"github.com/siderolabs/talos/internal/integration/base"
|
|
"github.com/siderolabs/talos/pkg/machinery/client"
|
|
"github.com/siderolabs/talos/pkg/machinery/config/machine"
|
|
"github.com/siderolabs/talos/pkg/machinery/config/types/network"
|
|
"github.com/siderolabs/talos/pkg/machinery/nethelpers"
|
|
networkres "github.com/siderolabs/talos/pkg/machinery/resources/network"
|
|
)
|
|
|
|
// NetworkConfigSuite ...
|
|
type NetworkConfigSuite struct {
|
|
base.APISuite
|
|
|
|
ctx context.Context //nolint:containedctx
|
|
ctxCancel context.CancelFunc
|
|
}
|
|
|
|
// SuiteName ...
|
|
func (suite *NetworkConfigSuite) SuiteName() string {
|
|
return "api.NetworkConfigSuite"
|
|
}
|
|
|
|
// SetupTest ...
|
|
func (suite *NetworkConfigSuite) SetupTest() {
|
|
suite.ctx, suite.ctxCancel = context.WithTimeout(context.Background(), 30*time.Second)
|
|
}
|
|
|
|
// TearDownTest ...
|
|
func (suite *NetworkConfigSuite) TearDownTest() {
|
|
if suite.ctxCancel != nil {
|
|
suite.ctxCancel()
|
|
}
|
|
}
|
|
|
|
// TestStaticHostConfig tests that /etc/hosts updates are working.
|
|
func (suite *NetworkConfigSuite) TestStaticHostConfig() {
|
|
node := suite.RandomDiscoveredNodeInternalIP()
|
|
nodeCtx := client.WithNode(suite.ctx, node)
|
|
|
|
suite.T().Logf("testing on node %q", node)
|
|
|
|
host1 := network.NewStaticHostConfigV1Alpha1("1.2.3.4")
|
|
host1.Hostnames = []string{"example.com", "example2"}
|
|
|
|
host2 := network.NewStaticHostConfigV1Alpha1("2001:db8::1")
|
|
host2.Hostnames = []string{"v6"}
|
|
|
|
suite.PatchMachineConfig(nodeCtx, host1, host2)
|
|
|
|
suite.EventuallyWithT(
|
|
func(collect *assert.CollectT) {
|
|
asrt := assert.New(collect)
|
|
|
|
hosts := suite.ReadFile(nodeCtx, "/etc/hosts")
|
|
scanner := bufio.NewScanner(strings.NewReader(hosts))
|
|
|
|
var found1, found2 bool
|
|
|
|
for scanner.Scan() {
|
|
line := scanner.Text()
|
|
|
|
switch {
|
|
case strings.HasPrefix(line, "1.2.3.4"):
|
|
found1 = true
|
|
|
|
asrt.Contains(line, "example.com", "expected to find hostname in IPv4 entry")
|
|
asrt.Contains(line, "example2", "expected to find hostname in IPv4 entry")
|
|
case strings.HasPrefix(line, "2001:db8::1"):
|
|
found2 = true
|
|
|
|
asrt.Contains(line, "v6", "expected to find hostname in IPv6 entry")
|
|
}
|
|
}
|
|
|
|
asrt.True(found1, "expected to find IPv4 entry in /etc/hosts")
|
|
asrt.True(found2, "expected to find IPv6 entry in /etc/hosts")
|
|
},
|
|
time.Second, time.Millisecond, "waiting for /etc/hosts to be updated",
|
|
)
|
|
|
|
suite.RemoveMachineConfigDocuments(nodeCtx, network.StaticHostKind)
|
|
|
|
suite.EventuallyWithT(
|
|
func(collect *assert.CollectT) {
|
|
asrt := assert.New(collect)
|
|
|
|
hosts := suite.ReadFile(nodeCtx, "/etc/hosts")
|
|
|
|
asrt.NotContains(hosts, "1.2.3.4", "expected to not find IPv4 entry in /etc/hosts")
|
|
asrt.NotContains(hosts, "2001:db8::1", "expected to not find IPv6 entry in /etc/hosts")
|
|
},
|
|
time.Second, time.Millisecond, "waiting for /etc/hosts to be updated",
|
|
)
|
|
}
|
|
|
|
// TestDummyLinkConfig tests creation of dummy link interfaces.
|
|
func (suite *NetworkConfigSuite) TestDummyLinkConfig() {
|
|
if suite.Cluster == nil {
|
|
suite.T().Skip("skipping if cluster is not qemu/docker")
|
|
}
|
|
|
|
node := suite.RandomDiscoveredNodeInternalIP(machine.TypeWorker)
|
|
nodeCtx := client.WithNode(suite.ctx, node)
|
|
|
|
suite.T().Logf("testing on node %q", node)
|
|
|
|
dummyName := fmt.Sprintf("dummy%d", rand.IntN(10000))
|
|
|
|
dummy := network.NewDummyLinkConfigV1Alpha1(dummyName)
|
|
dummy.HardwareAddressConfig = nethelpers.HardwareAddr{0x02, 0x00, 0x00, 0x00, byte(rand.IntN(256)), byte(rand.IntN(256))}
|
|
dummy.LinkUp = pointer.To(true)
|
|
dummy.LinkMTU = 9000
|
|
dummy.LinkAddresses = []network.AddressConfig{
|
|
{
|
|
AddressAddress: netip.MustParsePrefix("fd13:1234::1/64"),
|
|
AddressPriority: pointer.To[uint32](100),
|
|
},
|
|
}
|
|
dummy.LinkRoutes = []network.RouteConfig{
|
|
{
|
|
RouteDestination: network.Prefix{Prefix: netip.MustParsePrefix("fd13:1235::/64")},
|
|
RouteGateway: network.Addr{Addr: netip.MustParseAddr("fd13:1234::ffff")},
|
|
},
|
|
}
|
|
|
|
addressID := dummyName + "/fd13:1234::1/64"
|
|
routeID := dummyName + "/inet6/fd13:1234::ffff/fd13:1235::/64/1024"
|
|
addressRouteID := dummyName + "/inet6//fd13:1234::/64/100"
|
|
|
|
suite.PatchMachineConfig(nodeCtx, dummy)
|
|
|
|
rtestutils.AssertResource(nodeCtx, suite.T(), suite.Client.COSI, dummyName,
|
|
func(link *networkres.LinkStatus, asrt *assert.Assertions) {
|
|
asrt.Equal("dummy", link.TypedSpec().Kind)
|
|
asrt.Equal(dummy.HardwareAddressConfig, link.TypedSpec().HardwareAddr)
|
|
asrt.Equal(dummy.LinkMTU, link.TypedSpec().MTU)
|
|
},
|
|
)
|
|
|
|
rtestutils.AssertResource(nodeCtx, suite.T(), suite.Client.COSI, addressID,
|
|
func(addr *networkres.AddressStatus, asrt *assert.Assertions) {
|
|
asrt.Equal(dummyName, addr.TypedSpec().LinkName)
|
|
},
|
|
)
|
|
|
|
rtestutils.AssertResources(nodeCtx, suite.T(), suite.Client.COSI,
|
|
[]resource.ID{routeID, addressRouteID},
|
|
func(route *networkres.RouteStatus, asrt *assert.Assertions) {
|
|
asrt.Equal(dummyName, route.TypedSpec().OutLinkName)
|
|
},
|
|
)
|
|
|
|
suite.RemoveMachineConfigDocumentsByName(nodeCtx, network.DummyLinkKind, dummyName)
|
|
|
|
rtestutils.AssertNoResource[*networkres.LinkStatus](nodeCtx, suite.T(), suite.Client.COSI, dummyName)
|
|
rtestutils.AssertNoResource[*networkres.AddressStatus](nodeCtx, suite.T(), suite.Client.COSI, addressID)
|
|
rtestutils.AssertNoResource[*networkres.RouteStatus](nodeCtx, suite.T(), suite.Client.COSI, addressRouteID)
|
|
rtestutils.AssertNoResource[*networkres.RouteStatus](nodeCtx, suite.T(), suite.Client.COSI, routeID)
|
|
}
|
|
|
|
// TestLinkConfig tests configuring physical links.
|
|
func (suite *NetworkConfigSuite) TestLinkConfig() {
|
|
if suite.Cluster == nil || suite.Cluster.Provisioner() != base.ProvisionerQEMU {
|
|
suite.T().Skip("skipping if cluster is not qemu")
|
|
}
|
|
|
|
node := suite.RandomDiscoveredNodeInternalIP(machine.TypeWorker)
|
|
nodeCtx := client.WithNode(suite.ctx, node)
|
|
|
|
suite.T().Logf("testing on node %q", node)
|
|
|
|
// find the first physical link
|
|
links, err := safe.ReaderListAll[*networkres.LinkStatus](nodeCtx, suite.Client.COSI)
|
|
suite.Require().NoError(err)
|
|
|
|
var linkName string
|
|
|
|
for link := range links.All() {
|
|
if link.TypedSpec().Physical() {
|
|
linkName = link.Metadata().ID()
|
|
|
|
break
|
|
}
|
|
}
|
|
|
|
suite.Require().NotEmpty(linkName, "expected to find at least one physical link")
|
|
|
|
cfg := network.NewLinkConfigV1Alpha1(linkName)
|
|
cfg.LinkAddresses = []network.AddressConfig{
|
|
{
|
|
AddressAddress: netip.MustParsePrefix("fd13:1234::2/64"),
|
|
AddressPriority: pointer.To[uint32](2048),
|
|
},
|
|
}
|
|
|
|
addressID := linkName + "/fd13:1234::2/64"
|
|
|
|
suite.PatchMachineConfig(nodeCtx, cfg)
|
|
|
|
rtestutils.AssertResource(nodeCtx, suite.T(), suite.Client.COSI, addressID,
|
|
func(addr *networkres.AddressStatus, asrt *assert.Assertions) {
|
|
asrt.Equal(linkName, addr.TypedSpec().LinkName)
|
|
},
|
|
)
|
|
|
|
suite.RemoveMachineConfigDocumentsByName(nodeCtx, network.LinkKind, linkName)
|
|
|
|
rtestutils.AssertNoResource[*networkres.AddressStatus](nodeCtx, suite.T(), suite.Client.COSI, addressID)
|
|
}
|
|
|
|
func init() {
|
|
allSuites = append(allSuites, new(NetworkConfigSuite))
|
|
}
|