Skip to content

fix: snapshot release fails#3257

Open
csviri wants to merge 3 commits intooperator-framework:mainfrom
csviri:snapshot-release-fix
Open

fix: snapshot release fails#3257
csviri wants to merge 3 commits intooperator-framework:mainfrom
csviri:snapshot-release-fix

Conversation

@csviri
Copy link
Copy Markdown
Collaborator

@csviri csviri commented Mar 27, 2026

on first look it seems because of missing relative path

Signed-off-by: Attila Mészáros a_meszaros@apple.com

on first look it seems because of missing relative path

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Copilot AI review requested due to automatic review settings March 27, 2026 15:27
@openshift-ci openshift-ci bot requested review from metacosm and xstefank March 27, 2026 15:27
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses a snapshot/release build failure by making Maven parent resolution explicit for several modules and updates the observability docs to reflect the namespace tag behavior for events.received.

Changes:

  • Add <relativePath>../pom.xml</relativePath> to the <parent> section of several module POMs.
  • Update the MicrometerMetricsV2 documentation for events.received tags/description.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
test-index-processor/pom.xml Explicitly sets Maven parent relativePath to root pom.
sample-operators/pom.xml Explicitly sets Maven parent relativePath to root pom.
migration/pom.xml Explicitly sets Maven parent relativePath to root pom.
micrometer-support/pom.xml Explicitly sets Maven parent relativePath to root pom.
bootstrapper-maven-plugin/pom.xml Explicitly sets Maven parent relativePath to root pom.
docs/content/en/docs/documentation/observability.md Documents events.received namespace tag as optional (with withNamespaceAsTag()) and updates description.

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@metacosm
Copy link
Copy Markdown
Collaborator

Any idea of why that change is needed?

@csviri
Copy link
Copy Markdown
Collaborator Author

csviri commented Mar 27, 2026

Any idea of why that change is needed?

Not entirely sure why this popped up now, but also it is an issue here:
#3228

Pls take a look, maybe you will have a better idea. (I did not spend too much time on it to analyizing)

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.

3 participants