Tags: cnf-testsuite/helm
Tags
Helm module refactors (#6) * Refactor Helm module - Restructure logging: * Add scopes with .for() method * Change .info logs to be informative feedback as to what is happening * Scope logs to .info and .debug categories more accurately * Delete many lines of unnecessary .info and .debug logs * Completely delete developer logs like random arguments or words - Change signatures of methods corresponding to helm operations (pull, install...) from accepting cli strings to accepting arguments (from which cli commands are built). Helm module has a place in this codebase as layer of abstraction over helm binary only when providing api that can be used declaratively. It doesn't have much sense when many of the methods are accepting complete cli commands, it defeats the purpose of this module (testsuite could just use Process.run in place of method calls) and makes calls to those methods completely unconsistent throughout the codebase. - Simplify methods without changing logic wherever possible - Delete module methods related to helm v2 support - Delete prints to stdout_ from module (moved to main cnti_testsuite) - there are still prints in utils.cr marked as TODO. - Run official crystal-lang autoformatter - Delete unused constants from top of the file Signed-off-by: Rafal Lal <rafal.lal@tietoevry.com>