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

[jaeger] allInOne add option tolerations #588

Merged
merged 1 commit into from
Jul 11, 2024

Conversation

chenlujjj
Copy link
Contributor

@chenlujjj chenlujjj commented Jul 11, 2024

What this PR does

Add tolerations option to allInOne deployment

Checklist

  • DCO signed
  • Commits are GPG signed
  • Chart Version bumped
  • Title of the PR starts with chart name ([jaeger] or [jaeger-operator])
  • README.md has been updated to match version/contain new values

@pavelnikolov
Copy link
Contributor

pavelnikolov commented Jul 11, 2024

@chenlujjj thank you for your PR. Could you, please, GPG-sign your commits? You have to also bump the chart version. See other PRs.

Copy link
Contributor

@pavelnikolov pavelnikolov left a comment

Choose a reason for hiding this comment

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

Make sure you bump the chart version and GPG-sign your commits, please.

@chenlujjj chenlujjj force-pushed the allinone-tolerations branch from 647b6a0 to e594f48 Compare July 11, 2024 10:05
@chenlujjj
Copy link
Contributor Author

sure, let me check how to do that

Signed-off-by: chenlujjj <953546398@qq.com>
@chenlujjj chenlujjj force-pushed the allinone-tolerations branch from e594f48 to 76b42dd Compare July 11, 2024 12:49
@chenlujjj
Copy link
Contributor Author

hi @pavelnikolov I've signed the commit, thanks

@pavelnikolov pavelnikolov merged commit ebdb6ea into jaegertracing:main Jul 11, 2024
2 checks passed
hellspawn679 pushed a commit to hellspawn679/helm-charts that referenced this pull request Nov 1, 2024
Signed-off-by: chenlujjj <953546398@qq.com>
Signed-off-by: mehul <mehulsharam4786@gmail.com>
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.

2 participants