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

directory "_chaosfs__zookeeper__" not removed after iochaos experiment is finished and deleted #3217

Open
alelerac opened this issue May 3, 2022 · 2 comments · May be fixed by chaos-mesh/toda#35
Assignees
Labels
chaos/io type/enhancement New feature or request

Comments

@alelerac
Copy link

alelerac commented May 3, 2022

Bug Report

What version of Kubernetes are you using?

Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.1", GitCommit:"c4d752765b3bbac2237bf87cf0b1c2e307844666", GitTreeState:"clean", BuildDate:"2020-12-18T12:09:25Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.4", GitCommit:"ec2760d6d916781de466541a6babb4309766c995", GitTreeState:"clean", BuildDate:"2021-02-27T17:18:03Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"linux/amd64"}

What version of Chaos Mesh are you using?

Controller manager Version: version.Info{GitVersion:"v2.1.5", GitCommit:"42e2aaa590b4e738e069766aef062f6468ff81d4", BuildDate:"2022-04-18T09:44:56Z", GoVersion:"go1.17.5", Compiler:"gc", Platform:"linux/amd64"}

What did you do?

Run below experiment.

apiVersion: chaos-mesh.org/v1alpha1
kind: IOChaos
metadata:
name: io-delay-zk
namespace: zk
spec:
action: latency
mode: one
selector:
labelSelectors:
"app": zk
"statefulset.kubernetes.io/pod-name": zk-0
volumePath: /var/lib/zookeeper/
delay: "100ms"
percent: 100
duration: "50s"

Then delete experiment with kubectl delete iochaos io-delay-zk -n zk

What did you expect to see?

Cleanup after experiment.
The created directory "chaosfs__zookeeper_" is removed

What did you see instead?

Directory "chaosfs__zookeeper" is not removed

ls -l /var/lib/
total 4
drwxr-xr-x 2 root root 6 May 3 14:49 chaosfs__zookeeper
drwxr-xr-x 1 root root 42 Jun 13 2017 apt

@STRRL STRRL self-assigned this May 5, 2022
@STRRL STRRL added type/enhancement New feature or request chaos/io labels May 5, 2022
@STRRL
Copy link
Member

STRRL commented May 5, 2022

I think that is in expectation. Currently, chaos-daemon (in detail toda) would not remove the directory __chaosfs__<origin-name>__, but only umount from it, and it's enough to let the injected process be recovered from chaos experiment.

I think it could be an enhancement. Do you have interests to help us to make it? :D @alelerac

@basit9958
Copy link
Contributor

/assign

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chaos/io type/enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants