Note: This bug is displayed in read-only format because
the product is no longer active in Red Hat Bugzilla.
RHEL Engineering is moving the tracking of its product development work on RHEL 6 through RHEL 9 to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "RHEL project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs in the statuses "NEW", "ASSIGNED", and "POST" are being migrated throughout September 2023. Bugs of Red Hat partners with an assigned Engineering Partner Manager (EPM) are migrated in late September as per pre-agreed dates. Bugs against components "kernel", "kernel-rt", and "kpatch" are only migrated if still in "NEW" or "ASSIGNED". If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "RHEL project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/RHEL-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Some information in ovirt.log was being missed due to using an incorrect function, logging.info("") this has been rectified using the correct function logger.info("") to record all log information in ovirt.log.
Description of problem:
Miss some log info in ovirt.log by using function "logging.info".
Should change "logging.info" into "logger.info".
Also better to change the position between " logging.info(("Setting value for %s to %s " %
(self.__dict__[i_short], OVIRT_VARS[i])))"
and "i_short = i_short.replace("MIN_", "")"
Maybe using the follow patch can resole this issue:
if i in OVIRT_VARS:
if int(OVIRT_VARS[i]) < int(self.__dict__[i_short]):
logger.error(("%s is smaller than minimum required size " +
"of: %s") % (i, self.__dict__[i_short]))
print (("\n%s is smaller than minimum required size of: " +
"%s") % (i, self.__dict__[i_short]))
return False
i_short = i_short.replace("MIN_", "")
logger.info(("Setting value for %s to %s " %
(self.__dict__[i_short], OVIRT_VARS[i])))
self.__dict__[i_short] = int(OVIRT_VARS[i])
else:
logger.info("Using default value for: %s" % i_short)
[root@localhost data]# diff storage.py.Old-Version storage.py.New-Version
106c106,107
< logging.info(("Setting value for %s to %s " %
---
> i_short = i_short.replace("MIN_", "")
> logger.info(("Setting value for %s to %s " %
108d108
< i_short = i_short.replace("MIN_", "")
111c111
< logging.info("Using default value for: %s" % i_short)
---
> logger.info("Using default value for: %s" % i_short)
Version-Release number of selected component (if applicable):
rhev-hypervisor6-6.4-20121126.0.el6
How reproducible:
100%
Steps to Reproduce:
Actual result:
Expect result:
Test version:
rhev-hypervisor6-6.5-20131017.0.iso
ovirt-node-3.0.1-4.el6
Could see the follow log info in ovirt.log
-------------------------
.......
2013-10-23 02:25:26,115 - INFO - storage - Using default value for: BOOT_SIZE
2013-10-23 02:25:26,115 - INFO - storage - Setting value for 256 to 256
2013-10-23 02:25:26,115 - INFO - storage - Setting value for 5 to 5
2013-10-23 02:25:26,115 - INFO - storage - Setting value for 5 to 2048
2013-10-23 02:25:26,115 - INFO - storage - Setting value for -1 to -1
2013-10-23 02:25:26,115 - INFO - storage - Using default value for: SWAP2_SIZE
2013-10-23 02:25:26,115 - INFO - storage - Using default value for: DATA2_SIZE
2013-10-23 02:25:26,116 - INFO - storage - Setting value for 256 to 256
.......
so this bug has been fixed, change the status into "VERIFIED"
This bug is currently attached to errata RHBA-2013:15277. If this change is not to be documented in the text for this errata please either remove it from the errata, set the requires_doc_text flag to minus (-), or leave a "Doc Text" value of "--no tech note required" if you do not have permission to alter the flag.
Otherwise to aid in the development of relevant and accurate release documentation, please fill out the "Doc Text" field above with these four (4) pieces of information:
* Cause: What actions or circumstances cause this bug to present.
* Consequence: What happens when the bug presents.
* Fix: What was done to fix the bug.
* Result: What now happens when the actions or circumstances above occur. (NB: this is not the same as 'the bug doesn't present anymore')
Once filled out, please set the "Doc Type" field to the appropriate value for the type of change made and submit your edits to the bug.
For further details on the Cause, Consequence, Fix, Result format please refer to:
https://bugzilla.redhat.com/page.cgi?id=fields.html#cf_release_notes
Thanks in advance.
(In reply to Charlie from comment #13)
> Not quite sure what the problem was. Would you clarify it for the errata
> text?
This bug was describing about using wrong logging.info("") function to lead to miss some log info in ovirt.log.
Actually it should use logger.info("") not logging.info("") to record log info in ovirt.log.
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.
For information on the advisory, and where to find the updated
files, follow the link below.
If the solution does not work for you, open a new bug report.
http://rhn.redhat.com/errata/RHBA-2014-0033.html
Description of problem: Miss some log info in ovirt.log by using function "logging.info". Should change "logging.info" into "logger.info". Also better to change the position between " logging.info(("Setting value for %s to %s " % (self.__dict__[i_short], OVIRT_VARS[i])))" and "i_short = i_short.replace("MIN_", "")" Maybe using the follow patch can resole this issue: if i in OVIRT_VARS: if int(OVIRT_VARS[i]) < int(self.__dict__[i_short]): logger.error(("%s is smaller than minimum required size " + "of: %s") % (i, self.__dict__[i_short])) print (("\n%s is smaller than minimum required size of: " + "%s") % (i, self.__dict__[i_short])) return False i_short = i_short.replace("MIN_", "") logger.info(("Setting value for %s to %s " % (self.__dict__[i_short], OVIRT_VARS[i]))) self.__dict__[i_short] = int(OVIRT_VARS[i]) else: logger.info("Using default value for: %s" % i_short) [root@localhost data]# diff storage.py.Old-Version storage.py.New-Version 106c106,107 < logging.info(("Setting value for %s to %s " % --- > i_short = i_short.replace("MIN_", "") > logger.info(("Setting value for %s to %s " % 108d108 < i_short = i_short.replace("MIN_", "") 111c111 < logging.info("Using default value for: %s" % i_short) --- > logger.info("Using default value for: %s" % i_short) Version-Release number of selected component (if applicable): rhev-hypervisor6-6.4-20121126.0.el6 How reproducible: 100% Steps to Reproduce: Actual result: Expect result: