You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The build_release_artifacts github action utilizes grpc clients via Python to test grpc-labview generated server code from a proto file. It does this by installing a Python virtual environment via the CreatePythonVirtualEnv.bat file. However, this batch file doesn't specify specific versions of packages to install. As a result, it will always install the latest versions which can result in regression failures which are unrelated to the PR being submitted. This happened when trying to submit #377, and took considerable time to diagnose. Rather than always installing the latest dependencies, we should really be using Poetry projects to manage dependencies and using Poetry to install which will automatically create the venv. For an example of other NI repos that do this, see here.
For now, the batch file has been updated to pin the version for the problematic dependency. This change should be reverted once this issue is fixed.
AB#2856073
The text was updated successfully, but these errors were encountered:
The
build_release_artifacts
github action utilizes grpc clients via Python to test grpc-labview generated server code from a proto file. It does this by installing a Python virtual environment via the CreatePythonVirtualEnv.bat file. However, this batch file doesn't specify specific versions of packages to install. As a result, it will always install the latest versions which can result in regression failures which are unrelated to the PR being submitted. This happened when trying to submit #377, and took considerable time to diagnose. Rather than always installing the latest dependencies, we should really be using Poetry projects to manage dependencies and using Poetry to install which will automatically create the venv. For an example of other NI repos that do this, see here.For now, the batch file has been updated to pin the version for the problematic dependency. This change should be reverted once this issue is fixed.
AB#2856073
The text was updated successfully, but these errors were encountered: