Coverage increased (+0.3%) to 79.004% when pulling c8735ae on kayrus:response-headers into 53d02bc on gophercloud:master.

Build failed.

• gophercloud-unittest : SUCCESS in 3m 41s
• gophercloud-acceptance-test : TIMED_OUT in 3h 01m 44s
• gophercloud-acceptance-test-ironic : SUCCESS in 32m 14s

recheck

Build failed.

• gophercloud-unittest : SUCCESS in 3m 40s
• gophercloud-acceptance-test : FAILURE in 2h 28m 25s
• gophercloud-acceptance-test-ironic : SUCCESS in 32m 02s

recheck

Build succeeded.

• gophercloud-unittest : SUCCESS in 3m 26s
• gophercloud-acceptance-test : SUCCESS in 2h 26m 28s
• gophercloud-acceptance-test-ironic : SUCCESS in 47m 28s

@jtopjian ready for review. This PR also fixes potential null pointer exceptions.

I mentioned places, where you may want to get some clarifications:

• JSONBody is redundant here, it is already passed directly to Patch method:

  gophercloud/openstack/baremetal/v1/nodes/requests.go

  Line 305 in 53d02bc

  JSONBody: &body,

• Body.Close() is also redundant here:

  gophercloud/openstack/objectstorage/v1/containers/requests.go

  Line 116 in 53d02bc

  resp.Body.Close()

• Optimized this call:

  gophercloud/openstack/keymanager/v1/secrets/requests.go

  Line 313 in 53d02bc

  resp, err := client.Put(url, nil, nil, &gophercloud.RequestOpts{

  and added a body test in unit tests.
• Optimized this call:

  gophercloud/openstack/workflow/v2/workflows/requests.go

  Line 53 in 53d02bc

  _, r.Err = client.Post(url, nil, &r.Body, &gophercloud.RequestOpts{

• Optimized this call:

  gophercloud/openstack/objectstorage/v1/objects/requests.go

  Line 241 in 53d02bc

  resp, err := c.Put(url, nil, nil, &gophercloud.RequestOpts{

  and also added body test in unit tests.

Build succeeded.

• gophercloud-unittest : SUCCESS in 3m 09s
• gophercloud-acceptance-test : SUCCESS in 1h 34m 17s
• gophercloud-acceptance-test-ironic : SUCCESS in 28m 51s