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.

Sign up for GitHub

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

Jump to bottom

TRSS query should not have _xl in the plfatform #557

Open
llxia opened this issue May 29, 2024 · 0 comments
Open

TRSS query should not have _xl in the plfatform #557

llxia opened this issue May 29, 2024 · 0 comments

Comments

@llxia
Copy link
Contributor

llxia commented May 29, 2024 •
edited
Loading

TKG queries TRSS for test execution time. The platform for aarch64_mac should not have _xl

00:02:11.422  Attempting to get test duration data from TRSS.
00:02:11.422  curl --silent --max-time 120 https://trssrtp1.fyre.ibm.com/api/getTestAvgDuration?limit=10&jdkVersion=11&impl=j9&platform=aarch64_mac_xl&group=functional&level=sanity

aarch64_mac test build
x86-64_mac test build is correct.

source code:

TKG/src/org/testKitGen/TestDivider.java

Line 228 in 3ed8f8b

String URL = constructURL(impl, plat, group, level);

annaibm added a commit to annaibm/TKG that referenced this issue Aug 13, 2024
@annaibm
Removed _xl for mac platform in TRSS query
fd80ed5 
- The platform for aarch64_mac should not include _xl in TRSS url when querying for test execution times,
- Adjusted the code to remove _xl from the platform string if the OS is macOS.

related: adoptium#557

Signed-off-by: Anna Babu Palathingal <[email protected]>
@annaibm annaibm mentioned this issue Aug 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Adoptium Backlog
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
1 participant
@llxia