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

Default global.rc file gives validation error #1865

Closed
trwhitcomb opened this issue May 25, 2016 · 2 comments
Closed

Default global.rc file gives validation error #1865

trwhitcomb opened this issue May 25, 2016 · 2 comments
Assignees
Labels
bug Something is wrong :(
Milestone

Comments

@trwhitcomb
Copy link
Collaborator

I got a new workstation, so I'm setting it up with the latest cylc version (6.10.1-37-g2c9bb), and I'm now getting validation errors with the global.rc file. I cleared out the old and ran:

cylc get-site-config > $HOME/.cylc/global.rc

to generate the new configuration file. Then, I tried validating a suite and got:

[whitcomb@whitcomb-dev ~]$ cylc validate -v /ftp/receive/whitcomb/suite.rc
Loading site/user config files
Reading file /users/whitcomb/.cylc/global.rc
WARNING: ignoring bad site config /users/whitcomb/.cylc/global.rc:
Illegal ISO 8601 interval value: [task events][execution timeout]

it's also not suite-specific:

[whitcomb@whitcomb-dev ~]$ cylc version
WARNING: ignoring bad site config /users/whitcomb/.cylc/global.rc:
Illegal ISO 8601 interval value: [task events][execution timeout]
6.10.1-37-g2c9bb

If I look at the file, it shows that execution timeout entry as blank. I haven't changed anything, but it appears that the validation code is looking for something that's not present in the internal configuration.

Expected result is to not get a bad site config by using the software's method to dump the current configuration from scratch - the actual result is a validation error with a pristine global.rc file.

@trwhitcomb trwhitcomb changed the title Default global.rc file gives error when validating suites Default global.rc file gives validation error May 25, 2016
@hjoliver hjoliver added the bug Something is wrong :( label May 25, 2016
@hjoliver hjoliver self-assigned this May 25, 2016
@hjoliver
Copy link
Member

Reproduced. Thanks for the report.

@hjoliver
Copy link
Member

This goes back to the 6.6.0 release. Most users just set the site config items they need, rather than start from a file full of default values. However, that certainly should be possible; we'd better add a test for it.

@matthewrmshin matthewrmshin added this to the soon milestone May 26, 2016
@matthewrmshin matthewrmshin modified the milestones: next release, soon May 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is wrong :(
Projects
None yet
Development

No branches or pull requests

3 participants