This repository has been archived by the owner on Jul 4, 2022. It is now read-only.
Merge #24
Merged
Merge #24
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Problem: .gitlab folder continues to exist
Solution: removing .gitlab folder
Testing: Build test done.
Signed-off-by: Mohammed Shahid <mohammed.shahid@seagate.com>
EOS-13562: Remove .gitlab folder from NFS repos
Update README.md
Add a Codacy badge to README.md
List of modified files:
modified: src/CMakeLists.txt
modified: src/kvstore/kvstore_base.c
Change description:
Add new performance counters and integrate them further with other modules using newly added APIs
Addb APIs integration for read path
Unit test:
Compilation with and without ENABLE_TSDB_ADDB flag
Addb unit test while integrated with read handler
Regular UTs (both with and without ENABLE_TSDB_ADDB flag)
addb dump o/p from read handler after integration:
* 2020-09-29-17:27:47.484505618 210ab ? 1?, ? 1?, ? 5035?, ? 1?
* 2020-09-29-17:27:47.484506598 210cd ? 1?, ? 2?, ? 5035?, ? 5?, ? 0?
* 2020-09-29-17:27:47.484507284 210cd ? 1?, ? 2?, ? 5035?, ? 6?, ? 1?
* 2020-09-29-17:27:47.484508075 210cd ? 1?, ? 2?, ? 5035?, ? 7?, ? 1000?
* 2020-09-29-17:27:47.484524037 210ab ? 4?, ? 1?, ? 5036?, ? 1?
* 2020-09-29-17:27:47.484524850 210ef ? 4?, ? 3?, ? 6?, ? 5036?, ? 5035?
* 2020-09-29-17:27:47.484525701 210cd ? 4?, ? 2?, ? 5036?, ? a?, ? 1000?
* 2020-09-29-17:27:47.484526538 210cd ? 4?, ? 2?, ? 5036?, ? b?, ? 0?
* 2020-09-29-17:27:47.484528239 210ab ? 2?, ? 1?, ? 5037?, ? 1?
* 2020-09-29-17:27:47.484528994 210ef ? 2?, ? 3?, ? 10?, ? 5037?, ? 5035?
* 2020-09-29-17:27:47.484529869 210cd ? 2?, ? 2?, ? 5037?, ? 10?, ? 12?
* 2020-09-29-17:27:47.484531526 210cd ? 2?, ? 2?, ? 5037?, ? 11?, ? 0?
* 2020-09-29-17:27:47.484532228 210ab ? 2?, ? 1?, ? 5037?, ? 2?
* 2020-09-29-17:27:47.484533046 210ab ? 3?, ? 1?, ? 5038?, ? 1?
* 2020-09-29-17:27:47.484533789 210ef ? 3?, ? 3?, ? 10?, ? 5038?, ? 5035?
* 2020-09-29-17:27:47.484534649 210cd ? 3?, ? 2?, ? 5038?, ? 17?, ? 12?
* 2020-09-29-17:27:47.484535507 210cd ? 3?, ? 2?, ? 5038?, ? 18?, ? 0?
* 2020-09-29-17:27:47.487947750 210cd ? 3?, ? 2?, ? 5038?, ? 19?, ? 0?
* 2020-09-29-17:27:47.487948775 210ab ? 3?, ? 1?, ? 5038?, ? 2?
EOS-8896: NFS ADDB: NFS-only tracepoints for READ (repo nsal)
Signed-off-by: Gaurav Chaudhari <gaurav.chaudhari@seagate.com>
Signed-off-by: Swapnil Gaonkar <swapnil.gaonkar@seagate.com> Co-authored-by: Sachin Punadikar <sachin.punadikar@seagate.com>
Problem Statement: Setup CodeQL Analysis GitHub Actions. It discovers vulnerabilities across a codebase using CodeQL for repositories. Solution: Setup workflow and add codeql-analysis.yml file to perform scanning using github actions.
CodeQL has some issues during code scans.Will reopen a new PR with better tool as compared to CodeQL
Problem: Add AGPL V3 License Badge and Slack Chat Badge to README.md file Solution: Badge link for AGPL V3 License and Slack Chat added to README.md file
Signed-off-by: vimalraghwani20 <vimal.raghwani@seagate.com>
Signed-off-by: Sachin Punadikar <sachin.punadikar@seagate.com> Conflicts: README.md src/kvstore/kvstore_base.c
s-arya
approved these changes
Oct 19, 2020
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.