Skip to content

Adding support for the CDN service #505

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 61 commits into from
Jan 6, 2015
Merged

Adding support for the CDN service #505

merged 61 commits into from
Jan 6, 2015

Conversation

ycombinator
Copy link
Contributor

This PR adds support for the OpenStack Poppy service. It includes:

@ycombinator
Allow using PUT (default) or PATCH as update method depending on prop…
e959b01 
…erty.
@ycombinator
Setting memory limit needed for code coverage generation.
87d3116
@ycombinator
Parameterize type of service in catalog so it can be specified when b…
601e18c 
…uilding a service object.
@ycombinator
Adding tests and implementation for the CDN service.
733ddd6
@ycombinator
Whitespace fixes.
f156f47
@ycombinator
Fixing docblock for Flavor class.
bde9c66
@ycombinator
Looping over object properties while changing them is a bad idea!
95eada8
@ycombinator
Use the maximum pagination limit supported by the API, unless client …
6d65fca 
…provides a lower one.
@ycombinator
Do not use the top-level property in the create JSON.
f963aef
@ycombinator
Adding smoke tests for CDN.
020f7b3
@ycombinator
Adding code samples.
1b0eef9
@ycombinator
Adding Getting Started Guide for CDN service.
07d62a0
@ycombinator
Adding another property alias.
8f54c4e
@ycombinator
Clarifying definition.
260f68f
@ycombinator
Disambiguating "service" by providing additional context.
038e2da
@ycombinator
Fixing link.
f525941
@ycombinator
Adding Complete User Guide for CDN service.
157ea27
@ycombinator
Whitespace lint fixes.
ad71565
@ycombinator
Removing limits.
1c59c45
@ycombinator
Fixing formatting.
ffb915d
@ycombinator
Fixing typo.
92cbf98
@ycombinator
Changing syntax to be PHP5.3-compliant.
ee47135
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling f45dc27 on ycombinator:cdn into 5228802 on rackspace:working.

@ycombinator
Copy link
Contributor Author

This PR is ready for review except for a the Update Service smoke test. That requires a working API endpoint, which is not available yet.

@ycombinator ycombinator changed the title [WIP] Adding support for the CDN service Adding support for the CDN service Jan 6, 2015
@jamiehannaford
Copy link
Contributor

LGTM :shipit:

jamiehannaford pushed a commit that referenced this pull request Jan 6, 2015
Merge pull request #505 from ycombinator/cdn
890fddc 
Adding support for the CDN service
@jamiehannaford jamiehannaford merged commit 890fddc into rackspace:working Jan 6, 2015
@ycombinator ycombinator mentioned this pull request Jan 6, 2015
Merged
@ycombinator ycombinator mentioned this pull request Jan 6, 2015
Merged
@ycombinator ycombinator deleted the cdn branch March 19, 2015 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
PR: New feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ycombinator @coveralls @rgbkrk @jamiehannaford @amitgandhinz