Skip to content

Conversation

@Mixaster995
Copy link
Contributor

Signed-off-by: Mikhail Avramenko [email protected]

Signed-off-by: Mikhail Avramenko <[email protected]>
@Mixaster995 Mixaster995 marked this pull request as ready for review December 28, 2021 03:41
@Mixaster995 Mixaster995 self-assigned this Dec 28, 2021
Timeout time.Duration `default:"5s" desc:"Context timeout for kubernetes queries" split_words:"true"`
WorkerCount int `default:"8" desc:"Number of log collector workers" split_words:"true"`
AllowedNamespaces string `default:"(ns-.*)|(nsm-system)" desc:"Regex of allowed namespaces" split_words:"true"`
AllowedNamespaces string `default:"(^ns-.*)|(nsm-system)|spire|calico-vpp-dataplane" desc:"Regex of allowed namespaces" split_words:"true"`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Calico is not a default value for the env.

Suggested change
AllowedNamespaces string `default:"(^ns-.*)|(nsm-system)|spire|calico-vpp-dataplane" desc:"Regex of allowed namespaces" split_words:"true"`
AllowedNamespaces string `default:"(^ns-.*)|(nsm-system)|(spire)" desc:"Regex of allowed namespaces" split_words:"true"`

Copy link
Member

@denis-tingaikin denis-tingaikin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix the comment.

f
Signed-off-by: Mikhail Avramenko <[email protected]>
f
Signed-off-by: Mikhail Avramenko <[email protected]>
@Mixaster995 Mixaster995 marked this pull request as draft December 30, 2021 07:00
@sol-0 sol-0 removed their request for review August 2, 2022 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants