Skip to content

Remove code for CDN (poppy)#2809

Merged
EmilienM merged 1 commit intomasterfrom
cdn_removal
Oct 13, 2023
Merged

Remove code for CDN (poppy)#2809
EmilienM merged 1 commit intomasterfrom
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

This was approved here by original maintainers: #2296

@github-actions github-actions bot added the semver:major Breaking change label Oct 12, 2023
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 77.445% (+0.004%) from 77.441% when pulling 4dc5596 on cdn_removal into d00e5c5 on master.

@EmilienM EmilienM merged commit fed1858 into master Oct 13, 2023
@EmilienM EmilienM deleted the cdn_removal branch October 13, 2023 18:18
mandre added a commit to shiftstack/gophercloud that referenced this pull request Mar 22, 2024
The PR removing the code for CDN [1] merged around the same time the
fixtures were modified to no longer be part of the public API [2],
causing these files to stay in the tree.

This commit cleans up files that shouldn't be here.

[1] gophercloud#2809
[2] gophercloud#2811
@mandre mandre mentioned this pull request Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:major Breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants