FIP 0027 integration

[arajasek]

Integrates a change to FIP-0027. o

[ZenGround0]

@arajasek I can't tag you for it but need your review here

[ZenGround0]

TestRestartOnlyProviderDataTransfer appears to be flaky

[codecov-commenter]

Codecov Report

Merging #694 (954803e) into master (70dbd2f) will increase coverage by 0.15%.
The diff coverage is 55.56%.

@@            Coverage Diff             @@
##           master     #694      +/-   ##
==========================================
+ Coverage   58.60%   58.74%   +0.15%     
==========================================
  Files          64       64              
  Lines        5362     5390      +28     
==========================================
+ Hits         3142     3166      +24     
+ Misses       1882     1880       -2     
- Partials      338      344       +6     

Impacted Files	Coverage Δ
storagemarket/impl/client.go	21.38% <ø> (ø)
storagemarket/impl/provider_environments.go	9.35% <0.00%> (+0.18%)	⬆️
storagemarket/impl/providerutils/providerutils.go	94.29% <ø> (ø)
storagemarket/types.go	85.37% <ø> (ø)
...oragemarket/impl/providerstates/provider_states.go	84.35% <33.34%> (-0.63%)	⬇️
storagemarket/impl/clientutils/clientutils.go	40.55% <55.56%> (+1.84%)	⬆️
storagemarket/impl/provider.go	26.88% <63.34%> (+3.99%)	⬆️

[arajasek]

Approve to all of @ZenGround0's commits! If someone can review the first commit and green check this, we can merge!

[dirkmc]

I'm approving this PR but I do have one non-blocking question.
There is a commented out test in storagemarket/impl/providerstates/provider_states_test.go.
Why is this test commented out? If it's no longer relevant let's remove it.

[ZenGround0]

There is a commented out test in storagemarket/impl/providerstates/provider_states_test.go.
Why is this test commented out? If it's no longer relevant let's remove it.

Thanks, this test is made irrelevant by the new label type which forbids invalidly long labels at the type level. Removing.