-
Notifications
You must be signed in to change notification settings - Fork 9
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
Update the OS version definition for each server #42
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 (
main
branch instead of one branch per major version). The problem was caused by the fact that the (obsolete) default OS version is not part of themain
branch.