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 OpenCost Support, Update README, and Refactor GitHub Actions #1347

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chiukapoor
Copy link
Contributor

Updates

  • Create a script for installing the KubePlus operator and KubePlus kubectl plugin
  • Update README.md to include a "Quick Installation" section and move the getting-started example to a separate file to streamline the README
  • Add an OpenCost example to the getting-started file
  • Update GitHub Actions PR workflow to use the install.sh script for installing the KubePlus operator and kubectl plugin; modularized into distinct steps

Issues

@chiukapoor chiukapoor force-pushed the opencost branch 8 times, most recently from 9296f83 to 2925ebb Compare August 28, 2024 13:03
* Create a script for installing the KubePlus operator and KubePlus kubectl plugin
* Update README.md to include a "Quick Installation" section and move the getting-started example to a separate file to streamline the README
* Add an OpenCost example to the getting-started file
* Update GitHub Actions PR workflow to use the install.sh script for installing the KubePlus operator and kubectl plugin; modularized into distinct steps
@chiukapoor chiukapoor marked this pull request as draft August 29, 2024 17:36
@devdattakulkarni
Copy link
Contributor

@chiukapoor Looks like this PR was adding the Opencost and Prometheus installation as part of install.sh.

I would suggest let's not do that. We can just add a section in getting-started (https://github.com/cloud-ark/kubeplus/blob/master/examples/getting-started.md) towards the end showing the steps on how to use Opencost.

You can name it:
Getting per instance costs using Opencost

Add it after step 8 in the current getting started doc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants