Hi Wilfred Great progress, thank you. Could you please create a PR so it is easier for people to review and comment?
Thanks Weiwei On Fri, Nov 6, 2020 at 6:17 AM Wilfred Spiegelenburg <wilfr...@apache.org> wrote: > Over the last couple of days I have added all the unit tests and the e2e > tests for YUNIKORN-317 > > The test changes have been committed in three steps: unit tests, e2e tests > and a fix for a broken unit test/lint issues. > In both test changes there is also a limited number of linter clean up > changes included. > > The e2e tests the refactor was limited to using the newly introduced > functions and linter clean up. The test and mock needed for these tests > have now all been moved to *_test.go files. This means that they do not > leave traces in the production code anymore. > > Changes in the non test code that were made as part of these changes all > were fixes of porting issues or for test failures. > All tests pass locally, lint check is far cleaner than it has been in a > long time. Travis has some issues with the lint check which is a known > issue. > > Please review the changes, comment and feedback is welcome. > There is nothing left to do for this task. > I'll check after the weekend what needs to be merged into the branch to > allow merging into trunk. > > Cheers, > Wilfred > > > On Tue, 20 Oct 2020 at 15:01, Weiwei Yang <abvclo...@gmail.com> wrote: > > > Hi Wilfred > > > > Thanks for the update, I will look at the changes this week. > > Thanks for the efforts in putting these together. Thank you! > > > > Weiwei > > > > > > On Mon, Oct 19, 2020 at 6:37 PM Wilfred Spiegelenburg < > wilfr...@apache.org> > > wrote: > > > >> Hi, > >> > >> Just a status update on the big change of removing the cache from the > >> core. > >> Wangda started working on this a while ago. I have just finished a move > of > >> that code and some further refactoring into the dev branch YUNIKORN-317. > >> > >> In the next day or two I will also add the tests to the branch. > >> > >> Please feel free to start looking at the changes and comment or ask > >> questions if you have any. > >> > >> Wilfred > >> > > >