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.
Bug 2109810 - Search for string in n-v-r.a format fails for custom packages but not for Red Hat packages
Summary: Search for string in n-v-r.a format fails for custom packages but not for Red...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Repositories
Version: 6.10.7
Hardware: Unspecified
OS: Unspecified
urgent
medium
Target Milestone: 6.12.0
Assignee: Pavel Moravec
QA Contact: Cole Higgins
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-07-22 08:06 UTC by Eric Nothen
Modified: 2023-02-21 15:48 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-11-16 13:34:44 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 35290 0 Normal New Katello rpm search via nvra also 2022-07-28 13:08:43 UTC
Red Hat Issue Tracker SAT-11989 0 None None None 2022-08-17 14:57:36 UTC
Red Hat Product Errata RHSA-2022:8506 0 None None None 2022-11-16 13:35:05 UTC

Description Eric Nothen 2022-07-22 08:06:39 UTC 
Description of problem:

When searching for a package using name-version-release.architecture format, the search returns the correct results if the package is a provided by Red Hat in an official repository. When doing the same search for a custom package uploaded into a custom product and repository, the search returns no results, but removing the architecture component makes the search work.

Version-Release number of selected component (if applicable):
Customer reported the issue on Satellite 6.10.7, but I was able to reproduce it on 6.11.0 as well.

How reproducible:

Always reproducible with a custom rpm.

Steps to Reproduce:
1. Create a custom product
~~~
# hammer product create --name Google --organization-id 1 
Product created.
~~~

2. Create a custom repository
~~~
# hammer repository create --content-type yum --name Chrome --product Google
Repository created.
~~~

3. Upload a package to the custom repository
~~~
# hammer repository upload-content --id 245 --path ./google-chrome-stable_current_x86_64.rpm 
Successfully uploaded file 'google-chrome-stable_current_x86_64.rpm'
~~~

4. Get the n-v-r.a of the uploaded package
~~~
# rpm -qp --queryformat '%{NAME}-%{VERSION}-%{RELEASE}.%{ARCH}\n' ./google-chrome-stable_current_x86_64.rpm 2>/dev/null
google-chrome-stable-103.0.5060.134-1.x86_64
~~~

5. Search for the package
~~~
# curl -u admin 'https://satellite.keller.lab/katello/api/v2/packages?per_page=10000&page=1&repository_id=245&search=google-chrome-stable-103.0.5060.134-1.x86_64' | python3 -m json.tool
Enter host password for user 'admin':
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   184    0   184    0     0    392      0 --:--:-- --:--:-- --:--:--   392
{
    "total": 1,
    "subtotal": 0,
    "selectable": 0,
    "page": "1",
    "per_page": "10000",
    "error": null,
    "search": "google-chrome-stable-103.0.5060.134-1.x86_64",
    "sort": {
        "by": null,
        "order": null
    },
    "results": []
}
~~~

Actual results:

Search brings no results, but removing the architecture works:
~~~
# curl -u admin 'https://satellite.keller.lab/katello/api/v2/packages?per_page=10000&page=1&repository_id=245&search=google-chrome-stable-103.0.5060.134-1' | python3 -m json.tool
Enter host password for user 'admin':
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   848    0   848    0     0   1775      0 --:--:-- --:--:-- --:--:--  1777
{
    "total": 1,
    "subtotal": 1,
    "selectable": 1,
    "page": "1",
    "per_page": "10000",
    "error": null,
    "search": "google-chrome-stable-103.0.5060.134-1",
    "sort": {
        "by": null,
        "order": null
    },
    "results": [
        {
            "id": 48234,
            "pulp_id": "/pulp/api/v3/content/rpm/packages/4e7c2aa5-cd03-48ef-89e8-daad2acde79c/",
            "name": "google-chrome-stable",
            "version": "103.0.5060.134",
            "release": "1",
            "arch": "x86_64",
            "epoch": "0",
            "summary": "Google Chrome",
            "modular": false,
            "filename": "google-chrome-stable_current_x86_64.rpm",
            "sourcerpm": "google-chrome-stable-103.0.5060.134-1.src.rpm",
            "checksum": "0cb17f6ed4f421fe2c61e66d6db28dd442c3a5f4fed07629f23cc070d19165ea",
            "nvrea": "google-chrome-stable-103.0.5060.134-1.x86_64",
            "nvra": "google-chrome-stable-103.0.5060.134-1.x86_64",
            "uuid": "/pulp/api/v3/content/rpm/packages/4e7c2aa5-cd03-48ef-89e8-daad2acde79c/",
            "hosts_available_count": 0,
            "hosts_applicable_count": 0
        }
    ]
}
~~~


