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

Add Gosec to CI #358

Merged
merged 2 commits into from
Jul 8, 2021
Merged

Add Gosec to CI #358

merged 2 commits into from
Jul 8, 2021

Conversation

jsirianni
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Jul 8, 2021

Codecov Report

Merging #358 (2c59312) into master (7d4488e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #358   +/-   ##
=======================================
  Coverage   69.46%   69.46%           
=======================================
  Files         123      123           
  Lines        6525     6525           
=======================================
  Hits         4532     4532           
- Misses       1511     1515    +4     
+ Partials      482      478    -4     
Impacted Files Coverage Δ
operator/builtin/output/otlp/otlp.go 61.73% <0.00%> (ø)

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 7d4488e...2c59312. Read the comment docs.

@djaglowski
Copy link
Member

Log Files Logs / Second CPU Avg (%) CPU Avg Δ (%) Memory Avg (MB) Memory Avg Δ (MB)
1 1000 1.4310582 +0.034497976 128.58783 -0.33485413
1 5000 5.069024 +0.18960428 137.79405 -0.24487305
1 10000 9.983026 +0.32768822 148.62567 +2.3671875
1 50000 49.880245 -2.1380386 176.49258 +3.1641846
1 100000 95.94962 -1.3342896 237.58742 -5.2861023
10 100 1.8276225 -0.24135673 132.3164 -2.261322
10 500 5.8966784 -0.01702261 138.90302 +0.25094604
10 1000 11.621121 -0.4826336 146.49353 +1.9311676
10 5000 55.000862 +0.07850647 176.68602 +0.08418274
10 10000 104.89699 +3.0198593 229.68346 +10.661499

@djaglowski
Copy link
Member

Log Files Logs / Second CPU Avg (%) CPU Avg Δ (%) Memory Avg (MB) Memory Avg Δ (MB)
1 1000 1.5000426 +0.06899178 129.1863 -1.1551666
1 5000 5.0346746 +0.08627033 138.60265 +0.8426819
1 10000 10.103805 +0.37961292 148.99811 +6.2331543
1 50000 50.293537 +1.9820023 171.17915 -1.3244781
1 100000 96.41189 -2.34478 239.47603 +1.0218201
10 100 1.965545 +0.06891489 133.69356 +0.5391998
10 500 5.9311733 +0.12081957 137.39171 -2.4426117
10 1000 12.275976 +0.9654474 148.67255 +2.1697235
10 5000 60.105392 +4.448593 182.96228 +5.111664
10 10000 109.10597 +1.9933853 225.43225 +1.7046051

@jsirianni jsirianni requested a review from schmikei July 8, 2021 17:20
@jsirianni jsirianni merged commit e001332 into master Jul 8, 2021
@jsirianni jsirianni deleted the revert-339-revert-335-gosec branch July 8, 2021 17:40
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.

3 participants