[GitHub] [accumulo-testing] brianloss commented on a diff in pull request #222: Upgrade Maven, ZooKeeper, and Hadoop versions

[GitBox <gi...@apache.org>]

brianloss commented on code in PR #222:
URL: https://github.com/apache/accumulo-testing/pull/222#discussion_r974202990


##########
contrib/terraform-testing-infrastructure/modules/config-files/templates/initialize_accumulo.sh.tftpl:
##########
@@ -22,7 +22,7 @@ docker run -d --name jaeger \
     -p 6832:6832/udp -p 5778:5778 \
     -p 16686:16686 -p 14268:14268 \
     -p 14250:14250 -p 9411:9411 \
-    jaegertracing/all-in-one:1.29
+    artifactory.adv.evoforge.org/docker-remote/jaegertracing/all-in-one:1.29

Review Comment:
   I don't believe this is a publicly accessible image repository. At least, I'm not able to pull from it.
   ```bash
   $ docker pull artifactory.adv.evoforge.org/docker-remote/jaegertracing/all-in-one:1.29
   Error response from daemon: Get "https://artifactory.adv.evoforge.org/v2/": Failed to lookup host: artifactory.adv.evoforge.org
   ```



##########
contrib/terraform-testing-infrastructure/modules/config-files/templates/initialize_accumulo.sh.tftpl:
##########
@@ -36,7 +36,7 @@ docker run --ulimit nofile=66000:66000 -d \
     -v /data/metrics/grafana:/var/lib/grafana \
     -v ${software_root}/telegraf/conf:/etc/telegraf \
     -v ${software_root}/grafana/dashboards:/etc/grafana/provisioning/dashboards \
-    artlov/docker-telegraf-influxdb-grafana:latest
+    artifactory.adv.evoforge.org/docker-remote/artlov/docker-telegraf-influxdb-grafana:latest

Review Comment:
   Same issue here...



-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org