Skip to content

Commit

Permalink
fix: unused field in alertConfig discord
Browse files Browse the repository at this point in the history
  • Loading branch information
azrod committed Oct 7, 2024
1 parent a372e05 commit 4b93cc2
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 22 deletions.
11 changes: 1 addition & 10 deletions api/v1alpha1/alert_config_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,15 +15,6 @@ type (
// +kubebuilder:validation:Required
WebhookURL ValueOrValueFrom `json:"webhookURL"`

// +kubebuilder:validation:Optional
// +kubebuilder:description:List of users or roles to notify.
Mentions []string `json:"mentions,omitempty"`

// +kubebuilder:validation:Optional
// +kubebuilder:description:Timeout specifies a time limit for the request to be made.
// +kubebuilder:default:10s
Timeout string `json:"timeout,omitempty"`

// +kubebuilder:validation:Optional
// +kubebuilder:validation:Schemaless
// +kubebuilder:validation:Type=string
Expand All @@ -36,7 +27,7 @@ type (

// +kubebuilder:object:root=true
// +kubebuilder:subresource:status
// +kubebuilder:resource:path=alertconfig,scope=Cluster
// +kubebuilder:resource:path=alertconfig,scope=Namespaced

type AlertConfig struct {
metav1.TypeMeta `json:",inline"`
Expand Down
5 changes: 0 additions & 5 deletions api/v1alpha1/zz_generated.deepcopy.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 1 addition & 7 deletions config/crd/bases/kimup.cloudavenue.io_alertconfig.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ spec:
listKind: AlertConfigList
plural: alertconfig
singular: alertconfig
scope: Cluster
scope: Namespaced
versions:
- name: v1alpha1
schema:
Expand Down Expand Up @@ -40,14 +40,8 @@ spec:
discord:
description: AlertDiscordSpec defines the desired state of AlertDiscord
properties:
mentions:
items:
type: string
type: array
templateBody:
type: string
timeout:
type: string
webhookURL:
properties:
value:
Expand Down

0 comments on commit 4b93cc2

Please sign in to comment.