Skip to content

Commit ba89e35

Browse files
committed
testiso - enable journal forwarding for iscsi test
1 parent 8a56909 commit ba89e35

File tree

1 file changed

+21
-3
lines changed

1 file changed

+21
-3
lines changed

mantle/cmd/kola/testiso.go

+21-3
Original file line numberDiff line numberDiff line change
@@ -393,13 +393,23 @@ func newQemuBuilder(outdir string) (*platform.QemuBuilder, *conf.Conf, error) {
393393
if err != nil {
394394
return nil, nil, err
395395
}
396-
journalPipe, err := builder.VirtioJournal(config, "")
396+
397+
err = forwardJournal(outdir, builder, config)
397398
if err != nil {
398399
return nil, nil, err
399400
}
401+
402+
return builder, config, nil
403+
}
404+
405+
func forwardJournal(outdir string, builder *platform.QemuBuilder, config *conf.Conf) error {
406+
journalPipe, err := builder.VirtioJournal(config, "")
407+
if err != nil {
408+
return err
409+
}
400410
journalOut, err := os.OpenFile(filepath.Join(outdir, "journal.txt"), os.O_WRONLY|os.O_CREATE, 0644)
401411
if err != nil {
402-
return nil, nil, err
412+
return err
403413
}
404414

405415
go func() {
@@ -409,7 +419,7 @@ func newQemuBuilder(outdir string) (*platform.QemuBuilder, *conf.Conf, error) {
409419
}
410420
}()
411421

412-
return builder, config, nil
422+
return nil
413423
}
414424

415425
func newQemuBuilderWithDisk(outdir string) (*platform.QemuBuilder, *conf.Conf, error) {
@@ -1023,12 +1033,20 @@ func testLiveInstalliscsi(ctx context.Context, inst platform.Install, outdir str
10231033
if err != nil {
10241034
return 0, err
10251035
}
1036+
err = forwardJournal(outdir, builder, config)
1037+
if err != nil {
1038+
return 0, err
1039+
}
1040+
10261041
// Add a failure target to stop the test if something go wrong rather than waiting for the 10min timeout
10271042
config.AddSystemdUnit("coreos-test-entered-emergency-target.service", signalFailureUnit, conf.Enable)
10281043

10291044
// enable network
10301045
builder.EnableUsermodeNetworking([]platform.HostForwardPort{}, "")
10311046

1047+
// keep auto-login enabled for easier debug when running console
1048+
config.AddAutoLogin()
1049+
10321050
builder.SetConfig(config)
10331051

10341052
mach, err := builder.Exec()

0 commit comments

Comments
 (0)