Bug 1219216
Summary: | virt-v2v fails to match kernel line in grub.conf | |||
---|---|---|---|---|
Product: | Red Hat Enterprise Linux 6 | Reporter: | Dylan Gross <dgross> | |
Component: | augeas | Assignee: | Dominic Cleal <dcleal> | |
Status: | CLOSED WONTFIX | QA Contact: | Virtualization Bugs <virt-bugs> | |
Severity: | medium | Docs Contact: | ||
Priority: | unspecified | |||
Version: | 6.6 | CC: | dgross, rjones | |
Target Milestone: | rc | |||
Target Release: | --- | |||
Hardware: | Unspecified | |||
OS: | Unspecified | |||
Whiteboard: | ||||
Fixed In Version: | Doc Type: | Bug Fix | ||
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 1239053 (view as bug list) | Environment: | ||
Last Closed: | 2015-07-03 11:40:18 UTC | Type: | Bug | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: |
Description
Dylan Gross
2015-05-06 20:29:54 UTC
(In reply to Dylan Gross from comment #0) > We use aug_match which does not return the value. The problem is the two > lines > related to the serial console > > serial -unit=0 -speed=115200 > terminal -timeout=10 serial console > " Are these actually valid and working configuration directives? The documentation suggests it must be two hyphens, not one: https://www.gnu.org/software/grub/manual/legacy/Serial-terminal.html https://www.gnu.org/software/grub/manual/legacy/serial.html https://www.gnu.org/software/grub/manual/legacy/terminal.html I see nothing there to suggest that "-unit=0" is valid. Checked the GRUB 0.97 source code too, and it doesn't parse options with only one hyphen, which makes me believe this is a configuration error. Augeas generally only tries to parse valid configs, not be tolerant of errors. Customer confirmed that we he did remedy those incorrect arguments with double-hyphens, the virt-v2v no longer failed. If that is the intended behaviour of augeas when parsing, then this is probably "NOTABUG". Perhaps the error about not being able to match a kernel line could be more specific about failing on a different line. Or maybe not. I don't know enough about how the parsing is being done to know if it can get that granular. (In reply to Dylan Gross from comment #4) > Customer confirmed that we he did remedy those incorrect arguments with > double-hyphens, the virt-v2v no longer failed. If that is the intended > behaviour of augeas when parsing, then this is probably "NOTABUG". Indeed. > Perhaps the error about not being able to match a kernel line could be more > specific about failing on a different line. Or maybe not. I don't know > enough about how the parsing is being done to know if it can get that > granular. Yeah, it looks like virt-v2v isn't showing that Augeas hasn't managed to parse the file. It should be possible to report Augeas' own error output (from /augeas//error), like we did here for netcf: https://lists.fedorahosted.org/pipermail/netcf-devel/2014-September/000907.html or Puppet: https://github.com/puppetlabs/puppet/blob/4.1.0/lib/puppet/provider/augeas/augeas.rb#L327-L354 I don't know whether you'd prefer to move this bug or recreate one against v2v. |