Avoid recording multiple events

[adrianchiris]

In case update operation fail in node status writer we should not record an event.

move event sending logic to its own method
and call it after status has been successfully updated.

[github-actions]

Thanks for your PR,
To run vendors CIs use one of:

• /test-all: To run all tests for all vendors.
• /test-e2e-all: To run all E2E tests for all vendors.
• /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

• /skip-all: To skip all tests for all vendors.
• /skip-e2e-all: To skip all E2E tests for all vendors.
• /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
  Best regards.

[adrianchiris]

@rollandf PTAL.

still need to test this one on my setup.

[coveralls]

Pull Request Test Coverage Report for Build 6560379546

• 0 of 23 (0.0%) changed or added relevant lines in 1 file are covered.
• 6 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-0.003%) to 25.18%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
pkg/daemon/writer.go	0	23	0.0%

Files with Coverage Reduction	New Missed Lines	%
api/v1/helper.go	3	42.04%
pkg/daemon/writer.go	3	0.0%

Totals
Change from base Build 6552022763:	-0.003%
Covered Lines:	2270
Relevant Lines:	9015

---

💛 - Coveralls

[rollandf]

LGTM

[zeeke]

LGTM

Good catch

[adrianchiris]

This is working as expected.

There is another issue with roles in the operator following events introduction, ill send fix in a separarte PR.