-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix automatic database downloads in CICD system's Github actions #1914
Comments
this is an example of the import error we get, fyi https://github.com/RTXteam/RTX/actions/runs/3107811327/jobs/5036326534 |
just an fyi for everyone - as discussed at today's AHM, I added an explanation to the ARAX wiki of the manual step that needs to happen until this issue is addressed. https://github.com/RTXteam/RTX/wiki/CI-system-(via-GitHub-actions) |
Hey @amykglen I know this has been sitting forever but I think it is now solved and all the pytest.yml commands are running with no import error.
To:
|
I will update the docs and close this issue once you've given it a once-over. |
Link to test build https://github.com/RTXteam/RTX/actions/runs/5139600151 |
Closing issues |
@sundareswarpullela and I discussed this in detail today - documenting a summary here
we had a step in RTX/.github/workflows/pytest.yml that automatically downloads databases as needed (and deletes old ones as needed) via the ARAX Database Manager; that Database Manager function works correctly but Github actions wasn't able to run it, saying it was unable to import
RTXConfiguration
. (this step is commented out for now.)it seems to be that the
sys.path.append([abs_path])
method for imports doesn't work for Github actions - it uses a $GITHUB_PATH... though we also weren't successful today when trying to use that.we have two options for a fix:
but before we can work on a fix, we should get set up so we can run Github Actions locally. (which seems to be weirdly hard to do...) this might help? being able to run the actions locally will allow us to do some real debugging..
The text was updated successfully, but these errors were encountered: