You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, in the Metal³ dev-env, BMHs are applied in the metal3 namespace and then inspected during the make process. However, when reusing CAPI tests such as md_remediations.go, which utilizes random namespace we are applying bmh in that namespace through the hook PostNamespaceCreated, it becomes evident that applying BMHs in the metal3 namespace and then deleting them in e2e tests, only to recreate them in a different namespace, is not the most efficient approach.
To streamline the testing process and ensure alignment with evolving testing methodologies, it's recommended to update dev-env so that BMHs are applied as part of the test setup, directly in the appropriate namespace, rather than being applied by default in the metal3 namespace as part of the make process.
Proposed Solution:
Remove the step of applying BMHs in the verify.sh script and inspecting them.
Update the Ansible tests under /tests to apply the BMHs from a separate file before running the tests.
Eliminate the part where BMHs are deleted from the metal3 namespace and instead include applying them as part of every e2e test.
Remove the BMH file bmhosts_crs.yaml since it will no longer be needed (#1353)
/triage accepted
I am willing to accept it but please whoever fixes this make it so that the BMHs can be deployed without running the tests and deploying CPI and CAPM3, this would be a big help for those who are developing only BMO functionality without higher level Metal3 components.
Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues will close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Currently, in the Metal³ dev-env, BMHs are applied in the
metal3
namespace and then inspected during the make process. However, when reusing CAPI tests such asmd_remediations.go
, which utilizes random namespace we are applying bmh in that namespace through the hookPostNamespaceCreated
, it becomes evident that applying BMHs in themetal3
namespace and then deleting them in e2e tests, only to recreate them in a different namespace, is not the most efficient approach.To streamline the testing process and ensure alignment with evolving testing methodologies, it's recommended to update dev-env so that BMHs are applied as part of the test setup, directly in the appropriate namespace, rather than being applied by default in the
metal3
namespace as part of the make process.Proposed Solution:
Reference: metal3-io/cluster-api-provider-metal3#1080
The text was updated successfully, but these errors were encountered: