Fixes #38924 - Allow scoped search on 'persistence' field at /api/v2/hosts/:id/packages

[qcjames53]

What are the changes introduced in this pull request?

On the /api/v2/hosts/:id/packages endpoint, 'persistence' is now a scoped_search field, meaning we can use --search and --order in hammer on persistence info. This is particularly important for users who would just like to see their transient host installed packages on a bootc environment.

Considerations taken when implementing this change?

What are the testing steps for this pull request?

1. Pull this PR and this hammer-cli-katello PR for your hammer environment.
2. Verify hammer host package list --host-id <id> --order 'persistence ASC' works.
3. Verify hammer host package list --host-id <id> --search 'persistence = transient' works.

Summary by Sourcery

New Features:

• Expose the package persistence attribute as a scoped search field on the host packages endpoint to support search and ordering from clients like hammer.

[sourcery-ai]

Hey there - I've reviewed your changes and they look great!

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[jeremylenz]

Based on the scoped_search docs I'd recommend using :complete_value => { true => true, false => false }. I'm pretty sure we do that in a few other places in Katello. Also consider only_explicit: true since this won't commonly be searched without the field name.

[qcjames53]

How does the new code look? I ended up pulling the complete_value constants from the persistence model itself.

[jeremylenz]

index_by(&:itself), nifty! TIL

LGTM 👍

[ianballou]

I'm hitting a problem with this:

hammer -d host package list --host-id 1 --search "persistence = transient"

[DEBUG 2025-11-28T03:05:08 API] Headers: {
    :params => {
          "search" => "persistence = transient",
            "page" => 1,
        "per_page" => 1000
    }
}
[DEBUG 2025-11-28T03:05:08 API] Using authenticator: HammerCLIForeman::Api::InteractiveBasicAuth
[DEBUG 2025-11-28T03:05:08 API] Response: {
         "total" => 0,
      "subtotal" => 0,
    "selectable" => 0,
          "page" => nil,
      "per_page" => nil,
         "error" => "'persistence' should be one of 'transient, persistent', but the query was 'transient'",
        "search" => "persistence = transient",
          "sort" => {
           "by" => "name",
        "order" => "asc"
    },
       "results" => []
}

Similarly:

[ianballou]

Sending transient alone in the search I think looks instead at the package name by default.

[qcjames53]

I have a fix ready in #11584, sorry about that.