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

MCO-395: daemon: Completely remove imageInspect code #3821

Closed

Commits on Sep 6, 2023

  1. daemon: Completely remove imageInspect code

    Actually, I don't believe any of this logic is necessary anymore.
    
    - We can now unambiguously rely on the container image being bootable;
      if it's not, it should be rpm-ostree that errors out!  There's
      no value in us "second guessing" it and having a duplicate fetch
      path; it just creates more problems
    - Now crucially, `checkOS` had a special case for comparing the
      ostree commit digest, I think to try to deal with the firstboot
      problem.  But actually, what we've been doing for a while for
      scaleup of old bootimages is a double reboot, and hence we don't
      need special case logic for this!
    
    This entirely deletes a lot of unnecessary code, more fully
    deferring OS updates to rpm-ostree.
    cgwalters committed Sep 6, 2023
    Configuration menu
    Copy the full SHA
    03389cf View commit details
    Browse the repository at this point in the history