-
Notifications
You must be signed in to change notification settings - Fork 480
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
HDDS-3142. Create isolated enviornment for OM to test it without SCM. #656
Conversation
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.
Thanks @elek for working on this. Tested, works well.
Would be nice to include mock-scm.btm
script in hadoop-ozone/dist/target
for testing.
...ds/common/src/main/java/org/apache/hadoop/hdds/freon/FakeScmBlockLocationProtocolClient.java
Outdated
Show resolved
Hide resolved
...ommon/src/main/java/org/apache/hadoop/hdds/freon/FakeScmContainerLocationProtocolClient.java
Outdated
Show resolved
Hide resolved
hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/freon/FakeClusterTopology.java
Outdated
Show resolved
Hide resolved
Thanks the review @adoroszlai, I fixed the typo bugs.
100% agree. I created an issue to track this: https://issues.apache.org/jira/browse/HDDS-3232 |
Thanks @elek for updating the patch. |
…out SCM. (apache#656)" This reverts commit 281faf3.
What changes were proposed in this pull request?
OmKeyGenerator
class from Freon can generate keys (open key + commit key). But this test tests both OM and SCM performance. It seems to be useful to have a method to test only the OM performance with faking the response from SCM.Can be done easily with the same approach what we have in HDDS-3023: A simple utility class can be implemented and with byteman we can replace the client calls with the fake method.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-3142
How was this patch tested?
ozone om --init
+ozone om
) with the following JVM parameters: (change the path)ozone freon omkg
Expected result: It should be possible to init and start OM without SCM and test it with the key generator)