Blkio weight/leafWeight pointer value

[cclerget]

Prevent weight/leafweight from returning address value instead :

type LinuxWeightDevice struct {
	linuxBlockIODevice
	Weight *uint16 `json:"weight,omitempty"`
	LeafWeight *uint16 `json:"leafWeight,omitempty"`
}

Not sure if this is the right thing to do by returning "1000" when pointer is null ...

[codecov-io]

Codecov Report

Merging #69 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #69      +/-   ##
==========================================
- Coverage    14.3%   14.27%   -0.03%     
==========================================
  Files          23       23              
  Lines        4579     4588       +9     
==========================================
  Hits          655      655              
- Misses       3803     3812       +9     
  Partials      121      121

Impacted Files	Coverage Δ
blkio.go	20.77% <0%> (-0.95%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3bc6dde...64bade4. Read the comment docs.

[crosbymichael]

@cclerget i'd make https://github.com/containerd/cgroups/pull/69/files#diff-a0138ebe17a246a844ed97a795eecd9eR194 and https://github.com/containerd/cgroups/pull/69/files#diff-a0138ebe17a246a844ed97a795eecd9eR206 conditional. Don't place them in the slice if they have nil weights

[cclerget]

@crosbymichael Sorry about confusion, updated with your suggestions

[crosbymichael]

Looks great @cclerget ! maybe a simple test to make sure this does not panic anymore would be amazing.

[crosbymichael]

And can you squash your commits into one before we merge?

[cclerget]

Yep, I successfully run same tests (from opencontainers/runtime-tools validation tests) without panic

[crosbymichael]

LGTM

[dqminh]

LGTM