Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add builtin:logmonitoring.log-storage-settings to CR and reconciler #4053

Merged
merged 21 commits into from
Nov 25, 2024
Merged

Add builtin:logmonitoring.log-storage-settings to CR and reconciler #4053

Show file tree
Hide file tree
Changes from 14 commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
82eab51
Add IngestRuleMatchers to CR
waodim Nov 13, 2024
3027f1f
Add client logic for builtin:logmonitoring.log-storage-settings schema
waodim Nov 13, 2024
6cc9fad
Add logic to reconciler and add condition
waodim Nov 13, 2024
bde9ad7
Run mockery to generate mocks
waodim Nov 13, 2024
72e0aab
Fix linting
waodim Nov 13, 2024
a4767e4
Export client logic in reconciler to own function
waodim Nov 13, 2024
13e26a6
Fix linting by excluding dupl linter in the 'perform' methods
waodim Nov 13, 2024
17e84f8
Change maxDescLen because it was too large
waodim Nov 14, 2024
98b8685
Make naming private and more precise
waodim Nov 14, 2024
5a09114
Make naming more consistent
waodim Nov 14, 2024
6a09f0c
Make schemaID not public and rerun mockery
waodim Nov 14, 2024
c8d6905
Apply feedback from review
waodim Nov 18, 2024
e499de8
Merge branch 'main' into feature/add-ingestrulematchers
waodim Nov 18, 2024
2c76f1a
Merge branch 'main' into feature/add-ingestrulematchers
waodim Nov 19, 2024
15ba04c
Refactor TotalCount check, add condition removal
waodim Nov 19, 2024
13961d0
Apply feedback
waodim Nov 19, 2024
2320f89
Return in case lm is not enabled, move log message
waodim Nov 21, 2024
0070bb8
Fix linting
waodim Nov 21, 2024
9d32839
Remove unnecessary if condition
waodim Nov 21, 2024
bac6d41
Merge branch 'main' into feature/add-ingestrulematchers
waodim Nov 25, 2024
9ed8f4d
Merge branch 'main' into feature/add-ingestrulematchers
waodim Nov 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
274 changes: 89 additions & 185 deletions config/crd/bases/dynatrace.com_dynakubes.yaml
View file
Open in desktop

Large diffs are not rendered by default.

40 changes: 10 additions & 30 deletions config/crd/bases/dynatrace.com_edgeconnects.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,7 @@ spec:
Variable references $(VAR_NAME) are expanded
using the previously defined environment variables in the container and
any service environment variables. If a variable cannot be resolved,
the reference in the input string will be unchanged. Double $$ are reduced
to a single $, which allows for escaping the $(VAR_NAME) syntax: i.e.
the reference in the input string will be unchanged.
type: string
valueFrom:
description: Source for the environment variable's value. Cannot
Expand Down Expand Up @@ -339,7 +338,7 @@ spec:
Requests describes the minimum amount of compute resources required.
If Requests is omitted for a container, it defaults to Limits if that is explicitly specified,
otherwise to an implementation-defined value. Requests cannot exceed Limits.
More info: https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/
More info: https://kubernetes.
type: object
type: object
serviceAccountName:
Expand Down Expand Up @@ -375,8 +374,7 @@ spec:
description: |-
TolerationSeconds represents the period of time the toleration (which must be
of effect NoExecute, otherwise this field is ignored) tolerates the taint. By default,
it is not set, which means tolerate the taint forever (do not evict). Zero and
negative values will be treated as 0 (evict immediately) by the system.
it is not set, which means tolerate the taint forever (do not evict).
format: int64
type: integer
value:
Expand Down Expand Up @@ -445,10 +443,7 @@ spec:
matchLabelKeys:
description: |-
MatchLabelKeys is a set of pod label keys to select the pods over which
spreading will be calculated. The keys are used to lookup values from the
incoming pod labels, those key-value labels are ANDed with labelSelector
to select the group of existing pods over which spreading will be calculated
for the incoming pod.
spreading will be calculated.
items:
type: string
type: array
Expand All @@ -472,7 +467,7 @@ spec:
NodeAffinityPolicy indicates how we will treat Pod's nodeAffinity/nodeSelector
when calculating pod topology spread skew. Options are:
- Honor: only nodes matching nodeAffinity/nodeSelector are included in the calculations.
- Ignore: nodeAffinity/nodeSelector are ignored. All nodes are included in the calculations.
- Ignore: nodeAffinity/nodeSelector are ignored.
type: string
nodeTaintsPolicy:
description: |-
Expand All @@ -488,16 +483,12 @@ spec:
and identical values are considered to be in the same topology.
We consider each <key, value> as a "bucket", and try to put balanced number
of pods into each bucket.
We define a domain as a particular instance of a topology.
type: string
whenUnsatisfiable:
description: |-
WhenUnsatisfiable indicates how to deal with a pod if it doesn't satisfy
the spread constraint.
- DoNotSchedule (default) tells the scheduler not to schedule it.
- ScheduleAnyway tells the scheduler to schedule the pod in any location,
but giving higher precedence to topologies that would help reduce the
skew.
type: string
required:
- maxSkew
Expand Down Expand Up @@ -545,7 +536,6 @@ spec:
Producers of specific condition types may define expected values and meanings for this field,
and whether the values are considered a guaranteed API.
The value should be a CamelCase string.
This field may not be empty.
maxLength: 1024
minLength: 1
pattern: ^[A-Za-z]([A-Za-z0-9_,:]*[A-Za-z0-9_])?$
Expand Down Expand Up @@ -679,8 +669,7 @@ spec:
Variable references $(VAR_NAME) are expanded
using the previously defined environment variables in the container and
any service environment variables. If a variable cannot be resolved,
the reference in the input string will be unchanged. Double $$ are reduced
to a single $, which allows for escaping the $(VAR_NAME) syntax: i.e.
the reference in the input string will be unchanged.
type: string
valueFrom:
description: Source for the environment variable's value. Cannot
Expand Down Expand Up @@ -929,7 +918,7 @@ spec:
Requests describes the minimum amount of compute resources required.
If Requests is omitted for a container, it defaults to Limits if that is explicitly specified,
otherwise to an implementation-defined value. Requests cannot exceed Limits.
More info: https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/
More info: https://kubernetes.
type: object
type: object
serviceAccountName:
Expand Down Expand Up @@ -964,8 +953,7 @@ spec:
description: |-
TolerationSeconds represents the period of time the toleration (which must be
of effect NoExecute, otherwise this field is ignored) tolerates the taint. By default,
it is not set, which means tolerate the taint forever (do not evict). Zero and
negative values will be treated as 0 (evict immediately) by the system.
it is not set, which means tolerate the taint forever (do not evict).
format: int64
type: integer
value:
Expand Down Expand Up @@ -1034,10 +1022,7 @@ spec:
matchLabelKeys:
description: |-
MatchLabelKeys is a set of pod label keys to select the pods over which
spreading will be calculated. The keys are used to lookup values from the
incoming pod labels, those key-value labels are ANDed with labelSelector
to select the group of existing pods over which spreading will be calculated
for the incoming pod.
spreading will be calculated.
items:
type: string
type: array
Expand All @@ -1061,7 +1046,7 @@ spec:
NodeAffinityPolicy indicates how we will treat Pod's nodeAffinity/nodeSelector
when calculating pod topology spread skew. Options are:
- Honor: only nodes matching nodeAffinity/nodeSelector are included in the calculations.
- Ignore: nodeAffinity/nodeSelector are ignored. All nodes are included in the calculations.
- Ignore: nodeAffinity/nodeSelector are ignored.
type: string
nodeTaintsPolicy:
description: |-
Expand All @@ -1077,16 +1062,12 @@ spec:
and identical values are considered to be in the same topology.
We consider each <key, value> as a "bucket", and try to put balanced number
of pods into each bucket.
We define a domain as a particular instance of a topology.
type: string
whenUnsatisfiable:
description: |-
WhenUnsatisfiable indicates how to deal with a pod if it doesn't satisfy
the spread constraint.
- DoNotSchedule (default) tells the scheduler not to schedule it.
- ScheduleAnyway tells the scheduler to schedule the pod in any location,
but giving higher precedence to topologies that would help reduce the
skew.
type: string
required:
- maxSkew
Expand Down Expand Up @@ -1134,7 +1115,6 @@ spec:
Producers of specific condition types may define expected values and meanings for this field,
and whether the values are considered a guaranteed API.
The value should be a CamelCase string.
This field may not be empty.
maxLength: 1024
minLength: 1
pattern: ^[A-Za-z]([A-Za-z0-9_,:]*[A-Za-z0-9_])?$
Expand Down
Loading
Loading