Bug 970663
Summary: | Provide more information for failure to create content | ||
---|---|---|---|
Product: | Red Hat Satellite | Reporter: | Og Maciel <omaciel> |
Component: | Provisioning | Assignee: | Marek Hulan <mhulan> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Og Maciel <omaciel> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | 6.0.1 | CC: | bkearney, cwelton, mhulan, tkolhar |
Target Milestone: | Unspecified | Keywords: | Triaged |
Target Release: | Unused | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
URL: | http://projects.theforeman.org/issues/2615 | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-07-18 21:21:16 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
Og Maciel
2013-06-04 14:18:18 UTC
This one seems to be related only to environments created from Katello awaiting review in https://github.com/theforeman/foreman/pull/695 *** Bug 968774 has been marked as a duplicate of this bug. *** Moved to ON_QA as part of puddle 4.5. Hi, Please provide verification steps Thanks (fixing needinfo) Reading the github, I'm not so sure the issue was necessarily fixed, either. omaciel is/was looking for certain verbosity for such error messages in logs and UI, whereas the fix for this (may have?) been to resolve the issue that was causing the error. If that defect is fixed, I suppose I can verify it, but I'm not so certain we won't see the (lack of) error messaging in logs/UI appear in some other edge case. The bug as was described occurred only in specific scenario. Errors were displayed on casual validations, only when saving crashed because of current organization was selected, hostgroup was not saved and no errors were displayed. However this was not a validation error. It was bug in creation process. So the solution was not to display an error but fix the bug. Adding any more error logging in here does not make sense to me. Only thing we can tell is that some unknown error occurred which is already logged. The only thing that comes to my mind is we could enforce record saving with .save! method to obtain expcetion message however calling this in "already failed" branch could create inconsistency for example when the first save fail because of network problem. Fixing the bug was definitely a good thing but I'm not sure I agree with the logic behind not adding more logging. Anyhow, have not seen this happen with latest code so I'll mark it as VERIFIED: * apr-util-ldap-1.3.9-3.el6_0.1.x86_64 * candlepin-0.8.9-1.el6_4.noarch * candlepin-scl-1-5.el6_4.noarch * candlepin-scl-quartz-2.1.5-5.el6_4.noarch * candlepin-scl-rhino-1.7R3-1.el6_4.noarch * candlepin-scl-runtime-1-5.el6_4.noarch * candlepin-selinux-0.8.9-1.el6_4.noarch * candlepin-tomcat6-0.8.9-1.el6_4.noarch * elasticsearch-0.19.9-8.el6sat.noarch * foreman-1.1.10011-1.noarch * foreman-compute-1.1.10011-1.noarch * foreman-installer-puppet-concat-0-2.d776701.git.0.21ef926.el6sat.noarch * foreman-installer-puppet-dhcp-0-5.3a4a13c.el6sat.noarch * foreman-installer-puppet-dns-0-7.fcae203.el6sat.noarch * foreman-installer-puppet-foreman-0-6.568c5c4.el6sat.noarch * foreman-installer-puppet-foreman_proxy-0-8.bd1e35d.el6sat.noarch * foreman-installer-puppet-puppet-0-3.ab46748.el6sat.noarch * foreman-installer-puppet-tftp-0-5.ea6c5e5.el6sat.noarch * foreman-installer-puppet-xinetd-0-50a267b8.git.0.44aca6a.el6sat.noarch * foreman-libvirt-1.1.10011-1.noarch * foreman-postgresql-1.1.10011-1.noarch * foreman-proxy-1.1.10003-1.el6sat.noarch * foreman-proxy-installer-1.0.1-10.f5ae2cd.el6sat.noarch * katello-1.4.2-14.el6sat.noarch * katello-all-1.4.2-14.el6sat.noarch * katello-candlepin-cert-key-pair-1.0-1.noarch * katello-certs-tools-1.4.2-2.el6sat.noarch * katello-cli-1.4.2-7.el6sat.noarch * katello-cli-common-1.4.2-7.el6sat.noarch * katello-common-1.4.2-14.el6sat.noarch * katello-configure-1.4.3-16.el6sat.noarch * katello-configure-foreman-1.4.3-16.el6sat.noarch * katello-foreman-all-1.4.2-14.el6sat.noarch * katello-glue-candlepin-1.4.2-14.el6sat.noarch * katello-glue-elasticsearch-1.4.2-14.el6sat.noarch * katello-glue-pulp-1.4.2-14.el6sat.noarch * katello-qpid-broker-key-pair-1.0-1.noarch * katello-qpid-client-key-pair-1.0-1.noarch * katello-selinux-1.4.3-3.el6sat.noarch * openldap-2.4.23-31.el6.x86_64 * pulp-rpm-plugins-2.1.2-0.3.beta.el6sat.noarch * pulp-selinux-2.1.2-0.3.beta.el6sat.noarch * pulp-server-2.1.2-0.3.beta.el6sat.noarch * python-ldap-2.3.10-1.el6.x86_64 * ruby193-rubygem-ldap_fluff-0.2.2-1.el6sat.noarch * ruby193-rubygem-net-ldap-0.3.1-2.el6sat.noarch * signo-0.0.18-1.el6sat.noarch * signo-katello-0.0.18-1.el6sat.noarch mass move to CLOSED:CURRENTRELEASE since MDP1 has been released. |