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.

Sign up for GitHub

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

Jump to bottom

FinOps on EKS #33

Merged
merged 46 commits into from
Feb 13, 2025
Merged

FinOps on EKS #33

merged 46 commits into from
Feb 13, 2025

Conversation

maria-reynoso
Copy link
Contributor

No description provided.

@maria-reynoso maria-reynoso marked this pull request as ready for review November 29, 2024 16:00
cyberbliss
cyberbliss previously approved these changes Nov 29, 2024
View reviewed changes
jontutcher
jontutcher reviewed Dec 2, 2024
View reviewed changes
jontutcher
jontutcher reviewed Dec 3, 2024
View reviewed changes
maria-reynoso and others added 23 commits February 13, 2025 16:52
@maria-reynoso
helmfile
f3f129f
@maria-reynoso
prometheus exporter
360eee1
@maria-reynoso
Update docs
7ac297a
@maria-reynoso
new line
75c5cf1
@maria-reynoso
update
c5ad787
@maria-reynoso
env eks file
4de7211
@maria-reynoso
fix helmfile apply command
e512688
@maria-reynoso
fix opencost templates chart
c86edde
@maria-reynoso
fix opencost
5f3193b
@maria-reynoso
update env file
461662e
@maria-reynoso
fix opencost template chart
05c2e2d
@maria-reynoso
remove opencost from prometheus
ac75e15
@maria-reynoso
no aws access key needed
37a216c
@maria-reynoso
add dashboard to kind
0b0fefc
@maria-reynoso
update doc
0a8a84b
@maria-reynoso
new line
c066b71
@maria-reynoso
Re-word
4a64949
@maria-reynoso
Add scan pipeline
f7aec59
@dependabot @maria-reynoso
Bump golang.org/x/crypto in /scripts/chart-dep-updater (#37)
da27454 
Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.26.0 to 0.31.0.
- [Commits](golang/crypto@v0.26.0...v0.31.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@maria-reynoso
Re-word
6126f26
@maria-reynoso
Update docs
9a46312
@maria-reynoso
Update docs
57bf639
@maria-reynoso
Merge branch 'add-eks-support' of https://github.com/jetstack/finops-…
8b78949 
…stack into add-eks-support
jontutcher
jontutcher approved these changes Feb 13, 2025
View reviewed changes
Copy link
Contributor

@jontutcher jontutcher left a comment

Choose a reason for hiding this comment

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

I'm ok with this going in now - docs / structure should be re-factored (across all install methods). Maybe neaten up / squash the commits in the PR before merging?

davidcollom
davidcollom approved these changes Feb 13, 2025
View reviewed changes
Copy link
Collaborator

@davidcollom davidcollom left a comment

Choose a reason for hiding this comment

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

If @jontutcher 's good - I'm good! 😆

@davidcollom davidcollom merged commit 957115e into main Feb 13, 2025
4 checks passed
@davidcollom davidcollom deleted the add-eks-support branch February 13, 2025 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidcollom davidcollom davidcollom approved these changes

@jontutcher jontutcher jontutcher approved these changes

@BeckyPauley BeckyPauley Awaiting requested review from BeckyPauley BeckyPauley is a code owner

@cyberbliss cyberbliss Awaiting requested review from cyberbliss cyberbliss is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@maria-reynoso @davidcollom @cyberbliss @jontutcher