Skip to content

Adds environment variables for log exporter #3037

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
srikanthccv merged 13 commits into from
Jan 11, 2023
Merged

Adds environment variables for log exporter #3037

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
ab5f6a0
Adds environment variables for log exporter
pridhi-arora Nov 16, 2022
3ddc60d
Add tests for env variables introduced in logger.
pridhi-arora Nov 17, 2022
fccf547
Debugging tests
pridhi-arora Dec 8, 2022
b17d41f
Tried to fix tests
pridhi-arora Dec 12, 2022
9447d0e
Finding out properties of log exporter
pridhi-arora Dec 12, 2022
e10fed6
Fixed the logger tests finally
pridhi-arora Dec 15, 2022
7307117
Fixes lint
pridhi-arora Dec 15, 2022
66a0787
Update ChangeLog
pridhi-arora Dec 15, 2022
dbfa4c2
Updated http log env tests
pridhi-arora Dec 15, 2022
17aab8d
Merge branch 'main' into logenv
srikanthccv Jan 2, 2023
a22506d
Merge branch 'main' into logenv
srikanthccv Jan 8, 2023
d6748e7
Merge branch 'main' into logenv
srikanthccv Jan 11, 2023
d8f2405
Updated opetelemetry sdk version
pridhi-arora Jan 11, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Tried to fix tests
  • Loading branch information
@pridhi-arora
pridhi-arora committed Dec 12, 2022
commit b17d41f416536c2b2ceea082c979d941d413ed30
34 changes: 17 additions & 17 deletions exporter/opentelemetry-exporter-otlp-proto-grpc/tests/logs/test_otlp_logs_exporter.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,8 +111,8 @@ def Export(self, request, context):


class TestOTLPLogExporter(TestCase):
def setUp(self):

def setUp(self):
self.exporter = OTLPLogExporter()

self.server = server(ThreadPoolExecutor(max_workers=10))
Expand Down Expand Up @@ -182,26 +182,26 @@ def test_exporting(self):
OTEL_EXPORTER_OTLP_LOGS_ENDPOINT: "logs:4317",
OTEL_EXPORTER_OTLP_LOGS_CERTIFICATE: THIS_DIR
+ "/fixtures/test.cert",
OTEL_EXPORTER_OTLP_LOGS_HEADERS: " key1=value1,KEY2 = value=2",
OTEL_EXPORTER_OTLP_LOGS_HEADERS: " key1=value1,KEY2 = VALUE=2",
OTEL_EXPORTER_OTLP_LOGS_TIMEOUT: "10",
OTEL_EXPORTER_OTLP_LOGS_COMPRESSION: "gzip",
},
)
@patch(
"opentelemetry.exporter.otlp.proto.grpc.exporter.OTLPExporterMixin.__init__"
)
def test_env_variables(self, mock_exporter_mixin):
print("*************************")
print(mock_exporter_mixin.call_args_list)
self.assertTrue(len(mock_exporter_mixin.call_args_list) == 1)
_, kwargs = mock_exporter_mixin.call_args_list[0]

self.assertEqual(kwargs["endpoint"], "log:4317")
self.assertEqual(kwargs["headers"], " key1=value1,KEY2 = value=2")
self.assertEqual(kwargs["timeout"], 10)
self.assertEqual(kwargs["compression"], Compression.Gzip)
self.assertIsNotNone(kwargs["credentials"])
self.assertIsInstance(kwargs["credentials"], ChannelCredentials)
def test_env_variables(self):
exporter = OTLPLogExporter()
self.assertEqual(exporter._endpoint, "logs:4317")
self.assertEqual(exporter._timeout, 10)
self.assertEqual(exporter._compression, Compression.Gzip)
self.assertIsNotNone(exporter._credentials)
self.assertIsInstance(exporter._credentials, ChannelCredentials)
self.assertEqual(
exporter._headers,
(
("key1", "value1"),
("key2", "VALUE=2"),
("user-agent", "OTel OTLP Exporter Python/" + __version__),
),
)

@patch(
"opentelemetry.exporter.otlp.proto.grpc.exporter.ssl_channel_credentials"
Expand Down