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

Add kopia server connect helper methods #888

Merged
merged 4 commits into from
Feb 5, 2021
Merged

Add kopia server connect helper methods #888

merged 4 commits into from
Feb 5, 2021

Conversation

PrasadG193
Copy link
Contributor

@PrasadG193 PrasadG193 commented Jan 27, 2021

Signed-off-by: Prasad Ghangal prasad.ghangal@gmail.com

Change Overview

New pkg kopia with kopia server connect helper methods

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

Test Plan

Will wait for server side implementation for the e2e testing

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

@pavannd1
Copy link
Contributor

Looks good overall. We may have to see if the changes to repository in the latest kopia master would require changes to this. We can merge and follow up.

@pavannd1 pavannd1 self-requested a review January 28, 2021 20:04
@PrasadG193 PrasadG193 marked this pull request as ready for review January 29, 2021 07:59
Signed-off-by: Prasad Ghangal <prasad.ghangal@gmail.com>
Signed-off-by: Prasad Ghangal <prasad.ghangal@gmail.com>
@pavannd1
Copy link
Contributor

pavannd1 commented Feb 3, 2021

@PrasadG193 let's queue this behind #869 if possible. Some API changes would be required when we update Kopia

@pavannd1
Copy link
Contributor

pavannd1 commented Feb 5, 2021

@PrasadG193 The other PR merged. Please update the helpers accordingly

pkg/kopia/connect.go Outdated Show resolved Hide resolved
@mergify mergify bot merged commit 23448c1 into master Feb 5, 2021
@mergify mergify bot deleted the kopia-connect branch February 5, 2021 22:28
chaitanya-baraskar pushed a commit to chaitanya-baraskar/kanister that referenced this pull request Apr 5, 2021
* Add kopia server connect helper methods

Signed-off-by: Prasad Ghangal <prasad.ghangal@gmail.com>

* Add licence headers

Signed-off-by: Prasad Ghangal <prasad.ghangal@gmail.com>

* Update connect options

Co-authored-by: Pavan Navarathna <pavan@kasten.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants