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

Update weaver-gke to the base weaver version 0.18.0. #81

Merged
merged 1 commit into from
Jul 24, 2023
Merged

Conversation

spetrovic77
Copy link
Contributor

Changes required for the upgrade:

  • Use the new versioning libraries.
  • Local copy of the version command, since the corresponding command in weaver has been moved to its internal/.
  • Change the protocol to use protos.TraceSpans.
  • Trace dashboard changes.

@spetrovic77 spetrovic77 self-assigned this Jul 24, 2023
Changes required for the upgrade:
  * Use the new versioning libraries.
  * Local copy of the version command, since the corresponding
    command in weaver has been moved to its internal/.
  * Change the protocol to use `protos.TraceSpans`.
  * Trace dashboard changes.
Copy link
Member

@mwhittaker mwhittaker left a comment

Choose a reason for hiding this comment

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

Nice! Thanks Srdjan!

@spetrovic77 spetrovic77 merged commit 72a695e into main Jul 24, 2023
7 checks passed
@spetrovic77 spetrovic77 deleted the update branch July 24, 2023 23:25
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