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

RFC Refactoring #667

Open
7 of 15 tasks
bowrocker opened this issue Jul 29, 2014 · 8 comments
Open
7 of 15 tasks

RFC Refactoring #667

bowrocker opened this issue Jul 29, 2014 · 8 comments
Assignees

Comments

@bowrocker
Copy link
Contributor

Introduction

Refactoring relating to the rfc here:

https://gist.github.com/kellymclaughlin/f28b75ac295b3d397ef0

Work Items

  • Refactor and break up rt module into appropriate subdomain functions (JA)
  • Add specs to all rt* functions and get them passing dialyzer
  • Uniform naming for all rt modules
  • Create a harness behavior and have each harness implement it (JA)
  • Better integration points for Riak CS: backends, node lists, etc.
  • Improve node/deploy cluster API (KM)
  • Write a module to hide the details of the rt_properties record (getter setter) (KM)
  • Versioning for the rt_properties record (KM)
  • Add enforcement of valid backend fields from RT properties
  • Update all the 2i tests not to run with bit cask
  • Convert all tests to new conventions
  • Determine which tests are worth running as rolling upgrade tests
  • Type annotations so we can run test categories (JA)
  • Concurrent test runnings
  • Ability to change the logging verbosity of the framework logging and the nodes under test independently via the command line or the configuration file. (BH)

Branch

https://github.com/basho/riak_test/compare/feature/decouple-cluster-setup

@reiddraper
Copy link
Contributor

Really happy to see this!

@jburwell jburwell assigned jburwell and unassigned bowrocker Jul 29, 2014
@jburwell jburwell modified the milestone: 2.1 Jul 29, 2014
@jburwell jburwell removed their assignment Jul 29, 2014
@jburwell jburwell removed the feature label Jul 29, 2014
@jburwell
Copy link
Contributor

I mixed up browser tabs -- modifying the wrong ticket.

@kellymclaughlin
Copy link
Contributor

#668 is a PR for the following items from the task list:

  • Write a module to hide the details of the rt_properties record
  • Versioning for the rt_properties record

@bowrocker
Copy link
Contributor Author

#669 is a PR for the item from the task list:

  • Refactor and break up rt module into appropriate subdomain functions (JA)

@bowrocker bowrocker self-assigned this Aug 1, 2014
@kellymclaughlin
Copy link
Contributor

PR for riak_test_escript refactor: #670

@bowrocker
Copy link
Contributor Author

PR for test harness behaviour: #672

@bowrocker
Copy link
Contributor Author

PR for test attributes: #674

@kellymclaughlin
Copy link
Contributor

PR for work that overhauls how node setup and deployment works: #710

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

No branches or pull requests

4 participants