Expected results:

Search would behave the same way as with packages provided by Red Hat repositories, in which including the architecture or not makes no difference.

Search for abrt-addon-kerneloops-2.10.9-10.el8.x86_64 brings 1 result:
~~~
# curl -u admin 'https://satellite.keller.lab/katello/api/v2/packages?per_page=10000&page=1&repository_id=131&search=abrt-addon-kerneloops-2.10.9-10.el8.x86_64' | python3 -m json.tool
Enter host password for user 'admin':
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   849    0   849    0     0    822      0 --:--:--  0:00:01 --:--:--   822
{
    "total": 13452,
    "subtotal": 1,
    "selectable": 1,
    "page": "1",
    "per_page": "10000",
    "error": null,
    "search": "abrt-addon-kerneloops-2.10.9-10.el8.x86_64",
    "sort": {
        "by": null,
        "order": null
    },
    "results": [
        {
            "id": 36281,
            "pulp_id": "/pulp/api/v3/content/rpm/packages/b7908aec-2ba2-47bf-93c0-1a73b57e4f00/",
            "name": "abrt-addon-kerneloops",
            "version": "2.10.9",
            "release": "10.el8",
            "arch": "x86_64",
            "epoch": "0",
            "summary": "abrt's kerneloops addon",
            "modular": false,
            "filename": "abrt-addon-kerneloops-2.10.9-10.el8.x86_64.rpm",
            "sourcerpm": "abrt-2.10.9-10.el8.src.rpm",
            "checksum": "0aedcc7340b780821663dfafe63e49890d22930df4d799a6adbc257be7608195",
            "nvrea": "abrt-addon-kerneloops-2.10.9-10.el8.x86_64",
            "nvra": "abrt-addon-kerneloops-2.10.9-10.el8.x86_64",
            "uuid": "/pulp/api/v3/content/rpm/packages/b7908aec-2ba2-47bf-93c0-1a73b57e4f00/",
            "hosts_available_count": 0,
            "hosts_applicable_count": 0
        }
    ]
}
~~~

Search for abrt-addon-kerneloops-2.10.9-10.el8 also brings one result:
~~~
# curl -u admin 'https://satellite.keller.lab/katello/api/v2/packages?per_page=10000&page=1&repository_id=131&search=abrt-addon-kerneloops-2.10.9-10.el8' | python3 -m json.tool
Enter host password for user 'admin':
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   842    0   842    0     0    821      0 --:--:--  0:00:01 --:--:--   822
{
    "total": 13452,
    "subtotal": 1,
    "selectable": 1,
    "page": "1",
    "per_page": "10000",
    "error": null,
    "search": "abrt-addon-kerneloops-2.10.9-10.el8",
    "sort": {
        "by": null,
        "order": null
    },
    "results": [
        {
            "id": 36281,
            "pulp_id": "/pulp/api/v3/content/rpm/packages/b7908aec-2ba2-47bf-93c0-1a73b57e4f00/",
            "name": "abrt-addon-kerneloops",
            "version": "2.10.9",
            "release": "10.el8",
            "arch": "x86_64",
            "epoch": "0",
            "summary": "abrt's kerneloops addon",
            "modular": false,
            "filename": "abrt-addon-kerneloops-2.10.9-10.el8.x86_64.rpm",
            "sourcerpm": "abrt-2.10.9-10.el8.src.rpm",
            "checksum": "0aedcc7340b780821663dfafe63e49890d22930df4d799a6adbc257be7608195",
            "nvrea": "abrt-addon-kerneloops-2.10.9-10.el8.x86_64",
            "nvra": "abrt-addon-kerneloops-2.10.9-10.el8.x86_64",
            "uuid": "/pulp/api/v3/content/rpm/packages/b7908aec-2ba2-47bf-93c0-1a73b57e4f00/",
            "hosts_available_count": 0,
            "hosts_applicable_count": 0
        }
    ]
}
~~~
Comment 1 Brad Buckingham 2022-07-25 13:10:58 UTC 
Is this a regression from Satellite 6.9?
Comment 2 Pavel Moravec 2022-07-28 10:19:21 UTC 
I can not reproduce it on my satellite-6.10.7-1.el7sat.noarch :


