Skip to content
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

Enable multi-container pod runner #304

Closed
wants to merge 3 commits into from
Closed

Conversation

xinze-zheng
Copy link
Collaborator

Add an optional extra field operator_container_name in config.json. Deploy object will parse the config file and extract the operator_container_name to its instance variable. Runner will query the deploy object to access the operator container name.

Copy link
Contributor

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  acto
  deploy.py 58-62, 149
  acto/lib
  operator_config.py
  acto/post_process
  post_diff_test.py 328, 381, 421
  acto/runner
  runner.py 46, 316-323
Project Total  

This report was generated by python-coverage-comment-action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant