Director interface #1623
Merged
Director interface #1623
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
CannonLock
commented
Oct 8, 2024
CannonLock
commented
- Add Origin metrics interface v1
- Add Director metrics interface v1
- Add Graph Overlay and Context for control
- Move time context to root layout
- Update the Nginx Proxy to auto reload credentials
- Add test framework and tests
- Add Boxplot graphs
- Add usage Bar graphs
- Add Origin metrics interface v1 - Add Graph Overlay and Context for control - Move time context to root layout - Update the Nginx Proxy to auto reload credentials
- Add generalized metrics box plots - Add Transferred Bytes Bar Graph - Add Storage Table - Add url saving for range information
- Lint and SSR when possible
- Add Dates
This was
linked to
issues
Oct 8, 2024
- Log scaled the boxplots
- Log scaled the boxplots
|
@h2zh I made the changes we discussed. Do you mind approving this PR when you get a chance? |
h2zh
approved these changes
Oct 9, 2024
h2zh
requested changes
Oct 9, 2024
There was a problem hiding this comment.
seems like the ubuntu check failure might be a problem. btw I re-run the ubuntu check in the latest merged PR (#1633) and it still succeeds. I consulted Emma for this and she suggested not to merge it in in case breaking the test completely in main
h2zh
approved these changes
Oct 9, 2024
|
Overriding integration test failure - known that was broken when the PR was posted. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.