-
Notifications
You must be signed in to change notification settings - Fork 819
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
Add missing validation for GameServerSet, refactor #791
Add missing validation for GameServerSet, refactor #791
Conversation
Build Failed 😱 Build Id: 25c05457-3086-4047-bb46-2e494173e5ed To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
a532da9
to
ed0e9fd
Compare
Build Failed 😱 Build Id: 23783433-6002-4efb-96f3-acc014a06263 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Something went wrong with
|
ed0e9fd
to
59f3634
Compare
Build Failed 😱 Build Id: 15cfadd2-614c-423f-b755-0a6bf57c6a12 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
59f3634
to
85fb14c
Compare
Build Succeeded 👏 Build Id: 6c93c76f-1cb8-4b53-bf4f-5a7fdb97c27a The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
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.
Mega minor potential name change. But if you don't like it - then just merge 👍
Nice refactor!
pkg/apis/stable/v1alpha1/common.go
Outdated
) | ||
|
||
// crdName is an interface to get Name and Kind of CRD | ||
type crdName interface { |
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.
type crdName interface { | |
type crd interface { |
Maybe? 🤷♂️
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.
Good naming for this small file, applied proposed changes
Add common file which contains all common validations for GS, GSSet and Fleet. Add const for some err messages. GSSet now checks Name on Update and GS Spec on Create.
85fb14c
to
b260d97
Compare
Build Succeeded 👏 Build Id: d48e57ff-3d32-4138-a1dd-a1f066de1231 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
Add common file which contains all common validations for GS, GSSet and
Fleet. Add const for some err messages.
GSSet now checks Name on Update and GS Spec on Create.