-
Notifications
You must be signed in to change notification settings - Fork 489
Cost logs for API Deployments #1167
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
Merged
Merged
Changes from all commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
0919c34
Added logs for cost per file
pk-zipstack dccb7a0
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] b29a261
Fixed pre-sommit issues
pk-zipstack 12937cb
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] f52b1e9
fixed pre commit issues
pk-zipstack d0061a0
Update backend/workflow_manager/workflow_v2/execution.py
pk-zipstack 4fabd40
Update backend/workflow_manager/workflow_v2/execution.py
pk-zipstack 2c83d08
Made changes to log the total cost in get_aggregated_cost function
pk-zipstack 6fdbe5b
added error handling to prevent TypeError and ZeroDivisionError
pk-zipstack d4755ed
Removed devision by zero error for publish_average_cost_log
pk-zipstack a819e84
Updated the way the error logs are displayed to the user.
pk-zipstack a3ee996
Merge branch 'main' into UN-1799-Cost-log-for-ETL
pk-zipstack 8a138cb
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 4108df8
Moved publish_average_cost_log inside publish_final_workflow_logs
pk-zipstack 3394549
Update backend/workflow_manager/workflow_v2/execution.py
pk-zipstack e745bd7
Merge branch 'main' into UN-1799-Cost-log-for-ETL
pk-zipstack f940d33
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 7fc7b3e
Updated execution.py to fix line too long issue
pk-zipstack 1f33197
Converted log type "error" to "warning" in publish_average_cost_log
pk-zipstack 56dcf64
Moved get_aggregated_cost to WorkflowExectuion class as a property
pk-zipstack 965a3b7
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] fad083a
Update backend/workflow_manager/workflow_v2/models/execution.py
pk-zipstack dce0255
Update backend/workflow_manager/workflow_v2/models/execution.py
pk-zipstack 899638e
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 9f77f90
Merge branch 'main' into UN-1799-Cost-log-for-ETL
pk-zipstack 061db21
removed NoneType error handling in execution.py
pk-zipstack 948942c
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] f48b23d
Merge branch 'main' into UN-1799-Cost-log-for-ETL
pk-zipstack 6d62694
Merge branch 'main' into UN-1799-Cost-log-for-ETL
pk-zipstack 550019d
Merge branch 'main' into UN-1799-Cost-log-for-ETL
pk-zipstack 8e5fd12
Updated tool structure version
pk-zipstack 1a065cf
Update sample.env
pk-zipstack 471e6b9
Merge branch 'main' into UN-1799-Cost-log-for-ETL
pk-zipstack e71386c
Added .exists() for getting aggregated cost in execution.py
pk-zipstack babad72
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 64729ad
fixed line to long in execution.py
pk-zipstack 94e3289
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 82c423b
Removed redundant rounding of cost in exexution.py
pk-zipstack b6d95b8
Added indexing to execution_id in Usage model
pk-zipstack 9f9323a
Merge branch 'main' into UN-1799-Cost-log-for-ETL
pk-zipstack 3ac473c
Deleted a file
pk-zipstack d7d65c0
Updated structure tool version to 0.0.70
pk-zipstack File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
19 changes: 19 additions & 0 deletions
19
backend/usage_v2/migrations/0003_usage_usage_executi_4deb35_idx.py
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
# Generated by Django 4.2.1 on 2025-04-03 08:37 | ||
|
||
from django.db import migrations, models | ||
|
||
|
||
class Migration(migrations.Migration): | ||
|
||
dependencies = [ | ||
("usage_v2", "0002_alter_usage_run_id"), | ||
] | ||
|
||
operations = [ | ||
migrations.AddIndex( | ||
model_name="usage", | ||
index=models.Index( | ||
fields=["execution_id"], name="usage_executi_4deb35_idx" | ||
), | ||
), | ||
] |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -77,3 +77,4 @@ class SettingsKeys: | |
TOOL = "tool" | ||
METRICS = "metrics" | ||
INDEXING = "indexing" | ||
EXECUTION_ID = "execution_id" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.