Store ints as ints in the configuration object

Fixes open-telemetry#1118

Author: ocelotl

Diff for: opentelemetry-api/CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## Unreleased
 
+- Store `int`s as `int`s in the global Configuration object
+  ([#1118](https://github.com/open-telemetry/opentelemetry-python/pull/1118))
 - Refactor `SpanContext.is_valid` from a method to a data attribute
   ([#1005](https://github.com/open-telemetry/opentelemetry-python/pull/1005))
 - Moved samplers from API to SDK

Diff for: opentelemetry-api/src/opentelemetry/configuration/__init__.py

@@ -128,11 +128,10 @@ def __new__(cls) -> "Configuration":
                         try:
                             value = int(value_str)
                         except ValueError:
-                            pass
-                        try:
-                            value = float(value_str)
-                        except ValueError:
-                            pass
+                            try:
+                                value = float(value_str)
+                            except ValueError:
+                                pass
 
                     instance._config_map[key] = value
 

Diff for: opentelemetry-api/tests/configuration/test_configuration.py

@@ -119,15 +119,18 @@ def test_boolean(self) -> None:
         },
     )
     def test_integer(self) -> None:
+        # pylint: disable=no-member
+        self.assertTrue(isinstance(Configuration().POSITIVE_INTEGER, int))
         self.assertEqual(
             Configuration().POSITIVE_INTEGER, 123
-        )  # pylint: disable=no-member
+        )
+        self.assertTrue(isinstance(Configuration().NEGATIVE_INTEGER, int))
         self.assertEqual(
             Configuration().NEGATIVE_INTEGER, -123
-        )  # pylint: disable=no-member
+        )
         self.assertEqual(
             Configuration().NON_INTEGER, "-12z3"
-        )  # pylint: disable=no-member
+        )
 
     @patch.dict(
         "os.environ",  # type: ignore