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.
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.
A file is suggested to debug the problem. We should try to avoid using it in the fix if possible.
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.
I have removed the use of the file
response.json
in the code as suggested.Grinder links:
TARGET: sanity.functional
mac:
42294
win:
42295
aarch64_linux:
#42296
s390x_linux:
#42292
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.
Thanks @annaibm . It looks like the fix is to add quotes around
URL
. Do we need the following changes (L244 - L319)?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.
I have removed changes from (L244 - L319) . The fix for constructing the curl command based on the operating system . Added quotes around the URL for Windows, while no quotes are used for Linux. Also added the -k option in the curl command.
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.
Grinder links:
x86_64_linux
https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/42341/console
s390x_linux
https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/42340/
ppc64_aix
https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/42338/console
aarch64_linux
https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/42344/console
x86_64_windows
https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/42343/consoleFull
x86_64_mac
https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/42342/console