-
Notifications
You must be signed in to change notification settings - Fork 21
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
Including Grafana Links #111
Conversation
local-compose.sh
Outdated
@@ -11,7 +11,7 @@ CPT_CONFIG=${CPT_CONFIG:-"$PWD/backend/ocpperf.toml"} | |||
podman rm -f front back | |||
|
|||
podman build -f backend/backend.containerfile --tag backend | |||
podman build -f frontend/frontend.containerfile --tag frontend | |||
# podman build -f frontend/frontend.containerfile --tag frontend |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For testing this PR perhaps?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
var min = ("0" + (d.getMinutes())).slice(-2); | ||
var sec = ("0" + (d.getSeconds())).slice(-2); | ||
return year + "-" + month + "-" + day + " " + hour + ":" + min + ":" + sec; | ||
let year = d.getFullYear(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just curious, does it matter between var
vs let
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using var
variables can be re-declared and updated within the same scope but with let
variables can be updated but can't be re-declared in the same scope.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Appart from the commented podman build
all looking good!! Thank you @MVarshini
Type of change
Description
Based on the systems build Grafana URL and display it on row expansion in OCP page table
Related Tickets & Documents
Checklist before requesting a review
Testing