-
Notifications
You must be signed in to change notification settings - Fork 76
kola/tests: add nested partitions and raid test #699
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
kola/tests/misc/raid.go
Outdated
Run: NestedRootOnRaid, | ||
ClusterSize: 0, | ||
Platforms: []string{"qemu"}, | ||
MinVersion: semver.Version{Major: 1520}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will need to be updated, of course.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1534 good? It's dependent on Ignition changes (which will require a ignition bump in the overlay).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like a reasonable guess. If it turns out to be wrong, the test will start failing and we'll bump again.
kola/tests/misc/raid.go
Outdated
nestedRaidRootUserData = conf.ContainerLinuxConfig(`storage: | ||
disks: | ||
- device: /dev/vdb | ||
wipe_table: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
None of the wipe_table
s should be necessary here.
kola/tests/misc/raid.go
Outdated
@@ -80,6 +152,18 @@ func init() { | |||
Name: "coreos.disk.raid.root", | |||
}) | |||
register.Register(®ister.Test{ | |||
// This test needs additional disks which is only supported on qemu since Ignition | |||
// does not support creating partitions on disks without wiping the partition table |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
stale copypaste
0b2e509
to
138e4f2
Compare
Depends on coreos/ignition#434