Skip to content

currency: Automate python tracer currency report generation#528

Merged
GSVarsha merged 9 commits into
masterfrom
currency-update
May 24, 2024
Merged

currency: Automate python tracer currency report generation#528
GSVarsha merged 9 commits into
masterfrom
currency-update

Conversation

@GSVarsha

@GSVarsha GSVarsha commented May 8, 2024

Copy link
Copy Markdown
Contributor

Automate python tracer currency report generation and automatically push the report to the tracer-reports repository everyday if updates available.

@GSVarsha GSVarsha self-assigned this May 8, 2024
@GSVarsha GSVarsha added the currency Keeping supported versions updated. label May 8, 2024
@GSVarsha GSVarsha added this to the H1-2024 milestone May 8, 2024
@GSVarsha GSVarsha requested review from Ferenc- and pvital May 8, 2024 08:17

@Ferenc- Ferenc- left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could we also extend the tekton documentation?

Comment thread .tekton/.currency/utils/tekton-ci-output.txt Outdated
Comment thread .tekton/.currency/scripts/get-tekton-ci-output.sh Outdated
Comment thread .tekton/.currency/scripts/get-tekton-ci-output.sh Outdated
Comment thread .tekton/.currency/scripts/get-tekton-ci-output.sh Outdated
Comment thread .tekton/.currency/scripts/generate_report.py Outdated
Comment thread .tekton/.currency/currency-tasks.yaml Outdated
Comment thread .tekton/.currency/currency-tasks.yaml Outdated
Comment thread .tekton/.currency/scripts/get-tekton-ci-output.sh Outdated
Comment thread .tekton/.currency/scripts/generate_report.py Outdated
Comment thread .tekton/.currency/scripts/generate_report.py Outdated
@GSVarsha GSVarsha force-pushed the currency-update branch 4 times, most recently from e1d5d2e to d8118eb Compare May 13, 2024 08:02
@GSVarsha

Copy link
Copy Markdown
Contributor Author

Could we also extend the tekton documentation?

@Ferenc- I'm thinking of opening a separate PR for documentation since this PR is already huge. Should I create a separate README.md file under .tekton/.currency?

@GSVarsha GSVarsha requested a review from Ferenc- May 13, 2024 08:36
@Ferenc-

Ferenc- commented May 13, 2024

Copy link
Copy Markdown
Member

Could we also extend the tekton documentation?

@Ferenc- I'm thinking of opening a separate PR for documentation since this PR is already huge. Should I create a separate README.md file under .tekton/.currency?

That's fine by me.

Comment thread .tekton/.currency/scripts/generate_report.py Outdated
Comment thread .tekton/.currency/scripts/generate_report.py Outdated
Comment thread .tekton/.currency/scripts/generate_report.py Outdated
@GSVarsha GSVarsha requested a review from Ferenc- May 14, 2024 04:36
Comment thread .tekton/.currency/currency-rbac.yaml Outdated
Comment thread .tekton/.currency/currency-rbac.yaml Outdated
Comment thread .tekton/.currency/currency-tasks.yaml Outdated
Comment thread .tekton/.currency/scripts/generate_report.py Outdated
Comment thread .tekton/.currency/scripts/generate_report.py Outdated
Comment thread .tekton/.currency/scripts/generate_report.py Outdated
Comment thread .tekton/.currency/scripts/generate_report.py Outdated
@GSVarsha GSVarsha requested a review from Ferenc- May 20, 2024 15:10
Comment thread .tekton/.currency/scripts/generate_report.py Outdated
@GSVarsha GSVarsha requested a review from Ferenc- May 22, 2024 07:20
Comment thread .tekton/.currency/currency-tasks.yaml Outdated

@Ferenc- Ferenc- left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

GSVarsha added 2 commits May 24, 2024 21:41
Signed-off-by: Varsha GS <varsha.gs@ibm.com>
Signed-off-by: Varsha GS <varsha.gs@ibm.com>
GSVarsha added 6 commits May 24, 2024 21:41
Signed-off-by: Varsha GS <varsha.gs@ibm.com>
…uster

Signed-off-by: Varsha GS <varsha.gs@ibm.com>
…cessful git clone

Signed-off-by: Varsha GS <varsha.gs@ibm.com>
…t-starlette-task

Signed-off-by: Varsha GS <varsha.gs@ibm.com>
Signed-off-by: Varsha GS <varsha.gs@ibm.com>
Signed-off-by: Varsha GS <varsha.gs@ibm.com>
- add docstrings
- uploadlatest currency report

Signed-off-by: Varsha GS <varsha.gs@ibm.com>
@GSVarsha GSVarsha merged commit 5c31b63 into master May 24, 2024
@GSVarsha GSVarsha deleted the currency-update branch May 24, 2024 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

currency Keeping supported versions updated.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants