Skip to content

Commit 01f0dae

Browse files
mnasiadkammalchuk
authored andcommitted
Unify curl options
Currently we use couple of curl options throughout Dockerfiles, this change adds all common options to curlrc (-sSLf) and removes usage of those in Dockerfiles. Also fixed conflicts added by I75927b6016bdb7577fbff354632bdab6407f359c and I0bb8b2429906e698858f6b699e4c4d8bacc2dd74 Change-Id: I46b77978926fc2b578a68d1aaa944b2198af0685 (cherry picked from commit 22c32a3) Signed-off-by: Maksim Malchuk <[email protected]>
1 parent 5f35b50 commit 01f0dae

File tree

22 files changed

+24
-22
lines changed

22 files changed

+24
-22
lines changed

docker/base/Dockerfile.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -392,7 +392,7 @@ COPY apt_preferences /etc/apt/preferences.d/kolla-custom
392392
{% endif -%}
393393
{% endfor %}
394394
{% for key in remote_apt_keys | customizable('remote_apt_keys') %}
395-
{%- if loop.first %} RUN {% else %} && {% endif %}curl -L {{ key }} | apt-key add -
395+
{%- if loop.first %} RUN {% else %} && {% endif %}curl {{ key }} | apt-key add -
396396
{%- if not loop.last %} \
397397
{% endif -%}
398398
{% endfor %}

docker/base/curlrc

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
# curl default options
2+
--fail
3+
--location
24
--silent
35
--show-error
46
--write-out "curl (%{url_effective}): response: %{http_code}, time: %{time_total}, size: %{size_download}\n"

docker/fluentd/Dockerfile.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ RUN ln -s /bin/mkdir /usr/bin/mkdir
108108
ARG monasca_output_plugin_tag=1.0.2
109109
ARG monasca_output_plugin_url=https://github.com/monasca/fluentd-monasca/archive/$monasca_output_plugin_tag.tar.gz
110110

111-
RUN curl -sSL $monasca_output_plugin_url -o /tmp/fluentd-monasca.tar.gz \
111+
RUN curl $monasca_output_plugin_url -o /tmp/fluentd-monasca.tar.gz \
112112
&& tar -xvf /tmp/fluentd-monasca.tar.gz -C /tmp \
113113
&& cd /tmp/fluentd-monasca-* \
114114
{%- if fluentd_user == 'fluentd' %}

docker/kafka/Dockerfile.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ ENV kafka_pkg_sha512sum=9773a85ef2898b4bae20481df4cfd5488bd195fffd700fcc874a9fa5
3636
{% endblock %}
3737

3838
{% block kafka_install %}
39-
RUN curl -sSL -o /tmp/kafka.tgz ${kafka_url} \
39+
RUN curl -o /tmp/kafka.tgz ${kafka_url} \
4040
&& echo "${kafka_pkg_sha512sum} /tmp/kafka.tgz" | sha512sum -c \
4141
&& tar --strip 1 -xvf /tmp/kafka.tgz -C /opt/kafka \
4242
&& rm -f /tmp/kafka.tgz

docker/kolla-toolbox/Dockerfile.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
7474

7575
{% block kolla_toolbox_upper_constraints %}
7676
RUN mkdir -p /requirements \
77-
&& curl -sSL -o /requirements/upper-constraints.txt https://releases.openstack.org/constraints/upper/{{ openstack_release }} \
77+
&& curl -o /requirements/upper-constraints.txt https://releases.openstack.org/constraints/upper/{{ openstack_release }} \
7878
&& {{ macros.install_pip(kolla_toolbox_pip_virtualenv_packages | customizable("pip_virtualenv_packages")) }} \
7979
&& virtualenv --system-site-packages {{ virtualenv_path }}
8080
{% endblock %}

docker/mariadb/mariadb-clustercheck/Dockerfile.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
2020
{{ macros.install_packages(mariadb_clustercheck_packages | customizable("packages")) }}
2121

2222
{% if base_package_type == 'deb' %}
23-
RUN curl -sSL -o /usr/bin/clustercheck https://src.fedoraproject.org/rpms/mariadb/raw/10.3/f/clustercheck.sh \
23+
RUN curl -o /usr/bin/clustercheck https://src.fedoraproject.org/rpms/mariadb/raw/10.3/f/clustercheck.sh \
2424
&& chmod 755 /usr/bin/clustercheck
2525
{% endif %}
2626

