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

Add Job Cacher to the managed set #4033

Closed
wants to merge 1 commit into from

Conversation

darinpope
Copy link
Contributor

Add Job Cacher to the managed set

Testing done

  • LINE=weekly PLUGINS=jobcacher bash local-test.sh
  • LINE=2.479.x PLUGINS=jobcacher bash local-test.sh
  • LINE=2.462.x PLUGINS=jobcacher bash local-test.sh
  • LINE=2.452.x PLUGINS=jobcacher bash local-test.sh

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

@darinpope darinpope added the full-test Test all LTS lines in this PR and do not halt upon first error. label Nov 26, 2024
@darinpope
Copy link
Contributor Author

not going to merge this because of the consistent "can't find Docker" failures

@darinpope darinpope closed this Nov 27, 2024
@darinpope darinpope deleted the add-jobcacher branch November 27, 2024 02:59
@jonesbusy
Copy link
Contributor

FYI some tests will be ignored when docker is not available on bom tests: jenkinsci/jobcacher-plugin#375

@darinpope
Copy link
Contributor Author

Cool. Once that comes in, I'll make another pass to make sure the basics work.

However, we'll probably need to wait for 2.479.x to become the oldest line (as 2.452.x is as of 2 December 2024) since I was pulling in older versions for 2.452.x and 2.462.x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
full-test Test all LTS lines in this PR and do not halt upon first error.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants