-
Notifications
You must be signed in to change notification settings - Fork 9
Update the OS version definition for each server #42
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
Conversation
|
Ah, failing due to quattor/configuration-modules-core#1787 |
- Remove default version - Make sure each test server has an entry defining its OS version
2b8a0d7 to
f5686bd
Compare
|
The remaining error is caused by quattor/configuration-modules-core#1787. I'm reluctant to workaround the problem in the affected profiles. The problem comes from that the fact |
|
@jrha the test is still failing because merging quattor/configuration-modules-core#1787 is not enough, I forgot! The test uses the configuration module templates in |
|
That does remind me that I'd like to discuss options for not having |
|
I suppose manually fixing the schema for now is fine. |
I agree it is not ideal but I don't have any idea on how avoiding it. Except moving the manually maintained content in another repo and importing it the same way as we do for configuration module templates. But at the price that it is much more difficult to test modifications in example profiles, as we do in |
|
Indeed, I don't see any obviously good options. |
The changes in this PR fix the problem identified in quattor/template-library-core#237 after the structure change of grid/OS repositories (
mainbranch instead of one branch per major version). The problem was caused by the fact that the (obsolete) default OS version is not part of themainbranch.