Fixes issue 235 by completing CLI deploy command

[maximilien]

fixes issue 235 but does not work. Needed:

• --dry-run option for Deploy
  OR
• fix issue when deploying now without a deployments directory (maybe allow to specify path)

@akihikokuroda let's do the easiest thing to get this working then we can iterate and improve

This is the output when I run: test-fail-output.txt

[akihikokuroda]

Adding "DRY_RUN=1" here enables the dry run.

[akihikokuroda]

The Deploy constructor is expecting like this:

deploy = Deploy("../tests/examples/condition_agents.yaml", "../tests/examples/condition_workflow.yaml", "BEE_API_KEY=sk-proj-testkey BEE_API=http://192.168.86.45:4000")

The format of the environment variables is like "key1=value1 key2=value2"

[akihikokuroda]

There is an issue that the Deploy class has to be executed at src directory now. I'll look into it.

[akihikokuroda]

#254 fixes the working directory issue.

[maximilien]

Thank you @akihikokuroda. I will rebase and resubmit this PR.

[maximilien]

@akihikokuroda getting a series of issues ... maybe you can take a look

1. deploy tests are failing but seems like I am doing the right thing... since the deploy starts but then fails
2. when deploy tests fails it leaves the tmp2 directory and does not clean up. This causes all tests to fail until that directory is deleted.

I am passing --dry-run BTW

You can check this by doing the following:

gh pr checkout 257
...
pytest

Including output here: issue235_pytest_output.txt

[maximilien]

Made some changes @akihikokuroda and got further but still failing. FYI

The issue of tmp2 directory still there. I think it's because the test fails and the deploy does not have a chance to clean up... at a minimum we need to have that tmp2 be located in the user's temp directory so it does not polute the current. WDYT?

[akihikokuroda]

@maximilien Yes, I want to move the temp directory somewhere else. I'll look into why the directory returned by tempfile.gettempdir is not writable in my Mac.

I put fix in #264

[maximilien]

OK will try this again now that #264 is merged. Likely in a few hours