Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test with config file provider plugin #133

Merged

Conversation

MarkEWaite
Copy link
Contributor

Test with config file provider plugin

  • Require Jenkins 2.332.4 or newer
  • Declare config file provider test dependency

Declare config file provider test dependency

Tests run successfully fine without this dependency when executed from the usual Apache Maven command line as in:

$ mvn clean verify

Tests fail when run in the BOM evaluation. They report that a class from the config file provider plugin could not be found.

jenkinsci/bom#1435 (comment) provides details.

Also increments minimum Jenkins version to 2.332.4 in order to reduce maintenance burden.

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

Jenkins 2.289.1 is no longer supported by the Jenkins update center.
Jenkins versions prior to 2.332.4 have multiple known security
vulnerabilities.

Over 80% of users of the most recent two releases of text finder plugin
are running Jenkins 2.332.1 or newer.  Chose 2.332.4 as the baseline
rather than 2.346.3 because only about 60% of those who installed the
most recent two releases are running 2.346.1 or newer.
Tests run successfully fine without this dependency when executed from
the usual Apache Maven command line as in:

$ `mvn clean verify`

Tests fail when run in the BOM evaluation.  They report that a class
from the config file provider plugin could not be found.

jenkinsci/bom#1435 (comment)
provides details.
Copy link
Member

@basil basil left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

@basil basil merged commit e0851ab into jenkinsci:master Nov 14, 2022
@jglick
Copy link
Member

jglick commented Nov 14, 2022

Looks right. Did you verify the effect on PCT in bom in advance, or just hoping it works?

@basil
Copy link
Member

basil commented Nov 14, 2022

Released in 1.21.

@MarkEWaite MarkEWaite deleted the tests-depend-on-config-file-provider branch November 14, 2022 20:18
@MarkEWaite
Copy link
Contributor Author

MarkEWaite commented Nov 14, 2022

Looks right. Did you verify the effect on PCT in bom in advance, or just hoping it works?

Just hoping it works. Had never done a build in bom with an unreleased plugin.

@jglick
Copy link
Member

jglick commented Nov 14, 2022

a build in bom with an unreleased plugin

You can file the proposed plugin PR as draft, wait for an incrementals deploy, then patch BOM to use that incremental version and test the result either locally or in a draft bom PR.

It is probably possible to do this entirely locally also. PCT supports -localCheckout though I do not think bom/local-test.sh can pass this along yet.

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