Skip to content

Reduce Jenkinsfile diffs to archetype#121

Merged
MarkEWaite merged 2 commits into
jenkinsci:masterfrom
MarkEWaite:use-Jenkinsfile-from-archetype
Aug 27, 2023
Merged

Reduce Jenkinsfile diffs to archetype#121
MarkEWaite merged 2 commits into
jenkinsci:masterfrom
MarkEWaite:use-Jenkinsfile-from-archetype

Conversation

@MarkEWaite

Copy link
Copy Markdown
Contributor

Reduce Jenkinsfile diffs to archetype

Use the Jenkins plugin archetype as much as possible as we can so that the differences to the archetype are clear and intentionally used.

Tests with Java 21 on Linux and Java 17 on Windows. Accepts that tests on Java 11 are redundant since we've not detected relevant issues that are specific to Java 11.

Testing done

Confirmed that tests pass on Linux with Java 17 and Java 21.

Rely on ci.jenkins.io to test Windows.

Submitter checklist

  • 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

Use the Jenkins plugin archetype as much as possible as we can so that
the differences to the archetype are clear and intentionally used.

Tests with Java 21 on Linux and Java 17 on Windows.  Accepts that tests
on Java 11 are redundant since we've not detected relevant issues that
are specific to Java 11.

Confirmed that tests pass on Linux with Java 17 and Java 21.

Rely on ci.jenkins.io to test Windows.

Includes forkCount setting because it improves test performance on
ci.jenkins.io and reduces cost on ci.jenkins.io.
Use the Jenkins plugin archetype as much as possible as we can so that
the differences to the archetype are clear and intentionally used.

Tests with Java 21 on Linux and Java 17 on Windows.  Accepts that tests
on Java 11 are redundant since we've not detected relevant issues that
are specific to Java 11.

Confirmed that tests pass on Linux with Java 17 and Java 21.

Rely on ci.jenkins.io to test Windows.

Does not include forkCount setting because the setting was not enabled
previously.
@MarkEWaite MarkEWaite added the chore General project maintenance label Aug 27, 2023

@gounthar gounthar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me.
Thanks, Mark. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore General project maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants