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

[Docs] link to on-premises guide is outdated/broken #4865

Closed
2 tasks done
ALMerrill opened this issue Feb 8, 2024 · 2 comments · Fixed by #4868
Closed
2 tasks done

[Docs] link to on-premises guide is outdated/broken #4865

ALMerrill opened this issue Feb 8, 2024 · 2 comments · Fixed by #4868
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@ALMerrill
Copy link
Contributor

Description

The link to the "hard way" docs in github under the "Flyte in on-premises infrastructure" header is broken

Docs section link: https://docs.flyte.org/en/latest/deployment/deployment/cloud_simple.html#flyte-in-on-premises-infrastructure

Current link: https://github.com/davidmirror-ops/flyte-the-hard-way/blob/main/docs/on-premises/001-configure-local-k8s.md

I think its supposed to be this link? https://github.com/davidmirror-ops/flyte-the-hard-way/blob/main/docs/on-premises/single-node/001-configure-single-node-k8s.md

Are you sure this issue hasn't been raised already?

  • Yes

Have you read the Code of Conduct?

  • Yes
@ALMerrill ALMerrill added documentation Improvements or additions to documentation untriaged This issues has not yet been looked at by the Maintainers labels Feb 8, 2024
Copy link

dosubot bot commented Feb 8, 2024

🤖

It's great to see you've identified the outdated link and the potential fix for it. Your contribution to updating the documentation would be highly valuable. Feel free to create a pull request with the proposed fix, and the maintainers will review it. Thank you for your willingness to improve the project!

About Dosu

This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.

Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.

If you want to continue the conversation, start your reply with @dosu-bot.

@eapolinario eapolinario removed the untriaged This issues has not yet been looked at by the Maintainers label Feb 8, 2024
@ALMerrill
Copy link
Contributor Author

I can definitely help make this change in a PR, if the above change looks correct

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants