Brief Description
-----------------
Zuul tox job now failing for stx-puppet.
In /home/zuul/src/opendev.org/starlingx/stx-puppet/puppet-manifests/src/bin/apply_network_config.sh line 374: upgr_bootstrap=$? ^-- SC2319 (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten.
Possible fixes:
* Modify the problematic shell code apply_network_config.sh . (not great, the code isn't that bad)
Brief Description
-----------------
Zuul tox job now failing for stx-puppet.
In /home/zuul/ src/opendev. org/starlingx/ stx-puppet/ puppet- manifests/ src/bin/ apply_network_ config. sh line 374:
upgr_bootstrap =$?
^-- SC2319 (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten.
Possible fixes: config. sh . (not great, the code isn't that bad)
* Modify the problematic shell code apply_network_
* Pin the shellcheck version: py-0.9. 0.6 py-0.10. 0.1
passes with: shellcheck-
fails with: shellcheck-
* Relax the specific checks that are failing: SC2119, SC2120
# shellcheck disable=
or, -eSC2319
Severity
--------
Critical: Prevents code submission.
Steps to Reproduce
------------------
Update stx-puppet with a patchset.
Expected Behavior
------------------
Previous codebase should pass existing linters.
Actual Behavior
----------------
Changed upstream version of shellcheck has different behavior for specific tests.
Reproducibility
---------------
100%
System Configuration ------- ------
-------
gerrit review
Branch/Pull Time/Commit ------- ------- --
-------
NA
Last Pass
---------
Recent.
Timestamp/Logs
--------------
NA
Test Activity
-------------
Development
Workaround
----------
None.