-
Notifications
You must be signed in to change notification settings - Fork 104
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
Rename the CRD files. #1210
Rename the CRD files. #1210
Conversation
It was never used (verified by eyeballing `git log -p` all the way down), we don't seem to know what it's for, and it has no counterpart in the structs.
This is in preparation for moving to `controller-gen`, which is somewhat opinionated in that it does not allow us to pick the exact file names. I guess we do not really care what these are called anyway - the only place where these names show up is this integration test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm the dot in the file name - not a big fan :/ also we lost the version there, how are we going to distinguish multiple versions?
Really no way to customize this? :(
May I ask why it matters?
Looks like this will be handled automagically If we have more than one.
Looking at that code, it doesn't seem like it... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May I ask why it matters?
- just a matter of taste
Looks like this will be handled automagically If we have more than one.
- oh that's version of CRD resource, not version of our CRDs, anyway, now I realized that multiple versions are actually represented by one yaml so all good. 👍
This is in preparation for moving to `controller-gen`, which is somewhat opinionated in that it does not allow us to pick the exact file names. I guess we do not really care *that much* what these are called anyway - the only place where these names show up is this integration test. Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
What this PR does / why we need it:
This is in preparation for moving to
controller-gen
, which is somewhatopinionated in that it does not allow us to pick the exact file names.
I guess we do not really care what these are called anyway - the only
place on all of GitHub where these names show up is this integration test - so pleasing controller-gen now will make future diffs smaller.
This is another baby step towards #862