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

Config changes #80

Merged
merged 2 commits into from
Apr 25, 2024
Merged

Config changes #80

merged 2 commits into from
Apr 25, 2024

Conversation

rafsalas19
Copy link
Collaborator

  • Unify Configuration files to Single top directory config
  • Add GPU sample rate and profiling as config settings and removing them as arguments
  • Modify scripts and documentation for the new config path and settings
  • bug fixes

@rafsalas19 rafsalas19 marked this pull request as ready for review April 24, 2024 21:30
LiquidPT
LiquidPT reviewed Apr 24, 2024
View reviewed changes
dockerfile/moneo-exporter-nvidia_entrypoint.sh Show resolved Hide resolved
moneo_config.json Outdated Show resolved Hide resolved
linux_service/moneo_service_deploy.sh Show resolved Hide resolved
src/worker/exporters/nvidia_exporter.py Outdated Show resolved Hide resolved
src/worker/start_geneva.sh Show resolved Hide resolved
moneo.py Outdated Show resolved Hide resolved
Adding/refactoring unified config
58c42b4 
fixes to the conf functionality

lint fix

add configuration data to readme

address pr comments
@rafsalas19 rafsalas19 requested a review from LiquidPT April 25, 2024 15:33
@rafsalas19 rafsalas19 merged commit 845ebe2 into Azure:main Apr 25, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LiquidPT LiquidPT LiquidPT approved these changes

@jithinjosepkl jithinjosepkl Awaiting requested review from jithinjosepkl

@RyoYang RyoYang Awaiting requested review from RyoYang

@moes1 moes1 Awaiting requested review from moes1

@sandeeponcall sandeeponcall Awaiting requested review from sandeeponcall

@ElizabethFatade ElizabethFatade Awaiting requested review from ElizabethFatade

@shivanispatel shivanispatel Awaiting requested review from shivanispatel

@jeseszhang1010 jeseszhang1010 Awaiting requested review from jeseszhang1010

@yakovdyadkin yakovdyadkin Awaiting requested review from yakovdyadkin

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
@rafsalas19 @LiquidPT