-
Notifications
You must be signed in to change notification settings - Fork 16
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
This commit fixes the issue of integration tests that were never run. Reason for this bug? This has mostly happened when apiserver & etcd binaries were upgraded. The new binaries make use of new flags & deprecates use of old as well as insecure flags. The framework package present in test/integration did not handle these new changes and was rather swallowing the startup errors resulting in passing these tests that never ran. What has been done? The changes include copying of integration test related framework code from controller runtime library. The copied codebase fits the current integration test framework of Metac nicely. This approach also avoid import of unrequired dependencies & associated go vendoring problems that often arises due to import of controller-runtime libraries. closes #136 Refer - https://github.com/kubernetes-sigs/controller-runtime/tree/master/pkg/internal/testing Signed-off-by: AmitKumarDas <[email protected]>
- Loading branch information
Amit Kumar Das
authored
Apr 21, 2020
1 parent
1ae68f8
commit afeebdf
Showing
38 changed files
with
2,242 additions
and
779 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,6 +2,7 @@ | |
/vendor/ | ||
/metac | ||
/hack/bin/ | ||
/test/integration/framework/assets/bin/ | ||
/manifests/crds/ | ||
.*.swp | ||
.history |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.