Uploaded image for project: 'Container Tools'
  1. Container Tools
  2. RUN-3930

[containers/podman] port podman over to new moby/moby module instead of the old docker/docker one

XMLWordPrintable

    • False
    • Hide

      None

      Show
      None
    • False
    • Not Selected
    • rhel-container-tools

      [3625243800] Upstream Reporter: Paul Holzinger
      Upstream issue status: Open
      Upstream description:

      Image was already moved in https://github.com/containers/container-libs/pull/459

      We do vendor update it in https://github.com/containers/podman/pull/27342 which broke swagger so I did a quick fix in https://github.com/containers/podman/pull/27342/commits/6f9bf07a348cb824e0249817812befd54a08ea19 for that to unblock vendoring.

      However we need someone to port all the types to use the moby modules in podman so the docker module will get dropped.


      On another note to we actually benifit from reusing the docker/moby types instead of having our own definitions? All the past updates were quite painful as they keep moving things around so we already spend a ton of work on these changes. And in practice it doesn't gain much, like just because they add new field it still doesn't mean it works in podman becuase podman of course still has to actually populate these fields with our data to return them in the API. And right now we still do not properly track and implement the new docker API changes so these updates are not helping for that either.


      Upstream URL: https://github.com/containers/podman/issues/27536

              Unassigned Unassigned
              upstream-sync Upstream Sync
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated: