opentelemetry-sdk-extension-aws: make beanstalk detector less chatty

Don't print warnings if we are not running inside beanstalk so we can load the
resource detector more generally and avoid warnings in stderr.

Author: xrmx

sdk-extension/opentelemetry-sdk-extension-aws/src/opentelemetry/sdk/extension/aws/resource/beanstalk.py

@@ -41,6 +41,9 @@ def detect(self) -> "Resource":
         else:
             conf_file_path = "/var/elasticbeanstalk/xray/environment.conf"
 
+        if not os.path.exists(conf_file_path):
+            return Resource.get_empty()
+
         try:
             with open(conf_file_path, encoding="utf-8") as conf_file:
                 parsed_data = json.load(conf_file)

sdk-extension/opentelemetry-sdk-extension-aws/tests/resource/test_beanstalk.py

@@ -41,9 +41,19 @@ class AwsBeanstalkResourceDetectorTest(unittest.TestCase):
         new_callable=mock_open,
         read_data=f'{{"deployment_id":"{MockBeanstalkResourceAttributes[ResourceAttributes.SERVICE_INSTANCE_ID]}","environment_name":"{MockBeanstalkResourceAttributes[ResourceAttributes.SERVICE_NAMESPACE]}","version_label":"{MockBeanstalkResourceAttributes[ResourceAttributes.SERVICE_VERSION]}"}}',
     )
-    def test_simple_create(self, mock_open_function):
+    @patch("os.path.exists", return_value=True)
+    def test_simple_create(self, mock_path_exists, mock_open_function):
         actual = AwsBeanstalkResourceDetector().detect()
         self.assertDictEqual(
             actual.attributes.copy(),
             OrderedDict(MockBeanstalkResourceAttributes),
         )
+
+    @patch("os.name", "posix")
+    @patch("os.path.exists", return_value=False)
+    def test_not_on_beanstalk(self, mock_path_exists):
+        actual = AwsBeanstalkResourceDetector().detect()
+        self.assertDictEqual(actual.attributes.copy(), {})
+        mock_path_exists.assert_called_once_with(
+            "/var/elasticbeanstalk/xray/environment.conf"
+        )