{"action":"created","issue":{"url":"https://api.github.com/repos/ceph/teuthology/issues/891","repository_url":"https://api.github.com/repos/ceph/teuthology","labels_url":"https://api.github.com/repos/ceph/teuthology/issues/891/labels{/name}","comments_url":"https://api.github.com/repos/ceph/teuthology/issues/891/comments","events_url":"https://api.github.com/repos/ceph/teuthology/issues/891/events","html_url":"https://github.com/ceph/teuthology/pull/891","id":162864835,"number":891,"title":"install.upgrade: add pre-upgrade sanity check","user":{"login":"smithfarm","id":3974844,"avatar_url":"https://avatars.githubusercontent.com/u/3974844?v=3","gravatar_id":"","url":"https://api.github.com/users/smithfarm","html_url":"https://github.com/smithfarm","followers_url":"https://api.github.com/users/smithfarm/followers","following_url":"https://api.github.com/users/smithfarm/following{/other_user}","gists_url":"https://api.github.com/users/smithfarm/gists{/gist_id}","starred_url":"https://api.github.com/users/smithfarm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smithfarm/subscriptions","organizations_url":"https://api.github.com/users/smithfarm/orgs","repos_url":"https://api.github.com/users/smithfarm/repos","events_url":"https://api.github.com/users/smithfarm/events{/privacy}","received_events_url":"https://api.github.com/users/smithfarm/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"zmc","id":58307,"avatar_url":"https://avatars.githubusercontent.com/u/58307?v=3","gravatar_id":"","url":"https://api.github.com/users/zmc","html_url":"https://github.com/zmc","followers_url":"https://api.github.com/users/zmc/followers","following_url":"https://api.github.com/users/zmc/following{/other_user}","gists_url":"https://api.github.com/users/zmc/gists{/gist_id}","starred_url":"https://api.github.com/users/zmc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zmc/subscriptions","organizations_url":"https://api.github.com/users/zmc/orgs","repos_url":"https://api.github.com/users/zmc/repos","events_url":"https://api.github.com/users/zmc/events{/privacy}","received_events_url":"https://api.github.com/users/zmc/received_events","type":"User","site_admin":false},"assignees":[{"login":"zmc","id":58307,"avatar_url":"https://avatars.githubusercontent.com/u/58307?v=3","gravatar_id":"","url":"https://api.github.com/users/zmc","html_url":"https://github.com/zmc","followers_url":"https://api.github.com/users/zmc/followers","following_url":"https://api.github.com/users/zmc/following{/other_user}","gists_url":"https://api.github.com/users/zmc/gists{/gist_id}","starred_url":"https://api.github.com/users/zmc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zmc/subscriptions","organizations_url":"https://api.github.com/users/zmc/orgs","repos_url":"https://api.github.com/users/zmc/repos","events_url":"https://api.github.com/users/zmc/events{/privacy}","received_events_url":"https://api.github.com/users/zmc/received_events","type":"User","site_admin":false}],"milestone":null,"comments":30,"created_at":"2016-06-29T08:08:15Z","updated_at":"2016-06-29T23:24:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ceph/teuthology/pulls/891","html_url":"https://github.com/ceph/teuthology/pull/891","diff_url":"https://github.com/ceph/teuthology/pull/891.diff","patch_url":"https://github.com/ceph/teuthology/pull/891.patch"},"body":"Add a pre-upgrade sanity check to the install.upgrade task. This sanity check:\r\n\r\n1. determines the version of Ceph installed on the remote (\"installed_version\")\r\n2. determines the version of Ceph in the target gitbuilder repo\r\n(\"upgrade_version\")\r\n3. if \"installed_version\" is greater than \"upgrade_version\", fail immediately\r\n\r\nSince the upgrade itself cannot succeed if upgrade_version < installed_version,\r\nit's better to fail before attempting the upgrade.\r\n\r\nBefore this patch, we were failing in the verify_package_version() sanity check\r\nwhich is run after attempting (and failing) to upgrade the packages.\r\n\r\nFixes: http://tracker.ceph.com/issues/16521\r\nSigned-off-by: Nathan Cutler <ncutler@suse.com>"},"comment":{"url":"https://api.github.com/repos/ceph/teuthology/issues/comments/229518829","html_url":"https://github.com/ceph/teuthology/pull/891#issuecomment-229518829","issue_url":"https://api.github.com/repos/ceph/teuthology/issues/891","id":229518829,"user":{"login":"smithfarm","id":3974844,"avatar_url":"https://avatars.githubusercontent.com/u/3974844?v=3","gravatar_id":"","url":"https://api.github.com/users/smithfarm","html_url":"https://github.com/smithfarm","followers_url":"https://api.github.com/users/smithfarm/followers","following_url":"https://api.github.com/users/smithfarm/following{/other_user}","gists_url":"https://api.github.com/users/smithfarm/gists{/gist_id}","starred_url":"https://api.github.com/users/smithfarm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smithfarm/subscriptions","organizations_url":"https://api.github.com/users/smithfarm/orgs","repos_url":"https://api.github.com/users/smithfarm/repos","events_url":"https://api.github.com/users/smithfarm/events{/privacy}","received_events_url":"https://api.github.com/users/smithfarm/received_events","type":"User","site_admin":false},"created_at":"2016-06-29T23:24:41Z","updated_at":"2016-06-29T23:24:41Z","body":"Let me know if you want me to explore this path further or if I can drop these last two commits."}}
{"actor":{"display_login":"smithfarm"}}