-
Notifications
You must be signed in to change notification settings - Fork 39
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
Pull request for performance analysis #161
base: develop
Are you sure you want to change the base?
Conversation
Initial version of performance test.
Adding a reporting listener for performance analysis.
Adding metrics into the listener.
Getting changes from develop.
Adding todo metris in the listener.
Integrating google spreadsheet writer.
Commiting the credential store.
Adding hyperlink for git commit id.
Updating the git branch url.
Changing the url to point to the respective commit in github.
Updating the ruleset parameter to accept more than one ruleset.
Getting changes from develop.
.build()); | ||
setErrorsCount(new ErrorSpecification.Builder("<constants.Constants: void <clinit>()>") | ||
.withTPs(HardCodedError.class, 1) | ||
.build()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you revert the lines on pupose?
Updating the changes from develop branch. Added three maven benchmark projects for performance analysis.
Removing average fields.
- Credentials for the google sheet is to be passed as maven test case argument. - Added new performance metrics - number of findings
Refreshing the token after every request.
Removing the stored token credentials.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we dynamically checkout the repository (and the resprective commit) prior to running the analysis? I don't want to checking the code of other projects into our repo.
…namically download them before the start of analysis
Adding line number to the command line reporter.
Adding the sarif schema version being used in the reporter.
Updating the jenkins file to include new stage for performance analysis. Moving the google sheet credentials to Jenkins secret text.
Fixing the build error.
Hey @johspaeth & @sritejakv , Linking @AnakinSklavenwalker to take it from here. |
Hi @kruegers, this pull request is to run a performance test in Jenkins for each commit in CrypotAnalysis (master and develop branches). It downloads three applications from GitHub and runs CryptoAnalysis to record few metrics which are stored in a Google spreadsheet online. This might be helpful in identifying sudden performance changes after merging new features into the develop or master branches. |
# Conflicts: # CryptoAnalysis/pom.xml # CryptoAnalysis/src/main/java/crypto/reporting/CommandLineReporter.java
Pull request for performance analysis