[testing-on-gke part 6.9] Export output to a gsheet #2524
Conversation
This file contains 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
gargnitingoogle
commented
Sep 20, 2024
perfmetrics/scripts/testing_on_gke/examples/utils/gsheet_test.py
Outdated
Show resolved
Hide resolved
gargnitingoogle
changed the title
gargnitingoogle
changed the title
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/improvements-in-run-script
branch
from
169a527
to
b927c1b
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
427c67b
to
2cb51c3
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/improvements-in-run-script
branch
from
b927c1b
to
edb8dc3
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
2 times, most recently
from
5264bbf
to
09294ec
Compare
gargnitingoogle
commented
Sep 24, 2024
|
|
||
| exportToCsvFile(output_file_path=args.output_file, header=_HEADER, rows=rows) | ||
|
|
||
| def exportToGsheet( |
There was a problem hiding this comment.
Move this function to parse_logs_common
| f" '{args.output_worksheet_name}' in {url(args.output_gsheet_id)}" | ||
| ) | ||
|
|
||
| exportToGsheet( |
There was a problem hiding this comment.
Move this function to parse_logs_common
|
|
||
| exportToCsvFile(output_file_path=args.output_file, header=_HEADER, rows=rows) | ||
|
|
||
| def exportToGsheet( |
There was a problem hiding this comment.
Move this function to parse_logs_common
| f" '{args.output_worksheet_name}' in {url(args.output_gsheet_id)}" | ||
| ) | ||
|
|
||
| exportToGsheet( |
There was a problem hiding this comment.
Move this function to parse_logs_common
perfmetrics/scripts/testing_on_gke/examples/utils/parse_logs_common.py
Outdated
Show resolved
Hide resolved
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
09294ec
to
3bac14d
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/improvements-in-run-script
branch
from
edb8dc3
to
8e19d51
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
2 times, most recently
from
a94bc19
to
5dc2ea9
Compare
gargnitingoogle
changed the base branch from
garnitin/add-gke-load-testing/improvements-in-run-script
to
garnitin/add-gke-load-testing/add-gsheet-utility
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
3a6f7ab
to
172032d
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
5dc2ea9
to
6f9e067
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
172032d
to
390fe46
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
6f9e067
to
a73f23a
Compare
gargnitingoogle
requested review from
sethiay
and removed request for
a team
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
390fe46
to
b081112
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
a73f23a
to
06ab8c9
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
b081112
to
3ed138f
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
06ab8c9
to
f908de4
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
3ed138f
to
9d14972
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
2bf79fa
to
eb09af2
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
417e02f
to
a1a2dc1
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
eb09af2
to
78240a9
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
a1a2dc1
to
ea6eefa
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
78240a9
to
18c2f27
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
ea6eefa
to
4d51a48
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
18c2f27
to
1a38752
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
4d51a48
to
79cf9be
Compare
gargnitingoogle
changed the title
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
1a38752
to
f524114
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
79cf9be
to
780519d
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
f524114
to
e2ff044
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
780519d
to
f50caca
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
e2ff044
to
3af14bf
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
f50caca
to
0018699
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
3af14bf
to
3d46fbd
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
0018699
to
a296868
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
3d46fbd
to
2dde58b
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
a296868
to
773695f
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
2dde58b
to
6b4bfa1
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
773695f
to
5bf8515
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
6b4bfa1
to
b1f82bc
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
5bf8515
to
a998865
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
b1f82bc
to
89a9ada
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
a998865
to
ed56ec3
Compare
add generic utility to append to a gsheet Add utility for gsheet improve gsheet utility export fio output to gsheet encapsulate cpu/memory calculation in fio disable repeat operations for quick testing add dlio output export to gsheet fix a bug in dlio output parsing fix a column-name in fio csv output Revert "disable repeat operations for quick testing" This reverts commit 04bf834. add log of successful addition to gsheet clean-up code changes added some error-checking wip wip fix calls to download_gcs_objects support key-file on gcs in gsheet put back cpu/memory metrics fix couple of logs fix couple of help messages put back accidentally deleted command
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/add-gsheet-utility
branch
from
89a9ada
to
1fcf030
Compare
gargnitingoogle
force-pushed
the
garnitin/add-gke-load-testing/upload-outputs-to-gsheet
branch
from
d50d8d8
to
e6a68ea
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
This is on top of #2528 and is followed up in #2538 .
Link to the issue in case of a bug fix.
NA
Testing details