{"action":"created","issue":{"url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/3991","repository_url":"https://api.github.com/repos/SatelliteQE/robottelo","labels_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/3991/labels{/name}","comments_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/3991/comments","events_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/3991/events","html_url":"https://github.com/SatelliteQE/robottelo/pull/3991","id":189173938,"number":3991,"title":"Fix naive proxy retrieval","user":{"login":"svtkachenko","id":2845814,"avatar_url":"https://avatars.githubusercontent.com/u/2845814?v=3","gravatar_id":"","url":"https://api.github.com/users/svtkachenko","html_url":"https://github.com/svtkachenko","followers_url":"https://api.github.com/users/svtkachenko/followers","following_url":"https://api.github.com/users/svtkachenko/following{/other_user}","gists_url":"https://api.github.com/users/svtkachenko/gists{/gist_id}","starred_url":"https://api.github.com/users/svtkachenko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/svtkachenko/subscriptions","organizations_url":"https://api.github.com/users/svtkachenko/orgs","repos_url":"https://api.github.com/users/svtkachenko/repos","events_url":"https://api.github.com/users/svtkachenko/events{/privacy}","received_events_url":"https://api.github.com/users/svtkachenko/received_events","type":"User","site_admin":false},"labels":[{"id":468870987,"url":"https://api.github.com/repos/SatelliteQE/robottelo/labels/6.3","name":"6.3","color":"e99695","default":false},{"id":206130061,"url":"https://api.github.com/repos/SatelliteQE/robottelo/labels/Downstream%20Failure","name":"Downstream Failure","color":"bfe5bf","default":false},{"id":404622871,"url":"https://api.github.com/repos/SatelliteQE/robottelo/labels/review","name":"review","color":"00ff00","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2016-11-14T17:28:43Z","updated_at":"2016-11-15T19:04:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/SatelliteQE/robottelo/pulls/3991","html_url":"https://github.com/SatelliteQE/robottelo/pull/3991","diff_url":"https://github.com/SatelliteQE/robottelo/pull/3991.diff","patch_url":"https://github.com/SatelliteQE/robottelo/pull/3991.patch"},"body":"Fixes #3942\r\nTest run results for 6.3\r\n```python\r\n% nosetests tests/foreman/cli/test_host.py:{HostCreateTestCase,HostDeleteTestCase,HostUpdateTestCase,HostParameterTestCase}           \r\n..............................\r\n----------------------------------------------------------------------\r\nRan 30 tests in 1684.901s\r\nOK\r\n```\r\n```python\r\n% nosetests tests/foreman/cli/test_hostgroup.py:HostGroupTestCase.test_positive_create_with_puppet_ca_proxy \r\n.                                                                                                      \r\n----------------------------------------------------------------------\r\nRan 1 test in 12.701s\r\nOK\r\n```\r\n```python\r\n% nosetests tests/foreman/api/test_host.py:HostTestCase.{test_positive_create_with_puppet_proxy,test_positive_create_with_puppet_ca_proxy,test_positive_update_puppet_proxy,test_positive_update_puppet_ca_proxy}\r\n....python\r\n----------------------------------------------------------------------\r\nRan 4 tests in 75.776s\r\nOK\r\n```\r\n```python\r\n% nosetests tests/foreman/api/test_hostgroup.py:HostGroupTestCase.{test_positive_create_with_puppet_ca_proxy,test_positive_create_with_realm,test_positive_create_with_puppet_proxy,test_positive_create_with_content_source,test_positive_update_puppet_ca_proxy,test_positive_update_realm,test_positive_update_puppet_proxy,test_positive_update_content_source}      \r\n.S..SSSS\r\n----------------------------------------------------------------------\r\nRan 8 tests in 20.023s\r\nOK (SKIP=5)\r\n```\r\n`tests/foreman/api/test_smartproxy.py:SmartProxyMissingAttrTestCase` skipped becaus of bug.\r\n`tests/foreman/api/test_organization.py:OrganizationUpdateTestCase.test_positive_add_smart_proxy` fails because of bug in 6.3. However, test passed with the change on 6.2.\r\n`tests/foreman/cli/test_hostgroup.py:HostGroupTestCase.test_positive_create_with_multiple_entities` fails because of bug in 6.3. However, test passed with the change on 6.2.\r\n`tests/foreman/api/test_location.py:LocationTestCase` tests are failing because of bug in 6.3. However, tests passed with the change on 6.2."},"comment":{"url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/comments/260734635","html_url":"https://github.com/SatelliteQE/robottelo/pull/3991#issuecomment-260734635","issue_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/3991","id":260734635,"user":{"login":"abalakh","id":11719030,"avatar_url":"https://avatars.githubusercontent.com/u/11719030?v=3","gravatar_id":"","url":"https://api.github.com/users/abalakh","html_url":"https://github.com/abalakh","followers_url":"https://api.github.com/users/abalakh/followers","following_url":"https://api.github.com/users/abalakh/following{/other_user}","gists_url":"https://api.github.com/users/abalakh/gists{/gist_id}","starred_url":"https://api.github.com/users/abalakh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abalakh/subscriptions","organizations_url":"https://api.github.com/users/abalakh/orgs","repos_url":"https://api.github.com/users/abalakh/repos","events_url":"https://api.github.com/users/abalakh/events{/privacy}","received_events_url":"https://api.github.com/users/abalakh/received_events","type":"User","site_admin":false},"created_at":"2016-11-15T19:04:09Z","updated_at":"2016-11-15T19:04:09Z","body":"@rplevka better not to rely on id=1 imo. We had numerous issues with lots of 'default' entities not present after roll back on jenkins (not sure which approach we currently use, foreman reset? or maybe vm snapshot?) and as a result entity with id 1 was not the default one, but some custom one created by test. \r\nI wanted to give you default domain as an example - as it wasn't created after roll back some time ago, but i've just ran one test on our qe machine and was surprised it failed. Failure looks like this:\r\n```\r\nhammer> medium create --operatingsystem-ids=\"1\" --name=\"bJEclJ\" --organization-ids=\"1\" --location-ids=\"2\" --path=\"http://VvRKBH\"\r\nCould not create the installation medium:\r\n  Resource medium not found by id ''\r\n```\r\nPretty weird message, yeah? The root cause is this:\r\n```\r\nhammer> os info --id 1\r\nResource operatingsystem not found by id '1'\r\n```\r\nFor some reason default OS is not present there anymore.\r\nBasically, that's why me & Alex are always using some complex logic with default entities like  'find the default entity by name, not id, if it was found - then use it, otherwise create a new entity instead' and encouraging others not to rely on id=1.\r\nDefault org/loc are less likely to be affected, but still, who knows, maybe next time after roll back their ids will be 3 and 4 correspondingly."}}
{"actor":{"display_login":"abalakh"}}