Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Use the refactored runner manager (#351)
* Patch runner log path in tests * Add missing fixture * Fix the scope of fixture * Fix tmp_path scope issue * Fix monkeypatch fixture scope issue * Add patch of metric log path * Fix return type of create_runners * Add health check test * Fix arg naming * Add debug statement * Add more debug statement * Move debug statement * Merge tests * Handle openstack errors with delete runner * Fix delete server * Fix test variable reference * Fix OpenstackInstance creation * Add some docstrings * Fix args issues with RunnerInstance * Add more docs * Fix GithubRunnerState construction * Fix instance-id parsing from full name * Add delete idle runner test. * Add busy flush to test * Spawn a manual test env * Disable spawning on manual test env * Remove useless class * Fix runner deletion * Fix import error * Add more docs * Fix get no-existing openstack server * Add debug statement * Fix variable name and function name mixup * Fix id variable name, function name mixup * Add debug statement * Move debug * Add busy runner test * Add debug statement. * Disable some test * Disable some test * Fix runner label in workflow * Fix lambda * Debug * Debug * Add debug * Start new manual test env * Add none check * Fix missing prefix * Add more logging * Refactor runner manager one runner fixture * Fix error string formatting * Adding the docstring for github_runner_manager * Fix test fixture scope * Add docstring on cloud_runner_manager * Add debug * Fix docstring for cloud runner manager * Add more docstrings * Add metrics for deleted and cleanup runners * Enable tests again * Add debug * Get runner info not on GitHub * Fix dict access * Add debug of userdata * Fix metric path * Debug metric * Fix variable naming * Test * Fix iterator * Debug * Debug * Fix for iterator return value * Add more log path patching * Fix path naming * Fix monkey patch * Start a arm64 manual test env * Not spawning manual test env * Update fmt * Fix metric storage implementation for openstack * Fix metric storage provider usage in openstack runner manager * Debug * Fix iterator * Add debug * Fix None in iterator * Add debug * Trying fix for get runner filter * Add test * Patch the path for logs * Add cleanup test * Debug * Fix github state determining busy runner * Fix wrong naming for method in ABC * Remove debugging * Add more docstrings * Fix runner deletion * Add more docs * Fix typing * Debug * Update SSH health check * Tmp disable a passing test * Add deubg * Remove a debug * Fix Cloud runner state init * Change clean up to cleanup * Fix attr naming issue in openstack cloud * Fix reference to non-existing instance_name in openstack cloud * Add metric log processing to test * Enable all tests * Fix health check return value * Fix all flake8 lints * Fix test * Fix all lints * Fix unit test issue due to method sig change * Ignore openstack cloud from coverage due to the test requires private endpoint * Enable all tests * Remove a repeated test * Re-enable test.yaml * Fix integration tests workflwo * Add docs on cleanup method of cloud runner manager * Add parallel spawning of runners. * Enable dev testing * Fix parallel spawn * Allow openstack server to take a bit of time on deletion * Refactor test detection of no runners * Re-enable the tests * Fix lints * Disable tests again * Disable some test * Add wait until runner is running * Enable openstack runner manager tests * Add debug * Wait for github state * Refactor wait until runner spawn * Add keyfile erorr * Remove debug statement * Re-enable all tests * Update src/manager/github_runner_manager.py Co-authored-by: Yanks Yoon <[email protected]> * Update src/openstack_cloud/openstack_cloud.py Co-authored-by: Yanks Yoon <[email protected]> * Suggestions * Refactor remove openstack server * Test spawning two runners. * Fix test * Fix naming * Fix according comment * Fix clouds yaml write issue. * Fix format * Add delete runner by amount * Add getting runner health state for metrics * Fix security group ID issues * Fix according to review * Refactor health state for runner * Fix lint issues * Add missing docs * Update the github state enum to use auto * Rename class to fit convension * Fix according to review * Fix name_prefix property cloud runner manager * Add class for scaling runners * Fix lints * Fix unit test * Fix according to review comment * Fix test according to comments * Fix unit test * Fix typo of attr * Add debug * Add debug statement * Debug * Fix return code of the kill command * Remove debug * Add comments on the flush kill command * Add debug * Fix debug * Debug * Debug * Remove debug * Add cleanup during idle and busy runner test * Debug * Disable tests during debug * Debug missing keyfiles * Fix keyfile path matching issue * testing * debug * Add debug * Use OR * debug * Debug * Debug * Debug * Debug * Debug * Fix flush mode * Remove debug * Re-enable all tests * Initial unit test for runner scaler * Add more unit tests for runner scaler * Add more tests * Fix merge issues * Fix states in get_runners methods * Add docstring for unit test mocks * Fix construction of repo-policy-compliance from config * Fix get_runners action output * Fix the lints * Fix a naming issue * Fix naming prefix of runner * Improve unit test * Remove the old OpenstackRunnerManager * Fix test contstruction of runner manager. * Fix flavor naming * Fix flush action result output. * Fix flavor of metric * Testing out a integration test fikx * change flush runner to flush idle. * Add debug in integration test * Manual test mode * Start new manual test env * Spawn x64 manual test env. * Improve logging during reconcile * Fix crashed metric collection * Remove debug workflow * Format * Test * Add reactive back in * Fix flushing of runners * Debug workflow * Add debug * Fix logging of health state * Remove debug * Debug * Fix set contruction * Fix SSH key path in integration test setup * Add more checks to repo-policy-compliance setup in tests * Fix key path check * Fix format string issue * Fix format string typo * Add some logging of test setup * Fix missing await * Revert config-change flushing * Add maintance status for image relation change * Fix HTTP format * Update coverage ignore of github_runner_manager * Minor fix in test comments --------- Co-authored-by: Yanks Yoon <[email protected]> Co-authored-by: Christopher Bartz <[email protected]>
- Loading branch information
3 people
authored
Sep 5, 2024
1 parent
cb1b70e
commit 5b8c99a