Skip to content

Commit

Permalink
fix: align process command arguments to spec defenition
Browse files Browse the repository at this point in the history
  • Loading branch information
tamirdavid authored and Tamir David committed Jul 8, 2024
1 parent de8b9e0 commit 21f6c64
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -348,7 +348,7 @@ def detect(self) -> "Resource":
_process_executable_path = os.path.dirname(_process_executable_name)
_process_command = sys.argv[0]
_process_command_line = " ".join(sys.argv)
_process_command_args = sys.argv[1:]
_process_command_args = sys.argv
resource_info = {
PROCESS_RUNTIME_DESCRIPTION: sys.version,
PROCESS_RUNTIME_NAME: sys.implementation.name,
Expand Down
2 changes: 1 addition & 1 deletion opentelemetry-sdk/tests/resources/test_resources.py
Original file line number Diff line number Diff line change
Expand Up @@ -608,7 +608,7 @@ def test_process_detector(self):
)
self.assertEqual(
aggregated_resource.attributes[PROCESS_COMMAND_ARGS],
tuple(sys.argv[1:]),
tuple(sys.argv),
)

def test_resource_detector_entry_points_default(self):
Expand Down

0 comments on commit 21f6c64

Please sign in to comment.