-
Notifications
You must be signed in to change notification settings - Fork 38
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
init fleet api to install flagger and testloader #519
init fleet api to install flagger and testloader #519
Conversation
✅ Deploy Preview for kurator-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -507,6 +507,32 @@ type Device struct { | |||
Config map[string]string `json:"config,omitempty"` | |||
} | |||
|
|||
type FlaggerConfig struct { |
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.
I think you forget to add it to plugin
Signed-off-by: LiZhenCheng9527 <lizhencheng6@huawei.com>
f54809c
to
70e6ea9
Compare
pkg/apis/fleet/v1alpha1/types.go
Outdated
@@ -507,6 +509,32 @@ type Device struct { | |||
Config map[string]string `json:"config,omitempty"` | |||
} | |||
|
|||
type FlaggerConfig struct { | |||
// Chart defines the helm chart config of the flagger. | |||
// default values is |
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.
// default values is | |
// default value is |
// Chart defines the helm chart config of the flagger. | ||
// default values is | ||
// | ||
// chart: |
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.
wrap with
8c00330
to
7c53d1d
Compare
Signed-off-by: LiZhenCheng9527 <lizhencheng6@huawei.com>
3284553
to
696bb98
Compare
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hzxuzhonghu The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind api-change
/kind feature
What this PR does / why we need it:
Add configuretion to install flagger in fleet api.