UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Update Jaeger to 1.27.0

Merged Jordan Olachea requested to merge bb-811-update-jaeger into main
All threads resolved!

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Micah Nagel added statusdoing label and removed statusreview label

    added statusdoing label and removed statusreview label

  • Jordan Olachea added 2 commits

    added 2 commits

    • 366b9cb5 - update "chart" (https://github.com/jaegertracing/helm-charts) from...
    • 9e931dde - Removed changelog duplication

    Compare with previous version

  • Jordan Olachea added 2 commits

    added 2 commits

    • 51333a40 - update "chart" (https://github.com/jaegertracing/helm-charts) from...
    • 77c75e5f - update "chart" (https://github.com/jaegertracing/helm-charts) from...

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • f1d2fb44 - Updated all values to 1.27.0

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Tunde Oladipupo added 4 commits

    added 4 commits

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Tunde Oladipupo changed title from update {-"chart" (https://github.com/jaegertracing/helm-charts) from...-} to Update Jaeger to 1.27.0

    changed title from update {-"chart" (https://github.com/jaegertracing/helm-charts) from...-} to Update Jaeger to 1.27.0

  • Tunde Oladipupo changed the description

    changed the description

  • Tunde Oladipupo added 32 commits

    added 32 commits

    • 333a865b...c6daa631 - 6 commits from branch main
    • 8d514421 - update "chart" (https://github.com/jaegertracing/helm-charts) from...
    • 497a3a5b - Revert "update "chart" (https://github.com/jaegertracing/helm-charts) from...
    • eca2c5b2 - Fixed chart and changelog- still need to check crd
    • e35837c1 - Updated values.yaml
    • 90b35c60 - Fixed chart and changelog- still need to check crd
    • eb16b769 - update "chart" (https://github.com/jaegertracing/helm-charts) from...
    • 47d4ba1d - Revert "update "chart" (https://github.com/jaegertracing/helm-charts) from...
    • 24dcdb03 - Fixed chart and changelog- still need to check crd
    • 559acf0d - Fixed chart and changelog- still need to check crd
    • 237329d2 - updated changelog
    • f0eebef3 - updated chart.yaml
    • b7d7f9a2 - Fixed chart and changelog- still need to check crd
    • 367482ed - updated chart version
    • 11388a16 - Fixed chart and changelog- still need to check crd
    • 323f6a27 - update "chart" (https://github.com/jaegertracing/helm-charts) from...
    • c13039a2 - update "chart" (https://github.com/jaegertracing/helm-charts) from...
    • 5158ef6f - update "chart" (https://github.com/jaegertracing/helm-charts) from...
    • 2e87b1bc - Updated all values to 1.27.0
    • 68c6bfe7 - Updated changelog
    • 421666eb - updated kpt
    • fdfcc8ac - updated crds pkg
    • 2d1f7021 - updated crds
    • 9d3fab94 - updated crds
    • 8525a2f8 - updated crds to v1.27.0
    • 90c0f5c8 - updated changelog
    • a50ed0bc - updated changelog

    Compare with previous version

  • Tunde Oladipupo added statusreview label and removed statusdoing label

    added statusreview label and removed statusdoing label

  • Jordan Olachea resolved all threads

    resolved all threads

    • Resolved by Micah Nagel

      I think we can get rid of the extra kptfile/crd file for jaegertracing.io_jaegers_crd.yaml. We had that in there before because the upstream chart CRD wasn't the same as the CRD in the upstream operator repo. I did a diff on the two CRDs and nothing of substance is different now so we should be able to just use the upstream chart one. We only want one copy of the CRD otherwise it'll make things angry potentially :smile:

      Edited by Micah Nagel
  • Otherwise changes look good. Testing now.

  • Micah Nagel requested review from @micah.nagel

    requested review from @micah.nagel

  • Everything worked fine for me on both clean install and upgrade. Will approve once that comment is resolved regarding CRDs.

  • mentioned in issue #16 (closed)

  • added 1 commit

    • 4bb84acb - removed jaegertracing.io_jaegers_crd.yaml

    Compare with previous version

  • Micah Nagel approved this merge request

    approved this merge request

  • Micah Nagel resolved all threads

    resolved all threads

  • Micah Nagel mentioned in commit 7d9d339e

    mentioned in commit 7d9d339e

  • merged

  • Please register or sign in to reply
    Loading