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

volgroup_required option #73

Open
seraf1 opened this issue Jun 26, 2015 · 5 comments
Open

volgroup_required option #73

seraf1 opened this issue Jun 26, 2015 · 5 comments

Comments

@seraf1
Copy link

seraf1 commented Jun 26, 2015

Since the 14.8.0 version of the file quattor/aii/ks/config ends with the line:

"/system/aii/osinstall/ks/volgroup_required" = true;

Ks files generated and always have the volgroup inserted rows , also for sl6 and 7 which causes an error in the installation.

If I change this line as follows:

"/system/aii/osinstall/ks/volgroup_required" ?= true;

Then the files to sl6 ks and 7 no longer have these insertions, which is okay.

Regards.

@jouvin
Copy link
Contributor

jouvin commented Jun 26, 2015

I tend to agree that this line should be a default, not an enforced value. And it clearly makes the value define in the variant for el6 and el6 useless (false). I am just wondering why nobody reported this problem before... Could another KS expert comment?

@jrha jrha added this to the 15.10 milestone Jul 17, 2015
@jrha jrha modified the milestones: 16.2, 15.12 Dec 7, 2015
@jrha jrha removed this from the 16.2 milestone Feb 12, 2016
@seraf1
Copy link
Author

seraf1 commented Mar 30, 2016

Why has this been removed?

@jrha
Copy link
Member

jrha commented Mar 30, 2016

From the 16.2 milestone? Because there was no PR to fix it. We made a decision some time ago to only set milestones for issues that are being worked on and leave the rest as a proper backlog.

@seraf1
Copy link
Author

seraf1 commented Mar 31, 2016

It is for all versions.

I have to make changes in the code each time I make an update of Quattor

@jrha
Copy link
Member

jrha commented Apr 5, 2016

Open a PR and it will be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants