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

Query git issue url to get issue status #2701

Closed
llxia opened this issue Jun 30, 2021 · 6 comments
Closed

Query git issue url to get issue status #2701

llxia opened this issue Jun 30, 2021 · 6 comments
Assignees

Comments

@llxia
Copy link
Contributor

llxia commented Jun 30, 2021

As part of Create a Disabled tests pipeline, we need to query GIT_ISSUE url to get the issue status (open/closed).

  • input JSON file:
[{
"JDK_VERSION" : 17,
"JDK_IMPL" : "hotspot",
"TARGET" : "jdk_custom",
"CUSTOM_TARGET" : "java/lang/ClassLoader/nativeLibrary/NativeLibraryTest.java",
"GIT_ISSUE" : "https://github.com/adoptium/temurin-build/issues/248",
"PLATFORM" : "all"
},{
"JDK_VERSION" : 16,
"JDK_IMPL" : "openj9",
"TARGET" : "jdk_custom",
"CUSTOM_TARGET" : "java/lang/ClassLoader/Assert.java",
"GIT_ISSUE" : "https://github.com/eclipse-openj9/openj9/issues/6668",
"PLATFORM" : "x86-64_mac"
}]
  • query git to get the issue status and add GIT_ISSUE_STATUS in JSON file. Expected output:
[{
"JDK_VERSION" : 17,
"JDK_IMPL" : "hotspot",
"TARGET" : "jdk_custom",
"CUSTOM_TARGET" : "java/lang/ClassLoader/nativeLibrary/NativeLibraryTest.java",
"GIT_ISSUE" : "https://github.com/adoptium/temurin-build/issues/248",
"PLATFORM" : "all",
"GIT_ISSUE_STATUS": "closed"
},{
"JDK_VERSION" : 16,
"JDK_IMPL" : "openj9",
"TARGET" : "jdk_custom",
"CUSTOM_TARGET" : "java/lang/ClassLoader/Assert.java",
"GIT_ISSUE" : "https://github.com/eclipse-openj9/openj9/issues/6668",
"PLATFORM" : "x86-64_mac",
"GIT_ISSUE_STATUS": "open"
}]
@llxia
Copy link
Contributor Author

llxia commented Sep 8, 2021

@xius666
Copy link
Contributor

xius666 commented Sep 18, 2021

I finished and please review it thanks

@King-kay
Copy link
Contributor

I would like to fix this issue. @smlambert

@King-kay
Copy link
Contributor

@smlambert please check out #2974 (comment)

@smlambert
Copy link
Contributor

@King-kay - I believe @xius666 has already resolved this one (but we failed to assign him), you'll have to find another issue to grab.

@smlambert
Copy link
Contributor

Fixed via #2882

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants