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.
i think there is reason why such warning is checked. better not to remove this. what else warning are we seeing?
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 agree with @lguohan comments. I was able to verify it without removing that line #206. Just the below line should work i believe.
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 checked other test cases. No one else checked warning output.
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 am not fully convinced that we can ignore the warning here.
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.
$ grep run_script src/sonic-config-engine/tests/ -rn
src/sonic-config-engine/tests/test_cfggen.py:50: def run_script(self, argument, check_stderr=False, verbose=False):
src/sonic-config-engine/tests/test_cfggen.py:71: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:76: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:81: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:86: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:91: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:96: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:101: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:106: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:111: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:116: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:123: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:128: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:133: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:138: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:145: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:169: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:174: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:182: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:196: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:206: output = self.run_script(argument, True, True)
src/sonic-config-engine/tests/test_cfggen.py:225:# output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:230: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:238: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:244: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:259: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:284: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:289: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:299: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:307: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:317: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:325: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:333: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:341: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:349: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:360: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:374: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:383: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:392: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:433: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:474: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:482: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:490: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:498: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:504: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:510: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:518: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:561: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:604: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:647:# output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:652: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:660: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:666: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:672: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:677: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:685: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:693: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:729: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:747: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:752: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:766: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:806: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:811: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:816: output = json.loads(self.run_script(argument))
src/sonic-config-engine/tests/test_cfggen.py:825: json.loads(self.run_script(argument)),
src/sonic-config-engine/tests/test_cfggen.py:843: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:856: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:869: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:885: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:896: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:904: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:911: output = self.run_script(argument)
src/sonic-config-engine/tests/test_cfggen.py:919: output = self.run_script(argument)
We can see only this test case checked warning. What's the different between this case and others?