Skip to content

Commit c774835

Browse files
committed
test: update talosctl debug air-gapped
While working on another issue, I discovered we can update to use new config format. I couldn't reproduce another issue, so this is the only thing that is left. Signed-off-by: Andrey Smirnov <[email protected]>
1 parent ddd695d commit c774835

File tree

2 files changed

+29
-26
lines changed

2 files changed

+29
-26
lines changed

cmd/talosctl/cmd/mgmt/debug/air-gapped.go

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,9 @@ import (
3030
"golang.org/x/sync/errgroup"
3131

3232
"github.com/siderolabs/talos/pkg/cli"
33+
"github.com/siderolabs/talos/pkg/machinery/config/container"
3334
"github.com/siderolabs/talos/pkg/machinery/config/encoder"
35+
"github.com/siderolabs/talos/pkg/machinery/config/types/security"
3436
"github.com/siderolabs/talos/pkg/machinery/config/types/v1alpha1"
3537
)
3638

@@ -73,21 +75,13 @@ var airgappedCmd = &cobra.Command{
7375
}
7476

7577
func generateConfigPatch(caPEM []byte) error {
76-
patch := &v1alpha1.Config{
78+
patch1 := &v1alpha1.Config{
7779
MachineConfig: &v1alpha1.MachineConfig{
7880
MachineEnv: map[string]string{
7981
"http_proxy": fmt.Sprintf("http://%s", net.JoinHostPort(airgappedFlags.advertisedAddress.String(), strconv.Itoa(airgappedFlags.proxyPort))),
8082
"https_proxy": fmt.Sprintf("http://%s", net.JoinHostPort(airgappedFlags.advertisedAddress.String(), strconv.Itoa(airgappedFlags.proxyPort))),
8183
"no_proxy": fmt.Sprintf("%s/24", airgappedFlags.advertisedAddress.String()),
8284
},
83-
MachineFiles: []*v1alpha1.MachineFile{
84-
{
85-
FilePath: "/etc/ssl/certs/ca-certificates",
86-
FileContent: string(caPEM),
87-
FilePermissions: 0o644,
88-
FileOp: "append",
89-
},
90-
},
9185
},
9286
ClusterConfig: &v1alpha1.ClusterConfig{
9387
ExtraManifests: []string{
@@ -96,7 +90,16 @@ func generateConfigPatch(caPEM []byte) error {
9690
},
9791
}
9892

99-
patchBytes, err := encoder.NewEncoder(patch, encoder.WithComments(encoder.CommentsDisabled)).Encode()
93+
patch2 := security.NewTrustedRootsConfigV1Alpha1()
94+
patch2.MetaName = "air-gapped-ca"
95+
patch2.Certificates = string(caPEM)
96+
97+
ctr, err := container.New(patch1, patch2)
98+
if err != nil {
99+
return err
100+
}
101+
102+
patchBytes, err := ctr.EncodeBytes(encoder.WithComments(encoder.CommentsDisabled))
100103
if err != nil {
101104
return err
102105
}

website/content/v1.10/advanced/developing-talos.md

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -259,29 +259,29 @@ Generated machine configuration patch looks like:
259259

260260
```yaml
261261
machine:
262-
files:
263-
- content: |
264-
-----BEGIN CERTIFICATE-----
265-
MIIBijCCAS+gAwIBAgIBATAKBggqhkjOPQQDAjAUMRIwEAYDVQQKEwlUZXN0IE9u
266-
bHkwHhcNMjIwODA0MTI0MzE0WhcNMjIwODA1MTI0MzE0WjAUMRIwEAYDVQQKEwlU
267-
ZXN0IE9ubHkwWTATBgcqhkjOPQIBBggqhkjOPQMBBwNCAAQfOJdaOFSOI1I+EeP1
268-
RlMpsDZJaXjFdoo5zYM5VYs3UkLyTAXAmdTi7JodydgLhty0pwLEWG4NUQAEvip6
269-
EmzTo3IwcDAOBgNVHQ8BAf8EBAMCBaAwHQYDVR0lBBYwFAYIKwYBBQUHAwEGCCsG
270-
AQUFBwMCMA8GA1UdEwEB/wQFMAMBAf8wHQYDVR0OBBYEFCwxL+BjG0pDwaH8QgKW
271-
Ex0J2mVXMA8GA1UdEQQIMAaHBKwUAAEwCgYIKoZIzj0EAwIDSQAwRgIhAJoW0z0D
272-
JwpjFcgCmj4zT1SbBFhRBUX64PHJpAE8J+LgAiEAvfozZG8Or6hL21+Xuf1x9oh4
273-
/4Hx3jozbSjgDyHOLk4=
274-
-----END CERTIFICATE-----
275-
permissions: 0o644
276-
path: /etc/ssl/certs/ca-certificates
277-
op: append
278262
env:
279263
http_proxy: http://172.20.0.1:8002
280264
https_proxy: http://172.20.0.1:8002
281265
no_proxy: 172.20.0.1/24
282266
cluster:
283267
extraManifests:
284268
- https://172.20.0.1:8001/debug.yaml
269+
---
270+
apiVersion: v1alpha1
271+
kind: TrustedRootsConfig
272+
name: air-gapped-ca
273+
certificates: |
274+
-----BEGIN CERTIFICATE-----
275+
MIIBiTCCAS+gAwIBAgIBATAKBggqhkjOPQQDAjAUMRIwEAYDVQQKEwlUZXN0IE9u
276+
bHkwHhcNMjUwMTE1MTE1OTI3WhcNMjUwMTE2MTE1OTI3WjAUMRIwEAYDVQQKEwlU
277+
ZXN0IE9ubHkwWTATBgcqhkjOPQIBBggqhkjOPQMBBwNCAAReznBeEcQFcB/y1yqI
278+
HQcP0IWBMvgwGTeaaTBM6rV+AjbnyxgCrXAnmJ0t45Eur27eW9J/1T5tzA6fe24f
279+
YyY9o3IwcDAOBgNVHQ8BAf8EBAMCBaAwHQYDVR0lBBYwFAYIKwYBBQUHAwEGCCsG
280+
AQUFBwMCMA8GA1UdEwEB/wQFMAMBAf8wHQYDVR0OBBYEFEGBbafXsyzxVhVqfjzy
281+
7aBmVvtaMA8GA1UdEQQIMAaHBKwUAAEwCgYIKoZIzj0EAwIDSAAwRQIhAPAFm6Lv
282+
1Bw+M55Z1SEDLyILJSS0En5F6n8Q9LyGGT4fAiBi+Fm3wSQcvgGPG9OfokFaXmGp
283+
Pa6c4ZrarKO8ZxWigA==
284+
-----END CERTIFICATE-----
285285
```
286286
287287
The first section appends a self-signed certificate of the HTTPS server to the list of trusted certificates,

0 commit comments

Comments
 (0)