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.

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

Update k8s example #348

Merged
merged 1 commit into from
Jul 1, 2021
Merged

Update k8s example #348

merged 1 commit into from
Jul 1, 2021

Conversation

jsirianni
Copy link
Member

@jsirianni jsirianni commented Jun 30, 2021

Description of Changes

  • Switched to kubernetes_container plugin
  • Use stdout and kubectl logs to view stanza output, instead of file_output inside the container
  • Use persistent storage for offset database
  • Switch to latest rbac api

This example can run onMminikube with zero configuration.

Please check that the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • Add a changelog entry (for non-trivial bug fixes / features)
  • CI passes

@jsirianni jsirianni requested a review from schmikei June 30, 2021 22:40
@djaglowski
Copy link
Member

Log Files Logs / Second CPU Avg (%) CPU Avg Δ (%) Memory Avg (MB) Memory Avg Δ (MB)
1 1000 1.4655333 +0.06897259 128.95204 +0.19908142
1 5000 5.155183 -0.17252064 139.2015 +1.8138428
1 10000 9.534603 -0.3796444 144.16487 -0.08552551
1 50000 46.4489 -0.43344498 173.8843 -2.3411865
1 100000 98.91476 +10.6117935 232.83783 +6.5176544
10 100 1.862072 +0.1379385 133.91891 +0.9352112
10 500 6.241338 +0.8964801 138.04836 -2.5196686
10 1000 11.707051 +0.9484129 146.1968 +0.4731903
10 5000 54.44919 +0.29341888 178.64238 +1.2901459
10 10000 97.93902 -9.587624 214.8226 -11.48114

@codecov
Copy link

codecov bot commented Jun 30, 2021

Codecov Report

Merging #348 (d1f5733) into master (c254bee) will decrease coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #348      +/-   ##
==========================================
- Coverage   69.58%   69.45%   -0.12%     
==========================================
  Files         123      123              
  Lines        6495     6495              
==========================================
- Hits         4519     4511       -8     
- Misses       1494     1504      +10     
+ Partials      482      480       -2     
Impacted Files Coverage Δ
operator/builtin/input/tcp/tcp.go 69.30% <0.00%> (-4.39%) ⬇️
operator/builtin/output/forward/forward.go 55.07% <0.00%> (-4.35%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c254bee...d1f5733. Read the comment docs.

@jsirianni jsirianni merged commit 1f09156 into master Jul 1, 2021
@jsirianni jsirianni deleted the k8s-example branch July 1, 2021 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants