Bug 1839533
Summary: | engine-setup fails if admin password in answerfile contains a "%" | ||
---|---|---|---|
Product: | [oVirt] otopi | Reporter: | Stephen Panicho <s.panicho> |
Component: | General | Assignee: | Yedidyah Bar David <didi> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Nikolai Sednev <nsednev> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | master | CC: | bugs, didi, emarcus, nsoffer |
Target Milestone: | ovirt-4.4.1 | Keywords: | Regression, Triaged |
Target Release: | 1.9.2 | Flags: | sbonazzo:
ovirt-4.4?
sbonazzo: planning_ack? sbonazzo: devel_ack+ sbonazzo: testing_ack? |
Hardware: | x86_64 | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | otopi-1.9.2 | Doc Type: | Bug Fix |
Doc Text: |
Cause:
otopi, and therefore engine-setup, uses python's configparser module for reading answer/configuration files.
In 4.4, otopi uses python3, which changed the behavior of configparser to do "Interpolation", treating '%' as a special character.
Consequence:
engine-setup with an answer file containing '%' failed, emitting an error.
Fix:
otopi was changed to use the class RawConfigParser, which does not do interpolation, instead of ConfigParser.
Result:
engine-setup succeeds.
|
Story Points: | --- |
Clone Of: | Environment: | ||
Last Closed: | 2020-07-08 08:26:50 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | Integration | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 1700864 |
Description
Stephen Panicho
2020-05-24 14:42:57 UTC
Did, isn't this modified now? Works for me on latest Software Version:4.4.1.7-0.3.el8ev. otopi-common-1.9.2-1.el8ev.noarch ovirt-hosted-engine-ha-2.4.4-1.el8ev.noarch ovirt-hosted-engine-setup-2.4.5-1.el8ev.noarch Linux 4.18.0-193.12.1.el8_2.x86_64 #1 SMP Thu Jul 2 15:48:14 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux Red Hat Enterprise Linux release 8.2 (Ootpa) Reported issue no longer exists. This bugzilla is included in oVirt 4.4.1 release, published on July 8th 2020. Since the problem described in this bug report should be resolved in oVirt 4.4.1 release, it has been closed with a resolution of CURRENT RELEASE. If the solution does not work for you, please open a new bug report. Hi Didi, should't we list this as an enhancement or rebase issue? since it is the first release of RHV 4.4, it should not be a bug fix I would phrase it something like: "The otopi component has been updated to work with Python 3." or "The otopi component used in Self-Hosted Engine deployment, has been updated to work with Python 3." (In reply to Eli Marcus from comment #4) > Hi Didi, > should't we list this as an enhancement or rebase issue? > since it is the first release of RHV 4.4, it should not be a bug fix Indeed, but this is an oVirt bug, and 4.4.0 was released without a fix for it. > > I would phrase it something like: > > "The otopi component has been updated to work with Python 3." > > or > > "The otopi component used in Self-Hosted Engine deployment, has been updated > to work with Python 3." For RHV, I agree, or just drop it from release notes. Or, include a general comment that most (I think all, not sure - definitely all of it for the main components) of the code was ported to python3, no need to reference particular bugs or components, IMO. |