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.

Sign up for GitHub

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

Jump to bottom

chore(tracing-apps): Update jaeger operator to v2.37.0 #881

Merged
merged 2 commits into from
Nov 25, 2022
Merged

chore(tracing-apps): Update jaeger operator to v2.37.0 #881

merged 2 commits into from
Nov 25, 2022

Conversation

isantospardo
Copy link
Contributor

@isantospardo isantospardo commented Nov 23, 2022 •
edited
Loading

Description

Fixes #878

Issues

n/a

Checklist

  • This PR contains a description of the changes I'm making
  • I updated the version in Chart.yaml
  • I updated applicable README.md files using pre-commit run
  • I documented any high-level concepts I'm introducing in docs/
  • CI is currently green and this is ready for review
  • I am ready to test changes after they are applied and released

@isantospardo isantospardo requested a review from a team as a code owner November 23, 2022 09:03
@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Nov 23, 2022
hairmare
hairmare requested changes Nov 23, 2022
View reviewed changes
Copy link
Contributor

@hairmare hairmare left a comment

Choose a reason for hiding this comment

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

tracing-apps already has an artifacthub changelog in Chart.yaml, please replace it with corresponding changes from jaeger-operator

@github-actions github-actions bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 23, 2022
@isantospardo isantospardo force-pushed the chore/jeagerOperator-update-to-2.37.0 branch from 7e98ef9 to ac2d73f Compare November 23, 2022 10:14
@github-actions github-actions bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 23, 2022
@isantospardo isantospardo force-pushed the chore/jeagerOperator-update-to-2.37.0 branch from ac2d73f to 58c708d Compare November 25, 2022 09:00
@github-actions github-actions bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Nov 25, 2022
@isantospardo isantospardo enabled auto-merge (squash) November 25, 2022 09:00
@isantospardo isantospardo merged commit 00973e2 into main Nov 25, 2022
@isantospardo isantospardo deleted the chore/jeagerOperator-update-to-2.37.0 branch November 25, 2022 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hairmare hairmare hairmare approved these changes

@inisitijitty inisitijitty Awaiting requested review from inisitijitty inisitijitty is a code owner automatically assigned from adfinis/helm-charts

Assignees
No one assigned
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Version 2.37.0 of jaegerOperator is available!
2 participants
@isantospardo @hairmare