{"action":"created","comment":{"url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/comments/165957694","pull_request_review_id":93972236,"id":165957694,"diff_hunk":"@@ -61,11 +62,18 @@ sub run {\n     # DLM process needs to be started\n     ensure_process_running 'dlm_controld';\n \n+    # ocfs2 package should be installed by default\n     if ($fs_type eq 'ocfs2') {\n         # Test if OCFS2 kernel module package is installed\n-        die 'ocfs2 kernel package is not installed' unless is_package_installed 'ocfs2-kmp-default';\n+        if (!is_package_installed 'ocfs2-kmp-default') {\n+            record_info 'ocfs2 kernel package is not installed';\n+            zypper_call 'in ocfs2-kmp-default';","path":"tests/ha/filesystem.pm","position":26,"original_position":26,"commit_id":"25331c438e51fae4c2e823691340ec6c980f0b00","original_commit_id":"25331c438e51fae4c2e823691340ec6c980f0b00","user":{"login":"ldevulder","id":25102165,"avatar_url":"https://avatars3.githubusercontent.com/u/25102165?v=4","gravatar_id":"","url":"https://api.github.com/users/ldevulder","html_url":"https://github.com/ldevulder","followers_url":"https://api.github.com/users/ldevulder/followers","following_url":"https://api.github.com/users/ldevulder/following{/other_user}","gists_url":"https://api.github.com/users/ldevulder/gists{/gist_id}","starred_url":"https://api.github.com/users/ldevulder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ldevulder/subscriptions","organizations_url":"https://api.github.com/users/ldevulder/orgs","repos_url":"https://api.github.com/users/ldevulder/repos","events_url":"https://api.github.com/users/ldevulder/events{/privacy}","received_events_url":"https://api.github.com/users/ldevulder/received_events","type":"User","site_admin":false},"body":"I searched in my already opened bugs, because I already had the issue, and I've changed the code to record a failure with the appropriate bug number instead of the `record_info`.\r\nShould be better now :)","created_at":"2018-02-05T12:25:54Z","updated_at":"2018-02-05T12:25:54Z","html_url":"https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/4339#discussion_r165957694","pull_request_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/4339","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/comments/165957694"},"html":{"href":"https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/4339#discussion_r165957694"},"pull_request":{"href":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/4339"}},"in_reply_to_id":165810002},"pull_request":{"url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/4339","id":166831702,"html_url":"https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/4339","diff_url":"https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/4339.diff","patch_url":"https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/4339.patch","issue_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/issues/4339","number":4339,"state":"open","locked":false,"title":"Fix xfsprogs missing sometimes on SLEHA15","user":{"login":"ldevulder","id":25102165,"avatar_url":"https://avatars3.githubusercontent.com/u/25102165?v=4","gravatar_id":"","url":"https://api.github.com/users/ldevulder","html_url":"https://github.com/ldevulder","followers_url":"https://api.github.com/users/ldevulder/followers","following_url":"https://api.github.com/users/ldevulder/following{/other_user}","gists_url":"https://api.github.com/users/ldevulder/gists{/gist_id}","starred_url":"https://api.github.com/users/ldevulder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ldevulder/subscriptions","organizations_url":"https://api.github.com/users/ldevulder/orgs","repos_url":"https://api.github.com/users/ldevulder/repos","events_url":"https://api.github.com/users/ldevulder/events{/privacy}","received_events_url":"https://api.github.com/users/ldevulder/received_events","type":"User","site_admin":false},"body":"- xfsprogs package is not installed by default, so we need to install it if needed\r\n- use `is_package_installed` function instead of `script_run` call to `rpm -q`\r\n- add year 2018 in copyright messages\r\n\r\n- Verification run: http://moon.qa.suse.cz/tests/1527\r\n","created_at":"2018-02-02T15:34:56Z","updated_at":"2018-02-05T12:25:54Z","closed_at":null,"merged_at":null,"merge_commit_sha":"c428d0ac9ea35fffb217ea544fb1b458468ce3bd","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"milestone":null,"commits_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/4339/commits","review_comments_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/4339/comments","review_comment_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/comments{/number}","comments_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/issues/4339/comments","statuses_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/statuses/25331c438e51fae4c2e823691340ec6c980f0b00","head":{"label":"ldevulder:fix-mkfs-xfs-sleha15","ref":"fix-mkfs-xfs-sleha15","sha":"25331c438e51fae4c2e823691340ec6c980f0b00","user":{"login":"ldevulder","id":25102165,"avatar_url":"https://avatars3.githubusercontent.com/u/25102165?v=4","gravatar_id":"","url":"https://api.github.com/users/ldevulder","html_url":"https://github.com/ldevulder","followers_url":"https://api.github.com/users/ldevulder/followers","following_url":"https://api.github.com/users/ldevulder/following{/other_user}","gists_url":"https://api.github.com/users/ldevulder/gists{/gist_id}","starred_url":"https://api.github.com/users/ldevulder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ldevulder/subscriptions","organizations_url":"https://api.github.com/users/ldevulder/orgs","repos_url":"https://api.github.com/users/ldevulder/repos","events_url":"https://api.github.com/users/ldevulder/events{/privacy}","received_events_url":"https://api.github.com/users/ldevulder/received_events","type":"User","site_admin":false},"repo":{"id":103379584,"name":"os-autoinst-distri-opensuse","full_name":"ldevulder/os-autoinst-distri-opensuse","owner":{"login":"ldevulder","id":25102165,"avatar_url":"https://avatars3.githubusercontent.com/u/25102165?v=4","gravatar_id":"","url":"https://api.github.com/users/ldevulder","html_url":"https://github.com/ldevulder","followers_url":"https://api.github.com/users/ldevulder/followers","following_url":"https://api.github.com/users/ldevulder/following{/other_user}","gists_url":"https://api.github.com/users/ldevulder/gists{/gist_id}","starred_url":"https://api.github.com/users/ldevulder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ldevulder/subscriptions","organizations_url":"https://api.github.com/users/ldevulder/orgs","repos_url":"https://api.github.com/users/ldevulder/repos","events_url":"https://api.github.com/users/ldevulder/events{/privacy}","received_events_url":"https://api.github.com/users/ldevulder/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/ldevulder/os-autoinst-distri-opensuse","description":"os-autoinst test cases for openSUSE","fork":true,"url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse","forks_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/forks","keys_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/teams","hooks_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/hooks","issue_events_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/issues/events{/number}","events_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/events","assignees_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/assignees{/user}","branches_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/branches{/branch}","tags_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/tags","blobs_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/git/refs{/sha}","trees_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/statuses/{sha}","languages_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/languages","stargazers_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/stargazers","contributors_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/contributors","subscribers_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/subscribers","subscription_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/subscription","commits_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/commits{/sha}","git_commits_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/git/commits{/sha}","comments_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/comments{/number}","issue_comment_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/issues/comments{/number}","contents_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/contents/{+path}","compare_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/merges","archive_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/downloads","issues_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/issues{/number}","pulls_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/pulls{/number}","milestones_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/milestones{/number}","notifications_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/labels{/name}","releases_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/releases{/id}","deployments_url":"https://api.github.com/repos/ldevulder/os-autoinst-distri-opensuse/deployments","created_at":"2017-09-13T09:20:12Z","updated_at":"2017-09-13T09:20:16Z","pushed_at":"2018-02-02T15:43:25Z","git_url":"git://github.com/ldevulder/os-autoinst-distri-opensuse.git","ssh_url":"git@github.com:ldevulder/os-autoinst-distri-opensuse.git","clone_url":"https://github.com/ldevulder/os-autoinst-distri-opensuse.git","svn_url":"https://github.com/ldevulder/os-autoinst-distri-opensuse","homepage":"https://progress.opensuse.org/projects/openqatests/issues","size":87426,"stargazers_count":0,"watchers_count":0,"language":"Perl","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":null,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"os-autoinst:master","ref":"master","sha":"c3d11f262b355f778e92c4c46b74b519ee9898b2","user":{"login":"os-autoinst","id":6918706,"avatar_url":"https://avatars2.githubusercontent.com/u/6918706?v=4","gravatar_id":"","url":"https://api.github.com/users/os-autoinst","html_url":"https://github.com/os-autoinst","followers_url":"https://api.github.com/users/os-autoinst/followers","following_url":"https://api.github.com/users/os-autoinst/following{/other_user}","gists_url":"https://api.github.com/users/os-autoinst/gists{/gist_id}","starred_url":"https://api.github.com/users/os-autoinst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/os-autoinst/subscriptions","organizations_url":"https://api.github.com/users/os-autoinst/orgs","repos_url":"https://api.github.com/users/os-autoinst/repos","events_url":"https://api.github.com/users/os-autoinst/events{/privacy}","received_events_url":"https://api.github.com/users/os-autoinst/received_events","type":"Organization","site_admin":false},"repo":{"id":17630288,"name":"os-autoinst-distri-opensuse","full_name":"os-autoinst/os-autoinst-distri-opensuse","owner":{"login":"os-autoinst","id":6918706,"avatar_url":"https://avatars2.githubusercontent.com/u/6918706?v=4","gravatar_id":"","url":"https://api.github.com/users/os-autoinst","html_url":"https://github.com/os-autoinst","followers_url":"https://api.github.com/users/os-autoinst/followers","following_url":"https://api.github.com/users/os-autoinst/following{/other_user}","gists_url":"https://api.github.com/users/os-autoinst/gists{/gist_id}","starred_url":"https://api.github.com/users/os-autoinst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/os-autoinst/subscriptions","organizations_url":"https://api.github.com/users/os-autoinst/orgs","repos_url":"https://api.github.com/users/os-autoinst/repos","events_url":"https://api.github.com/users/os-autoinst/events{/privacy}","received_events_url":"https://api.github.com/users/os-autoinst/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/os-autoinst/os-autoinst-distri-opensuse","description":"os-autoinst test cases for openSUSE","fork":false,"url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse","forks_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/forks","keys_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/keys{/key_id}","collaborators_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/teams","hooks_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/hooks","issue_events_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/issues/events{/number}","events_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/events","assignees_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/assignees{/user}","branches_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/branches{/branch}","tags_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/tags","blobs_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/git/refs{/sha}","trees_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/git/trees{/sha}","statuses_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/statuses/{sha}","languages_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/languages","stargazers_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/stargazers","contributors_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/contributors","subscribers_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/subscribers","subscription_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/subscription","commits_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/commits{/sha}","git_commits_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/git/commits{/sha}","comments_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/comments{/number}","issue_comment_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/issues/comments{/number}","contents_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/contents/{+path}","compare_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/compare/{base}...{head}","merges_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/merges","archive_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/downloads","issues_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/issues{/number}","pulls_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls{/number}","milestones_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/milestones{/number}","notifications_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/labels{/name}","releases_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/releases{/id}","deployments_url":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/deployments","created_at":"2014-03-11T12:54:42Z","updated_at":"2017-11-10T06:18:19Z","pushed_at":"2018-02-05T12:14:43Z","git_url":"git://github.com/os-autoinst/os-autoinst-distri-opensuse.git","ssh_url":"git@github.com:os-autoinst/os-autoinst-distri-opensuse.git","clone_url":"https://github.com/os-autoinst/os-autoinst-distri-opensuse.git","svn_url":"https://github.com/os-autoinst/os-autoinst-distri-opensuse","homepage":"https://progress.opensuse.org/projects/openqatests/issues","size":87880,"stargazers_count":28,"watchers_count":28,"language":"Perl","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":119,"mirror_url":null,"archived":false,"open_issues_count":43,"license":null,"forks":119,"open_issues":43,"watchers":28,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/4339"},"html":{"href":"https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/4339"},"issue":{"href":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/issues/4339"},"comments":{"href":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/issues/4339/comments"},"review_comments":{"href":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/4339/comments"},"review_comment":{"href":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/pulls/4339/commits"},"statuses":{"href":"https://api.github.com/repos/os-autoinst/os-autoinst-distri-opensuse/statuses/25331c438e51fae4c2e823691340ec6c980f0b00"}},"author_association":"CONTRIBUTOR"}}
{"actor":{"display_login":"ldevulder"}}