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: remove tutorials section and move its content under appropriate sections #2097

Merged
merged 3 commits into from
Feb 15, 2024

Conversation

mtardy
Copy link
Member

@mtardy mtardy commented Feb 14, 2024

Fixes #2086.
Also first tickbox of #1974.

Compatibility is maintained thanks to page aliases: i.e. removed page will redirect to the appropriate content and not 404.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
@mtardy mtardy added area/documentation Improvements or additions to documentation release-note/docs This PR updates the documentation. labels Feb 14, 2024
@mtardy mtardy requested a review from a team as a code owner February 14, 2024 16:48
Copy link

netlify bot commented Feb 14, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 956c888
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/65ccfc635d43b30008147b64
😎 Deploy Preview https://deploy-preview-2097--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tixxdz
Copy link
Member

tixxdz commented Feb 14, 2024

Hi Mahé thanks for the change, but the troubleshooting is more for production ;-) !

@mtardy
Copy link
Member Author

mtardy commented Feb 14, 2024

Hi Mahé thanks for the change, but the troubleshooting is more for production ;-) !

yeah we can keep bade7ad out of this PR. It was discussed during hive time that it could make sense to put it down there but I don't mind.

@kkourt
Copy link
Contributor

kkourt commented Feb 14, 2024

Agreed, let's keep troubleshooting as is.

It was discussed that this guide was redundant with the getting started
guides. It might get reused as an independent blog post.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
@mtardy mtardy force-pushed the pr/mtardy/docs-remove-tutorials branch from e5040b1 to 956c888 Compare February 14, 2024 17:46
@mtardy
Copy link
Member Author

mtardy commented Feb 14, 2024

Okay I removed the commit!

@mtardy mtardy merged commit be97a74 into main Feb 15, 2024
10 checks passed
@mtardy mtardy deleted the pr/mtardy/docs-remove-tutorials branch February 15, 2024 09:27
mtardy added a commit that referenced this pull request Feb 15, 2024
This section was removed by #2097.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
tixxdz pushed a commit that referenced this pull request Feb 15, 2024
This section was removed by #2097.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
janvi01 pushed a commit to janvi01/tetragon that referenced this pull request Feb 16, 2024
This section was removed by cilium#2097.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
michi-covalent pushed a commit that referenced this pull request May 25, 2024
[ upstream commit ad5c95d ]

This section was removed by #2097.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
michi-covalent pushed a commit that referenced this pull request May 25, 2024
[ upstream commit ad5c95d ]

This section was removed by #2097.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
kkourt pushed a commit that referenced this pull request May 27, 2024
[ upstream commit ad5c95d ]

This section was removed by #2097.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation release-note/docs This PR updates the documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

docs: remove the tutorial section and move its content
3 participants