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

[Feature] Rename cnf_cleanup, cnf_setup and setup tasks #2181

Open
kosstennbl opened this issue Nov 20, 2024 · 0 comments
Open

[Feature] Rename cnf_cleanup, cnf_setup and setup tasks #2181

kosstennbl opened this issue Nov 20, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@kosstennbl
Copy link
Collaborator

In cnf-testsuite, there are multiple ways for process of installation and deletion of CNFs, in methods/documentations/tasks they often differ between cnf_setup/cnf_cleanup and cnf install/cnf uninstall. It needs to be brought to one common naming to avoid confusion for new users and make tasks more understandable without digging through documentation.

"setup" task is responsible for installation of all dependencies and needed tools for functionality of testsuite and the task name doesn't represent that clearly. Some other name should be selected for that process.

@kosstennbl kosstennbl added the enhancement New feature or request label Nov 20, 2024
kosstennbl pushed a commit that referenced this issue Nov 29, 2024
Rename cnf_setup, cnf_cleanup and setup tasks to
cnf_install, cnf_uninstall, install_dependencies
Change documentation, specs and code accordingly

Refs: #2181
Signed-off-by: Konstantin Yarovoy <[email protected]>
kosstennbl pushed a commit that referenced this issue Nov 29, 2024
Rename cnf_setup, cnf_cleanup and setup tasks to
cnf_install, cnf_uninstall, install_dependencies
Change documentation, specs and code accordingly

Refs: #2181
Signed-off-by: Konstantin Yarovoy <[email protected]>
kosstennbl pushed a commit that referenced this issue Dec 3, 2024
Rename cnf_setup, cnf_cleanup and setup tasks to
cnf_install, cnf_uninstall, install_dependencies
Change documentation, specs and code accordingly

Refs: #2181
Signed-off-by: Konstantin Yarovoy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant