[GitHub] [cloudstack-cloudmonkey] rohityadavcloud commented on a diff in pull request #131: Fix #115: cmk ignores the CLI profile arg when loading the cache file

Mon, 17 Apr 2023 23:22:50 -0700 


rohityadavcloud commented on code in PR #131:
URL: 
https://github.com/apache/cloudstack-cloudmonkey/pull/131#discussion_r1169546073


##########
cmk.go:
##########
@@ -70,6 +82,10 @@ func main() {
        }
 
        if *profile != "" {
+               if !existsProfileCache(*profile, cfg) {
+                       fmt.Printf("Cannot find a cache file for the profile: 
%s\n", *profile)
+                       os.Exit(1)

Review Comment:
   I would have thought of it using a cache file otherwise giving a warning to 
the user when not found, and when it’s found just use it. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to