docker/monasca/monasca-grafana/Dockerfile.j2

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ ARG monasca_grafana_app_url=https://github.com/monasca/monasca-grafana/archive/$
5757
# NPM installs itself in /usr/local/bin (the default in Ubuntu) which is not in the PATH.
5858
# This is forced for all distros to avoid conflicts with native packages.
5959
RUN gem install fpm \
60-
&& curl -sSL -o /tmp/monasca-grafana.tgz ${monasca_grafana_url} \
60+
&& curl -o /tmp/monasca-grafana.tgz ${monasca_grafana_url} \
6161
&& mkdir -p ${monasca_grafana_build_path} \
6262
&& tar --strip 1 -xvf /tmp/monasca-grafana.tgz -C ${monasca_grafana_build_path} \
6363
&& rm -f /tmp/monasca-grafana.tgz \
@@ -89,7 +89,7 @@ RUN grafana-cli plugins install monasca-datasource \
8989
&& chmod 755 /usr/local/bin/kolla_extend_start
9090

9191
RUN mkdir -p /var/lib/grafana/plugins/monasca-grafana \
92-
&& curl -Lo tmp.tgz ${monasca_grafana_app_url} \
92+
&& curl -o tmp.tgz ${monasca_grafana_app_url} \
9393
&& tar zxvf tmp.tgz -C /var/lib/grafana/plugins/monasca-grafana --strip-components=1 \
9494
&& rm tmp.tgz \
9595
&& npm install -g grunt \

docker/prometheus/prometheus-alertmanager/Dockerfile.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ ENV prometheus_alertmanager_version=0.20.0
1010
{% endblock %}
1111

1212
{% block prometheus_alertmanager_install %}
13-
RUN curl -sSL -o /tmp/prometheus_alertmanager.tar.gz https://github.com/prometheus/alertmanager/releases/download/v${prometheus_alertmanager_version}/alertmanager-${prometheus_alertmanager_version}.linux-{{debian_arch}}.tar.gz \
13+
RUN curl -o /tmp/prometheus_alertmanager.tar.gz https://github.com/prometheus/alertmanager/releases/download/v${prometheus_alertmanager_version}/alertmanager-${prometheus_alertmanager_version}.linux-{{debian_arch}}.tar.gz \
1414
&& tar xvf /tmp/prometheus_alertmanager.tar.gz -C /opt/ \
1515
&& rm -f /tmp/prometheus_alertmanager.tar.gz \
1616
&& ln -s /opt/alertmanager* /opt/prometheus_alertmanager

docker/prometheus/prometheus-blackbox-exporter/Dockerfile.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ ENV blackbox_exporter_version=0.16.0
2424
{{ macros.install_packages(blackbox_packages | customizable("packages")) }}
2525

2626
{% block prometheus_blackbox_exporter_install %}
27-
RUN curl -sSL -o /tmp/blackbox_exporter.tar.gz https://github.com/prometheus/blackbox_exporter/releases/download/v${blackbox_exporter_version}/blackbox_exporter-${blackbox_exporter_version}.linux-{{debian_arch}}.tar.gz \
27+
RUN curl -o /tmp/blackbox_exporter.tar.gz https://github.com/prometheus/blackbox_exporter/releases/download/v${blackbox_exporter_version}/blackbox_exporter-${blackbox_exporter_version}.linux-{{debian_arch}}.tar.gz \
2828
&& tar xvf /tmp/blackbox_exporter.tar.gz -C /opt/ \
2929
&& rm -f /tmp/blackbox_exporter.tar.gz \
3030
&& ln -s /opt/blackbox_exporter* /opt/blackbox_exporter

docker/prometheus/prometheus-cadvisor/Dockerfile.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ ENV prometheus_cadvisor_version=0.35.0
1414
{% if base_arch in ['x86_64'] %}
1515

1616
{% block prometheus_cadvisor_install %}
17-
RUN curl -ssL -o /opt/cadvisor https://github.com/google/cadvisor/releases/download/v${prometheus_cadvisor_version}/cadvisor && chmod 0755 /opt/cadvisor
17+
RUN curl -o /opt/cadvisor https://github.com/google/cadvisor/releases/download/v${prometheus_cadvisor_version}/cadvisor && chmod 0755 /opt/cadvisor
1818
{% endblock %}
1919

2020
{% else %}

0 commit comments

Comments
 (0)