Pat St. laurent

> To repeat my question from earlier: the issue this PR is fixing, #46658, states the following: > > > We should switch to getting the `state` property, **and do...

> The subtlety here is that dockerd does not _depend_ on FirewallD in the traditional sense. As you have pointed out, on non-FirewallD systems we fall back to raw iptables...

@thaJeztah When you have a free moment, I made the change you requested.

> This change behaves incorrectly. > > ``` > root@56e9443a7364:/go/src/github.com/docker/docker# cat /etc/docker/daemon.json > { > "max-download-attempts": 7, > "runtimes": { > "crun": { > "path": "/usr/local/bin/crun" > } > }...

> This PR is described as fixing #44921. The internal user story says "Description and A/C in #44921." The acceptance criteria for #44921 are to fix the regression _for all...

> I'm aware of the description, we discussed changing a single flag as an introduction to docker. I wasn't supposed to handle the entire issue. The tasks are only for...

Is there something wrong with the validate/validate-md check? The diff in my PR doesn't show any of the changes that are reflected in the check. A git diff locally doesn't...