Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

provider/aws: Convert Route 53 Zone, Record to upstream #1519

Merged
merged 2 commits into from
Apr 16, 2015

Conversation

catsby
Copy link
Contributor

@catsby catsby commented Apr 14, 2015

  • Converts both Route 53 Zone and Records to upstream aws-sdk-go.
  • Converts helper tag library
  • Moves a Route 53 only method from structure.go to structure_sdk.go, but without the SDK suffix since Route53 is the only resource that uses it
  • Acceptance tests are running passed

@ocxo
Copy link

ocxo commented Apr 14, 2015

👍 You beat me to it on this one. I was getting tripped up on the flattenResourceRecords. Thanks for getting these all ported over!

@catsby
Copy link
Contributor Author

catsby commented Apr 14, 2015

@fromonesrc I got stumped there briefly too, and I'm the one who wrote it 😄

Doing vpn gateway next, if you're curious. Should wrap #1488 today/tomorrow 🎊

@phinze
Copy link
Contributor

phinze commented Apr 16, 2015

LGTM

@catsby catsby force-pushed the f-aws-upstream-route53 branch from 7d2750c to 1da9bc8 Compare April 16, 2015 18:43
catsby added a commit that referenced this pull request Apr 16, 2015
provider/aws: Convert Route 53 Zone, Record to upstream
@catsby catsby merged commit 94d9f69 into master Apr 16, 2015
@catsby catsby deleted the f-aws-upstream-route53 branch April 16, 2015 18:47
@ghost
Copy link

ghost commented May 3, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators May 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants