[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

[via GitHub]

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: dev-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org