feat: automatically append talos.config to the Environment

Fixes #460

With this change, it's no longer necessary to supply `talos.config` in
the Environment - it is appended dynamically during the boot process.

So `talos.config` is removed even from the default Environment, as this
makes it update automatically if Sidero endpoint gets changed.

Signed-off-by: Andrey Smirnov <andrey.smirnov@talos-systems.com>
(cherry picked from commit acb5f57)

app/sidero-controller-manager/api/v1alpha1/environment_types.go

@@ -78,7 +78,6 @@ func EnvironmentDefaultSpec(talosRelease, apiEndpoint string, apiPort uint16) *E
 	args = append(args, kernel.DefaultArgs...)
 	args = append(args, "console=tty0", "console=ttyS0", "earlyprintk=ttyS0")
 	args = append(args, "initrd=initramfs.xz", "talos.platform=metal")
-	args = append(args, fmt.Sprintf("talos.config=http://%s:%d/configdata?uuid=", apiEndpoint, apiPort))
 	sort.Strings(args)
 
 	return &EnvironmentSpec{

app/sidero-controller-manager/internal/ipxe/ipxe_server.go

@@ -25,6 +25,7 @@ import (
 	"sigs.k8s.io/controller-runtime/pkg/client"
 
 	"github.com/talos-systems/go-procfs/procfs"
+	talosconstants "github.com/talos-systems/talos/pkg/machinery/constants"
 
 	infrav1 "github.com/talos-systems/sidero/app/caps-controller-manager/api/v1alpha3"
 	metalv1alpha1 "github.com/talos-systems/sidero/app/sidero-controller-manager/api/v1alpha1"
@@ -364,6 +365,8 @@ func newDefaultEnvironment() (env *metalv1alpha1.Environment, err error) {
 		return nil, err
 	}
 
+	appendTalosConfigArgument(env)
+
 	return env, nil
 }
 
@@ -374,6 +377,8 @@ func newEnvironmentFromServer(server *metalv1alpha1.Server) (env *metalv1alpha1.
 		return nil, err
 	}
 
+	appendTalosConfigArgument(env)
+
 	return env, nil
 }
 
@@ -384,17 +389,39 @@ func newEnvironmentFromServerClass(serverBinding *infrav1.ServerBinding) (env *m
 		return nil, err
 	}
 
-	if serverClassResource.Spec.EnvironmentRef != nil {
-		env = &metalv1alpha1.Environment{}
+	if serverClassResource.Spec.EnvironmentRef == nil {
+		return env, nil
+	}
 
-		if err := c.Get(context.Background(), types.NamespacedName{Namespace: "", Name: serverClassResource.Spec.EnvironmentRef.Name}, env); err != nil {
-			return nil, err
-		}
+	env = &metalv1alpha1.Environment{}
+
+	if err := c.Get(context.Background(), types.NamespacedName{Namespace: "", Name: serverClassResource.Spec.EnvironmentRef.Name}, env); err != nil {
+		return nil, err
 	}
 
+	appendTalosConfigArgument(env)
+
 	return env, nil
 }
 
+func appendTalosConfigArgument(env *metalv1alpha1.Environment) {
+	args := env.Spec.Kernel.Args
+
+	talosConfigPrefix := talosconstants.KernelParamConfig + "="
+
+	for _, arg := range args {
+		if strings.HasPrefix(arg, talosConfigPrefix) {
+			// Environment already has talos.config
+			return
+		}
+	}
+
+	// patch environment with the link to the metadata server
+	env.Spec.Kernel.Args = append(env.Spec.Kernel.Args,
+		fmt.Sprintf("%s=http://%s:%d/configdata?uuid=", talosconstants.KernelParamConfig, apiEndpoint, apiPort),
+	)
+}
+
 func markAsPXEBooted(server *metalv1alpha1.Server) error {
 	patchHelper, err := patch.NewHelper(server, c)
 	if err != nil {

sfyra/pkg/tests/environment.go

@@ -6,7 +6,6 @@ package tests
 
 import (
 	"context"
-	"fmt"
 	"testing"
 	"time"
 
@@ -91,7 +90,6 @@ func TestEnvironmentCreate(ctx context.Context, metalClient client.Client, clust
 			cmdline.Append("panic", "1")
 			cmdline.Append("talos.platform", "metal")
 			cmdline.Append("talos.shutdown", "halt")
-			cmdline.Append("talos.config", fmt.Sprintf("http://%s:8081/configdata?uuid=", cluster.SideroComponentsIP()))
 			cmdline.Append("initrd", "initramfs.xz")
 
 			environment.APIVersion = constants.SideroAPIVersion