-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Investigate how to split CWA refactoring #847
Comments
It might be possible to use a tool to start most of the coding between python and golang. We need to consider if we just write the go things that interest us, if we write the full golang code, compared to the original python code, and if we write the full golang code, including the rok sections. |
VolumesFollowing discussion of 27/01/22
An idea could be to have a 'setup' that disables the ui of them + and Authorization Policy that blocks it completelyu. |
We might also want to include some tests to make sure the golang code works as expected. |
Idea of splitting work:
These items will be added to #831 (comment) |
Overview and rationale for proposed strategy: First task will be an MVP which will be the setting up upstream's CWA folder structure in our kubeflow repo, and porting over the necessary setup/infrastructure code from our jupyter-api repo to just be able to access the Notebooks Server page (no additional functionality). This allows us to confirm a good base to continue development on. Next task will be dividing up the coding of upstream's CWA into Go. This will give a good base structure for us to then add in our customizations, and also provide a good foundation of the app's new structure and design, which will help us make decisions when adding in our custom components. This also allows for iterative testing at this stage. Once the base app is in Go, we can then tackle in parallel the addition of our custom components, and the testing of successful integration of each at this point. |
Regarding the "EVERYTHING into Go (need to divide up, maybe per folder/file)" bullet we do not have a solid idea for how to divide this just yet as we are all inexperienced here. The most important folder to tackle first is probably going to be the crud-web-app/common/backend/kubeflow/kubeflow/crud_backend/api as these get used by the files in jupyter/backend/apps. These These files can be assigned with a file per person initially where say an 'equivalent' or Zach's work exists jupyter-apis first. It may also be good to just have a folder named |
Closing as investigation is complete, with strategy going forward planned out as above. First step is the MVP ticket, which has been created and ready for next sprint. Outline for Go tickets has been established, and will be created during next sprint in time for the following sprint when work can begin after the MVP is in place. |
This ticket is related to #831
Still feel like even if it's
maybe instead of starting straight from scratch, it might make more sense to take the refactor with our code.
if we maintain python, let's pull in
we pull in this jupyter-apis into our statcan/kubeflow, as is. with the goal of refactoring it to have the same structure of the python but in go.
Need to figure:
Most important is
Possible things to do:
Thing we want to make sure to do
Old ticket about 1.2 that might be useful to look into
The text was updated successfully, but these errors were encountered: