Closed mquhuy closed 3 months ago
/cc @tuminoid @Rozzii @mboukhalfa @Sunnatillo @adilGhaffarDev @huutomerkki
@adilGhaffarDev: adding LGTM is restricted to approvers and reviewers in OWNERS files.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dtantsur
The full list of commands accepted by this bot can be found here.
The pull request process is described here
A minor PR to not use gomega's
Expect()
inLoadBMCConfig()
.Reason: Using
Expect()
inside the function prevents reusage of the function in locations where ginkgo is not in use (e.g. https://github.com/metal3-io/baremetal-operator/pull/1767). This inclusion is unnecessary anyway, andExpect()
should be located in the e2e suite.