-
Notifications
You must be signed in to change notification settings - Fork 414
terraform assets and support for CenturyLinkCloud #1095
Conversation
Thanks! This is really exciting. Do you think we can have a quick google hangout next week to discuss the integration? Send me a note at [email protected]
|
Could we remove the inventory commit from this PR now that we have mantl/terraform.py#36? Note that we'll have to merge that and update terraform.py before we can merge this. |
f3d8e83
to
3352dfd
Compare
yanked the inventory edits as requested, rebased and squashed commits. |
@ack Thanks for the changes! Have you been able to discuss those topics with steve? |
Yes, I spoke with steve a while ago. The tf configs for clc should be working. One of our customers has already deployed it (with a custom .terraformrc setup) and we should be all set to merge into mantl. The following PR against terraform is nearing completion and with any luck, clc will be included in the provider set for the next tf release. @stevendborrelli and I also discussed the possibility of integrating acceptance testing on our platform. The next step would be to have someone from mantl/ciscocloud set up an account on our platform. We'd be able to grant recurring compute credits for automated testing purposes. |
@ack Sorry, I just merged a conflicting PR. Does the same thing as yours on the Traefik config. Can you fix the conflict? Then we'll merge this, and we can get to work on the CI build immediately after. |
yanked the traefik commit |
@ack And I'm sorry I didn't see this sooner, but I just noticed that the documentation is in the wrong format/spot. Could you add it to |
migrated docs and rebased |
@ack Thanks so much for your work on this! It's all looking good to me, I'm excited to get automated testing up and running. |
terraform assets and support for CenturyLinkCloud
No description provided.