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

Archive leftoverResults logs for perf benchmarks runs #5615

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

sophiaxu0424
Copy link
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
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
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 marked this pull request as ready for review September 17, 2024 19:53
@sophiaxu0424
Copy link
Contributor Author

@llxia
Copy link
Contributor

llxia commented Sep 17, 2024

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

Copy link
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

@LongyuZhang LongyuZhang merged commit 0c3f98b into adoptium:master Sep 17, 2024
2 checks passed
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