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 arg to connect to kopia repos as readonly #2004

Merged
merged 5 commits into from
Aug 23, 2023
Merged

Conversation

adowair
Copy link
Contributor

@adowair adowair commented Apr 11, 2023

Change Overview

This commit adds a new field to the repository and repository server connect arg structs to connect to a repository/server in read-only mode. As of kopia/kopia#2799, there are new kopia workflows which require a read only connection.

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

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E
Running tool: /usr/local/bin/go test -timeout 30s -coverprofile=/var/folders/pz/vhp6gb9900x1hw1cmkcn74s40000gp/T/vscode-goLblpkp/go-code-cover -run ^Test$ github.com/kanisterio/kanister/pkg/kopia/command

ok  	github.com/kanisterio/kanister/pkg/kopia/command	1.019s	coverage: 87.0% of statements

This commit adds a new field to the repository and repository server
connect arg structs to connect to a repository/server in read-only
mode. As of kopia/kopia/#2799, there are new kopia workflows which
require a read only connection.
@infraq infraq added this to In Progress in Kanister Apr 11, 2023
@adowair
Copy link
Contributor Author

adowair commented Apr 11, 2023

@aaron-kasten kopia dependency in kanister needs to be bumped to kopia/kopia#2799 (kopia/kopia@c65613c) for this PR to make sense. There are downstream dependencies on this PR.

@adowair
Copy link
Contributor Author

adowair commented Apr 11, 2023

Closing for now, as it is possible to get around this downstream

@adowair adowair closed this Apr 11, 2023
Kanister automation moved this from In Progress to Done Apr 11, 2023
@julio-lopez julio-lopez reopened this Jul 22, 2023
Kanister automation moved this from Done to In Progress Jul 22, 2023
Copy link
Contributor

@ankitjain235 ankitjain235 left a comment

Choose a reason for hiding this comment

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

LGTM

@julio-lopez
Copy link
Contributor

julio-lopez commented Aug 19, 2023

@ankitjain235 or @aaron-kasten Can one of you drive merging this PR?

@kanisterio kanisterio deleted a comment from github-actions bot Aug 19, 2023
@ankitjain235
Copy link
Contributor

@julio-lopez The Kopia commit we are referring to is after the one mentioned in the comment above. I don't see that PR needs any changes, and we can merge it as it is. cc @pavannd1

Kanister automation moved this from In Progress to Reviewer approved Aug 23, 2023
@pavannd1 pavannd1 added the kueue label Aug 23, 2023
@mergify mergify bot merged commit 4a51782 into master Aug 23, 2023
14 checks passed
Kanister automation moved this from Reviewer approved to Done Aug 23, 2023
@mergify mergify bot deleted the read-only-connect branch August 23, 2023 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants