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

Drop the 2.462.x line #4650

Merged
merged 1 commit into from
Mar 5, 2025
Merged

Conversation

MarkEWaite
Copy link
Contributor

Drop the 2.462.x line

The transition to the 2.479.x line is progressing very well. That is causing the 2.462.x line to become an independent, outdated copy of bom-weekly.

There are 1800 lines in bom-weekly and 1500 lines in bom-2.462.x. New releases of the bom-2.462.x line will have far fewer plugin updates than is typical for the oldest line in the bom.

Removing the 2.462.x line may also allow us to successfully complete a full build on ci.jenkins.io. That is a temporary condition, but it will allow us more time to identify the issues on ci.jenkins.io and resolve them.

Testing done

Confirmed that example in contributing file passes:

PLUGINS=structs,mailer TEST=InjectedTest 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

The transition to the 2.479.x line is progressing very well.  That is
causing the 2.462.x line to become an independent, outdated copy
of bom-weekly.

There are 1800 lines in bom-weekly and 1500 lines in bom-2.462.x.
New releases of the bom-2.462.x line will have far fewer plugin updates
than is typical for the oldest line in the bom.

Removing the 2.462.x line may also allow us to successfully complete
a full build on ci.jenkins.io.  That is a temporary condition, but it
will allow us more time to identify the issues on ci.jenkins.io and
resolve them.
@MarkEWaite MarkEWaite requested a review from a team as a code owner March 5, 2025 02:50
@MarkEWaite MarkEWaite added the full-test Test all LTS lines in this PR and do not halt upon first error. label Mar 5, 2025
Copy link
Member

@jglick jglick left a comment

Choose a reason for hiding this comment

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

Sensible.

@MarkEWaite MarkEWaite removed the full-test Test all LTS lines in this PR and do not halt upon first error. label Mar 5, 2025
Copy link
Member

@alecharp alecharp left a comment

Choose a reason for hiding this comment

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

This makes a lot of sense.

@alecharp alecharp merged commit 89e5675 into jenkinsci:master Mar 5, 2025
5 checks passed
@MarkEWaite MarkEWaite deleted the remove-2.462.x-line branch March 5, 2025 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants