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

Possible Improvement Scope: Leftover Installation in the Ansible test scripts #1768

Open
Talismanic opened this issue Jan 14, 2021 · 1 comment

Comments

@Talismanic
Copy link

Greetings,
I am an engineer who is interested in infrastructure as code testing. Currently, I am looking for testing anti-patterns in iac test scripts. I noticed that for testing purpose we sometimes install different packages, but do not clean those after the testing is completed. This may lead to problem in sub-sequent tests if the same environment is used in multiple testing or keep some unwanted packages installed in the environment.
For example, in this file, we can see that task is installing python3.4 and some pre-requisite dependencies. After the test, it is not cleaning any of them. Though in this case, it should not cause any problem as python is harmless. But in case someone uses same environment for testing with some python scripts compatible with 2.7, they can face some issue theoretically.

So, I have the following queries:
Do you principally agree that if any package is installed for testing purpose, we should remove the package?

Do you want to add a step to remove the packages?

Any feedback is appreciated.

@JunqiZhang0
Copy link
Contributor

Hi @Talismanic , thank you for giving your opinion to linchpin! Linchpin is now under maintenance mode, it only supports business critical features/bug fixes, as this doesn't cause any problem, we will just leave it as is. You can always submit a PR and we will review it. Thanks!

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

No branches or pull requests

2 participants