{"action":"created","issue":{"url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/2730","labels_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/2730/labels{/name}","comments_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/2730/comments","events_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/2730/events","html_url":"https://github.com/SatelliteQE/robottelo/pull/2730","id":104764989,"number":2730,"title":"Initial incremental update tests","user":{"login":"sthirugn","id":3309189,"avatar_url":"https://avatars.githubusercontent.com/u/3309189?v=3","gravatar_id":"","url":"https://api.github.com/users/sthirugn","html_url":"https://github.com/sthirugn","followers_url":"https://api.github.com/users/sthirugn/followers","following_url":"https://api.github.com/users/sthirugn/following{/other_user}","gists_url":"https://api.github.com/users/sthirugn/gists{/gist_id}","starred_url":"https://api.github.com/users/sthirugn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sthirugn/subscriptions","organizations_url":"https://api.github.com/users/sthirugn/orgs","repos_url":"https://api.github.com/users/sthirugn/repos","events_url":"https://api.github.com/users/sthirugn/events{/privacy}","received_events_url":"https://api.github.com/users/sthirugn/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/SatelliteQE/robottelo/labels/API","name":"API","color":"fbca04"},{"url":"https://api.github.com/repos/SatelliteQE/robottelo/labels/CLI","name":"CLI","color":"009800"}],"state":"open","locked":false,"assignee":{"login":"sthirugn","id":3309189,"avatar_url":"https://avatars.githubusercontent.com/u/3309189?v=3","gravatar_id":"","url":"https://api.github.com/users/sthirugn","html_url":"https://github.com/sthirugn","followers_url":"https://api.github.com/users/sthirugn/followers","following_url":"https://api.github.com/users/sthirugn/following{/other_user}","gists_url":"https://api.github.com/users/sthirugn/gists{/gist_id}","starred_url":"https://api.github.com/users/sthirugn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sthirugn/subscriptions","organizations_url":"https://api.github.com/users/sthirugn/orgs","repos_url":"https://api.github.com/users/sthirugn/repos","events_url":"https://api.github.com/users/sthirugn/events{/privacy}","received_events_url":"https://api.github.com/users/sthirugn/received_events","type":"User","site_admin":false},"milestone":null,"comments":13,"created_at":"2015-09-03T19:29:03Z","updated_at":"2015-09-09T18:27:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/SatelliteQE/robottelo/pulls/2730","html_url":"https://github.com/SatelliteQE/robottelo/pull/2730","diff_url":"https://github.com/SatelliteQE/robottelo/pull/2730.diff","patch_url":"https://github.com/SatelliteQE/robottelo/pull/2730.patch"},"body":"This PR does the following:\r\n1. Introduces the longrun package to accomodate all long running tests. We are still discussing a strategy to run these long run tests which will be finalized in next few days.\r\n2. Added tests for incremental update feature - This PR has one api test and one cli test.  But the setUpClass() is designed in such a way to accomodate more future tests.  So it is possible that some of the things setup in setUpClass() may not actually get tested.  But they will be tested in the future tests for sure\r\n3. The setUpClass() has 15 steps which are identified with comments like Step 1.....Step 16\r\n4. The tearDownClass() takes care of killing the vms used and also reset the nailgun timeout to its default values\r\n5. setUpClass() is deliberately planned to be very thorough to mimic customer SOE with: 2 Content views (one with errata filters), 2 lifecycle environments, content hosts subscribed to different content views, etc.\r\n6. Lastly I have introduced the concept of tagging when creating vms.  So while vm clean up in ginger, we know which tests are making those vm hanging there and clean up those tests.\r\n\r\nPylint and flake8 looks clean for test_inc_updates.py:\r\n```sh\r\n# pylint tests/foreman/longrun/test_inc_updates.py\r\nNo config file found, using default configuration\r\n************* Module tests.foreman.longrun.test_inc_updates\r\nR: 47, 4: Too many local variables (27/15) (too-many-locals)\r\nR: 47, 4: Too many branches (17/12) (too-many-branches)\r\nR: 47, 4: Too many statements (62/50) (too-many-statements)\r\n\r\n...\r\n...\r\nGlobal evaluation\r\n-----------------\r\nYour code has been rated at 9.74/10 (previous run: 9.74/10, +0.00)\r\n\r\n\r\n# flake8 tests/foreman/longrun/test_inc_updates.py\r\n- No output -\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/comments/138998669","html_url":"https://github.com/SatelliteQE/robottelo/pull/2730#issuecomment-138998669","issue_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/2730","id":138998669,"user":{"login":"sthirugn","id":3309189,"avatar_url":"https://avatars.githubusercontent.com/u/3309189?v=3","gravatar_id":"","url":"https://api.github.com/users/sthirugn","html_url":"https://github.com/sthirugn","followers_url":"https://api.github.com/users/sthirugn/followers","following_url":"https://api.github.com/users/sthirugn/following{/other_user}","gists_url":"https://api.github.com/users/sthirugn/gists{/gist_id}","starred_url":"https://api.github.com/users/sthirugn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sthirugn/subscriptions","organizations_url":"https://api.github.com/users/sthirugn/orgs","repos_url":"https://api.github.com/users/sthirugn/repos","events_url":"https://api.github.com/users/sthirugn/events{/privacy}","received_events_url":"https://api.github.com/users/sthirugn/received_events","type":"User","site_admin":false},"created_at":"2015-09-09T18:27:45Z","updated_at":"2015-09-09T18:27:45Z","body":"bump - this needs another ack/merge."}}