jrushford commented on a change in pull request #5938:
URL: https://github.com/apache/trafficcontrol/pull/5938#discussion_r652314341



##########
File path: cache-config/testing/docker/ort_test/Dockerfile
##########
@@ -28,15 +28,17 @@
 # The recommended minimum size for each block devices is 1G.
 # For example, `sudo modprobe brd rd_size=1048576 rd_nr=2`
 
-FROM centos:7
+FROM centos:${RHEL_VERSION:-8}
 MAINTAINER [email protected]
 
+RUN yum -y install epel-release
+
+RUN yum repolist
+
 RUN yum install -y \
-  
https://download.postgresql.org/pub/repos/yum/reporpms/EL-7-x86_64/pgdg-redhat-repo-latest.noarch.rpm
 \
-  epel-release initscripts postgresql13.x86_64 git gcc lua-5.1.4-15.el7 
lua-devel-5.1.4-15.el7 \
-  ImageMagick-c++-devel && \
-  # jq is used in run.sh to update tc-fixtures.json with the ATS RPM version
-  yum install -y jq
+  initscripts git jq \
+  git gcc jq \

Review comment:
       fixed

##########
File path: cache-config/testing/docker/ort_test/Dockerfile
##########
@@ -28,15 +28,17 @@
 # The recommended minimum size for each block devices is 1G.
 # For example, `sudo modprobe brd rd_size=1048576 rd_nr=2`
 
-FROM centos:7
+FROM centos:${RHEL_VERSION:-8}
 MAINTAINER [email protected]
 
+RUN yum -y install epel-release
+
+RUN yum repolist
+
 RUN yum install -y \
-  
https://download.postgresql.org/pub/repos/yum/reporpms/EL-7-x86_64/pgdg-redhat-repo-latest.noarch.rpm
 \
-  epel-release initscripts postgresql13.x86_64 git gcc lua-5.1.4-15.el7 
lua-devel-5.1.4-15.el7 \
-  ImageMagick-c++-devel && \
-  # jq is used in run.sh to update tc-fixtures.json with the ATS RPM version
-  yum install -y jq
+  initscripts git jq \
+  git gcc jq \

Review comment:
       fixed




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to