sensu-settings 1.9.0 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c40db4538a2eb9d126977fc84d1c1a7276517986
-  data.tar.gz: 5f13e4dac37f2b7abda31f79c6580244c50ff6ab
+  metadata.gz: ca8f0399f312c1be636acd8b5e2821e506293e27
+  data.tar.gz: f555802b381e16ee23947c3e384c370b6c865eca
 SHA512:
-  metadata.gz: 694e9e8e2590d42fc060e938226aa9c2460c79fdec5dedec4d83946a1b19370b6bb22a2da5697bd583070573a1e0d1813d76167ca6b7300966e32569838aed79
-  data.tar.gz: 456f4622d3e33dc8bdb00f7703655541edda5939d3f5cb1fd69c089232f124d7fd1c350e592722c6a26b0037f88cfa59211984315bd94580a4be4974ceb12d1b
+  metadata.gz: b797e36e3831943b86bbc9ff9fa38cac622b70a987b35041ed5eb8fe5b8a72db9143037d7e34379872490cd9104f7c8e1899a3b5a7c1aefdaed42d58a872b4d4
+  data.tar.gz: fea30269b1e5d2ee02db99adee8dfcb035fb8bac8ef334d0331137f6e1808a2ded3926dc3f0e91566ecbdc2fc9ea2a9c3d0b1e29ab9bc0c743688d265a4f2a8d

data/lib/sensu/settings/loader.rb

@@ -1,5 +1,6 @@
 require "sensu/settings/validator"
 require "multi_json"
+require "tempfile"
 
 module Sensu
   module Settings
@@ -135,11 +136,26 @@ module Sensu
         end
       end
 
+      # Create a temporary file containing the colon delimited list of
+      # loaded configuration files.
+      #
+      # @return [String] tempfile path.
+      def create_loaded_tempfile!
+        file = Tempfile.new("sensu_loaded_files")
+        file.write(@loaded_files.join(":"))
+        file.close
+        file.path
+      end
+
       # Set Sensu settings related environment variables. This method
-      # currently sets SENSU_CONFIG_FILES, a colon delimited list of
-      # loaded config files.
+      # sets `SENSU_LOADED_TEMPFILE` to a new temporary file path,
+      # a file containing the colon delimited list of loaded
+      # configuration files (using `create_loaded_tempfile!()`. The
+      # environment variable `SENSU_CONFIG_FILES` has been removed,
+      # due to the exec ARG_MAX (E2BIG) error when spawning processes
+      # after loading many configuration files (e.g. > 2000).
       def set_env!
-        ENV["SENSU_CONFIG_FILES"] = @loaded_files.join(":")
+        ENV["SENSU_LOADED_TEMPFILE"] = create_loaded_tempfile!
       end
 
       # Validate the loaded settings.

data/sensu-settings.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |spec|
   spec.name          = "sensu-settings"
-  spec.version       = "1.9.0"
+  spec.version       = "2.0.0"
   spec.authors       = ["Sean Porter"]
   spec.email         = ["portertech@gmail.com"]
   spec.summary       = "The Sensu settings library, loader and validator"

data/spec/loader_spec.rb

@@ -130,7 +130,9 @@ describe "Sensu::Settings::Loader" do
     @loader.load_directory(@config_dir)
     expect(@loader.loaded_files.size).to eq(3)
     @loader.set_env!
-    expect(ENV["SENSU_CONFIG_FILES"].split(":")).to eq(@loader.loaded_files)
+    expect(ENV["SENSU_LOADED_TEMPFILE"]).to match(/sensu_loaded_files/)
+    loaded_files = IO.read(ENV["SENSU_LOADED_TEMPFILE"])
+    expect(loaded_files.split(":")).to eq(@loader.loaded_files)
   end
 
   it "can load settings and determine if certain definitions exist" do

data/spec/settings_spec.rb

@@ -38,7 +38,9 @@ describe "Sensu::Settings" do
     expect(settings[:checks][:merger][:command]).to eq("echo -n merger")
     expect(settings[:checks][:merger][:subscribers]).to eq(["foo", "bar"])
     expect(settings[:checks][:nested][:command]).to eq("true")
-    expect(ENV["SENSU_CONFIG_FILES"].split(":")).to eq(settings.loaded_files)
+    expect(ENV["SENSU_LOADED_TEMPFILE"]).to match(/sensu_loaded_files/)
+    loaded_files = IO.read(ENV["SENSU_LOADED_TEMPFILE"])
+    expect(loaded_files.split(":")).to eq(settings.loaded_files)
     ENV["RABBITMQ_URL"] = nil
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sensu-settings
 version: !ruby/object:Gem::Version
-  version: 1.9.0
+  version: 2.0.0
 platform: ruby
 authors:
 - Sean Porter
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-20 00:00:00.000000000 Z
+date: 2015-06-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json