Cache EndpointConfig session
#12886
Cache EndpointConfig session
#12886
Conversation
ancalita
added
the
tools:clear-poetry-cache-unit-tests
ancalita
removed
the
tools:clear-poetry-cache-unit-tests
ancalita
changed the title
EndpointConfig session
ancalita
requested review from
miraai,
souvikg10,
a team and
radovanZRasa
and removed request for
a team
|
|
||
| - name: Install ddtrace on Windows | ||
| if: needs.changes.outputs.backend == 'true' && matrix.os == 'windows-2019' | ||
| run: py -m pip install -U 'ddtrace<2.0.0' |
There was a problem hiding this comment.
@miraai Is there a unified command provided by Github actions that allows us to install a dependency with a constraint for both OS types?
There was a problem hiding this comment.
@ancalita We can use RUNNER_OS envar to put this into a single step, your call though if it is worth it
- name: Install ddtrace
if: needs.changes.outputs.backend == 'true'
run: |
if [ "$RUNNER_OS" == "Linux" ]; then
poetry run pip install -U 'ddtrace<2.0.0'
elif [ "$RUNNER_OS" == "Windows" ]; then
py -m pip install -U 'ddtrace<2.0.0'
else
echo "$RUNNER_OS not supported"
exit 1
fi
There was a problem hiding this comment.
This is documented here, https://docs.github.com/en/actions/learn-github-actions/variables#default-environment-variables
There was a problem hiding this comment.
Though I'm not sure if matrix.os == runner.os
There was a problem hiding this comment.
@miraai what approach do you recommend?
|
🚀 A preview of the docs have been deployed at the following URL: https://12886--rasahq-docs-rasa-v2.netlify.app/docs/rasa |
Proposed changes:
ddtracein the CI workflows because the new version2.0.0released on Tue 3rd Oct upgradeswraptto1.15and uninstalls Tensorflow's constraint to be pinned at<1.15, this causes 30 job failures.Status (please check what you already did):
black(please check Readme for instructions)