feat: setup controller runtime logger

[britaniar]

Description of your changes

Fixes #

I have: setup controller runtime logger to remove trace stack and share logs

• Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

Using DevMode to make it more human readable.

Example logs:

[Copilot]

Pull Request Overview

This PR sets up the controller-runtime logger using the zap package to improve log readability by removing trace stack clutter. It adds the necessary import and logger initialization in both the memberagent and hubagent main packages.

• In cmd/memberagent/main.go, the logger is configured with zap in development mode.
• In cmd/hubagent/main.go, similar logger initialization is added.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
cmd/memberagent/main.go	Added zap import and initialized the controller-runtime logger using DevMode.
cmd/hubagent/main.go	Added zap import and similarly set up the controller-runtime logger.

[Copilot]

[nitpick] Consider making the logger mode configurable (e.g., via a command-line flag or environment variable) instead of hard-coding dev mode to better support production environments.

Suggested change

	ctrl.SetLogger(zap.New(zap.UseDevMode(true)))
	ctrl.SetLogger(zap.New(zap.UseDevMode(*devMode)))

[Copilot]

[nitpick] Consider making the zap logger's development mode configurable to allow switching between development and production logging settings.

Suggested change

	ctrl.SetLogger(zap.New(zap.UseDevMode(true)))
	ctrl.SetLogger(zap.New(zap.UseDevMode(devMode)))

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
cmd/hubagent/main.go	0.00%	2 Missing ⚠️
cmd/memberagent/main.go	0.00%	2 Missing ⚠️

📢 Thoughts on this report? Let us know!