# curl -u admin:$PASS "https://$(hostname -f)/katello/api/v2/packages?per_page=10000&page=1&repository_id=9294&search=google-chrome-stable-103.0.5060.134-1.x86_64"
{"total":2,"subtotal":1,"page":"1","per_page":"10000","error":null,"search":"google-chrome-stable-103.0.5060.134-1.x86_64","sort":{"by":null,"order":null},"results":[{"id":305245,"pulp_id":"/pulp/api/v3/content/rpm/packages/4ca2bb56-e889-4b55-b7c4-3f4c59f9f701/","name":"google-chrome-stable","version":"103.0.5060.134","release":"1","arch":"x86_64","epoch":"0","summary":"Google Chrome","modular":false,"filename":"google-chrome-stable-103.0.5060.134-1.x86_64.rpm","sourcerpm":"google-chrome-stable-103.0.5060.134-1.src.rpm","checksum":"0cb17f6ed4f421fe2c61e66d6db28dd442c3a5f4fed07629f23cc070d19165ea","nvrea":"google-chrome-stable-103.0.5060.134-1.x86_64","nvra":"google-chrome-stable-103.0.5060.134-1.x86_64","uuid":"/pulp/api/v3/content/rpm/packages/4ca2bb56-e889-4b55-b7c4-3f4c59f9f701/","hosts_available_count":0,"hosts_applicable_count":0}]}

# curl -u admin:$PASS "https://$(hostname -f)/katello/api/v2/packages?per_page=10000&page=1&repository_id=9294&search=google-chrome-stable-103.0.5060.134-1"
{"total":2,"subtotal":1,"page":"1","per_page":"10000","error":null,"search":"google-chrome-stable-103.0.5060.134-1","sort":{"by":null,"order":null},"results":[{"id":305245,"pulp_id":"/pulp/api/v3/content/rpm/packages/4ca2bb56-e889-4b55-b7c4-3f4c59f9f701/","name":"google-chrome-stable","version":"103.0.5060.134","release":"1","arch":"x86_64","epoch":"0","summary":"Google Chrome","modular":false,"filename":"google-chrome-stable-103.0.5060.134-1.x86_64.rpm","sourcerpm":"google-chrome-stable-103.0.5060.134-1.src.rpm","checksum":"0cb17f6ed4f421fe2c61e66d6db28dd442c3a5f4fed07629f23cc070d19165ea","nvrea":"google-chrome-stable-103.0.5060.134-1.x86_64","nvra":"google-chrome-stable-103.0.5060.134-1.x86_64","uuid":"/pulp/api/v3/content/rpm/packages/4ca2bb56-e889-4b55-b7c4-3f4c59f9f701/","hosts_available_count":0,"hosts_applicable_count":0}]}

The repo 9294 is a custom repo with a few other packages (from Red Hat, but still..)


I will diagnose more via the support case.
Comment 3 Pavel Moravec 2022-07-28 12:34:25 UTC 
Here is the cause:

If RPM filename does not follow NVRA pattern, but source RPM filename follows NVR pattern, then we get this issue. Since this is the key search done:

# search="google-chrome-stable-103.0.5060.134-1"
# su - postgres -c "psql foreman -c \"SELECT name,version,release,arch,epoch,filename,sourcerpm,checksum FROM katello_rpms WHERE ((katello_rpms.name ILIKE '%${search}%' OR katello_rpms.version = '${search}' OR katello_rpms.release = '${search}' OR katello_rpms.arch ILIKE '%${search}%' OR katello_rpms.epoch ILIKE '%${search}%' OR katello_rpms.filename ILIKE '%${search}%' OR katello_rpms.sourcerpm ILIKE '%${search}%' OR katello_rpms.checksum ILIKE '%${search}%'));\""
         name         |    version     | release |  arch  | epoch |                filename                 |                   sourcerpm                   |                             chec
ksum                            
----------------------+----------------+---------+--------+-------+-----------------------------------------+-----------------------------------------------+---------------------------------
---------------------------------
 google-chrome-stable | 103.0.5060.134 | 1       | x86_64 | 0     | google-chrome-stable_current_x86_64.rpm | google-chrome-stable-103.0.5060.134-1.src.rpm | 0cb17f6ed4f421fe2c61e66d6db28dd4
42c3a5f4fed07629f23cc070d19165ea
(1 row)

#

but:

# search="google-chrome-stable-103.0.5060.134-1.x86_64"
# su - postgres -c "psql foreman -c \"SELECT name,version,release,arch,epoch,filename,sourcerpm,checksum FROM katello_rpms WHERE ((katello_rpms.name ILIKE '%${search}%' OR katello_rpms.version = '${search}' OR katello_rpms.release = '${search}' OR katello_rpms.arch ILIKE '%${search}%' OR katello_rpms.epoch ILIKE '%${search}%' OR katello_rpms.filename ILIKE '%${search}%' OR katello_rpms.sourcerpm ILIKE '%${search}%' OR katello_rpms.checksum ILIKE '%${search}%'));\""
 name | version | release | arch | epoch | filename | sourcerpm | checksum
