Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rpm-ostree: Use upstream Rust bindings #491

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Commits on Mar 8, 2021

  1. rpm-ostree: Use upstream Rust bindings

    Depends: coreos/rpm-ostree#2636
    
    Use the upstream client bindings for the status data.
    
    An interesting note: In the upstream API, the commit
    metadata is exposed as a generic `HashMap<>` because
    that's how it's used by both ostree and rpm-ostree.  It'd
    be a layering violation for us to hardcode `coreos-assembler.basearch`
    in the rpm-ostree git for example, not to mention `fedora-coreos.stream`.
    So those constants stay here in zincati.
    
    I dropped the `_json` terminology from various functions
    because that's weird - we parsed the data from JSON, but
    that's not really very relevant except as an implementation
    detail.  It's just a `Deployment`, not a `DeploymentJSON`.
    
    Also I dropped for now the optimization of using `--booted`;
    it's not a huge amount of data.  If we care we can re-add that
    later.  Note it makes caching more complex because then we
    need to carefully check the booted state too.
    cgwalters committed Mar 8, 2021
    Configuration menu
    Copy the full SHA
    9a9c5f9 View commit details
    Browse the repository at this point in the history