This repository has been archived by the owner on Jul 4, 2022. It is now read-only.
Merge #33
Merged
Merge #33
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>
Co-authored-by: Sachin Punadikar <sachin.punadikar@seagate.com>
* Added README.md Added README.md Signed-off-by: Saumya Sunder <saumya.sunder@seagate.com> * Added SUPPORT.md Added SUPPORT.md Signed-off-by: Saumya Sunder <saumya.sunder@seagate.com> * Added SECURITY.md Added SECURITY.md Signed-off-by: Saumya Sunder <saumya.sunder@seagate.com> * Added CONTRIBUTING.md Added CONTRIBUTING.md Signed-off-by: Saumya Sunder <saumya.sunder@seagate.com> * Added CODE_OF_CONDUCT.md Added CODE_OF_CONDUCT.md Signed-off-by: Saumya Sunder <saumya.sunder@seagate.com> Co-authored-by: Sachin Punadikar <sachin.punadikar@seagate.com>
Signed-off-by: vimalraghwani20 <vimal.raghwani@seagate.com>
Added ADDB tracepoint for directory operations like mkdir, rmdir, readdir and lookup. Signed-off-by: Sachin Punadikar <sachin.punadikar@seagate.com>
At times one need to work on a global index to store global information. Currently supported functionality do not provide such global framework, and everytime one need to mention which index one want to work with in that KVStore. Functionality added such that one can specify an index which can be treated as global index, and support KV operations. Operations include set a KV pair, get the KV pair and delete the KV pair. Unit test is also provided. Signed-off-by: Sachin Punadikar <sachin.punadikar@seagate.com>
* fix: disable auto-close issues/PR feature of stale-bot Signed-off-by: Gaurav Chaudhari <gaurav.chaudhari@seagate.com> * feat: add github action for issue triage Signed-off-by: Gaurav Chaudhari <gaurav.chaudhari@seagate.com>
Delete CODE_OF_CONDUCT.md because it is moved to the .git org level folder. Signed-off-by: Saumya Sunder saumya.sunder@seagate.com
Problem: The perf counter definitions should be included in cortxfs but it is currently in utils
Change description:
Refactor function tags, maps and attribute enums and move out repo specific code. Introduce ranges. move out plugin.
List of modified files:
modified: src/common/motr/m0common.c
new file: src/include/cfs_nsal_perfc.h
modified: src/kvstore/kvstore_base.c
modified: src/kvstore/plugins/cortx/cortx_kvstore.c
modified: src/kvtree/kvtree.c
Test:
Full build, executed testing scripts.
Signed-off-by: Mohammed Shahid <mohammed.shahid@seagate.com>
Signed-off-by: Sachin Punadikar <sachin.punadikar@seagate.com>
s-arya
approved these changes
Nov 5, 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.