Skip to content

Archive leftoverResults logs for perf benchmarks runs#5615

Merged
LongyuZhang merged 1 commit intoadoptium:masterfrom
sophiaxu0424:issue40
Sep 17, 2024
Merged

Archive leftoverResults logs for perf benchmarks runs#5615
LongyuZhang merged 1 commit intoadoptium:masterfrom
sophiaxu0424:issue40

Conversation

@sophiaxu0424
Copy link
Copy Markdown
Contributor

@sophiaxu0424 sophiaxu0424 commented Sep 17, 2024

We need to make sure the leftoverresults folder's logs for benchmarks runs are correctly archived and uploaded to artifactory

Fixes: https://github.ibm.com/runtimes/automation/issues/40

@sophiaxu0424
Copy link
Copy Markdown
Contributor Author

sophiaxu0424 commented Sep 17, 2024

Posted the test links in our issue: https://github.ibm.com/runtimes/automation/issues/40#issuecomment-91185512

@sophiaxu0424 sophiaxu0424 changed the title Archive leftoverResults logs to artifactory Archive leftoverResults logs for perf benchmarks runs Sep 17, 2024
@llxia
Copy link
Copy Markdown
Contributor

llxia commented Sep 17, 2024

@sophiaxu0424 Please use git keywords for linking issues. Please refer to https://github.com/adoptium/aqa-tests/blob/master/Contributing.md for details.

@sophiaxu0424 sophiaxu0424 marked this pull request as draft September 17, 2024 18:52
@sophiaxu0424 sophiaxu0424 force-pushed the issue40 branch 2 times, most recently from 6b6e74a to 117bb76 Compare September 17, 2024 18:57
@sophiaxu0424 sophiaxu0424 marked this pull request as ready for review September 17, 2024 19:53
@sophiaxu0424 sophiaxu0424 requested a review from llxia September 17, 2024 19:53
Fixes: https://github.ibm.com/runtimes/automation/issues/40

Signed-off-by: sophiaxu0424 <xuminghong0424@gmail.com>
@sophiaxu0424
Copy link
Copy Markdown
Contributor Author

@llxia
Copy link
Copy Markdown
Contributor

llxia commented Sep 17, 2024

re #5615 (comment), please update PR description as well.

Copy link
Copy Markdown
Contributor

@llxia llxia left a comment

Choose a reason for hiding this comment

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

LGTM

@llxia llxia requested a review from LongyuZhang September 17, 2024 21:06
@LongyuZhang LongyuZhang merged commit 0c3f98b into adoptium:master Sep 17, 2024
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.

3 participants