-
Notifications
You must be signed in to change notification settings - Fork 65
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 info about the grafana deployer action and api key #929
Add info about the grafana deployer action and api key #929
Conversation
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 a few quick thoughts - looks good in general though!
I just tried inviting the @2i2c-org/tech-team to review this, to see if we can test out our slack notification :-) |
Fix typos Co-authored-by: Chris Holdgraf <choldgraf@gmail.com>
Co-authored-by: Chris Holdgraf <choldgraf@gmail.com>
Thanks @choldgraf! I'll postpone merging it to check that we get the Slack reminders 🚀 |
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.
I made some suggestions related to the confusing terminology by GitHub.
My understanding is that:
- "GitHub Actions" is like saying "TravisCI" or "CircleCI", it is the name of the system to run CI/CD stuff.
- Within GitHub Actions, GitHub's CI system, you can run "workflows", and each workflow has a list of "jobs", and each job has a list of "steps".
- A "GitHub action" is a pre-defined "step" that can be re-used across workflows defined in projects across GitHub
Co-authored-by: Erik Sundell <erik.i.sundell@gmail.com>
Co-authored-by: Erik Sundell <erik.i.sundell@gmail.com>
Co-authored-by: Erik Sundell <erik.i.sundell@gmail.com>
Thank you @consideRatio for clarifying these 🌼 |
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.
LGTM!
Thanks everyone 🌼 🚀 |
When researching about #328 realized that we haven't documented the grafana deployer action and where the api key is stored right now.
I believe it closes 2i2c-org/team-compass#176
Probably this will change when we'll have one grafana for all the clusters.