Only run checkpoint/restore tests on Fedora >= 29
Only Fedora 29 and newer has an updated container-selinux and selinux-policy new enough to support CRIU in restoring threaded processes in a container with SELinux enabled. Also skip checkpoint/restore tests if rootless. CRIU requires root. Signed-off-by: Adrian Reber <areber@redhat.com>
This commit is contained in:
		
							parent
							
								
									5db7b17d79
								
							
						
					
					
						commit
						76df31830c
					
				|  | @ -21,6 +21,7 @@ var _ = Describe("Podman checkpoint", func() { | ||||||
| 	) | 	) | ||||||
| 
 | 
 | ||||||
| 	BeforeEach(func() { | 	BeforeEach(func() { | ||||||
|  | 		SkipIfRootless() | ||||||
| 		tempdir, err = CreateTempDirInTempDir() | 		tempdir, err = CreateTempDirInTempDir() | ||||||
| 		if err != nil { | 		if err != nil { | ||||||
| 			os.Exit(1) | 			os.Exit(1) | ||||||
|  | @ -41,11 +42,12 @@ var _ = Describe("Podman checkpoint", func() { | ||||||
| 		if !criu.CheckForCriu() { | 		if !criu.CheckForCriu() { | ||||||
| 			Skip("CRIU is missing or too old.") | 			Skip("CRIU is missing or too old.") | ||||||
| 		} | 		} | ||||||
| 		// TODO: Remove the skip when the current CRIU SELinux problem is solved.
 | 		// Only Fedora 29 and newer has a new enough selinux-policy and
 | ||||||
| 		// See: https://github.com/containers/libpod/issues/2334
 | 		// container-selinux package to support CRIU in correctly
 | ||||||
|  | 		// restoring threaded processes
 | ||||||
| 		hostInfo := podmanTest.Host | 		hostInfo := podmanTest.Host | ||||||
| 		if hostInfo.Distribution == "fedora" { | 		if hostInfo.Distribution == "fedora" && hostInfo.Version < "29" { | ||||||
| 			Skip("Checkpointing containers on Fedora currently broken.") | 			Skip("Checkpoint/Restore with SELinux only works on Fedora >= 29") | ||||||
| 		} | 		} | ||||||
| 
 | 
 | ||||||
| 	}) | 	}) | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue