Closed msherif1234 closed 3 months ago
@msherif1234: This pull request references NETOBSERV-1532 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set.
Attention: Patch coverage is 30.00000%
with 7 lines
in your changes are missing coverage. Please review.
Project coverage is 34.01%. Comparing base (
3a12ba2
) to head (aa041de
).
Files | Patch % | Lines |
---|---|---|
pkg/agent/agent.go | 0.00% | 4 Missing :warning: |
pkg/prometheus/prom_server.go | 50.00% | 1 Missing and 2 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
New image: quay.io/netobserv/netobserv-ebpf-agent:edd4cb8
It will expire after two weeks.
To deploy this build, run from the operator repo, assuming the operator is running:
USER=netobserv VERSION=edd4cb8 make set-agent-image
/ok-to-test
New image: quay.io/netobserv/netobserv-ebpf-agent:b466e4a
It will expire after two weeks.
To deploy this build, run from the operator repo, assuming the operator is running:
USER=netobserv VERSION=b466e4a make set-agent-image
@msherif1234 - I tried to enable metrics with TLS with below config in flowcollector:
metrics:
enable: true
server:
port: 9090
tls:
insecureSkipVerify: false
type: Auto
ebpf pods are landing in error state:
time="2024-03-26T15:28:50Z" level=info msg="starting NetObserv eBPF Agent"
time="2024-03-26T15:28:50Z" level=info msg="initializing Flows agent" component=agent.Flows
time="2024-03-26T15:28:50Z" level=info msg="StartServerAsync: addr = :9090" component=prometheus
time="2024-03-26T15:28:50Z" level=info msg="push CTRL+C or send SIGTERM to interrupt execution"
time="2024-03-26T15:28:50Z" level=info msg="starting Flows agent" component=agent.Flows
time="2024-03-26T15:28:50Z" level=warning msg="can't detect any network-namespaces err: open /var/run/netns: no such file or directory [Ignore if the agent privileged flag is not set]" component=ifaces.Watcher
time="2024-03-26T15:28:50Z" level=warning msg="failed to add watcher to netns directory err: no such file or directory [Ignore if the agent privileged flag is not set]" component=ifaces.Watcher
time="2024-03-26T15:28:50Z" level=fatal msg="error in http.ListenAndServe: open tls.crt: no such file or directory" component=prometheus
@msherif1234 - I tried to enable metrics with TLS with below config in flowcollector:
metrics: enable: true server: port: 9090 tls: insecureSkipVerify: false type: Auto
ebpf pods are landing in error state:
time="2024-03-26T15:28:50Z" level=info msg="starting NetObserv eBPF Agent" time="2024-03-26T15:28:50Z" level=info msg="initializing Flows agent" component=agent.Flows time="2024-03-26T15:28:50Z" level=info msg="StartServerAsync: addr = :9090" component=prometheus time="2024-03-26T15:28:50Z" level=info msg="push CTRL+C or send SIGTERM to interrupt execution" time="2024-03-26T15:28:50Z" level=info msg="starting Flows agent" component=agent.Flows time="2024-03-26T15:28:50Z" level=warning msg="can't detect any network-namespaces err: open /var/run/netns: no such file or directory [Ignore if the agent privileged flag is not set]" component=ifaces.Watcher time="2024-03-26T15:28:50Z" level=warning msg="failed to add watcher to netns directory err: no such file or directory [Ignore if the agent privileged flag is not set]" component=ifaces.Watcher time="2024-03-26T15:28:50Z" level=fatal msg="error in http.ListenAndServe: open tls.crt: no such file or directory" component=prometheus
@memodi there was missing mounts in the operator side I just updated the operator PR to do the proper mounts
/ok-to-test
New image: quay.io/netobserv/netobserv-ebpf-agent:e418bc9
It will expire after two weeks.
To deploy this build, run from the operator repo, assuming the operator is running:
USER=netobserv VERSION=e418bc9 make set-agent-image
Attention: Patch coverage is 25.00000%
with 9 lines
in your changes are missing coverage. Please review.
Project coverage is 33.84%. Comparing base (
a5bcf49
) to head (68f00d3
).
Files | Patch % | Lines |
---|---|---|
pkg/agent/agent.go | 0.00% | 6 Missing :warning: |
pkg/prometheus/prom_server.go | 50.00% | 1 Missing and 2 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
/label qe-approved
@msherif1234: This pull request references NETOBSERV-1532 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set.
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: msherif1234
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Description
Add the ability to use TLS for the metrics server,
Dependencies
n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.