fix review comments

cloud-bulldozer · Oct 15, 2021 · 6188c13 · 6188c13
1 parent 4f87bda
commit 6188c13
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/snafu/benchmarks/coremarkpro/Dockerfile b/snafu/benchmarks/coremarkpro/Dockerfile
@@ -1,5 +1,5 @@
 FROM registry.access.redhat.com/ubi8:latest
-MAINTAINER Sai Sindhur Malleni <smalleni@redhat.org>
+MAINTAINER Ed Chong <edchong@redhat.com>
 
 RUN dnf install -y --nodocs make git gcc && dnf clean all
 RUN git clone https://github.com/eembc/coremark-pro.git && cd coremark-pro && ls util/make && make TARGET=linux64 build
@@ -8,6 +8,5 @@ WORKDIR /output/
 RUN dnf install -y --nodocs python3.8 python38-devel procps-ng iproute net-tools ethtool nmap iputils && dnf clean all
 RUN ln -s /usr/bin/python3 /usr/bin/python
 RUN pip3 install --upgrade pip
-#RUN echo -e '[global]\nextra-index-url=https://www.piwheels.org/simple' > /etc/pip.conf
 COPY . /opt/snafu
 RUN pip3 install -r /opt/snafu/requirements/py38-reqs/install.txt -e /opt/snafu
diff --git a/snafu/benchmarks/coremarkpro/__init__.py b/snafu/benchmarks/coremarkpro/__init__.py
@@ -1,5 +1,5 @@
 #!/usr/bin/env python3
 # -*- coding: utf-8 -*-
 # flake8: noqa
-"""Uperf benchmark"""
+"""Coremarkpro benchmark"""
 from snafu.benchmarks.coremarkpro.coremarkpro import Coremarkpro
diff --git a/snafu/benchmarks/coremarkpro/ci_test.sh b/snafu/benchmarks/coremarkpro/ci_test.sh
@@ -8,4 +8,3 @@ source ci/common.sh
 image_spec=$SNAFU_WRAPPER_IMAGE_PREFIX/coremark-pro:$SNAFU_IMAGE_TAG
 build_and_push snafu/coremarkpro/Dockerfile $image_spec
 pushd ripsaw
-source tests/test_coremarkpro.sh