Thanks. 

Yeah I agree, it was kinda tough to come up with terms and names for some of the methods and things and I am definitely not attached to any of them. 

Same thing with the workflow, once we start working on this more we will probably come up with a better one. 

--Perry

On Mon, Aug 31, 2020 at 9:50 AM Ondrej Lichtner <olichtne@redhat.com> wrote:
On Thu, Aug 27, 2020 at 03:56:14PM -0400, pgagne@redhat.com wrote:
> From: Perry Gagne <pgagne@redhat.com>
>
> Controller.py: Add log_dir to RecipeRun call.
>
> Job.py: Added __get_state__ to control what gets pickled during Recipe Run export.
> Remove netns since it contains a lot of stuff that cant be easily exported.
>
> Recipe.py: Added log_dir paramter to RecipeRun in order to more centrally track it
> so it can be used for exporting.
>
> RecipeResults.py: Added __getstate__ to DeviceConfigResult to remove _device during pickling.
>
> RecipeRunExport.py: Add RecipeRunExporter and RecipeRunData and related stuff to be used for exporting.
>
> Signed-off-by: Perry Gagne <pgagne@redhat.com>
> ---
>  lnst/Controller/Controller.py      |   2 +-
>  lnst/Controller/Job.py             |   7 ++
>  lnst/Controller/Recipe.py          |   7 +-
>  lnst/Controller/RecipeResults.py   |   5 ++
>  lnst/Controller/RecipeRunExport.py | 110 +++++++++++++++++++++++++++++
>  5 files changed, 129 insertions(+), 2 deletions(-)
>  create mode 100644 lnst/Controller/RecipeRunExport.py

I think we may want to change some of the specifics of the API with
regards to the parameters/method names. But I'm not sure exactly how
yet, I think we'll have to use this for a bit to figure out if/how we
want to improve it.

For now I'm pushing the set, thanks.

-Ondrej