{"action":"created","comment":{"url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/comments/240204088","pull_request_review_id":183176410,"id":240204088,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDI0MDIwNDA4OA==","diff_hunk":"@@ -289,12 +289,13 @@ def post(test_target) {\n \t\t\t\t}\n \t\t\t}\n \t\t\t//for performance test, achive regardless the build result\n+\t\t\t//TODO: Need to soft-code the output dir path\n \t\t\tdef benchmark_test_output_dir = 'jvmtest/performance/odm/ilog_wodm881/leftoverResults';\n \t\t\tif (fileExists(benchmark_test_output_dir)) {\n-\t\t\t\tsh \"tar -zcf ${testTarName} benchmark_test_output.tar.gz\"\n+\t\t\t\tsh \"zip -r benchmarkOutput.zip ${benchmark_test_output_dir}\"\n \t\t\t\tif (!params.ARTIFACTORY_SERVER) {\n \t\t\t\t\techo \"ARTIFACTORY_SERVER is not set. Saving artifacts on jenkins.\"\n-\t\t\t\t\tarchiveArtifacts artifacts: 'benchmark_test_output.tar.gz', fingerprint: true, allowEmptyArchive: true\n+\t\t\t\t\tarchiveArtifacts artifacts: 'benchmarkOutput.zip', fingerprint: true, allowEmptyArchive: true","path":"buildenv/jenkins/JenkinsfileBase","position":21,"original_position":21,"commit_id":"b70ec0f368c3b968619f92d69ed29ca47efe7b2e","original_commit_id":"b70ec0f368c3b968619f92d69ed29ca47efe7b2e","user":{"login":"llxia","id":19273206,"node_id":"MDQ6VXNlcjE5MjczMjA2","avatar_url":"https://avatars0.githubusercontent.com/u/19273206?v=4","gravatar_id":"","url":"https://api.github.com/users/llxia","html_url":"https://github.com/llxia","followers_url":"https://api.github.com/users/llxia/followers","following_url":"https://api.github.com/users/llxia/following{/other_user}","gists_url":"https://api.github.com/users/llxia/gists{/gist_id}","starred_url":"https://api.github.com/users/llxia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/llxia/subscriptions","organizations_url":"https://api.github.com/users/llxia/orgs","repos_url":"https://api.github.com/users/llxia/repos","events_url":"https://api.github.com/users/llxia/events{/privacy}","received_events_url":"https://api.github.com/users/llxia/received_events","type":"User","site_admin":false},"body":"Any reason for changing tar to zip. All the `*_test_output.tar.gz` will later upload to artifactory. This change breaks file upload later.","created_at":"2018-12-10T13:16:03Z","updated_at":"2018-12-10T13:16:10Z","html_url":"https://github.com/AdoptOpenJDK/openjdk-tests/pull/737#discussion_r240204088","pull_request_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/737","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/comments/240204088"},"html":{"href":"https://github.com/AdoptOpenJDK/openjdk-tests/pull/737#discussion_r240204088"},"pull_request":{"href":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/737"}}},"pull_request":{"url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/737","id":237010776,"node_id":"MDExOlB1bGxSZXF1ZXN0MjM3MDEwNzc2","html_url":"https://github.com/AdoptOpenJDK/openjdk-tests/pull/737","diff_url":"https://github.com/AdoptOpenJDK/openjdk-tests/pull/737.diff","patch_url":"https://github.com/AdoptOpenJDK/openjdk-tests/pull/737.patch","issue_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/issues/737","number":737,"state":"open","locked":false,"title":"Deploy ODM Benchmark","user":{"login":"piyush286","id":10422081,"node_id":"MDQ6VXNlcjEwNDIyMDgx","avatar_url":"https://avatars3.githubusercontent.com/u/10422081?v=4","gravatar_id":"","url":"https://api.github.com/users/piyush286","html_url":"https://github.com/piyush286","followers_url":"https://api.github.com/users/piyush286/followers","following_url":"https://api.github.com/users/piyush286/following{/other_user}","gists_url":"https://api.github.com/users/piyush286/gists{/gist_id}","starred_url":"https://api.github.com/users/piyush286/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/piyush286/subscriptions","organizations_url":"https://api.github.com/users/piyush286/orgs","repos_url":"https://api.github.com/users/piyush286/repos","events_url":"https://api.github.com/users/piyush286/events{/privacy}","received_events_url":"https://api.github.com/users/piyush286/received_events","type":"User","site_admin":false},"body":"• Enabled ODM benchmark since benchmark package has been staged (https://github.com/AdoptOpenJDK/openjdk-tests/issues/457) and numactl has been installed (https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/394) on the perf machines\r\n• Fixed various environment variables\r\n• Updated the playlist to copy the benchmark config file in the correct directory before running it\r\n• Changed tar to zip to package the benchmark output files\r\n• Cleaned up some redundant code\r\n• Other fixes\r\n\r\nSigned-off-by: Piyush <piyush@ca.ibm.com>","created_at":"2018-12-07T22:25:37Z","updated_at":"2018-12-10T13:16:10Z","closed_at":null,"merged_at":null,"merge_commit_sha":"f7ea4c2a5f28dc1cfac0cdf9a084e83ba49f5e9e","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/737/commits","review_comments_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/737/comments","review_comment_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/comments{/number}","comments_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/issues/737/comments","statuses_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/statuses/b70ec0f368c3b968619f92d69ed29ca47efe7b2e","head":{"label":"piyush286:master","ref":"master","sha":"b70ec0f368c3b968619f92d69ed29ca47efe7b2e","user":{"login":"piyush286","id":10422081,"node_id":"MDQ6VXNlcjEwNDIyMDgx","avatar_url":"https://avatars3.githubusercontent.com/u/10422081?v=4","gravatar_id":"","url":"https://api.github.com/users/piyush286","html_url":"https://github.com/piyush286","followers_url":"https://api.github.com/users/piyush286/followers","following_url":"https://api.github.com/users/piyush286/following{/other_user}","gists_url":"https://api.github.com/users/piyush286/gists{/gist_id}","starred_url":"https://api.github.com/users/piyush286/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/piyush286/subscriptions","organizations_url":"https://api.github.com/users/piyush286/orgs","repos_url":"https://api.github.com/users/piyush286/repos","events_url":"https://api.github.com/users/piyush286/events{/privacy}","received_events_url":"https://api.github.com/users/piyush286/received_events","type":"User","site_admin":false},"repo":{"id":158589443,"node_id":"MDEwOlJlcG9zaXRvcnkxNTg1ODk0NDM=","name":"openjdk-tests","full_name":"piyush286/openjdk-tests","private":false,"owner":{"login":"piyush286","id":10422081,"node_id":"MDQ6VXNlcjEwNDIyMDgx","avatar_url":"https://avatars3.githubusercontent.com/u/10422081?v=4","gravatar_id":"","url":"https://api.github.com/users/piyush286","html_url":"https://github.com/piyush286","followers_url":"https://api.github.com/users/piyush286/followers","following_url":"https://api.github.com/users/piyush286/following{/other_user}","gists_url":"https://api.github.com/users/piyush286/gists{/gist_id}","starred_url":"https://api.github.com/users/piyush286/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/piyush286/subscriptions","organizations_url":"https://api.github.com/users/piyush286/orgs","repos_url":"https://api.github.com/users/piyush286/repos","events_url":"https://api.github.com/users/piyush286/events{/privacy}","received_events_url":"https://api.github.com/users/piyush286/received_events","type":"User","site_admin":false},"html_url":"https://github.com/piyush286/openjdk-tests","description":"Home of test infrastructure for AdoptOpenJDK builds","fork":true,"url":"https://api.github.com/repos/piyush286/openjdk-tests","forks_url":"https://api.github.com/repos/piyush286/openjdk-tests/forks","keys_url":"https://api.github.com/repos/piyush286/openjdk-tests/keys{/key_id}","collaborators_url":"https://api.github.com/repos/piyush286/openjdk-tests/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/piyush286/openjdk-tests/teams","hooks_url":"https://api.github.com/repos/piyush286/openjdk-tests/hooks","issue_events_url":"https://api.github.com/repos/piyush286/openjdk-tests/issues/events{/number}","events_url":"https://api.github.com/repos/piyush286/openjdk-tests/events","assignees_url":"https://api.github.com/repos/piyush286/openjdk-tests/assignees{/user}","branches_url":"https://api.github.com/repos/piyush286/openjdk-tests/branches{/branch}","tags_url":"https://api.github.com/repos/piyush286/openjdk-tests/tags","blobs_url":"https://api.github.com/repos/piyush286/openjdk-tests/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/piyush286/openjdk-tests/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/piyush286/openjdk-tests/git/refs{/sha}","trees_url":"https://api.github.com/repos/piyush286/openjdk-tests/git/trees{/sha}","statuses_url":"https://api.github.com/repos/piyush286/openjdk-tests/statuses/{sha}","languages_url":"https://api.github.com/repos/piyush286/openjdk-tests/languages","stargazers_url":"https://api.github.com/repos/piyush286/openjdk-tests/stargazers","contributors_url":"https://api.github.com/repos/piyush286/openjdk-tests/contributors","subscribers_url":"https://api.github.com/repos/piyush286/openjdk-tests/subscribers","subscription_url":"https://api.github.com/repos/piyush286/openjdk-tests/subscription","commits_url":"https://api.github.com/repos/piyush286/openjdk-tests/commits{/sha}","git_commits_url":"https://api.github.com/repos/piyush286/openjdk-tests/git/commits{/sha}","comments_url":"https://api.github.com/repos/piyush286/openjdk-tests/comments{/number}","issue_comment_url":"https://api.github.com/repos/piyush286/openjdk-tests/issues/comments{/number}","contents_url":"https://api.github.com/repos/piyush286/openjdk-tests/contents/{+path}","compare_url":"https://api.github.com/repos/piyush286/openjdk-tests/compare/{base}...{head}","merges_url":"https://api.github.com/repos/piyush286/openjdk-tests/merges","archive_url":"https://api.github.com/repos/piyush286/openjdk-tests/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/piyush286/openjdk-tests/downloads","issues_url":"https://api.github.com/repos/piyush286/openjdk-tests/issues{/number}","pulls_url":"https://api.github.com/repos/piyush286/openjdk-tests/pulls{/number}","milestones_url":"https://api.github.com/repos/piyush286/openjdk-tests/milestones{/number}","notifications_url":"https://api.github.com/repos/piyush286/openjdk-tests/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/piyush286/openjdk-tests/labels{/name}","releases_url":"https://api.github.com/repos/piyush286/openjdk-tests/releases{/id}","deployments_url":"https://api.github.com/repos/piyush286/openjdk-tests/deployments","created_at":"2018-11-21T18:18:57Z","updated_at":"2018-12-07T22:24:01Z","pushed_at":"2018-12-07T22:24:00Z","git_url":"git://github.com/piyush286/openjdk-tests.git","ssh_url":"git@github.com:piyush286/openjdk-tests.git","clone_url":"https://github.com/piyush286/openjdk-tests.git","svn_url":"https://github.com/piyush286/openjdk-tests","homepage":"","size":26154,"stargazers_count":0,"watchers_count":0,"language":"Shell","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"apache-2.0","name":"Apache License 2.0","spdx_id":"Apache-2.0","url":"https://api.github.com/licenses/apache-2.0","node_id":"MDc6TGljZW5zZTI="},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"AdoptOpenJDK:master","ref":"master","sha":"f3856860ea0a30c936240d4959284443bb636181","user":{"login":"AdoptOpenJDK","id":1673867,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE2NzM4Njc=","avatar_url":"https://avatars0.githubusercontent.com/u/1673867?v=4","gravatar_id":"","url":"https://api.github.com/users/AdoptOpenJDK","html_url":"https://github.com/AdoptOpenJDK","followers_url":"https://api.github.com/users/AdoptOpenJDK/followers","following_url":"https://api.github.com/users/AdoptOpenJDK/following{/other_user}","gists_url":"https://api.github.com/users/AdoptOpenJDK/gists{/gist_id}","starred_url":"https://api.github.com/users/AdoptOpenJDK/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AdoptOpenJDK/subscriptions","organizations_url":"https://api.github.com/users/AdoptOpenJDK/orgs","repos_url":"https://api.github.com/users/AdoptOpenJDK/repos","events_url":"https://api.github.com/users/AdoptOpenJDK/events{/privacy}","received_events_url":"https://api.github.com/users/AdoptOpenJDK/received_events","type":"Organization","site_admin":false},"repo":{"id":88991896,"node_id":"MDEwOlJlcG9zaXRvcnk4ODk5MTg5Ng==","name":"openjdk-tests","full_name":"AdoptOpenJDK/openjdk-tests","private":false,"owner":{"login":"AdoptOpenJDK","id":1673867,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE2NzM4Njc=","avatar_url":"https://avatars0.githubusercontent.com/u/1673867?v=4","gravatar_id":"","url":"https://api.github.com/users/AdoptOpenJDK","html_url":"https://github.com/AdoptOpenJDK","followers_url":"https://api.github.com/users/AdoptOpenJDK/followers","following_url":"https://api.github.com/users/AdoptOpenJDK/following{/other_user}","gists_url":"https://api.github.com/users/AdoptOpenJDK/gists{/gist_id}","starred_url":"https://api.github.com/users/AdoptOpenJDK/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AdoptOpenJDK/subscriptions","organizations_url":"https://api.github.com/users/AdoptOpenJDK/orgs","repos_url":"https://api.github.com/users/AdoptOpenJDK/repos","events_url":"https://api.github.com/users/AdoptOpenJDK/events{/privacy}","received_events_url":"https://api.github.com/users/AdoptOpenJDK/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/AdoptOpenJDK/openjdk-tests","description":"Home of test infrastructure for AdoptOpenJDK builds","fork":false,"url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests","forks_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/forks","keys_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/keys{/key_id}","collaborators_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/teams","hooks_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/hooks","issue_events_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/issues/events{/number}","events_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/events","assignees_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/assignees{/user}","branches_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/branches{/branch}","tags_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/tags","blobs_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/git/refs{/sha}","trees_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/git/trees{/sha}","statuses_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/statuses/{sha}","languages_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/languages","stargazers_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/stargazers","contributors_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/contributors","subscribers_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/subscribers","subscription_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/subscription","commits_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/commits{/sha}","git_commits_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/git/commits{/sha}","comments_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/comments{/number}","issue_comment_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/issues/comments{/number}","contents_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/contents/{+path}","compare_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/compare/{base}...{head}","merges_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/merges","archive_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/downloads","issues_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/issues{/number}","pulls_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls{/number}","milestones_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/milestones{/number}","notifications_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/labels{/name}","releases_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/releases{/id}","deployments_url":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/deployments","created_at":"2017-04-21T14:24:21Z","updated_at":"2018-12-08T18:32:29Z","pushed_at":"2018-12-10T12:55:24Z","git_url":"git://github.com/AdoptOpenJDK/openjdk-tests.git","ssh_url":"git@github.com:AdoptOpenJDK/openjdk-tests.git","clone_url":"https://github.com/AdoptOpenJDK/openjdk-tests.git","svn_url":"https://github.com/AdoptOpenJDK/openjdk-tests","homepage":"","size":26176,"stargazers_count":17,"watchers_count":17,"language":"Shell","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":47,"mirror_url":null,"archived":false,"open_issues_count":86,"license":{"key":"apache-2.0","name":"Apache License 2.0","spdx_id":"Apache-2.0","url":"https://api.github.com/licenses/apache-2.0","node_id":"MDc6TGljZW5zZTI="},"forks":47,"open_issues":86,"watchers":17,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/737"},"html":{"href":"https://github.com/AdoptOpenJDK/openjdk-tests/pull/737"},"issue":{"href":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/issues/737"},"comments":{"href":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/issues/737/comments"},"review_comments":{"href":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/737/comments"},"review_comment":{"href":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/pulls/737/commits"},"statuses":{"href":"https://api.github.com/repos/AdoptOpenJDK/openjdk-tests/statuses/b70ec0f368c3b968619f92d69ed29ca47efe7b2e"}},"author_association":"NONE"}}
{"actor":{"display_login":"llxia"}}