Skip to content

Remove code for CDN (poppy) - #2296

Closed
EmilienM wants to merge 1 commit into
gophercloud:masterfrom
EmilienM:cdn_removal
Closed

EmilienM wants to merge 1 commit into
gophercloud: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

EmilienM commented Dec 14, 2021 •

Copy link
Copy Markdown
Contributor Author

@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

theopenlab-ci Bot commented Dec 14, 2021

Copy link
Copy Markdown

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.

@mandre mandre mentioned this pull request Mar 4, 2023
4 of 11 tasks
@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