Skip to content

Conversation

@sym-stiller
Copy link

Description

These changes enable users of the Tyk Helm charts to specify a priorityClassName for each component. By default, no priority class is assigned to the components.

Related Issue

#411

Motivation and Context

When no priority class is specified, Kubernetes will assign the lowest possible priority to pods. These low-priority pods may be preempted by higher-priority pods, which is not ideal for an API gateway.

Test Coverage For This Change

I have templated the charts locally like this:

helm dependency build # only necessary for the umbrella charts, not the component charts
helm template tyk ./ -n tyk --debug

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Refactoring or add test (improvements in base code or adds test coverage to functionality)
  • Documentation updates or improvements.

Checklist

  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side). If PRing from your fork, don't come from your master!
  • Make sure you are making a pull request against our master branch (left side). Also, it would be best if you started your change off our latest master.
  • My change requires a change to the documentation.
    • I have manually updated the README(s)/documentation accordingly.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@sym-stiller sym-stiller requested a review from a team as a code owner July 15, 2025 09:02
@sym-stiller sym-stiller requested review from olamilekan000 and removed request for a team July 15, 2025 09:02
@sym-stiller sym-stiller force-pushed the feat/priority-class branch from 7041192 to 3b62d0b Compare July 30, 2025 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant