| manadart_ | achilleasa: Need a review for the AWS VPC re-use. | 11:09 |
|---|---|---|
| manadart_ | https://github.com/juju/juju/pull/11872 | 11:09 |
| achilleasa | manadart_: looking in 5' | 11:12 |
| achilleasa | manadart_: are you using venv to test ^^^ ? | 11:46 |
| manadart_ | achilleasa: Yes. | 11:46 |
| achilleasa | we will need to bump the requirements: https://pastebin.canonical.com/p/JQg2k6T7c8/ | 11:49 |
| achilleasa | I will try to get it up & running and send you a diff for requirements.txt | 11:50 |
| achilleasa | looks like it's only some of the azure bits... | 11:55 |
| achilleasa | manadart_: https://pastebin.canonical.com/p/6ghJG7gC2q/ not sure if removing these will break winrm.py... | 11:56 |
| achilleasa | manadart_: are the 400 responses at the end of the test the known cleanup issues that you mention in the PR description? | 12:03 |
| manadart_ | achilleasa: For the security groups? | 12:06 |
| achilleasa | yeap | 12:06 |
| manadart_ | achilleasa: Have not yet looked at those. I always get a destroy-model failure. | 12:07 |
| manadart_ | achilleasa: https://github.com/juju/juju/pull/11873 merges that single patch forward. | 13:47 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!