Skip to content

Remove code for CDN (poppy)#2296

Closed
EmilienM wants to merge 1 commit intogophercloud:masterfrom
EmilienM:cdn_removal
Closed

Remove code for CDN (poppy)#2296
EmilienM wants to merge 1 commit intogophercloud:masterfrom
EmilienM:cdn_removal

Conversation

@EmilienM
Copy link
Copy Markdown
Contributor

OpenStack CDN (Poppy) isn't maintained anymore, last commit was 5 years
ago:
https://opendev.org/x/poppy/commits/branch/master

This patch removes all the code that called CDN v1 API, which is not
tested, and the upstream project doesn't seem to be alive anymore.

This effort is made for helping to maintain Gophercloud and clean the
things we don't need/use anymore.

OpenStack CDN (Poppy) isn't maintained anymore, last commit was 5 years
ago:
https://opendev.org/x/poppy/commits/branch/master

This patch removes all the code that called CDN v1 API, which is not
tested, and the upstream project doesn't seem to be alive anymore.

This effort is made for helping to maintain Gophercloud and clean the
things we don't need/use anymore.
@EmilienM
Copy link
Copy Markdown
Contributor Author

EmilienM commented Dec 14, 2021

@jtopjian @jrperritt would you be against it?
If you tell me this is used somewhere, I'm happy to close this PR.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 79.917% when pulling 6edd6fa on EmilienM:cdn_removal into 6626c35 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Dec 14, 2021

Build succeeded.

@jrperritt
Copy link
Copy Markdown
Contributor

@jrperritt would you be against it?

No, sounds fine to me.

@jtopjian
Copy link
Copy Markdown
Contributor

Ditto - I am in favour of removing this.

@EmilienM EmilienM closed this by deleting the head repository Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants