Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

terraform assets and support for CenturyLinkCloud #1095

Merged
merged 2 commits into from
Mar 11, 2016

Conversation

ack
Copy link
Contributor

@ack ack commented Feb 5, 2016

No description provided.

@metahertz metahertz changed the title terraform assets and support for CenturyLinkCloud terraform assets and support for CenturyLinkCloud Feb 5, 2016
@stevendborrelli
Copy link
Contributor

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]

  • CLC terraform provisioner
  • Travis-CI integration

@langston-barrett
Copy link
Contributor

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.

@ack ack force-pushed the clc branch 3 times, most recently from f3d8e83 to 3352dfd Compare March 8, 2016 23:06
@ack
Copy link
Contributor Author

ack commented Mar 8, 2016

yanked the inventory edits as requested, rebased and squashed commits.

@langston-barrett
Copy link
Contributor

@ack Thanks for the changes! Have you been able to discuss those topics with steve?

@ack
Copy link
Contributor Author

ack commented Mar 9, 2016

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.
hashicorp/terraform#4893

@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.

@langston-barrett
Copy link
Contributor

@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.

@ack
Copy link
Contributor Author

ack commented Mar 10, 2016

yanked the traefik commit

@langston-barrett
Copy link
Contributor

@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 docs/getting_started/clc.rst and add that page to the table of contents in docs/getting_started/index.rst? If you're not sure how to do that, we can merge this and then I'll work on it.

@ack
Copy link
Contributor Author

ack commented Mar 11, 2016

migrated docs and rebased

@langston-barrett
Copy link
Contributor

@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.

langston-barrett added a commit that referenced this pull request Mar 11, 2016
terraform assets and support for CenturyLinkCloud
@langston-barrett langston-barrett merged commit dd5730a into mantl:master Mar 11, 2016
@ryane ryane added this to the 1.1 milestone Mar 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants