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 power manager #963

Merged
merged 1 commit into from
Apr 19, 2024
Merged

add power manager #963

merged 1 commit into from
Apr 19, 2024

Conversation

HermioneKT
Copy link
Contributor

Summary

This PR uses k8 Power Manager to tune CPU frequency and run experiments (workload sensitivity and internode scaling) to investigate how tuning frequencies can help to optimise power consumption in serverless computing

Implementation Notes ⚒️

set up K8 power manager (power profiles, power workloads)
run the experiments using sleeping, spinning, auth, aes benchmarks from vSwarm

External Dependencies 🍀

Breaking API Changes ⚠️

Simply specify none (N/A) if not applicable.

Copy link
Contributor

@leokondrashov leokondrashov left a comment

Choose a reason for hiding this comment

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

Please also fix the spellchecker issue.

docs/power_manager.md Outdated Show resolved Hide resolved
docs/power_manager.md Outdated Show resolved Hide resolved
scripts/power_manager/setup_power_manager.sh Outdated Show resolved Hide resolved
scripts/power_manager/setup_power_manager.sh Show resolved Hide resolved
scripts/power_manager/shared-workload.yaml Outdated Show resolved Hide resolved
examples/power_manager/util.go Outdated Show resolved Hide resolved
docs/power_manager.md Outdated Show resolved Hide resolved
docs/power_manager.md Outdated Show resolved Hide resolved
examples/power_manager/README.md Outdated Show resolved Hide resolved
power_manager/util.go Outdated Show resolved Hide resolved
power_manager/util.go Outdated Show resolved Hide resolved
docs/power_manager.md Outdated Show resolved Hide resolved
docs/power_manager.md Outdated Show resolved Hide resolved
scripts/power_manager/shared-workload.yaml Outdated Show resolved Hide resolved
scripts/power_manager/setup_power_manager.sh Outdated Show resolved Hide resolved
scripts/power_manager/setup_power_manager.sh Outdated Show resolved Hide resolved
@leokondrashov
Copy link
Contributor

Looks better, please fix the linter failure, couple of my comments. And try to use powerprofiles for setting up the nodes.

@leokondrashov
Copy link
Contributor

LGTM

@XunyiiZ Please check that it works (with the downgraded go version) and we can merge it.

@XunyiiZ
Copy link
Collaborator

XunyiiZ commented Apr 11, 2024

LGTM

@XunyiiZ Please check that it works (with the downgraded go version) and we can merge it.

Yes it works

@leokondrashov
Copy link
Contributor

@HermioneKT We also need an entry in changelog.md. Please add.

@leokondrashov
Copy link
Contributor

Please squash your commits and rebase on current main branch.

@lrq619 Please take a look

@HermioneKT HermioneKT force-pushed the add_power_manager branch 2 times, most recently from bc85987 to 39c2d0a Compare April 15, 2024 03:24
Signed-off-by: HermioneKT <hermionegrangerkt@gmail.com>
@leokondrashov leokondrashov merged commit 28fbb7a into main Apr 19, 2024
23 checks passed
@leokondrashov leokondrashov deleted the add_power_manager branch April 19, 2024 04:28
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.

None yet

4 participants