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
After provisioner mode related code was merged 1df6bcd , we have 2 components in the internal package:
provisioner mode
static mode
Additionally, we have shared "framework" code that both provisioner and static mode use.
To reflect that, for better code organization, refactor the packages as below:
- internal
- provisonermode
- manager.go
... other files
- staticmode
- manager.go
- nginx
- state
- status // status-related code, only relevant for static mode
- framework // common framework code
- events
- controller
- helpers
- status // general purpose status-related code, both relevant for provisioner and static mode
As part of it, refactor the status package into two package, so that the one in framework/status doesn't depend on the graph.
The text was updated successfully, but these errors were encountered:
Discussed in #707
Originally posted by pleshakov June 1, 2023
After provisioner mode related code was merged 1df6bcd , we have 2 components in the
internal
package:Additionally, we have shared "framework" code that both provisioner and static mode use.
To reflect that, for better code organization, refactor the packages as below:
As part of it, refactor the status package into two package, so that the one in framework/status doesn't depend on the graph.
The text was updated successfully, but these errors were encountered: