Closed smoshiur1237 closed 1 week ago
/test metal3-ubuntu-e2e-integration-test-main
One test would be enough to check the added directory, so overriding the 2nd test. /override metal3-centos-e2e-integration-test-main
@smoshiur1237: Overrode contexts on behalf of smoshiur1237: metal3-centos-e2e-integration-test-main
/cc @tuminoid @mboukhalfa
Manifest collection is done under different namespace directory: https://jenkins.nordix.org/view/Metal3/job/metal3-ubuntu-e2e-integration-test-main/310/
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mboukhalfa, tuminoid
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@smoshiur1237 Is it intentional that these new directories are empty?
/hold
Just holding until I get answer
Okay, i have got my answer offline, probably in the future we should improve on this in terms of not creating all the "kind" directories for all the namespaces because many of them will be just empty dirs but this PR is a step in the right direction. /hold cancel /lgtm
There is a possibility to miss same kinds of manifest in different namespace. So this PR will add the namespace directory in manifest collection to gather all the manifests which will improve debugging capability.