Skip to content

Add pprof-bind-address flag support#371

Merged
openshift-merge-bot[bot] merged 2 commits into
openstack-k8s-operators:mainfrom
stuggi:operator_sdk_1.41.1_pprofBindAddress
Nov 25, 2025
Merged

Add pprof-bind-address flag support#371
openshift-merge-bot[bot] merged 2 commits into
openstack-k8s-operators:mainfrom
stuggi:operator_sdk_1.41.1_pprofBindAddress

Conversation

@stuggi

@stuggi stuggi commented Nov 25, 2025

Copy link
Copy Markdown
Contributor

Add optional pprof profiling endpoint configuration via --pprof-bind-address flag, matching neutron-operator implementation. Disabled by default (empty string).

Add optional pprof profiling endpoint configuration via
--pprof-bind-address flag, matching neutron-operator implementation.
Disabled by default (empty string).

Signed-off-by: Martin Schuppert <mschuppert@redhat.com>
Re-adds support for configuring the webhook server port through the
--webhook-bind-address command-line flag (default: 9443).

This flag was previously removed and needs to be restored to support
local development and testing scenarios.

Signed-off-by: Martin Schuppert <mschuppert@redhat.com>

@abays abays left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci

openshift-ci Bot commented Nov 25, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, stuggi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@stuggi

stuggi commented Nov 25, 2025

Copy link
Copy Markdown
Contributor Author

/retest

1 similar comment
@stuggi

stuggi commented Nov 25, 2025

Copy link
Copy Markdown
Contributor Author

/retest

@openshift-merge-bot openshift-merge-bot Bot merged commit 42e6ab0 into openstack-k8s-operators:main Nov 25, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants