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

Metrics #27

Merged
merged 16 commits into from
Jul 27, 2020
Merged

Metrics #27

merged 16 commits into from
Jul 27, 2020

Conversation

plamenmpetrov
Copy link
Collaborator

No description provided.

@plamenmpetrov
Added metrics module
55dbdeb 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
@plamenmpetrov
Updated ctriface with metrics module
90f3571 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
ustiugov
ustiugov reviewed Jul 22, 2020
View reviewed changes
Copy link
Member

@ustiugov ustiugov left a comment

Choose a reason for hiding this comment

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

looks good, think of integrating into the orchestrator (ctriface?)

@plamenmpetrov
Extended metrics module
a9d6974 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
@plamenmpetrov
Updated metrics for ctriface
e9f1a1f 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
@plamenmpetrov
Extended metrics with Serve stats
96d5e1c 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
ustiugov
ustiugov requested changes Jul 23, 2020
View reviewed changes
ctriface/bench_test.go Outdated Show resolved Hide resolved
@plamenmpetrov
Serve benchmarking
d81b19a 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
@ustiugov
Copy link
Member

can you please do the following:

  • in the benchmark (before running), create a folder bench_results/ in the current directory
  • instead of running only helloworld, run all available functions (take them from TestAllFunctions) in a loop
  • for every function, create a separate file called <FUNCNAME>.txt
  • add same benchmarks that would run 4 functions of the same kind in goroutines (e.g., Load+Resume of 4 independent helloworlds at a time)

@plamenmpetrov
Resume Metrics
4128c20 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
@plamenmpetrov
Measure all functions in ctriface
841ab08 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
ustiugov
ustiugov requested changes Jul 24, 2020
View reviewed changes
ctriface/bench_test.go Outdated Show resolved Hide resolved
functions.go Outdated Show resolved Hide resolved
functions.go Outdated Show resolved Hide resolved
functions.go Outdated Show resolved Hide resolved
functions.go Outdated Show resolved Hide resolved
metrics/resumeVMStat.go Outdated Show resolved Hide resolved
metrics/serveStat.go Outdated Show resolved Hide resolved
metrics/startVMStat.go Outdated Show resolved Hide resolved
metrics/startVMStat.go Outdated Show resolved Hide resolved
metrics/startVMStat.go Outdated Show resolved Hide resolved
@plamenmpetrov
top module tests
62c5b35 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
@ustiugov
Copy link
Member

  • ignore comments inside metrics
  • add parallel function start and load benchmarks

@plamenmpetrov
Updated metrics to map
b838e9b 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
@plamenmpetrov
ctriface metrics
3da0319 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
ustiugov
ustiugov requested changes Jul 27, 2020
View reviewed changes
bench_test.go Outdated Show resolved Hide resolved
ctriface/Makefile Outdated Show resolved Hide resolved
ctriface/iface.go Outdated Show resolved Hide resolved
ctriface/iface.go Show resolved Hide resolved
ctriface/iface.go Show resolved Hide resolved
functions.go Outdated Show resolved Hide resolved
functions.go Show resolved Hide resolved
functions.go Outdated Show resolved Hide resolved
functions.go Outdated Show resolved Hide resolved
metrics/metrics.go Show resolved Hide resolved
@plamenmpetrov
Rename GetResponse to FuncInvocation
ea27568 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
@plamenmpetrov
ctriface metrics, added checks
f2d08f1 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
@plamenmpetrov
Top module metrics
9e3ef3c 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
ustiugov
ustiugov previously approved these changes Jul 27, 2020
View reviewed changes
@plamenmpetrov
Parallel tests
d9e659d 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
ustiugov
ustiugov previously approved these changes Jul 27, 2020
View reviewed changes
@plamenmpetrov
test names
f55a58a 
Signed-off-by: Plamen Petrov <plamb0brt@gmail.com>
@ustiugov ustiugov self-requested a review July 27, 2020 15:42
@ustiugov ustiugov merged commit 31291ae into master Jul 27, 2020
@ustiugov ustiugov deleted the metrics branch November 30, 2020 20:29
HermioneKT pushed a commit that referenced this pull request Jan 31, 2024
@KTCyber @HermioneKT
# This is a combination of 154 commits.
6674807 
# This is the 1st commit message:

refactor

# This is the commit message #2:

modify powerNodeSelector

# This is the commit message #3:

modify powerNodeSelector

# This is the commit message #4:

modify powerNodeSelector

# This is the commit message #5:

modify namespace

# This is the commit message #6:

modify namespace

# This is the commit message #7:

add namespace

# This is the commit message #8:

init power manager script

# This is the commit message #9:

add go code

# This is the commit message #10:

test

# This is the commit message #11:

test

# This is the commit message #12:

test

# This is the commit message #13:

test

# This is the commit message #14:

go.sum

# This is the commit message #15:

go sum

# This is the commit message #16:

go mod tidy

# This is the commit message #17:

test

# This is the commit message #18:

test

# This is the commit message #19:

test

# This is the commit message #20:

test

# This is the commit message #21:

test

# This is the commit message #22:

test

# This is the commit message #23:

test

# This is the commit message #24:

test

# This is the commit message #25:

test

# This is the commit message #26:

test

# This is the commit message #27:

test

# This is the commit message #28:

test

# This is the commit message #29:

print output

# This is the commit message #30:

test

# This is the commit message #31:

test

# This is the commit message #32:

test

# This is the commit message #33:

test

# This is the commit message #34:

test

# This is the commit message #35:

test

# This is the commit message #36:

test

# This is the commit message #37:

test

# This is the commit message #38:

test

# This is the commit message #39:

test

# This is the commit message #40:

refactor

# This is the commit message #41:

change val

# This is the commit message #42:

test

# This is the commit message #43:

test

# This is the commit message #44:

test

# This is the commit message #45:

exp1

# This is the commit message #46:

add url

# This is the commit message #47:

test

# This is the commit message #48:

test

# This is the commit message #49:

test auth

# This is the commit message #50:

test

# This is the commit message #51:

test

# This is the commit message #52:

test

# This is the commit message #53:

test

# This is the commit message #54:

test

# This is the commit message #55:

fix bug

# This is the commit message #56:

remove log

# This is the commit message #57:

test

# This is the commit message #58:

test

# This is the commit message #59:

test

# This is the commit message #60:

fix loop

# This is the commit message #61:

increase invocation

# This is the commit message #62:

test

# This is the commit message #63:

test

# This is the commit message #64:

test

# This is the commit message #65:

test

# This is the commit message #66:

test

# This is the commit message #67:

test

# This is the commit message #68:

test

# This is the commit message #69:

Test

# This is the commit message #70:

test

# This is the commit message #71:

test

# This is the commit message #72:

test

# This is the commit message #73:

test

# This is the commit message #74:

test

# This is the commit message #75:

test

# This is the commit message #76:

test

# This is the commit message #77:

test

# This is the commit message #78:

test

# This is the commit message #79:

test

# This is the commit message #80:

test

# This is the commit message #81:

test

# This is the commit message #82:

test

# This is the commit message #83:

test

# This is the commit message #84:

test

# This is the commit message #85:

test

# This is the commit message #86:

test

# This is the commit message #87:

test

# This is the commit message #88:

test

# This is the commit message #89:

test

# This is the commit message #90:

test1

# This is the commit message #91:

test

# This is the commit message #92:

test

# This is the commit message #93:

test

# This is the commit message #94:

test

# This is the commit message #95:

test

# This is the commit message #96:

test

# This is the commit message #97:

test

# This is the commit message #98:

test

# This is the commit message #99:

test

# This is the commit message #100:

test

# This is the commit message #101:

test

# This is the commit message #102:

test

# This is the commit message #103:

test

# This is the commit message #104:

test

# This is the commit message #105:

test

# This is the commit message #106:

test

# This is the commit message #107:

test

# This is the commit message #108:

test

# This is the commit message #109:

test1

# This is the commit message #110:

test

# This is the commit message #111:

test

# This is the commit message #112:

Test

# This is the commit message #113:

test

# This is the commit message #114:

test

# This is the commit message #115:

test

# This is the commit message #116:

test

# This is the commit message #117:

test

# This is the commit message #118:

test

# This is the commit message #119:

Test

# This is the commit message #120:

Test

# This is the commit message #121:

test

# This is the commit message #122:

test

# This is the commit message #123:

Test

# This is the commit message #124:

test

# This is the commit message #125:

test

# This is the commit message #126:

test

# This is the commit message #127:

test

# This is the commit message #128:

test

# This is the commit message #129:

test

# This is the commit message #130:

test

# This is the commit message #131:

test

# This is the commit message #132:

test

# This is the commit message #133:

test

# This is the commit message #134:

test

# This is the commit message #135:

test

# This is the commit message #136:

test

# This is the commit message #137:

test

# This is the commit message #138:

test

# This is the commit message #139:

test

# This is the commit message #140:

test

# This is the commit message #141:

test

# This is the commit message #142:

test

# This is the commit message #143:

Test

# This is the commit message #144:

Test

# This is the commit message #145:

Test

# This is the commit message #146:

Test

# This is the commit message #147:

test

# This is the commit message #148:

test

# This is the commit message #149:

test

# This is the commit message #150:

test

# This is the commit message #151:

test

# This is the commit message #152:

test

# This is the commit message #153:

test

# This is the commit message #154:

Test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ustiugov ustiugov ustiugov 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
@plamenmpetrov @ustiugov