Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#964: add backwards comp restart #1027

Merged

Conversation

benfitzpatrick
Copy link
Contributor

This adds a test for restarting from a pre-cylc-6 state file. This closes #964.

@hjoliver, please review.

@matthewrmshin matthewrmshin added this to the cylc-6 milestone Jul 21, 2014
hjoliver added a commit that referenced this pull request Jul 23, 2014
@hjoliver hjoliver merged commit 2ef7019 into cylc:119.iso8601-cycling Jul 23, 2014
@arjclark
Copy link
Contributor

@benfitzpatrick / @hjoliver - while it's nice in theory to be able to do this, I guess we wouldn't recommend this as a general practice?

@hjoliver
Copy link
Member

@arjclark - I guess not, but backward compatibility really requires it. There will undoubtedly be many suites that get restarted after cylc is upgraded at a site, under users who have not set $CYLC_VERSION.

@hjoliver
Copy link
Member

(see #1032)

@benfitzpatrick benfitzpatrick deleted the 964.back-comp-restart branch October 15, 2014 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants