Skip to content
This repository has been archived by the owner on Jan 20, 2024. It is now read-only.

Contrib agent Support for Machine Target Executor #106

Merged
merged 9 commits into from
Jan 13, 2022

Conversation

AlfinST
Copy link
Contributor

@AlfinST AlfinST commented Jan 3, 2022

Summary

@yahoo/ychaos-dev

Implementing Contrib agents

Fixes: #93

Checklist

Checklist (Developer)

Prerequisites

  • I have read the contribution guidelines

Code Analysis

  • Covered by Unittests
  • Security warnings ignored (Why?!)

Project related

  • Schema changed and is backward compatible.
  • Introduces a new sub-command for ychaos CLI

Autogenerated Files

  • Auto generated schema

Pre-Merge Checklist

  • All the build jobs are passing

Checklist (Reviewer 1)

  • Reviewed Source Code
  • Reviewed Tests (If applicable)
  • Reviewed Documentation (If applicable)

Checklist (Reviewer 2)

  • Reviewed Source Code
  • Reviewed Tests (If applicable)
  • Reviewed Documentation (If applicable)

@codecov
Copy link

codecov bot commented Jan 3, 2022

Codecov Report

Merging #106 (57031cc) into main (cc0f12e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #106   +/-   ##
=======================================
  Coverage   98.68%   98.68%           
=======================================
  Files          56       56           
  Lines        2428     2441   +13     
  Branches      361      363    +2     
=======================================
+ Hits         2396     2409   +13     
  Misses         20       20           
  Partials       12       12           
Impacted Files Coverage Δ
src/ychaos/agents/contrib.py 100.00% <100.00%> (ø)
src/ychaos/core/executor/MachineTargetExecutor.py 98.43% <100.00%> (+0.39%) ⬆️

@AlfinST AlfinST changed the title fix documentation for contrib agent Contrib agent Support for Machine Target Executor Jan 6, 2022
@AlfinST AlfinST marked this pull request as ready for review January 7, 2022 06:35
@AlfinST AlfinST requested a review from a team as a code owner January 7, 2022 06:35
@AlfinST AlfinST force-pushed the contrib_agent_impl branch 2 times, most recently from 7e0a396 to 379cab3 Compare January 10, 2022 08:52
Copy link
Member

@shashankrnr32 shashankrnr32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nits.

src/ychaos/agents/contrib.py Outdated Show resolved Hide resolved
tests/resources/contrib_agent/dummy_agent.py Outdated Show resolved Hide resolved
tests/resources/contrib_agent/dummy_agent.py Outdated Show resolved Hide resolved
tests/resources/contrib_agent/dummy_agent.py Outdated Show resolved Hide resolved
tests/resources/contrib_agent/dummy_agent.py Outdated Show resolved Hide resolved
tests/resources/contrib_agent/dummy_agent.py Outdated Show resolved Hide resolved
tests/resources/contrib_agent/contribAgentTestplan.yaml Outdated Show resolved Hide resolved
Comment on lines 328 to 330
modified_testplan.attack.agents[i].config[
"path"
] = "./ychaos_ws/{}".format(filename.name)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
modified_testplan.attack.agents[i].config[
"path"
] = "./ychaos_ws/{}".format(filename.name)
modified_testplan.attack.agents[i].config.path = "./ychaos_ws/{}".format(filename.name)

Copy link
Member

@shashankrnr32 shashankrnr32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit; Looks good to me otherwise

src/ychaos/core/executor/MachineTargetExecutor.py Outdated Show resolved Hide resolved
shashankrnr32
shashankrnr32 previously approved these changes Jan 13, 2022
Copy link
Member

@shashankrnr32 shashankrnr32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you 👍 . LGTM

r-r-2
r-r-2 previously approved these changes Jan 13, 2022
@shashankrnr32 shashankrnr32 added the enhancement New feature or request label Jan 13, 2022
@shashankrnr32 shashankrnr32 linked an issue Jan 13, 2022 that may be closed by this pull request
@AlfinST AlfinST merged commit c3651ca into yahoo:main Jan 13, 2022
@AlfinST AlfinST deleted the contrib_agent_impl branch January 13, 2022 11:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Contrib Agents in Executors
3 participants