These changes are for cloud-ctl error lablels #405
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ISSUE(S):
https://platform9.atlassian.net/browse/PCD-607
SUMMARY
This is related cloud-ctl as the error message should have cloud-ctl rather than pf9ctl so for the ld-flag introduced a variable so the value can be changed dynamically in cloud-ctl and it won't affect pf9ctl as well
ISSUE TYPE
IMPACTED FEATURES/COMPONENTS:
RELATED ISSUE(S):
DEPENDS ON:
TESTING DONE
in progress
Reviewers
Summary by Bito
This PR enhances error message clarity by making them dynamically include tool names and improving configuration-related code. It updates error messages in pkg/config/config.go for missing configurations, introduces a new 'Ctl' constant in pkg/util/constants.go, and refines error messages for version conflicts and sudo privileges using fmt.Sprintf.Unit tests added: False
Estimated effort to review (1-5, lower is better): 1