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

Update Status pages to handle changes from removing nomad #953

Open
davidsmejia opened this issue Jun 30, 2021 · 2 comments
Open

Update Status pages to handle changes from removing nomad #953

davidsmejia opened this issue Jun 30, 2021 · 2 comments

Comments

@davidsmejia
Copy link
Collaborator

We need to handle the changed attribute names like nomad_pending_jobs in:

  • the placeholder src/apiData.json
  • src/pages/ExecDashboard/AppRunningSpeed.js
  • src/pages/Dashboard/Index.js

And also remove the now extraneous code for getRunningJobs

@wvauclain
Copy link
Contributor

One question that I have is what we want the Dashboard to look like when we are running the API locally (not in AWS). Right now, the API doesn't add any batch data when not running in the cloud, but depending on how the dashboard is written this might cause problems.

@kurtwheeler
Copy link
Contributor

I don't think the dashboard is particularly useful if we're running the API locally as there's no Batch for it to pull stats from.

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

3 participants