-
Notifications
You must be signed in to change notification settings - Fork 200
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
Grayscale #50
Grayscale #50
Conversation
@@ -31,6 +31,8 @@ const ( | |||
GridSelectorName = "superedge.io/grid-selector" | |||
// GridSelectorKey corresponds to gridUniqKey of the custom-defined workload | |||
GridSelectorUniqKeyName = "superedge.io/grid-uniq-key" | |||
// TemplateHashKey is a key for storing template's hash value in labels. | |||
TemplateHashKey = "service_group_template_hash_key" |
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.
please rename “ service_group_template_hash_key”,keep the same format with GridSelectorName and GridSelectorUniqKeyName
"hash/fnv" | ||
) | ||
|
||
func GenerateHash(template interface{}) uint64 { |
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.
Uint64 has a small range and is prone to hash collision. Please pay attention to user feedback
/lgtm |
* add protocol required * deploymentgrid grayscale * statefulsetgrid grayscale * add copyright&fix bug * gofmt * file mode * grayscale doc * rename Signed-off-by: 00pf00 <feeli202012@gmail.com>
* add protocol required * deploymentgrid grayscale * statefulsetgrid grayscale * add copyright&fix bug * gofmt * file mode * grayscale doc * rename Signed-off-by: 00pf00 <feeli202012@gmail.com>
What this PR does:
Which issue(s) this PR fixes:
Fixes #
#24