Skip to content

Commit

Permalink
fix warning and add log
Browse files Browse the repository at this point in the history
Signed-off-by: Gidi233 <qpbtyfh@gmail.com>
  • Loading branch information
Gidi233 committed Sep 2, 2024
1 parent f42c88f commit f561601
Show file tree
Hide file tree
Showing 5 changed files with 33 additions and 33 deletions.
2 changes: 1 addition & 1 deletion docs/content/en/references/fleet_v1alpha1_types.html
Original file line number Diff line number Diff line change
Expand Up @@ -739,7 +739,7 @@ <h3 id="fleet.kurator.dev/v1alpha1.FlaggerConfig">FlaggerConfig
</tr>
<tr>
<td>
<code>Config</code><br>
<code>config</code><br>
<em>
<a href="#fleet.kurator.dev/v1alpha1.Config">
Config
Expand Down
60 changes: 30 additions & 30 deletions manifests/charts/fleet-manager/crds/fleet.kurator.dev_fleets.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2465,7 +2465,36 @@ spec:
description: Flagger defines the configuration for the kurator
rollout engine.
properties:
Config:
chart:
description: |-
Chart defines the helm chart config of the flagger.
default value is
```yaml
chart:
repository: oci://ghcr.io/fluxcd/charts
name: flagger
version: 1.x
```
properties:
name:
description: |-
Name defines the name of the chart.
Default value depends on the kind of the component.
type: string
repository:
description: |-
Repository defines the repository of chart.
Default value depends on the kind of the component.
type: string
version:
description: |-
Version defines the version of the chart.
Default value depends on the kind of the component.
type: string
type: object
config:
description: ProviderConfig defines the configuration for
the TrafficRoutingProvider.
properties:
Expand Down Expand Up @@ -2496,35 +2525,6 @@ spec:
You can pass in values according to your needs.
x-kubernetes-preserve-unknown-fields: true
type: object
chart:
description: |-
Chart defines the helm chart config of the flagger.
default value is
```yaml
chart:
repository: oci://ghcr.io/fluxcd/charts
name: flagger
version: 1.x
```
properties:
name:
description: |-
Name defines the name of the chart.
Default value depends on the kind of the component.
type: string
repository:
description: |-
Repository defines the repository of chart.
Default value depends on the kind of the component.
type: string
version:
description: |-
Version defines the version of the chart.
Default value depends on the kind of the component.
type: string
type: object
extraArgs:
description: |-
ExtraArgs is the set of extra arguments for flagger chart.
Expand Down
2 changes: 1 addition & 1 deletion pkg/apis/fleet/v1alpha1/types.go
Original file line number Diff line number Diff line change
Expand Up @@ -572,7 +572,7 @@ type FlaggerConfig struct {
PublicTestloader bool `json:"publicTestloader,omitempty"`
// ProviderConfig defines the configuration for the TrafficRoutingProvider.
// +optional
ProviderConfig *Config `json:"Config,omitempty"`
ProviderConfig *Config `json:"config,omitempty"`
}

type Config struct {
Expand Down
1 change: 0 additions & 1 deletion pkg/fleet-manager/application/rollout_helper.go
Original file line number Diff line number Diff line change
Expand Up @@ -494,7 +494,6 @@ func deleteIngressCreatedByKurator(ctx context.Context, kubeClient client.Client

// create/update ingress configuration
func renderIngress(ingress *ingressv1.Ingress, rollout *applicationapi.RolloutConfig) {

if labels := ingress.GetLabels(); labels == nil || labels[ingressLabelKey] == "" {
ingress.SetLabels(map[string]string{ingressLabelKey: rollout.ServiceName})
} else {
Expand Down
1 change: 1 addition & 0 deletions pkg/webhooks/application_webhook.go
Original file line number Diff line number Diff line change
Expand Up @@ -179,5 +179,6 @@ func defaultSyncPolicies(ctx context.Context, SyncPolicies []*v1alpha1.Applicati
}
}
}
log.Info("set SyncPolicies default success")
return nil
}

0 comments on commit f561601

Please sign in to comment.