UPSTREAM: 56872: Fix event generation

[gnufied]

Also fix error handling in operation hooks.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1540039

[gnufied]

verified that the PR brings back the metrics:

storage_operation_errors_total{operation_name="volume_attach",volume_plugin="kubernetes.io/aws-ebs"} 1

[gnufied]

/assign @jsafrane

[gnufied]

/retest

[childsb]

/approve

[childsb]

/lgtm

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: childsb, gnufied

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

• vendor/k8s.io/kubernetes/pkg/controller/volume/persistentvolume/OWNERS [childsb]
• vendor/k8s.io/kubernetes/pkg/volume/OWNERS [childsb,gnufied]

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

[gnufied]

/test extended_conformance_install

[gnufied]

/test extended_conformance_install

[jsafrane]

/retest

[gnufied]

test gcp is failing on:

+ ../../bin/ansible.sh ansible-playbook -e openshift_test_repo=https://storage.googleapis.com/origin-ci-test/pr-logs/pull/18442/test_pull_request_origin_extended_conformance_gce/15652/artifacts/rpms playbooks/gcp/openshift-cluster/launch.yml
Activated service account credentials for: [jenkins-ci-provisioner@openshift-gce-devel.iam.gserviceaccount.com]
 [WARNING]: Could not match supplied host pattern, ignoring: all
 [WARNING]: provided hosts list is empty, only localhost is available

Does not look like remotely related to this PR.

[gnufied]

/test gcp

[gnufied]

/retest

[openshift-merge-robot]

/test all [submit-queue is verifying that this PR is safe to merge]

[openshift-merge-robot]

Automatic merge from submit-queue.