Skip to content

Commit fa547a2

Browse files
authored
Use curl timeout over systemd-run to prevent possible network timeouts (#628)
* Remove incompatible options from `systemd-run` * Use `curl` timeout * Fix commit mistake
1 parent 28ce781 commit fa547a2

File tree

5 files changed

+10
-15
lines changed

5 files changed

+10
-15
lines changed

environment/setup.sh

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,8 @@ if [ ! -f "$FILE" ]; then
1818
sudo usermod -aG docker ubuntu
1919
sudo setfacl --modify user:ubuntu:rw /var/run/docker.sock
2020

21-
get_ecr_helper="curl https://amazon-ecr-credential-helper-releases.s3.us-east-2.amazonaws.com/0.5.0/linux-amd64/docker-credential-ecr-login --output /usr/bin/docker-credential-ecr-login"
22-
chmod_ecr_helper="chmod 755 /usr/bin/docker-credential-ecr-login"
23-
sudo systemd-run --same-dir --no-block --service-type=exec bash -c "$get_ecr_helper && $chmod_ecr_helper"
21+
curl --max-time 10 --output /usr/bin/docker-credential-ecr-login https://amazon-ecr-credential-helper-releases.s3.us-east-2.amazonaws.com/0.5.0/linux-amd64/docker-credential-ecr-login
22+
chmod a+x /usr/bin/docker-credential-ecr-login
2423

2524
curl -fsSL https://apt.releases.hashicorp.com/gpg | sudo apt-key add -
2625
sudo apt-add-repository "deb [arch=amd64] https://apt.releases.hashicorp.com $(lsb_release -cs) main"

iterative/testdata/script_template_cloud_aws.golden

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,8 @@ if [ ! -f "$FILE" ]; then
1919
sudo usermod -aG docker ubuntu
2020
sudo setfacl --modify user:ubuntu:rw /var/run/docker.sock
2121

22-
get_ecr_helper="curl https://amazon-ecr-credential-helper-releases.s3.us-east-2.amazonaws.com/0.5.0/linux-amd64/docker-credential-ecr-login --output /usr/bin/docker-credential-ecr-login"
23-
chmod_ecr_helper="chmod 755 /usr/bin/docker-credential-ecr-login"
24-
sudo systemd-run --same-dir --no-block --service-type=exec bash -c "$get_ecr_helper && $chmod_ecr_helper"
22+
curl --max-time 10 --output /usr/bin/docker-credential-ecr-login https://amazon-ecr-credential-helper-releases.s3.us-east-2.amazonaws.com/0.5.0/linux-amd64/docker-credential-ecr-login
23+
chmod a+x /usr/bin/docker-credential-ecr-login
2524

2625
curl -fsSL https://apt.releases.hashicorp.com/gpg | sudo apt-key add -
2726
sudo apt-add-repository "deb [arch=amd64] https://apt.releases.hashicorp.com $(lsb_release -cs) main"

iterative/testdata/script_template_cloud_azure.golden

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,8 @@ if [ ! -f "$FILE" ]; then
1919
sudo usermod -aG docker ubuntu
2020
sudo setfacl --modify user:ubuntu:rw /var/run/docker.sock
2121

22-
get_ecr_helper="curl https://amazon-ecr-credential-helper-releases.s3.us-east-2.amazonaws.com/0.5.0/linux-amd64/docker-credential-ecr-login --output /usr/bin/docker-credential-ecr-login"
23-
chmod_ecr_helper="chmod 755 /usr/bin/docker-credential-ecr-login"
24-
sudo systemd-run --same-dir --no-block --service-type=exec bash -c "$get_ecr_helper && $chmod_ecr_helper"
22+
curl --max-time 10 --output /usr/bin/docker-credential-ecr-login https://amazon-ecr-credential-helper-releases.s3.us-east-2.amazonaws.com/0.5.0/linux-amd64/docker-credential-ecr-login
23+
chmod a+x /usr/bin/docker-credential-ecr-login
2524

2625
curl -fsSL https://apt.releases.hashicorp.com/gpg | sudo apt-key add -
2726
sudo apt-add-repository "deb [arch=amd64] https://apt.releases.hashicorp.com $(lsb_release -cs) main"

iterative/testdata/script_template_cloud_gcp.golden

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,8 @@ if [ ! -f "$FILE" ]; then
1919
sudo usermod -aG docker ubuntu
2020
sudo setfacl --modify user:ubuntu:rw /var/run/docker.sock
2121

22-
get_ecr_helper="curl https://amazon-ecr-credential-helper-releases.s3.us-east-2.amazonaws.com/0.5.0/linux-amd64/docker-credential-ecr-login --output /usr/bin/docker-credential-ecr-login"
23-
chmod_ecr_helper="chmod 755 /usr/bin/docker-credential-ecr-login"
24-
sudo systemd-run --same-dir --no-block --service-type=exec bash -c "$get_ecr_helper && $chmod_ecr_helper"
22+
curl --max-time 10 --output /usr/bin/docker-credential-ecr-login https://amazon-ecr-credential-helper-releases.s3.us-east-2.amazonaws.com/0.5.0/linux-amd64/docker-credential-ecr-login
23+
chmod a+x /usr/bin/docker-credential-ecr-login
2524

2625
curl -fsSL https://apt.releases.hashicorp.com/gpg | sudo apt-key add -
2726
sudo apt-add-repository "deb [arch=amd64] https://apt.releases.hashicorp.com $(lsb_release -cs) main"

iterative/testdata/script_template_cloud_invalid.golden

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,8 @@ if [ ! -f "$FILE" ]; then
1919
sudo usermod -aG docker ubuntu
2020
sudo setfacl --modify user:ubuntu:rw /var/run/docker.sock
2121

22-
get_ecr_helper="curl https://amazon-ecr-credential-helper-releases.s3.us-east-2.amazonaws.com/0.5.0/linux-amd64/docker-credential-ecr-login --output /usr/bin/docker-credential-ecr-login"
23-
chmod_ecr_helper="chmod 755 /usr/bin/docker-credential-ecr-login"
24-
sudo systemd-run --same-dir --no-block --service-type=exec bash -c "$get_ecr_helper && $chmod_ecr_helper"
22+
curl --max-time 10 --output /usr/bin/docker-credential-ecr-login https://amazon-ecr-credential-helper-releases.s3.us-east-2.amazonaws.com/0.5.0/linux-amd64/docker-credential-ecr-login
23+
chmod a+x /usr/bin/docker-credential-ecr-login
2524

2625
curl -fsSL https://apt.releases.hashicorp.com/gpg | sudo apt-key add -
2726
sudo apt-add-repository "deb [arch=amd64] https://apt.releases.hashicorp.com $(lsb_release -cs) main"

0 commit comments

Comments
 (0)