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

Create redirects for Telemetry and Serverless #18099

Merged
merged 9 commits into from
Sep 4, 2023

Conversation

mmitoraj
Copy link
Contributor

@mmitoraj mmitoraj commented Sep 1, 2023

Description

Changes proposed in this pull request:

  • Redirects for Telemetry after docu migration from components to modules
  • Redirects for Serverless after docu migration from components to modules

Related issue(s)

@mmitoraj mmitoraj added area/documentation Issues or PRs related to documentation do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Sep 1, 2023
@mmitoraj mmitoraj requested a review from a team as a code owner September 1, 2023 12:11
@netlify
Copy link

netlify bot commented Sep 1, 2023

👷 Deploy request for kyma-project pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit fc0504c

@netlify
Copy link

netlify bot commented Sep 1, 2023

Deploy Preview for kyma-project-docs-preview ready!

Name Link
🔨 Latest commit fc0504c
🔍 Latest deploy log https://app.netlify.com/sites/kyma-project-docs-preview/deploys/64f580cd89cd3e00073a5cc4
😎 Deploy Preview https://deploy-preview-18099--kyma-project-docs-preview.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.

@kyma-bot kyma-bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 1, 2023
@mmitoraj mmitoraj removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 1, 2023
docs/_redirects Outdated
https://kyma-project.io/#/05-technical-reference/00-custom-resources/telemetry-03-tracepipeline https://kyma-project.io/#/telemetry-manager/user/resources/04-tracepipeline 301
https://kyma-project.io/#/01-overview/serverless/README https://kyma-project.io/#/serverless-manager/user/README 301
https://kyma-project.io/#/03-tutorials/00-serverless/* https://kyma-project.io/#/serverless-manager/user/tutorials/README 301
https://kyma-project.io/#/04-operation-guides/troubleshooting/seerverless/* https://kyma-project.io/#/serverless-manager/user/troubleshooting-guides/README 301
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
https://kyma-project.io/#/04-operation-guides/troubleshooting/seerverless/* https://kyma-project.io/#/serverless-manager/user/troubleshooting-guides/README 301
https://kyma-project.io/#/04-operation-guides/troubleshooting/serverless/* https://kyma-project.io/#/serverless-manager/user/troubleshooting-guides/README 301

?

docs/_redirects Outdated
https://kyma-project.io/#/05-technical-reference/00-custom-resources/telemetry-03-tracepipeline https://kyma-project.io/#/telemetry-manager/user/resources/04-tracepipeline 301
https://kyma-project.io/#/01-overview/serverless/README https://kyma-project.io/#/serverless-manager/user/README 301
https://kyma-project.io/#/03-tutorials/00-serverless/* https://kyma-project.io/#/serverless-manager/user/tutorials/README 301
https://kyma-project.io/#/04-operation-guides/troubleshooting/seerverless/* https://kyma-project.io/#/serverless-manager/user/troubleshooting-guides/README 301
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
https://kyma-project.io/#/04-operation-guides/troubleshooting/seerverless/* https://kyma-project.io/#/serverless-manager/user/troubleshooting-guides/README 301
https://kyma-project.io/#/04-operation-guides/troubleshooting/serverless/* https://kyma-project.io/#/serverless-manager/user/troubleshooting-guides/README 301

docs/_redirects Outdated Show resolved Hide resolved
valentinvieriu
valentinvieriu previously approved these changes Sep 4, 2023
@kyma-bot kyma-bot added the lgtm Looks good to me! label Sep 4, 2023
@kyma-bot kyma-bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed lgtm Looks good to me! size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Sep 4, 2023
valentinvieriu
valentinvieriu previously approved these changes Sep 4, 2023
@kyma-bot kyma-bot added the lgtm Looks good to me! label Sep 4, 2023
@kyma-bot kyma-bot added the lgtm Looks good to me! label Sep 4, 2023
@kyma-bot
Copy link
Contributor

kyma-bot commented Sep 4, 2023

@mmitoraj: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-kyma-governance fc0504c link false /test pull-kyma-governance

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@@ -10,6 +10,5 @@ A custom resource (CR) is an extension to the Kubernetes API which allows you to
| API Gateway | [APIRule](apix-01-apirule.md) |
| Eventing | [Subscription](evnt-01-subscription.md), [EventingBackend](evnt-02-eventingbackend.md) |
| Istio | [Istio](oper-01-istio.md) |
| Telemetry | [LogPipeline](telemetry-01-logpipeline.md), [LogParser](telemetry-02-logparser.md), [TracePipeline](telemetry-03-tracepipeline.md) |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for spotting this!

@kyma-bot kyma-bot merged commit 7bcfe7f into kyma-project:main Sep 4, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Issues or PRs related to documentation lgtm Looks good to me! size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants