Skip to content

Conversation

@dvaldivia
Copy link
Collaborator

Fixes #261

@dvaldivia dvaldivia added the WIP This PR is WIP and cannot be merged yet label Aug 31, 2020
@harshavardhana
Copy link
Member

Don't merge this as-is, keep in a branch such that we can decide on the input parameters later on to avoid this clobbering up the releases.

Since we need to comprehensively add remote targets as

  • Replication service
  • ILM service (data tiering)

Copy link
Collaborator

@bexsoft bexsoft left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@bexsoft bexsoft left a comment

Choose a reason for hiding this comment

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

Please fix

restapi/admin_remote_buckets.go:27:6: `registerAdminBucketRemoteHandlers` is unused (deadcode)
func registerAdminBucketRemoteHandlers(api *operations.ConsoleAPI) {
     ^
restapi/admin_remote_buckets.go:39:6: func `getRemoteBucketsResponse` is unused (unused)

@dvaldivia dvaldivia removed the WIP This PR is WIP and cannot be merged yet label Sep 17, 2020
dvaldivia and others added 5 commits September 22, 2020 20:32
# Conflicts:
#	go.mod
#	go.sum
#	portal-ui/bindata_assetfs.go
#	portal-ui/src/screens/Console/Buckets/ViewBucket/ViewBucket.tsx
#	portal-ui/src/screens/Console/Common/FormComponents/SelectWrapper/SelectWrapper.tsx
#	restapi/embedded_spec.go
#	restapi/operations/console_api.go
#	restapi/user_buckets.go
#	swagger.yml
@bexsoft
Copy link
Collaborator

bexsoft commented Sep 24, 2020

Replaced by #287

@bexsoft bexsoft closed this Sep 24, 2020
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.

Integrate Bucket Replication

4 participants