Skip to content

Multi pr #71

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 120 commits into
base: master
Choose a base branch
from
Open

Multi pr #71

wants to merge 120 commits into from

Conversation

@mkinney
Copy link

@mkinney mkinney commented Feb 4, 2020

Major refactoring.

I'd like to continue refactoring/enhancing, but this is a lot of work so far.

Major changes:

  • added flake8 for some static analysis and fixes for flake8 recommendations
  • added docs on most functions/methods
  • added integration tests that validate most functionality
  • can run multiple instances; most commands work without an instance name (refers to "all" instances)
  • provisioning runs when you 'start' or 'resume' instances
  • eliminated the data design (no state saved at this point except for the boxes)
  • files are not stored in user's home dir; this means you need to be in the directory that contains the Mechfile for commands to work
  • removed any "not implemented" options

Takashi Ando and others added 30 commits May 4, 2019 10:27
Fix OSError if mech destroy mechboxes#57
f1d55f5
This fixes mechboxes#59: json.decoder.JSONDecodeError: Expecting prop…
a67380c 
…erty name enclosed in double quotes if was broken
@mkinney
show instance name when ssh-config command is run
204c779
@mkinney
add command line option to disable shared folder
1180354
@mkinney
add ability to override vcpus/mem
ee1e270
@mkinney
Merge remote-tracking branch 'dodo5522/fix-json-decode-error-by-broke…
0f043f6 
…n-index' into multi-pr
@mkinney
Merge remote-tracking branch 'origin/disable-shared-folder-option' in…
b2d9b92 
…to multi-pr
@mkinney
Merge remote-tracking branch 'origin/mem-and-vcpu' into multi-pr
b173b0b
@mkinney
Merge remote-tracking branch 'origin/show-instance-name-in-ssh-config…
2684174 
…' into multi-pr
@mkinney
add info on how to install from diff repo; update example
d461bbf
@mkinney
add info in readme about new up options
321fd37
@mkinney
flake8 fixes; add contributing file
578a811
@mkinney
move all files under this dir .mech
7b20319
@mkinney
get list/destroy to work as expected
13e265a
@mkinney
got mech up and ls working with multiple
1681233
@mkinney
enable most commands to deal with single or multiple instances
3c4a20b
@mkinney
add docstrings; get scp working
fff0393
@mkinney
test that snapshot add/list/delete work as expected
377097d
@mkinney
validate mech box add/list/remove work as expected
46333dc
@mkinney
update help info
e6faeb6
@mkinney
add bats and simple int tests
f89d149
@mkinney
add two instances test; added name to up/delete
1e263c3
@mkinney
document the int tests
82e800b
@mkinney
update doc/TODOs
57d8021
@mkinney
add direnv
38842c6
@mkinney
migrate TODO notes into project board
8109fff
@mkinney
change debug logging; pass named values; validate creating box from file
a76f29e
@mkinney
improve int testing
19b9176
@mkinney
add int testing for most functionality
1223951
@mkinney
validate provisioning works as expected
e014e37
@mkinney
Copy link
Author

mkinney commented Feb 16, 2020

How cool is this: https://codecov.io/gh/mkinney/mech/branch/multi-pr ?!?

mkinney added 17 commits February 15, 2020 21:19
@mkinney
add codecov badge in readme
b4eeccc
@mkinney
add init from json int test
d336310
@mkinney
add more utils unittests
bc74773
@mkinney
add more unittests for utils
f40c0ee
@mkinney
remove options that are not implemented
8648061
@mkinney
add upgradevm option; but there seems to be a bug in vmrun
00aba98
@mkinney
add a few more lines of unittest coverage
533b81a
@mkinney
add some unittests to vmrun
c4611da
@mkinney
set executable/provider in test
96d506b
@mkinney
added unit test for vmrun.get_guest_ip_address
c41cdde
@mkinney
only create vmrun object with user/pass when we need to
1f63830
@mkinney
added add-me option
0e93177
@mkinney
add unit tests for vmrun class
ad91f68
@mkinney
add use-me/remove-vagrant options
5f12d24
@mkinney
revamp the ssh output
afe6089
@mkinney
fix issues that int tests caught; update readme with current help; li…
7b4dfd7 
…mit pytest jobs to 3 for more consistent runs
@mkinney
convert bats int tests to pure python
048386b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mkinney @ColdHeat @arizvisa