Skip to content

Conversation

@jasonhawkharris
Copy link
Contributor

@jasonhawkharris jasonhawkharris commented May 18, 2023

Test plan

  • Shipped with unit tests.
  • Tested manually with healthy, and unhealthy k8s instances of sourcegraph

jasonhawkharris and others added 30 commits April 28, 2023 16:12
- Prints all current resource allocations for each service (CPU, Memory, Storage) to the terminal
- Previously, you could only run `scout` on a k8s deployment
- Now, by specifying the `--docker` flag, you can run against a docker
  deployment
- Added unit tests for ResourcesK8s and ResourcesDocker
Co-authored-by: Jacob Pleiness <jdpleiness@users.noreply.github.com>
Co-authored-by: Jacob Pleiness <jdpleiness@users.noreply.github.com>
Co-authored-by: Jacob Pleiness <jdpleiness@users.noreply.github.com>
Co-authored-by: Jacob Pleiness <jdpleiness@users.noreply.github.com>
Co-authored-by: Jacob Pleiness <jdpleiness@users.noreply.github.com>
Co-authored-by: Jacob Pleiness <jdpleiness@users.noreply.github.com>
Co-authored-by: Jacob Pleiness <jdpleiness@users.noreply.github.com>
Co-authored-by: Jacob Pleiness <jdpleiness@users.noreply.github.com>
Co-authored-by: Jacob Pleiness <jdpleiness@users.noreply.github.com>
- change package from resources to resource
- renamed files accordingly
- changed from if statement to switch statement.
- deleted empty README.md
@jasonhawkharris jasonhawkharris requested review from jac and jdpleiness May 18, 2023 19:08
Copy link
Contributor

@jdpleiness jdpleiness left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good 😎

A few suggestions here and there.

@jasonhawkharris jasonhawkharris marked this pull request as ready for review May 22, 2023 17:05
Copy link
Contributor

@jdpleiness jdpleiness left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two minor fixes, but overall looks good!

@jasonhawkharris jasonhawkharris merged commit 4766869 into main May 23, 2023
@jasonhawkharris jasonhawkharris deleted the jhh/usage branch May 23, 2023 14:41
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.

3 participants