feature: add Mountpoints in containers/json api

[HusterWan]

Signed-off-by: Michael Wan [email protected]

Ⅰ. Describe what this PR did

add MountPoint in container body in api containers/json

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

[codecov]

Codecov Report

Merging #2816 into master will decrease coverage by 0.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #2816      +/-   ##
==========================================
- Coverage   69.34%   69.32%   -0.02%     
==========================================
  Files         278      278              
  Lines       17428    17432       +4     
==========================================
  Hits        12085    12085              
- Misses       3994     3996       +2     
- Partials     1349     1351       +2

Flag	Coverage Δ
#criv1alpha2_test	39.22% <0%> (-0.01%)	⬇️
#integration_test_0	36.51% <0%> (-0.05%)	⬇️
#integration_test_1	35.3% <0%> (-0.04%)	⬇️
#integration_test_2	36.53% <50%> (-0.02%)	⬇️
#integration_test_3	35.4% <50%> (+0.02%)	⬆️
#node_e2e_test	34.97% <0%> (-0.02%)	⬇️
#unittest	28.77% <0%> (-0.01%)	⬇️

Impacted Files	Coverage Δ
apis/server/container_bridge.go	90% <50%> (-0.45%)	⬇️
apis/server/utils.go	71.15% <0%> (-3.85%)	⬇️
daemon/logger/jsonfile/utils.go	71.54% <0%> (-1.63%)	⬇️
ctrd/client.go	69.72% <0%> (-1.63%)	⬇️
pkg/meta/store.go	67.44% <0%> (-1.56%)	⬇️
daemon/mgr/container_utils.go	82.38% <0%> (-1.14%)	⬇️
ctrd/container.go	53.88% <0%> (-0.38%)	⬇️
cri/v1alpha2/cri.go	71.22% <0%> (+0.63%)	⬆️
cri/ocicni/cni_manager.go	70.58% <0%> (+11.76%)	⬆️

[allencloud]

FWIW，in terms of compatibility, I compared pouchcontainer's returns struct and

moby's(https://github.com/moby/moby/blob/master/api/swagger.yaml#L3434-L3501), and found that we still miss:

Ports
State
SizeRw
SizeRootfs

Will all these missing fields affect your usecases? @wangforthinker @zjumoon01

[wangforthinker]

There are no effect at the moment. But it is better to add missing filed in returns struct. @allencloud

[allencloud]

LGTM