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

feat: oci logging analytics output plugin implementation #7830

Merged
merged 12 commits into from
Sep 22, 2023
Merged

feat: oci logging analytics output plugin implementation #7830

Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
afee9d9
out_oci_logan: plugin implementation
adiforluls Aug 15, 2023
8b98e9c
out_oci_logan: remove unwanted change from cmake lists
adiforluls Aug 15, 2023
43a9374
out_oci_logan: fix merge conflicts
adiforluls Aug 15, 2023
825d622
out_oci_logan: fix issues with metadata fields
adiforluls Aug 16, 2023
502e59b
out_oci_logan: adhere to coding standards and remove cruft
adiforluls Aug 16, 2023
f111da4
out_oci_logan: add copyrights
adiforluls Aug 16, 2023
c6508a8
out_oci_logan: address review comments
adiforluls Aug 21, 2023
5fa7885
out_oci_logan: address review comments
adiforluls Aug 21, 2023
7f44ae6
out_oci_logan: fix leaks and some cleanup
adiforluls Aug 30, 2023
b5a615d
rename plugin
adiforluls Sep 1, 2023
a8443d0
refactor plugin name references
adiforluls Sep 19, 2023
373cfd9
remove unused macros
adiforluls Sep 21, 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
1 change: 1 addition & 0 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -255,6 +255,7 @@ option(FLB_OUT_PROMETHEUS_REMOTE_WRITE "Enable Prometheus remote write plugin"
option(FLB_OUT_S3 "Enable AWS S3 output plugin" Yes)
option(FLB_OUT_VIVO_EXPORTER "Enabel Vivo exporter output plugin" Yes)
option(FLB_OUT_WEBSOCKET "Enable Websocket output plugin" Yes)
option(FLB_OUT_OCI_LOGAN "Enable OCI Logging analytics plugin" Yes)
Copy link
Contributor

Choose a reason for hiding this comment

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

Per the docs PR, I would expand OCI to be Oracle Cloud Infrastructure as it is a fairly generic acronym and may be confused with open container initiative.

Copy link
Member

Choose a reason for hiding this comment

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

what about oracle_log_analytics ? azure has azure_logs ingestion

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah sounds good to me, making it clear it is Oracle I think is the key one - we have AWS/Google equivalents too.

Copy link
Member Author

Choose a reason for hiding this comment

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

Good idea, I'll make these changes

Copy link
Member Author

Choose a reason for hiding this comment

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

Refactored @edsiper

option(FLB_OUT_CHRONICLE "Enable Google Chronicle output plugin" Yes)
option(FLB_FILTER_ALTER_SIZE "Enable alter_size filter" Yes)
option(FLB_FILTER_AWS "Enable aws filter" Yes)
Expand Down
1 change: 1 addition & 0 deletions plugins/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -304,6 +304,7 @@ REGISTER_OUT_PLUGIN("out_nats")
REGISTER_OUT_PLUGIN("out_nrlogs")
REGISTER_OUT_PLUGIN("out_null")
REGISTER_OUT_PLUGIN("out_opensearch")
REGISTER_OUT_PLUGIN("out_oci_logan")

if (NOT CMAKE_SYSTEM_NAME MATCHES "Windows")
REGISTER_OUT_PLUGIN("out_plot")
Expand Down
6 changes: 6 additions & 0 deletions plugins/out_oci_logan/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
set(src
oci_logan.c
oci_logan_conf.c
)

FLB_PLUGIN(out_oci_logan "${src}" "")
Loading