Create a dependancy between the libvirt network and virsh command adding a dns record

[eparis]

Since we used the variable name in both places terraform tried to run both at
the same time. We could have created an explicit dep, or done this via an
implicit dep. I did implicit.

Fixes: #138

[wking]

This is for #138?

[wking]

The e2e-aws error was:

1 error(s) occurred:

* aws_s3_bucket.tectonic: 1 error(s) occurred:

* aws_s3_bucket.tectonic: Error putting S3 ACL: OperationAborted: A conflicting conditional operation is currently in progress against this resource. Please try again.
	status code: 409, request id: 369D71D8DCE4D4B1, host id: 0QZRoCRLFOXmYmuu0lpj1RCRk08Jlc6XCZc1DMyrIihcLsYtxB0kjrZZ9t/WwZhA+B3v/oEo8zc=

That may be due to an over-reaching removal by my cleanup scripts, which I'm currently running.

[eparis]

Didn't know that existed, but it should. yes. I updated the top comment.

[wking]

/retest

(although without #121 and associated release updates, this libvirt change isn't being exercised by CI).

[jlebon]

I can confirm this fixes #138.

[wking]

/lgtm

I'll wait and see if #151 fixes the Ginkgo time out, and if it does I'll kick the tests here again.

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: eparis, wking

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [wking]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[sjenning]

tested and works for me
/retest

[eparis]

/retest