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

miniweb: braver and newer web #987

Merged
merged 13 commits into from Sep 25, 2017
Merged

Conversation

@jcrussell
Copy link
Contributor

@jcrussell jcrussell commented Sep 24, 2017

minimega:

  • display more info about namespaces in namespace API

miniweb:

  • force namespace with -namespace flag:
    /foo/vms gives you VMs from foo. /vms gives you VMs from the current
    namespace. /foo/bar/vms gives you an error.
  • add namespaces page: displays info about different namespaces
  • add files pages: directory walk of files for hosts in the namespace
  • add vlans page: displays VLAN aliases
  • add VM controls: start/stop/kill VMs

Fixes a few issues: #870, #872, #876.
Partially implements #873.

jcrussell added 8 commits Sep 22, 2017
Prepended all commands with namespace command. Fixes #876.
Add tabularHandler to replace hosts and vlans handlers.
/foo/vms gives you VMs from foo. /vms gives you VMs from the current
namespace. /foo/bar/vms gives you an error.

Change a bunch of relative paths so that we keep /foo when navigating
around. Make images, JavaScript, and other resources absolute.

Only thing missing is a way to see which namespaces there are.

Fixes #870.
Change `namespace` API so that it lists more information about the
namespaces in tabular form.
Displays list of namespaces with link to VMs page for each namespace.
Supports walking the directory tree and viewing VLAN aliases. Does not
allow downloading or uploading files.

Updates #873.
Need a new icon for...
Add functionality to start/stop/kill VMs from the VM page. Move the
connect and screenshot APIs underneath a /vm/ path along with controls.

Fixes #872.
@jcrussell jcrussell requested a review from jasoncontrib Sep 25, 2017
Should always use the -namespace and ignore one passed in the URL.
@jcrussell
Copy link
Contributor Author

@jcrussell jcrussell commented Sep 25, 2017

Small bug: namespace counts VMs on the current host and not across the namespace.

jcrussell added 4 commits Sep 25, 2017
Fix clipping and wrong arg to getElementById.
Could be useful elsewhere...
The VM count was limited to just VMs running on the head node. Tried
collecting host stats from all the nodes but that causes a deadlock
since InfoNamespaces holds the namespaceLock. Functional for now, could
fix later.
Deleted VMs column and didn't fix index for VLAN string.
@floren
floren approved these changes Sep 25, 2017
@floren floren merged commit 065f3b5 into sandia-minimega:master Sep 25, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jcrussell jcrussell deleted the jcrussell:braver-newer-web branch Sep 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants