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

Clean up old readthedocs sites #571

Closed
SgtCoDFish opened this issue May 25, 2021 · 5 comments
Closed

Clean up old readthedocs sites #571

SgtCoDFish opened this issue May 25, 2021 · 5 comments
Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.

Comments

@SgtCoDFish
Copy link
Member

There are other sites which mention cert-manager (e.g https://cert-manager-munnerz.readthedocs.io). These can appear in searches and might be confusing. We should take a look at ensuring that everything links to cert-manager.io

/kind cleanup

@jetstack-bot jetstack-bot added the kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. label May 25, 2021
@maelvls
Copy link
Member

maelvls commented May 25, 2021

Note: searching for issuerRef leads to this link:

https://docs.cert-manager.io/en/release-0.7/tasks/issuing-certificates/index.html

Is it also a readthedocs page?

@SgtCoDFish
Copy link
Member Author

docs.cert-manager.io

Some versions have redirects in place and others don't, hence why release-0.7 pops up in search results.

Ideally, there'd be support for adding redirects in the cert-manager readthedocs.io project. Otherwise, we could have to go back to really old release branches and update them manually.

cert-manager-munnerz.readthedocs.io

This is the last commit before the docs/ folder was cleaned out in the cert-manager repo - so this is what cert-manager-munnerz is built from AFAIK.

Fixes

The best fix would be if there are settings in the cert-manager and cert-manager-munnerz readthedocs.io projects for redirecting all pages to https://cert-manager.io. Failing that, it seems possible to set versions to "hidden" in the control panel.

In addition, we could add a robots.txt to the docs/ folder to prevent search engines indexing the old docs sites - this would work for both docs.cert-manager.io and cert-manager-munnerz.readthedocs.io. There are docs surrounding this.

@SgtCoDFish
Copy link
Member Author

I don't think I can do much more investigation without access to the readthedocs.io project, and the only person listed as a maintainer is @munnerz

James, do you still have access to these projects?

@SgtCoDFish
Copy link
Member Author

There's also a readthedocs webhook on https://github.com/jetstack/cert-manager which should be removed.

@SgtCoDFish
Copy link
Member Author

@munnerz has deleted both projects (:heart:), so in time they'll presumably be removed from search engines.

(see on slack)

I think this can be considered closed now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.
Projects
None yet
Development

No branches or pull requests

3 participants