Skip to content

Conversation

@sharadregoti
Copy link
Contributor

@sharadregoti sharadregoti commented Nov 21, 2025

PR Type

Documentation


Description

  • Update Tyk Pump default to enabled

  • Reflect Helm values flag consistency


Diagram Walkthrough

flowchart LR
  docs["Stack chart docs"] -- "update defaults" --> pump["Tyk Pump: false -> true"]
Loading

File Walkthrough

Relevant files
Documentation
tyk-stack-chart.mdx
Set Pump enabled by default in docs                                           

product-stack/tyk-charts/tyk-stack-chart.mdx

  • Change Pump default from false to true in table
  • Keep flag reference global.components.pump unchanged
+1/-1     

@github-actions
Copy link

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

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

Docs Accuracy

Verify that the Helm chart’s actual default for global.components.pump is now true and that the flag name matches the chart values schema across versions to avoid misleading users.

| Tyk Pump                        | true               | global.components.pump      |
| Tyk Enterprise Developer Portal | false              | global.components.devPortal | 
| Tyk Operator                    | false              | global.components.operator  |

To enable or disable each component, change the corresponding enabled flag.
Consistency

Confirm other components’ flags are consistently represented (e.g., showing n/a vs explicit flags) and whether Gateway/Dashboard should also list flags for parity with Pump.

| Component                       | Enabled by Default | Flag                        |
| :--------------------------------- | :-------------------- | :----------------------------- |
| Tyk Gateway                     | true               | n/a                         |
| Tyk Dashboard                   | true               | n/a                         |
| Tyk Pump                        | true               | global.components.pump      |
| Tyk Enterprise Developer Portal | false              | global.components.devPortal | 
| Tyk Operator                    | false              | global.components.operator  |

@github-actions
Copy link

PR Code Suggestions ✨

No code suggestions found for the PR.

@sharadregoti sharadregoti merged commit a480ca7 into main Nov 21, 2025
7 checks passed
@buger
Copy link
Member

buger commented Nov 21, 2025

/release to release-5.8

@buger
Copy link
Member

buger commented Nov 21, 2025

/release to release-5.10

buger pushed a commit that referenced this pull request Nov 21, 2025
buger pushed a commit that referenced this pull request Nov 21, 2025
@github-actions
Copy link

✅ Cherry-pick successful. A PR was created and auto-merged (if allowed): #1007

@github-actions
Copy link

✅ Cherry-pick successful. A PR was created and auto-merged (if allowed): #1009

buger added a commit that referenced this pull request Nov 21, 2025
buger added a commit that referenced this pull request Nov 21, 2025
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.

3 participants