Run http_request test only 1 build in each pipeline

This repository uses matrix to run 50+ builds in a pipeline.
This results in simultaneous requests being sent to httpbin.org,
similar to a DDoS attack, often causing 500 errors and test failures.

e.g. https://github.com/itamae-kitchen/itamae/actions/runs/17045210601/job/48319099479?pr=379

Therefore, I suppressed simultaneous requests to httpbin.org.

Author: sue445

.github/workflows/test.yml

@@ -67,6 +67,15 @@ jobs:
     with:
       ruby: ${{ matrix.ruby }}
       rubyopt: ${{ matrix.rubyopt }}
+
+      # This repository uses matrix to run 50+ builds in a pipeline.
+      # This results in simultaneous requests being sent to httpbin.org,
+      # similar to a DDoS attack, often causing 500 errors and test failures.
+      #
+      # e.g. https://github.com/itamae-kitchen/itamae/actions/runs/17045210601/job/48319099479?pr=379
+      #
+      # Therefore, I suppressed simultaneous requests to httpbin.org.
+      skip_http_request_test: ${{ (matrix.ruby == '2.3' && matrix.rubyopt == '') && 'false' || 'true' }}
     secrets:
       SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
 

.github/workflows/test_main.yml

@@ -10,6 +10,9 @@ on:
         required: false
         type: string
         default: ""
+      skip_http_request_test:
+        required: true
+        type: string
     secrets:
       SLACK_WEBHOOK:
         required: true
@@ -72,6 +75,7 @@ jobs:
       - run: bundle exec rake spec:integration:docker:boot
         env:
           RUBYOPT: ${{ inputs.rubyopt }}
+          SKIP_HTTP_REQUEST_TEST: ${{ inputs.skip_http_request_test }}
 
       - run: bundle exec rake spec:integration:docker:provision
         env:
@@ -83,6 +87,7 @@ jobs:
       - run: bundle exec rake spec:integration:docker:serverspec
         env:
           RUBYOPT: ${{ inputs.rubyopt }}
+          SKIP_HTTP_REQUEST_TEST: ${{ inputs.skip_http_request_test }}
 
       - run: bundle exec rake spec:integration:docker:clean_docker_container
         env:
@@ -120,10 +125,12 @@ jobs:
       - run: bundle exec rake spec:integration:local:main
         env:
           RUBYOPT: ${{ inputs.rubyopt }}
+          SKIP_HTTP_REQUEST_TEST: ${{ inputs.skip_http_request_test }}
 
       - run: bundle exec rake spec:integration:local:ordinary_user
         env:
           RUBYOPT: ${{ inputs.rubyopt }}
+          SKIP_HTTP_REQUEST_TEST: ${{ inputs.skip_http_request_test }}
 
       # TODO: for develop
       # - name: Slack Notification (not success)

Rakefile

@@ -29,7 +29,7 @@ namespace :spec do
     namespace :docker do
       desc "Run docker"
       task :boot do
-        sh "docker run --privileged -d --name #{container_name} #{TEST_IMAGE} /sbin/init"
+        sh "docker run --env SKIP_HTTP_REQUEST_TEST --privileged -d --name #{container_name} #{TEST_IMAGE} /sbin/init"
       end
 
       desc "Run itamae"

spec/integration/default_spec.rb

@@ -77,34 +77,34 @@
   it { should_not be_file }
 end
 
-describe file('/tmp/http_request.html') do
+describe file('/tmp/http_request.html'), unless: ENV["SKIP_HTTP_REQUEST_TEST"] == "true"  do
   it { should be_file }
   its(:content) { should match(/"from":\s*"itamae"/) }
 end
 
-describe file('/tmp/http_request_delete.html') do
+describe file('/tmp/http_request_delete.html'), unless: ENV["SKIP_HTTP_REQUEST_TEST"] == "true"  do
   it { should be_file }
   its(:content) { should match(/"from":\s*"itamae"/) }
 end
 
-describe file('/tmp/http_request_post.html') do
+describe file('/tmp/http_request_post.html'), unless: ENV["SKIP_HTTP_REQUEST_TEST"] == "true"  do
   it { should be_file }
   its(:content) { should match(/"from":\s*"itamae"/) }
   its(:content) { should match(/"love":\s*"sushi"/) }
 end
 
-describe file('/tmp/http_request_put.html') do
+describe file('/tmp/http_request_put.html'), unless: ENV["SKIP_HTTP_REQUEST_TEST"] == "true"  do
   it { should be_file }
   its(:content) { should match(/"from":\s*"itamae"/) }
   its(:content) { should match(/"love":\s*"sushi"/) }
 end
 
-describe file('/tmp/http_request_headers.html') do
+describe file('/tmp/http_request_headers.html'), unless: ENV["SKIP_HTTP_REQUEST_TEST"] == "true"  do
   it { should be_file }
   its(:content) { should match(/"User-Agent":\s*"Itamae"/) }
 end
 
-xdescribe file('/tmp/http_request_redirect.html') do
+xdescribe file('/tmp/http_request_redirect.html'), unless: ENV["SKIP_HTTP_REQUEST_TEST"] == "true"  do
   it { should be_file }
   its(:content) { should match(/"from":\s*"itamae"/) }
 end

spec/integration/ordinary_user_spec.rb

@@ -84,22 +84,22 @@
 
 ###
 
-describe file('/tmp/http_request.html') do
+describe file('/tmp/http_request.html'), unless: ENV["SKIP_HTTP_REQUEST_TEST"] == "true" do
   it { should be_file }
   it { should be_owned_by "ordinary_san" }
   it { should be_grouped_into "ordinary_san" }
   its(:content) { should match(/"from":\s*"itamae"/) }
 end
 
-describe file('/tmp/http_request_root.html') do
+describe file('/tmp/http_request_root.html'), unless: ENV["SKIP_HTTP_REQUEST_TEST"] == "true"  do
   it { should be_file }
   it { should be_owned_by "root" }
   it { should be_grouped_into "root" }
   its(:content) { should match(/"from":\s*"itamae"/) }
 end
 
 %w[/tmp/http_request_another_ordinary.html /tmp/http_request_another_ordinary_with_root.html].each do |path|
-  describe file(path) do
+  describe file(path), unless: ENV["SKIP_HTTP_REQUEST_TEST"] == "true"  do
     it { should be_file }
     it { should be_owned_by "itamae" }
     it { should be_grouped_into "itamae" }

spec/integration/recipes/default.rb

@@ -192,36 +192,38 @@
 
 ######
 
-http_request "/tmp/http_request.html" do
-  url "https://httpbin.org/get?from=itamae"
-end
+unless ENV["SKIP_HTTP_REQUEST_TEST"] == "true"
+  http_request "/tmp/http_request.html" do
+    url "https://httpbin.org/get?from=itamae"
+  end
 
-http_request "/tmp/http_request_delete.html" do
-  action :delete
-  url "https://httpbin.org/delete?from=itamae"
-end
+  http_request "/tmp/http_request_delete.html" do
+    action :delete
+    url "https://httpbin.org/delete?from=itamae"
+  end
 
-http_request "/tmp/http_request_post.html" do
-  action :post
-  message "love=sushi"
-  url "https://httpbin.org/post?from=itamae"
-end
+  http_request "/tmp/http_request_post.html" do
+    action :post
+    message "love=sushi"
+    url "https://httpbin.org/post?from=itamae"
+  end
 
-http_request "/tmp/http_request_put.html" do
-  action :put
-  message "love=sushi"
-  url "https://httpbin.org/put?from=itamae"
-end
+  http_request "/tmp/http_request_put.html" do
+    action :put
+    message "love=sushi"
+    url "https://httpbin.org/put?from=itamae"
+  end
 
-http_request "/tmp/http_request_headers.html" do
-  headers "User-Agent" => "Itamae"
-  url "https://httpbin.org/get"
-end
+  http_request "/tmp/http_request_headers.html" do
+    headers "User-Agent" => "Itamae"
+    url "https://httpbin.org/get"
+  end
 
-# http_request "/tmp/http_request_redirect.html" do
-#   redirect_limit 1
-#   url "https://httpbin.org/redirect-to?url=https%3A%2F%2Fhttpbin.org%2Fget%3Ffrom%3Ditamae"
-# end
+  # http_request "/tmp/http_request_redirect.html" do
+  #   redirect_limit 1
+  #   url "https://httpbin.org/redirect-to?url=https%3A%2F%2Fhttpbin.org%2Fget%3Ffrom%3Ditamae"
+  # end
+end
 
 link "/tmp-link" do
   to "/tmp"

spec/integration/recipes/ordinary_user.rb

@@ -83,27 +83,29 @@
 
 ###
 
-http_request "/tmp/http_request.html" do
-  url "https://httpbin.org/get?from=itamae"
-end
-
-http_request "/tmp/http_request_root.html" do
-  user 'root'
-  owner 'root'
-  group 'root'
-  url "https://httpbin.org/get?from=itamae"
-end
-
-http_request "/tmp/http_request_another_ordinary.html" do
-  user 'itamae'
-  owner 'itamae'
-  group 'itamae'
-  url "https://httpbin.org/get?from=itamae"
-end
-
-http_request "/tmp/http_request_another_ordinary_with_root.html" do
-  user 'root'
-  owner 'itamae'
-  group 'itamae'
-  url "https://httpbin.org/get?from=itamae"
+unless ENV["SKIP_HTTP_REQUEST_TEST"] == "true"
+  http_request "/tmp/http_request.html" do
+    url "https://httpbin.org/get?from=itamae"
+  end
+
+  http_request "/tmp/http_request_root.html" do
+    user 'root'
+    owner 'root'
+    group 'root'
+    url "https://httpbin.org/get?from=itamae"
+  end
+
+  http_request "/tmp/http_request_another_ordinary.html" do
+    user 'itamae'
+    owner 'itamae'
+    group 'itamae'
+    url "https://httpbin.org/get?from=itamae"
+  end
+
+  http_request "/tmp/http_request_another_ordinary_with_root.html" do
+    user 'root'
+    owner 'itamae'
+    group 'itamae'
+    url "https://httpbin.org/get?from=itamae"
+  end
 end