bugfix: cli should not split args by comma for env/labels

[fuweid]

Signed-off-by: Wei Fu [email protected]

Ⅰ. Describe what this PR did

flagset.StringSlice will split args by comma so that the env/labels
parse job fails. Should use StringArray for env/labels.

Ⅱ. Does this pull request fix one issue?

NONE

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

already added it.

Ⅳ. Describe how to verify it

CI

Ⅴ. Special notes for reviews

[codecov-io]

Codecov Report

Merging #2171 into master will increase coverage by 12.82%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #2171       +/-   ##
===========================================
+ Coverage   51.22%   64.04%   +12.82%     
===========================================
  Files         209      209               
  Lines       16725    16904      +179     
===========================================
+ Hits         8567    10827     +2260     
+ Misses       7150     4735     -2415     
- Partials     1008     1342      +334

Flag	Coverage Δ
#criv1alpha1test	32.9% <ø> (?)
#criv1alpha2test	33.58% <ø> (?)
#integrationtest	39.25% <ø> (ø)	⬆️
#unittest	23.9% <ø> (ø)	⬆️

Impacted Files	Coverage Δ
ctrd/image.go	75% <0%> (-2.2%)	⬇️
ctrd/container.go	40.48% <0%> (-1.29%)	⬇️
daemon/mgr/container_storage.go	50.93% <0%> (+0.46%)	⬆️
pkg/utils/utils.go	84.3% <0%> (+0.58%)	⬆️
daemon/mgr/container_types.go	83.63% <0%> (+2.72%)	⬆️
daemon/config/config.go	44.28% <0%> (+2.85%)	⬆️
daemon/mgr/container_utils.go	84.33% <0%> (+3.01%)	⬆️
pkg/meta/local.go	63.91% <0%> (+3.09%)	⬆️
daemon/mgr/container.go	56.98% <0%> (+3.28%)	⬆️
pkg/user/user.go	40% <0%> (+3.33%)	⬆️
... and 39 more

[allencloud]

LGTM