Skip to content

[main] Fix Metrics issue causing BRO failing to startup#1008

Open
mallardduck wants to merge 13 commits into
rancher:mainfrom
mallardduck:metrics-failing-startup
Open

[main] Fix Metrics issue causing BRO failing to startup#1008
mallardduck wants to merge 13 commits into
rancher:mainfrom
mallardduck:metrics-failing-startup

Conversation

@mallardduck

Copy link
Copy Markdown
Member

This PR:

  • Updates BRO CRDs from using obsolete methods to use controller-gen
  • Adds necessary kubebuilder annotations to get equivalent CRD yaml generated
  • Fixes some issues with the previous OpenAPI results being generated (most things were nullable even if required)
  • Fixes a bug related to metrics which causes BRO to fail to start

…c, and clean up unused files.

Signed-off-by: Dan Pock <self@danpock.me>
Signed-off-by: Dan Pock <self@danpock.me>
…for file copying

Signed-off-by: Dan Pock <self@danpock.me>
…degen.

Signed-off-by: Dan Pock <self@danpock.me>
Signed-off-by: Dan Pock <self@danpock.me>
Signed-off-by: Dan Pock <self@danpock.me>
…rue"

Signed-off-by: Dan Pock <self@danpock.me>
Signed-off-by: Dan Pock <self@danpock.me>
…nd Restore resources

Signed-off-by: Dan Pock <self@danpock.me>
Signed-off-by: Dan Pock <self@danpock.me>
- Introduced a reusable `scripts/e2e` script for running e2e tests locally and in CI.
- Added restore tests to verify CRD default behavior for the Prune field and backward compatibility.
- Included a unit test for `FmtVersionInfo` in the version package.

Signed-off-by: Dan Pock <self@danpock.me>
@mallardduck mallardduck force-pushed the metrics-failing-startup branch from 0e43771 to 851e2c3 Compare July 3, 2026 14:17
Signed-off-by: Dan Pock <self@danpock.me>
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.

1 participant