-
-
Notifications
You must be signed in to change notification settings - Fork 402
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
chore: logging an example when generate-config fails #1308
Conversation
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
Apply Sweep Rules to your PR?
|
will review by EOD |
I think few other changes have got added in the PR. Please check once mainly in go.mod and go.sum file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please address the comments and check for changes in go.mod and go.sum file as those changes are from other commits. Rest looks good to me.
results, err := yaml.Marshal(node.Content[0]) | ||
if err != nil { | ||
g.logger.Fatal("Failed to marshal the config", zap.Error(err)) | ||
} | ||
|
||
err = os.WriteFile(filePath, results, os.ModePerm) | ||
finalOutput := append(results, []byte(utils.ConfigGuide)...) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this tested that we are getting same config file as before..?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ship it!
Related Issue
Improving the logging of Keploy when it fails due to the keploy config file.
Fixes: #1259
Describe the changes you've made
Added a log in record and test mode where we log an example to use the keploy-config file, whenever keploy fails because of it.
Type of change
Checklist:
Screenshots (if any)