Skip to content
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

Jump to bottom

filterx: otel dict #114

Merged
merged 22 commits into from
May 21, 2024
Merged

filterx: otel dict #114

merged 22 commits into from
May 21, 2024

Conversation

alltilla
Copy link
Member

No description provided.

alltilla added 22 commits May 21, 2024 10:54
@alltilla
otel: rename LogRecord functions to snake_case
cab940c 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: pass const std::string & where possible
b1de55d 
This can save a copy.

Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: add extract_string_from_object() helper fn
41ed14d 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: port LogRecord to implement the Dict interface
bab170d 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: add get_protobuf_message_set_field_count()
3461444 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: implement len() for LogRecord
17f9228 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: add Unset() to ProtobufField
3646630 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: implement unset for LogRecord
9ce814a 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: add IsSet() to ProtobufField
6442d2f 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: implement is_key_set for LogRecord
2ed39cd 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: add iter_on_otel_protobuf_message_fields()
7527ce6 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: implement iter() for LogRecord
45848ae 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: port Resource to implement the Dict interface
7a96897 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: implement the rest of the Dict interface for Resource
0b28f39 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: port Scope to implement the Dict interface
fe4773e 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: implement the rest of the Dict interface for Scope
224c2e2 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
otel: add list and dict factory for Resource, Scope, LogRecord
641035d 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
filterx: add filterx_dict_merge()
a3a11ad 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
filterx: add filterx_list_merge()
003df12 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
filterx: use dict merging in json and otel related functions
6af1e67 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
filterx: use list merging in json and otel related functions
46ed4e8 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@alltilla
filterx: add E2E tests to OTEL Res, Sco, Log <-> JSON convert
7808171 
Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
@tarokkk tarokkk requested a review from bazsi May 21, 2024 08:56
bazsi
bazsi approved these changes May 21, 2024
View reviewed changes
Copy link
Member

@bazsi bazsi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved based on my review in syslog-ng/syslog-ng

@alltilla alltilla merged commit edfed2c into axoflow:main May 21, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bazsi bazsi bazsi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alltilla @bazsi