Skip to content

Conversation

@bexsoft
Copy link
Collaborator

@bexsoft bexsoft commented Jan 19, 2022

What does this do?

  • Added Edit option for replication
  • Sorted by priority in replication list
  • Fixed an issue where storage class was not saved for replication rule

NOTE: Please test add & edit rules roughly as both modules were changed

How does it look?

Screen Shot 2022-01-18 at 21 42 59

Screen Shot 2022-01-19 at 19 17 44

Screen Shot 2022-01-19 at 19 17 33

Signed-off-by: Benjamin Perez [email protected]

@bexsoft bexsoft self-assigned this Jan 19, 2022
@bexsoft bexsoft added the WIP This PR is WIP and cannot be merged yet label Jan 19, 2022
@bexsoft bexsoft force-pushed the edit-replication-rule branch 2 times, most recently from a9edb1b to 66727bb Compare January 19, 2022 03:53
@bexsoft bexsoft removed the WIP This PR is WIP and cannot be merged yet label Jan 19, 2022
@bexsoft bexsoft changed the title Added new features tu bucket replication rules panel Added new features to bucket replication rules panel Jan 19, 2022
@bexsoft bexsoft force-pushed the edit-replication-rule branch from 66727bb to fdb4151 Compare January 19, 2022 03:55
dvaldivia
dvaldivia previously approved these changes Jan 19, 2022
Copy link
Collaborator

@dvaldivia dvaldivia left a comment

Choose a reason for hiding this comment

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

LGTM.

dvaldivia
dvaldivia previously approved these changes Jan 19, 2022
@bexsoft bexsoft force-pushed the edit-replication-rule branch from edb62f3 to 4e1e7ef Compare January 20, 2022 01:12
@bexsoft bexsoft requested a review from poornas January 20, 2022 01:18
@bexsoft bexsoft force-pushed the edit-replication-rule branch from 4e1e7ef to 41096d8 Compare January 20, 2022 01:21
@bexsoft bexsoft added the WIP This PR is WIP and cannot be merged yet label Jan 20, 2022
@dvaldivia
Copy link
Collaborator

@poornas PTAL

@bexsoft bexsoft force-pushed the edit-replication-rule branch from d096789 to 5fb2b7d Compare January 20, 2022 05:23
@bexsoft
Copy link
Collaborator Author

bexsoft commented Jan 20, 2022

Added selector for metadata replication. Regarding removing the validation for not deleting the last rule, minio-go sdk still contains this restriction. Will keep this behavior for now
Screen Shot 2022-01-19 at 23 25 44
Screen Shot 2022-01-19 at 23 25 37
Screen Shot 2022-01-19 at 23 25 28
Screen Shot 2022-01-19 at 23 20 57
Screen Shot 2022-01-19 at 23 19 05

- Added Edit option for replication
- Sorted by priority in replication list
- Fixed an issue where storage class was not saved for replication rule
- Added metadata replication selector to both add & edit windows

Signed-off-by: Benjamin Perez <[email protected]>
@bexsoft bexsoft force-pushed the edit-replication-rule branch from af31e66 to 92a9075 Compare January 20, 2022 05:28
@bexsoft bexsoft removed the WIP This PR is WIP and cannot be merged yet label Jan 20, 2022
@dvaldivia dvaldivia merged commit 7edab62 into minio:master Jan 20, 2022
@dvaldivia dvaldivia deleted the edit-replication-rule branch January 20, 2022 07:30
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.

3 participants