Note: This bug is displayed in read-only format because
the product is no longer active in Red Hat Bugzilla.
Red Hat Satellite engineering is moving the tracking of its product development work on Satellite 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 "Satellite project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs will be migrated starting at the end of May. 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 "Satellite project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/SAT-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.
Description of problem:
If you check the sync status under content management tab, you will find the "Synchronize Now" button active even if no repo is selected for sync.
When you click on this button without selecting any repo, UI throws :
undefined method `each' for nil:NilClass
Version-Release number of selected component (if applicable):
[root@dhcp201-198 ~]# rpm -qa | grep pulp
mod_wsgi-3.2-6.pulp.fc15.x86_64
pulp-0.0.244-5.fc15.noarch
m2crypto-0.21.1.pulp-5.fc15.x86_64
katello-glue-pulp-0.1.98-1.git.33.a663155.fc15.x86_64
pulp-common-0.0.244-5.fc15.noarch
[root@dhcp201-198 ~]# rpm -qa | grep katello
katello-common-0.1.98-1.git.33.a663155.fc15.noarch
katello-all-0.1.98-1.git.33.a663155.fc15.x86_64
katello-cli-0.1.11-1.git.0.facb3bb.fc15.noarch
katello-cli-common-0.1.10-1.git.1133.ea1fcc6.fc15.noarch
katello-glue-foreman-0.1.98-1.git.33.a663155.fc15.x86_64
katello-configure-0.1.10-1.git.0.78fbe70.fc15.noarch
katello-glue-pulp-0.1.98-1.git.33.a663155.fc15.x86_64
katello-0.1.101-1.git.6.83ea7cf.fc15.x86_64
katello-glue-candlepin-0.1.98-1.git.33.a663155.fc15.x86_64
katello-repos-0.1.3-1.git.0.db2bd1d.fc15.noarch
[root@dhcp201-198 ~]#
How reproducible:
always
Steps to Reproduce:
1. Add a repo
2. open sync status page under content management
3. without selecting any repo, click "Synchronize Now"
Actual results:
undefined method `each' for nil:NilClass
Expected results:
"Synchronize Now" button should be disabled if user id not selecting any repo for sync.
Additional info:
The error message seems to be working or already fixed. If you click on 'synchronize Now' you should not be seeing the undefined method error. Regarding the the click action graying the button, seems to be an issue with version of the browser, newer versions of firefox dont give the same behaviour.
Talking to shughes, we decided to remove the logic to disable button and keep it enabled.
commit b386ffe283254c249afd6d8c62b420028364bc01
Verified with following katello build:
[root@dhcp201-197 ~]# rpm -qa | grep katello
katello-glue-pulp-0.1.155-1.el6.noarch
katello-configure-0.1.48-1.el6.noarch
katello-trusted-ssl-cert-1.0-1.noarch
katello-cli-common-0.1.31-1.el6.noarch
katello-glue-foreman-0.1.155-1.el6.noarch
katello-common-0.1.155-1.el6.noarch
katello-httpd-ssl-key-pair-1.0-1.noarch
katello-0.1.155-1.el6.noarch
katello-qpid-broker-key-pair-1.0-1.noarch
katello-repos-0.1.4-1.el6.noarch
katello-cli-0.1.31-1.el6.noarch
katello-glue-candlepin-0.1.155-1.el6.noarch
katello-all-0.1.155-1.el6.noarch
katello-certs-tools-1.0.1-1.el6.noarch
[root@dhcp201-197 ~]#
The reported error "undefined method `each' for nil:NilClass" is not reproducible while clicking on 'Sysncronize Now' button and withoiut selecting any repo.
However a long traceback is generated in production.log. I'll log a separate defect for that. Moving this to verified.