Skip to content

Add Profiling option#290

Draft
LionelJouin wants to merge 1 commit intomasterfrom
profiler
Draft

Add Profiling option#290
LionelJouin wants to merge 1 commit intomasterfrom
profiler

Conversation

@LionelJouin
Copy link
Copy Markdown
Member

@LionelJouin LionelJouin commented Sep 14, 2022

Description

  • Profilier disabled by default
  • Exposed on port 9995 by default except in fe where 9996 is used (to
    avoid port conflict with load-balancer)
  • Exposed via http (no TLS)
  • Remove logrus from NSP and IPAM (no usage of NSM in these containers)

Issue link

Checklist

  • Purpose
    • Bug fix
    • New functionality
    • Documentation
    • Refactoring
    • CI
  • Test
    • Unit test
    • E2E Test
    • Tested manually
  • Introduce a breaking change
    • Yes (description required)
    • No
  • Introduce changes in the Operator
    • Yes (description required)
    • No

- Profilier disabled by default
- Exposed on port 9995 by default except in fe where 9996 is used (to
  avoid port conflict with load-balancer)
- Exposed via http (no TLS)
- Remove logrus from NSP and IPAM (no usage of NSM in these containers)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 🏗 In progress

Development

Successfully merging this pull request may close these issues.

1 participant