------+---------+---------+------+-------+----------+-----------+----------
(0 rows)

#

Katello matches the search (sub)string against:
- name
- version
- release
- arch
- epoch
- filename
- sourcerpm

We shall extend the search to "nvra" attribute, then the search would succeed:


# search="google-chrome-stable-103.0.5060.134-1.x86_64"
# su - postgres -c "psql foreman -c \"SELECT name,version,release,arch,epoch,filename,sourcerpm,checksum FROM katello_rpms WHERE ((katello_rpms.name ILIKE '%${search}%' OR katello_rpms.version = '${search}' OR katello_rpms.release = '${search}' OR katello_rpms.arch ILIKE '%${search}%' OR katello_rpms.epoch ILIKE '%${search}%' OR katello_rpms.filename ILIKE '%${search}%' OR katello_rpms.sourcerpm ILIKE '%${search}%' OR katello_rpms.checksum ILIKE '%${search}%' OR katello_rpms.nvra ILIKE '%${search}%'));\""
         name         |    version     | release |  arch  | epoch |                filename                 |                   sourcerpm                   |                             chec
ksum                            
----------------------+----------------+---------+--------+-------+-----------------------------------------+-----------------------------------------------+---------------------------------
---------------------------------
 google-chrome-stable | 103.0.5060.134 | 1       | x86_64 | 0     | google-chrome-stable_current_x86_64.rpm | google-chrome-stable-103.0.5060.134-1.src.rpm | 0cb17f6ed4f421fe2c61e66d6db28dd4
42c3a5f4fed07629f23cc070d19165ea
(1 row)

#


This should be a trivial fix, I would say.
Comment 4 Pavel Moravec 2022-07-28 12:46:01 UTC 
.. and here is the patch (for Satellite, aply in /opt/theforeman/tfm/root/usr/share/gems/gems/katello-* directory):



--- a/app/models/katello/rpm.rb
+++ b/app/models/katello/rpm.rb
@@ -16,6 +16,7 @@ module Katello
     scoped_search :on => :evr, :ext_method => :scoped_search_evr, :only_explicit => true
     scoped_search :on => :filename, :complete_value => true
     scoped_search :on => :sourcerpm, :complete_value => true
+    scoped_search :on => :nvra, :complete_value => true
     scoped_search :on => :modular, :complete_value => true, :only_explicit => true
     scoped_search :on => :checksum
Comment 5 Bryan Kearney 2022-07-28 16:05:13 UTC 
Upstream bug assigned to pmoravec
Comment 6 Bryan Kearney 2022-07-28 16:05:16 UTC 
Upstream bug assigned to pmoravec
Comment 7 Eric Nothen 2022-08-09 07:29:23 UTC 
(In reply to Brad Buckingham from comment #1)
> Is this a regression from Satellite 6.9?

I don't have a running Satellite 6.9, but looking at tfm-rubygem-katello-3.18.1.53-1.el7sat.noarch.rpm, it doesn't look like, as it's the same as Pavel fixed on latest:

~~~
$ egrep -n "scoped_search :on" opt/theforeman/tfm/root/usr/share/gems/gems/katello-3.18.1.53/app/models/katello/rpm.rb
11:    scoped_search :on => :name, :complete_value => true
12:    scoped_search :on => :version, :complete_value => true, :ext_method => :scoped_search_version
13:    scoped_search :on => :release, :complete_value => true, :ext_method => :scoped_search_release
14:    scoped_search :on => :arch, :complete_value => true
15:    scoped_search :on => :epoch, :complete_value => true
16:    scoped_search :on => :evr, :ext_method => :scoped_search_evr, :only_explicit => true
17:    scoped_search :on => :filename, :complete_value => true
18:    scoped_search :on => :sourcerpm, :complete_value => true
19:    scoped_search :on => :modular, :complete_value => true, :only_explicit => true
20:    scoped_search :on => :checksum
~~~
Comment 8 Bryan Kearney 2022-08-11 20:05:06 UTC 
Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/35290 has been resolved.
Comment 13 errata-xmlrpc 2022-11-16 13:34:44 UTC 
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 (Important: Satellite 6.12 Release), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2022:8506
Note You need to log in before you can comment on or make changes to this bug.