ORC-1113: Remove CentOS 8 from docker-based tests (#1040)

dongjoon-hyun · web-flow · commit e62a16bb41fb · 2022-02-01T22:33:27.000-08:00
### What changes were proposed in this pull request? This PR aims to remove CentOS 8 from docker-based tests. ### Why are the changes needed? CentOS 8 became EOL and Docker image build is failing now. - https://www.centos.org/centos-linux-eol/ ``` #6 0.762 CentOS Linux 8 - AppStream 83 B/s | 38 B 00:00 #6 0.767 Error: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist ``` ### How was this patch tested? Manually.
diff --git a/docker/README.md b/docker/README.md
@@ -1,6 +1,6 @@
 ## Supported OSes
 
-* CentOS 7 and 8
+* CentOS 7
 * Debian 9 and 10
 * Ubuntu 18 and 20
 
diff --git a/docker/centos8/Dockerfile b/docker/centos8/Dockerfile
diff --git a/docker/os-list.txt b/docker/os-list.txt
@@ -1,5 +1,4 @@
 centos7
-centos8
 debian9
 debian10
 debian11
diff --git a/site/_docs/building.md b/site/_docs/building.md
@@ -9,7 +9,7 @@ dockerUrl: https://github.com/apache/orc/blob/main/docker
 
 The C++ library is supported on the following operating systems:
 
-* CentOS 7 or 8
+* CentOS 7
 * Debian 9 to 11
 * MacOS 11 to 12.1
 * Ubuntu 18.04 or 20.04
@@ -26,7 +26,6 @@ For each version of Linux, please check the corresponding Dockerfile, which
 is in the docker subdirectory, for the list of packages required to build ORC:
 
 * [CentOS 7]({{ page.dockerUrl }}/centos7/Dockerfile)
-* [CentOS 8]({{ page.dockerUrl }}/centos8/Dockerfile)
 * [Debian 9]({{ page.dockerUrl }}/debian9/Dockerfile)
 * [Debian 10]({{ page.dockerUrl }}/debian10/Dockerfile)
 * [Debian 11]({{ page.dockerUrl }}/debian11/Dockerfile)

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`centos7`
`2`		`-centos8`
`3`	`2`	`debian9`
`4`	`3`	`debian10`
`5`	`4`	`debian11`