Use `Expect(err).NotTo(HaveOccurred())` instead of `Expect(err).Should(Succeed())`

[joelanford]

Nit: I know this is a backport and this line is basically unchanged in this PR but for better readability this probably should be:

Expect(err).NotTo(HaveOccurred())

Succeed() is used more for the function itself:

Ω(DoSomethingSimple()).Should(Succeed())

https://onsi.github.io/gomega/#handling-errors

Originally posted by @hasbro17 in #3895 (comment)

[DanielXLee]

@joelanford I want to try to fix this issue, can I pick it up?

[camilamacedo86]

HI @DanielXLee,

Please, feel free to help with. Your collab is more than welcome. You can assign the issue to yourself by comment /assign <your-gh-name>

[DanielXLee]

Thanks @camilamacedo86
/assign DanielXLee