-
Notifications
You must be signed in to change notification settings - Fork 75
kola/tests: add nested partitions and raid test #699
base: master
Are you sure you want to change the base?
Conversation
bgilbert
left a comment
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.
👍
| // creating them | ||
| Run: NestedRootOnRaid, | ||
| ClusterSize: 0, | ||
| Platforms: []string{"qemu"}, |
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_tables should be necessary here.
| Name: "coreos.disk.raid.root", | ||
| }) | ||
| register.Register(®ister.Test{ | ||
| // This test needs additional disks which is only supported on qemu since Ignition |
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