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

Implement machine management tool #9

Merged
merged 43 commits into from
Feb 22, 2019
Merged

Conversation

atoghill
Copy link
Contributor

@atoghill atoghill commented Feb 20, 2019

Based on #8 this PR adds an additional widget to the Additional Information page. With this new widget it should be possible to restart or stop the machine it is running on.

image

@atoghill atoghill force-pushed the feature/machine-management-tool branch from 5dd9051 to cd2462e Compare February 20, 2019 15:01
Andrew added 28 commits February 22, 2019 13:16
All of the VPN related shell commands would by default output a bunch to
the console. Instead of adding the option to suppress this to every line
it is much easier to simply extract this logic to a generic method and
pass the command into it.
This should indicate that the button's actions are more destructive in
nature than the rest.
Now that the VPN controls live on the main cluster page this logic needs
to move with them.
@atoghill atoghill force-pushed the feature/machine-management-tool branch from cd2462e to 5f7b14f Compare February 22, 2019 13:17
@atoghill atoghill merged commit 495a5e2 into develop Feb 22, 2019
@atoghill atoghill mentioned this pull request Feb 22, 2019
@atoghill atoghill deleted the feature/machine-management-tool branch February 22, 2019 13:19
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.

1 participant