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

New IA - Troubleshooting Debugging Page #5838

Merged
merged 10 commits into from
Dec 23, 2024

Conversation

sharadregoti
Copy link
Contributor

@sharadregoti sharadregoti commented Dec 20, 2024

User description

For internal users - Please add a Jira DX PR ticket to the subject!



Preview Link


Description


Screenshots (if appropriate)


Checklist

  • I have added a preview link to the PR description.
  • I have reviewed the suggestions made by our AI (PR Agent) and updated them accordingly (spelling errors, rephrasing, etc.)
  • I have reviewed the guidelines for contributing to this repository.
  • I have read the technical guidelines for contributing to this repository.
  • Make sure you have started your change off our latest master.
  • I labeled the PR

PR Type

Documentation


Description

  • Introduced a new comprehensive troubleshooting and debugging guide for Tyk components, covering Gateway, Dashboard, Pump, Streams, MongoDB, and Self-Managed setups.
  • Added solutions for common errors, configuration tips, and debugging techniques, along with links to relevant documentation.
  • Updated the menu to include the new troubleshooting and debugging page under API Management.
  • Added multiple aliases to ensure backward compatibility and ease of navigation for the new page.

Changes walkthrough 📝

Relevant files
Documentation
troubleshooting-debugging.md
Added a detailed troubleshooting and debugging guide for Tyk
components.

tyk-docs/content/api-management/troubleshooting-debugging.md

  • Added a comprehensive troubleshooting and debugging guide for Tyk
    components.
  • Included detailed sections for Gateway, Dashboard, Pump, Streams,
    MongoDB, and Self-Managed setups.
  • Provided solutions for common errors, configuration tips, and
    debugging techniques.
  • Added links to relevant documentation and external resources.
  • +1375/-0
    Configuration changes
    menu.yaml
    Updated menu to include troubleshooting and debugging page.

    tyk-docs/data/menu.yaml

  • Added a new menu entry for the troubleshooting and debugging page
    under API Management.
  • +4/-188 
    alias.json
    Added aliases for troubleshooting and debugging page.       

    tyk-docs/data/alias.json

  • Added multiple aliases for the troubleshooting and debugging page.
  • Mapped old paths to new sections for better navigation and backward
    compatibility.
  • +44/-0   

    💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 4 🔵🔵🔵🔵⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Content Duplication
    The newly added troubleshooting and debugging page consolidates a significant amount of content from previously existing pages. Ensure that the consolidation does not lead to loss of context, broken links, or missing critical information.

    Alias Mapping Accuracy
    The alias mappings added in the alias.json file should be verified for accuracy and completeness to ensure that all deleted pages are correctly redirected to the appropriate sections in the new troubleshooting and debugging page.

    Menu Structure Changes
    The menu structure has been significantly altered. Verify that the new structure is intuitive and that all references to deleted pages are correctly updated to point to the new consolidated page.

    Copy link

    netlify bot commented Dec 20, 2024

    PS. Pls add /docs/nightly to the end of url

    Name Link
    🔨 Latest commit 0d77cda
    🔍 Latest deploy log https://app.netlify.com/sites/tyk-docs/deploys/67692779163f7e0008030416
    😎 Deploy Preview https://deploy-preview-5838--tyk-docs.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.

    Copy link
    Contributor

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Score
    Possible issue
    Fix inconsistent quotation marks in configuration examples to ensure proper parsing

    Address the inconsistent use of quotation marks in the proxy.service_discovery
    configuration example to prevent potential parsing errors.

    tyk-docs/content/api-management/troubleshooting-debugging.md [97-98]

    -proxy.service_discovery.data_path = ""Address”
    -proxy.service_discovery.port_data_path = “ServicePort""
    +proxy.service_discovery.data_path = "Address"
    +proxy.service_discovery.port_data_path = "ServicePort"
    Suggestion importance[1-10]: 10

    Why: The suggestion addresses a critical issue with inconsistent quotation marks in the configuration example, which could cause parsing errors. Correcting this ensures the example is valid and functional, making it highly impactful for users following the documentation.

    10
    Correct the syntax of configuration examples to prevent potential misconfiguration issues

    Ensure that the tyk.conf configuration examples provided in the "Users receive 502
    error in the Gateway" section are accurate and syntactically correct, as any
    misconfiguration could lead to runtime errors or unexpected behavior.

    tyk-docs/content/api-management/troubleshooting-debugging.md [85-86]

    -enable_detailed_recording: false, 
    -enable_jsvm: false,
    +"enable_detailed_recording": false,
    +"enable_jsvm": false,
    Suggestion importance[1-10]: 9

    Why: The suggestion correctly identifies and resolves a syntax issue in the configuration examples, which could lead to runtime errors or misconfigurations if left unaddressed. This change ensures the examples are syntactically accurate and adheres to JSON standards, improving clarity and usability for users.

    9
    Ensure JSON examples are correctly formatted to avoid import failures

    Verify that the JSON example for importing keys into Tyk CE is properly formatted
    and valid, as any syntax errors could cause the import process to fail.

    tyk-docs/content/api-management/troubleshooting-debugging.md [290-294]

     "quota_renewal_rate": 60,
     "access_rights": {
         "3": {
    -    "api_name": "Tyk Test API",
    -    "api_id": "3"
    +        "api_name": "Tyk Test API",
    +        "api_id": "3"
         }
     },
    Suggestion importance[1-10]: 8

    Why: The suggestion improves the readability and correctness of the JSON example by fixing indentation issues, which could otherwise lead to confusion or errors during the import process. This enhancement is important for maintaining the accuracy of the documentation.

    8
    General
    Update the bootstrap script example to use a valid hostname and port number

    Ensure that the example command for running the bootstrap script includes a valid
    hostname and port number, as incorrect values may lead to script failure.

    tyk-docs/content/api-management/troubleshooting-debugging.md [714]

    -./bootstrap.sh new-tyk-instance.com:3000
    +./bootstrap.sh valid-hostname.com:3000
    Suggestion importance[1-10]: 7

    Why: The suggestion improves the example command by emphasizing the need for a valid hostname and port number, which is crucial for the script's success. While the change is minor, it enhances clarity and reduces the likelihood of user errors.

    7

    @sharadregoti sharadregoti merged commit c4fef22 into master Dec 23, 2024
    9 checks passed
    @sharadregoti sharadregoti deleted the ia-page-troubleshooting-debugging branch December 23, 2024 09:10
    @sharadregoti
    Copy link
    Contributor Author

    /release to release-5.7

    Copy link

    tykbot bot commented Dec 23, 2024

    Working on it! Note that it can take a few minutes.

    tykbot bot pushed a commit that referenced this pull request Dec 23, 2024
    Copy link

    tykbot bot commented Dec 23, 2024

    @sharadregoti Succesfully merged PR

    buger added a commit that referenced this pull request Dec 23, 2024
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    1 participant