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.

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

Update agitator to read cluster.yml #183

Closed
milleruntime opened this issue Jan 3, 2022 · 2 comments · Fixed by #184
Closed

Update agitator to read cluster.yml #183

milleruntime opened this issue Jan 3, 2022 · 2 comments · Fixed by #184
Assignees

Comments

@milleruntime
Copy link
Contributor

This ticket is for a solution to have the agitator scripts read the new cluster.yml file. The files referenced in the agitator scripts (master-agitator.pl, tserver-agitator.pl, datanode-agitator.pl) no longer exist since apache/accumulo#2238. Related to #182

One solution would be to use the perl library "tiny" to real the YAML file but this will require the install of an additional perl package. For example, if the test cluster were running centos it would need to have the package perl-yaml-tiny installed. I am not sure if this is the best solution.

@dlmarion
Copy link
Contributor

dlmarion commented Jan 3, 2022

Something I just thought of is to do what the accumulo-cluster script does. It invokes a Java class to parse the yaml file and outputs the information in a key-value format.

I have never really looked at the agitator scripts before, it looks like they just start/stop/restart/kill processes. I think we could make the scripts more modular, reuse some code, and maybe include other types of failure. For example, the old Netflix Chaos Monkey scripts use Linux host commands to introduce various things into the environment - high CPU / IO usage, packet loss, packet latency, packet corruption, bad routing, etc.

@milleruntime
Copy link
Contributor Author

Something I just thought of is to do what the accumulo-cluster script does. It invokes a Java class to parse the yaml file and outputs the information in a key-value format.

I like this idea. There is a bash script calling the perl scripts so we could probably just eliminate the perl scripts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants