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

Add SCI dashboard for Falco #72

Merged
merged 1 commit into from
Mar 13, 2024
Merged

Add SCI dashboard for Falco #72

merged 1 commit into from
Mar 13, 2024

Conversation

nikimanoledaki
Copy link
Contributor

@nikimanoledaki nikimanoledaki commented Mar 12, 2024

What type of PR is this?

kind/feature

What this PR does / why we need it:

This PR adds the scaffolding for a Grafana dashboard that shows the SCI of Falco. There are still a few TODOs! I wanted to get early feedback on the progress @raymundovr and I have made so far.

The dashboard can be found here. Checkout the login info for the public Grafana instance here.

Which issue(s) this PR fixes:

Part of #38
Part of #39

^ not closing either until docs are added.

-->

Special notes for your reviewer (optional):

Will open a separate PR for adding info to docs :)

@nikimanoledaki nikimanoledaki force-pushed the add-sci-dashboard branch 6 times, most recently from 07906c1 to dc61836 Compare March 12, 2024 12:43
Co-authored-by: raymundovr <raymundo.vr@protonmail.com>

Signed-off-by: nikimanoledaki <niki.manoledaki@grafana.com>
Copy link
Contributor

@rossf7 rossf7 left a comment

Choose a reason for hiding this comment

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

@nikimanoledaki @raymundovr This is awesome, great work!

I left some comments including with the value of M for the Equinix server.

clusters/base/falco-sci.yaml Show resolved Hide resolved
"sort": "none"
}
},
"title": "Embodied (M) - Falco",
Copy link
Contributor

Choose a reason for hiding this comment

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

@nikimanoledaki the value of M for the m3.small we're using is 3.92 gCO2eq per 15 mins.
Could you add that?

See #73 for the write up

Copy link
Contributor

Choose a reason for hiding this comment

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

I'll add this to the calculations :)

clusters/base/falco-sci.yaml Show resolved Hide resolved
Copy link
Member

@leonardpahlke leonardpahlke left a comment

Choose a reason for hiding this comment

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

Just took a very quick look. Great, looks good to me as a first version that we will iterate and refine over the next months.

Copy link
Contributor

@guidemetothemoon guidemetothemoon left a comment

Choose a reason for hiding this comment

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

LGTM, very cool and a great start! Great work, Niki and Raymundo😃

@leonardpahlke leonardpahlke merged commit dca1a77 into main Mar 13, 2024
1 check passed
@leonardpahlke leonardpahlke deleted the add-sci-dashboard branch March 13, 2024 12:03
@raymundovr raymundovr mentioned this pull request Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants