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

Added Cost Estimate Workflow job #477

Merged
merged 47 commits into from
Jan 25, 2023
Merged

Added Cost Estimate Workflow job #477

merged 47 commits into from
Jan 25, 2023

Conversation

Gordonby
Copy link
Collaborator

@Gordonby Gordonby commented Nov 28, 2022

PR Summary

Adds a new job to the workflow that estimates the monthly cost of the deployment, which is then saved as a workflow artifact.

The purpose is to showcase additional features that can be integrated into Customers CI/CD pipelines, this does not affect the bicep/helper in any way.

image

Closes #469

BYO workflow

  • Added cost prediction for environment. Ref Cost Estimates #469
  • Bumped versions of dependant actions

Json output from the Estimate workflow step;

image

Private cluster workflow

OSS Workflow

Additionally it solves the "OSS workflow, Get Latest AKSC release" item in #454 (Get latest AKSC version)

PR Checklist

  • PR has a meaningful title
  • Summarized changes
  • This PR is ready to merge and is not Work in Progress
  • Link to a filed issue

@Gordonby Gordonby temporarily deployed to csu November 28, 2022 09:56 Inactive
@Gordonby Gordonby temporarily deployed to csu November 28, 2022 09:57 Inactive
@Gordonby Gordonby temporarily deployed to csu November 28, 2022 10:02 Inactive
@Gordonby Gordonby temporarily deployed to csu November 29, 2022 08:48 Inactive
@Gordonby Gordonby temporarily deployed to csu November 29, 2022 08:48 Inactive
@Gordonby Gordonby temporarily deployed to csu November 29, 2022 08:55 Inactive
@Gordonby Gordonby temporarily deployed to csu November 29, 2022 08:58 Inactive
@Gordonby Gordonby temporarily deployed to csu November 29, 2022 09:23 Inactive
@Gordonby Gordonby temporarily deployed to csu November 29, 2022 11:48 Inactive
@Gordonby
Copy link
Collaborator Author

Once this issue gets fixed, i think we're good to merge this PR.

TheCloudTheory/arm-estimator#149

@Gordonby Gordonby enabled auto-merge (squash) January 17, 2023 16:15
@Gordonby
Copy link
Collaborator Author

Once this issue gets fixed, i think we're good to merge this PR.

TheCloudTheory/arm-estimator#149

Ready!

@Gordonby Gordonby requested a review from a team January 20, 2023 08:44
@Gordonby
Copy link
Collaborator Author

Thanks for the feedback @samaea
I've set the workflow to not run Cost Estimate on PR.

image

@Gordonby Gordonby modified the milestones: 0.9.8, 0.9.9 Jan 24, 2023
@Gordonby Gordonby added the ci-cd sample 🤖 Sample code in a GitHub action workflow label Jan 25, 2023
@Gordonby Gordonby temporarily deployed to csu January 25, 2023 16:09 — with GitHub Actions Inactive
@Gordonby Gordonby temporarily deployed to csu January 25, 2023 16:09 — with GitHub Actions Inactive
@Gordonby Gordonby temporarily deployed to csu January 25, 2023 16:17 — with GitHub Actions Inactive
@Gordonby Gordonby temporarily deployed to csu January 25, 2023 16:33 — with GitHub Actions Inactive
@Gordonby Gordonby temporarily deployed to csu January 25, 2023 16:33 — with GitHub Actions Inactive
@Gordonby Gordonby temporarily deployed to csu January 25, 2023 16:33 — with GitHub Actions Inactive
@Gordonby Gordonby temporarily deployed to csu January 25, 2023 16:33 — with GitHub Actions Inactive
Copy link
Contributor

@samaea samaea left a comment

Choose a reason for hiding this comment

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

Lgtm 👍

@Gordonby Gordonby merged commit 2a790de into main Jan 25, 2023
@Gordonby Gordonby deleted the gb-costestiamte branch January 25, 2023 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-cd sample 🤖 Sample code in a GitHub action workflow enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cost Estimates
2 participants