You need to sign in or sign up before continuing.
Remove config/default and config/manager from our doc guidelines
The following discussion from !702 (merged) should be addressed:
-
@Alexand started a discussion: (+2 comments) Thanks, @clemensbeck! Given this that all the changes in between versions are additive non-breaking ones, this LGTM.
Just 2 questions out of curiosity and for my personal learning here.
- Have we already run a full test pipeline since Update Operator SDK from 1.14.0 to 1.32.0 (gitlab-org/gitlab-build-images!741 - merged) got merged?
- I see that we're introducing some of the suggested changes, but the indication is that they should be changed in different files, which we don't have. Namely
config/default/manager_auth_proxy_patch.yaml
andconfig/manager/manager.yaml
. Is this because we have a different set of scaffolded files and we'd have to re-scafold our project?- I also see that in our development guide - project structure we have mention of the existence of
config/default
andconfig/manager
. So perhaps it's something that we moved at some point?
- I also see that in our development guide - project structure we have mention of the existence of