Check tmp dir of all datastores#219
Merged
pawbu merged 9 commits intoworkflow_multitenancy_testsfrom Jan 8, 2026
Merged
Conversation
Use env in workflows
Update urllib3
DanielElisenberg
changed the base branch from
main
to
workflow_multitenancy_tests
# Conflicts: # job_executor/app.py # uv.lock
|
DanielElisenberg
approved these changes
Jan 7, 2026
DanielElisenberg
added a commit
that referenced
this pull request
Jan 14, 2026
* workflow_multitenancy_tests: move code from app * workflow_multitenancy_tests: move code to manager * workflow_multitenancy_tests: update to use job context * workflow_multitenancy_tests: restart from scratch tests * workflow_multitenancy_tests: gitkeeps * Remove dotenv (#212) * remove_dotenv: move to conftest * remove_dotenv: remove dependency * More integration tests (#213) * more_integration_tests: new datastore resouce dirs * more_integration_tests: add to common functions * more_integration_tests: add rollback tests * more_integration_tests: fix * more_integration_tests: fix * Reintroduce unit tests (#214) * reintroduce_unit_tests: adapter tests * reintroduce_unit_tests: config tests * reintroduce_unit_tests: manager tests * reintroduce_unit_tests: worker tests * reintroduce_unit_tests: fix resources * reintroduce_unit_tests: fixes manager test * reintroduce_unit_tests: fixes * reintroduce_unit_tests: missing gitkeep * reintroduce_unit_tests: fix imports * reintroduce_unit_tests: test fixes * update_urllib3: update (#215) * reintroduce_fs_models_test: reintroduce (#217) * Check tmp dir of all datastores (#219) * pin_actions: better security (#210) * Use variables from env in workflow scripts * remove double env in workflow-step * update INTEGRATION_TEST_WORKFLOW and DEPLOY_WORKFLOW in env * Update urllib3 * Check tmp dir of all datastores --------- Co-authored-by: linnbjornerud <linnchb@gmail.com> Co-authored-by: Linn Bjørnerud <125447659+linnbjornerud@users.noreply.github.com> * Extra assertions integration tests (#221) * extra_assertions_integration_tests: added metadata update assertions * extra_assertions_integration_tests: more assertions, update resources * extra_assertions_integration_tests: update dependencies * extra_assertions_integration_tests: ruff --------- Co-authored-by: pawbu <pawbu@users.noreply.github.com> Co-authored-by: linnbjornerud <linnchb@gmail.com> Co-authored-by: Linn Bjørnerud <125447659+linnbjornerud@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
The code gets all RDNs from datastore-api, then loops through them but uses the hardcoded RDN from environment. The fix is to use the current RDN we're iterating over.