{"action":"created","issue":{"url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/549","labels_url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/549/labels{/name}","comments_url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/549/comments","events_url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/549/events","html_url":"https://github.com/riptano/cassandra-dtest/pull/549","id":106397462,"number":549,"title":"improve and consolidate upgrade path decision logic","user":{"login":"mambocab","id":3753769,"avatar_url":"https://avatars.githubusercontent.com/u/3753769?v=3","gravatar_id":"","url":"https://api.github.com/users/mambocab","html_url":"https://github.com/mambocab","followers_url":"https://api.github.com/users/mambocab/followers","following_url":"https://api.github.com/users/mambocab/following{/other_user}","gists_url":"https://api.github.com/users/mambocab/gists{/gist_id}","starred_url":"https://api.github.com/users/mambocab/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mambocab/subscriptions","organizations_url":"https://api.github.com/users/mambocab/orgs","repos_url":"https://api.github.com/users/mambocab/repos","events_url":"https://api.github.com/users/mambocab/events{/privacy}","received_events_url":"https://api.github.com/users/mambocab/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-09-14T18:08:56Z","updated_at":"2015-09-15T14:07:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/riptano/cassandra-dtest/pulls/549","html_url":"https://github.com/riptano/cassandra-dtest/pull/549","diff_url":"https://github.com/riptano/cassandra-dtest/pull/549.diff","patch_url":"https://github.com/riptano/cassandra-dtest/pull/549.patch"},"body":"This rewrites the logic for determining the upgrade path in the storage engine upgrade tests. Rather than working off the branch name, it works off the version of the It works locally for me, both for running as-is, and for specifying paths with `UPGRADE_TO`. Demonstration of the latter is here:\r\n\r\nhttp://cassci.datastax.com/job/scratch_mambocab-storage_engine_upgrades-2.1/17\r\n\r\nThis should deal with problems like [the problem tjake had](https://issues.apache.org/jira/browse/CASSANDRA-10261?focusedCommentId=14740734&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14740734) where the test ran the upgrade from cassandra-3.0 to the pre-3.0 dev branch, which isn't a supported path."},"comment":{"url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/comments/140404672","html_url":"https://github.com/riptano/cassandra-dtest/pull/549#issuecomment-140404672","issue_url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/549","id":140404672,"user":{"login":"mambocab","id":3753769,"avatar_url":"https://avatars.githubusercontent.com/u/3753769?v=3","gravatar_id":"","url":"https://api.github.com/users/mambocab","html_url":"https://github.com/mambocab","followers_url":"https://api.github.com/users/mambocab/followers","following_url":"https://api.github.com/users/mambocab/following{/other_user}","gists_url":"https://api.github.com/users/mambocab/gists{/gist_id}","starred_url":"https://api.github.com/users/mambocab/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mambocab/subscriptions","organizations_url":"https://api.github.com/users/mambocab/orgs","repos_url":"https://api.github.com/users/mambocab/repos","events_url":"https://api.github.com/users/mambocab/events{/privacy}","received_events_url":"https://api.github.com/users/mambocab/received_events","type":"User","site_admin":false},"created_at":"2015-09-15T14:07:42Z","updated_at":"2015-09-15T14:07:42Z","body":"@ptnapoleon I'm getting errors like this:\r\n\r\nhttp://cassci.datastax.com/view/All_Jobs/job/scratch_mambocab-dtest_windows/1/testReport/upgrade_tests.cql_tests/TestCQL/blobAs_functions_test/\r\n\r\non the Windows job. The other two jobs are reasonable:\r\n\r\nhttp://cassci.datastax.com/view/All_Jobs/job/scratch_mambocab-storage_engine_upgrades-3.0/lastCompletedBuild/testReport/\r\n\r\nhttp://cassci.datastax.com/view/All_Jobs/job/scratch_mambocab-storage_engine_upgrades-trunk/lastCompletedBuild/\r\n\r\nWould it be all right to add a `@skipIf(sys.platform == 'win32', 'Skip upgrade tests on Windows')` for now and merge?"}}