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 kopia version to 1e98511 #2848

Closed
wants to merge 6 commits into from
Closed

Conversation

redgoat650
Copy link
Contributor

Change Overview

See kopia/kopia@1e98511

This PR updates the binary version in the kanister-tools image and the module version. It also updates the golangci-lint version in the build image and updates the build image to v0.0.31.

Pull request type

Please check the type of change your PR introduces:

  • 🚧 Work in Progress
  • 🌈 Refactoring (no functional changes, no api changes)
  • 🐹 Trivial/Minor
  • 🐛 Bugfix
  • 🌻 Feature
  • 🗺️ Documentation
  • 🤖 Test

Issues

  • fixes #issue-number

Test Plan

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

@redgoat650 redgoat650 mentioned this pull request Apr 26, 2024
10 tasks
@redgoat650
Copy link
Contributor Author

Adding hold off merging to sort out the govulncheck warnings

Copy link
Contributor

@julio-lopez julio-lopez left a comment

Choose a reason for hiding this comment

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

LGTM ✅
🥇Thanks for doing this.

@julio-lopez
Copy link
Contributor

@redgoat650 Not merging since you added the hold off merging label. Feel free to merge (while it is still up to date 😉 )

@redgoat650
Copy link
Contributor Author

I think we'll have to postpone the kopia update, now that we've reverted the gRPC changes due to ipv6 issues. I'll close this ticket and reopen once the needed kopia fixes are in place.

#2849

@redgoat650 redgoat650 closed this Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants