Skip to content

Avoid warnings for ignored abstract non-Describable properties#2853

Merged
timja merged 1 commit into
jenkinsci:masterfrom
somiljain2006:Warning-issue
Jun 20, 2026
Merged

Avoid warnings for ignored abstract non-Describable properties#2853
timja merged 1 commit into
jenkinsci:masterfrom
somiljain2006:Warning-issue

Conversation

@somiljain2006

@somiljain2006 somiljain2006 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Fixes #2532

JCasC currently emits a warning when attribute discovery encounters an abstract non-Describable type, despite ignoring the attribute by returning null. For properties such as SCMSource#owner, this results in confusing log messages even though the configuration proceeds normally.

Reduce the log level from WARNING to FINE to avoid unnecessary log noise while retaining diagnostic information for troubleshooting.

Your checklist for this pull request

🚨 Please review the guidelines for contributing to this repository.

  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side) and not your master branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or in Jenkins JIRA
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Did you provide a test case? That demonstrates a feature works or fixes the issue.

@somiljain2006
somiljain2006 requested a review from a team as a code owner June 20, 2026 05:56
@somiljain2006
somiljain2006 marked this pull request as draft June 20, 2026 06:51
@somiljain2006 somiljain2006 changed the title Ignore inherited transient properties during attribute discovery Avoid warnings for ignored abstract non-Describable properties Jun 20, 2026
@somiljain2006
somiljain2006 marked this pull request as ready for review June 20, 2026 08:30

@timja timja left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should be ok, users don't need to be spammed with this.

@timja timja added the feature A PR that adds a feature - used by Release Drafter label Jun 20, 2026
@timja
timja merged commit 970a0ba into jenkinsci:master Jun 20, 2026
17 checks passed
@somiljain2006
somiljain2006 deleted the Warning-issue branch June 20, 2026 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature A PR that adds a feature - used by Release Drafter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BaseConfigurator warns about #owner field that should be ignored

2 participants